From 3bee6e50801e75d32840a520648023e8f6f946d6 Mon Sep 17 00:00:00 2001 From: Madhura68 Date: Wed, 6 May 2026 10:05:03 +0200 Subject: [PATCH] chore: sync make-plan.md prompt from scrum4me upstream MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Bron-aanpassing in scrum4me/lib/idea-prompts/make-plan.md (PR madhura68/Scrum4Me#130) wordt hierin gesynced naar de embedded kopie die de worker daadwerkelijk leest via getIdeaPromptText. Inhoudelijke wijziging: nieuwe verplichte stap-3 in de werkwijze ("Bij removal/refactor: doe een dependency-cascade-grep") + complete sectie met grep-protocol per type wijziging (Prisma-model, component, type, hernoemen, veld) + eind-taak `npm run typecheck`. Achtergrond: tijdens ST-1236 (Todo-applicatielaag verwijderen) miste het plan de cascade naar 4 consumer-bestanden + de v3-landing. Lint en tests slaagden, next build brak. De upstream prompt-update voorkomt dit voor toekomstige IDEA_MAKE_PLAN-jobs — deze sync zorgt dat workers het ook echt zien. Verified: tsc + vitest (153/153) groen. Co-Authored-By: Claude Opus 4.7 (1M context) --- src/prompts/idea/make-plan.md | 43 ++++++++++++++++++++++++++++++++--- 1 file changed, 40 insertions(+), 3 deletions(-) diff --git a/src/prompts/idea/make-plan.md b/src/prompts/idea/make-plan.md index ea7f1a8..86891a0 100644 --- a/src/prompts/idea/make-plan.md +++ b/src/prompts/idea/make-plan.md @@ -28,9 +28,46 @@ PBI + stories + taken via `materializeIdeaPlanAction`. 1. Lees `idea.grill_md` volledig. 2. Verken de repo voor patronen, bestaande modules, en `docs/`-structuur. -3. Bouw het plan op in de **strikte format** hieronder. -4. Roep `mcp__scrum4me__update_idea_plan_md({ idea_id, markdown })`. -5. Roep `mcp__scrum4me__update_job_status({ job_id, status: 'done', summary })`. +3. **Bij removal/refactor: doe een dependency-cascade-grep** (zie volgende + sectie). Voeg per geraakte file een taak toe vóór de schema/code-edit zelf. +4. Bouw het plan op in de **strikte format** hieronder. +5. Roep `mcp__scrum4me__update_idea_plan_md({ idea_id, markdown })`. +6. Roep `mcp__scrum4me__update_job_status({ job_id, status: 'done', summary })`. + +## Dependency-cascade-grep (verplicht bij removal/refactor) + +Wanneer het idee een **bestaand symbool, model, route of component +verwijdert of hernoemt**, MOET je éérst de consumers in kaart brengen voordat +je het plan vaststelt. Anders breekt `next build` op type-errors die `lint` +en `vitest run` niet pakken (zie hieronder waarom). + +**Concreet:** + +- Verwijder je een Prisma-model `Foo`? + ```bash + grep -rn "prisma\.foo\b\|prisma\.foos\b" actions/ app/ components/ lib/ \ + --include="*.ts" --include="*.tsx" + ``` + Voeg per geraakt bestand één of meer taken toe ("schoon `actions/foos.ts` + op", "verwijder `app/(app)/foos/`-route", "haal Foo-tegel uit + `app/page.tsx`-feature-grid", etc.) **vóór** de schema-edit-taak. + +- Verwijder je een component / utility / type? Idem: grep op de + bestandspaden en exports en plan per consumer een taak. + +- Hernoem je een model/route/component? Plan per geraakt bestand een edit-taak. + +- Wijzig je een `prisma.x.create`-veld (verplicht ↔ optioneel)? Grep op + `prisma.x.create` en `prisma.x.update` voor type-mismatches. + +- Voeg óók een **eind-taak** toe: `npm run typecheck` (= `tsc --noEmit`) + als sanity-check, los van `lint && test && build`. Type-errors verschijnen + daar het eerst en zijn 10× sneller dan een full `next build`. + +**Waarom zo strikt?** `eslint` doet geen diepe type-check. `vitest` met +esbuild-transpile slaat type-errors over. `next build` is de eerste step die +álles type-checkt — en die zit aan het einde van de pijp. Een gemist +consumer-bestand wordt pas zichtbaar bij verify, niet bij implementation. ## STEL GEEN VRAGEN