opencode-pilot 0.18.0 → 0.18.1

package/package.json

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

package/service/poll-service.js

@@ -124,7 +124,12 @@ export async function pollOnce(options = {}) {
     // Fetch items from source
     if (!skipMcp) {
       try {
-        toolProviderConfig = getToolProviderConfig(source.tool.mcp);
+        // Get provider config - for MCP sources use source.tool.mcp, for CLI sources detect provider
+        let provider = source.tool.mcp;
+        if (!provider && Array.isArray(source.tool?.command) && source.tool.command[0] === 'gh') {
+          provider = 'github'; // CLI-based GitHub source
+        }
+        toolProviderConfig = getToolProviderConfig(provider);
         items = await pollGenericSource(source, { toolProviderConfig });
         debug(`Fetched ${items.length} items from ${sourceName}`);
         

package/service/poller.js

@@ -12,7 +12,7 @@ import { SSEClientTransport } from "@modelcontextprotocol/sdk/client/sse.js";
 import fs from "fs";
 import path from "path";
 import os from "os";
-import { getNestedValue } from "./utils.js";
+import { getNestedValue, hasNonBotFeedback } from "./utils.js";
 
 /**
  * Expand template string with item fields
@@ -501,6 +501,22 @@ export async function fetchGitHubComments(item, options = {}) {
   }
 }
 
+/**
+ * Check if a source is a GitHub source (MCP or CLI-based)
+ * @param {object} source - Source configuration
+ * @returns {boolean} True if this is a GitHub source
+ */
+function isGitHubSource(source) {
+  // MCP-based GitHub source
+  if (source.tool?.mcp === "github") return true;
+  
+  // CLI-based GitHub source (uses gh command)
+  const command = source.tool?.command;
+  if (Array.isArray(command) && command[0] === "gh") return true;
+  
+  return false;
+}
+
 /**
  * Enrich items with comments for bot filtering
  * 
@@ -514,7 +530,7 @@ export async function fetchGitHubComments(item, options = {}) {
  */
 export async function enrichItemsWithComments(items, source, options = {}) {
   // Skip if not configured or not a GitHub source
-  if (!source.filter_bot_comments || source.tool?.mcp !== "github") {
+  if (!source.filter_bot_comments || !isGitHubSource(source)) {
     return items;
   }
   
@@ -618,16 +634,11 @@ export function computeAttentionLabels(items, source) {
       reasons.push('Conflicts');
     }
     
-    // Check for human feedback (non-bot, non-author comments)
+    // Check for human feedback using the shared hasNonBotFeedback utility
+    // This properly handles known bots like 'linear' that don't have [bot] suffix
     if (item._comments && item._comments.length > 0) {
       const authorUsername = item.user?.login || item.author?.login;
-      const hasHumanFeedback = item._comments.some(comment => {
-        const commenter = comment.user?.login || comment.author?.login;
-        const isBot = commenter?.includes('[bot]') || comment.user?.type === 'Bot';
-        const isAuthor = commenter === authorUsername;
-        return !isBot && !isAuthor;
-      });
-      if (hasHumanFeedback) {
+      if (hasNonBotFeedback(item._comments, authorUsername)) {
         reasons.push('Feedback');
       }
     }

package/test/unit/poller.test.js

@@ -1013,5 +1013,25 @@ describe('poller.js', () => {
       assert.strictEqual(result[0]._attention_label, 'PR');
       assert.strictEqual(result[0]._has_attention, false);
     });
+
+    test('ignores known bots without [bot] suffix (e.g., linear)', async () => {
+      const { computeAttentionLabels } = await import('../../service/poller.js');
+      
+      const items = [{
+        number: 123,
+        title: 'Test PR',
+        user: { login: 'author' },
+        _mergeable: 'MERGEABLE',
+        _comments: [
+          // Linear bot posts linkback comments without [bot] suffix
+          { user: { login: 'linear', type: 'User' }, body: '<!-- linear-linkback -->' }
+        ]
+      }];
+      
+      const result = computeAttentionLabels(items, {});
+      
+      assert.strictEqual(result[0]._attention_label, 'PR');
+      assert.strictEqual(result[0]._has_attention, false);
+    });
   });
 });