instar 0.28.100 → 0.28.102

package/dist/commands/server.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"server.d.ts","sourceRoot":"","sources":["../../src/commands/server.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAkQH,UAAU,YAAY;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,GAAG,CAAC,EAAE,MAAM,CAAC;IACb;2DACuD;IACvD,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB;AAw4CD,wBAAsB,WAAW,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO,CAAC,IAAI,CAAC,CAknKtE;AAED,wBAAsB,UAAU,CAAC,OAAO,EAAE;IAAE,GAAG,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAsDzE;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,aAAa,CAAC,OAAO,EAAE;IAAE,GAAG,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAuD5E"}
+{"version":3,"file":"server.d.ts","sourceRoot":"","sources":["../../src/commands/server.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAkQH,UAAU,YAAY;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,GAAG,CAAC,EAAE,MAAM,CAAC;IACb;2DACuD;IACvD,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB;AAw4CD,wBAAsB,WAAW,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO,CAAC,IAAI,CAAC,CA8nKtE;AAED,wBAAsB,UAAU,CAAC,OAAO,EAAE;IAAE,GAAG,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAsDzE;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,aAAa,CAAC,OAAO,EAAE;IAAE,GAAG,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAuD5E"}

package/dist/commands/server.js

@@ -4341,14 +4341,20 @@ export async function startServer(options) {
         let presenceProxy;
         if (sharedIntelligence && telegram) {
             try {
-                const { PresenceProxy } = await import('../monitoring/PresenceProxy.js');
+                const { PresenceProxy, isBriefAck } = await import('../monitoring/PresenceProxy.js');
                 const { execSync: shellExecSync } = await import('child_process');
                 const messagesLogPath = path.join(config.stateDir, 'telegram-messages.jsonl');
                 const slackMessagesLogPath = path.join(config.stateDir, 'slack-messages.jsonl');
-                // Shared helper: check a messages log file for agent responses after a timestamp.
-                // System/proxy-message filtering is delegated to isSystemOrProxyMessage so all
-                // three callsites (this one, PresenceProxy.isSystemMessage, and the compaction
-                // recoverFn) stay in sync.
+                // Shared helper: check a messages log file for SUBSTANTIVE agent
+                // responses after a timestamp. Filters out system/proxy messages via
+                // isSystemOrProxyMessage AND brief acks via isBriefAck — both kinds
+                // of messages are non-cancelling from PresenceProxy's perspective.
+                //
+                // The brief-ack filter is what closes the gap from PR #128: that PR
+                // taught the event path (recordAgentMessage) to ignore acks, but the
+                // log-reading race guard at PresenceProxy.fireTier didn't share the
+                // same classifier. The result was Tier 2 silently cancelling because
+                // the ack was on disk. Same isBriefAck export feeds both paths now.
                 const checkLogForAgentResponse = (logPath, topicId, sinceIso) => {
                     try {
                         const content = fs.readFileSync(logPath, 'utf-8');
@@ -4361,8 +4367,14 @@ export async function startServer(options) {
                                 const matchesTopic = msg.topicId === topicId
                                     || (topicId < 0 && msg.channelId && slackChannelToSyntheticId(String(msg.channelId)) === topicId);
                                 if (matchesTopic && !msg.fromUser && msg.timestamp > sinceIso) {
-                                    if (!isSystemOrProxyMessage(msg.text))
-                                        return true;
+                                    // Non-cancelling agent message kinds: system/proxy
+                                    // chrome, and brief acks ("On it"). Both leave tier
+                                    // timers running.
+                                    const isNonCancelling = isSystemOrProxyMessage(msg.text)
+                                        || isBriefAck(msg.text);
+                                    if (isNonCancelling)
+                                        continue;
+                                    return true;
                                 }
                             }
                             catch { /* skip malformed lines */ }