From 25f93a3b6499a2a99dc9312c9186933cd1b26c31 Mon Sep 17 00:00:00 2001 From: Alex Verkhovsky Date: Sun, 4 Jan 2026 20:59:58 -0800 Subject: [PATCH] refactor(code-review): simplify workflow.md --- .../4-implementation/code-review/workflow.md | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/src/modules/bmm/workflows/4-implementation/code-review/workflow.md b/src/modules/bmm/workflows/4-implementation/code-review/workflow.md index 46c3bdeb..afc4149d 100644 --- a/src/modules/bmm/workflows/4-implementation/code-review/workflow.md +++ b/src/modules/bmm/workflows/4-implementation/code-review/workflow.md @@ -1,23 +1,14 @@ --- name: code-review -description: 'Perform an ADVERSARIAL Senior Developer code review with dual-phase analysis: context-aware story validation plus context-independent adversarial diff review. Finds 3-10 specific problems in every story. NEVER accepts "looks good" - must find minimum issues and can auto-fix with user approval.' +description: 'Code review for dev-story output. Audits acceptance criteria against implementation, performs adversarial diff review, can auto-fix with approval. A different LLM than the implementer is recommended.' --- # Code Review Workflow -**Goal:** Execute a comprehensive two-phase code review that validates story claims AND performs context-independent adversarial analysis. - -**Your Role:** You are an elite senior developer performing adversarial review. Challenge everything. Verify claims against reality. Find what's wrong or missing. - ---- - -## WORKFLOW ARCHITECTURE - -This uses **step-file architecture** for focused execution: +## WORKFLOW ARCHITECTURE: STEP FILES - Each step loads fresh to combat "lost in the middle" - State persists via variables: `{story_path}`, `{story_key}`, `{context_aware_findings}`, `{asymmetric_findings}` -- Dual-phase review: context-aware (step 3) + adversarial asymmetric (step 4) ---