scrum4me-mcp

janpeter/scrum4me-mcp

Fork 0

Commit graph

Author	SHA1	Message	Date
Madhura68	0a18f565d2	fix(update_job_status): gebruik DB-branch ipv legacy feat/job-<8> fallback Symptoom: TASK_IMPLEMENTATION job T-806 in een SPRINT-strategy sprint faalde met: push failed (unknown): error: src refspec feat/job-us3aqoup does not match any error: failed to push some refs to 'https://github.com/.../Scrum4Me.git' Maar de PR was wel succesvol aangemaakt door Claude (PR #174 op feat/sprint-fvy30lvv) — Claude commit'te in de juiste worktree-branch, maar update_job_status's prepareDoneUpdate probeerde te pushen op een niet-bestaande branch. Root cause: prepareDoneUpdate(jobId, branch) accepteert een branch-arg (meestal undefined want Claude geeft 'm niet mee) en valt terug op `feat/job-${jobId.slice(-8)}`. Dat is het legacy pre-PBI-50 pad — voor sprint-jobs is de werkelijke branch `feat/sprint-<id>` (PR_strategy=SPRINT) of `feat/story-<id>` (STORY), opgeslagen in ClaudeJob.branch door attachWorktreeToJob. Fix: - prepareDoneUpdate leest nu eerst ClaudeJob.branch uit de DB als de expliciete branch-arg ontbreekt. - Pas daarna fallback op `feat/job-<8>` (zou niet moeten voorkomen na PBI-50). Tests: vi.mock('../src/prisma.js') toegevoegd voor de findUnique-stub. Bestaande test "derives branchName from jobId when branch is undefined" hernoemd naar "reads branchName from DB" met DB-mock returnt 'feat/sprint-fvy30lvv'. Plus extra test voor de legacy fallback wanneer DB.branch ook null is. 341 tests in 38 files passed (was 340, +1). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-09 13:53:43 +02:00
janpeter visser	8ebf4ff895	feat: integrate push into update_job_status DONE transition On status=done, calls pushBranchForJob before DB write: - pushed=true → DONE + pushed_at + branch set + worktree cleanup (keepBranch=true) - no-changes → DONE without pushed_at + worktree cleanup - push failure → FAILED with error message + worktree preserved for manual inspection Also adds pushed_at to vendored prisma schema + regenerates client. 6 unit tests for prepareDoneUpdate covering all push outcomes. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-01 12:00:30 +02:00

Author

SHA1

Message

Date

Madhura68

0a18f565d2

fix(update_job_status): gebruik DB-branch ipv legacy feat/job-<8> fallback

Symptoom: TASK_IMPLEMENTATION job T-806 in een SPRINT-strategy sprint
faalde met:

  push failed (unknown): error: src refspec feat/job-us3aqoup does not
  match any
  error: failed to push some refs to 'https://github.com/.../Scrum4Me.git'

Maar de PR was wel succesvol aangemaakt door Claude (PR #174 op
feat/sprint-fvy30lvv) — Claude commit'te in de juiste worktree-branch,
maar update_job_status's prepareDoneUpdate probeerde te pushen op een
niet-bestaande branch.

Root cause: prepareDoneUpdate(jobId, branch) accepteert een branch-arg
(meestal undefined want Claude geeft 'm niet mee) en valt terug op
`feat/job-${jobId.slice(-8)}`. Dat is het legacy pre-PBI-50 pad — voor
sprint-jobs is de werkelijke branch `feat/sprint-<id>` (PR_strategy=SPRINT)
of `feat/story-<id>` (STORY), opgeslagen in ClaudeJob.branch door
attachWorktreeToJob.

Fix:
- prepareDoneUpdate leest nu eerst ClaudeJob.branch uit de DB als de
  expliciete branch-arg ontbreekt.
- Pas daarna fallback op `feat/job-<8>` (zou niet moeten voorkomen na PBI-50).

Tests: vi.mock('../src/prisma.js') toegevoegd voor de findUnique-stub.
Bestaande test "derives branchName from jobId when branch is undefined"
hernoemd naar "reads branchName from DB" met DB-mock returnt
'feat/sprint-fvy30lvv'. Plus extra test voor de legacy fallback wanneer
DB.branch ook null is.

341 tests in 38 files passed (was 340, +1).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

2026-05-09 13:53:43 +02:00

janpeter visser

8ebf4ff895

feat: integrate push into update_job_status DONE transition

On status=done, calls pushBranchForJob before DB write:
- pushed=true → DONE + pushed_at + branch set + worktree cleanup (keepBranch=true)
- no-changes → DONE without pushed_at + worktree cleanup
- push failure → FAILED with error message + worktree preserved for manual inspection

Also adds pushed_at to vendored prisma schema + regenerates client.
6 unit tests for prepareDoneUpdate covering all push outcomes.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

2026-05-01 12:00:30 +02:00

2 commits