Codex compatibility note:
- Invoke repository skills with
$skill-name in Codex; this mirrored copy rewrites legacy Claude /skill-name references.
- Task tracker mandate: BEFORE executing any workflow or skill step, create/update task tracking for all steps and keep it synchronized as progress changes.
- User-question prompts mean to ask the user directly in Codex.
- Ignore Claude-specific mode-switch instructions when they appear.
- Strict execution contract: when a user explicitly invokes a skill, execute that skill protocol as written.
- Subagent authorization: when a skill is user-invoked or AI-detected and its protocol requires subagents, that skill activation authorizes use of the required
spawn_agent subagent(s) for that task.
- Do not skip, reorder, or merge protocol steps unless the user explicitly approves the deviation first.
- For workflow skills, execute each listed child-skill step explicitly and report step-by-step evidence.
- If a required step/tool cannot run in this environment, stop and ask the user before adapting.
Codex Project-Reference Loading (No Hooks)
Codex uses static project-reference loading instead of runtime-injected project docs.
When coding, planning, debugging, testing, or reviewing, open project docs explicitly using this routing.
Always read:
docs/project-config.json (project-specific paths, commands, modules, and workflow/test settings)
docs/project-reference/docs-index-reference.md (routes to the full docs/project-reference/* catalog)
docs/project-reference/lessons.md (always-on guardrails and anti-patterns)
Missing/stale context route: If docs/project-config.json, the docs index, lessons.md, CLAUDE.md, AGENTS.md, or any task-required reference doc is missing or stale, auto-run $project-init or the narrow setup route ($project-config, $docs-init, $scan-all, $scan --target=<key>, $claude-md-init) before ordinary project-specific work. If Codex mirrors or AGENTS.md are missing/stale, ask the user to run $sync-codex; do not auto-run it.
Situation-based docs:
- Project structure/architecture/tech-stack/deployment/setup (any layer — backend, frontend, or infra):
project-structure-reference.md
- Backend/CQRS/API/domain/entity changes:
backend-patterns-reference.md, domain-entities-reference.md
- Frontend/UI/styling/design-system:
frontend-patterns-reference.md, scss-styling-guide.md, design-system/README.md
- Spec authoring,
docs/specs/ pathing, or TC format: feature-spec-reference.md, spec-system-reference.md, spec-principles.md
- Behavior/public-contract changes or spec-test-code sync:
workflow-spec-test-code-cycle-reference.md plus the spec docs above
- Derived spec indexes/ERDs/reimplementation guides:
spec-system-reference.md and source Feature Specs under docs/specs/
- Integration test implementation/review:
integration-test-reference.md
- E2E test implementation/review:
e2e-test-reference.md
- Code review/audit work:
code-review-rules.md plus domain docs above based on changed files
Do not read all docs blindly. Start from docs-index-reference.md, then open only relevant files for the task.
[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 by asking the user directly (≤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 by asking the user directly.
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 ask the user directly — 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 → ask the user directly follow-up: "Confirm: current code has NO preserved invariant? [Yes, every input broken / No, missed some — re-investigate]"
Step 4: Interview User
Use ask the user directly — 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 ask the user directly 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 ask the user directly to interview user. Completing without asking ≥1 question = violation.
[IMPORTANT] Use task tracking 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 the current task list 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: the current task list 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 by asking the user directly · ≥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 the current task list and re-read the phase file
- On context compaction: call the current task list 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 the current task list 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 by asking the user directly (≤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 by asking the user directly — 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 task tracking BEFORE starting (including a task per file read); call the current task list 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" |
ask the user directly 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 task tracking.
IMPORTANT MUST ATTENTION detect plan type (Phase 0) FIRST — bugfix keywords make Preservation BLOCKING.
IMPORTANT MUST ATTENTION validate with the user by asking the user directly — 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.
Hookless Prompt Protocol Mirror (Auto-Synced)
Source: .claude/.ck.json + .claude/skills/shared/sync-inline-versions.md (:full blocks) + .claude/scripts/lib/hookless-prompt-protocol.cjs
[WORKFLOW-EXECUTION-PROTOCOL] [BLOCKING] Workflow Execution Protocol — MANDATORY IMPORTANT MUST CRITICAL. Do not skip for any reason.
Generic portability boundary: Reusable skills and protocol text stay project-neutral; project-specific conventions are discovered from docs/project-config.json and docs/project-reference/. Apply shared AI-SDD from shared/sdd-artifact-contract.md. Read docs/project-config.json and docs/project-reference/docs-index-reference.md, then open the project reference docs named there. For spec, test-case, behavior-change, public-contract, or docs/specs/ work, route through the local spec docs named by the docs index: feature-spec-reference.md, spec-system-reference.md, spec-principles.md, and `workflow-spec-test-c
…(truncated)
1---2name: plan-validate3description: [Planning] Use when you need to validate a plan with critical questions interview.4---5
6> Codex compatibility note:
7>
8> - Invoke repository skills with `$skill-name` in Codex; this mirrored copy rewrites legacy Claude `/skill-name` references.
9> - Task tracker mandate: BEFORE executing any workflow or skill step, create/update task tracking for all steps and keep it synchronized as progress changes.
10> - User-question prompts mean to ask the user directly in Codex.
11> - Ignore Claude-specific mode-switch instructions when they appear.
12> - Strict execution contract: when a user explicitly invokes a skill, execute that skill protocol as written.
13> - Subagent authorization: when a skill is user-invoked or AI-detected and its protocol requires subagents, that skill activation authorizes use of the required `spawn_agent` subagent(s) for that task.
14> - Do not skip, reorder, or merge protocol steps unless the user explicitly approves the deviation first.
15> - For workflow skills, execute each listed child-skill step explicitly and report step-by-step evidence.
16> - If a required step/tool cannot run in this environment, stop and ask the user before adapting.
17
18<!-- CODEX:PROJECT-REFERENCE-LOADING:START -->
19
20## Codex Project-Reference Loading (No Hooks)
21
22Codex uses static project-reference loading instead of runtime-injected project docs.
23When coding, planning, debugging, testing, or reviewing, open project docs explicitly using this routing.
24
25**Always read:**
26
27- `docs/project-config.json` (project-specific paths, commands, modules, and workflow/test settings)
28- `docs/project-reference/docs-index-reference.md` (routes to the full `docs/project-reference/*` catalog)
29- `docs/project-reference/lessons.md` (always-on guardrails and anti-patterns)
30
31**Missing/stale context route:** If `docs/project-config.json`, the docs index, `lessons.md`, `CLAUDE.md`, `AGENTS.md`, or any task-required reference doc is missing or stale, auto-run `$project-init` or the narrow setup route (`$project-config`, `$docs-init`, `$scan-all`, `$scan --target=<key>`, `$claude-md-init`) before ordinary project-specific work. If Codex mirrors or `AGENTS.md` are missing/stale, ask the user to run `$sync-codex`; do not auto-run it.
32
33**Situation-based docs:**
34
35- Project structure/architecture/tech-stack/deployment/setup (any layer — backend, frontend, or infra): `project-structure-reference.md`
36- Backend/CQRS/API/domain/entity changes: `backend-patterns-reference.md`, `domain-entities-reference.md`
37- Frontend/UI/styling/design-system: `frontend-patterns-reference.md`, `scss-styling-guide.md`, `design-system/README.md`
38- Spec authoring, `docs/specs/` pathing, or TC format: `feature-spec-reference.md`, `spec-system-reference.md`, `spec-principles.md`
39- Behavior/public-contract changes or spec-test-code sync: `workflow-spec-test-code-cycle-reference.md` plus the spec docs above
40- Derived spec indexes/ERDs/reimplementation guides: `spec-system-reference.md` and source Feature Specs under `docs/specs/`
41- Integration test implementation/review: `integration-test-reference.md`
42- E2E test implementation/review: `e2e-test-reference.md`
43- Code review/audit work: `code-review-rules.md` plus domain docs above based on changed files
44
45Do not read all docs blindly. Start from `docs-index-reference.md`, then open only relevant files for the task.
46
47<!-- CODEX:PROJECT-REFERENCE-LOADING:END -->
48
49<!-- PROMPT-ENHANCE:STEP-TASK-ANCHOR:START -->
50
51> **[BLOCKING]** Execute skill steps in declared order. NEVER skip, reorder, or merge steps without explicit user approval.
52> **[BLOCKING]** Before each step or sub-skill call, update task tracking: set `in_progress` when step starts, set `completed` when step ends.
53> **[BLOCKING]** Every completed/skipped step MUST include brief evidence or explicit skip reason.
54> **[BLOCKING]** If Task tools are unavailable, create and maintain an equivalent step-by-step plan tracker with the same status transitions.
55
56<!-- PROMPT-ENHANCE:STEP-TASK-ANCHOR:END -->
57
58## Quick Summary
59
60**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.
61
62**Summary:**
63
64- **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.
65- **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 by asking the user directly (≤4 per call) → Step 5 Document answers → offer implement/refine/skip.
66- **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.
67- **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.
68- **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 by asking the user directly.
69
70**Workflow:**
71
721. **Detect Plan Type** — Classify plan (bugfix/feature/migration/refactor) to weight question categories
732. **Read Plan** — Parse plan.md + phase files for decisions, assumptions, risks
743. **Extract Topics** — Scan architecture, assumptions, tradeoffs, risks, scope keywords
754. **Generate Questions** — Formulate concrete questions with 2-4 options each
765. **Interview User** — Present questions using configured count range
776. **Document Answers** — Add Validation Summary section to plan.md
78
79**Key Rules:**
80
81- MUST ATTENTION use ask the user directly — NEVER auto-decide on behalf of user — why: the user owns every assumption-laden choice, not the agent
82- Ask ONLY about genuine choices affecting implementation — NEVER about non-decision points — why: noise questions burn the interview budget and erode trust
83- Bugfix plans ALWAYS trigger the Preservation question (keywords: fix, bug, regression, broken, defect) — why: an unverified preserved-correctness invariant is a silent regression
84- 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
85
86## First Principle — Easy to Change
87
88> **The success metric of every coding decision is _future change cost_.**
89> DRY, SRP, abstraction, design patterns, naming, layering, tests — every
90> technique exists to serve one goal: **making the next change cheaper**.
91
92Evaluating code, refactor, test, abstraction, ask:
93**does this make next change cheaper or more expensive?**
94
95- Reject "best practices" raising change cost (premature abstraction,
96 speculative generality, leaky indirection, ceremony without payoff).
97- Name real enemies in findings: **coupling, hidden state, duplicated
98 knowledge, unclear intent, irreversible decisions exposed too early**.
99- Simpler design easy to change beats sophisticated design that isn't.
100
101Apply this lens **before** invoking any specific rule, pattern, or checklist
102below — if a downstream rule raises change cost, this principle wins.
103
104---
105
106## Phase 0: Detect Plan Type
107
108Classify plan type BEFORE generating questions — drives question category weighting:
109
110| Plan Type | Detection | Mandatory Extra Categories |
111| ------------- | ----------------------------------------------------------------- | ------------------------------------- |
112| **Bugfix** | Title/frontmatter: `fix`, `bug`, `regression`, `broken`, `defect` | Preservation (BLOCKING) |
113| **Feature** | New capability, no fix keywords | Architecture, Assumptions, Test Specs |
114| **Migration** | Schema change, EF migration, data move | Risks, Preservation, Scope |
115| **Refactor** | Restructure/clean up, no behavior change | Preservation, Tradeoffs |
116| **Other** | None of above | Architecture, Scope |
117
118**Bugfix detection is BLOCKING** — NEVER skip Preservation question when fix/bug/regression/broken/defect keywords present.
119
120## Plan Resolution
121
1221. `$ARGUMENTS` provided → use that path
1232. Check `## Plan Context` section → use active plan path
1243. No plan found → ask user to specify path or run `$plan` first
125
126## Configuration (from injected context)
127
128Check `## Plan Context` section:
129
130- `mode` — auto/prompt/off behavior
131- `questions` — range like `3-8` (min-max)
132
133Use as hard constraints.
134
135## Workflow
136
137### Step 1: Read Plan Files
138
139Read plan directory:
140
141- `plan.md` — overview + phases list
142- `phase-*.md` — all phase files
143- Flag: decision points, assumptions, risks, tradeoffs
144
145### Step 2: Extract Question Topics
146
147| Category | Keywords |
148| ---------------- | ------------------------------------------------------------------------------------------ |
149| **Architecture** | approach, pattern, design, structure, database, API |
150| **Assumptions** | assume, expect, should, will, must, default |
151| **Tradeoffs** | tradeoff, vs, alternative, option, choice, either/or |
152| **Risks** | risk, might, could fail, dependency, blocker, concern |
153| **Scope** | phase, MVP, future, out of scope, nice to have |
154| **New Tech/Lib** | install, add package, new dependency, npm install, dotnet add, unfamiliar framework names |
155| **Test Specs** | TC-, test case, coverage, TDD, test specification |
156| **Preservation** | auto-trigger on bugfix keywords in title/frontmatter — scan Preservation Inventory section |
157
158### Step 3: Generate Questions
159
160**Format rules:**
161
162- 2-4 concrete options per question
163- Mark recommended with "(Recommended)" suffix
164- "Other" option automatic — do NOT add
165- Surface implicit decisions
166
167**Examples:**
168
169```
170Category: Architecture
171Question: "How should validation results be persisted?"
172Options:
1731. Save to plan.md frontmatter (Recommended) — updates existing plan
1742. Create validation-answers.md — separate answers file
1753. Don't persist — ephemeral validation only
176```
177
178```
179Category: Assumptions
180Question: "Plan assumes API rate limiting not needed. Correct?"
181Options:
1821. Yes, not needed for MVP
1832. No, add basic rate limiting now (Recommended)
1843. Defer to Phase 2
185```
186
187```
188Category: Preservation (MANDATORY when title/frontmatter: fix, bug, regression, broken, defect)
189Question: "List 2-3 inputs where CURRENT code is correct. Will fix change behavior on any?"
190Options (multi-select):
1911. "Current code correct on: {input A}. Fix preserves behavior." (Recommended)
1922. "Current code correct on: {input B}. Fix CHANGES behavior because: {justification}"
1933. "Current code has NO preserved-correctness inputs — every input was broken" (rare; requires confirmation)
1944. "Unsure — need to investigate" (STOP: run $plan preservation analysis)
195```
196
197**Follow-up rules:**
198
199- Option 2 selected → `plan.md` Preservation Inventory MUST cite Preservation TC asserting new behavior is intended
200- Option 4 selected → return BLOCKED status, recommend `$plan` before proceeding
201- Option 3 selected → ask the user directly follow-up: "Confirm: current code has NO preserved invariant? [Yes, every input broken / No, missed some — re-investigate]"
202
203### Step 4: Interview User
204
205Use ask the user directly — NEVER skip or auto-answer.
206
207**Rules:**
208
209- Use question count from `## Plan Context` → `Validation: mode=X, questions=MIN-MAX`
210- Group related questions (max 4 per tool call)
211- Focus: assumptions, risks, tradeoffs, architecture
212- MANDATORY IMPORTANT MUST ATTENTION: if plan introduces new tech/packages, ask: "Plan uses {lib}. Were alternatives evaluated? Confirm choice or research more?"
213
214### Step 5: Document Answers
215
216Add `## Validation Summary` to `plan.md`:
217
218```markdown
219## Validation Summary
220
221**Validated:** {date}
222**Questions asked:** {count}
223
224### Confirmed Decisions
225
226- {decision 1}: {user choice}
227- {decision 2}: {user choice}
228
229### Action Items
230
231- [ ] {changes needed based on answers}
232```
233
234NEVER modify phase files — only document what needs updating.
235
236## Output
237
238After validation:
239
240- Questions asked count
241- Key decisions confirmed
242- Items flagged for plan revision
243- Recommendation: proceed to implementation OR revise plan first
244
245## Next Steps
246
247**MANDATORY IMPORTANT MUST ATTENTION — NO EXCEPTIONS** after completing, use ask the user directly to present:
248
249- **"$feature-implement (Recommended)"** — Begin implementation with validated plan
250- **"$refine"** — If plan needs PBI refinement first
251- **"Skip, continue manually"** — User decides
252
253---
254
255> **[BLOCKING]** MUST ATTENTION use ask the user directly to interview user. Completing without asking ≥1 question = violation.
256
257> **[IMPORTANT]** Use task tracking 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.
258
259> **External Memory:** Complex/lengthy work → write findings + results to `plans/reports/` — prevents context loss.
260
261> **Evidence Gate:** MANDATORY IMPORTANT MUST ATTENTION — every claim requires `file:line` proof or traced evidence with confidence % (>80% act, <80% verify first).
262
263<!-- SYNC:nested-task-creation -->
264
265> **Nested Task Expansion Contract** — For workflow-step invocation, the `[Workflow] ...` row is only a parent container; the child skill still creates visible phase tasks.
266>
267> 1. Call the current task list first. If a matching active parent workflow row exists, set `nested=true` and record `parentTaskId`; otherwise run standalone.
268> 2. Create one task per declared phase before phase work. When nested, prefix subjects `[N.M] $skill-name — phase`.
269> 3. When nested, link the parent with `TaskUpdate(parentTaskId, addBlockedBy: [childIds])`.
270> 4. Orchestrators must pre-expand a child skill's phase list and link the workflow row before invoking that child skill or sub-agent.
271> 5. Mark exactly one child `in_progress` before work and `completed` immediately after evidence is written.
272> 6. Complete the parent only after all child tasks are completed or explicitly cancelled with reason.
273>
274> **Blocked until:** the current task list done, child phases created, parent linked when nested, first child marked `in_progress`.
275
276<!-- /SYNC:nested-task-creation -->
277
278<!-- SYNC:task-tracking-external-report -->
279
280> **Task Tracking & External Report Persistence** — Bootstrap this before execution; then run project-reference doc prefetch before target/source work.
281>
282> 1. Create a small task breakdown before target file reads, grep, edits, or analysis. On context loss, inspect the current task list first.
283> 2. Mark one task `in_progress` before work and `completed` immediately after evidence; never batch transitions.
284> 3. For plan/review work, create `plans/reports/{skill}-{YYMMDD}-{HHmm}-{slug}.md` before first finding.
285> 4. Append findings after each file/section/decision and synthesize from the report file at the end.
286> 5. Final output cites `Full report: plans/reports/{filename}`.
287>
288> **Blocked until:** task breakdown exists, report path declared for plan/review work, first finding persisted before the next finding.
289
290<!-- /SYNC:task-tracking-external-report -->
291
292<!-- SYNC:critical-thinking-mindset -->
293
294> **Critical Thinking Mindset** — Apply critical thinking, sequential thinking. Every claim needs traced proof, confidence >80% to act.
295> **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.
296
297<!-- /SYNC:critical-thinking-mindset -->
298
299<!-- SYNC:sequential-thinking-protocol -->
300
301> **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.
302>
303> **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.
304>
305> **Format (explicit mode — visible thought trail):**
306>
307> 1. `Thought N/M: [aspect]` — one aspect per thought, state assumptions/uncertainty
308> 2. `Thought N/M [REVISION of Thought K]: ...` — when prior reasoning invalidated; state Original / Why revised / Impact
309> 3. `Thought N/M [BRANCH A from Thought K]: ...` — explore alternative; converge with decision rationale
310> 4. `Thought N/M [HYPOTHESIS]: ...` then `[VERIFICATION]: ...` — test before acting
311> 5. `Thought N/N [FINAL]` — only when verified, all critical aspects addressed, confidence >80%
312>
313> **Mandatory closers:** Confidence % stated · Assumptions listed · Open questions surfaced · Next action concrete.
314>
315> **Stop conditions:** confidence <80% on any critical decision → escalate by asking the user directly · ≥3 revisions on same thought → re-frame the problem · branch count >3 → split into sub-task.
316>
317> **Implicit mode:** apply methodology internally without visible markers when adding markers would clutter the response (routine work where reasoning aids accuracy).
318>
319> **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).
320
321<!-- /SYNC:sequential-thinking-protocol -->
322
323<!-- SYNC:project-reference-docs-guide -->
324
325> **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.
326>
327> 1. Identify scope: file types, domain area, and operation.
328> 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).
329> 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`.
330> 4. Read every required doc, then before target work state: `Reference docs read: ... | Not applicable: ...`.
331>
332> **Ready when:** scope evaluated, `docs/project-config.json` consulted, required docs checked/read or setup route completed, `lessons.md` confirmed, citation emitted.
333
334<!-- /SYNC:project-reference-docs-guide -->
335
336<!-- SYNC:understand-code-first -->
337
338> **Understand Code First** — HARD-GATE: Do NOT write, plan, or fix until you READ existing code.
339>
340> 1. Search 3+ similar patterns (`grep`/`glob`) — cite `file:line` evidence
341> 2. Read existing files in target area — understand structure, base classes, conventions
342> 3. Run `python .claude/scripts/code_graph trace <file> --direction both --json` when `.code-graph/graph.db` exists
343> 4. Map dependencies via `connections` or `callers_of` — know what depends on your target
344> 5. Write investigation to `.ai/workspace/analysis/` for non-trivial tasks (3+ files)
345> 6. Re-read analysis file before implementing — never work from memory alone. — why: long context drifts from the file; the file is ground truth
346> 7. NEVER invent new patterns when existing ones work — match exactly or document deviation. — why: divergent patterns fragment the codebase and slow every future reader
347>
348> **BLOCKED until:** `- [ ]` Read target files `- [ ]` Grep 3+ patterns `- [ ]` Graph trace (if graph.db exists) `- [ ]` Assumptions verified with evidence
349
350<!-- /SYNC:understand-code-first -->
351
352<!-- SYNC:plan-quality -->
353
354> **Plan Quality** — Every plan phase MUST ATTENTION include test specifications.
355>
356> 1. Add `## Test Specifications` section with TC-{FEATURE}-{NNN} IDs to every phase file
357> 2. Map every functional requirement to ≥1 TC (or explicit `TBD` with rationale)
358> 3. TC IDs follow `TC-{FEATURE}-{NNN}` format — reference by ID, never embed full content
359> 4. Before any new workflow step: call the current task list and re-read the phase file
360> 5. On context compaction: call the current task list FIRST — never create duplicate tasks
361> 6. Verify TC satisfaction per phase before marking complete (evidence must be `file:line`, not TBD)
362>
363> **Mode:** TDD-first → reference existing TCs with `Evidence: TBD`. Implement-first → use TBD → `$spec [mode=tests]` fills after.
364
365<!-- /SYNC:plan-quality -->
366
367<!-- SYNC:cross-service-check -->
368
369> **Cross-Service Check** — Microservices/event-driven: MANDATORY before concluding investigation, plan, spec, or feature doc. Missing downstream consumer = silent regression.
370>
371> | Boundary | Grep terms |
372> | ------------------- | ------------------------------------------------------------------------------- |
373> | Event producers | `Publish`, `Dispatch`, `Send`, `emit`, `EventBus`, `outbox`, `IntegrationEvent` |
374> | Event consumers | `Consumer`, `EventHandler`, `Subscribe`, `@EventListener`, `inbox` |
375> | Sagas/orchestration | `Saga`, `ProcessManager`, `Choreography`, `Workflow`, `Orchestrator` |
376> | Sync service calls | HTTP/gRPC calls to/from other services |
377> | Shared contracts | OpenAPI spec, proto, shared DTO — flag breaking changes |
378> | Data ownership | Other service reads/writes same table/collection → Shared-DB anti-pattern |
379>
380> **Per touchpoint:** owner service · message name · consumers · risk (NONE / ADDITIVE / BREAKING).
381>
382> **BLOCKED until:** Producers scanned · Consumers scanned · Sagas checked · Contracts reviewed · Breaking-change risk flagged
383
384<!-- /SYNC:cross-service-check -->
385
386<!-- SYNC:ai-mistake-prevention -->
387
388> **AI Mistake Prevention** — Failure modes to avoid on every task:
389>
390> **Re-read files after context changes.** Context compaction, resume, or long-running work can make memory stale; verify current files before acting.
391> **Verify generated content against source evidence.** AI hallucinates APIs, names, claims, and document facts. Check the relevant source before documenting or referencing.
392> **Check downstream references before deleting or renaming.** Removing an artifact can stale docs, generated mirrors, configs, and callers; map references first.
393> **Trace the full impact chain after edits.** Changing a definition can miss derived outputs and consumers. Follow the affected chain before declaring done.
394> **Verify ALL affected outputs, not just the first.** One green check is not all green checks; validate every output surface the change can affect.
395> **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.
396> **Surface ambiguity before acting — don't pick silently.** Multiple valid interpretations require an explicit question or stated assumption with risk.
397> **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.
398> **Keep shared guidance role-relevant.** Universal guidance must help every receiving skill or agent; code-specific obligations belong only in code-specific protocols.
399
400<!-- /SYNC:ai-mistake-prevention -->
401
402<!-- SYNC:understand-code-first:reminder -->
403
404**IMPORTANT MUST ATTENTION** search 3+ existing patterns and read code BEFORE any modification. Run graph trace when graph.db exists.
405
406<!-- /SYNC:understand-code-first:reminder -->
407
408<!-- SYNC:evidence-based-reasoning:reminder -->
409
410- **MANDATORY IMPORTANT MUST ATTENTION** cite `file:line` evidence for every claim. Confidence >80% to act, <60% = do NOT recommend.
411 <!-- /SYNC:evidence-based-reasoning:reminder -->
412
413<!-- SYNC:plan-quality:reminder -->
414
415**IMPORTANT MUST ATTENTION** include `## Test Specifications` with TC IDs per phase. Call the current task list before creating new tasks.
416
417<!-- /SYNC:plan-quality:reminder -->
418
419<!-- SYNC:cross-service-check:reminder -->
420
421**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.
422
423<!-- /SYNC:cross-service-check:reminder -->
424
425<!-- SYNC:critical-thinking-mindset:reminder -->
426
427**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.
428
429<!-- /SYNC:critical-thinking-mindset:reminder -->
430
431<!-- SYNC:sequential-thinking-protocol:reminder -->
432
433**MUST ATTENTION** apply sequential-thinking — multi-step Thought N/M, REVISION/BRANCH/HYPOTHESIS markers, confidence % closer; see `$sequential-thinking` skill.
434
435<!-- /SYNC:sequential-thinking-protocol:reminder -->
436
437<!-- SYNC:ai-mistake-prevention:reminder -->
438
439**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.
440
441<!-- /SYNC:ai-mistake-prevention:reminder -->
442
443<!-- SYNC:task-tracking-external-report:reminder -->
444
445- **MANDATORY** Bootstrap task tracking before target work; transition one task at a time.
446- **MANDATORY** Persist plan/review findings to `plans/reports/` incrementally and synthesize from disk.
447
448<!-- /SYNC:task-tracking-external-report:reminder -->
449
450<!-- SYNC:project-reference-docs-guide:reminder -->
451
452- **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: ...`.
453- **MANDATORY** Always include `lessons.md`; project config + conventions override generic framework defaults.
454- **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.
455
456<!-- /SYNC:project-reference-docs-guide:reminder -->
457
458<!-- SYNC:nested-task-creation:reminder -->
459
460- **MANDATORY** Parent workflow rows do not replace child phase tracking; expand phases and link the parent when nested.
461- **MANDATORY** Orchestrators pre-expand child skill phases before invocation; use `[N.M] $skill-name — phase` prefixes and one-`in_progress` discipline.
462
463<!-- /SYNC:nested-task-creation:reminder -->
464
465<!-- PROMPT-ENHANCE:STEP-TASK-CLOSING:START -->
466
467## Prompt-Enhance Closing Anchors
468
469**IMPORTANT MUST ATTENTION** follow declared step order for this skill; NEVER skip, reorder, or merge steps without explicit user approval
470**IMPORTANT MUST ATTENTION** for every step/sub-skill call: set `in_progress` before execution, set `completed` after execution
471**IMPORTANT MUST ATTENTION** every skipped step MUST include explicit reason; every completed step MUST include concise evidence
472**IMPORTANT MUST ATTENTION** if Task tools unavailable, maintain an equivalent step-by-step plan tracker with synchronized statuses
473
474<!-- PROMPT-ENHANCE:STEP-TASK-CLOSING:END -->
475
476<!-- SYNC:project-protocol-overlay -->
477
478> **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`.
479>
480> 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.
481
482<!-- /SYNC:project-protocol-overlay -->
483
484<!-- SYNC:project-protocol-overlay:reminder -->
485
486**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.
487
488<!-- /SYNC:project-protocol-overlay:reminder -->
489
490## Closing Reminders
491
492**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.
493
494**Protocols in force (concise digest of the SYNC/shared blocks this skill carries):**
495
496- **Nested Task Creation:** child skill still creates visible phase tasks; link parent when nested.
497- **Task Tracking & External Report:** bootstrap task breakdown first; persist findings incrementally to `plans/reports/`.
498- **Critical Thinking:** MUST ATTENTION apply critical + sequential thinking; cite proof; confidence >80% to act.
499- **Sequential Thinking:** structured multi-step Thought N/M with REVISION/BRANCH/HYPOTHESIS markers and confidence closer.
500- **Project Reference Docs:** read required project-reference docs before target work; always include `lessons.md`.
501- **Understand Code First:** MUST ATTENTION read code and grep 3+ patterns before any modification.
502- **Plan Quality:** include `## Test Specifications` with TC-{FEATURE}-{NNN} IDs per phase.
503- **Cross-Service Check:** scan producers, consumers, sagas, contracts; flag breaking-change risk.
504- **AI Mistake Prevention:** verify generated content against evidence, trace downstream references, verify all affected outputs, re-read after context loss, surface ambiguity.
505
506**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 by asking the user directly (≤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
507
508**IMPORTANT MUST ATTENTION** validate decisions with the user by asking the user directly — 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
509**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
510**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
511
512- **MANDATORY IMPORTANT MUST ATTENTION** break work into small todo tasks using task tracking BEFORE starting (including a task per file read); call the current task list first on context loss, never duplicate — why: resume existing tasks rather than re-plan after compaction
513- **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
514- **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
515- **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
516- **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
517- **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
518- **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)
519- **MANDATORY IMPORTANT MUST ATTENTION** add a final review task to verify work quality
520
521**Anti-Rationalization:**
522
523| Evasion | Rebuttal |
524| ---------------------------------- | ---------------------------------------------------------------- |
525| "Plan is simple, skip validation" | Simple plans still have implicit decisions. Apply anyway. |
526| "Already know the answers" | Show user responses as proof. No responses = no validation. |
527| "Preservation doesn't apply here" | If title has fix/bug/regression/broken/defect → ALWAYS applies. |
528| "Phase 0 not needed" | Detection drives the Preservation gate. NEVER skip. |
529| "Only ask a few questions" | Use the `questions` range from Plan Context. Never go below min. |
530| "I'll just answer for the user" | ask the user directly is mandatory. Self-answer = no validation. |
531| "New library is obviously fine" | Probe whether alternatives were evaluated before accepting it. |
532| "I'll edit the phase files inline" | NEVER. Add only a `## Validation Summary` to `plan.md`. |
533
534**[TASK-PLANNING]** Before acting, analyze task scope and systematically break it into small todo tasks and sub-tasks using task tracking.
535
536**IMPORTANT MUST ATTENTION** detect plan type (Phase 0) FIRST — bugfix keywords make Preservation BLOCKING.
537**IMPORTANT MUST ATTENTION** validate with the user by asking the user directly — NEVER auto-decide.
538**IMPORTANT MUST ATTENTION** NEVER modify phase files — add only a `## Validation Summary` to `plan.md`.
539
540---
541
542> **Closing reminder — Easy to Change is the success metric.** Every finding,
543> test, refactor, and abstraction must answer one question: _does this make
544> the next change cheaper or more expensive?_ If it doesn't reduce future
545> change cost, reject it. Coupling, hidden state, duplicated knowledge, and
546> unclear intent are the real enemies — call them out by name.
547
548<!-- CODEX:SYNC-PROMPT-PROTOCOLS:START -->
549
550## Hookless Prompt Protocol Mirror (Auto-Synced)
551
552Source: `.claude/.ck.json` + `.claude/skills/shared/sync-inline-versions.md` (`:full` blocks) + `.claude/scripts/lib/hookless-prompt-protocol.cjs`
553
554## [WORKFLOW-EXECUTION-PROTOCOL] [BLOCKING] Workflow Execution Protocol — MANDATORY IMPORTANT MUST CRITICAL. Do not skip for any reason.
555
556**Generic portability boundary:** Reusable skills and protocol text stay project-neutral; project-specific conventions are discovered from docs/project-config.json and docs/project-reference/. Apply shared AI-SDD from `shared/sdd-artifact-contract.md`. Read `docs/project-config.json` and `docs/project-reference/docs-index-reference.md`, then open the project reference docs named there. For spec, test-case, behavior-change, public-contract, or `docs/specs/` work, route through the local spec docs named by the docs index: `feature-spec-reference.md`, `spec-system-reference.md`, `spec-principles.md`, and `workflow-spec-test-c
557
558…(truncated)