From 004824b242c86fca20ebdbfb8ea74f5200a27b8b Mon Sep 17 00:00:00 2001 From: Sam Xie Date: Wed, 25 Feb 2026 06:55:55 -0800 Subject: [PATCH] Fix ordering number and orphaned fences --- .../commands/workflows/review.md | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/plugins/compound-engineering/commands/workflows/review.md b/plugins/compound-engineering/commands/workflows/review.md index d0ba78f..89c43ff 100644 --- a/plugins/compound-engineering/commands/workflows/review.md +++ b/plugins/compound-engineering/commands/workflows/review.md @@ -106,7 +106,7 @@ These agents are run ONLY when the PR matches specific criteria. Check the PR fi -### 4. Ultra-Thinking Deep Dive Phases +### 2. Ultra-Thinking Deep Dive Phases For each phase below, spend maximum cognitive effort. Think step by step. Consider all angles. Question assumptions. And bring all reviews in a synthesis to the user. @@ -114,7 +114,7 @@ These agents are run ONLY when the PR matches specific criteria. Check the PR fi Complete system context map with component interactions -#### Phase 3: Stakeholder Perspective Analysis +#### Phase 1: Stakeholder Perspective Analysis ULTRA-THINK: Put yourself in each stakeholder's shoes. What matters to them? What are their pain points? @@ -154,7 +154,7 @@ Complete system context map with component interactions - How does this affect time-to-market? - What's the total cost of ownership? -#### Phase 4: Scenario Exploration +#### Phase 2: Scenario Exploration ULTRA-THINK: Explore edge cases and failure scenarios. What could go wrong? How does the system behave under stress? @@ -171,7 +171,7 @@ Complete system context map with component interactions - [ ] **Data Corruption**: Partial writes, inconsistency - [ ] **Cascading Failures**: Downstream service issues -### 6. Multi-Angle Review Perspectives +### 3. Multi-Angle Review Perspectives #### Technical Excellence Angle @@ -437,9 +437,7 @@ After creating all todo files, present comprehensive summary: - Optimization opportunities - Documentation updates -``` - -### 7. End-to-End Testing (Optional) +### 6. End-to-End Testing (Optional) @@ -525,4 +523,3 @@ The subagent will: ### Important: P1 Findings Block Merge Any **🔴 P1 (CRITICAL)** findings must be addressed before merging the PR. Present these prominently and ensure they're resolved before accepting the PR. -```