@hienlh/ppm 0.9.60 → 0.9.62

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## [0.9.62] - 2026-04-08
+
+### Fixed
+- **SDK subprocess crash auto-retry**: When the Claude Code subprocess crashes (exit code 1), PPM now automatically retries once with a fresh subprocess after a 1s delay, instead of immediately showing the error. Only surfaces the crash message if the retry also fails.
+
+## [0.9.61] - 2026-04-08
+
+### Fixed
+- **Telegram message clarity**: Thinking blocks now wrapped in `<blockquote>` (indented with vertical bar), tool calls in `<pre>` blocks (monospace background). Much easier to distinguish from actual response text.
+
 ## [0.9.60] - 2026-04-08
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hienlh/ppm",
-  "version": "0.9.60",
+  "version": "0.9.62",
   "description": "Personal Project Manager — mobile-first web IDE with AI assistance",
   "author": "hienlh",
   "license": "MIT",

package/src/providers/claude-agent-sdk.ts

@@ -702,6 +702,13 @@ export class ClaudeAgentSdkProvider implements AIProvider {
         includePartialMessages: true,
       };
 
+      // Crash retry: if subprocess exits with non-zero code before producing events,
+      // clean up and retry once with a fresh query before surfacing the error.
+      const MAX_CRASH_RETRIES = 1;
+      let crashRetryCount = 0;
+
+      crashRetryLoop: for (;;) {
+      try {
       // Streaming input: create message channel and persistent query
       const { generator: streamGen, controller: streamCtrl } = createMessageChannel();
       const firstMsg = {
@@ -1373,28 +1380,45 @@ export class ClaudeAgentSdkProvider implements AIProvider {
       }
       break; // Exit retryLoop — normal completion
       } // end retryLoop
-    } catch (e) {
-      const msg = (e as Error).message ?? String(e);
-      console.error(`[sdk] session=${sessionId} cwd=${meta.projectPath} error: ${msg}`);
-      if (msg.includes("abort") || msg.includes("closed")) {
+      break crashRetryLoop; // Normal completion — exit crash retry loop
+    } catch (crashErr) {
+      const crashMsg = (crashErr as Error).message ?? String(crashErr);
+      console.error(`[sdk] session=${sessionId} cwd=${meta.projectPath} error: ${crashMsg}`);
+
+      // Clean up crashed subprocess before retry or error
+      this.activeQueries.delete(sessionId);
+      const ss = this.streamingSessions.get(sessionId);
+      if (ss) { ss.controller.done(); ss.query.close(); this.streamingSessions.delete(sessionId); }
+      console.log(`[sdk] session=${sessionId} streaming session ended`);
+
+      if (crashMsg.includes("abort") || crashMsg.includes("closed")) {
         // User-initiated abort or WS closed — nothing to report
-      } else if (msg.includes("exited with code")) {
-        // Subprocess crashed — session will auto-recover on next message
-        console.warn(`[sdk] session=${sessionId} subprocess crashed: ${msg}`);
+      } else if (crashMsg.includes("exited with code") && crashRetryCount < MAX_CRASH_RETRIES) {
+        // Subprocess crashed — auto-retry once before surfacing the error
+        crashRetryCount++;
+        console.warn(`[sdk] session=${sessionId} subprocess crashed: ${crashMsg} — auto-retrying (attempt ${crashRetryCount}/${MAX_CRASH_RETRIES})`);
+        await new Promise((r) => setTimeout(r, 1000));
+        continue crashRetryLoop;
+      } else if (crashMsg.includes("exited with code")) {
+        console.warn(`[sdk] session=${sessionId} subprocess crashed after retry: ${crashMsg}`);
         yield { type: "error", message: `SDK subprocess crashed. Send another message to auto-recover.` };
       } else {
-        yield { type: "error", message: `SDK error: ${msg}` };
+        yield { type: "error", message: `SDK error: ${crashMsg}` };
       }
+      break crashRetryLoop; // Exit after error handling (non-retryable)
+    }
+    } // end crashRetryLoop
+
+    } catch (outerErr) {
+      // Setup errors (account auth, env) — not retryable
+      const msg = (outerErr as Error).message ?? String(outerErr);
+      console.error(`[sdk] session=${sessionId} setup error: ${msg}`);
+      yield { type: "error", message: `SDK error: ${msg}` };
     } finally {
+      // Final cleanup — ensure no leaked streaming session
       this.activeQueries.delete(sessionId);
-      // Properly close streaming session: terminate subprocess + generator
       const ss = this.streamingSessions.get(sessionId);
-      if (ss) {
-        ss.controller.done();
-        ss.query.close();
-        this.streamingSessions.delete(sessionId);
-      }
-      console.log(`[sdk] session=${sessionId} streaming session ended`);
+      if (ss) { ss.controller.done(); ss.query.close(); this.streamingSessions.delete(sessionId); }
     }
 
     // Final done event when query ends (crash, close, generator done)

package/src/services/ppmbot/ppmbot-streamer.ts

@@ -66,7 +66,7 @@ function renderSegments(segments: Segment[]): string {
   return segments
     .map((s) => {
       if (s.type === "md") return markdownToTelegramHtml(s.text);
-      if (s.type === "thinking") return `\n<i>💭 ${escapeHtml(s.text)}</i>\n`;
+      if (s.type === "thinking") return `\n<blockquote>💭 <i>${escapeHtml(s.text)}</i></blockquote>\n`;
       return s.text;
     })
     .join("");
@@ -192,7 +192,7 @@ export async function streamToTelegram(
             const inputPreview = formatToolInput(event.input);
             appendHtml(
               segments,
-              `\n🔧 <code>${escapeHtml(toolName)}</code>(${escapeHtml(inputPreview)})\n`,
+              `\n<pre>🔧 ${escapeHtml(toolName)}: ${escapeHtml(inputPreview)}</pre>\n`,
             );
             await editCurrent();
           }
@@ -203,7 +203,7 @@ export async function streamToTelegram(
           if (config.showToolCalls && event.isError) {
             appendHtml(
               segments,
-              `\n⚠️ <code>${escapeHtml(event.output.slice(0, 200))}</code>\n`,
+              `\n<pre>⚠️ ${escapeHtml(event.output.slice(0, 200))}</pre>\n`,
             );
             await editCurrent();
           }