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

feat(settings): show ui-ux skill tag status #69

Merged
janpeter merged 1 commit from codex/ui-ux-pro-max-worker-catalog into main 2026-07-17 15:49:16 +02:00
Conversation 1 Commits 1 Files changed 4 +13 -1
janpeter commented 2026-07-17 15:25:30 +02:00
Owner
Copy link

Samenvatting

  • bump scrum4me-shared naar de gemergde registrycommit 2a6b9a7
  • toon een echte upstreamtag of 'niet gekoppeld' in de reviewed skillcatalogus
  • bewijs dat ui-ux-pro-max beschikbaar maar niet automatisch geselecteerd is

Acceptatiebewijs

De tabel toont voor ui-ux-pro-max: upstream 2.11.0, tag niet gekoppeld, commit f8ac5e1, intern 2.11.0+f8ac5e1.s4m.1, MIT, Claude + Codex en APPROVED.

Verificatie

  • gerichte catalogus/config/editor-tests: 50/50 groen
  • npm run verify: lint zonder errors, typecheck groen, 718 tests groen en 10 skipped
  • Docker→gemergde shared renderer-parity: 3/3 groen

Bekende bestaande lintwarning in components/jobs/job-card.tsx; niet door deze PR veroorzaakt.

## Samenvatting - bump scrum4me-shared naar de gemergde registrycommit 2a6b9a7 - toon een echte upstreamtag of 'niet gekoppeld' in de reviewed skillcatalogus - bewijs dat ui-ux-pro-max beschikbaar maar niet automatisch geselecteerd is ## Acceptatiebewijs De tabel toont voor ui-ux-pro-max: upstream 2.11.0, tag niet gekoppeld, commit f8ac5e1, intern 2.11.0+f8ac5e1.s4m.1, MIT, Claude + Codex en APPROVED. ## Verificatie - gerichte catalogus/config/editor-tests: 50/50 groen - npm run verify: lint zonder errors, typecheck groen, 718 tests groen en 10 skipped - Docker→gemergde shared renderer-parity: 3/3 groen Bekende bestaande lintwarning in components/jobs/job-card.tsx; niet door deze PR veroorzaakt.
feat(settings): show worker skill tag status
Some checks failed
CI / Verify (pull_request) Failing after 2m27s
Details
034f14cdec
s4m-codex-reviewer approved these changes 2026-07-17 15:31:01 +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 niet-blokkerende findings.

Review-notities

  • De wijziging toont upstreamTag compact naast de bestaande upstreamversie en commitinformatie, met een duidelijke fallback voor ontbrekende tags.
  • De componenttest dekt zowel null (Tag: niet gekoppeld) als een concrete tag (Tag: v1.0.0).
  • De job-kind-config-editor-test borgt dat de nieuwe skill-metadata de bestaande skills-selectie niet per ongeluk activeert.
  • De submodule-update van vendor/scrum4me-shared is zichtbaar als pointerwijziging; de inhoudelijke shared-diff is niet in de unified diff opgenomen, maar de zichtbare consumentcode en tests zijn consistent met het nieuwe upstreamTag-contract.
# Verdict: APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings Geen blokkerende of niet-blokkerende findings. ## Review-notities - De wijziging toont `upstreamTag` compact naast de bestaande upstreamversie en commitinformatie, met een duidelijke fallback voor ontbrekende tags. - De componenttest dekt zowel `null` (`Tag: niet gekoppeld`) als een concrete tag (`Tag: v1.0.0`). - De job-kind-config-editor-test borgt dat de nieuwe skill-metadata de bestaande skills-selectie niet per ongeluk activeert. - De submodule-update van `vendor/scrum4me-shared` is zichtbaar als pointerwijziging; de inhoudelijke shared-diff is niet in de unified diff opgenomen, maar de zichtbare consumentcode en tests zijn consistent met het nieuwe `upstreamTag`-contract.
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-workers!69
No description provided.