Documentation audit cmukt69k3003zk47roinpfpfw #170

Merged

janpeter merged 1 commit from docs-audit/cmukt69k3003zk47roinpfpfw into main

2026-09-28 10:45:56 +02:00

janpeter commented

2026-09-28 07:40:17 +02:00

Owner

Runner-validated markdown audit for job cmukt69k3003zk47roinpfpfw. Review and merge through the repository release workflow.

janpeter added 1 commit

2026-09-28 07:40:17 +02:00

docs(audit): local_llm-taakjobs — alleen expliciete repo-root, geen auto-PR/cascade (PR #169 )

CI / PR candidate (never published) (pull_request) Successful in 6m8s

Details

CI / Final merge attestation and immutable publication (pull_request) Has been skipped

Details

1085858fe3

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

s4m-codex-reviewer requested changes

2026-09-28 07:46:09 +02:00

s4m-codex-reviewer left a comment

REQUEST_CHANGES

Findings

error — CLAUDE.md:130: De nieuwe uitzondering beschrijft local_llm task jobs, maar de claimroutering reserveert required_capability = 'local_llm' voor system-IDEA_CHAT-jobs. Daardoor documenteert deze instructie niet-bestaand taakgedrag.
error — CLAUDE.md:202: De tekst stelt dat een local_llm-job uitsluitend via een expliciet geconfigureerde root kan resolven en anders direct faalt. De huidige resolveRepoRoot past geen local_llm-specifieke beperking toe: zij probeert ook de ~/Projects/<naam>-conventie en on-demand cloning. Dit kan operators tot onjuiste configuratie- en foutverwachtingen leiden.
error — README.md:527: Dezelfde onjuiste repo-resolutieclaim staat in de gebruikersdocumentatie en moet met de werkelijke implementatie worden gesynchroniseerd.

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

## REQUEST_CHANGES ### Findings - **error — CLAUDE.md:130:** De nieuwe uitzondering beschrijft `local_llm task jobs`, maar de claimroutering reserveert `required_capability = 'local_llm'` voor system-`IDEA_CHAT`-jobs. Daardoor documenteert deze instructie niet-bestaand taakgedrag. - **error — CLAUDE.md:202:** De tekst stelt dat een `local_llm`-job uitsluitend via een expliciet geconfigureerde root kan resolven en anders direct faalt. De huidige `resolveRepoRoot` past geen `local_llm`-specifieke beperking toe: zij probeert ook de `~/Projects/<naam>`-conventie en on-demand cloning. Dit kan operators tot onjuiste configuratie- en foutverwachtingen leiden. - **error — README.md:527:** Dezelfde onjuiste repo-resolutieclaim staat in de gebruikersdocumentatie en moet met de werkelijke implementatie worden gesynchroniseerd. Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

janpeter commented

2026-09-28 10:45:55 +02:00

Author

Owner

Review-evaluatie (Claude, op verzoek van JP): de drie errors kloppen niet tegen main @ 9e49f16 (bevat M3, PR #169).

CLAUDE.md:130 — local_llm-taakjobs bestaan sinds M3: update-job-status.ts:1106 (geen auto-PR), :1338 (geen statuspropagatie), :1639 (geen PBI fail-cascade), telkens job.required_capability !== 'local_llm'.
CLAUDE.md:202 — resolveRepoRoot(…, { explicitRootsOnly }) (wait-for-job.ts:77-88) zet on-demand clone uit, stopt bij een cross-repo taak zonder expliciete root (:139) en slaat de ~/Projects-conventie over (:170); attachWorktreeToJob zet explicitRootsOnly: isLocal (:441-445) en zet de job direct op FAILED zonder rollback (:449-461).
README.md:527 — idem.

De documentatie beschrijft de huidige implementatie correct. JP heeft akkoord gegeven om te mergen.

Review-evaluatie (Claude, op verzoek van JP): de drie errors kloppen niet tegen `main` @ 9e49f16 (bevat M3, PR #169). 1. **CLAUDE.md:130** — `local_llm`-taakjobs bestaan sinds M3: `update-job-status.ts:1106` (geen auto-PR), `:1338` (geen statuspropagatie), `:1639` (geen PBI fail-cascade), telkens `job.required_capability !== 'local_llm'`. 2. **CLAUDE.md:202** — `resolveRepoRoot(…, { explicitRootsOnly })` (`wait-for-job.ts:77-88`) zet on-demand clone uit, stopt bij een cross-repo taak zonder expliciete root (`:139`) en slaat de `~/Projects`-conventie over (`:170`); `attachWorktreeToJob` zet `explicitRootsOnly: isLocal` (`:441-445`) en zet de job direct op FAILED zonder rollback (`:449-461`). 3. **README.md:527** — idem. De documentatie beschrijft de huidige implementatie correct. JP heeft akkoord gegeven om te mergen.

janpeter merged commit d58927ef29 into main

2026-09-28 10:45:56 +02:00

janpeter referenced this pull request from a commit

2026-09-28 10:45:56 +02:00

Merge pull request 'Documentation audit cmukt69k3003zk47roinpfpfw' (#170) from docs-audit/cmukt69k3003zk47roinpfpfw into main