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

feat(skills): expose generated reviewed catalog #38

Merged
janpeter merged 1 commit from codex/generated-skills-registry into main 2026-07-17 07:49:26 +02:00
Conversation 1 Commits 1 Files changed 2 +73 -25
janpeter commented 2026-07-17 07:44:29 +02:00
Owner
Copy link

Samenvatting

  • vervangt de handmatige skills-allowlist door de gegenereerde Docker-catalogusprojectie
  • exporteert catalogus-SHA, volledige reviewmetadata en approved-only selecteerbaarheid
  • ondersteunt een volledig ingetrokken allowlist zonder TypeScript-fout

Verificatie

  • npm run verify (21 testbestanden, 231 tests)
  • byte-parity met scrum4me-docker renderer
  • onafhankelijke review: P2 emergency-revoke opgelost
## Samenvatting - vervangt de handmatige skills-allowlist door de gegenereerde Docker-catalogusprojectie - exporteert catalogus-SHA, volledige reviewmetadata en approved-only selecteerbaarheid - ondersteunt een volledig ingetrokken allowlist zonder TypeScript-fout ## Verificatie - npm run verify (21 testbestanden, 231 tests) - byte-parity met scrum4me-docker renderer - onafhankelijke review: P2 emergency-revoke opgelost
feat(skills): expose generated reviewed catalog
All checks were successful
CI / Verify (pull_request) Successful in 21s
Details
68c42279a4
s4m-codex-reviewer approved these changes 2026-07-17 07:46:06 +02:00
s4m-codex-reviewer left a comment
Copy link

APPROVED

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

Findings

  • Geen blocking findings.

Review-notities

  • lib/skills-registry.ts: de wijziging blijft framework-loos en dependency-vrij, passend bij de scrum4me-shared architectuur als pure data + pure functies.
  • __tests__/skills-registry.test.ts: de tests dekken de nieuwe catalogushash, verrijkte metadata en de afleiding van APPROVED_SKILLS uit goedgekeurde registry entries.
# APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - Geen blocking findings. ## Review-notities - `lib/skills-registry.ts`: de wijziging blijft framework-loos en dependency-vrij, passend bij de `scrum4me-shared` architectuur als pure data + pure functies. - `__tests__/skills-registry.test.ts`: de tests dekken de nieuwe catalogushash, verrijkte metadata en de afleiding van `APPROVED_SKILLS` uit goedgekeurde registry entries.
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-shared!38
No description provided.