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

fix(skills): project upstream tag status #67

Merged
janpeter merged 1 commit from codex/skills-registry-upstream-tag into master 2026-07-17 15:22:54 +02:00
Conversation 1 Commits 1 Files changed 2 +5 -1
janpeter commented 2026-07-17 15:18:10 +02:00
Owner
Copy link

Probleem

De catalogus bevat source.tag, maar de gegenereerde shared-registry liet dit veld weg. Daardoor kon de workers-tabel de in ST-028 vereiste tagstatus 'niet gekoppeld' niet tonen.

Oplossing

  • projecteer nullable upstreamTag via de bestaande generator
  • borg zowel een echte tag als null in de renderertest

TDD en verificatie

  • test eerst rood op ontbrekend upstreamTag
  • gerichte renderertest 3/3 groen
  • volledige suite 44/44 groen
  • skills:verify: 3 approved, catalogushash ongewijzigd

Dit is de minimale generieke correctie vóór de ui-ux-pro-max-canary; geen catalogus- of imagewijziging.

## Probleem De catalogus bevat source.tag, maar de gegenereerde shared-registry liet dit veld weg. Daardoor kon de workers-tabel de in ST-028 vereiste tagstatus 'niet gekoppeld' niet tonen. ## Oplossing - projecteer nullable upstreamTag via de bestaande generator - borg zowel een echte tag als null in de renderertest ## TDD en verificatie - test eerst rood op ontbrekend upstreamTag - gerichte renderertest 3/3 groen - volledige suite 44/44 groen - skills:verify: 3 approved, catalogushash ongewijzigd Dit is de minimale generieke correctie vóór de ui-ux-pro-max-canary; geen catalogus- of imagewijziging.
fix(skills): project upstream tag status
All checks were successful
CI / Compose config (pull_request) Successful in 4s
Details
CI / Docker build (pull_request) Successful in 6s
Details
794dcae8b2
s4m-codex-reviewer approved these changes 2026-07-17 15:21:06 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: APPROVED

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

Findings

Geen findings.

Review-notities

De diff voegt upstreamTag consequent toe aan de gegenereerde skill metadata: runtime-rendering in scripts/render-skills-registry.mjs, type-definitie als string | null, en testdekking voor zowel een concrete tag als null. Dit sluit aan op de bestaande rendererpatronen voor nullable registry-velden.

# Verdict: APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings Geen findings. ## Review-notities De diff voegt `upstreamTag` consequent toe aan de gegenereerde skill metadata: runtime-rendering in `scripts/render-skills-registry.mjs`, type-definitie als `string | null`, en testdekking voor zowel een concrete tag als `null`. Dit sluit aan op de bestaande rendererpatronen voor nullable registry-velden.
janpeter merged commit 9acc53c8e9 into master 2026-07-17 15:22:54 +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!67
No description provided.