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

chore(shared): bump naar 69cb33f voor de COMMENT-ReviewVerdict #73

Merged
janpeter merged 1 commit from fix/review-verdict-comment into main 2026-07-26 01:36:16 +02:00
Conversation 1 Commits 1 Files changed 1 +1 -1
janpeter commented 2026-07-26 01:15:36 +02:00
Owner
Copy link

Wat

Alleen de submodule-pointer: vendor/scrum4me-shared → 69cb33f (scrum4me-shared#42).

Waarom een eigen PR nodig is

Workers leest review_logs.verdict (lib/jobs-mapper.ts, components/jobs/job-detail-pane.tsx) en genereert zijn Prisma-client bij build uit de vendored shared (postinstall/prebuild → gen-schema.sh). Zonder deze bump kent die client COMMENT niet, en geeft de jobs-pagina P2023 zodra de eerste COMMENT-rij bestaat — exact het incident van 2026-06-10 met de PR_REVIEW-enum, waar een consumer met een oude client crashte op een enum-waarde die de DB al kende.

Er is géén code-wijziging nodig: het verdict wordt als opaque string gerenderd (<TextValue value={job.reviewLog?.verdict} mono />), dus er is hier geen exhaustieve map die uitgebreid moet worden. Dat is bevestigd door de typecheck — in Scrum4Me lag dat anders en dáár moest wél een pill bij.

Verificatie

npm run verify (lint + typecheck + test): 778 tests groen, 10 skipped, 113 bestanden. De gegenereerde prisma/schema.prisma (gitignored) bevat na de bump de vierde enum-waarde.

Volgorde

Deze PR moet live vóór scrum4me-mcp#93 — dat is de schrijver. Ketting: shared#42 → Scrum4Me#141 → deze → scrum4me-mcp#93.

Scrum4Me: PBI-21 / ST-036 / T-121.

🤖 Generated with Claude Code

## Wat Alleen de submodule-pointer: `vendor/scrum4me-shared` → `69cb33f` (scrum4me-shared#42). ## Waarom een eigen PR nodig is Workers leest `review_logs.verdict` (`lib/jobs-mapper.ts`, `components/jobs/job-detail-pane.tsx`) en genereert zijn Prisma-client **bij build** uit de vendored shared (`postinstall`/`prebuild` → `gen-schema.sh`). Zonder deze bump kent die client `COMMENT` niet, en geeft de jobs-pagina P2023 zodra de eerste COMMENT-rij bestaat — exact het incident van 2026-06-10 met de PR_REVIEW-enum, waar een consumer met een oude client crashte op een enum-waarde die de DB al kende. Er is géén code-wijziging nodig: het verdict wordt als opaque string gerenderd (`<TextValue value={job.reviewLog?.verdict} mono />`), dus er is hier geen exhaustieve map die uitgebreid moet worden. Dat is bevestigd door de typecheck — in Scrum4Me lag dat anders en dáár moest wél een pill bij. ## Verificatie `npm run verify` (lint + typecheck + test): **778 tests groen**, 10 skipped, 113 bestanden. De gegenereerde `prisma/schema.prisma` (gitignored) bevat na de bump de vierde enum-waarde. ## Volgorde Deze PR moet live **vóór** scrum4me-mcp#93 — dat is de schrijver. Ketting: shared#42 → Scrum4Me#141 → **deze** → scrum4me-mcp#93. Scrum4Me: PBI-21 / ST-036 / T-121. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
chore(shared): bump naar 69cb33f voor de COMMENT-ReviewVerdict
All checks were successful
CI / Verify (pull_request) Successful in 1m54s
Details
2897641a61 
Workers leest review_logs.verdict (lib/jobs-mapper.ts,
components/jobs/job-detail-pane.tsx) en genereert zijn Prisma-client bij
build uit de vendored shared. Zonder deze bump kent die client COMMENT
niet en geeft de jobs-pagina P2023 zodra de eerste COMMENT-rij bestaat --
hetzelfde incident als bij de PR_REVIEW-enum op 2026-06-10.

Alleen de pointer: het verdict wordt als opaque string gerenderd, dus er
is geen exhaustieve map die uitgebreid moet worden.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
s4m-codex-reviewer reviewed 2026-07-26 01:21:11 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: COMMENT

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

Findings

  • minor — vendor/scrum4me-shared:1 — De PR is uitsluitend een submodule-bump van be67dbe naar 69cb33f. De onderliggende shared-diff voegt COMMENT toe aan ReviewVerdict en wijzigt documenterend commentaar in lib/queue-identity.ts; dat past inhoudelijk bij de PR-titel en ik zie geen directe blocker in deze repository-diff. De reviewzekerheid blijft wel beperkt omdat er in scrum4me-workers zelf geen consumer-code of test-aanpassing zichtbaar is die bewijst dat de nieuwe enumwaarde overal correct wordt verwerkt.

Conclusie

Geen blokkerende bevindingen gevonden in de zichtbare diff, maar door de beperkte submodule-only scope en ontbrekende gekoppelde plan-/acceptatiecriteria kies ik safe-default COMMENT in plaats van APPROVED.

# Verdict: COMMENT Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - **minor** — `vendor/scrum4me-shared:1` — De PR is uitsluitend een submodule-bump van `be67dbe` naar `69cb33f`. De onderliggende shared-diff voegt `COMMENT` toe aan `ReviewVerdict` en wijzigt documenterend commentaar in `lib/queue-identity.ts`; dat past inhoudelijk bij de PR-titel en ik zie geen directe blocker in deze repository-diff. De reviewzekerheid blijft wel beperkt omdat er in `scrum4me-workers` zelf geen consumer-code of test-aanpassing zichtbaar is die bewijst dat de nieuwe enumwaarde overal correct wordt verwerkt. ## Conclusie Geen blokkerende bevindingen gevonden in de zichtbare diff, maar door de beperkte submodule-only scope en ontbrekende gekoppelde plan-/acceptatiecriteria kies ik safe-default `COMMENT` in plaats van `APPROVED`.
Sign in to join this conversation.
Reviewers
No reviewers
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!73
No description provided.