janpeter/scrum4me-docker
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

docs(runner): PR_REVIEW = read-only, geen worktree (Phase 2) #35

Merged
janpeter merged 1 commit from feat/codex-pr-review-phase2 into master 2026-06-10 09:51:22 +02:00
Conversation 0 Commits 1 Files changed 1 +4 -2
janpeter commented 2026-06-10 09:09:23 +02:00
Owner
Copy link

Phase 2 docker-slice. Bevinding: geen gedragswijziging nodig — de worktree-attach in bin/run-one-job.ts is al een TASK_IMPLEMENTATION-allowlist (regel 308), dus een PR_REVIEW-job slaat de worktree vanzelf over: cwd valt terug op /opt/agent (hetzelfde Phase 0-bewezen pad als PLAN_CHAT), de payload draagt de PR-diff/meta zelf, en de cleanup is no-op zonder worktree. De prompt-selectie (getKindPromptText(ctx.kind, runtime), regel ~365) pakt pr/review.codex.md op uit de mcp-clone in de image; MCP_GIT_REF default main pakt de gemergde mcp 65f0197 (PR 45) bij de volgende rebuild met cache-bust.

Deze PR synct alleen de kind-enumeratie-comments (2 plekken) zodat de docs PR_REVIEW benoemen.

Spec/plan: scrum4me-mcp docs/superpowers 2026-06-09 (codex 3x GO + 154 GO); mcp#45 gemerged; workers-PR volgt.

🤖 Generated with Claude Code

Phase 2 docker-slice. **Bevinding: geen gedragswijziging nodig** — de worktree-attach in bin/run-one-job.ts is al een TASK_IMPLEMENTATION-allowlist (regel 308), dus een PR_REVIEW-job slaat de worktree vanzelf over: cwd valt terug op /opt/agent (hetzelfde Phase 0-bewezen pad als PLAN_CHAT), de payload draagt de PR-diff/meta zelf, en de cleanup is no-op zonder worktree. De prompt-selectie (getKindPromptText(ctx.kind, runtime), regel ~365) pakt pr/review.codex.md op uit de mcp-clone in de image; MCP_GIT_REF default main pakt de gemergde mcp 65f0197 (PR 45) bij de volgende rebuild met cache-bust. Deze PR synct alleen de kind-enumeratie-comments (2 plekken) zodat de docs PR_REVIEW benoemen. Spec/plan: scrum4me-mcp docs/superpowers 2026-06-09 (codex 3x GO + 154 GO); mcp#45 gemerged; workers-PR volgt. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Geen gedragswijziging: de attach is al een TASK_IMPLEMENTATION-allowlist, dus PR_REVIEW slaat de worktree vanzelf over (cwd -> /opt/agent, Phase 0-bewezen pad). Alleen de kind-enumeratie-comments gesynct. MCP_GIT_REF default main pakt de gemergde mcp (65f0197) bij rebuild.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
janpeter merged commit 1ac55560ed into master 2026-06-10 09:51:22 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
janpeter/scrum4me-docker!35
No description provided.