opencode-pilot 0.2.2 → 0.4.0

package/examples/config.yaml

@@ -10,21 +10,33 @@
 
 # =============================================================================
 # TOOLS - Field mappings for MCP servers (normalize different APIs)
+#
+# Each tool provider can specify:
+#   response_key: Key to extract array from response (e.g., "reminders", "nodes")
+#   mappings: Field name translations (e.g., { body: "description" })
 # =============================================================================
 tools:
   github:
+    response_key: items             # GitHub search returns { items: [...] }
     mappings: {}                    # GitHub already uses standard field names
 
   linear:
+    response_key: nodes             # Linear returns { nodes: [...] }
     mappings:
       body: title                         # Use title as body
       number: "url:/([A-Z0-9]+-[0-9]+)/"  # Extract PROJ-123 from URL
 
+  apple-reminders:
+    response_key: reminders         # Apple Reminders returns { reminders: [...] }
+    mappings:
+      title: name                   # Reminders uses 'name' not 'title'
+      body: notes                   # Reminders uses 'notes' for details
+
 # =============================================================================
 # SOURCES - What to poll (generic MCP tool references)
 # =============================================================================
 sources:
-  # GitHub issues assigned to me - work in devcontainer
+  # GitHub issues assigned to me
   - name: my-issues
     tool:
       mcp: github
@@ -33,7 +45,7 @@ sources:
       q: "is:issue assignee:@me state:open"
     item:
       id: "{html_url}"
-    prompt: devcontainer
+    prompt: worktree
     agent: plan
 
   # GitHub PRs needing review
@@ -48,7 +60,7 @@ sources:
     prompt: review
     agent: plan
 
-  # Linear issues - work in devcontainer
+  # Linear issues
   # NOTE: Replace teamId and assigneeId with your actual Linear UUIDs
   # Find these via: linear_list_teams and check user IDs in team members
   - name: linear-work
@@ -62,5 +74,5 @@ sources:
     item:
       id: "linear:{id}"
     working_dir: ~/code/myproject
-    prompt: devcontainer
+    prompt: worktree
     agent: plan

package/examples/templates/devcontainer.md

@@ -1,7 +1,12 @@
-Start a devcontainer for branch issue-{number} and work on this issue:
+Work on this issue in an isolated devcontainer:
 
 {title}
 
 {body}
 
-Follow TDD: write failing tests first, then implement. Create a PR when complete.
+First, set up an isolated workspace:
+1. Create a shallow clone in a temp directory for branch `issue-{number}`
+2. Open the devcontainer in that clone
+3. Work from inside the devcontainer
+
+Complete the implementation and create a PR when ready.

package/examples/templates/worktree.md

@@ -0,0 +1,12 @@
+Work on this issue:
+
+{title}
+
+{body}
+
+First, create a git worktree for this work:
+1. Create branch `issue-{number}` if it doesn't exist
+2. Create a worktree at `../$(basename $PWD)-issue-{number}`
+3. Switch to that worktree directory
+
+Follow TDD: write failing tests first, then implement. Create a PR when complete.

package/package.json

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

package/service/poll-service.js

@@ -9,7 +9,7 @@
  * 5. Track processed items to avoid duplicates
  */
 
-import { loadRepoConfig, getRepoConfig, getAllSources, getToolMappings } from "./repo-config.js";
+import { loadRepoConfig, getRepoConfig, getAllSources, getToolProviderConfig } from "./repo-config.js";
 import { createPoller, pollGenericSource } from "./poller.js";
 import { evaluateReadiness, sortByPriority } from "./readiness.js";
 import { executeAction, buildCommand } from "./actions.js";
@@ -100,8 +100,8 @@ export async function pollOnce(options = {}) {
     // Fetch items from source
     if (!skipMcp) {
       try {
-        const mappings = getToolMappings(source.tool.mcp);
-        items = await pollGenericSource(source, { mappings });
+        const toolProviderConfig = getToolProviderConfig(source.tool.mcp);
+        items = await pollGenericSource(source, { toolProviderConfig });
         debug(`Fetched ${items.length} items from ${sourceName}`);
       } catch (err) {
         console.error(`[poll] Error fetching from ${sourceName}: ${err.message}`);

package/service/poller.js

@@ -107,14 +107,28 @@ export function transformItems(items, idTemplate) {
 
 /**
  * Parse JSON text as an array with error handling
+ * @param {string} text - JSON text to parse
+ * @param {string} sourceName - Source name for error logging
+ * @param {string} [responseKey] - Key to extract array from response object
+ * @returns {Array} Parsed array of items
  */
-function parseJsonArray(text, sourceName) {
+export function parseJsonArray(text, sourceName, responseKey) {
   try {
     const data = JSON.parse(text);
+    
+    // If already an array, return it
     if (Array.isArray(data)) return data;
-    if (data.items) return data.items;
-    if (data.issues) return data.issues;
-    if (data.nodes) return data.nodes;
+    
+    // If response_key is configured, use it to extract the array
+    if (responseKey) {
+      const items = data[responseKey];
+      if (Array.isArray(items)) return items;
+      // response_key was specified but not found or not an array
+      console.error(`[poller] Response key '${responseKey}' not found or not an array in ${sourceName} response`);
+      return [];
+    }
+    
+    // No response_key - wrap single object as array
     return [data];
   } catch (err) {
     console.error(`[poller] Failed to parse ${sourceName} response:`, err.message);
@@ -206,13 +220,15 @@ function createTimeout(ms, operation) {
  * @param {object} [options] - Additional options
  * @param {number} [options.timeout] - Timeout in ms (default: 30000)
  * @param {string} [options.opencodeConfigPath] - Path to opencode.json for MCP config
- * @param {object} [options.mappings] - Field mappings to apply to items
+ * @param {object} [options.toolProviderConfig] - Tool provider config (response_key, mappings)
  * @returns {Promise<Array>} Array of items from the source with IDs and mappings applied
  */
 export async function pollGenericSource(source, options = {}) {
   const toolConfig = getToolConfig(source);
   const timeout = options.timeout || DEFAULT_MCP_TIMEOUT;
-  const mappings = options.mappings || null;
+  const toolProviderConfig = options.toolProviderConfig || {};
+  const responseKey = toolProviderConfig.response_key;
+  const mappings = toolProviderConfig.mappings || null;
   const mcpConfig = getMcpConfig(toolConfig.mcpServer, options.opencodeConfigPath);
   const client = new Client({ name: "opencode-pilot", version: "1.0.0" });
 
@@ -235,7 +251,7 @@ export async function pollGenericSource(source, options = {}) {
     const text = result.content?.[0]?.text;
     if (!text) return [];
     
-    const rawItems = parseJsonArray(text, source.name);
+    const rawItems = parseJsonArray(text, source.name, responseKey);
     
     // Apply field mappings before transforming
     const mappedItems = mappings 

package/service/repo-config.js

@@ -134,6 +134,23 @@ export function getToolMappings(provider) {
   return toolConfig.mappings;
 }
 
+/**
+ * Get full tool provider configuration (response_key, mappings, etc.)
+ * @param {string} provider - Tool provider name (e.g., "github", "linear", "apple-reminders")
+ * @returns {object|null} Tool config including response_key and mappings, or null if not configured
+ */
+export function getToolProviderConfig(provider) {
+  const config = getRawConfig();
+  const tools = config.tools || {};
+  const toolConfig = tools[provider];
+
+  if (!toolConfig) {
+    return null;
+  }
+
+  return toolConfig;
+}
+
 /**
  * Load a template from the templates directory
  * @param {string} templateName - Template name (without .md extension)

package/test/unit/poller.test.js

@@ -312,6 +312,65 @@ describe('poller.js', () => {
     });
   });
 
+  describe('parseJsonArray', () => {
+    test('parses direct array response', async () => {
+      const { parseJsonArray } = await import('../../service/poller.js');
+      
+      const text = JSON.stringify([{ id: '1' }, { id: '2' }]);
+      const result = parseJsonArray(text, 'test');
+      
+      assert.strictEqual(result.length, 2);
+      assert.strictEqual(result[0].id, '1');
+    });
+
+    test('extracts array using response_key', async () => {
+      const { parseJsonArray } = await import('../../service/poller.js');
+      
+      const text = JSON.stringify({
+        reminders: [
+          { id: 'reminder-1', name: 'Task 1', completed: false },
+          { id: 'reminder-2', name: 'Task 2', completed: false },
+          { id: 'reminder-3', name: 'Task 3', completed: false }
+        ],
+        count: 3
+      });
+      const result = parseJsonArray(text, 'agent-tasks', 'reminders');
+      
+      assert.strictEqual(result.length, 3);
+      assert.strictEqual(result[0].id, 'reminder-1');
+      assert.strictEqual(result[0].name, 'Task 1');
+      assert.strictEqual(result[1].id, 'reminder-2');
+      assert.strictEqual(result[2].id, 'reminder-3');
+    });
+
+    test('wraps single object as array when no response_key', async () => {
+      const { parseJsonArray } = await import('../../service/poller.js');
+      
+      const text = JSON.stringify({ id: '1', title: 'Single item' });
+      const result = parseJsonArray(text, 'test');
+      
+      assert.strictEqual(result.length, 1);
+      assert.strictEqual(result[0].id, '1');
+    });
+
+    test('returns empty array for invalid JSON', async () => {
+      const { parseJsonArray } = await import('../../service/poller.js');
+      
+      const result = parseJsonArray('not valid json', 'test');
+      
+      assert.strictEqual(result.length, 0);
+    });
+
+    test('returns empty array when response_key not found', async () => {
+      const { parseJsonArray } = await import('../../service/poller.js');
+      
+      const text = JSON.stringify({ items: [{ id: '1' }] });
+      const result = parseJsonArray(text, 'test', 'reminders');
+      
+      assert.strictEqual(result.length, 0);
+    });
+  });
+
   describe('transformItems with mappings', () => {
     test('applies mappings to all items', async () => {
       const { transformItems, applyMappings } = await import('../../service/poller.js');

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

@@ -314,6 +314,45 @@ sources: []
       
       assert.strictEqual(mappings, null);
     });
+
+    test('getToolProviderConfig returns full tool config with response_key', async () => {
+      writeFileSync(configPath, `
+tools:
+  apple-reminders:
+    response_key: reminders
+    mappings:
+      body: notes
+
+sources: []
+`);
+
+      const { loadRepoConfig, getToolProviderConfig } = await import('../../service/repo-config.js');
+      loadRepoConfig(configPath);
+      
+      const toolConfig = getToolProviderConfig('apple-reminders');
+      
+      assert.strictEqual(toolConfig.response_key, 'reminders');
+      assert.deepStrictEqual(toolConfig.mappings, { body: 'notes' });
+    });
+
+    test('getToolProviderConfig returns config without response_key', async () => {
+      writeFileSync(configPath, `
+tools:
+  github:
+    mappings:
+      url: html_url
+
+sources: []
+`);
+
+      const { loadRepoConfig, getToolProviderConfig } = await import('../../service/repo-config.js');
+      loadRepoConfig(configPath);
+      
+      const toolConfig = getToolProviderConfig('github');
+      
+      assert.strictEqual(toolConfig.response_key, undefined);
+      assert.deepStrictEqual(toolConfig.mappings, { url: 'html_url' });
+    });
   });
 
   describe('repo resolution for sources', () => {