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

test(skills): enforce shared catalog parity #65

Merged
janpeter merged 1 commit from codex/skills-registry-render into master 2026-07-17 09:10:30 +02:00
Conversation 1 Commits 1 Files changed 3 +295
janpeter commented 2026-07-17 07:51:35 +02:00
Owner
Copy link

Samenvatting

  • genereert de rijke shared skills-registry deterministisch uit catalog.lock.json
  • controleert TypeScript escaping, byte-ordering en cross-repo parity
  • houdt niet-goedgekeurde skills zichtbaar maar niet selecteerbaar
  • ondersteunt een volledig ingetrokken allowlist zonder TypeScript-fout

Verificatie

  • SCRUM4ME_SHARED_PATH=... npm test: 38/38
  • npm run skills:verify
  • byte-parity met scrum4me-shared PR #38

Review

De onafhankelijke review vond één P2 voor een all-revoked catalogus; opgelost en afgedekt met een echte TypeScript-regressietest. Reviewverslag: skills/reviews/2026-07-17-st-026-generated-registry.md

Afhankelijk van: janpeter/scrum4me-shared#38

## Samenvatting - genereert de rijke shared skills-registry deterministisch uit catalog.lock.json - controleert TypeScript escaping, byte-ordering en cross-repo parity - houdt niet-goedgekeurde skills zichtbaar maar niet selecteerbaar - ondersteunt een volledig ingetrokken allowlist zonder TypeScript-fout ## Verificatie - SCRUM4ME_SHARED_PATH=... npm test: 38/38 - npm run skills:verify - byte-parity met scrum4me-shared PR #38 ## Review De onafhankelijke review vond één P2 voor een all-revoked catalogus; opgelost en afgedekt met een echte TypeScript-regressietest. Reviewverslag: skills/reviews/2026-07-17-st-026-generated-registry.md Afhankelijk van: https://git.jp-visser.nl/janpeter/scrum4me-shared/pulls/38
test(skills): enforce shared catalog parity
All checks were successful
CI / Compose config (pull_request) Successful in 4s
Details
CI / Docker build (pull_request) Successful in 6s
Details
8df1ff621e
s4m-codex-reviewer approved these changes 2026-07-17 07:56:14 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: APPROVED

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

Findings

Geen findings.

Toetsing

  • De renderer sluit aan op de bestaande skill-catalog.mjs validatie en houdt de runtime-selectie fail-closed via APPROVED_SKILLS/isKnownSkill.
  • De generated registry gebruikt brede SkillMeta-typering, waardoor een volledig ingetrokken catalogus niet opnieuw in TS2367 vastloopt.
  • De nieuwe tests dekken deterministische rendering, shared parity opt-in via SCRUM4ME_SHARED_PATH, escaping/sortering en all-revoked typecheck.
  • Verificatie uitgevoerd: npm run skills:verify slaagde; een renderer-smoke-run tegen skills/catalog.lock.json slaagde. Vitest kon in de tijdelijke review-checkout niet starten door lokale /tmp/native-binding execution-beperkingen, niet door een diff-fout.
# Verdict: APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings Geen findings. ## Toetsing - De renderer sluit aan op de bestaande `skill-catalog.mjs` validatie en houdt de runtime-selectie fail-closed via `APPROVED_SKILLS`/`isKnownSkill`. - De generated registry gebruikt brede `SkillMeta`-typering, waardoor een volledig ingetrokken catalogus niet opnieuw in TS2367 vastloopt. - De nieuwe tests dekken deterministische rendering, shared parity opt-in via `SCRUM4ME_SHARED_PATH`, escaping/sortering en all-revoked typecheck. - Verificatie uitgevoerd: `npm run skills:verify` slaagde; een renderer-smoke-run tegen `skills/catalog.lock.json` slaagde. Vitest kon in de tijdelijke review-checkout niet starten door lokale `/tmp`/native-binding execution-beperkingen, niet door een diff-fout.
janpeter merged commit 077ab4f409 into master 2026-07-17 09:10:30 +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!65
No description provided.