opencode-conductor-cdd-plugin 1.0.0-beta.19 → 1.0.0-beta.20

package/README.md

@@ -92,18 +92,31 @@ We highly recommend pinning the `@cdd` agent to a "flash" model for optimal perf
 
 ### oh-my-opencode-slim Config
 **File:** `~/.config/opencode/oh-my-opencode-slim.json`
+
+**⚠️ Important**: For synergy to activate with oh-my-opencode-slim, you **must** configure a `cdd` agent in the config file. The presence of the file alone is not sufficient.
+
 ```json
 {
   "agents": {
     "cdd": {
       "model": "anthropic/claude-3-5-sonnet"
+    },
+    "designer": {
+      "model": "google/gemini-flash-1.5"
+    },
+    "explorer": {
+      "model": "google/gemini-flash-1.5"
     }
   }
 }
 ```
 
 **Framework Detection Priority:**
-When multiple synergy frameworks are detected, Conductor CDD prioritizes oh-my-opencode-slim over oh-my-opencode. If neither framework config is found, Conductor operates in manual mode.
+When multiple synergy frameworks are detected, Conductor CDD prioritizes oh-my-opencode-slim over oh-my-opencode. However, synergy will **only activate** if the selected framework has a `cdd` agent explicitly configured:
+
+1. **Check slim**: If `~/.config/opencode/oh-my-opencode-slim.json` exists **and** has `agents.cdd` → use slim (highest priority)
+2. **Check OMO**: If `~/.config/opencode/oh-my-opencode.json` exists **and** has `agents.cdd` → use OMO (fallback)
+3. **No synergy**: If neither config has a `cdd` agent → Conductor operates in manual mode
 
 **Agent Mappings for oh-my-opencode-slim:**
 Conductor CDD automatically maps generic agent names to slim-specific agent names:
@@ -140,9 +153,12 @@ When using **OhMyOpenCode**, `@cdd` acts as your Technical Lead. While **Sisyphu
 ### oh-my-opencode-slim Integration
 When using **oh-my-opencode-slim**, `@cdd` integrates with the lightweight agent framework, automatically mapping to slim-specific agents like `explorer`, `designer`, and `librarian` for specialized tasks.
 
+**Requirements**: You **must** configure a `cdd` agent in your `oh-my-opencode-slim.json` for synergy to activate. Simply having the file present is not sufficient.
+
 ### Key Features
-- **Automatic Detection**: Conductor CDD automatically detects which framework you're using (if any)
-- **Priority Resolution**: When both frameworks are present, oh-my-opencode-slim takes priority
+- **Automatic Detection**: Conductor CDD automatically detects which framework you're using based on the presence of a `cdd` agent in the config
+- **Strict CDD Requirement**: Synergy only activates if the framework config explicitly includes a `cdd` agent (file presence alone is insufficient)
+- **Priority Resolution**: When both frameworks have a `cdd` agent configured, oh-my-opencode-slim takes priority
 - **Agent Mapping**: Generic agent names are automatically mapped to framework-specific names
 - **Graceful Fallback**: If a requested agent is unavailable or disabled, tasks fall back to `@cdd`
 - **Loop Protection**: Built-in safeguards ensure no conflicts with framework continuation enforcers during interactive Q&A

package/dist/utils/configDetection.d.ts

@@ -1,10 +1,22 @@
 export type SynergyFramework = 'none' | 'oh-my-opencode' | 'oh-my-opencode-slim';
+/**
+ * Result of CDD configuration detection across multiple config files.
+ * Checks ~/.config/opencode/{opencode.json, oh-my-opencode.json, oh-my-opencode-slim.json}
+ */
 export interface ConfigDetectionResult {
+    /** Whether CDD agent is configured in opencode.json */
     hasCDDInOpenCode: boolean;
+    /** Whether CDD agent is configured in oh-my-opencode.json */
     hasCDDInOMO: boolean;
+    /** Whether CDD agent is configured in oh-my-opencode-slim.json */
+    hasCDDInSlim: boolean;
+    /** Whether synergy mode should be activated (true if any synergy framework detected) */
     synergyActive: boolean;
+    /** The CDD model string extracted from configs (priority: slim > OMO > opencode) */
     cddModel?: string;
+    /** Which synergy framework to use (priority: slim > OMO > none) */
     synergyFramework: SynergyFramework;
+    /** Available agents from slim config (filtered by disabled_agents) */
     slimAgents?: string[];
 }
 export declare function detectCDDConfig(): ConfigDetectionResult;

package/dist/utils/configDetection.js

@@ -8,6 +8,7 @@ export function detectCDDConfig() {
     const slimJsonPath = join(opencodeConfigDir, "oh-my-opencode-slim.json");
     let hasCDDInOpenCode = false;
     let hasCDDInOMO = false;
+    let hasCDDInSlim = false;
     let cddModel;
     let synergyFramework = 'none';
     let slimAgents;
@@ -17,8 +18,16 @@ export function detectCDDConfig() {
             const config = JSON.parse(readFileSync(slimJsonPath, "utf-8"));
             // Check if config is not empty and has actual content
             if (config && Object.keys(config).length > 0) {
-                synergyFramework = 'oh-my-opencode-slim';
-                // Extract available agents (filter out disabled ones)
+                // Check for CDD agent in slim config (strict detection)
+                if (config.agents && config.agents.cdd) {
+                    hasCDDInSlim = true;
+                    synergyFramework = 'oh-my-opencode-slim';
+                    // Extract model from slim config (priority over OMO and opencode.json)
+                    if (config.agents.cdd.model) {
+                        cddModel = config.agents.cdd.model;
+                    }
+                }
+                // Extract available agents (filter out disabled ones) regardless of CDD presence
                 const coreAgents = ['explorer', 'librarian', 'oracle', 'designer'];
                 const disabledAgents = new Set(config.disabled_agents ?? []);
                 slimAgents = coreAgents.filter(agent => !disabledAgents.has(agent));
@@ -28,15 +37,19 @@ export function detectCDDConfig() {
             // Silently fail on parse errors
         }
     }
-    // Check oh-my-opencode.json (only if slim is not active)
-    if (synergyFramework === 'none' && existsSync(omoJsonPath)) {
+    // Check oh-my-opencode.json (only if slim doesn't have CDD)
+    // NOTE: We still check OMO to set hasCDDInOMO flag even if slim has priority
+    if (existsSync(omoJsonPath)) {
         try {
             const config = JSON.parse(readFileSync(omoJsonPath, "utf-8"));
             if (config.agents && config.agents.cdd) {
                 hasCDDInOMO = true;
-                synergyFramework = 'oh-my-opencode';
-                // Extract model from oh-my-opencode.json
-                if (config.agents.cdd.model) {
+                // Only activate OMO synergy if slim doesn't have CDD (slim takes priority)
+                if (synergyFramework === 'none') {
+                    synergyFramework = 'oh-my-opencode';
+                }
+                // Extract model from oh-my-opencode.json (only if not already set by slim)
+                if (!cddModel && config.agents.cdd.model) {
                     cddModel = config.agents.cdd.model;
                 }
             }
@@ -45,13 +58,13 @@ export function detectCDDConfig() {
             // Silently fail on parse errors
         }
     }
-    // Check opencode.json (fallback if model not found in OMO)
+    // Check opencode.json (fallback if model not found in slim or OMO)
     if (existsSync(opencodeJsonPath)) {
         try {
             const config = JSON.parse(readFileSync(opencodeJsonPath, "utf-8"));
             if (config.agent && config.agent.cdd) {
                 hasCDDInOpenCode = true;
-                // Only use this model if we didn't find one in oh-my-opencode.json
+                // Only use this model if we didn't find one in slim or oh-my-opencode.json
                 if (!cddModel && config.agent.cdd.model) {
                     cddModel = config.agent.cdd.model;
                 }
@@ -64,6 +77,7 @@ export function detectCDDConfig() {
     return {
         hasCDDInOpenCode,
         hasCDDInOMO,
+        hasCDDInSlim,
         synergyActive: synergyFramework !== 'none',
         cddModel,
         synergyFramework,

package/dist/utils/configDetection.test.js

@@ -119,7 +119,7 @@ describe("configDetection", () => {
     });
     // New tests for oh-my-opencode-slim detection
     describe("oh-my-opencode-slim detection", () => {
-        it("should detect oh-my-opencode-slim when only slim config is present", () => {
+        it("should NOT detect synergy when slim config has no cdd agent", () => {
             vi.mocked(existsSync).mockImplementation((path) => path === slimJsonPath);
             vi.mocked(readFileSync).mockImplementation((path) => {
                 if (path === slimJsonPath) {
@@ -130,11 +130,11 @@ describe("configDetection", () => {
                 return "";
             });
             const result = detectCDDConfig();
-            expect(result.synergyActive).toBe(true);
-            expect(result.synergyFramework).toBe('oh-my-opencode-slim');
+            expect(result.synergyActive).toBe(false);
+            expect(result.synergyFramework).toBe('none');
             expect(result.slimAgents).toEqual(['explorer', 'librarian', 'oracle', 'designer']);
         });
-        it("should prioritize oh-my-opencode-slim over oh-my-opencode when both present", () => {
+        it("should prioritize oh-my-opencode over slim when slim has no cdd agent", () => {
             vi.mocked(existsSync).mockReturnValue(true);
             vi.mocked(readFileSync).mockImplementation((path) => {
                 if (path === slimJsonPath) {
@@ -146,10 +146,11 @@ describe("configDetection", () => {
                 return "";
             });
             const result = detectCDDConfig();
-            expect(result.synergyFramework).toBe('oh-my-opencode-slim');
+            expect(result.synergyFramework).toBe('oh-my-opencode');
             expect(result.synergyActive).toBe(true);
+            expect(result.cddModel).toBe('model-from-omo');
         });
-        it("should filter out disabled agents from slim config", () => {
+        it("should filter out disabled agents from slim config (but require cdd for synergy)", () => {
             vi.mocked(existsSync).mockImplementation((path) => path === slimJsonPath);
             vi.mocked(readFileSync).mockImplementation((path) => {
                 if (path === slimJsonPath) {
@@ -160,7 +161,8 @@ describe("configDetection", () => {
                 return "";
             });
             const result = detectCDDConfig();
-            expect(result.synergyFramework).toBe('oh-my-opencode-slim');
+            expect(result.synergyFramework).toBe('none');
+            expect(result.synergyActive).toBe(false);
             expect(result.slimAgents).toEqual(['explorer', 'librarian']);
         });
         it("should handle empty oh-my-opencode-slim config", () => {
@@ -203,4 +205,199 @@ describe("configDetection", () => {
             expect(result.synergyActive).toBe(false);
         });
     });
+    // New tests for CDD agent detection in oh-my-opencode-slim
+    describe("CDD agent detection in oh-my-opencode-slim", () => {
+        it("should detect cdd agent in slim config and activate synergy", () => {
+            vi.mocked(existsSync).mockImplementation((path) => path === slimJsonPath);
+            vi.mocked(readFileSync).mockImplementation((path) => {
+                if (path === slimJsonPath) {
+                    return JSON.stringify({
+                        agents: {
+                            cdd: { model: "anthropic/claude-3-5-sonnet" },
+                            designer: { model: "google/gemini-3-flash" }
+                        }
+                    });
+                }
+                return "";
+            });
+            const result = detectCDDConfig();
+            expect(result.hasCDDInSlim).toBe(true);
+            expect(result.synergyFramework).toBe('oh-my-opencode-slim');
+            expect(result.synergyActive).toBe(true);
+            expect(result.cddModel).toBe("anthropic/claude-3-5-sonnet");
+        });
+        it("should not activate synergy when slim config exists but no cdd agent", () => {
+            vi.mocked(existsSync).mockImplementation((path) => path === slimJsonPath);
+            vi.mocked(readFileSync).mockImplementation((path) => {
+                if (path === slimJsonPath) {
+                    return JSON.stringify({
+                        agents: {
+                            designer: { model: "google/gemini-3-flash" }
+                        }
+                    });
+                }
+                return "";
+            });
+            const result = detectCDDConfig();
+            expect(result.hasCDDInSlim).toBe(false);
+            expect(result.synergyFramework).toBe('none');
+            expect(result.synergyActive).toBe(false);
+        });
+        it("should not activate synergy when slim config has empty agents object", () => {
+            vi.mocked(existsSync).mockImplementation((path) => path === slimJsonPath);
+            vi.mocked(readFileSync).mockImplementation((path) => {
+                if (path === slimJsonPath) {
+                    return JSON.stringify({ agents: {} });
+                }
+                return "";
+            });
+            const result = detectCDDConfig();
+            expect(result.hasCDDInSlim).toBe(false);
+            expect(result.synergyFramework).toBe('none');
+            expect(result.synergyActive).toBe(false);
+        });
+        it("should handle malformed slim config gracefully with hasCDDInSlim false", () => {
+            vi.mocked(existsSync).mockImplementation((path) => path === slimJsonPath);
+            vi.mocked(readFileSync).mockImplementation((path) => {
+                if (path === slimJsonPath) {
+                    return "invalid json";
+                }
+                return "";
+            });
+            const result = detectCDDConfig();
+            expect(result.hasCDDInSlim).toBe(false);
+            expect(result.synergyFramework).toBe('none');
+            expect(result.synergyActive).toBe(false);
+        });
+        it("should extract cdd model from slim config when present", () => {
+            vi.mocked(existsSync).mockImplementation((path) => path === slimJsonPath);
+            vi.mocked(readFileSync).mockImplementation((path) => {
+                if (path === slimJsonPath) {
+                    return JSON.stringify({
+                        agents: {
+                            cdd: { model: "anthropic/claude-3-5-haiku" }
+                        }
+                    });
+                }
+                return "";
+            });
+            const result = detectCDDConfig();
+            expect(result.cddModel).toBe("anthropic/claude-3-5-haiku");
+            expect(result.hasCDDInSlim).toBe(true);
+        });
+        it("should handle cdd agent without model field in slim", () => {
+            vi.mocked(existsSync).mockImplementation((path) => path === slimJsonPath);
+            vi.mocked(readFileSync).mockImplementation((path) => {
+                if (path === slimJsonPath) {
+                    return JSON.stringify({
+                        agents: {
+                            cdd: {}
+                        }
+                    });
+                }
+                return "";
+            });
+            const result = detectCDDConfig();
+            expect(result.hasCDDInSlim).toBe(true);
+            expect(result.cddModel).toBeUndefined();
+        });
+        it("should prioritize slim model over oh-my-opencode model when both have cdd", () => {
+            vi.mocked(existsSync).mockReturnValue(true);
+            vi.mocked(readFileSync).mockImplementation((path) => {
+                if (path === slimJsonPath) {
+                    return JSON.stringify({
+                        agents: {
+                            cdd: { model: "model-from-slim" }
+                        }
+                    });
+                }
+                if (path === omoJsonPath) {
+                    return JSON.stringify({
+                        agents: {
+                            cdd: { model: "model-from-omo" }
+                        }
+                    });
+                }
+                return "";
+            });
+            const result = detectCDDConfig();
+            expect(result.cddModel).toBe("model-from-slim");
+            expect(result.synergyFramework).toBe('oh-my-opencode-slim');
+            expect(result.hasCDDInSlim).toBe(true);
+            expect(result.hasCDDInOMO).toBe(true);
+        });
+        it("should select oh-my-opencode when only OMO has cdd", () => {
+            vi.mocked(existsSync).mockReturnValue(true);
+            vi.mocked(readFileSync).mockImplementation((path) => {
+                if (path === slimJsonPath) {
+                    return JSON.stringify({
+                        agents: {
+                            designer: { model: "google/gemini-3-flash" }
+                        }
+                    });
+                }
+                if (path === omoJsonPath) {
+                    return JSON.stringify({
+                        agents: {
+                            cdd: { model: "model-from-omo" }
+                        }
+                    });
+                }
+                return "";
+            });
+            const result = detectCDDConfig();
+            expect(result.hasCDDInSlim).toBe(false);
+            expect(result.hasCDDInOMO).toBe(true);
+            expect(result.synergyFramework).toBe('oh-my-opencode');
+        });
+        it("should select slim when only slim has cdd", () => {
+            vi.mocked(existsSync).mockReturnValue(true);
+            vi.mocked(readFileSync).mockImplementation((path) => {
+                if (path === slimJsonPath) {
+                    return JSON.stringify({
+                        agents: {
+                            cdd: { model: "model-from-slim" }
+                        }
+                    });
+                }
+                if (path === omoJsonPath) {
+                    return JSON.stringify({
+                        agents: {
+                            sisyphus: { model: "google/gemini-3-flash" }
+                        }
+                    });
+                }
+                return "";
+            });
+            const result = detectCDDConfig();
+            expect(result.hasCDDInSlim).toBe(true);
+            expect(result.hasCDDInOMO).toBe(false);
+            expect(result.synergyFramework).toBe('oh-my-opencode-slim');
+        });
+        it("should not activate synergy when neither framework has cdd", () => {
+            vi.mocked(existsSync).mockReturnValue(true);
+            vi.mocked(readFileSync).mockImplementation((path) => {
+                if (path === slimJsonPath) {
+                    return JSON.stringify({
+                        agents: {
+                            designer: { model: "google/gemini-3-flash" }
+                        }
+                    });
+                }
+                if (path === omoJsonPath) {
+                    return JSON.stringify({
+                        agents: {
+                            sisyphus: { model: "google/gemini-3-flash" }
+                        }
+                    });
+                }
+                return "";
+            });
+            const result = detectCDDConfig();
+            expect(result.hasCDDInSlim).toBe(false);
+            expect(result.hasCDDInOMO).toBe(false);
+            expect(result.synergyFramework).toBe('none');
+            expect(result.synergyActive).toBe(false);
+        });
+    });
 });

package/dist/utils/synergyState.test.js

@@ -15,6 +15,7 @@ describe("synergyState", () => {
             vi.mocked(detectCDDConfig).mockReturnValue({
                 hasCDDInOpenCode: false,
                 hasCDDInOMO: false,
+                hasCDDInSlim: true,
                 synergyActive: true,
                 synergyFramework: 'oh-my-opencode-slim',
                 slimAgents: ['explorer', 'librarian', 'oracle', 'designer'],
@@ -28,6 +29,7 @@ describe("synergyState", () => {
             vi.mocked(detectCDDConfig).mockReturnValue({
                 hasCDDInOpenCode: false,
                 hasCDDInOMO: false,
+                hasCDDInSlim: true,
                 synergyActive: true,
                 synergyFramework: 'oh-my-opencode-slim',
                 slimAgents: ['explorer', 'librarian'],
@@ -42,6 +44,7 @@ describe("synergyState", () => {
                 .mockReturnValueOnce({
                 hasCDDInOpenCode: false,
                 hasCDDInOMO: false,
+                hasCDDInSlim: true,
                 synergyActive: true,
                 synergyFramework: 'oh-my-opencode-slim',
                 slimAgents: ['explorer'],
@@ -49,6 +52,7 @@ describe("synergyState", () => {
                 .mockReturnValueOnce({
                 hasCDDInOpenCode: false,
                 hasCDDInOMO: true,
+                hasCDDInSlim: false,
                 synergyActive: true,
                 synergyFramework: 'oh-my-opencode',
                 cddModel: 'model-1',
@@ -64,13 +68,15 @@ describe("synergyState", () => {
                 .mockReturnValueOnce({
                 hasCDDInOpenCode: false,
                 hasCDDInOMO: false,
+                hasCDDInSlim: true,
                 synergyActive: true,
                 synergyFramework: 'oh-my-opencode-slim',
                 slimAgents: ['explorer'],
             })
                 .mockReturnValueOnce({
                 hasCDDInOpenCode: false,
-                hasCDDInOMO: false,
+                hasCDDInOMO: true,
+                hasCDDInSlim: false,
                 synergyActive: true,
                 synergyFramework: 'oh-my-opencode',
                 cddModel: 'model-changed',
@@ -85,6 +91,7 @@ describe("synergyState", () => {
             vi.mocked(detectCDDConfig).mockReturnValue({
                 hasCDDInOpenCode: true,
                 hasCDDInOMO: false,
+                hasCDDInSlim: false,
                 synergyActive: false,
                 synergyFramework: 'none',
                 cddModel: 'model-1',
@@ -99,13 +106,15 @@ describe("synergyState", () => {
                 .mockReturnValueOnce({
                 hasCDDInOpenCode: false,
                 hasCDDInOMO: false,
+                hasCDDInSlim: true,
                 synergyActive: true,
                 synergyFramework: 'oh-my-opencode-slim',
                 slimAgents: ['explorer'],
             })
                 .mockReturnValueOnce({
                 hasCDDInOpenCode: false,
-                hasCDDInOMO: false,
+                hasCDDInOMO: true,
+                hasCDDInSlim: false,
                 synergyActive: true,
                 synergyFramework: 'oh-my-opencode',
                 cddModel: 'model-1',
@@ -124,6 +133,7 @@ describe("synergyState", () => {
             vi.mocked(detectCDDConfig).mockReturnValue({
                 hasCDDInOpenCode: false,
                 hasCDDInOMO: false,
+                hasCDDInSlim: false,
                 synergyActive: false,
                 synergyFramework: 'none',
             });
@@ -136,6 +146,7 @@ describe("synergyState", () => {
             vi.mocked(detectCDDConfig).mockReturnValue({
                 hasCDDInOpenCode: false,
                 hasCDDInOMO: false,
+                hasCDDInSlim: true,
                 synergyActive: true,
                 synergyFramework: 'oh-my-opencode-slim',
                 slimAgents: ['explorer', 'oracle'],
@@ -147,6 +158,7 @@ describe("synergyState", () => {
             vi.mocked(detectCDDConfig).mockReturnValue({
                 hasCDDInOpenCode: false,
                 hasCDDInOMO: true,
+                hasCDDInSlim: false,
                 synergyActive: true,
                 synergyFramework: 'oh-my-opencode',
                 cddModel: 'model-1',
@@ -163,13 +175,15 @@ describe("synergyState", () => {
                 .mockReturnValueOnce({
                 hasCDDInOpenCode: false,
                 hasCDDInOMO: false,
+                hasCDDInSlim: true,
                 synergyActive: true,
                 synergyFramework: 'oh-my-opencode-slim',
                 slimAgents: ['explorer'],
             })
                 .mockReturnValueOnce({
                 hasCDDInOpenCode: false,
-                hasCDDInOMO: false,
+                hasCDDInOMO: true,
+                hasCDDInSlim: false,
                 synergyActive: true,
                 synergyFramework: 'oh-my-opencode',
                 cddModel: 'changed',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-conductor-cdd-plugin",
-  "version": "1.0.0-beta.19",
+  "version": "1.0.0-beta.20",
   "description": "Context-Driven Development (CDD) plugin for OpenCode - Transform your AI coding workflow with structured specifications, plans, and implementation tracking",
   "type": "module",
   "repository": {
@@ -53,6 +53,7 @@
     "@semantic-release/npm": "^12.0.1",
     "@semantic-release/release-notes-generator": "^14.0.0",
     "@types/node": "^20.0.0",
+    "@vitest/coverage-v8": "^4.0.17",
     "semantic-release": "^24.2.1",
     "typescript": "^5.0.0",
     "vitest": "^4.0.16"