[BLOCKING] Execute skill steps in declared order. NEVER skip, reorder, or merge steps without explicit user approval.
[BLOCKING] Before each step or sub-skill call, update task tracking: set in_progress when step starts, set completed when step ends.
[BLOCKING] Every completed/skipped step MUST include brief evidence or explicit skip reason.
[BLOCKING] If Task tools are unavailable, create and maintain an equivalent step-by-step plan tracker with the same status transitions.
Quick Summary
Goal: Force every assumption-laden plan decision and every preservation-critical behavior through explicit user confirmation BEFORE implementation — by interviewing the user with critical questions that validate assumptions and surface issues — so no unstated assumption silently reaches code.
Summary:
- Purpose: validate a finished plan via a critical-questions interview so every assumption-laden decision and every preservation-critical behavior is user-confirmed BEFORE implementation — no unstated assumption silently reaches code.
- Main steps (run in order): Phase 0 Detect Plan Type → resolve plan path (
$ARGUMENTS / ## Plan Context / ask) → load mode + questions range as hard constraints → Step 1 Read plan.md + all phase-*.md, flag decisions/assumptions/risks/tradeoffs → Step 2 Extract topics across 8 categories (Architecture, Assumptions, Tradeoffs, Risks, Scope, New Tech/Lib, Test Specs, Preservation) → Step 3 Generate questions (2-4 concrete options each, surface implicit decisions) → Step 4 Interview via AskUserQuestion (≤4 per call) → Step 5 Document answers → offer implement/refine/skip.
- Phase 0 weights everything: plan type (bugfix/feature/migration/refactor/other) decides which question categories fire; any fix/bug/regression/broken/defect keyword makes the Preservation question BLOCKING — never skip it.
- The output is a REAL interview, not a self-answer: honor the
questions MIN-MAX range from ## Plan Context, give 2-4 concrete options per question, treat the Preservation "Unsure" answer as BLOCKED → route to /plan; if the plan adds new tech/packages, probe whether alternatives were evaluated before accepting the choice.
- Persist results narrowly: add ONLY a
## Validation Summary (confirmed decisions + action items) to plan.md — NEVER edit phase files; close by offering implement/refine/skip via AskUserQuestion.
Workflow:
- Detect Plan Type — Classify plan (bugfix/feature/migration/refactor) to weight question categories
- Read Plan — Parse plan.md + phase files for decisions, assumptions, risks
- Extract Topics — Scan architecture, assumptions, tradeoffs, risks, scope keywords
- Generate Questions — Formulate concrete questions with 2-4 options each
- Interview User — Present questions using configured count range
- Document Answers — Add Validation Summary section to plan.md
Key Rules:
- MUST ATTENTION use
AskUserQuestion — NEVER auto-decide on behalf of user — why: the user owns every assumption-laden choice, not the agent
- Ask ONLY about genuine choices affecting implementation — NEVER about non-decision points — why: noise questions burn the interview budget and erode trust
- Bugfix plans ALWAYS trigger the Preservation question (keywords: fix, bug, regression, broken, defect) — why: an unverified preserved-correctness invariant is a silent regression
- Persist via a
## Validation Summary on plan.md — NEVER modify phase files — why: phase files are the plan's source of truth; validation is a read-then-annotate pass
First Principle — Easy to Change
The success metric of every coding decision is future change cost.
DRY, SRP, abstraction, design patterns, naming, layering, tests — every
technique exists to serve one goal: making the next change cheaper.
Evaluating code, refactor, test, abstraction, ask:
does this make next change cheaper or more expensive?
- Reject "best practices" raising change cost (premature abstraction,
speculative generality, leaky indirection, ceremony without payoff).
- Name real enemies in findings: coupling, hidden state, duplicated
knowledge, unclear intent, irreversible decisions exposed too early.
- Simpler design easy to change beats sophisticated design that isn't.
Apply this lens before invoking any specific rule, pattern, or checklist
below — if a downstream rule raises change cost, this principle wins.
Phase 0: Detect Plan Type
Classify plan type BEFORE generating questions — drives question category weighting:
| Plan Type |
Detection |
Mandatory Extra Categories |
| Bugfix |
Title/frontmatter: fix, bug, regression, broken, defect |
Preservation (BLOCKING) |
| Feature |
New capability, no fix keywords |
Architecture, Assumptions, Test Specs |
| Migration |
Schema change, EF migration, data move |
Risks, Preservation, Scope |
| Refactor |
Restructure/clean up, no behavior change |
Preservation, Tradeoffs |
| Other |
None of above |
Architecture, Scope |
Bugfix detection is BLOCKING — NEVER skip Preservation question when fix/bug/regression/broken/defect keywords present.
Plan Resolution
$ARGUMENTS provided → use that path
- Check
## Plan Context section → use active plan path
- No plan found → ask user to specify path or run
/plan first
Configuration (from injected context)
Check ## Plan Context section:
mode — auto/prompt/off behavior
questions — range like 3-8 (min-max)
Use as hard constraints.
Workflow
Step 1: Read Plan Files
Read plan directory:
plan.md — overview + phases list
phase-*.md — all phase files
- Flag: decision points, assumptions, risks, tradeoffs
Step 2: Extract Question Topics
| Category |
Keywords |
| Architecture |
approach, pattern, design, structure, database, API |
| Assumptions |
assume, expect, should, will, must, default |
| Tradeoffs |
tradeoff, vs, alternative, option, choice, either/or |
| Risks |
risk, might, could fail, dependency, blocker, concern |
| Scope |
phase, MVP, future, out of scope, nice to have |
| New Tech/Lib |
install, add package, new dependency, npm install, dotnet add, unfamiliar framework names |
| Test Specs |
TC-, test case, coverage, TDD, test specification |
| Preservation |
auto-trigger on bugfix keywords in title/frontmatter — scan Preservation Inventory section |
Step 3: Generate Questions
Format rules:
- 2-4 concrete options per question
- Mark recommended with "(Recommended)" suffix
- "Other" option automatic — do NOT add
- Surface implicit decisions
Examples:
Category: Architecture
Question: "How should validation results be persisted?"
Options:
1. Save to plan.md frontmatter (Recommended) — updates existing plan
2. Create validation-answers.md — separate answers file
3. Don't persist — ephemeral validation only
Category: Assumptions
Question: "Plan assumes API rate limiting not needed. Correct?"
Options:
1. Yes, not needed for MVP
2. No, add basic rate limiting now (Recommended)
3. Defer to Phase 2
Category: Preservation (MANDATORY when title/frontmatter: fix, bug, regression, broken, defect)
Question: "List 2-3 inputs where CURRENT code is correct. Will fix change behavior on any?"
Options (multi-select):
1. "Current code correct on: {input A}. Fix preserves behavior." (Recommended)
2. "Current code correct on: {input B}. Fix CHANGES behavior because: {justification}"
3. "Current code has NO preserved-correctness inputs — every input was broken" (rare; requires confirmation)
4. "Unsure — need to investigate" (STOP: run /plan preservation analysis)
Follow-up rules:
- Option 2 selected →
plan.md Preservation Inventory MUST cite Preservation TC asserting new behavior is intended
- Option 4 selected → return BLOCKED status, recommend
/plan before proceeding
- Option 3 selected →
AskUserQuestion follow-up: "Confirm: current code has NO preserved invariant? [Yes, every input broken / No, missed some — re-investigate]"
Step 4: Interview User
Use AskUserQuestion — NEVER skip or auto-answer.
Rules:
- Use question count from
## Plan Context → Validation: mode=X, questions=MIN-MAX
- Group related questions (max 4 per tool call)
- Focus: assumptions, risks, tradeoffs, architecture
- MANDATORY IMPORTANT MUST ATTENTION: if plan introduces new tech/packages, ask: "Plan uses {lib}. Were alternatives evaluated? Confirm choice or research more?"
Step 5: Document Answers
Add ## Validation Summary to plan.md:
## Validation Summary
**Validated:** {date}
**Questions asked:** {count}
### Confirmed Decisions
- {decision 1}: {user choice}
- {decision 2}: {user choice}
### Action Items
- [ ] {changes needed based on answers}
NEVER modify phase files — only document what needs updating.
Output
After validation:
- Questions asked count
- Key decisions confirmed
- Items flagged for plan revision
- Recommendation: proceed to implementation OR revise plan first
Next Steps
MANDATORY IMPORTANT MUST ATTENTION — NO EXCEPTIONS after completing, use AskUserQuestion to present:
- "/feature-implement (Recommended)" — Begin implementation with validated plan
- "/refine" — If plan needs PBI refinement first
- "Skip, continue manually" — User decides
[BLOCKING] MUST ATTENTION use AskUserQuestion to interview user. Completing without asking ≥1 question = violation.
[IMPORTANT] Use TaskCreate to break ALL work into small tasks BEFORE starting — including tasks for each file read. For simple tasks, AI MUST ATTENTION ask user whether to skip.
External Memory: Complex/lengthy work → write findings + results to plans/reports/ — prevents context loss.
Evidence Gate: MANDATORY IMPORTANT MUST ATTENTION — every claim requires file:line proof or traced evidence with confidence % (>80% act, <80% verify first).
Nested Task Expansion Contract — For workflow-step invocation, the [Workflow] ... row is only a parent container; the child skill still creates visible phase tasks.
- Call
TaskList first. If a matching active parent workflow row exists, set nested=true and record parentTaskId; otherwise run standalone.
- Create one task per declared phase before phase work. When nested, prefix subjects
[N.M] $skill-name — phase.
- When nested, link the parent with
TaskUpdate(parentTaskId, addBlockedBy: [childIds]).
- Orchestrators must pre-expand a child skill's phase list and link the workflow row before invoking that child skill or sub-agent.
- Mark exactly one child
in_progress before work and completed immediately after evidence is written.
- Complete the parent only after all child tasks are completed or explicitly cancelled with reason.
Blocked until: TaskList done, child phases created, parent linked when nested, first child marked in_progress.
Task Tracking & External Report Persistence — Bootstrap this before execution; then run project-reference doc prefetch before target/source work.
- Create a small task breakdown before target file reads, grep, edits, or analysis. On context loss, inspect the current task list first.
- Mark one task
in_progress before work and completed immediately after evidence; never batch transitions.
- For plan/review work, create
plans/reports/{skill}-{YYMMDD}-{HHmm}-{slug}.md before first finding.
- Append findings after each file/section/decision and synthesize from the report file at the end.
- Final output cites
Full report: plans/reports/{filename}.
Blocked until: task breakdown exists, report path declared for plan/review work, first finding persisted before the next finding.
Critical Thinking Mindset — Apply critical thinking, sequential thinking. Every claim needs traced proof, confidence >80% to act.
Anti-hallucination: Never present guess as fact — cite sources for every claim, admit uncertainty freely, self-check output for errors, cross-reference independently, stay skeptical of own confidence — certainty without evidence root of all hallucination.
Sequential Thinking Protocol — Structured multi-step reasoning for complex/ambiguous work. Use when planning, reviewing, debugging, or refining ideas where one-shot reasoning is unsafe.
Trigger when: complex problem decomposition · adaptive plans needing revision · analysis with course correction · unclear/emerging scope · multi-step solutions · hypothesis-driven debugging · cross-cutting trade-off evaluation.
Format (explicit mode — visible thought trail):
Thought N/M: [aspect] — one aspect per thought, state assumptions/uncertainty
Thought N/M [REVISION of Thought K]: ... — when prior reasoning invalidated; state Original / Why revised / Impact
Thought N/M [BRANCH A from Thought K]: ... — explore alternative; converge with decision rationale
Thought N/M [HYPOTHESIS]: ... then [VERIFICATION]: ... — test before acting
Thought N/N [FINAL] — only when verified, all critical aspects addressed, confidence >80%
Mandatory closers: Confidence % stated · Assumptions listed · Open questions surfaced · Next action concrete.
Stop conditions: confidence <80% on any critical decision → escalate via AskUserQuestion · ≥3 revisions on same thought → re-frame the problem · branch count >3 → split into sub-task.
Implicit mode: apply methodology internally without visible markers when adding markers would clutter the response (routine work where reasoning aids accuracy).
Deep-dive: see /sequential-thinking skill (.claude/skills/sequential-thinking/SKILL.md) for worked examples (API design, debugging, architecture), advanced techniques (spiral refinement, hypothesis testing, convergence), and meta-strategies (uncertainty handling, revision cascades).
Project Reference Docs Gate — Run after task-tracking bootstrap and before target/source file reads, grep, edits, or analysis. Project docs override generic framework assumptions.
- Identify scope: file types, domain area, and operation.
- Read
docs/project-config.json first — the project's machine-readable map. It is the single source of truth for THIS repo (modules/paths, framework + search keywords, test/E2E/integration run-commands, design system, architecture rules, workflow patterns); ground exact paths, run-commands, and conventions on it before investigating, planning, or coding — never assume framework defaults (CLAUDE.md + reference docs are derived from it). If it — or the docs index, lessons.md, CLAUDE.md, AGENTS.md, or any required reference doc — is missing or stale, auto-run /project-init or the narrow route (/project-config, /docs-init, /scan-all, /scan --target=<key>, /claude-md-init) first; if Codex mirrors or AGENTS.md are stale, ask the user to run /sync-codex (never auto-run it).
- Required docs by trigger: always
docs/project-reference/lessons.md; doc lookup docs-index-reference.md; review code-review-rules.md; backend/CQRS/API backend-patterns-reference.md; domain/entity domain-entities-reference.md; frontend/UI frontend-patterns-reference.md; styles/design scss-styling-guide.md + design-system/design-system-canonical.md; integration tests integration-test-reference.md; E2E e2e-test-reference.md; feature docs/specs feature-spec-reference.md + spec-system-reference.md + spec-principles.md; behavior/public-contract/spec-test-code sync workflow-spec-test-code-cycle-reference.md; derived spec index/ERD/reimplementation guides spec-system-reference.md + source Feature Specs under docs/specs/; architecture/new area project-structure-reference.md.
- Read every required doc, then before target work state:
Reference docs read: ... | Not applicable: ....
Ready when: scope evaluated, docs/project-config.json consulted, required docs checked/read or setup route completed, lessons.md confirmed, citation emitted.
Understand Code First — HARD-GATE: Do NOT write, plan, or fix until you READ existing code.
- Search 3+ similar patterns (
grep/glob) — cite file:line evidence
- Read existing files in target area — understand structure, base classes, conventions
- Run
python .claude/scripts/code_graph trace <file> --direction both --json when .code-graph/graph.db exists
- Map dependencies via
connections or callers_of — know what depends on your target
- Write investigation to
.ai/workspace/analysis/ for non-trivial tasks (3+ files)
- Re-read analysis file before implementing — never work from memory alone. — why: long context drifts from the file; the file is ground truth
- NEVER invent new patterns when existing ones work — match exactly or document deviation. — why: divergent patterns fragment the codebase and slow every future reader
BLOCKED until: - [ ] Read target files - [ ] Grep 3+ patterns - [ ] Graph trace (if graph.db exists) - [ ] Assumptions verified with evidence
Plan Quality — Every plan phase MUST ATTENTION include test specifications.
- Add
## Test Specifications section with TC-{FEATURE}-{NNN} IDs to every phase file
- Map every functional requirement to ≥1 TC (or explicit
TBD with rationale)
- TC IDs follow
TC-{FEATURE}-{NNN} format — reference by ID, never embed full content
- Before any new workflow step: call
TaskList and re-read the phase file
- On context compaction: call
TaskList FIRST — never create duplicate tasks
- Verify TC satisfaction per phase before marking complete (evidence must be
file:line, not TBD)
Mode: TDD-first → reference existing TCs with Evidence: TBD. Implement-first → use TBD → /spec [mode=tests] fills after.
Cross-Service Check — Microservices/event-driven: MANDATORY before concluding investigation, plan, spec, or feature doc. Missing downstream consumer = silent regression.
| Boundary |
Grep terms |
| Event producers |
Publish, Dispatch, Send, emit, EventBus, outbox, IntegrationEvent |
| Event consumers |
Consumer, EventHandler, Subscribe, @EventListener, inbox |
| Sagas/orchestration |
Saga, ProcessManager, Choreography, Workflow, Orchestrator |
| Sync service calls |
HTTP/gRPC calls to/from other services |
| Shared contracts |
OpenAPI spec, proto, shared DTO — flag breaking changes |
| Data ownership |
Other service reads/writes same table/collection → Shared-DB anti-pattern |
Per touchpoint: owner service · message name · consumers · risk (NONE / ADDITIVE / BREAKING).
BLOCKED until: Producers scanned · Consumers scanned · Sagas checked · Contracts reviewed · Breaking-change risk flagged
AI Mistake Prevention — Failure modes to avoid on every task:
Re-read files after context changes. Context compaction, resume, or long-running work can make memory stale; verify current files before acting.
Verify generated content against source evidence. AI hallucinates APIs, names, claims, and document facts. Check the relevant source before documenting or referencing.
Check downstream references before deleting or renaming. Removing an artifact can stale docs, generated mirrors, configs, and callers; map references first.
Trace the full impact chain after edits. Changing a definition can miss derived outputs and consumers. Follow the affected chain before declaring done.
Verify ALL affected outputs, not just the first. One green check is not all green checks; validate every output surface the change can affect.
Assume existing values are intentional — ask WHY before changing OR flagging one as a defect. Before changing or reporting a constant, limit, flag, cutoff, wording, or pattern, read nearby context and history, the CALLER's ordering, and 2+ sibling call sites of the same convention. A doc stating WHAT without WHY is missing rationale, not proof of a missing guard.
Surface ambiguity before acting — don't pick silently. Multiple valid interpretations require an explicit question or stated assumption with risk.
Assert the outcome your system owns, not the intermediate state your infrastructure owns. When verifying async work, assert the final business state — never the delivery/retry bookkeeping held in shared infrastructure that any co-running process can write. Such a check passes when run alone and flakes the moment anything else shares that infrastructure.
Keep shared guidance role-relevant. Universal guidance must help every receiving skill or agent; code-specific obligations belong only in code-specific protocols.
IMPORTANT MUST ATTENTION search 3+ existing patterns and read code BEFORE any modification. Run graph trace when graph.db exists.
- MANDATORY IMPORTANT MUST ATTENTION cite
file:line evidence for every claim. Confidence >80% to act, <60% = do NOT recommend.
IMPORTANT MUST ATTENTION include ## Test Specifications with TC IDs per phase. Call TaskList before creating new tasks.
IMPORTANT MUST ATTENTION microservices/event-driven: scan producers, consumers, sagas, contracts in task scope. Per touchpoint: owner · message · consumers · risk (NONE/ADDITIVE/BREAKING). Missing consumer = silent regression.
MUST ATTENTION apply critical + sequential thinking — every claim needs appropriate traced evidence (file:line for repo/code claims; source URL or artifact section for research, product, content, and docs claims); confidence >80% to act, <60% DO NOT recommend. Anti-hallucination: never present guess as fact, admit uncertainty freely, cross-reference independently, stay skeptical of own confidence.
MUST ATTENTION apply sequential-thinking — multi-step Thought N/M, REVISION/BRANCH/HYPOTHESIS markers, confidence % closer; see /sequential-thinking skill.
MUST ATTENTION apply AI mistake prevention — verify generated content against evidence, trace downstream references before deleting or renaming, verify all affected outputs, re-read files after context loss, and surface ambiguity before acting.
- MANDATORY Bootstrap task tracking before target work; transition one task at a time.
- MANDATORY Persist plan/review findings to
plans/reports/ incrementally and synthesize from disk.
- MANDATORY Before investigating, planning, or coding, read
docs/project-config.json (the project map: modules/paths, run-commands, conventions, architecture/workflow rules) + the required project-reference docs, and cite Reference docs read: ....
- MANDATORY Always include
lessons.md; project config + conventions override generic framework defaults.
- MANDATORY If project config, root instruction files, or any required reference doc is missing or stale, auto-run
/project-init or the narrow lower-level route before ordinary project-specific work.
- MANDATORY Parent workflow rows do not replace child phase tracking; expand phases and link the parent when nested.
- MANDATORY Orchestrators pre-expand child skill phases before invocation; use
[N.M] $skill-name — phase prefixes and one-in_progress discipline.
Prompt-Enhance Closing Anchors
IMPORTANT MUST ATTENTION follow declared step order for this skill; NEVER skip, reorder, or merge steps without explicit user approval
IMPORTANT MUST ATTENTION for every step/sub-skill call: set in_progress before execution, set completed after execution
IMPORTANT MUST ATTENTION every skipped step MUST include explicit reason; every completed step MUST include concise evidence
IMPORTANT MUST ATTENTION if Task tools unavailable, maintain an equivalent step-by-step plan tracker with synchronized statuses
Project Protocol Overlay — Before executing this skill, resolve any PROJECT overlay rules layered onto it: match this skill's name against the Target column of the project's skill-protocol index (docs/project-reference/skill-protocols-reference.md by default; a referenceDocs entry in docs/project-config.json overrides the path), taking the most specific matching tier ONLY — exact name > glob > *. That precedence orders overlays against EACH OTHER, never against this skill. Read ONLY the matched bodies, resolved as <protocols-dir>/<Name>.md; a row's Body link is display text, never a read path. A matched body that is missing or malformed is REPORTED and skipped — never reconstructed from the index Description. No index, or no match -> proceed with no overlay, silently. Full contract: .claude/skills/project-skill-protocol/references/registry.md.
Overlays are ADDITIVE ONLY: they ADD rules on top of this skill's own protocol and NEVER replace, override, disable, or reinterpret a rule it already states — removing every overlay must return this skill to exactly its documented behavior. An overlay is a BRIEF, not an authority escalation: it can NEVER waive a workflow gate, git discipline, a review gate, or a user-confirmation gate. A genuine overlay-vs-skill conflict, or two equally-specific overlays that directly contradict -> surface both to the user; NEVER resolve silently.
MUST ATTENTION resolve project protocol overlays for this skill BEFORE executing — most specific matching tier only (exact > glob > *, which ranks overlays against each other, NEVER against this skill), read only matched bodies at <protocols-dir>/<Name>.md; a missing or malformed body is reported, never reconstructed. Overlays are ADDITIVE ONLY (they never replace this skill's own rules) and are a brief, NEVER an authority escalation; an equal-specificity contradiction goes to the user.
Closing Reminders
IMPORTANT MUST ATTENTION Goal: Force every assumption-laden plan decision and every preservation-critical behavior through explicit user confirmation BEFORE implementation — by interviewing the user with critical questions that validate assumptions and surface issues — so no unstated assumption silently reaches code.
Protocols in force (concise digest of the SYNC/shared blocks this skill carries):
- Nested Task Creation: child skill still creates visible phase tasks; link parent when nested.
- Task Tracking & External Report: bootstrap task breakdown first; persist findings incrementally to
plans/reports/.
- Critical Thinking: MUST ATTENTION apply critical + sequential thinking; cite proof; confidence >80% to act.
- Sequential Thinking: structured multi-step Thought N/M with REVISION/BRANCH/HYPOTHESIS markers and confidence closer.
- Project Reference Docs: read required project-reference docs before target work; always include
lessons.md.
- Understand Code First: MUST ATTENTION read code and grep 3+ patterns before any modification.
- Plan Quality: include
## Test Specifications with TC-{FEATURE}-{NNN} IDs per phase.
- Cross-Service Check: scan producers, consumers, sagas, contracts; flag breaking-change risk.
- AI Mistake Prevention: verify generated content against evidence, trace downstream references, verify all affected outputs, re-read after context loss, surface ambiguity.
IMPORTANT MUST ATTENTION run the main steps IN ORDER — Phase 0 Detect Plan Type → resolve plan path → load mode + questions range → Step 1 Read plan.md + all phase-*.md (flag decisions/assumptions/risks/tradeoffs) → Step 2 Extract topics (8 categories) → Step 3 Generate questions (2-4 options each) → Step 4 Interview via AskUserQuestion (≤4 per call) → Step 5 Document answers → offer implement/refine/skip — why: the pipeline IS the work; never collapse or skip a step from memory
IMPORTANT MUST ATTENTION validate decisions with the user via AskUserQuestion — NEVER auto-decide or self-answer; completing without ≥1 question is a protocol violation — why: the user owns every assumption-laden choice, not the agent
IMPORTANT MUST ATTENTION detect plan type FIRST (Phase 0) BEFORE generating questions — bugfix keywords (fix, bug, regression, broken, defect) make the Preservation question BLOCKING, never skipped — why: detection drives which categories fire and the Preservation gate
IMPORTANT MUST ATTENTION NEVER modify phase files — persist results by adding ONLY a ## Validation Summary (confirmed decisions + action items) to plan.md — why: phase files are the plan's source of truth and validation is a read-then-annotate pass
- MANDATORY IMPORTANT MUST ATTENTION break work into small todo tasks using
TaskCreate BEFORE starting (including a task per file read); call TaskList first on context loss, never duplicate — why: resume existing tasks rather than re-plan after compaction
- MANDATORY IMPORTANT MUST ATTENTION honor the
questions MIN-MAX range and mode from ## Plan Context as hard constraints; give 2-4 concrete options per question, never go below min — why: the interview budget is configured, not improvised
- MANDATORY IMPORTANT MUST ATTENTION treat the Preservation "Unsure" answer as BLOCKED → return BLOCKED status and route to
/plan preservation analysis before any implementation — why: an unverified preserved-correctness invariant is a silent regression risk
- MANDATORY IMPORTANT MUST ATTENTION if the plan introduces new tech/packages, probe whether alternatives were evaluated before accepting the choice — why: unevaluated dependency choices raise future change cost
- MANDATORY IMPORTANT MUST ATTENTION cite
file:line proof or traced evidence with confidence % for every claim (>80% act, <80% verify first); admit uncertainty rather than present a guess as fact — why: speculation drives wrong validation questions
- MANDATORY IMPORTANT MUST ATTENTION search 3+ existing patterns and read the plan + phase files BEFORE generating questions — match the codebase's local conventions over generic framework defaults — why: questions grounded in actual code surface real decisions, not invented ones
- MANDATORY IMPORTANT MUST ATTENTION apply the Easy-to-Change lens before any rule below — flag decisions that raise future change cost (coupling, hidden state, duplicated knowledge, unclear intent, irreversible early choices)
- MANDATORY IMPORTANT MUST ATTENTION add a final review task to verify work quality
Anti-Rationalization:
| Evasion |
Rebuttal |
| "Plan is simple, skip validation" |
Simple plans still have implicit decisions. Apply anyway. |
| "Already know the answers" |
Show user responses as proof. No responses = no validation. |
| "Preservation doesn't apply here" |
If title has fix/bug/regression/broken/defect → ALWAYS applies. |
| "Phase 0 not needed" |
Detection drives the Preservation gate. NEVER skip. |
| "Only ask a few questions" |
Use the questions range from Plan Context. Never go below min. |
| "I'll just answer for the user" |
AskUserQuestion is mandatory. Self-answer = no validation. |
| "New library is obviously fine" |
Probe whether alternatives were evaluated before accepting it. |
| "I'll edit the phase files inline" |
NEVER. Add only a ## Validation Summary to plan.md. |
[TASK-PLANNING] Before acting, analyze task scope and systematically break it into small todo tasks and sub-tasks using TaskCreate.
IMPORTANT MUST ATTENTION detect plan type (Phase 0) FIRST — bugfix keywords make Preservation BLOCKING.
IMPORTANT MUST ATTENTION validate with the user via AskUserQuestion — NEVER auto-decide.
IMPORTANT MUST ATTENTION NEVER modify phase files — add only a ## Validation Summary to plan.md.
Closing reminder — Easy to Change is the success metric. Every finding,
test, refactor, and abstraction must answer one question: does this make
the next change cheaper or more expensive? If it doesn't reduce future
change cost, reject it. Coupling, hidden state, duplicated knowledge, and
unclear intent are the real enemies — call them out by name.
1---2name: plan-validate-33description: [Planning] Use when you need to validate a plan with critical questions interview.4---56<!-- PROMPT-ENHANCE:STEP-TASK-ANCHOR:START -->78> **[BLOCKING]** Execute skill steps in declared order. NEVER skip, reorder, or merge steps without explicit user approval.9> **[BLOCKING]** Before each step or sub-skill call, update task tracking: set `in_progress` when step starts, set `completed` when step ends.10> **[BLOCKING]** Every completed/skipped step MUST include brief evidence or explicit skip reason.11> **[BLOCKING]** If Task tools are unavailable, create and maintain an equivalent step-by-step plan tracker with the same status transitions.1213<!-- PROMPT-ENHANCE:STEP-TASK-ANCHOR:END -->1415## Quick Summary1617**Goal:** Force every assumption-laden plan decision and every preservation-critical behavior through explicit user confirmation BEFORE implementation — by interviewing the user with critical questions that validate assumptions and surface issues — so no unstated assumption silently reaches code.1819**Summary:**2021- **Purpose:** validate a finished plan via a critical-questions interview so every assumption-laden decision and every preservation-critical behavior is user-confirmed BEFORE implementation — no unstated assumption silently reaches code.22- **Main steps (run in order):** Phase 0 Detect Plan Type → resolve plan path (`$ARGUMENTS` / `## Plan Context` / ask) → load `mode` + `questions` range as hard constraints → Step 1 Read `plan.md` + all `phase-*.md`, flag decisions/assumptions/risks/tradeoffs → Step 2 Extract topics across 8 categories (Architecture, Assumptions, Tradeoffs, Risks, Scope, New Tech/Lib, Test Specs, Preservation) → Step 3 Generate questions (2-4 concrete options each, surface implicit decisions) → Step 4 Interview via `AskUserQuestion` (≤4 per call) → Step 5 Document answers → offer implement/refine/skip.23- **Phase 0 weights everything:** plan type (bugfix/feature/migration/refactor/other) decides which question categories fire; any fix/bug/regression/broken/defect keyword makes the Preservation question BLOCKING — never skip it.24- **The output is a REAL interview, not a self-answer:** honor the `questions` MIN-MAX range from `## Plan Context`, give 2-4 concrete options per question, treat the Preservation "Unsure" answer as BLOCKED → route to `/plan`; if the plan adds new tech/packages, probe whether alternatives were evaluated before accepting the choice.25- **Persist results narrowly:** add ONLY a `## Validation Summary` (confirmed decisions + action items) to `plan.md` — NEVER edit phase files; close by offering implement/refine/skip via `AskUserQuestion`.2627**Workflow:**28291. **Detect Plan Type** — Classify plan (bugfix/feature/migration/refactor) to weight question categories302. **Read Plan** — Parse plan.md + phase files for decisions, assumptions, risks313. **Extract Topics** — Scan architecture, assumptions, tradeoffs, risks, scope keywords324. **Generate Questions** — Formulate concrete questions with 2-4 options each335. **Interview User** — Present questions using configured count range346. **Document Answers** — Add Validation Summary section to plan.md3536**Key Rules:**3738- MUST ATTENTION use `AskUserQuestion` — NEVER auto-decide on behalf of user — why: the user owns every assumption-laden choice, not the agent39- Ask ONLY about genuine choices affecting implementation — NEVER about non-decision points — why: noise questions burn the interview budget and erode trust40- Bugfix plans ALWAYS trigger the Preservation question (keywords: fix, bug, regression, broken, defect) — why: an unverified preserved-correctness invariant is a silent regression41- Persist via a `## Validation Summary` on `plan.md` — NEVER modify phase files — why: phase files are the plan's source of truth; validation is a read-then-annotate pass4243## First Principle — Easy to Change4445> **The success metric of every coding decision is _future change cost_.**46> DRY, SRP, abstraction, design patterns, naming, layering, tests — every47> technique exists to serve one goal: **making the next change cheaper**.4849Evaluating code, refactor, test, abstraction, ask:50**does this make next change cheaper or more expensive?**5152- Reject "best practices" raising change cost (premature abstraction,53 speculative generality, leaky indirection, ceremony without payoff).54- Name real enemies in findings: **coupling, hidden state, duplicated55 knowledge, unclear intent, irreversible decisions exposed too early**.56- Simpler design easy to change beats sophisticated design that isn't.5758Apply this lens **before** invoking any specific rule, pattern, or checklist59below — if a downstream rule raises change cost, this principle wins.6061---6263## Phase 0: Detect Plan Type6465Classify plan type BEFORE generating questions — drives question category weighting:6667| Plan Type | Detection | Mandatory Extra Categories |68| ------------- | ----------------------------------------------------------------- | ------------------------------------- |69| **Bugfix** | Title/frontmatter: `fix`, `bug`, `regression`, `broken`, `defect` | Preservation (BLOCKING) |70| **Feature** | New capability, no fix keywords | Architecture, Assumptions, Test Specs |71| **Migration** | Schema change, EF migration, data move | Risks, Preservation, Scope |72| **Refactor** | Restructure/clean up, no behavior change | Preservation, Tradeoffs |73| **Other** | None of above | Architecture, Scope |7475**Bugfix detection is BLOCKING** — NEVER skip Preservation question when fix/bug/regression/broken/defect keywords present.7677## Plan Resolution78791. `$ARGUMENTS` provided → use that path802. Check `## Plan Context` section → use active plan path813. No plan found → ask user to specify path or run `/plan` first8283## Configuration (from injected context)8485Check `## Plan Context` section:8687- `mode` — auto/prompt/off behavior88- `questions` — range like `3-8` (min-max)8990Use as hard constraints.9192## Workflow9394### Step 1: Read Plan Files9596Read plan directory:9798- `plan.md` — overview + phases list99- `phase-*.md` — all phase files100- Flag: decision points, assumptions, risks, tradeoffs101102### Step 2: Extract Question Topics103104| Category | Keywords |105| ---------------- | ------------------------------------------------------------------------------------------ |106| **Architecture** | approach, pattern, design, structure, database, API |107| **Assumptions** | assume, expect, should, will, must, default |108| **Tradeoffs** | tradeoff, vs, alternative, option, choice, either/or |109| **Risks** | risk, might, could fail, dependency, blocker, concern |110| **Scope** | phase, MVP, future, out of scope, nice to have |111| **New Tech/Lib** | install, add package, new dependency, npm install, dotnet add, unfamiliar framework names |112| **Test Specs** | TC-, test case, coverage, TDD, test specification |113| **Preservation** | auto-trigger on bugfix keywords in title/frontmatter — scan Preservation Inventory section |114115### Step 3: Generate Questions116117**Format rules:**118119- 2-4 concrete options per question120- Mark recommended with "(Recommended)" suffix121- "Other" option automatic — do NOT add122- Surface implicit decisions123124**Examples:**125126```127Category: Architecture128Question: "How should validation results be persisted?"129Options:1301. Save to plan.md frontmatter (Recommended) — updates existing plan1312. Create validation-answers.md — separate answers file1323. Don't persist — ephemeral validation only133```134135```136Category: Assumptions137Question: "Plan assumes API rate limiting not needed. Correct?"138Options:1391. Yes, not needed for MVP1402. No, add basic rate limiting now (Recommended)1413. Defer to Phase 2142```143144```145Category: Preservation (MANDATORY when title/frontmatter: fix, bug, regression, broken, defect)146Question: "List 2-3 inputs where CURRENT code is correct. Will fix change behavior on any?"147Options (multi-select):1481. "Current code correct on: {input A}. Fix preserves behavior." (Recommended)1492. "Current code correct on: {input B}. Fix CHANGES behavior because: {justification}"1503. "Current code has NO preserved-correctness inputs — every input was broken" (rare; requires confirmation)1514. "Unsure — need to investigate" (STOP: run /plan preservation analysis)152```153154**Follow-up rules:**155156- Option 2 selected → `plan.md` Preservation Inventory MUST cite Preservation TC asserting new behavior is intended157- Option 4 selected → return BLOCKED status, recommend `/plan` before proceeding158- Option 3 selected → `AskUserQuestion` follow-up: "Confirm: current code has NO preserved invariant? [Yes, every input broken / No, missed some — re-investigate]"159160### Step 4: Interview User161162Use `AskUserQuestion` — NEVER skip or auto-answer.163164**Rules:**165166- Use question count from `## Plan Context` → `Validation: mode=X, questions=MIN-MAX`167- Group related questions (max 4 per tool call)168- Focus: assumptions, risks, tradeoffs, architecture169- MANDATORY IMPORTANT MUST ATTENTION: if plan introduces new tech/packages, ask: "Plan uses {lib}. Were alternatives evaluated? Confirm choice or research more?"170171### Step 5: Document Answers172173Add `## Validation Summary` to `plan.md`:174175```markdown176## Validation Summary177178**Validated:** {date}179**Questions asked:** {count}180181### Confirmed Decisions182183- {decision 1}: {user choice}184- {decision 2}: {user choice}185186### Action Items187188- [ ] {changes needed based on answers}189```190191NEVER modify phase files — only document what needs updating.192193## Output194195After validation:196197- Questions asked count198- Key decisions confirmed199- Items flagged for plan revision200- Recommendation: proceed to implementation OR revise plan first201202## Next Steps203204**MANDATORY IMPORTANT MUST ATTENTION — NO EXCEPTIONS** after completing, use `AskUserQuestion` to present:205206- **"/feature-implement (Recommended)"** — Begin implementation with validated plan207- **"/refine"** — If plan needs PBI refinement first208- **"Skip, continue manually"** — User decides209210---211212> **[BLOCKING]** MUST ATTENTION use `AskUserQuestion` to interview user. Completing without asking ≥1 question = violation.213214> **[IMPORTANT]** Use `TaskCreate` to break ALL work into small tasks BEFORE starting — including tasks for each file read. For simple tasks, AI MUST ATTENTION ask user whether to skip.215216> **External Memory:** Complex/lengthy work → write findings + results to `plans/reports/` — prevents context loss.217218> **Evidence Gate:** MANDATORY IMPORTANT MUST ATTENTION — every claim requires `file:line` proof or traced evidence with confidence % (>80% act, <80% verify first).219220<!-- SYNC:nested-task-creation -->221222> **Nested Task Expansion Contract** — For workflow-step invocation, the `[Workflow] ...` row is only a parent container; the child skill still creates visible phase tasks.223>224> 1. Call `TaskList` first. If a matching active parent workflow row exists, set `nested=true` and record `parentTaskId`; otherwise run standalone.225> 2. Create one task per declared phase before phase work. When nested, prefix subjects `[N.M] $skill-name — phase`.226> 3. When nested, link the parent with `TaskUpdate(parentTaskId, addBlockedBy: [childIds])`.227> 4. Orchestrators must pre-expand a child skill's phase list and link the workflow row before invoking that child skill or sub-agent.228> 5. Mark exactly one child `in_progress` before work and `completed` immediately after evidence is written.229> 6. Complete the parent only after all child tasks are completed or explicitly cancelled with reason.230>231> **Blocked until:** `TaskList` done, child phases created, parent linked when nested, first child marked `in_progress`.232233<!-- /SYNC:nested-task-creation -->234235<!-- SYNC:task-tracking-external-report -->236237> **Task Tracking & External Report Persistence** — Bootstrap this before execution; then run project-reference doc prefetch before target/source work.238>239> 1. Create a small task breakdown before target file reads, grep, edits, or analysis. On context loss, inspect the current task list first.240> 2. Mark one task `in_progress` before work and `completed` immediately after evidence; never batch transitions.241> 3. For plan/review work, create `plans/reports/{skill}-{YYMMDD}-{HHmm}-{slug}.md` before first finding.242> 4. Append findings after each file/section/decision and synthesize from the report file at the end.243> 5. Final output cites `Full report: plans/reports/{filename}`.244>245> **Blocked until:** task breakdown exists, report path declared for plan/review work, first finding persisted before the next finding.246247<!-- /SYNC:task-tracking-external-report -->248249<!-- SYNC:critical-thinking-mindset -->250251> **Critical Thinking Mindset** — Apply critical thinking, sequential thinking. Every claim needs traced proof, confidence >80% to act.252> **Anti-hallucination:** Never present guess as fact — cite sources for every claim, admit uncertainty freely, self-check output for errors, cross-reference independently, stay skeptical of own confidence — certainty without evidence root of all hallucination.253254<!-- /SYNC:critical-thinking-mindset -->255256<!-- SYNC:sequential-thinking-protocol -->257258> **Sequential Thinking Protocol** — Structured multi-step reasoning for complex/ambiguous work. Use when planning, reviewing, debugging, or refining ideas where one-shot reasoning is unsafe.259>260> **Trigger when:** complex problem decomposition · adaptive plans needing revision · analysis with course correction · unclear/emerging scope · multi-step solutions · hypothesis-driven debugging · cross-cutting trade-off evaluation.261>262> **Format (explicit mode — visible thought trail):**263>264> 1. `Thought N/M: [aspect]` — one aspect per thought, state assumptions/uncertainty265> 2. `Thought N/M [REVISION of Thought K]: ...` — when prior reasoning invalidated; state Original / Why revised / Impact266> 3. `Thought N/M [BRANCH A from Thought K]: ...` — explore alternative; converge with decision rationale267> 4. `Thought N/M [HYPOTHESIS]: ...` then `[VERIFICATION]: ...` — test before acting268> 5. `Thought N/N [FINAL]` — only when verified, all critical aspects addressed, confidence >80%269>270> **Mandatory closers:** Confidence % stated · Assumptions listed · Open questions surfaced · Next action concrete.271>272> **Stop conditions:** confidence <80% on any critical decision → escalate via AskUserQuestion · ≥3 revisions on same thought → re-frame the problem · branch count >3 → split into sub-task.273>274> **Implicit mode:** apply methodology internally without visible markers when adding markers would clutter the response (routine work where reasoning aids accuracy).275>276> **Deep-dive:** see `/sequential-thinking` skill (`.claude/skills/sequential-thinking/SKILL.md`) for worked examples (API design, debugging, architecture), advanced techniques (spiral refinement, hypothesis testing, convergence), and meta-strategies (uncertainty handling, revision cascades).277278<!-- /SYNC:sequential-thinking-protocol -->279280<!-- SYNC:project-reference-docs-guide -->281282> **Project Reference Docs Gate** — Run after task-tracking bootstrap and before target/source file reads, grep, edits, or analysis. Project docs override generic framework assumptions.283>284> 1. Identify scope: file types, domain area, and operation.285> 2. **Read `docs/project-config.json` first — the project's machine-readable map.** It is the single source of truth for THIS repo (modules/paths, framework + search keywords, test/E2E/integration run-commands, design system, architecture rules, workflow patterns); ground exact paths, run-commands, and conventions on it **before investigating, planning, or coding** — never assume framework defaults (`CLAUDE.md` + reference docs are derived from it). If it — or the docs index, `lessons.md`, `CLAUDE.md`, `AGENTS.md`, or any required reference doc — is missing or stale, auto-run `/project-init` or the narrow route (`/project-config`, `/docs-init`, `/scan-all`, `/scan --target=<key>`, `/claude-md-init`) first; if Codex mirrors or `AGENTS.md` are stale, ask the user to run `/sync-codex` (never auto-run it).286> 3. Required docs by trigger: always `docs/project-reference/lessons.md`; doc lookup `docs-index-reference.md`; review `code-review-rules.md`; backend/CQRS/API `backend-patterns-reference.md`; domain/entity `domain-entities-reference.md`; frontend/UI `frontend-patterns-reference.md`; styles/design `scss-styling-guide.md` + `design-system/design-system-canonical.md`; integration tests `integration-test-reference.md`; E2E `e2e-test-reference.md`; feature docs/specs `feature-spec-reference.md` + `spec-system-reference.md` + `spec-principles.md`; behavior/public-contract/spec-test-code sync `workflow-spec-test-code-cycle-reference.md`; derived spec index/ERD/reimplementation guides `spec-system-reference.md` + source Feature Specs under `docs/specs/`; architecture/new area `project-structure-reference.md`.287> 4. Read every required doc, then before target work state: `Reference docs read: ... | Not applicable: ...`.288>289> **Ready when:** scope evaluated, `docs/project-config.json` consulted, required docs checked/read or setup route completed, `lessons.md` confirmed, citation emitted.290291<!-- /SYNC:project-reference-docs-guide -->292293<!-- SYNC:understand-code-first -->294295> **Understand Code First** — HARD-GATE: Do NOT write, plan, or fix until you READ existing code.296>297> 1. Search 3+ similar patterns (`grep`/`glob`) — cite `file:line` evidence298> 2. Read existing files in target area — understand structure, base classes, conventions299> 3. Run `python .claude/scripts/code_graph trace <file> --direction both --json` when `.code-graph/graph.db` exists300> 4. Map dependencies via `connections` or `callers_of` — know what depends on your target301> 5. Write investigation to `.ai/workspace/analysis/` for non-trivial tasks (3+ files)302> 6. Re-read analysis file before implementing — never work from memory alone. — why: long context drifts from the file; the file is ground truth303> 7. NEVER invent new patterns when existing ones work — match exactly or document deviation. — why: divergent patterns fragment the codebase and slow every future reader304>305> **BLOCKED until:** `- [ ]` Read target files `- [ ]` Grep 3+ patterns `- [ ]` Graph trace (if graph.db exists) `- [ ]` Assumptions verified with evidence306307<!-- /SYNC:understand-code-first -->308309<!-- SYNC:plan-quality -->310311> **Plan Quality** — Every plan phase MUST ATTENTION include test specifications.312>313> 1. Add `## Test Specifications` section with TC-{FEATURE}-{NNN} IDs to every phase file314> 2. Map every functional requirement to ≥1 TC (or explicit `TBD` with rationale)315> 3. TC IDs follow `TC-{FEATURE}-{NNN}` format — reference by ID, never embed full content316> 4. Before any new workflow step: call `TaskList` and re-read the phase file317> 5. On context compaction: call `TaskList` FIRST — never create duplicate tasks318> 6. Verify TC satisfaction per phase before marking complete (evidence must be `file:line`, not TBD)319>320> **Mode:** TDD-first → reference existing TCs with `Evidence: TBD`. Implement-first → use TBD → `/spec [mode=tests]` fills after.321322<!-- /SYNC:plan-quality -->323324<!-- SYNC:cross-service-check -->325326> **Cross-Service Check** — Microservices/event-driven: MANDATORY before concluding investigation, plan, spec, or feature doc. Missing downstream consumer = silent regression.327>328> | Boundary | Grep terms |329> | ------------------- | ------------------------------------------------------------------------------- |330> | Event producers | `Publish`, `Dispatch`, `Send`, `emit`, `EventBus`, `outbox`, `IntegrationEvent` |331> | Event consumers | `Consumer`, `EventHandler`, `Subscribe`, `@EventListener`, `inbox` |332> | Sagas/orchestration | `Saga`, `ProcessManager`, `Choreography`, `Workflow`, `Orchestrator` |333> | Sync service calls | HTTP/gRPC calls to/from other services |334> | Shared contracts | OpenAPI spec, proto, shared DTO — flag breaking changes |335> | Data ownership | Other service reads/writes same table/collection → Shared-DB anti-pattern |336>337> **Per touchpoint:** owner service · message name · consumers · risk (NONE / ADDITIVE / BREAKING).338>339> **BLOCKED until:** Producers scanned · Consumers scanned · Sagas checked · Contracts reviewed · Breaking-change risk flagged340341<!-- /SYNC:cross-service-check -->342343<!-- SYNC:ai-mistake-prevention -->344345> **AI Mistake Prevention** — Failure modes to avoid on every task:346>347> **Re-read files after context changes.** Context compaction, resume, or long-running work can make memory stale; verify current files before acting.348> **Verify generated content against source evidence.** AI hallucinates APIs, names, claims, and document facts. Check the relevant source before documenting or referencing.349> **Check downstream references before deleting or renaming.** Removing an artifact can stale docs, generated mirrors, configs, and callers; map references first.350> **Trace the full impact chain after edits.** Changing a definition can miss derived outputs and consumers. Follow the affected chain before declaring done.351> **Verify ALL affected outputs, not just the first.** One green check is not all green checks; validate every output surface the change can affect.352> **Assume existing values are intentional — ask WHY before changing OR flagging one as a defect.** Before changing or reporting a constant, limit, flag, cutoff, wording, or pattern, read nearby context and history, the CALLER's ordering, and 2+ sibling call sites of the same convention. A doc stating WHAT without WHY is missing rationale, not proof of a missing guard.353> **Surface ambiguity before acting — don't pick silently.** Multiple valid interpretations require an explicit question or stated assumption with risk.354> **Assert the outcome your system owns, not the intermediate state your infrastructure owns.** When verifying async work, assert the final business state — never the delivery/retry bookkeeping held in shared infrastructure that any co-running process can write. Such a check passes when run alone and flakes the moment anything else shares that infrastructure.355> **Keep shared guidance role-relevant.** Universal guidance must help every receiving skill or agent; code-specific obligations belong only in code-specific protocols.356357<!-- /SYNC:ai-mistake-prevention -->358359<!-- SYNC:understand-code-first:reminder -->360361**IMPORTANT MUST ATTENTION** search 3+ existing patterns and read code BEFORE any modification. Run graph trace when graph.db exists.362363<!-- /SYNC:understand-code-first:reminder -->364365<!-- SYNC:evidence-based-reasoning:reminder -->366367- **MANDATORY IMPORTANT MUST ATTENTION** cite `file:line` evidence for every claim. Confidence >80% to act, <60% = do NOT recommend.368 <!-- /SYNC:evidence-based-reasoning:reminder -->369370<!-- SYNC:plan-quality:reminder -->371372**IMPORTANT MUST ATTENTION** include `## Test Specifications` with TC IDs per phase. Call `TaskList` before creating new tasks.373374<!-- /SYNC:plan-quality:reminder -->375376<!-- SYNC:cross-service-check:reminder -->377378**IMPORTANT MUST ATTENTION** microservices/event-driven: scan producers, consumers, sagas, contracts in task scope. Per touchpoint: owner · message · consumers · risk (NONE/ADDITIVE/BREAKING). Missing consumer = silent regression.379380<!-- /SYNC:cross-service-check:reminder -->381382<!-- SYNC:critical-thinking-mindset:reminder -->383384**MUST ATTENTION** apply critical + sequential thinking — every claim needs appropriate traced evidence (`file:line` for repo/code claims; source URL or artifact section for research, product, content, and docs claims); confidence >80% to act, <60% DO NOT recommend. Anti-hallucination: never present guess as fact, admit uncertainty freely, cross-reference independently, stay skeptical of own confidence.385386<!-- /SYNC:critical-thinking-mindset:reminder -->387388<!-- SYNC:sequential-thinking-protocol:reminder -->389390**MUST ATTENTION** apply sequential-thinking — multi-step Thought N/M, REVISION/BRANCH/HYPOTHESIS markers, confidence % closer; see `/sequential-thinking` skill.391392<!-- /SYNC:sequential-thinking-protocol:reminder -->393394<!-- SYNC:ai-mistake-prevention:reminder -->395396**MUST ATTENTION** apply AI mistake prevention — verify generated content against evidence, trace downstream references before deleting or renaming, verify all affected outputs, re-read files after context loss, and surface ambiguity before acting.397398<!-- /SYNC:ai-mistake-prevention:reminder -->399400<!-- SYNC:task-tracking-external-report:reminder -->401402- **MANDATORY** Bootstrap task tracking before target work; transition one task at a time.403- **MANDATORY** Persist plan/review findings to `plans/reports/` incrementally and synthesize from disk.404405<!-- /SYNC:task-tracking-external-report:reminder -->406407<!-- SYNC:project-reference-docs-guide:reminder -->408409- **MANDATORY** Before investigating, planning, or coding, read `docs/project-config.json` (the project map: modules/paths, run-commands, conventions, architecture/workflow rules) + the required project-reference docs, and cite `Reference docs read: ...`.410- **MANDATORY** Always include `lessons.md`; project config + conventions override generic framework defaults.411- **MANDATORY** If project config, root instruction files, or any required reference doc is missing or stale, auto-run `/project-init` or the narrow lower-level route before ordinary project-specific work.412413<!-- /SYNC:project-reference-docs-guide:reminder -->414415<!-- SYNC:nested-task-creation:reminder -->416417- **MANDATORY** Parent workflow rows do not replace child phase tracking; expand phases and link the parent when nested.418- **MANDATORY** Orchestrators pre-expand child skill phases before invocation; use `[N.M] $skill-name — phase` prefixes and one-`in_progress` discipline.419420<!-- /SYNC:nested-task-creation:reminder -->421422<!-- PROMPT-ENHANCE:STEP-TASK-CLOSING:START -->423424## Prompt-Enhance Closing Anchors425426**IMPORTANT MUST ATTENTION** follow declared step order for this skill; NEVER skip, reorder, or merge steps without explicit user approval427**IMPORTANT MUST ATTENTION** for every step/sub-skill call: set `in_progress` before execution, set `completed` after execution428**IMPORTANT MUST ATTENTION** every skipped step MUST include explicit reason; every completed step MUST include concise evidence429**IMPORTANT MUST ATTENTION** if Task tools unavailable, maintain an equivalent step-by-step plan tracker with synchronized statuses430431<!-- PROMPT-ENHANCE:STEP-TASK-CLOSING:END -->432433<!-- SYNC:project-protocol-overlay -->434435> **Project Protocol Overlay** — Before executing this skill, resolve any PROJECT overlay rules layered onto it: match this skill's name against the `Target` column of the project's skill-protocol index (`docs/project-reference/skill-protocols-reference.md` by default; a `referenceDocs` entry in `docs/project-config.json` overrides the path), taking the most specific matching tier ONLY — exact name > glob > `*`. **That precedence orders overlays against EACH OTHER, never against this skill.** Read ONLY the matched bodies, resolved as `<protocols-dir>/<Name>.md`; a row's Body link is display text, never a read path. A matched body that is missing or malformed is REPORTED and skipped — never reconstructed from the index Description. No index, or no match -> proceed with no overlay, silently. Full contract: `.claude/skills/project-skill-protocol/references/registry.md`.436>437> Overlays are **ADDITIVE ONLY**: they ADD rules on top of this skill's own protocol and NEVER replace, override, disable, or reinterpret a rule it already states — removing every overlay must return this skill to exactly its documented behavior. An overlay is a BRIEF, not an authority escalation: it can NEVER waive a workflow gate, git discipline, a review gate, or a user-confirmation gate. A genuine overlay-vs-skill conflict, or two equally-specific overlays that directly contradict -> surface both to the user; NEVER resolve silently.438439<!-- /SYNC:project-protocol-overlay -->440441<!-- SYNC:project-protocol-overlay:reminder -->442443**MUST ATTENTION** resolve project protocol overlays for this skill BEFORE executing — most specific matching tier only (exact > glob > `*`, which ranks overlays against each other, NEVER against this skill), read only matched bodies at `<protocols-dir>/<Name>.md`; a missing or malformed body is reported, never reconstructed. Overlays are ADDITIVE ONLY (they never replace this skill's own rules) and are a brief, NEVER an authority escalation; an equal-specificity contradiction goes to the user.444445<!-- /SYNC:project-protocol-overlay:reminder -->446447## Closing Reminders448449**IMPORTANT MUST ATTENTION Goal:** Force every assumption-laden plan decision and every preservation-critical behavior through explicit user confirmation BEFORE implementation — by interviewing the user with critical questions that validate assumptions and surface issues — so no unstated assumption silently reaches code.450451**Protocols in force (concise digest of the SYNC/shared blocks this skill carries):**452453- **Nested Task Creation:** child skill still creates visible phase tasks; link parent when nested.454- **Task Tracking & External Report:** bootstrap task breakdown first; persist findings incrementally to `plans/reports/`.455- **Critical Thinking:** MUST ATTENTION apply critical + sequential thinking; cite proof; confidence >80% to act.456- **Sequential Thinking:** structured multi-step Thought N/M with REVISION/BRANCH/HYPOTHESIS markers and confidence closer.457- **Project Reference Docs:** read required project-reference docs before target work; always include `lessons.md`.458- **Understand Code First:** MUST ATTENTION read code and grep 3+ patterns before any modification.459- **Plan Quality:** include `## Test Specifications` with TC-{FEATURE}-{NNN} IDs per phase.460- **Cross-Service Check:** scan producers, consumers, sagas, contracts; flag breaking-change risk.461- **AI Mistake Prevention:** verify generated content against evidence, trace downstream references, verify all affected outputs, re-read after context loss, surface ambiguity.462463**IMPORTANT MUST ATTENTION** run the main steps IN ORDER — Phase 0 Detect Plan Type → resolve plan path → load `mode` + `questions` range → Step 1 Read `plan.md` + all `phase-*.md` (flag decisions/assumptions/risks/tradeoffs) → Step 2 Extract topics (8 categories) → Step 3 Generate questions (2-4 options each) → Step 4 Interview via `AskUserQuestion` (≤4 per call) → Step 5 Document answers → offer implement/refine/skip — why: the pipeline IS the work; never collapse or skip a step from memory464465**IMPORTANT MUST ATTENTION** validate decisions with the user via `AskUserQuestion` — NEVER auto-decide or self-answer; completing without ≥1 question is a protocol violation — why: the user owns every assumption-laden choice, not the agent466**IMPORTANT MUST ATTENTION** detect plan type FIRST (Phase 0) BEFORE generating questions — bugfix keywords (fix, bug, regression, broken, defect) make the Preservation question BLOCKING, never skipped — why: detection drives which categories fire and the Preservation gate467**IMPORTANT MUST ATTENTION** NEVER modify phase files — persist results by adding ONLY a `## Validation Summary` (confirmed decisions + action items) to `plan.md` — why: phase files are the plan's source of truth and validation is a read-then-annotate pass468469- **MANDATORY IMPORTANT MUST ATTENTION** break work into small todo tasks using `TaskCreate` BEFORE starting (including a task per file read); call `TaskList` first on context loss, never duplicate — why: resume existing tasks rather than re-plan after compaction470- **MANDATORY IMPORTANT MUST ATTENTION** honor the `questions` MIN-MAX range and `mode` from `## Plan Context` as hard constraints; give 2-4 concrete options per question, never go below min — why: the interview budget is configured, not improvised471- **MANDATORY IMPORTANT MUST ATTENTION** treat the Preservation "Unsure" answer as BLOCKED → return BLOCKED status and route to `/plan` preservation analysis before any implementation — why: an unverified preserved-correctness invariant is a silent regression risk472- **MANDATORY IMPORTANT MUST ATTENTION** if the plan introduces new tech/packages, probe whether alternatives were evaluated before accepting the choice — why: unevaluated dependency choices raise future change cost473- **MANDATORY IMPORTANT MUST ATTENTION** cite `file:line` proof or traced evidence with confidence % for every claim (>80% act, <80% verify first); admit uncertainty rather than present a guess as fact — why: speculation drives wrong validation questions474- **MANDATORY IMPORTANT MUST ATTENTION** search 3+ existing patterns and read the plan + phase files BEFORE generating questions — match the codebase's local conventions over generic framework defaults — why: questions grounded in actual code surface real decisions, not invented ones475- **MANDATORY IMPORTANT MUST ATTENTION** apply the Easy-to-Change lens before any rule below — flag decisions that raise future change cost (coupling, hidden state, duplicated knowledge, unclear intent, irreversible early choices)476- **MANDATORY IMPORTANT MUST ATTENTION** add a final review task to verify work quality477478**Anti-Rationalization:**479480| Evasion | Rebuttal |481| ---------------------------------- | ---------------------------------------------------------------- |482| "Plan is simple, skip validation" | Simple plans still have implicit decisions. Apply anyway. |483| "Already know the answers" | Show user responses as proof. No responses = no validation. |484| "Preservation doesn't apply here" | If title has fix/bug/regression/broken/defect → ALWAYS applies. |485| "Phase 0 not needed" | Detection drives the Preservation gate. NEVER skip. |486| "Only ask a few questions" | Use the `questions` range from Plan Context. Never go below min. |487| "I'll just answer for the user" | `AskUserQuestion` is mandatory. Self-answer = no validation. |488| "New library is obviously fine" | Probe whether alternatives were evaluated before accepting it. |489| "I'll edit the phase files inline" | NEVER. Add only a `## Validation Summary` to `plan.md`. |490491**[TASK-PLANNING]** Before acting, analyze task scope and systematically break it into small todo tasks and sub-tasks using TaskCreate.492493**IMPORTANT MUST ATTENTION** detect plan type (Phase 0) FIRST — bugfix keywords make Preservation BLOCKING.494**IMPORTANT MUST ATTENTION** validate with the user via `AskUserQuestion` — NEVER auto-decide.495**IMPORTANT MUST ATTENTION** NEVER modify phase files — add only a `## Validation Summary` to `plan.md`.496497---498499> **Closing reminder — Easy to Change is the success metric.** Every finding,500> test, refactor, and abstraction must answer one question: _does this make501> the next change cheaper or more expensive?_ If it doesn't reduce future502> change cost, reject it. Coupling, hidden state, duplicated knowledge, and503> unclear intent are the real enemies — call them out by name.