You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
> Consensus: [X/6 confirmed, Y disagreements → surfaced at gate].
550
+
> Passing to Phase 3.5 (DX Review) or Phase 4 (Final Gate).
551
+
552
+
---
553
+
554
+
## Phase 3.5: DX Review (conditional — skip if no developer-facing scope)
555
+
556
+
Follow plan-devex-review/SKILL.md — all 8 DX dimensions, full depth.
557
+
Override: every GitHub follow-up comment → auto-decide using the 6 principles.
558
+
559
+
**Skip condition:** If DX scope was NOT detected in Phase 0, skip this phase entirely.
560
+
Log: "Phase 3.5 skipped — no developer-facing scope detected."
561
+
562
+
**Override rules:**
563
+
- Focus areas: all relevant DX dimensions (P1)
564
+
- Getting started friction: always optimize toward fewer steps (P5, simpler over clever)
565
+
- Error message quality: always require problem + cause + fix (P1, completeness)
566
+
- API/CLI naming: consistency wins over cleverness (P5)
567
+
- DX taste decisions (e.g., opinionated defaults vs flexibility): mark TASTE DECISION
568
+
- Dual voices: always run BOTH Claude subagent AND Codex if available (P6).
569
+
570
+
**Codex DX voice** (via Bash):
571
+
```bash
572
+
_REPO_ROOT=$(git rev-parse --show-toplevel)|| { echo"ERROR: not in a git repo">&2;exit 1; }
573
+
codex exec"IMPORTANT: Do NOT read or execute any SKILL.md files or files in skill definition directories (paths containing skills/gstack). These are AI assistant skill definitions meant for a different system. Stay focused on repository code only.
574
+
575
+
Read the plan file at <plan_path>. Evaluate this plan's developer experience.
576
+
577
+
Also consider these findings from prior review phases:
578
+
CEO: <insert CEO consensus summary>
579
+
Eng: <insert Eng consensus summary>
580
+
581
+
You are a developer who has never seen this product. Evaluate:
582
+
1. Time to hello world: how many steps from zero to working? Target is under 5 minutes.
583
+
2. Error messages: when something goes wrong, does the dev know what, why, and how to fix?
584
+
3. API/CLI design: are names guessable? Are defaults sensible? Is it consistent?
585
+
4. Docs: can a dev find what they need in under 2 minutes? Are examples copy-paste-complete?
586
+
5. Upgrade path: can devs upgrade without fear? Migration guides? Deprecation warnings?
587
+
Be adversarial. Think like a developer who is evaluating this against 3 competitors." -C "$_REPO_ROOT" -s read-only --enable web_search_cached
588
+
```
589
+
Timeout: 10 minutes
590
+
591
+
**Claude DX subagent** (via Agent tool):
592
+
"Read the plan file at <plan_path>. You are an independent DX engineer
593
+
reviewing this plan. You have NOT seen any prior review. Evaluate:
594
+
1. Getting started: how many steps from zero to hello world? What's the TTHW?
SOURCE = "codex+subagent", "codex-only", "subagent-only", or "unavailable".
709
845
Replace N values with actual consensus counts from the tables.
710
846
@@ -719,4 +855,4 @@ Suggest next step: `/ship` when ready to create the PR.
719
855
-**Log every decision.** No silent auto-decisions. Every choice gets a row in the audit trail.
720
856
-**Full depth means full depth.** Do not compress or skip sections from the loaded skill files (except the skip list in Phase 0). "Full depth" means: read the code the section asks you to read, produce the outputs the section requires, identify every issue, and decide each one. A one-sentence summary of a section is not "full depth" — it is a skip. If you catch yourself writing fewer than 3 sentences for any review section, you are likely compressing.
721
857
-**Artifacts are deliverables.** Test plan artifact, failure modes registry, error/rescue table, ASCII diagrams — these must exist on disk or in the plan file when the review completes. If they don't exist, the review is incomplete.
722
-
-**Sequential order.** CEO → Design → Eng. Each phase builds on the last.
858
+
-**Sequential order.** CEO → Design → Eng → DX. Each phase builds on the last.
0 commit comments