plan reviews: replace context-pressure triage with the full-completion rule (s04 F1; manual merge of s05 M2)

plan-ceo-review, plan-design-review and plan-devex-review ranked sections
'under context pressure' ending in 'Everything else', which licensed dropping
sections their own anti-skip rules require. plan-eng-review already replaced
the same block with 'Complete every required stage ... do not preemptively
warn'. All three now use that rule and name what may never be shortened
(design: Step 0, Step 0.5 mockups unless the user opted out, Passes 1-7;
devex: Step 0 with persona/empathy/benchmark/magical moment/TTHW, Passes 1-8).

s04 F1 noted the same block in the s05 files; the design copy is a manual
merge of s05 M2 (drops 'non-negotiable', which contradicted the user's
explicit 'skip mockups' opt-out). No test pins the removed text.
expected red until the regeneration commit
This commit is contained in:
garrytan committed 2026-10-03 00:14:27 +00:00
1 parent d355df24c7
commit 166ce10245
3 files changed
+15 -12

No files matched your search

+5 -3
View File
@@ -88,9 +88,11 @@ Review only. Do not change code or implement.
* Plan partial deploys, rollbacks and feature flags.
* Add and maintain ASCII comments for complex state, pipelines, requests, mixins and test setup.
## Priority Hierarchy Under Context Pressure
Step 0 > System audit > Error/rescue map > Test diagram > Failure modes > Opinionated recommendations > Everything else.
Never skip Step 0, system audit, error/rescue map or failure modes.
## Priority hierarchy
Complete every required stage, decision gate and output. Shorten only optional
commentary, never Step 0, the system audit, Review Sections 1–10, the error/rescue
map, the test diagram, failure modes or required decision/report content. The
system handles context limits; do not preemptively warn.
{{ASIDE_RESEARCH}}
+5 -3
View File
@@ -128,10 +128,12 @@ When reviewing a plan, empathy as simulation runs automatically. When rating, pr
{{UX_PRINCIPLES}}
## Priority Hierarchy Under Context Pressure
## Priority hierarchy
Step 0 > Step 0.5 (mockups — generate by default) > Interaction State Coverage > AI Slop Risk > Information Architecture > User Journey > everything else.
Never skip Step 0 or mockup generation (when the designer is available). Mockups before review passes is non-negotiable. Text descriptions of UI designs are not a substitute for showing what it looks like.
Complete every required stage, decision gate and output. Shorten only optional
commentary, never Step 0, Step 0.5 mockups (when the designer is available and
the user hasn't opted out), Passes 1–7 or required decision/report content.
The system handles context limits; do not preemptively warn.
## PRE-REVIEW SYSTEM AUDIT (before Step 0)
+5 -6
View File
@@ -52,13 +52,12 @@ Keep the reviewed project cwd: read skills by absolute path and run any `cd` in
{{DX_FRAMEWORK}}
## Priority Hierarchy Under Context Pressure
## Priority hierarchy
Step 0 > Developer Persona > Empathy Narrative > Competitive Benchmark >
Magical Moment Design > TTHW Assessment > Error quality > Getting started >
API/CLI ergonomics > Everything else.
Never skip Step 0, the persona interrogation, or the empathy narrative.
Complete every required stage, decision gate and output. Shorten only optional
commentary, never Step 0 (persona, empathy narrative, competitive benchmark,
magical moment, TTHW), Passes 1–8 or required decision/report content.
The system handles context limits; do not preemptively warn.
### Decision gate