opencode-pilot 0.24.5 → 0.24.6

package/Formula/opencode-pilot.rb

@@ -1,8 +1,8 @@
 class OpencodePilot < Formula
   desc "Automation daemon for OpenCode - polls GitHub/Linear issues and spawns sessions"
   homepage "https://github.com/athal7/opencode-pilot"
-  url "https://github.com/athal7/opencode-pilot/archive/refs/tags/v0.24.4.tar.gz"
-  sha256 "7a697b9543877054b0ae42a4bbfff69f1bb9b91c3c724cbee9cc256bd75d2e83"
+  url "https://github.com/athal7/opencode-pilot/archive/refs/tags/v0.24.5.tar.gz"
+  sha256 "6a191feaa9f7ec7421272a0c9533ebbd734a45483ca7116135b62e4ba11e780d"
   license "MIT"
 
   depends_on "node"

package/package.json

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

package/service/poller.js

@@ -1275,6 +1275,10 @@ export function createPoller(options = {}) {
             const storedFeedbackAt = meta.latestFeedbackAt;
             const currentFeedbackAt = item._latest_feedback_at;
             
+            // No stored baseline: any current feedback is new (legacy state entries)
+            if (!storedFeedbackAt && currentFeedbackAt) {
+              return true;
+            }
             if (storedFeedbackAt && currentFeedbackAt && currentFeedbackAt > storedFeedbackAt) {
               return true;
             }

package/test/unit/poller.test.js

@@ -855,7 +855,7 @@ describe('poller.js', () => {
       );
     });
 
-    test('shouldReprocess returns false when attention stays true with no timestamps', async () => {
+    test('shouldReprocess returns false when attention stays true with no timestamps on either side', async () => {
       const { createPoller } = await import('../../service/poller.js');
       
       const poller = createPoller({ stateFile });
@@ -871,7 +871,28 @@ describe('poller.js', () => {
       assert.strictEqual(
         poller.shouldReprocess(item, { reprocessOn: ['attention'] }), 
         false,
-        'Should NOT reprocess when no timestamps available (legacy)'
+        'Should NOT reprocess when no timestamps available on either side'
+      );
+    });
+
+    test('shouldReprocess returns true when legacy entry has no stored feedback timestamp but current item does', async () => {
+      const { createPoller } = await import('../../service/poller.js');
+      
+      const poller = createPoller({ stateFile });
+      // Legacy item processed with attention but before latestFeedbackAt was tracked
+      poller.markProcessed('pr-1', { 
+        source: 'my-prs-attention', 
+        itemState: 'open',
+        hasAttention: true
+        // Note: no latestFeedbackAt
+      });
+      
+      // Item now has feedback with a timestamp (new review received)
+      const item = { id: 'pr-1', state: 'open', _has_attention: true, _latest_feedback_at: '2026-02-17T15:11:15Z' };
+      assert.strictEqual(
+        poller.shouldReprocess(item, { reprocessOn: ['attention'] }), 
+        true,
+        'Should reprocess legacy entries when new feedback is detected'
       );
     });