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

fix(runner): run-one-job — expliciete log-regel bij skills-promptinjectie #46

Merged
janpeter merged 1 commit from fix/run-one-job-skills-injection-log into master 2026-07-04 19:01:23 +02:00
Conversation 0 Commits 1 Files changed 1 +14 -5
janpeter commented 2026-07-04 18:57:38 +02:00
Owner
Copy link

Wat

bin/run-one-job.ts bouwt via buildSkillsSection() de sectie "## Skills beschikbaar voor deze job" die aan de claude -p-prompt wordt geappend, maar de stream-json run-log emit de -p-prompt niet. Bij de M18-e2e (2026-07-04) was de injectie daardoor niet uit transcripts te bewijzen — alleen uit code + gedrag.

Deze PR logt op het injectiepunt één expliciete stdout-regel (naast de bestaande audit-regels, zelfde log()-prefix):

<iso> [run-one-job] skills-sectie geïnjecteerd: idea-brainstorm

Alleen de skill-namen, niet de hele sectie. Lege/ontbrekende lijst → geen regel én geen sectie (onveranderd gedrag).

Hoe

  • Naam-filtering afgesplitst naar resolveSkillNames(); buildSkillsSection() neemt nu de gefilterde lijst. Log-regel en sectie gebruiken daardoor gegarandeerd dezelfde namen.
  • Geldt voor beide runtimes (CLAUDE én CODEX) — de log staat vóór de runtime-split, op het punt waar promptText wordt samengesteld.

Tests

Deel 2 van het verzoek (sectie-inhoud asserten in bestaande tests voor buildSkillsSection) vervalt: deze repo heeft geen test-infrastructuur (geen test-runner in package.json, geen *.test.ts). Geverifieerd via esbuild-parse-check + geïsoleerde run van beide functies (log-formaat, lege-lijst-gedrag, filtering van lege strings).

Aanleiding: s4m-queue-bericht ce2ed53b (server-agent) na de M18-rollout; zie ook docs/plans/M18-idea-grill-brainstorm-skill.md in de Scrum4Me-repo.

🤖 Generated with Claude Code

## Wat `bin/run-one-job.ts` bouwt via `buildSkillsSection()` de sectie "## Skills beschikbaar voor deze job" die aan de `claude -p`-prompt wordt geappend, maar de stream-json run-log emit de `-p`-prompt niet. Bij de M18-e2e (2026-07-04) was de injectie daardoor niet uit transcripts te bewijzen — alleen uit code + gedrag. Deze PR logt op het injectiepunt één expliciete stdout-regel (naast de bestaande audit-regels, zelfde `log()`-prefix): ``` <iso> [run-one-job] skills-sectie geïnjecteerd: idea-brainstorm ``` Alleen de skill-namen, niet de hele sectie. Lege/ontbrekende lijst → geen regel én geen sectie (onveranderd gedrag). ## Hoe - Naam-filtering afgesplitst naar `resolveSkillNames()`; `buildSkillsSection()` neemt nu de gefilterde lijst. Log-regel en sectie gebruiken daardoor gegarandeerd dezelfde namen. - Geldt voor beide runtimes (CLAUDE én CODEX) — de log staat vóór de runtime-split, op het punt waar `promptText` wordt samengesteld. ## Tests Deel 2 van het verzoek (sectie-inhoud asserten in bestaande tests voor `buildSkillsSection`) vervalt: deze repo heeft geen test-infrastructuur (geen test-runner in `package.json`, geen `*.test.ts`). Geverifieerd via esbuild-parse-check + geïsoleerde run van beide functies (log-formaat, lege-lijst-gedrag, filtering van lege strings). Aanleiding: s4m-queue-bericht ce2ed53b (server-agent) na de M18-rollout; zie ook `docs/plans/M18-idea-grill-brainstorm-skill.md` in de Scrum4Me-repo. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
De stream-json run-log bevat de -p prompt niet, waardoor de per-kind
skills-injectie (buildSkillsSection) niet uit transcripts te bewijzen was
(M18-e2e 2026-07-04). Log nu één stdout-regel met de skill-namen op het
injectiepunt: `[run-one-job] skills-sectie geïnjecteerd: <namen>`.
Naam-filtering afgesplitst naar resolveSkillNames zodat log en sectie
gegarandeerd dezelfde lijst gebruiken. Lege lijst → geen regel, geen sectie.

Aanleiding: s4m-queue-bericht ce2ed53b (server-agent) na de M18-rollout.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
janpeter merged commit eb287df845 into master 2026-07-04 19:01:23 +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!46
No description provided.