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

feat(skills): add reviewed and pinned worker skill foundation #64

Merged
janpeter merged 4 commits from codex/reviewed-pinned-skills into master 2026-07-17 07:15:03 +02:00
Conversation 1 Commits 4 Files changed 19 +1091 -58
janpeter commented 2026-07-17 07:07:35 +02:00
Owner
Copy link

Samenvatting

  • voeg een canonieke, deterministisch gehashte skillcatalogus toe
  • bouw uitsluitend APPROVED skills in de Claude- en Codex-images
  • publiceer de inventory veilig naar runtime discovery-paden
  • behandel onbekende jobskills terminaal zonder requeue
  • isoleer alle niet-approved artefacten in een aparte builder-stage
  • voeg reviewaudittrail en deploy/triage-runbook toe

Tasks ST-027.1 en ST-027.2 landen bewust atomair in deze PR.

Verificatie

  • npm test: 30/30 geslaagd
  • npm run skills:verify: 2 approved; catalogus-sha 2a40cb906668b719231f279550088afa73de4b7aa385458e696c5a3fc0cfe5c5
  • Dockerfile-check voor targets claude en codex: geslaagd
  • onafhankelijke Claude-review en herreview: definitief Ready to merge; geen open bevindingen

Operationele gate

De volledige lokale Docker-build ondervond vóór de projectlagen opnieuw flakiness op ports.ubuntu.com:80. De geplande Ops-flow voert daarom vóór deploy de volledige image-build en inventoryvergelijking voor beide targets uit.

## Samenvatting - voeg een canonieke, deterministisch gehashte skillcatalogus toe - bouw uitsluitend APPROVED skills in de Claude- en Codex-images - publiceer de inventory veilig naar runtime discovery-paden - behandel onbekende jobskills terminaal zonder requeue - isoleer alle niet-approved artefacten in een aparte builder-stage - voeg reviewaudittrail en deploy/triage-runbook toe Tasks ST-027.1 en ST-027.2 landen bewust atomair in deze PR. ## Verificatie - `npm test`: 30/30 geslaagd - `npm run skills:verify`: 2 approved; catalogus-sha `2a40cb906668b719231f279550088afa73de4b7aa385458e696c5a3fc0cfe5c5` - Dockerfile-check voor targets `claude` en `codex`: geslaagd - onafhankelijke Claude-review en herreview: definitief Ready to merge; geen open bevindingen ## Operationele gate De volledige lokale Docker-build ondervond vóór de projectlagen opnieuw flakiness op `ports.ubuntu.com:80`. De geplande Ops-flow voert daarom vóór deploy de volledige image-build en inventoryvergelijking voor beide targets uit.
fix(skills): address independent review findings
All checks were successful
CI / Compose config (pull_request) Successful in 4s
Details
CI / Docker build (pull_request) Successful in 40s
Details
ad05e9de68
s4m-codex-reviewer approved these changes 2026-07-17 07:11:08 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: APPROVED

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

Findings

  • Geen blokkerende of error-severity findings gevonden.

Review-notities

De PR introduceert een fail-closed skill-catalogus met approved-only staging in een aparte Docker build-stage, inventory-gestuurde runtime-publicatie, en runner-validatie voor geselecteerde skills. Dit past bij de productstandaard om klein, expliciet en verifieerbaar te bouwen: de runtime krijgt alleen de gestagede inventory, onbekende geselecteerde skills worden terminaal afgehandeld in plaats van opnieuw gequeued, en de publicatie naar user-scope skill-mappen laat niet-beheerde inhoud ongemoeid.

Tests en docs zijn aanwezig in de diff: catalogusvalidatie, entrypoint/installatiegedrag en runner-inventorygedrag zijn afgedekt met Vitest-tests; de deploy/triage-runbook beschrijft de relevante operationele gevolgen van de fail-closed inventory.

# Verdict: APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - Geen blokkerende of error-severity findings gevonden. ## Review-notities De PR introduceert een fail-closed skill-catalogus met approved-only staging in een aparte Docker build-stage, inventory-gestuurde runtime-publicatie, en runner-validatie voor geselecteerde skills. Dit past bij de productstandaard om klein, expliciet en verifieerbaar te bouwen: de runtime krijgt alleen de gestagede inventory, onbekende geselecteerde skills worden terminaal afgehandeld in plaats van opnieuw gequeued, en de publicatie naar user-scope skill-mappen laat niet-beheerde inhoud ongemoeid. Tests en docs zijn aanwezig in de diff: catalogusvalidatie, entrypoint/installatiegedrag en runner-inventorygedrag zijn afgedekt met Vitest-tests; de deploy/triage-runbook beschrijft de relevante operationele gevolgen van de fail-closed inventory.
janpeter merged commit d1b25fe833 into master 2026-07-17 07:15:03 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
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!64
No description provided.