opencode-pilot 0.24.1 → 0.24.2

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.23.1.tar.gz"
-  sha256 "106172281e27a0847f1c457837c503aa64058908ab4ac5df6fb7326db4c07079"
+  url "https://github.com/athal7/opencode-pilot/archive/refs/tags/v0.24.1.tar.gz"
+  sha256 "ea5cb01e8893bdebc24b7868659355c1762e6672bfc50b2cec775f0431381adb"
   license "MIT"
 
   depends_on "node"

package/package.json

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

package/service/presets/github.yaml

@@ -18,7 +18,7 @@ _provider:
 
 # Presets
 my-issues:
-  name: my-issues
+  name: github-my-issues
   tool:
     command: ["gh", "search", "issues", "--assignee=@me", "--state=open", "--json", "number,title,url,repository,state,body,updatedAt"]
   item:

package/service/presets/linear.yaml

@@ -13,7 +13,7 @@ _provider:
 
 # Presets
 my-issues:
-  name: my-issues
+  name: linear-my-issues
   tool:
     mcp: linear
     name: list_issues

package/test/unit/repo-config.test.js

@@ -656,7 +656,7 @@ sources:
       const sources = getSources();
 
       assert.strictEqual(sources.length, 1);
-      assert.strictEqual(sources[0].name, 'my-issues');
+      assert.strictEqual(sources[0].name, 'github-my-issues');
       // GitHub presets now use gh CLI instead of MCP
       assert.ok(Array.isArray(sources[0].tool.command), 'tool.command should be an array');
       assert.ok(sources[0].tool.command.includes('gh'), 'command should use gh CLI');
@@ -714,7 +714,7 @@ sources:
       loadRepoConfig(configPath);
       const sources = getSources();
 
-      assert.strictEqual(sources[0].name, 'my-issues');
+      assert.strictEqual(sources[0].name, 'linear-my-issues');
       assert.deepStrictEqual(sources[0].tool, { mcp: 'linear', name: 'list_issues' });
       assert.strictEqual(sources[0].args.teamId, 'team-uuid-123');
       assert.strictEqual(sources[0].args.assigneeId, 'user-uuid-456');
@@ -913,6 +913,43 @@ sources:
       assert.strictEqual(sources[0].worktree_name, '{number}', 'linear preset should use {number} (identifier)');
     });
 
+    test('all preset names are globally unique (no cross-provider collisions)', async () => {
+      const { listPresets, expandPreset } = await import('../../service/presets/index.js');
+      const presets = listPresets();
+      
+      const nameToPreset = new Map();
+      for (const presetKey of presets) {
+        const expanded = expandPreset(presetKey, {});
+        const name = expanded.name;
+        
+        assert.ok(!nameToPreset.has(name),
+          `Duplicate source name "${name}" from presets "${nameToPreset.get(name)}" and "${presetKey}". ` +
+          `Source names must be unique to prevent cross-source pollution in poll state tracking.`);
+        nameToPreset.set(name, presetKey);
+      }
+    });
+
+    test('github and linear my-issues presets have distinct names when used together', async () => {
+      writeFileSync(configPath, `
+sources:
+  - preset: linear/my-issues
+    args:
+      teamId: "team-uuid"
+      assigneeId: "user-uuid"
+  - preset: github/my-issues
+`);
+
+      const { loadRepoConfig, getSources } = await import('../../service/repo-config.js');
+      loadRepoConfig(configPath);
+      const sources = getSources();
+
+      assert.strictEqual(sources.length, 2);
+      assert.notStrictEqual(sources[0].name, sources[1].name,
+        'Linear and GitHub my-issues presets must have different names');
+      assert.strictEqual(sources[0].name, 'linear-my-issues');
+      assert.strictEqual(sources[1].name, 'github-my-issues');
+    });
+
   });
 
   describe('shorthand syntax', () => {