opencode-pilot 0.16.4 → 0.16.5

package/package.json

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

package/service/actions.js

@@ -525,19 +525,40 @@ export async function createSessionViaApi(serverUrl, directory, prompt, options
       messageBody.modelID = modelID;
     }
     
-    const messageResponse = await fetchFn(messageUrl.toString(), {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(messageBody),
-    });
+    // Use AbortController with timeout for the message POST
+    // The /session/{id}/message endpoint returns a chunked/streaming response
+    // that stays open until the agent completes. We only need to verify the
+    // request was accepted (2xx status), not wait for the full response.
+    const controller = new AbortController();
+    const timeoutId = setTimeout(() => controller.abort(), 10000); // 10 second timeout
     
-    if (!messageResponse.ok) {
-      const errorText = await messageResponse.text();
-      throw new Error(`Failed to send message: ${messageResponse.status} ${errorText}`);
+    try {
+      const messageResponse = await fetchFn(messageUrl.toString(), {
+        method: 'POST',
+        headers: { 'Content-Type': 'application/json' },
+        body: JSON.stringify(messageBody),
+        signal: controller.signal,
+      });
+      
+      clearTimeout(timeoutId);
+      
+      if (!messageResponse.ok) {
+        const errorText = await messageResponse.text();
+        throw new Error(`Failed to send message: ${messageResponse.status} ${errorText}`);
+      }
+      
+      debug(`createSessionViaApi: sent message to session ${session.id}`);
+    } catch (abortErr) {
+      clearTimeout(timeoutId);
+      // AbortError is expected - we intentionally abort after verifying the request started
+      // The server accepted our message, we just don't need to wait for the response
+      if (abortErr.name === 'AbortError') {
+        debug(`createSessionViaApi: message request started for session ${session.id} (response aborted as expected)`);
+      } else {
+        throw abortErr;
+      }
     }
     
-    debug(`createSessionViaApi: sent message to session ${session.id}`);
-    
     return {
       success: true,
       sessionId: session.id,

package/service/repo-config.js

@@ -352,7 +352,12 @@ export function resolveRepoForItem(source, item) {
     if (resolvedRepo) {
       return source.repos.includes(resolvedRepo) ? [resolvedRepo] : [];
     }
-    // No repo template - return empty (can't match without item context)
+    // No repo template - if exactly one repo, use it as default
+    // (e.g., Linear issues don't have repo context, user explicitly configures one repo)
+    if (source.repos.length === 1) {
+      return source.repos;
+    }
+    // Multiple repos but can't match without item context
     return [];
   }
 

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

@@ -769,6 +769,46 @@ sources:
       assert.deepStrictEqual(resolveRepoForItem(source, filteredItem), []);
     });
 
+    test('single-repo allowlist uses repo as default when no template', async () => {
+      // Linear issues don't have repository context - when exactly one repo is configured,
+      // use it as the default for all items from that source
+      writeFileSync(configPath, `
+sources:
+  - preset: linear/my-issues
+    repos:
+      - 0din-ai/odin
+`);
+
+      const { loadRepoConfig, getSources, resolveRepoForItem } = await import('../../service/repo-config.js');
+      loadRepoConfig(configPath);
+      const source = getSources()[0];
+
+      // Linear items don't have repository field
+      const linearItem = { id: 'linear:abc123', title: 'Fix bug', state: { name: 'In Progress' } };
+      assert.deepStrictEqual(resolveRepoForItem(source, linearItem), ['0din-ai/odin'],
+        'single-repo allowlist should use repo as default');
+    });
+
+    test('multi-repo allowlist returns empty when no template match', async () => {
+      // With multiple repos and no way to determine which one, return empty
+      writeFileSync(configPath, `
+sources:
+  - preset: linear/my-issues
+    repos:
+      - org/repo-a
+      - org/repo-b
+`);
+
+      const { loadRepoConfig, getSources, resolveRepoForItem } = await import('../../service/repo-config.js');
+      loadRepoConfig(configPath);
+      const source = getSources()[0];
+
+      // Can't determine which of the 2 repos to use
+      const linearItem = { id: 'linear:abc123', title: 'Fix bug' };
+      assert.deepStrictEqual(resolveRepoForItem(source, linearItem), [],
+        'multi-repo allowlist should return empty when no template');
+    });
+
     test('github presets include semantic session names', async () => {
       writeFileSync(configPath, `
 sources: