feat(migration): additive review_logs idea_id + pr_commit_id anchors #89

Merged
janpeter merged 1 commit from claude/web-reviewlog-migration into main 2026-06-16 19:25:41 +02:00
Owner

Fase 2 (migrator) van MCP-getriggerde reviews + geünificeerde ReviewLog (spec/plan in scrum4me-workers docs/superpowers/{specs,plans}/2026-06-15-mcp-review-dispatch-unified-reviewlog*). Hoort bij scrum4me-shared#20 (gemerged → d373021) en scrum4me-mcp#57.

Hand-geauthorde additieve migratie 20260616172251_add_reviewlog_idea_pr_anchors:

  • review_logs.idea_id TEXT + FK → ideas(id) ON DELETE SET NULL + index review_logs_idea_id_created_at_idx (anker IDEA_REVIEW_PLAN).
  • review_logs.pr_commit_id TEXT (anker PR_REVIEW).

Plus submodule-bump vendor/scrum4me-shared → d373021 + geregenereerd prisma/schema.prisma (delta = exact bovenstaande velden; prisma generate clean). Geen wijziging aan bestaande kolommen, geen enum, geen CHECK-constraint — FK/index volgen de naamconventie van 20260610140946_add_spec_task_review_reviewlog.

Toepassen op prod gebeurt ná merge via update_scrum4me_web op 154 (designated migrator) + psql-verify; niet hier.

🤖 Generated with Claude Code

Fase 2 (migrator) van **MCP-getriggerde reviews + geünificeerde ReviewLog** (spec/plan in scrum4me-workers `docs/superpowers/{specs,plans}/2026-06-15-mcp-review-dispatch-unified-reviewlog*`). Hoort bij scrum4me-shared#20 (gemerged → d373021) en scrum4me-mcp#57. Hand-geauthorde **additieve** migratie `20260616172251_add_reviewlog_idea_pr_anchors`: - `review_logs.idea_id TEXT` + FK → `ideas(id)` ON DELETE SET NULL + index `review_logs_idea_id_created_at_idx` (anker IDEA_REVIEW_PLAN). - `review_logs.pr_commit_id TEXT` (anker PR_REVIEW). Plus submodule-bump `vendor/scrum4me-shared` → d373021 + geregenereerd `prisma/schema.prisma` (delta = exact bovenstaande velden; `prisma generate` clean). Geen wijziging aan bestaande kolommen, geen enum, geen CHECK-constraint — FK/index volgen de naamconventie van `20260610140946_add_spec_task_review_reviewlog`. **Toepassen op prod gebeurt ná merge** via `update_scrum4me_web` op 154 (designated migrator) + psql-verify; niet hier. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
feat(migration): additive review_logs idea_id + pr_commit_id anchors
All checks were successful
CI / Lint, Typecheck, Test & Build (pull_request) Successful in 3m25s
CI / Deploy Manual (workflow_dispatch) (pull_request) Has been skipped
CI / Detect deploy-relevant changes (pull_request) Has been skipped
CI / Deploy Preview (PR) (pull_request) Has been skipped
CI / Deploy Production (main) (pull_request) Has been skipped
01c8679d58
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!89
No description provided.