oh-my-opencode-slim 0.8.6 → 0.9.0

package/dist/cli/index.js

@@ -13676,6 +13676,7 @@ var CouncilMasterConfigSchema = exports_external.object({
   variant: exports_external.string().optional(),
   prompt: exports_external.string().optional().describe("Optional role/guidance injected into the master synthesis prompt")
 });
+var CouncillorExecutionModeSchema = exports_external.enum(["parallel", "serial"]).default("parallel").describe('Execution mode for councillors. Use "serial" for single-model systems to avoid conflicts. ' + 'Use "parallel" for multi-model systems for faster execution.');
 var CouncilConfigSchema = exports_external.object({
   master: CouncilMasterConfigSchema,
   presets: exports_external.record(exports_external.string(), CouncilPresetSchema),
@@ -13690,7 +13691,8 @@ var CouncilConfigSchema = exports_external.object({
       return unique;
     }
     return val;
-  }).describe("Fallback models for the council master. Tried in order if the primary model fails. " + 'Example: ["anthropic/claude-sonnet-4-6", "openai/gpt-5.4"]')
+  }).describe("Fallback models for the council master. Tried in order if the primary model fails. " + 'Example: ["anthropic/claude-sonnet-4-6", "openai/gpt-5.4"]'),
+  councillor_execution_mode: CouncillorExecutionModeSchema.describe('Execution mode for councillors. "serial" runs them one at a time (required for single-model systems). "parallel" runs them concurrently (default, faster for multi-model systems).')
 });
 // src/config/schema.ts
 var ProviderModelIdSchema = exports_external.string().regex(/^[^/\s]+\/[^\s]+$/, "Expected provider/model format (provider/.../model)");
@@ -13863,7 +13865,7 @@ var RECOMMENDED_SKILLS = [
     name: "simplify",
     repo: "https://github.com/brianlovin/claude-config",
     skillName: "simplify",
-    allowedAgents: ["orchestrator"],
+    allowedAgents: ["oracle"],
     description: "YAGNI code simplification expert"
   },
   {

package/dist/config/council-schema.d.ts

@@ -61,6 +61,15 @@ export declare const CouncilMasterConfigSchema: z.ZodObject<{
     prompt: z.ZodOptional<z.ZodString>;
 }, z.core.$strip>;
 export type CouncilMasterConfig = z.infer<typeof CouncilMasterConfigSchema>;
+/**
+ * Execution mode for councillors.
+ * - parallel: Run all councillors concurrently (default, fastest for multi-model systems)
+ * - serial: Run councillors one at a time (required for single-model systems to avoid conflicts)
+ */
+export declare const CouncillorExecutionModeSchema: z.ZodDefault<z.ZodEnum<{
+    parallel: "parallel";
+    serial: "serial";
+}>>;
 /**
  * Top-level council configuration.
  *
@@ -77,7 +86,8 @@ export type CouncilMasterConfig = z.infer<typeof CouncilMasterConfigSchema>;
  *       }
  *     },
  *     "master_timeout": 300000,
- *     "councillors_timeout": 180000
+ *     "councillors_timeout": 180000,
+ *     "councillor_execution_mode": "serial"
  *   }
  * }
  * ```
@@ -104,8 +114,13 @@ export declare const CouncilConfigSchema: z.ZodObject<{
     councillors_timeout: z.ZodDefault<z.ZodNumber>;
     default_preset: z.ZodDefault<z.ZodString>;
     master_fallback: z.ZodPipe<z.ZodOptional<z.ZodArray<z.ZodString>>, z.ZodTransform<string[] | undefined, string[] | undefined>>;
+    councillor_execution_mode: z.ZodDefault<z.ZodEnum<{
+        parallel: "parallel";
+        serial: "serial";
+    }>>;
 }, z.core.$strip>;
 export type CouncilConfig = z.infer<typeof CouncilConfigSchema>;
+export type CouncillorExecutionMode = z.infer<typeof CouncillorExecutionModeSchema>;
 /**
  * A sensible default council configuration that users can copy into their
  * opencode.jsonc. Provides a 3-councillor preset using common models.

package/dist/config/schema.d.ts

@@ -238,6 +238,10 @@ export declare const PluginConfigSchema: z.ZodObject<{
         councillors_timeout: z.ZodDefault<z.ZodNumber>;
         default_preset: z.ZodDefault<z.ZodString>;
         master_fallback: z.ZodPipe<z.ZodOptional<z.ZodArray<z.ZodString>>, z.ZodTransform<string[] | undefined, string[] | undefined>>;
+        councillor_execution_mode: z.ZodDefault<z.ZodEnum<{
+            parallel: "parallel";
+            serial: "serial";
+        }>>;
     }, z.core.$strip>>;
 }, z.core.$strip>;
 export type PluginConfig = z.infer<typeof PluginConfigSchema>;

package/dist/index.js

@@ -3529,7 +3529,7 @@ var RECOMMENDED_SKILLS = [
     name: "simplify",
     repo: "https://github.com/brianlovin/claude-config",
     skillName: "simplify",
-    allowedAgents: ["orchestrator"],
+    allowedAgents: ["oracle"],
     description: "YAGNI code simplification expert"
   },
   {
@@ -17217,6 +17217,7 @@ var CouncilMasterConfigSchema = exports_external.object({
   variant: exports_external.string().optional(),
   prompt: exports_external.string().optional().describe("Optional role/guidance injected into the master synthesis prompt")
 });
+var CouncillorExecutionModeSchema = exports_external.enum(["parallel", "serial"]).default("parallel").describe('Execution mode for councillors. Use "serial" for single-model systems to avoid conflicts. ' + 'Use "parallel" for multi-model systems for faster execution.');
 var CouncilConfigSchema = exports_external.object({
   master: CouncilMasterConfigSchema,
   presets: exports_external.record(exports_external.string(), CouncilPresetSchema),
@@ -17231,7 +17232,8 @@ var CouncilConfigSchema = exports_external.object({
       return unique;
     }
     return val;
-  }).describe("Fallback models for the council master. Tried in order if the primary model fails. " + 'Example: ["anthropic/claude-sonnet-4-6", "openai/gpt-5.4"]')
+  }).describe("Fallback models for the council master. Tried in order if the primary model fails. " + 'Example: ["anthropic/claude-sonnet-4-6", "openai/gpt-5.4"]'),
+  councillor_execution_mode: CouncillorExecutionModeSchema.describe('Execution mode for councillors. "serial" runs them one at a time (required for single-model systems). "parallel" runs them concurrently (default, faster for multi-model systems).')
 });
 // src/config/loader.ts
 import * as fs from "fs";
@@ -17581,16 +17583,16 @@ You are an AI coding orchestrator that optimizes for quality, speed, cost, and r
 @oracle
 - 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 \u2022 Problems persisting after 2+ fix attempts \u2022 High-risk multi-system refactors \u2022 Costly trade-offs (performance vs maintainability) \u2022 Complex debugging with unclear root cause \u2022 Security/scalability/data integrity decisions \u2022 Genuinely uncertain and cost of wrong choice is high \u2022 When a workflow calls for a **reviewer** subagent
+- Capabilities: Deep architectural reasoning, system-level trade-offs, complex debugging, code review, simplification, maintainability review
+- **Delegate when:** Major architectural decisions with long-term impact \u2022 Problems persisting after 2+ fix attempts \u2022 High-risk multi-system refactors \u2022 Costly trade-offs (performance vs maintainability) \u2022 Complex debugging with unclear root cause \u2022 Security/scalability/data integrity decisions \u2022 Genuinely uncertain and cost of wrong choice is high \u2022 When a workflow calls for a **reviewer** subagent \u2022 Code needs simplification or YAGNI scrutiny
 - **Don't delegate when:** Routine decisions you're confident about \u2022 First bug fix attempt \u2022 Straightforward trade-offs \u2022 Tactical "how" vs strategic "should" \u2022 Time-sensitive good-enough decisions \u2022 Quick research/testing can answer
-- **Rule of thumb:** Need senior architect review? \u2192 @oracle. Need code review? \u2192 @oracle. Just do it and PR? \u2192 yourself.
+- **Rule of thumb:** Need senior architect review? \u2192 @oracle. Need code review or simplification? \u2192 @oracle. Just do it and PR? \u2192 yourself.
 
 @designer
 - Role: UI/UX specialist for intentional, polished experiences
 - Stats: 10x better UI/UX than orchestrator
-- Capabilities: Visual direction, interactions, responsive layouts, design systems with aesthetic intent
-- **Delegate when:** User-facing interfaces needing polish \u2022 Responsive layouts \u2022 UX-critical components (forms, nav, dashboards) \u2022 Visual consistency systems \u2022 Animations/micro-interactions \u2022 Landing/marketing pages \u2022 Refining functional\u2192delightful
+- Capabilities: Visual direction, interactions, responsive layouts, design systems with aesthetic intent, UI/UX review
+- **Delegate when:** User-facing interfaces needing polish \u2022 Responsive layouts \u2022 UX-critical components (forms, nav, dashboards) \u2022 Visual consistency systems \u2022 Animations/micro-interactions \u2022 Landing/marketing pages \u2022 Refining functional\u2192delightful \u2022 Reviewing existing UI/UX quality
 - **Don't delegate when:** Backend/logic with no visual \u2022 Quick prototypes where design doesn't matter yet
 - **Rule of thumb:** Users see it and polish matters? \u2192 @designer. Headless/functional? \u2192 yourself.
 
@@ -17598,9 +17600,9 @@ You are an AI coding orchestrator that optimizes for quality, speed, cost, and r
 - Role: Fast execution specialist for well-defined tasks, which empowers orchestrator with parallel, speedy executions
 - Stats: 2x faster code edits, 1/2 cost of orchestrator, 0.8x quality of orchestrator
 - Tools/Constraints: Execution-focused\u2014no research, no architectural decisions
-- **Delegate when:** For implementation work, think and triage first. If the change is non-trivial or multi-file, hand bounded execution to @fixer
+- **Delegate when:** For implementation work, think and triage first. If the change is non-trivial or multi-file, hand bounded execution to @fixer \u2022 Writing or updating tests \u2022 Tasks that touch test files, fixtures, mocks, or test helpers
 - **Don't delegate when:** Needs discovery/research/decisions \u2022 Single small change (<20 lines, one file) \u2022 Unclear requirements needing iteration \u2022 Explaining to fixer > doing \u2022 Tight integration with your current work \u2022 Sequential dependencies
-- **Rule of thumb:** Explaining > doing? \u2192 yourself. Orchestrator paths selection is vastly improved by Fixer. eg it can reduce overall speed if Orchestrator splits what's usually a single task into multiple subtasks and parallelize it with fixer.
+- **Rule of thumb:** Explaining > doing? \u2192 yourself. Test file modifications and bounded implementation work usually go to @fixer. Orchestrator paths selection is vastly improved by Fixer. eg it can reduce overall speed if Orchestrator splits what's usually a single task into multiple subtasks and parallelize it with fixer.
 
 @council
 - Role: Multi-LLM consensus engine for high-confidence answers
@@ -17648,9 +17650,17 @@ Balance: respect dependencies, avoid parallelizing what must be sequential.
 4. Integrate results
 5. Adjust if needed
 
+### Validation routing
+- Validation is a workflow stage owned by the Orchestrator, not a separate specialist
+- Route UI/UX validation and review to @designer
+- Route code review, simplification, maintainability review, and YAGNI checks to @oracle
+- Route test writing, test updates, and changes touching test files to @fixer
+- If a request spans multiple lanes, delegate only the lanes that add clear value
+
 ## 6. Verify
 - Run \`lsp_diagnostics\` for errors
-- Suggest \`simplify\` skill when applicable
+- Use validation routing when applicable instead of doing all review work yourself
+- If test files are involved, prefer @fixer for bounded test changes and @oracle only for test strategy or quality review
 - Confirm specialists completed successfully
 - Verify solution meets requirements
 
@@ -17901,9 +17911,9 @@ function createCouncillorAgent(model, customPrompt, customAppendPrompt) {
 }
 
 // src/agents/designer.ts
-var DESIGNER_PROMPT = `You are a Designer - a frontend UI/UX specialist who creates intentional, polished experiences.
+var DESIGNER_PROMPT = `You are a Designer - a frontend UI/UX specialist who creates and reviews intentional, polished experiences.
 
-**Role**: Craft cohesive UI/UX that balances visual impact with usability.
+**Role**: Craft and review cohesive UI/UX that balances visual impact with usability.
 
 ## Design Principles
 
@@ -17949,6 +17959,11 @@ var DESIGNER_PROMPT = `You are a Designer - a frontend UI/UX specialist who crea
 - Leverage component libraries where available
 - Prioritize visual excellence\u2014code perfection comes second
 
+## Review Responsibilities
+- Review existing UI for usability, responsiveness, visual consistency, and polish when asked
+- Call out concrete UX issues and improvements, not just abstract design advice
+- When validating, focus on what users actually see and feel
+
 ## Output Quality
 You're capable of extraordinary creative work. Commit fully to distinctive visions and show what's possible when breaking conventions thoughtfully.`;
 function createDesignerAgent(model, customPrompt, customAppendPrompt) {
@@ -17962,7 +17977,7 @@ ${customAppendPrompt}`;
   }
   return {
     name: "designer",
-    description: "UI/UX design and implementation. Use for styling, responsive design, component architecture and visual polish.",
+    description: "UI/UX design, review, and implementation. Use for styling, responsive design, component architecture and visual polish.",
     config: {
       model,
       temperature: 0.7,
@@ -18030,6 +18045,7 @@ var FIXER_PROMPT = `You are Fixer - a fast, focused implementation specialist.
 - Use the research context (file paths, documentation, patterns) provided
 - Read files before using edit/write tools and gather exact content before making changes
 - Be fast and direct - no research, no delegation, No multi-step research/planning; minimal execution sequence ok
+- Write or update tests when requested, especially for bounded tasks involving test files, fixtures, mocks, or test helpers
 - Run tests/lsp_diagnostics when relevant or requested (otherwise note as skipped with reason)
 - Report completion with summary of changes
 
@@ -18039,6 +18055,7 @@ var FIXER_PROMPT = `You are Fixer - a fast, focused implementation specialist.
 - No multi-step research/planning; minimal execution sequence ok
 - If context is insufficient: use grep/glob/lsp_diagnostics directly \u2014 do not delegate
 - Only ask for missing inputs you truly cannot retrieve yourself
+- Do not act as the primary reviewer; implement requested changes and surface obvious issues briefly
 
 **Output Format**:
 <summary>
@@ -18123,14 +18140,15 @@ ${customAppendPrompt}`;
 }
 
 // src/agents/oracle.ts
-var ORACLE_PROMPT = `You are Oracle - a strategic technical advisor.
+var ORACLE_PROMPT = `You are Oracle - a strategic technical advisor and code reviewer.
 
-**Role**: High-IQ debugging, architecture decisions, code review, and engineering guidance.
+**Role**: High-IQ debugging, architecture decisions, code review, simplification, and engineering guidance.
 
 **Capabilities**:
 - Analyze complex codebases and identify root causes
 - Propose architectural solutions with tradeoffs
-- Review code for correctness, performance, and maintainability
+- Review code for correctness, performance, maintainability, and unnecessary complexity
+- Enforce YAGNI and suggest simpler designs when abstractions are not pulling their weight
 - Guide debugging when standard approaches fail
 
 **Behavior**:
@@ -18138,6 +18156,7 @@ var ORACLE_PROMPT = `You are Oracle - a strategic technical advisor.
 - Provide actionable recommendations
 - Explain reasoning briefly
 - Acknowledge uncertainty when present
+- Prefer simpler designs unless complexity clearly earns its keep
 
 **Constraints**:
 - READ-ONLY: You advise, you don't implement
@@ -18154,7 +18173,7 @@ ${customAppendPrompt}`;
   }
   return {
     name: "oracle",
-    description: "Strategic technical advisor. Use for architecture decisions, complex debugging, code review, and engineering guidance.",
+    description: "Strategic technical advisor. Use for architecture decisions, complex debugging, code review, simplification, and engineering guidance.",
     config: {
       model,
       temperature: 0.1,
@@ -19294,6 +19313,7 @@ class CouncilManager {
     }
     const councillorsTimeout = councilConfig.councillors_timeout ?? 180000;
     const masterTimeout = councilConfig.master_timeout ?? 300000;
+    const executionMode = councilConfig.councillor_execution_mode ?? "parallel";
     const councillorCount = Object.keys(preset.councillors).length;
     log(`[council-manager] Starting council with preset "${resolvedPreset}"`, {
       councillors: Object.keys(preset.councillors)
@@ -19303,7 +19323,7 @@ class CouncilManager {
         error: err instanceof Error ? err.message : String(err)
       });
     });
-    const councillorResults = await this.runCouncillors(prompt, preset.councillors, parentSessionId, councillorsTimeout);
+    const councillorResults = await this.runCouncillors(prompt, preset.councillors, parentSessionId, councillorsTimeout, executionMode);
     const completedCount = councillorResults.filter((r) => r.status === "completed").length;
     log(`[council-manager] Councillors completed: ${completedCount}/${councillorResults.length}`);
     if (completedCount === 0) {
@@ -19404,59 +19424,96 @@ ${bestResult.result}` : undefined,
       }
     }
   }
-  async runCouncillors(prompt, councillors, parentSessionId, timeout) {
+  async runCouncillors(prompt, councillors, parentSessionId, timeout, executionMode = "parallel") {
     const entries = Object.entries(councillors);
-    const promises = entries.map(([name, config2], index) => (async () => {
-      if (index > 0) {
-        await new Promise((r) => setTimeout(r, index * COUNCILLOR_STAGGER_MS));
-      }
-      const modelLabel = shortModelLabel(config2.model);
-      try {
-        const result = await this.runAgentSession({
-          parentSessionId,
-          title: `Council ${name} (${modelLabel})`,
-          agent: "councillor",
-          model: config2.model,
-          promptText: formatCouncillorPrompt(prompt, config2.prompt),
-          variant: config2.variant,
-          timeout,
-          includeReasoning: false
-        });
-        return {
-          name,
-          model: config2.model,
-          status: "completed",
-          result
-        };
-      } catch (error48) {
-        const msg = error48 instanceof Error ? error48.message : String(error48);
-        return {
-          name,
-          model: config2.model,
-          status: msg.includes("timed out") ? "timed_out" : "failed",
-          error: `Councillor "${name}": ${msg}`
-        };
+    const results = [];
+    if (executionMode === "serial") {
+      for (const [name, config2] of entries) {
+        const modelLabel = shortModelLabel(config2.model);
+        log(`[council-manager] Running councillor "${name}" (${modelLabel}) serially`);
+        try {
+          const result = await this.runAgentSession({
+            parentSessionId,
+            title: `Council ${name} (${modelLabel})`,
+            agent: "councillor",
+            model: config2.model,
+            promptText: formatCouncillorPrompt(prompt, config2.prompt),
+            variant: config2.variant,
+            timeout,
+            includeReasoning: false
+          });
+          results.push({
+            name,
+            model: config2.model,
+            status: "completed",
+            result
+          });
+        } catch (error48) {
+          const msg = error48 instanceof Error ? error48.message : String(error48);
+          results.push({
+            name,
+            model: config2.model,
+            status: msg.includes("timed out") ? "timed_out" : "failed",
+            error: `Councillor "${name}": ${msg}`
+          });
+        }
       }
-    })());
-    const settled = await Promise.allSettled(promises);
-    return settled.map((result, index) => {
-      const [name, cfg] = entries[index];
-      if (result.status === "fulfilled") {
-        return {
-          name,
-          model: cfg.model,
-          status: result.value.status,
-          result: result.value.result,
-          error: result.value.error
-        };
+    } else {
+      const promises = entries.map(([name, config2], index) => (async () => {
+        if (index > 0) {
+          await new Promise((r) => setTimeout(r, index * COUNCILLOR_STAGGER_MS));
+        }
+        const modelLabel = shortModelLabel(config2.model);
+        try {
+          const result = await this.runAgentSession({
+            parentSessionId,
+            title: `Council ${name} (${modelLabel})`,
+            agent: "councillor",
+            model: config2.model,
+            promptText: formatCouncillorPrompt(prompt, config2.prompt),
+            variant: config2.variant,
+            timeout,
+            includeReasoning: false
+          });
+          return {
+            name,
+            model: config2.model,
+            status: "completed",
+            result
+          };
+        } catch (error48) {
+          const msg = error48 instanceof Error ? error48.message : String(error48);
+          return {
+            name,
+            model: config2.model,
+            status: msg.includes("timed out") ? "timed_out" : "failed",
+            error: `Councillor "${name}": ${msg}`
+          };
+        }
+      })());
+      const settled = await Promise.allSettled(promises);
+      for (let index = 0;index < settled.length; index++) {
+        const result = settled[index];
+        const [name, cfg] = entries[index];
+        if (result.status === "fulfilled") {
+          results.push({
+            name,
+            model: cfg.model,
+            status: result.value.status,
+            result: result.value.result,
+            error: result.value.error
+          });
+        } else {
+          results.push({
+            name,
+            model: cfg.model,
+            status: "failed",
+            error: result.reason instanceof Error ? result.reason.message : String(result.reason)
+          });
+        }
       }
-      return {
-        name,
-        model: cfg.model,
-        status: "failed",
-        error: result.reason instanceof Error ? result.reason.message : String(result.reason)
-      };
-    });
+    }
+    return results;
   }
   async runMaster(prompt, councillorResults, councilConfig, parentSessionId, timeout, presetMasterOverride) {
     const masterConfig = councilConfig.master;

package/oh-my-opencode-slim.schema.json

@@ -505,6 +505,15 @@
             "type": "string",
             "pattern": "^[^/\\s]+\\/[^\\s]+$"
           }
+        },
+        "councillor_execution_mode": {
+          "default": "parallel",
+          "description": "Execution mode for councillors. \"serial\" runs them one at a time (required for single-model systems). \"parallel\" runs them concurrently (default, faster for multi-model systems).",
+          "type": "string",
+          "enum": [
+            "parallel",
+            "serial"
+          ]
         }
       },
       "required": [

package/package.json

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