opencode-pilot 0.20.1 → 0.20.3

package/package.json

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

package/service/poll-service.js

@@ -50,6 +50,7 @@ export function buildActionConfigFromSource(source, repoConfig) {
     ...(source.agent && { agent: source.agent }),
     ...(source.model && { model: source.model }),
     ...(source.working_dir && { working_dir: source.working_dir }),
+    ...(source.worktree_name && { worktree_name: source.worktree_name }),
   };
 }
 

package/service/presets/github.yaml

@@ -10,6 +10,7 @@ _provider:
     html_url: url
     repository_full_name: repository.nameWithOwner
     updated_at: updatedAt
+    comments: commentsCount
   # Reprocess items when state changes (e.g., reopened issues)
   # Note: updated_at is NOT included because our own changes would trigger reprocessing
   reprocess_on:
@@ -23,6 +24,7 @@ my-issues:
   item:
     id: "{url}"
   repo: "{repository.nameWithOwner}"
+  worktree_name: "issue-{number}"
   session:
     name: "{title}"
 
@@ -34,6 +36,7 @@ review-requests:
     id: "{url}"
   repo: "{repository.nameWithOwner}"
   prompt: review
+  worktree_name: "pr-{number}"
   session:
     name: "Review: {title}"
 
@@ -46,6 +49,7 @@ my-prs-attention:
     id: "{url}"
   repo: "{repository.nameWithOwner}"
   prompt: review-feedback
+  worktree_name: "pr-{number}"
   session:
     # Dynamic name showing which conditions triggered (set by enrichment)
     name: "{_attention_label}: {title}"
@@ -55,7 +59,7 @@ my-prs-attention:
   readiness:
     # Require at least one attention condition (conflicts or human feedback)
     require_attention: true
-  # Reprocess when PR is updated
+  # Only reprocess when state changes (e.g., reopened)
+  # Note: updatedAt is NOT included - CI status changes would trigger reprocessing
   reprocess_on:
     - state
-    - updatedAt

package/service/presets/linear.yaml

@@ -21,5 +21,6 @@ my-issues:
     # teamId and assigneeId are required - user must provide
   item:
     id: "linear:{id}"
+  worktree_name: "{number}"
   session:
     name: "{title}"

package/test/unit/poll-service.test.js

@@ -154,6 +154,56 @@ sources:
       assert.strictEqual(config.agent, 'code');
     });
 
+    test('includes worktree_name from source config', async () => {
+      const { buildActionConfigFromSource } = await import('../../service/poll-service.js');
+      
+      const source = {
+        name: 'test-source',
+        worktree_name: 'pr-{number}'
+      };
+      const repoConfig = {
+        path: '~/code/default'
+      };
+      
+      const config = buildActionConfigFromSource(source, repoConfig);
+      
+      assert.strictEqual(config.worktree_name, 'pr-{number}');
+    });
+
+    test('worktree_name from source overrides repoConfig', async () => {
+      const { buildActionConfigFromSource } = await import('../../service/poll-service.js');
+      
+      const source = {
+        name: 'test-source',
+        worktree_name: 'pr-{number}'
+      };
+      const repoConfig = {
+        path: '~/code/default',
+        worktree_name: 'issue-{number}'  // Should be overridden
+      };
+      
+      const config = buildActionConfigFromSource(source, repoConfig);
+      
+      assert.strictEqual(config.worktree_name, 'pr-{number}');
+    });
+
+    test('falls back to repoConfig worktree_name when source has none', async () => {
+      const { buildActionConfigFromSource } = await import('../../service/poll-service.js');
+      
+      const source = {
+        name: 'test-source'
+        // No worktree_name
+      };
+      const repoConfig = {
+        path: '~/code/default',
+        worktree_name: 'issue-{number}'
+      };
+      
+      const config = buildActionConfigFromSource(source, repoConfig);
+      
+      assert.strictEqual(config.worktree_name, 'issue-{number}');
+    });
+
   });
 
   describe('per-item repo resolution', () => {

package/test/unit/poller.test.js

@@ -786,6 +786,31 @@ describe('poller.js', () => {
       
       assert.strictEqual(mapped.number, undefined);
     });
+
+    test('maps commentsCount to comments for GitHub PR enrichment', async () => {
+      const { applyMappings } = await import('../../service/poller.js');
+      
+      // gh search prs returns commentsCount, but enrichItemsWithComments checks for 'comments'
+      const item = {
+        number: 123,
+        title: 'Fix mobile overflow',
+        commentsCount: 4,
+        repository: { nameWithOwner: 'anomalyco/opencode' }
+      };
+      const mappings = {
+        comments: 'commentsCount',
+        repository_full_name: 'repository.nameWithOwner'
+      };
+      
+      const mapped = applyMappings(item, mappings);
+      
+      // comments field should be set from commentsCount
+      assert.strictEqual(mapped.comments, 4);
+      // Original commentsCount preserved
+      assert.strictEqual(mapped.commentsCount, 4);
+      // Other mappings work too
+      assert.strictEqual(mapped.repository_full_name, 'anomalyco/opencode');
+    });
   });
 
   describe('fetchGitHubComments', () => {

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

@@ -874,6 +874,45 @@ sources:
       assert.strictEqual(sources[0].session.name, '{title}', 'linear preset should use title');
     });
 
+    test('github presets include worktree_name for sandbox reuse', async () => {
+      writeFileSync(configPath, `
+sources:
+  - preset: github/my-issues
+  - preset: github/review-requests
+  - preset: github/my-prs-attention
+`);
+
+      const { loadRepoConfig, getSources } = await import('../../service/repo-config.js');
+      loadRepoConfig(configPath);
+      const sources = getSources();
+
+      // my-issues: worktree_name: "issue-{number}"
+      assert.strictEqual(sources[0].worktree_name, 'issue-{number}', 'my-issues should use issue-{number}');
+      
+      // review-requests: worktree_name: "pr-{number}"
+      assert.strictEqual(sources[1].worktree_name, 'pr-{number}', 'review-requests should use pr-{number}');
+      
+      // my-prs-attention: worktree_name: "pr-{number}"
+      assert.strictEqual(sources[2].worktree_name, 'pr-{number}', 'my-prs-attention should use pr-{number}');
+    });
+
+    test('linear preset includes worktree_name for sandbox reuse', async () => {
+      writeFileSync(configPath, `
+sources:
+  - preset: linear/my-issues
+    args:
+      teamId: "team-uuid"
+      assigneeId: "user-uuid"
+`);
+
+      const { loadRepoConfig, getSources } = await import('../../service/repo-config.js');
+      loadRepoConfig(configPath);
+      const sources = getSources();
+
+      // linear uses the issue identifier (e.g., "ABC-123")
+      assert.strictEqual(sources[0].worktree_name, '{number}', 'linear preset should use {number} (identifier)');
+    });
+
   });
 
   describe('shorthand syntax', () => {