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

feat(settings): show reviewed worker skill catalog #68

Merged
janpeter merged 1 commit from codex/reviewed-skill-catalog-ui into main 2026-07-17 13:45:18 +02:00
Conversation 1 Commits 1 Files changed 7 +243 -5
janpeter commented 2026-07-17 09:40:59 +02:00
Owner
Copy link

Samenvatting

  • toont de volledige gereviewde skills-catalogus read-only op /settings/job-kinds
  • gebruikt alleen APPROVED_SKILLS voor selecteerbare workerconfiguratie en servervalidatie
  • toont ingetrokken/bekende stale waarden expliciet en laat admins ze bewust verwijderen
  • pint vendor/scrum4me-shared exact op 001540ad5199ba829f603360617812ef6d28a505
  • bevat geen database-, Prisma- of migratiewijziging

Verificatie

  • npm run verify: lint en typecheck geslaagd; 718 tests geslaagd, 10 overgeslagen
  • gerichte skills/settings-suites: 50/50 geslaagd
  • task review: spec compliant en quality approved
  • finale branchreview: Ready to merge, geen Critical/Important findings

Reviewfixes

  • stale niet-approved skills kunnen expliciet en toegankelijk worden verwijderd voordat een config wordt opgeslagen
  • lange catalogushash wrapt op smalle schermen
  • projectbranch en PR uitsluitend via Forgejo
## Samenvatting - toont de volledige gereviewde skills-catalogus read-only op /settings/job-kinds - gebruikt alleen APPROVED_SKILLS voor selecteerbare workerconfiguratie en servervalidatie - toont ingetrokken/bekende stale waarden expliciet en laat admins ze bewust verwijderen - pint vendor/scrum4me-shared exact op 001540ad5199ba829f603360617812ef6d28a505 - bevat geen database-, Prisma- of migratiewijziging ## Verificatie - npm run verify: lint en typecheck geslaagd; 718 tests geslaagd, 10 overgeslagen - gerichte skills/settings-suites: 50/50 geslaagd - task review: spec compliant en quality approved - finale branchreview: Ready to merge, geen Critical/Important findings ## Reviewfixes - stale niet-approved skills kunnen expliciet en toegankelijk worden verwijderd voordat een config wordt opgeslagen - lange catalogushash wrapt op smalle schermen - projectbranch en PR uitsluitend via Forgejo
feat(settings): show reviewed worker skill catalog
Some checks failed
CI / Verify (pull_request) Failing after 2m14s
Details
1212c51fab
s4m-codex-reviewer approved these changes 2026-07-17 09:46:47 +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 gevonden.

Review-notities

  • De UI toont alleen APPROVED_SKILLS als selecteerbare opties en markeert bestaande niet-goedgekeurde opgeslagen skills expliciet als niet-selecteerbaar met een verwijderactie.
  • De server-side validatie blijft de security-grens: jobKindConfigInputSchema gebruikt isKnownSkill, dat in de bijgewerkte shared registry op goedgekeurde skills gebaseerd is.
  • De catalogustabel is read-only en toont de hash plus metadata uit de pinned shared registry.
  • Relevante tests zijn toegevoegd voor de selector, verwijderen van legacy/unapproved skills, schema-validatie en catalogusweergave.

Verificatiebeperking: in een tijdelijke clone kon ik de test-run niet uitvoeren omdat dependency-installatie strandde op execute-permissions voor gedownloade esbuild binaries onder /tmp; dit lijkt een lokale sandbox-/filesystembeperking en geen diff-fout.

# Verdict: APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings Geen blokkerende of niet-blokkerende findings gevonden. ## Review-notities - De UI toont alleen `APPROVED_SKILLS` als selecteerbare opties en markeert bestaande niet-goedgekeurde opgeslagen skills expliciet als niet-selecteerbaar met een verwijderactie. - De server-side validatie blijft de security-grens: `jobKindConfigInputSchema` gebruikt `isKnownSkill`, dat in de bijgewerkte shared registry op goedgekeurde skills gebaseerd is. - De catalogustabel is read-only en toont de hash plus metadata uit de pinned shared registry. - Relevante tests zijn toegevoegd voor de selector, verwijderen van legacy/unapproved skills, schema-validatie en catalogusweergave. Verificatiebeperking: in een tijdelijke clone kon ik de test-run niet uitvoeren omdat dependency-installatie strandde op execute-permissions voor gedownloade `esbuild` binaries onder `/tmp`; dit lijkt een lokale sandbox-/filesystembeperking en geen diff-fout.
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!68
No description provided.