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

feat(schema): COMMENT toevoegen aan ReviewVerdict #42

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

Wat

enum ReviewVerdict krijgt een vierde waarde: COMMENT.

Waarom

PR-reviews met verdict COMMENT worden wél op Forgejo gepost, maar landen niet in review_logs. scrum4me-mcp/src/tools/post-pr-review.ts slaat de ReviewLog-write bewust over met if (event !== 'COMMENT') — precies omdat deze enum geen COMMENT kent.

Gemeten op de prod-DB (2026-07-26): sinds 1 juli 149 afgeronde PR_REVIEW-jobs tegen 130 ReviewLog-rijen. Het gat is exact de 19 COMMENT-verdicts; APPROVED (83/83) en REQUEST_CHANGES (47/47) hebben volledige dekking. Die 19 reviews bevatten elk 1–4 findings die daarmee buiten de queryable audit-trail vallen — ~13% van het volume.

Waarom achteraan

De enum-volgorde bepaalt de pg_enum-sortorder. COMMENT achteraan houdt de bestaande drie waarden op hun plek.

Reikwijdte

Dit is stap 1 van 4 — de canonical bron. De submodule-bumps in Scrum4Me (migratie), scrum4me-mcp (de eigenlijke fix) en scrum4me-workers (lezer) wijzen naar déze commit en volgen hierna. Merge deze PR als eerste.

submit_review (SPEC/TASK/IDEA_REVIEW) blijft bewust 3-waardig: daar stuurt het verdict een toestandsmachine aan (APPROVED → PLAN_REVIEWED, CHANGES_REQUESTED → revisie-job) en COMMENT toevoegen vraagt om nieuwe semantiek die buiten dit gat valt.

Verificatie

npm run verify → 22 bestanden, 237 tests groen.

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

🤖 Generated with Claude Code

## Wat `enum ReviewVerdict` krijgt een vierde waarde: `COMMENT`. ## Waarom PR-reviews met verdict COMMENT worden wél op Forgejo gepost, maar landen niet in `review_logs`. `scrum4me-mcp/src/tools/post-pr-review.ts` slaat de ReviewLog-write bewust over met `if (event !== 'COMMENT')` — precies omdat deze enum geen COMMENT kent. Gemeten op de prod-DB (2026-07-26): sinds 1 juli 149 afgeronde `PR_REVIEW`-jobs tegen 130 ReviewLog-rijen. Het gat is **exact** de 19 COMMENT-verdicts; APPROVED (83/83) en REQUEST_CHANGES (47/47) hebben volledige dekking. Die 19 reviews bevatten elk 1–4 findings die daarmee buiten de queryable audit-trail vallen — ~13% van het volume. ## Waarom achteraan De enum-volgorde bepaalt de `pg_enum`-sortorder. COMMENT achteraan houdt de bestaande drie waarden op hun plek. ## Reikwijdte Dit is stap 1 van 4 — de canonical bron. De submodule-bumps in Scrum4Me (migratie), scrum4me-mcp (de eigenlijke fix) en scrum4me-workers (lezer) wijzen naar déze commit en volgen hierna. **Merge deze PR als eerste.** `submit_review` (SPEC/TASK/IDEA_REVIEW) blijft bewust 3-waardig: daar stuurt het verdict een toestandsmachine aan (APPROVED → PLAN_REVIEWED, CHANGES_REQUESTED → revisie-job) en COMMENT toevoegen vraagt om nieuwe semantiek die buiten dit gat valt. ## Verificatie `npm run verify` → 22 bestanden, 237 tests groen. Scrum4Me: PBI-21 / ST-036 / T-118. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
feat(schema): COMMENT toevoegen aan ReviewVerdict
All checks were successful
CI / Verify (pull_request) Successful in 30s
Details
69cb33f498 
PR-reviews met verdict COMMENT worden wel op Forgejo gepost maar vallen
buiten de unified ReviewLog: post_pr_review slaat de write bewust over
omdat er geen enum-waarde voor bestaat. Sinds 1 juli zijn dat 19 van de
149 afgeronde PR_REVIEW-jobs (~13%), elk met 1-4 findings die daarmee
niet queryable zijn.

COMMENT achteraan toegevoegd zodat de pg_enum-sortorder van de
bestaande drie waarden ongewijzigd blijft.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
s4m-codex-reviewer approved these changes 2026-07-26 01:01:11 +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.

De wijziging voegt COMMENT toe aan de canonieke Prisma enum ReviewVerdict in prisma/schema.prisma:182. Dit past bij de rol van scrum4me-shared als bron van waarheid voor schema-enums. Volgens de productdocs horen database-migraties uitsluitend in de consumer Scrum4Me, dus het ontbreken van een migratiebestand in deze shared-repo is hier conform de architectuur.

# Verdict: APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings Geen blokkerende of niet-blokkerende findings. De wijziging voegt `COMMENT` toe aan de canonieke Prisma enum `ReviewVerdict` in `prisma/schema.prisma:182`. Dit past bij de rol van `scrum4me-shared` als bron van waarheid voor schema-enums. Volgens de productdocs horen database-migraties uitsluitend in de consumer `Scrum4Me`, dus het ontbreken van een migratiebestand in deze shared-repo is hier conform de architectuur.
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-shared!42
No description provided.