Browse Source

Code review orchestrator

Alvin Unreal 2 weeks ago
parent
commit
979d02c89a
2 changed files with 6 additions and 7 deletions
  1. 1 1
      package.json
  2. 5 6
      src/agents/orchestrator.ts

+ 1 - 1
package.json

@@ -1,6 +1,6 @@
 {
   "name": "oh-my-opencode-slim",
-  "version": "0.8.4",
+  "version": "0.8.5",
   "description": "Lightweight agent orchestration plugin for OpenCode - a slimmed-down fork of oh-my-opencode",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

+ 5 - 6
src/agents/orchestrator.ts

@@ -45,13 +45,12 @@ You are an AI coding orchestrator that optimizes for quality, speed, cost, and r
 - **Rule of thumb:** "How does this library work?" → @librarian. "How does programming work?" → yourself.
 
 @oracle
-- Role: Strategic advisor for high-stakes decisions and persistent problems
-- Stats: 10x better decisions maker, problem solver, investigator than orchestrator, 0.8x speed of orchestrator, same cost.
-- Capabilities: Deep architectural reasoning, system-level trade-offs, complex debugging
-- Tools/Constraints: Slow, expensive, high-quality—use sparingly when thoroughness beats speed
-- **Delegate when:** Major architectural decisions with long-term impact • Problems persisting after 2+ fix attempts • High-risk multi-system refactors • Costly trade-offs (performance vs maintainability) • Complex debugging with unclear root cause • Security/scalability/data integrity decisions • Genuinely uncertain and cost of wrong choice is high
+- Role: Strategic advisor for high-stakes decisions and persistent problems, code reviewer
+- Stats: 5x better decision maker, problem solver, investigator than orchestrator, 0.8x speed of orchestrator, same cost.
+- Capabilities: Deep architectural reasoning, system-level trade-offs, complex debugging, code review
+- **Delegate when:** Major architectural decisions with long-term impact • Problems persisting after 2+ fix attempts • High-risk multi-system refactors • Costly trade-offs (performance vs maintainability) • Complex debugging with unclear root cause • Security/scalability/data integrity decisions • Genuinely uncertain and cost of wrong choice is high • When a workflow calls for a **reviewer** subagent
 - **Don't delegate when:** Routine decisions you're confident about • First bug fix attempt • Straightforward trade-offs • Tactical "how" vs strategic "should" • Time-sensitive good-enough decisions • Quick research/testing can answer
-- **Rule of thumb:** Need senior architect review? → @oracle. Just do it and PR? → yourself.
+- **Rule of thumb:** Need senior architect review? → @oracle. Need code review? → @oracle. Just do it and PR? → yourself.
 
 @designer
 - Role: UI/UX specialist for intentional, polished experiences