polygram 0.5.5 → 0.5.7

package/lib/abort-detector.js

@@ -6,14 +6,17 @@
  * interrupt the in-flight turn instead of queueing the message behind it.
  *
  * Conservative on purpose. False positives hijack user intent — "stop using
- * emoji" should NOT abort. So we require:
- *   1. The message (after stripping leading @-mention + trailing punctuation)
- *      must be an exact match against a known abort phrase, OR
- *   2. It must start with an explicit slash command: /stop, /abort, /cancel.
+ * emoji" should NOT abort. So we require ONE of:
+ *   1. The whole message (after stripping leading @-mention + trailing
+ *      punctuation) is an exact match against a known abort phrase, OR
+ *   2. It starts with an explicit slash command: /stop, /abort, /cancel, OR
+ *   3. The FIRST SENTENCE (split on . ! ?) is an exact abort phrase. This
+ *      catches "Stop. I'll ask in another session." — clear abort intent
+ *      with continuation explaining what comes next. Comma is not a split
+ *      character ("Stop, look here" is ambiguous and stays non-abort).
  *
  * Not detected (on purpose):
- *   - "wait a sec while I finish typing" → too long, real content
- *   - "stop using markdown" → has trailing content
+ *   - "stop using markdown" → first sentence is the whole thing, not exact
  *   - "I said stop" → not at start / not exact match
  */
 
@@ -54,10 +57,21 @@ function isAbortRequest(text) {
 
   const n = normalize(text);
   if (!n) return false;
-  // Cap length: a long message that happens to start with "stop" is real
-  // content, not an abort. 40 chars covers all phrases above with headroom.
-  if (n.length > 40) return false;
-  return ABORT_PHRASES.has(n);
+  // Whole-message exact match (capped — a long message that happens to
+  // start with "stop" is real content, not an abort).
+  if (n.length <= 40 && ABORT_PHRASES.has(n)) return true;
+
+  // First-sentence exact match. Splits on . ! ? (NOT comma — "Stop, look
+  // here" is ambiguous and stays non-abort). The leading @-mention has
+  // already been stripped by normalize but only on the whole string, so
+  // we strip it again on the raw text before splitting.
+  const head = text.trim().replace(LEADING_MENTION_RE, '');
+  const firstSentence = head.split(/[.!?]/, 1)[0]?.trim().toLowerCase();
+  if (firstSentence && firstSentence.length <= 40 && ABORT_PHRASES.has(firstSentence)) {
+    return true;
+  }
+
+  return false;
 }
 
 module.exports = { isAbortRequest, ABORT_PHRASES };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polygram",
-  "version": "0.5.5",
+  "version": "0.5.7",
   "description": "Telegram daemon for Claude Code that preserves the OpenClaw per-chat session model. Migration path for OpenClaw users moving to Claude Code.",
   "main": "lib/ipc-client.js",
   "bin": {

package/polygram.js

@@ -576,12 +576,32 @@ const inFlightHandlers = new Map(); // sessionKey → count
 // polygram is going down, in-flight handlers reject with "Process
 // killed" / "Process exited" but those failures aren't "real" — the
 // next boot's replay will re-dispatch them. Suppressing the user-facing
-// "Sorry, I couldn't process" during shutdown removes a misleading
-// post-mortem apology that the user shouldn't have seen in the first
-// place. (The boot replay's own _isReplay flag handles the OTHER half:
-// suppressing the apology if the replay itself fails.)
+// error reply during shutdown removes a misleading post-mortem apology
+// the user shouldn't see. (The boot replay's own _isReplay flag handles
+// the OTHER half: suppressing if the replay itself fails.)
 let isShuttingDown = false;
 
+// Map a handler-error to a user-facing reply that says what happened
+// and what to do next. The technical strings come from process-manager
+// (idle / wall-clock timeouts) and node child_process (Process exited /
+// killed). Anything we don't recognise falls back to a generic line
+// with a single-line snippet of the error so the user can at least
+// distinguish unique failures from the obvious "try again" cases.
+function errorReplyText(err) {
+  const msg = err?.message || '';
+  if (/idle with no Claude activity/i.test(msg)) {
+    return '⏳ I went quiet too long without finishing. Try resending or simplifying the task.';
+  }
+  if (/wall-clock ceiling/i.test(msg)) {
+    return '⏱ This was taking too long, so I stopped. Try resending or simplifying the task.';
+  }
+  if (/Process (exited|killed)/i.test(msg)) {
+    return '💥 Something crashed on my end. Try again.';
+  }
+  const reason = msg.split('\n')[0].slice(0, 120);
+  return `Hit a snag: ${reason || 'unknown error'}. Try resending.`;
+}
+
 // Sessions the operator just /stop'd (or natural-language "стоп"). Keyed
 // by sessionKey → timestamp of abort. ANY pending that rejects within
 // ABORT_GRACE_MS of the mark is considered abort-caused — its generic
@@ -638,7 +658,7 @@ function dispatchHandleMessage(sessionKey, chatId, msg, bot) {
       aborted: wasAborted || undefined,
       replay: isReplay || undefined,
     }), 'log handler-error');
-    // Suppress the "Sorry, I couldn't process" reply when:
+    // Suppress the user-facing error reply when:
     //  - boot replay (user typed this minutes ago and moved on)
     //  - polygram is shutting down (the failure is "Process killed" /
     //    "Process exited" which isn't a real error — boot replay will
@@ -647,7 +667,7 @@ function dispatchHandleMessage(sessionKey, chatId, msg, bot) {
     if (!wasAborted && !isReplay && !isShuttingDown) {
       tg(bot, 'sendMessage', {
         chat_id: chatId,
-        text: `Sorry, I couldn't process that message. The operator has been notified.`,
+        text: errorReplyText(err),
         reply_parameters: { message_id: msg.message_id },
       }, { source: 'error-reply', botName: BOT_NAME }).catch((replyErr) => {
         console.error(`[${sessionKey}] failed to send error reply: ${replyErr.message}`);