feat(jobs): PR_REVIEW/SPEC_REVIEW/TASK_REVIEW filterbaar in jobs-kolom (+ shared-bump) #44

Merged

janpeter merged 3 commits from claude/nice-jang-d7fb7a into main

2026-06-10 19:07:01 +02:00

janpeter commented

2026-06-10 19:01:27 +02:00

Owner

Wat

KIND_OPTIONS in components/jobs/jobs-column.tsx voedde de Soort-filter-pills met alleen de zes oorspronkelijke kinds — jobs van kind PR_REVIEW (Phase 2), SPEC_REVIEW en TASK_REVIEW (Phase 3) waren niet filterbaar.

KIND_OPTIONS: +3 entries (PR, SPEC-RVW, TASK-RVW) — zelfde volgorde en labels als de web-repo (Scrum4Me main).
KIND_LABELS: +SPEC_REVIEW/TASK_REVIEW in jobs-column.tsx én job-card.tsx (beide Record<ClaudeJobKind, string>, dus typecheck-verplicht na de enum-uitbreiding).

Waarom de submodule-bump

Op de oude pin (47fcd51) kende ClaudeJobKind de Phase 3-kinds nog niet → SPEC_REVIEW/TASK_REVIEW in KIND_OPTIONS zou niet typechecken. Shared-PR #12 (Phase 3 review-schema) is al gemerged op shared main, dus per de repo-flow ("PR op scrum4me-shared eerst, dan submodule-bump hier") bumpt commit 1 vendor/scrum4me-shared → e55a44c. Alleen prisma generate, geen migraties (workers is geen migrator).

Overlap met `feat/codex-spec-task-review-phase3`

Die branch bumpt dezelfde submodule naar hetzelfde commit en voegt dezelfde twee KIND_LABELS-regels toe in jobs-column.tsx — inhoudelijk identieke hunks, merge-volgorde maakt dus niet uit. De KIND_OPTIONS-entries en de job-card-labels zitten alleen in deze PR.

Verify

npm run verify groen: lint + typecheck + 498 tests (80 files) passed.

Geen tests aangepast: er bestaat geen test die de optie-lijst asserteert.

🤖 Generated with Claude Code

## Wat `KIND_OPTIONS` in `components/jobs/jobs-column.tsx` voedde de Soort-filter-pills met alleen de zes oorspronkelijke kinds — jobs van kind `PR_REVIEW` (Phase 2), `SPEC_REVIEW` en `TASK_REVIEW` (Phase 3) waren niet filterbaar. - `KIND_OPTIONS`: +3 entries (`PR`, `SPEC-RVW`, `TASK-RVW`) — zelfde volgorde en labels als de web-repo (Scrum4Me main). - `KIND_LABELS`: +`SPEC_REVIEW`/`TASK_REVIEW` in **jobs-column.tsx** én **job-card.tsx** (beide `Record<ClaudeJobKind, string>`, dus typecheck-verplicht na de enum-uitbreiding). ## Waarom de submodule-bump Op de oude pin (`47fcd51`) kende `ClaudeJobKind` de Phase 3-kinds nog niet → `SPEC_REVIEW`/`TASK_REVIEW` in `KIND_OPTIONS` zou niet typechecken. Shared-PR #12 (Phase 3 review-schema) is al gemerged op shared main, dus per de repo-flow ("PR op scrum4me-shared eerst, dan submodule-bump hier") bumpt commit 1 `vendor/scrum4me-shared` → `e55a44c`. Alleen `prisma generate`, geen migraties (workers is geen migrator). ## Overlap met `feat/codex-spec-task-review-phase3` Die branch bumpt dezelfde submodule naar **hetzelfde** commit en voegt **dezelfde** twee `KIND_LABELS`-regels toe in jobs-column.tsx — inhoudelijk identieke hunks, merge-volgorde maakt dus niet uit. De `KIND_OPTIONS`-entries en de job-card-labels zitten alleen in deze PR. ## Verify `npm run verify` groen: lint + typecheck + 498 tests (80 files) passed. Geen tests aangepast: er bestaat geen test die de optie-lijst asserteert. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

janpeter added 2 commits

2026-06-10 19:01:27 +02:00

chore(shared): bump scrum4me-shared naar e55a44c (Phase 3 review-schema) 25b3066510

ClaudeJobKind op de oude pin (47fcd51) kende SPEC_REVIEW/TASK_REVIEW niet.
Shared-PR #12 (Phase 3 review-kinds + ReviewVerdict + ReviewLog + doc_id)
is al gemerged op shared main; alleen prisma generate, geen migraties.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

feat(jobs): PR/SPEC-RVW/TASK-RVW filterbaar in jobs-kolom 471e8b443f

KIND_OPTIONS bevatte alleen de zes oorspronkelijke kinds, waardoor
PR_REVIEW- (Phase 2) en SPEC_REVIEW/TASK_REVIEW-jobs (Phase 3) niet
filterbaar waren. Entries + labels nu gelijk aan de web-repo (Scrum4Me).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

s4m-codex-reviewer added 1 commit

2026-06-10 19:04:36 +02:00