opencode-pilot 0.26.0 → 0.27.0

package/Formula/opencode-pilot.rb

@@ -1,8 +1,8 @@
 class OpencodePilot < Formula
   desc "Automation daemon for OpenCode - polls GitHub/Linear issues and spawns sessions"
   homepage "https://github.com/athal7/opencode-pilot"
-  url "https://github.com/athal7/opencode-pilot/archive/refs/tags/v0.25.1.tar.gz"
-  sha256 "f8be5b0d08bdd45d5c0155bf40e8930c431fc7a6e6ec77d422bb89471fe91cdc"
+  url "https://github.com/athal7/opencode-pilot/archive/refs/tags/v0.26.0.tar.gz"
+  sha256 "2dc50f9d92e16bcbc3e5956f26bf53d3613c0f3ce756204ea050bf0faba4d9f4"
   license "MIT"
 
   depends_on "node"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-pilot",
-  "version": "0.26.0",
+  "version": "0.27.0",
   "type": "module",
   "main": "plugin/index.js",
   "description": "Automation daemon for OpenCode - polls for work and spawns sessions",

package/service/actions.js

@@ -35,6 +35,62 @@ import os from "os";
 // These are generous upper bounds — if exceeded, the server is genuinely stuck.
 export const HEADER_TIMEOUT_MS = 60_000;
 
+/**
+ * Read model configuration from a target directory's OpenCode config file.
+ *
+ * Looks for `.opencode/opencode.json` (or `.jsonc`) in `dir`. If an `agentName`
+ * is provided, the per-agent model (`.agent.<name>.model`) takes precedence
+ * over the global `.model` value. Returns `{}` when no file is found or the
+ * file cannot be parsed.
+ *
+ * @param {string} dir - Absolute path to the target project directory
+ * @param {string} [agentName] - Optional agent name for per-agent model lookup
+ * @returns {{ model?: string }} Extracted config fields
+ */
+export function readTargetOpencodeConfig(dir, agentName) {
+  const candidates = [
+    path.join(dir, '.opencode', 'opencode.json'),
+    path.join(dir, '.opencode', 'opencode.jsonc'),
+  ];
+
+  let raw;
+  for (const candidate of candidates) {
+    if (existsSync(candidate)) {
+      try {
+        raw = readFileSync(candidate, 'utf-8');
+      } catch {
+        return {};
+      }
+      break;
+    }
+  }
+
+  if (!raw) return {};
+
+  let parsed;
+  try {
+    // Strip single-line comments for .jsonc support before parsing
+    const stripped = raw.replace(/\/\/[^\n]*/g, '');
+    parsed = JSON.parse(stripped);
+  } catch {
+    return {};
+  }
+
+  const result = {};
+
+  // Per-agent model takes priority over global model
+  const agentModel = agentName && parsed.agent?.[agentName]?.model;
+  const globalModel = parsed.model;
+
+  if (agentModel) {
+    result.model = agentModel;
+  } else if (globalModel) {
+    result.model = globalModel;
+  }
+
+  return result;
+}
+
 /**
  * Parse a slash command from the beginning of a prompt
  * Returns null if the prompt doesn't start with a slash command
@@ -367,9 +423,11 @@ export function buildPromptFromTemplate(templateName, item, templatesDir) {
  * @param {object} source - Source configuration (may include _explicit tracking from normalizeSource)
  * @param {object} repoConfig - Repository configuration
  * @param {object} defaults - Default configuration
+ * @param {object} [targetDirConfig] - Optional model config read from the target dir's opencode config.
+ *   Used as a low-priority fallback: only applied when no model is set by source, repo, or defaults.
  * @returns {object} Merged action config
  */
-export function getActionConfig(source, repoConfig, defaults) {
+export function getActionConfig(source, repoConfig, defaults, targetDirConfig = {}) {
   // _explicit tracks fields set directly on the source (not inherited from defaults).
   // When _explicit is absent (e.g., tests constructing source objects directly), fall
   // back to treating non-undefined source fields as explicit.
@@ -379,12 +437,14 @@ export function getActionConfig(source, repoConfig, defaults) {
     if (explicit) {
       if (explicit[field] !== undefined) return explicit[field];
       if (repoConfig[field] !== undefined) return repoConfig[field];
-      return defaults[field];
+      if (defaults[field] !== undefined) return defaults[field];
+      return targetDirConfig[field];
     }
-    // No tracking: source wins, then repo, then defaults
+    // No tracking: source wins, then repo, then defaults, then target dir
     if (source[field] !== undefined) return source[field];
     if (repoConfig[field] !== undefined) return repoConfig[field];
-    return defaults[field];
+    if (defaults[field] !== undefined) return defaults[field];
+    return targetDirConfig[field];
   };
 
   return {
@@ -977,6 +1037,8 @@ async function executeInDirectory(serverUrl, sessionCtx, item, config, options =
       command: apiCommand,
       directory: cwd,
       dryRun: true,
+      ...(config.model && { model: config.model }),
+      ...(config.agent && { agent: config.agent }),
     };
   }
   
@@ -1022,6 +1084,17 @@ export async function executeAction(item, config, options = {}) {
   }
   
   const baseCwd = expandPath(workingDir);
+
+  // Apply target directory's opencode config as a low-priority fallback for model.
+  // This reads <baseCwd>/.opencode/opencode.json and uses its model settings only
+  // when no model is specified in the pilot config (source > repo > defaults).
+  if (!config.model) {
+    const targetDirConfig = readTargetOpencodeConfig(baseCwd, config.agent);
+    if (targetDirConfig.model) {
+      debug(`executeAction: using target dir model=${targetDirConfig.model} from ${baseCwd}/.opencode/opencode.json`);
+      config = { ...config, model: targetDirConfig.model };
+    }
+  }
   
   // Discover running opencode server for this directory
   const discoverFn = options.discoverServer || discoverOpencodeServer;

package/test/unit/actions.test.js

@@ -281,6 +281,121 @@ Check for bugs and security issues.`;
 
   });
 
+  describe('readTargetOpencodeConfig', () => {
+    test('returns global model from .opencode/opencode.json', async () => {
+      const { readTargetOpencodeConfig } = await import('../../service/actions.js');
+
+      const opencodeDir = join(tempDir, '.opencode');
+      mkdirSync(opencodeDir);
+      writeFileSync(join(opencodeDir, 'opencode.json'), JSON.stringify({
+        model: 'anthropic/claude-haiku-3.5'
+      }));
+
+      const result = readTargetOpencodeConfig(tempDir);
+      assert.strictEqual(result.model, 'anthropic/claude-haiku-3.5');
+    });
+
+    test('returns per-agent model from .opencode/opencode.json', async () => {
+      const { readTargetOpencodeConfig } = await import('../../service/actions.js');
+
+      const opencodeDir = join(tempDir, '.opencode');
+      mkdirSync(opencodeDir);
+      writeFileSync(join(opencodeDir, 'opencode.json'), JSON.stringify({
+        model: 'anthropic/claude-sonnet-4-20250514',
+        agent: {
+          plan: { model: 'anthropic/claude-haiku-3.5' }
+        }
+      }));
+
+      const result = readTargetOpencodeConfig(tempDir, 'plan');
+      assert.strictEqual(result.model, 'anthropic/claude-haiku-3.5');
+    });
+
+    test('falls back to global model when agent has no model', async () => {
+      const { readTargetOpencodeConfig } = await import('../../service/actions.js');
+
+      const opencodeDir = join(tempDir, '.opencode');
+      mkdirSync(opencodeDir);
+      writeFileSync(join(opencodeDir, 'opencode.json'), JSON.stringify({
+        model: 'anthropic/claude-opus-4',
+        agent: {
+          plan: { temperature: 0.5 }
+        }
+      }));
+
+      const result = readTargetOpencodeConfig(tempDir, 'plan');
+      assert.strictEqual(result.model, 'anthropic/claude-opus-4');
+    });
+
+    test('returns empty object when no .opencode/opencode.json exists', async () => {
+      const { readTargetOpencodeConfig } = await import('../../service/actions.js');
+
+      const result = readTargetOpencodeConfig(tempDir);
+      assert.deepStrictEqual(result, {});
+    });
+
+    test('returns empty object when .opencode/opencode.json is malformed', async () => {
+      const { readTargetOpencodeConfig } = await import('../../service/actions.js');
+
+      const opencodeDir = join(tempDir, '.opencode');
+      mkdirSync(opencodeDir);
+      writeFileSync(join(opencodeDir, 'opencode.json'), 'not valid json {{{');
+
+      const result = readTargetOpencodeConfig(tempDir);
+      assert.deepStrictEqual(result, {});
+    });
+  });
+
+  describe('getActionConfig with target dir opencode config', () => {
+    test('uses target-dir model when no pilot model is set', async () => {
+      const { getActionConfig } = await import('../../service/actions.js');
+
+      const source = { name: 'my-issues' };
+      const repoConfig = { path: tempDir };
+      const defaults = {};
+      const targetDirConfig = { model: 'anthropic/claude-haiku-3.5' };
+
+      const config = getActionConfig(source, repoConfig, defaults, targetDirConfig);
+      assert.strictEqual(config.model, 'anthropic/claude-haiku-3.5');
+    });
+
+    test('pilot model overrides target-dir model', async () => {
+      const { getActionConfig } = await import('../../service/actions.js');
+
+      const source = { name: 'my-issues', model: 'anthropic/claude-opus-4' };
+      const repoConfig = { path: tempDir };
+      const defaults = {};
+      const targetDirConfig = { model: 'anthropic/claude-haiku-3.5' };
+
+      const config = getActionConfig(source, repoConfig, defaults, targetDirConfig);
+      assert.strictEqual(config.model, 'anthropic/claude-opus-4');
+    });
+
+    test('defaults model overrides target-dir model', async () => {
+      const { getActionConfig } = await import('../../service/actions.js');
+
+      const source = { name: 'my-issues' };
+      const repoConfig = { path: tempDir };
+      const defaults = { model: 'anthropic/claude-sonnet-4-20250514' };
+      const targetDirConfig = { model: 'anthropic/claude-haiku-3.5' };
+
+      const config = getActionConfig(source, repoConfig, defaults, targetDirConfig);
+      assert.strictEqual(config.model, 'anthropic/claude-sonnet-4-20250514');
+    });
+
+    test('target-dir model used when only defaults have no model', async () => {
+      const { getActionConfig } = await import('../../service/actions.js');
+
+      const source = { name: 'my-issues' };
+      const repoConfig = { path: tempDir };
+      const defaults = {};
+      const targetDirConfig = { model: 'openai/gpt-4o' };
+
+      const config = getActionConfig(source, repoConfig, defaults, targetDirConfig);
+      assert.strictEqual(config.model, 'openai/gpt-4o');
+    });
+  });
+
   describe('buildCommand', () => {
     test('builds display string for API call', async () => {
       const { buildCommand } = await import('../../service/actions.js');
@@ -989,6 +1104,64 @@ Check for bugs and security issues.`;
       assert.strictEqual(result.directory, '/data/worktree/calm-wizard',
         'Result should include worktree directory');
     });
+
+    test('uses target dir opencode config model when no pilot model set (dry run)', async () => {
+      const { executeAction } = await import('../../service/actions.js');
+
+      // Write .opencode/opencode.json into the temp project dir
+      const opencodeDir = join(tempDir, '.opencode');
+      mkdirSync(opencodeDir, { recursive: true });
+      writeFileSync(join(opencodeDir, 'opencode.json'), JSON.stringify({
+        model: 'anthropic/claude-haiku-3.5'
+      }));
+
+      const item = { number: 1, title: 'Fix bug' };
+      const config = {
+        path: tempDir,
+        prompt: 'default',
+        agent: 'plan',
+        // No model set in pilot config
+      };
+
+      const mockDiscoverServer = async () => 'http://localhost:4096';
+      const result = await executeAction(item, config, {
+        dryRun: true,
+        discoverServer: mockDiscoverServer,
+      });
+
+      assert.ok(result.dryRun);
+      assert.strictEqual(result.model, 'anthropic/claude-haiku-3.5',
+        'Should use target dir model as fallback');
+    });
+
+    test('target dir opencode config model does not override pilot model (dry run)', async () => {
+      const { executeAction } = await import('../../service/actions.js');
+
+      // Write .opencode/opencode.json into the temp project dir
+      const opencodeDir = join(tempDir, '.opencode');
+      mkdirSync(opencodeDir, { recursive: true });
+      writeFileSync(join(opencodeDir, 'opencode.json'), JSON.stringify({
+        model: 'anthropic/claude-haiku-3.5'
+      }));
+
+      const item = { number: 1, title: 'Fix bug' };
+      const config = {
+        path: tempDir,
+        prompt: 'default',
+        agent: 'plan',
+        model: 'anthropic/claude-opus-4',  // Pilot model is set — should win
+      };
+
+      const mockDiscoverServer = async () => 'http://localhost:4096';
+      const result = await executeAction(item, config, {
+        dryRun: true,
+        discoverServer: mockDiscoverServer,
+      });
+
+      assert.ok(result.dryRun);
+      assert.strictEqual(result.model, 'anthropic/claude-opus-4',
+        'Pilot model should override target dir model');
+    });
   });
 
   describe('createSessionViaApi', () => {