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

docs(runbooks): verdict-trail + cache-bust-eis bij de agent-runner-rebuild #69

Closed
janpeter wants to merge 2 commits from canary/reviewlog-comment into master
pull from: canary/reviewlog-comment
merge into: janpeter:master
Conversation 1 Commits 2 Files changed 1 +54
janpeter commented 2026-07-26 02:37:12 +02:00
Owner
Copy link

Documenteert twee eigenaardigheden van de PR-review-pijplijn die in déze repo thuishoren, omdat hier de agent-runner-images gebouwd worden:

  • Waar een verdict landt — Forgejo-review + review_logs-rij — en dat de event-namen en de enum-namen bewust niet gelijklopen (REQUEST_CHANGES → CHANGES_REQUESTED).
  • Dat een rebuild zonder MCP_CACHE_BUST stil een image met oude MCP-code oplevert. De Dockerfile cloont scrum4me-mcp als aparte laag; Docker ziet daar geen wijziging in bij nieuwe commits op main, dus de cache wordt hergebruikt en de container start gewoon healthy. Met de verificatie die dat onderscheidt: grep in de container op een verwachte nieuwe constante én op de afwezigheid van de oude.
  • Dat commit_id een volledige SHA-1 moet zijn, sinds scrum4me-mcp#95.

Achtergrond

Deze branch begon als wegwerp-canary voor de COMMENT-rollout van vannacht (scrum4me-shared#42 → Scrum4Me#141 → scrum4me-workers#73 → scrum4me-mcp#93). Dat wegwerpbestand is vervangen door deze runbook-notitie, zodat er iets overblijft dat bewaard kan worden — merge of sluit hem, allebei prima.

Scrum4Me: PBI-21 / ST-036 / T-122.

Documenteert twee eigenaardigheden van de PR-review-pijplijn die in déze repo thuishoren, omdat hier de agent-runner-images gebouwd worden: - **Waar een verdict landt** — Forgejo-review + `review_logs`-rij — en dat de event-namen en de enum-namen bewust niet gelijklopen (`REQUEST_CHANGES` → `CHANGES_REQUESTED`). - **Dat een rebuild zonder `MCP_CACHE_BUST` stil een image met oude MCP-code oplevert.** De Dockerfile cloont `scrum4me-mcp` als aparte laag; Docker ziet daar geen wijziging in bij nieuwe commits op main, dus de cache wordt hergebruikt en de container start gewoon healthy. Met de verificatie die dat onderscheidt: grep in de container op een verwachte nieuwe constante én op de afwezigheid van de oude. - **Dat `commit_id` een volledige SHA-1 moet zijn**, sinds scrum4me-mcp#95. ## Achtergrond Deze branch begon als wegwerp-canary voor de COMMENT-rollout van vannacht (scrum4me-shared#42 → Scrum4Me#141 → scrum4me-workers#73 → scrum4me-mcp#93). Dat wegwerpbestand is vervangen door deze runbook-notitie, zodat er iets overblijft dat bewaard kan worden — merge of sluit hem, allebei prima. Scrum4Me: PBI-21 / ST-036 / T-122.
canary: verifieer de ReviewLog-rij bij verdict COMMENT
All checks were successful
CI / Compose config (pull_request) Successful in 3s
Details
CI / Docker build (pull_request) Successful in 5s
Details
f72071f741 
Wegwerp-PR. Meet of post_pr_review sinds de rollout ook voor COMMENT een
review_logs-rij schrijft. Wordt na de meting gesloten.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
docs(runbooks): verdict-trail + cache-bust-eis bij de agent-runner-rebuild
All checks were successful
CI / Compose config (pull_request) Successful in 3s
Details
CI / Docker build (pull_request) Successful in 5s
Details
a517d6bf63 
Twee eigenaardigheden van de PR-review-pijplijn die in deze repo thuishoren
omdat hier de agent-runner-images gebouwd worden:

- waar een verdict landt (Forgejo-review + review_logs) en dat de
  event-namen en de enum-namen bewust niet gelijklopen
- dat een rebuild zonder MCP_CACHE_BUST stil een image met oude MCP-code
  oplevert: de clone-laag blijft gecached en de container start healthy

Vervangt het wegwerp-canary-bestand uit de vorige commit door iets dat
bewaard kan blijven.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
janpeter changed title from CANARY — verifieer de ReviewLog-rij bij verdict COMMENT (niet mergen) to docs(runbooks): verdict-trail + cache-bust-eis bij de agent-runner-rebuild 2026-07-26 03:18:56 +02:00
s4m-codex-reviewer reviewed 2026-07-26 03:21:04 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: COMMENT

Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Findings

  • minor — docs/INDEX.md:31: de PR voegt docs/runbooks/review-verdict-trail.md toe, maar neemt deze niet op in de top-level docs-index. Bestaande runbooks worden daar expliciet gelinkt, dus deze nieuwe operationele runbook is minder vindbaar via de standaard documentatie-ingang.

Review-notities

  • De inhoud van de nieuwe runbook is consistent met de bestaande productdocs over MCP_CACHE_BUST en fresh worker-builds.
  • Geen tests vereist gezien dit een markdown-only documentatiewijziging is.
# Verdict: COMMENT Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - **minor** — `docs/INDEX.md:31`: de PR voegt `docs/runbooks/review-verdict-trail.md` toe, maar neemt deze niet op in de top-level docs-index. Bestaande runbooks worden daar expliciet gelinkt, dus deze nieuwe operationele runbook is minder vindbaar via de standaard documentatie-ingang. ## Review-notities - De inhoud van de nieuwe runbook is consistent met de bestaande productdocs over `MCP_CACHE_BUST` en fresh worker-builds. - Geen tests vereist gezien dit een markdown-only documentatiewijziging is.
janpeter closed this pull request 2026-07-26 03:33:41 +02:00
All checks were successful
CI / Compose config (pull_request) Successful in 3s
Details
CI / Docker build (pull_request) Successful in 5s
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
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
2 participants
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!69
No description provided.