opencode-pilot 0.6.0 → 0.6.1

package/package.json

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

package/service/poll-service.js

@@ -185,7 +185,7 @@ export async function pollOnce(options = {}) {
           if (result.success) {
             // Mark as processed to avoid re-triggering
             if (pollerInstance) {
-              pollerInstance.markProcessed(item.id, { repoKey, command: result.command });
+              pollerInstance.markProcessed(item.id, { repoKey: item.repo_key, command: result.command });
             }
             console.log(`[poll] Started session for ${item.id}`);
           } else {

package/service/presets/github.yaml

@@ -3,7 +3,9 @@
 # Provider-level config (applies to all GitHub presets)
 _provider:
   response_key: items
-  mappings: {}
+  mappings:
+    # Extract repo full name from repository_url (e.g., "https://api.github.com/repos/owner/repo")
+    repository_full_name: "repository_url:/repos\\/([^/]+\\/[^/]+)$/"
 
 # Presets
 my-issues:
@@ -15,7 +17,7 @@ my-issues:
     q: "is:issue assignee:@me state:open"
   item:
     id: "{html_url}"
-  repo: "{repository.full_name}"
+  repo: "{repository_full_name}"
 
 review-requests:
   name: review-requests
@@ -26,7 +28,7 @@ review-requests:
     q: "is:pr review-requested:@me state:open"
   item:
     id: "{html_url}"
-  repo: "{repository.full_name}"
+  repo: "{repository_full_name}"
 
 my-prs-feedback:
   name: my-prs-feedback
@@ -37,4 +39,4 @@ my-prs-feedback:
     q: "is:pr author:@me state:open review:changes_requested"
   item:
     id: "{html_url}"
-  repo: "{repository.full_name}"
+  repo: "{repository_full_name}"

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

@@ -159,14 +159,15 @@ sources:
       loadRepoConfig(configPath);
       
       const source = getSources()[0];
+      // Item with repository_full_name (mapped from repository_url by GitHub provider)
       const item = { 
-        repository: { full_name: 'myorg/backend' },
+        repository_full_name: 'myorg/backend',
         number: 123,
         html_url: 'https://github.com/myorg/backend/issues/123'
       };
       
-      // Source should have repo field from preset
-      assert.strictEqual(source.repo, '{repository.full_name}');
+      // Source should have repo field from preset (uses mapped field)
+      assert.strictEqual(source.repo, '{repository_full_name}');
       
       // resolveRepoForItem should extract repo key from item
       const repoKeys = resolveRepoForItem(source, item);
@@ -194,8 +195,9 @@ sources:
       loadRepoConfig(configPath);
       
       const source = getSources()[0];
+      // Item with repository_full_name (mapped from repository_url by GitHub provider)
       const item = { 
-        repository: { full_name: 'unknown/repo' },
+        repository_full_name: 'unknown/repo',
         number: 456
       };
       

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

@@ -352,7 +352,9 @@ sources: []
       
       // GitHub preset has response_key: items, user config doesn't override it
       assert.strictEqual(toolConfig.response_key, 'items');
-      assert.deepStrictEqual(toolConfig.mappings, { url: 'html_url' });
+      // GitHub provider has mapping for repository_full_name extraction from URL
+      assert.strictEqual(toolConfig.mappings.url, 'html_url');
+      assert.ok(toolConfig.mappings.repository_full_name, 'Should have repository_full_name mapping');
     });
 
     test('getToolProviderConfig falls back to preset provider config', async () => {
@@ -628,11 +630,12 @@ sources:
       loadRepoConfig(configPath);
       const sources = getSources();
 
-      // All GitHub presets should have repo field for automatic resolution
-      const mockItem = { repository: { full_name: 'myorg/backend' } };
+      // All GitHub presets should have repo field that references repository_full_name
+      // (which is mapped from repository_url by the GitHub provider)
+      const mockItem = { repository_full_name: 'myorg/backend' };
       
       for (const source of sources) {
-        assert.strictEqual(source.repo, '{repository.full_name}', `Preset ${source.name} should have repo field`);
+        assert.strictEqual(source.repo, '{repository_full_name}', `Preset ${source.name} should have repo field`);
         const repos = resolveRepoForItem(source, mockItem);
         assert.deepStrictEqual(repos, ['myorg/backend'], `Preset ${source.name} should resolve repo from item`);
       }
@@ -651,12 +654,12 @@ sources:
       loadRepoConfig(configPath);
       const source = getSources()[0];
 
-      // Item from allowed repo should resolve
-      const allowedItem = { repository: { full_name: 'myorg/backend' } };
+      // Item from allowed repo should resolve (repository_full_name is mapped from repository_url)
+      const allowedItem = { repository_full_name: 'myorg/backend' };
       assert.deepStrictEqual(resolveRepoForItem(source, allowedItem), ['myorg/backend']);
 
       // Item from non-allowed repo should return empty (filtered out)
-      const filteredItem = { repository: { full_name: 'other/repo' } };
+      const filteredItem = { repository_full_name: 'other/repo' };
       assert.deepStrictEqual(resolveRepoForItem(source, filteredItem), []);
     });
   });