[Documentation] Use when generating, smart-merging or refactoring the root AI context (CLAUDE.md, AGENTS.md) and its Claude/Codex projections. Config file only: project-config.
The English summary is being prepared. Showing the original description.
[Quality] Use when a workflow step or the user asks for an agent quality harness with project guidance and automated/review feedback.
The English summary is being prepared. Showing the original description.
See how to installReview the instructions the agent will receive before you install.
<!-- PROMPT-ENHANCE:STEP-TASK-ANCHOR:END -->[BLOCKING] Execute skill steps in declared order. NEVER skip, reorder, or merge steps without explicit user approval. Inside a workflow run,
/start-workflow→ Workflow-step mode bounds this rule. [BLOCKING] Before each step or sub-skill call, update todo tracking: setin_progresswhen step starts, setcompletedwhen 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.
Goal: Wire every feedforward guide and feedback sensor into the greenfield project so all later AI coding agents operate with maximum guidance and self-correct against quality gates BEFORE human review — raising first-attempt quality and catching defects at the earliest, cheapest stage.
Summary:
N/A for non-applicable tiers.ask user question tool./linter-setup supplies computational sensors; this skill never installs them. Require intent-protecting evidence selected by profile, risk, tooling and budget; mutation/property sensors are optional, line coverage diagnostic; append inventory after every phase and keep it living.Main steps (run in order — each BLOCKS the next):
/linter-setup (linter config + pre-commit hook + CI gate); detect existing inventory and enhance it, never skip it.architecture --mode=design / tech-stack reports; write stack-profile.md; use ask user question tool for undetectable fields.docs/architecture/* notes, and pattern catalog; confirm via ask user question tool./linter-setup outputs and list config paths; invoke it if any are missing./why-review pre-impl · /code-quality-review pre-commit · /domain-analysis --mode=review post-impl · /production-readiness-review + /security-audit pre-release · /scan-codebase-health recurring · /integration-test --mode=review feature-area TC audit BOTH pre-release AND recurring, catching orphaned Section-8 TCs and uncovered behavior); record under ## Review Gates.test-strategy.md; NEVER gate on line %.harness-inventory.md with all sensors and gaps; present it via ask user question tool.ask user question tool to choose /feature-implement (recommended), /why-review, or skip.Produces:
/linter-setup linters, formatters, pre-commit hooks, and CI gatestmp/harness/harness-inventory.mdWhen invoked: After /scaffold + /linter-setup in greenfield workflow; scaffolding must be complete.
Does NOT do: Install linters or configure formatters; /linter-setup owns that work.
Check 1 — /linter-setup prerequisite (BLOCK if missing): Before phases, verify it completed by checking for a root linter config (e.g., .eslintrc, pyproject.toml, .editorconfig), pre-commit hook config (e.g., .husky/, .pre-commit-config.yaml), and CI quality gate definition. If any is missing → ask user question tool: "/linter-setup appears incomplete. Computational feedback sensors must be in place before harness setup. Run /linter-setup first, then return here?" BLOCK Phases A–E until verification passes.
Check 2 — Existing harness inventory: Check tmp/harness/harness-inventory.md. If found → ask user question tool: "Harness inventory already exists — re-run to enhance existing harness, or skip?" Existing CLAUDE.md/AGENTS.md are feedforward guides to enhance, NEVER skip signals.
Read, in order: plan.md frontmatter → architecture --mode=design report → tech-stack-comparison report. Extract:
Write detection result to tmp/harness/stack-profile.md.
If any field is undetectable → ask user question tool before proceeding.
Read .claude/skills/shared/protocols/context-engineering-principles.md when creating or changing persistent instruction files; apply its Instruction-file audit before finalizing those outputs, including later review-gate additions. Preserve generated ownership and existing confirmation gates.
For each guide type, check existence; create it or enhance an existing guide:
/architecture --mode=design, e.g., Clean Architecture, CQRS, Repository), "Anti-Patterns" (stack-specific), "Naming Conventions" (language-idiomatic), and "Module Boundaries" (allowed imports and dependency direction)./domain-analysis --mode=review; before commits → /code-quality-review.docs/architecture/ with bounded-contexts.md (boundaries/ownership), dependency-rules.md (allowed layer imports), and naming-conventions.md (project-specific file/class/function names).docs/architecture/pattern-catalog.md, document each /architecture --mode=design choice with DO/DON'T examples, and anchor examples to actual project files once scaffolding produces them.read <path> when <situation> trigger — a guide nothing routes to is never read. Put generated root-context changes through /ai-context-refresh, not a hand-edit of a generated section; run /prompt-enhance on each hand-owned guide that changed.Present created/updated guides via ask user question tool: "Feedforward guides above will be created/enhanced. Confirm or adjust?"
Confirm /linter-setup outputs by checking the root linter config (e.g., .eslintrc, pyproject.toml, .editorconfig), pre-commit hook config (e.g., .husky/, .pre-commit-config.yaml), and CI quality gate. If any is missing, invoke /linter-setup before continuing. Output confirmation with file paths.
Configure AI review skills by lifecycle stage. Present via ask user question tool: "Which inferential sensors should be mandatory vs optional for this repository?"
/why-review validates design rationale before the implementation approach is committed./code-quality-review./domain-analysis --mode=review when domain entity files are in the changeset./production-readiness-review for reliability/operations and /security-audit for production security./scan-codebase-health quarterly or on CI schedule. Also wire /integration-test --mode=review's Missing Integration Test / Spec-Coverage Gate feature-area TC audit (Phase 3 addendum), which catches orphaned Section-8 TCs and uncovered behavior, both pre-release alongside the two mandatory gates and on the same recurring cadence; a diff-scoped run cannot catch a Section-8 TC whose test regressed outside the current changeset.Add the agreed sensor configuration to CLAUDE.md under "## Review Gates".
Define the project behaviour harness:
ask user question tool: "Feature documentation format?" Options: feature-spec (8-section tech-free), TDD specs only, lightweight ADRs. Establish the business spec root (default docs/specs; a specRoots.business.path entry in docs/project-config.json overrides the path) or an equivalent spec home.test-strategy.md)Copy the architecture --mode=design contract into test-strategy.md; resolve every tier from verified project/configuration evidence before choosing tools:
| Tier | Applicability + evidence | Owner | Runner/framework + config | Test root | Data/fixture policy | Full command | Focused/partial command | Zero-match behavior | CI gate | Simple Windows/macOS/Linux entry point | Repeat proof |
|---|---|---|---|---|---|---|---|---|---|---|---|
| Unit | APPLICABLE / N/A — {evidence} | {owner} | {runner/config} | {root} | {fixtures/factories} | {command} | {filter} | {non-zero behavior} | {gate} | {command, or .cmd + .sh pair} | {result or planned owner} |
| Integration/System | APPLICABLE / N/A — {evidence} | {owner} | {runner/config} | {root} | {public-path + additive data} | {command} | {filter} | {non-zero behavior} | {gate} | {command, or .cmd + .sh pair} | {two no-reset runs} |
| E2E | APPLICABLE / N/A — {evidence} | {owner} | {configured browser/config} | {root} | {reachable journey data} | {command} | {filter} | {non-zero behavior} | {gate} | {command, or .cmd + .sh pair} | {result or evidence-backed N/A} |
APPLICABLE requires runner/framework/configuration/root/command evidence. If no E2E framework, configuration, and command are verified, record N/A — {config/source evidence}; do not infer a browser stack from generic examples. Full/focused commands must be copy-ready, report exact counts/exit status, and fail invalid or zero-match selection.
For each applicable persistent-state tier, record run/test identity generation and unique business-data suffix, supported public paths, realistic valid data, count-before-create idempotent/restart-safe reference setup, keyed additive accumulation with integrity checks, mutable-root/parallel-worker isolation, immutable shared data, realistic actor pacing, observable arrange barriers, and exact results. Require two consecutive no-reset full runs for an integration tier and one fresh green full run for any other tier; until executed, mark proof planned — {owner}, not PASS. Keep property/invariant, mutation, change, and behavior coverage meaningful; line coverage remains diagnostic only.
Test-strength sensors (NOT a line-coverage gate):
/integration-test --mode=review Gate 7. This is the right notion of "coverage"; the line-% is not.Document the agreed strategy in docs/architecture/test-strategy.md.
Write tmp/harness/harness-inventory.md:
# Harness Inventory
Generated: {date}
Stack: {detected stack from Phase A}
## Testability & Verification Contract
Copy the resolved `test-strategy.md` matrix into this inventory and keep the status current:
**Status:** `PASS | PARTIAL | BLOCKED`
| Tier | Applicability + evidence | Owner | Runner/config/root | Full | Focused/partial | Zero-match behavior | CI / simple Windows/macOS/Linux entry point | Identity/data/isolation/fidelity policy | Repeat proof |
| --- | --- | --- | --- | --- | --- | --- | --- | --- | --- |
| Unit | `APPLICABLE` / `N/A — {evidence}` | {owner} | {runner/config/root} | `{command}` | `{filter}` | `{non-zero behavior}` | {gate / command} | {policy reference} | {result/status} |
| Integration/System | `APPLICABLE` / `N/A — {evidence}` | {owner} | {runner/config/root} | `{command}` | `{filter}` | `{non-zero behavior}` | {gate / command} | {policy reference} | `{two no-reset runs}` |
| E2E | `APPLICABLE` / `N/A — {evidence}` | {owner} | {runner/config/root} | `{command}` | `{filter}` | `{non-zero behavior}` | {gate / command} | {policy reference} | `{result or evidence-backed N/A}` |
Missing/placeholder evidence is an open gap, not a PASS. The inventory must preserve the strategy's unique identity, additive-data, isolation, realistic-fidelity, and two-run proof fields; E2E N/A remains evidence-backed.
## Feedforward Guides
| Type | File/Skill | Purpose |
| ------------- | ------------------------------------ | ------------------------------- |
| Inferential | CLAUDE.md §Architecture Patterns | Shapes AI architectural choices |
| Inferential | CLAUDE.md §Anti-Patterns | Prevents known bad patterns |
| Inferential | docs/architecture/pattern-catalog.md | DO/DON'T examples per pattern |
| Computational | .editorconfig | Cross-IDE consistency |
## Feedback Sensors — Computational
| Stage | Tool/Hook | What it catches |
| ---------- | ------------------ | ---------------------------------------------- |
| Pre-commit | {linter} | Style violations, common errors |
| Pre-commit | {formatter} | Code formatting drift |
| CI | {type-checker} | Type errors |
| CI | {static-analyzer} | Security, complexity, dead code |
| CI | {selected strength sensor, or N/A with alternate evidence} | Protected intent; threshold only if justified |
| CI | {coverage-tool} | Untested areas (DIAGNOSTIC only — never gated) |
## Feedback Sensors — Inferential
| Stage | Skill/Agent | What it catches |
| ------------------- | ----------------------- | ------------------------------ |
| Pre-implementation | /why-review | Design rationale gaps |
| Pre-commit | /code-quality-review | Convention drift, logic errors |
| Post-implementation | /domain-analysis --mode=review | Domain model quality |
| Pre-release | /production-readiness-review | Operational readiness |
| Pre-release | /security-audit | Security vulnerabilities |
| Pre-release + Recurring | /integration-test --mode=review (feature-area TC audit) | Orphaned Section-8 TCs, uncovered changed behavior |
## Open Gaps
| Area | Reason | Risk |
| ------------------------ | -------- | -------------- |
| {area not yet harnessed} | {reason} | {LOW/MED/HIGH} |
Present inventory to user for review via ask user question tool.
Inside a workflow: when this skill runs as a step of an active workflow, skip the prompts in this section and return the result — the workflow's next step decides what runs (
/start-workflow→ Step Execution Protocol, Workflow-step mode).
ask user question tool:
<!-- PROTOCOL-GUIDES:START -->[IMPORTANT] Use
TaskCreateto break ALL work into small tasks BEFORE starting. This prevents context loss from long files. Keep task depth proportional to the work.
Protocol guides — Apply each protocol only when its read-when condition holds. If a protocol's text is not in your context, read its file below before you act on it.
engineering-foundation-gate — Seven engineering-foundation dimensions judged by project profile; creating or reviewing how a project is built, run, tested or checked → .claude/skills/shared/protocols/engineering-foundation-gate.mdharness-setup — Agent quality harness: feedforward guides and feedback sensors; setting up an agent quality harness → .claude/skills/shared/protocols/harness-setup.mdmeasured-capacity-engineering — Model demand, reduce measured work safely and prove capacity before scaling; planning, building, testing or reviewing hot paths, caches or capacity → .claude/skills/shared/protocols/measured-capacity-engineering.mdtest-architecture-execution-contract — Testability as an architecture condition: required test types and execution modes; setting up or reviewing a test architecture → .claude/skills/shared/protocols/test-architecture-execution-contract.mdIMPORTANT MUST ATTENTION follow declared step order for this skill; NEVER skip, reorder, or merge steps without explicit user approval; inside a workflow run, /start-workflow → Workflow-step mode bounds this
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
MUST ATTENTION Each assertion-bearing test names the behavior or technical contract it protects and asserts an outcome it owns. Use the project's configured/native test format — Given/When/Then is one valid format, never a framework-wide requirement. When specArtifacts is valid, link configured owner/case/variant identity and intent/contracts evidence; when absent, name the guarded business intent or technical contract. A malformed declared profile BLOCKS without fallback. Broad test-format migration → assign an owner and next step, never rewrite cases outside scope.
MUST ATTENTION Before implementation record evidence-backed applicability for the test types and modes the task/project contract requires: copy-ready full and focused commands where available, zero-match behavior, a useful platform-appropriate entry point, supported execution modes and environments, state-isolation requirements, exact results, and repeat evidence where persistent state makes it relevant. Exercise claimed modes; report a missing required capability as ENVIRONMENT-BLOCKED. Never invent production targets or impose a test format. Browser/UI E2E uses the configured runner's waits or project helper for observable readiness and outcomes; apply action pacing only where the project contract specifies it. Reuse the project's evidenced test organization — require a POM, base class, or component taxonomy only when the project actually selects it.
IMPORTANT MUST ATTENTION evidence-backed lifecycle/scale/criticality/repo/runtime profile; unknowns take lower tiers. Judge all 7 outcomes: F1 reproducible build/run/test · F2 exercise supported/required modes; dual modes only when warranted · F3 applicable local/CI/production-shaped test portability · F4 test-strength proof; no universal mutation tool · F5 measured performance at warranted scale/risk · F6 build/change scalability at meaningful module boundaries · F7 stack/profile-fit mechanical checks. Evidence-backed N/A-by-profile is valid; prevent over-engineering. Creation blocks warranted omissions; brownfield advises without score changes, with smallest next steps. Catalog: .claude/docs/engineering-foundation-catalog.md; update first, re-run inject_engineering_foundation_gate.py.
MUST ATTENTION capacity work: model demand/SLO and distinguish sessions from RPS/in-flight work; disclose load model and offered vs achieved demand; reduce measured work at a safe owner; preserve cache authorization/freshness/bounds; prove cold-state, overload recovery and justified headroom before scaling. Static review returns a verification plan, not invented throughput. Retain the hosting skill's scores, gates and authority.
<!-- /SYNC:measured-capacity-engineering:reminder -->IMPORTANT MUST ATTENTION Goal: Wire every feedforward guide and feedback sensor into the greenfield project so all later AI coding agents operate with maximum guidance and self-correct against quality gates BEFORE human review — raising first-attempt quality and catching defects at the earliest, cheapest stage. IMPORTANT MUST ATTENTION Testability contract: resolve evidence-backed Unit/Integration/System/E2E and warranted Performance/Scale rows, copy-ready full/focused commands, zero-match failures, owner/root/data, CI/simple Windows/macOS/Linux entry, supported host/container modes and environment reach, unique run identity, isolation, and repeat proof before claiming setup, review, or test completion.
IMPORTANT MUST ATTENTION Protocols in force (concise digest of the SYNC/shared blocks this skill carries):
IMPORTANT MUST ATTENTION Main steps (in order — each BLOCKS the next): Guards (verify /linter-setup) → A Stack Detection (stack-profile.md) → B Feedforward Guides (CLAUDE.md patterns/anti-patterns/naming/boundaries + skill-activation rules + pattern catalog) → C Computational Sensors (confirm linter/hook/CI) → D Inferential Sensors (wire /why-review, /code-quality-review, /domain-analysis --mode=review, /production-readiness-review, /security-audit, /scan-codebase-health, /integration-test --mode=review missing-test/spec-coverage gate to gates) → E Behaviour Harness (spec format + profile-fit test tiers + intent evidence + test-strategy.md) → F Inventory Report (harness-inventory.md) → Next Steps. NEVER skip or reorder — why: each phase consumes the prior phase's verified output.
IMPORTANT MUST ATTENTION BLOCK on the /linter-setup prerequisite first — ALWAYS verify computational sensors (linter config, pre-commit hook, CI gate) exist before any phase runs — why: keep quality left; cheapest gates must precede inferential ones, and this skill never installs them itself
IMPORTANT MUST ATTENTION NEVER auto-decide feedforward-guide or sensor content — present the draft and confirm via ask user question tool — why: harness conventions bind every future agent; silent choices propagate to all later sessions
IMPORTANT MUST ATTENTION write tmp/harness/harness-inventory.md incrementally (append after each phase) — NEVER hold findings in memory — why: long context drifts and silently drops findings
IMPORTANT MUST ATTENTION walk phases A→F as a hard barrier sequence — NEVER skip or reorder; each phase BLOCKS the next until its guard passes — why: a later phase consumes the prior phase's verified output
IMPORTANT MUST ATTENTION require profile-fit intent-protecting evidence for the behaviour harness — NEVER fail a build on a line-coverage % — why: lines execute without asserting intent, so coverage % is a diagnostic only, never a quality gate
IMPORTANT MUST ATTENTION wire /integration-test --mode=review's feature-area-wide TC audit as a Phase D sensor BOTH pre-release AND on the SAME recurring cadence as /scan-codebase-health — never pre-release only — why: a diff-scoped-only run cannot see a §8 TC whose covering test regressed outside the current change set; only a periodic feature-area sweep catches it
IMPORTANT MUST ATTENTION research tool choices per detected stack — NEVER hardcode a linter/formatter/mutation tool — present top 2-3 options, enforce strictest defaults, loosen only with explicit approval — why: harnessability depends on the actual stack, not a default
IMPORTANT MUST ATTENTION harness inventory is a LIVING document — update it when new sensors are added later — why: a stale inventory misrepresents the active feedback loop
IMPORTANT MUST ATTENTION grep 3+ existing guides/sensors before authoring a new one; verify fit (same stack, gate stage, lifecycle) before copying a nearby pattern — why: closest example ≠ matching preconditions
IMPORTANT MUST ATTENTION cite file:line / config-path evidence for every detected sensor and stack fact (confidence >80% to act, <60% DO NOT recommend) — NEVER speculate a tool exists; grep the config to confirm — why: a hallucinated sensor leaves a real gap unguarded
IMPORTANT MUST ATTENTION bootstrap task tracking before phases — TaskCreate one todo per phase, mark in_progress/completed as you go; on context loss TaskList first — why: resume work, never duplicate phases
Anti-Rationalization:
| Evasion | Rebuttal |
|---|---|
| "Linter probably set up — skip the prereq check" | Grep for the config files. No file:line proof = BLOCK Phase A/B/C/D/E until verified. |
| "I'll pick the obvious linter myself" | NEVER auto-decide — present top 2-3 via ask user question tool; the user owns binding conventions. |
| "High line coverage means tests are strong" | Coverage is a diagnostic, not a gate. Choose profile-fit intent evidence; lines run without asserting. |
| "Inventory's small, I'll hold it in memory" | Append per phase to the inventory file — context loss silently drops findings. |
| "CLAUDE.md exists, harness already done" | CLAUDE.md is a feedforward guide to ENHANCE, never a signal to skip phases. |
IMPORTANT MUST ATTENTION BLOCK on /linter-setup before any phase · NEVER auto-decide harness content (ask user question tool-gate) · require profile-fit intent evidence, NEVER a universal mutation tool or line-coverage % gate.
No reviews yet. Tell others what you thought.
No login needed. Share how it worked for you, ask a question, or note where it did not run.
No comments yet. Be the first to write one.
[Documentation] Use when generating, smart-merging or refactoring the root AI context (CLAUDE.md, AGENTS.md) and its Claude/Codex projections. Config file only: project-config.
The English summary is being prepared. Showing the original description.
[Code Quality] Use when a workflow step or the user asks for AI code or plan review: LLM calls, prompts, agents, RAG, evals and guardrails. --mode={code|plan}; --report-only is read-only.
The English summary is being prepared. Showing the original description.
[Architecture] Use when a workflow step or the user asks for architecture design, compliance review, scalability grading or whole-project audit via --mode={design|review|scalability|full}.
The English summary is being prepared. Showing the original description.
[Content] Use when a workflow step or the user asks for product/feature ideation with ranked opportunities and validation tests. --mode={roadmap|scope}.
The English summary is being prepared. Showing the original description.
[Content] Use when a workflow step or the user asks for business viability: Business Model Canvas, financials, risks and go-to-market recommendation.
The English summary is being prepared. Showing the original description.
[Code Quality] Use when a workflow step or the user asks for a focused code-change review: staged, unstaged, branch diff or simple pull request. --fix-loop fixes and re-reviews. Complex pull request: workflow-review-changes; rationale: why-review.
The English summary is being prepared. Showing the original description.