feat(TASK-001): add rigor/surfaces declaration layer to planning skills
design-session: Quality bar round in Step 2 (rigor/surfaces proposal, per-surface confirm/drop, other:<label> lane, autonomous fallback), rigor+surfaces frontmatter and Quality bar section in the Step 3 PLAN template, poc-dodge anti-pattern. task-tracking: optional per-task surfaces key (inherits the plan's list); rigor stays run-level only. plan-gatekeeping: manifest category 11 (Quality bar) with FRICTION/BLOCKING severity guidance.
This commit is contained in:
@@ -31,6 +31,18 @@ Produce a structured proposal (iterate with the user when interactive — load t
|
||||
- **Open questions** — ONLY questions the codebase cannot answer (business rules, priority calls). If none, say "No open questions."
|
||||
- **Task breakdown** — see below.
|
||||
|
||||
## Quality bar round (closes Step 2)
|
||||
|
||||
The last round of Step 2 sets the plan's quality bar. It is grilling-compatible — run it as numbered questions, each carrying a recommended answer, like any other frontier round:
|
||||
|
||||
1. **Propose `rigor`** — one of `poc | prototype | production` (default `production`). Infer the recommended value from the design doc's own language: "spike"/"demo" → `poc`; "iterate"/"internal" → `prototype`; otherwise `production`. Rigor calibrates which review-finding severities BLOCK downstream work: 🔴-critical findings block at EVERY rigor; at `poc`, suggestion/nitpick-level (🟢/💡) convention findings may be dropped from reports entirely. Anything a lower rigor defers is tracked as a follow-up — never silently dropped.
|
||||
2. **Propose `surfaces`** — zero or more of the closed enum: `rest-api`, `cli`, `library`, `worker`, `iac`, `db-migration`, `frontend`, `ci-cd` (`grpc`/`graphql` are aliases for `rest-api`), plus the escape hatch `other:<label>` for anything outside it. Infer from the approach: an HTTP handler → `rest-api`, a schema change → `db-migration`, and so on.
|
||||
3. **Per-surface confirm/drop** — for each accepted surface, present the headline best practices its reviewers will enforce and let the user confirm or drop each one. Every drop demands a one-line reason and lands in the plan's `## Quality bar` section — a dropped practice without a recorded reason WILL be re-litigated by a reviewer.
|
||||
|
||||
For an `other:<label>` surface no reviewer checklist exists, so the lane is: a librarian lookup distills an authoritative best-practice checklist for the label; the user confirms or drops each item; accepted items become task acceptance criteria where possible, otherwise they live under `## Quality bar → Long-tail criteria`. (The architect drives the lookup; this skill documents the shape the results take in the plan.)
|
||||
|
||||
Autonomous runs (no user to grill): take the inferred values, drop nothing, and note "quality bar inferred, not user-confirmed" in the plan.
|
||||
|
||||
## Task breakdown rules
|
||||
|
||||
| Rule | Why |
|
||||
@@ -51,6 +63,8 @@ Write `PLAN-<slug>.md` (kebab-case slug from the topic; verify no collision) to
|
||||
slug: <slug>
|
||||
status: draft # draft | active | implemented
|
||||
created: YYYY-MM-DD
|
||||
rigor: production # poc | prototype | production; omitted = production
|
||||
surfaces: [] # from the closed enum and/or other:<label>; omitted = []
|
||||
---
|
||||
|
||||
# <Title>
|
||||
@@ -60,6 +74,7 @@ created: YYYY-MM-DD
|
||||
## Approach
|
||||
## Alternatives considered
|
||||
## Constraints and risks
|
||||
## Quality bar
|
||||
## Open questions
|
||||
## Task breakdown
|
||||
|
||||
@@ -67,6 +82,8 @@ created: YYYY-MM-DD
|
||||
|---|------|------|-----------|-------|
|
||||
```
|
||||
|
||||
`## Quality bar` records the quality-bar round in human-readable form: the rigor line with its one-line reason, the surfaces line, the dropped-practices list (each entry with its one-line reason and the date it was decided), and the long-tail criteria block (`none`, or the distilled checklist for each `other:<label>` surface).
|
||||
|
||||
The plan is the implementers' entire context. Write for the "sealed container" standard: every question an implementer will hit is either answered inline or delegated via a pointer to the exact file/doc that answers it (where infra code goes, what DB tech, which layout to mirror, exact test commands). Paste short code snippets for load-bearing patterns — a path alone forces re-exploration; a stale claim fails the executor mid-implementation.
|
||||
|
||||
## Anti-patterns
|
||||
@@ -77,3 +94,4 @@ The plan is the implementers' entire context. Write for the "sealed container" s
|
||||
- Acceptance criteria describing implementation ("uses a for loop") instead of behavior.
|
||||
- Open questions the code could have answered — grep first, ask last.
|
||||
- Unrecorded decisions — every settled fork carries its reason.
|
||||
- Declaring `rigor: poc` to dodge review findings the user never agreed to drop.
|
||||
|
||||
Reference in New Issue
Block a user