feat(reviewlog): idea_id + pr_commit_id anchors + Idea.review_logs inverse #20

Merged
janpeter merged 1 commit from claude/reviewlog-idea-pr-anchors into main 2026-06-16 18:49:37 +02:00
Owner

Fase 1 van de cross-repo feature MCP-getriggerde reviews + geünificeerde ReviewLog (spec/plan in scrum4me-workers docs/superpowers/{specs,plans}/2026-06-15-mcp-review-dispatch-unified-reviewlog*).

Additieve schema-wijziging op ReviewLog zodat PR-reviews en idea-plan-reviews in één tabel kunnen landen:

  • ReviewLog.idea_id String? + FK → ideas(id) onDelete: SetNull + @@index([idea_id, created_at]) (anker voor IDEA_REVIEW_PLAN).
  • ReviewLog.pr_commit_id String? (gereviewde head-SHA, anker voor PR_REVIEW).
  • Idea.review_logs ReviewLog[] inverse-relatie (IdeaReviewLogs) — vereist voor Prisma-compilatie.

Puur additief: geen wijziging aan bestaande kolommen, geen enum-wijziging, geen CHECK-constraint. Geverifieerd via consumer-schema-gen → prisma validate groen.

Consumers (aparte PR's): Scrum4Me web-migratie + scrum4me-mcp sinks/get_review. Twee codex-reviewrondes op spec (v0.4) + plan (APPROVE).

🤖 Generated with Claude Code

Fase 1 van de cross-repo feature **MCP-getriggerde reviews + geünificeerde ReviewLog** (spec/plan in scrum4me-workers `docs/superpowers/{specs,plans}/2026-06-15-mcp-review-dispatch-unified-reviewlog*`). Additieve schema-wijziging op `ReviewLog` zodat PR-reviews en idea-plan-reviews in één tabel kunnen landen: - `ReviewLog.idea_id String?` + FK → `ideas(id)` `onDelete: SetNull` + `@@index([idea_id, created_at])` (anker voor IDEA_REVIEW_PLAN). - `ReviewLog.pr_commit_id String?` (gereviewde head-SHA, anker voor PR_REVIEW). - `Idea.review_logs ReviewLog[]` inverse-relatie (`IdeaReviewLogs`) — vereist voor Prisma-compilatie. Puur additief: geen wijziging aan bestaande kolommen, geen enum-wijziging, geen CHECK-constraint. Geverifieerd via consumer-schema-gen → `prisma validate` groen. Consumers (aparte PR's): `Scrum4Me` web-migratie + `scrum4me-mcp` sinks/get_review. Twee codex-reviewrondes op spec (v0.4) + plan (APPROVE). 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
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!20
No description provided.