@hienlh/ppm 0.9.77 → 0.9.79

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [0.9.79] - 2026-04-10
+
+### Fixed
+- **Silent hang on resume of never-completed sessions**: When a session's first attempt hung (no SDK `init` event received), subsequent messages tried to `resume` a non-existent JSONL file, causing the SDK subprocess to silently hang. Now detects missing SDK mapping and treats as a new session.
+
+## [0.9.78] - 2026-04-10
+
+### Added
+- **SDK subprocess stderr logging**: Real-time stderr output from Claude CLI subprocess is now logged to server console for debugging hangs and crashes.
+- **MCP servers diagnostic log**: Log which MCP servers are being passed to SDK query.
+- **Query creation log**: Log when `query()` async iterator is created to confirm subprocess started.
+
 ## [0.9.77] - 2026-04-10
 
 ### Fixed

package/package.json

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

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

@@ -648,6 +648,14 @@ export class ClaudeAgentSdkProvider implements AIProvider {
       // Resolve SDK's actual session ID for resume (may differ from PPM's UUID)
       // For fork: use the source session's SDK id
       const sdkId = shouldFork ? getSdkSessionId(forkSourceId!) : getSdkSessionId(sessionId);
+      // If messageCount > 0 (resumeSession ran) but no real SDK mapping exists,
+      // the session never received an init event (prior attempt hung/crashed).
+      // Treat as first message to avoid resuming a non-existent JSONL → silent hang.
+      const hasRealSdkMapping = getSessionMapping(sessionId) !== null && getSessionMapping(sessionId) !== sessionId;
+      const effectiveIsFirst = isFirstMessage || (!shouldFork && !hasRealSdkMapping);
+      if (effectiveIsFirst && !isFirstMessage) {
+        console.warn(`[sdk] session=${sessionId} no SDK mapping found — treating as new session (was isFirst=false)`);
+      }
       // Fallback cwd: SDK needs a valid working directory even when no project is selected.
       // On Windows daemons, undefined cwd can cause the subprocess to fail silently.
       // Resolve path and validate existence — invalid cwd causes spawn to hang on Windows.
@@ -717,25 +725,29 @@ export class ClaudeAgentSdkProvider implements AIProvider {
           console.warn(`[sdk] session=${sessionId} no account and no API key in env — Claude CLI will use its own auth (if any)`);
         }
       }
-      console.log(`[sdk] query: session=${sessionId} sdkId=${sdkId} isFirst=${isFirstMessage} fork=${shouldFork} cwd=${effectiveCwd} platform=${process.platform} accountMode=${!!account} permissionMode=${permissionMode} isBypass=${isBypass}`);
+      console.log(`[sdk] query: session=${sessionId} sdkId=${sdkId} isFirst=${isFirstMessage} effectiveIsFirst=${effectiveIsFirst} fork=${shouldFork} cwd=${effectiveCwd} platform=${process.platform} accountMode=${!!account} permissionMode=${permissionMode} isBypass=${isBypass}`);
 
       // Read MCP servers from PPM DB (fresh per query — user may add/remove between chats)
       const mcpServers = mcpConfigService.list();
       const hasMcp = Object.keys(mcpServers).length > 0;
 
-      // Buffer subprocess stderr for crash diagnostics
+      // Buffer subprocess stderr for crash diagnostics + log in real-time
       let stderrBuffer = "";
       const stderrCallback = (chunk: string) => {
         stderrBuffer += chunk;
-        // Keep only last 2KB to avoid unbounded growth
         if (stderrBuffer.length > 2048) stderrBuffer = stderrBuffer.slice(-2048);
+        const trimmed = chunk.trim();
+        if (trimmed) console.log(`[sdk] session=${sessionId} stderr: ${trimmed.slice(0, 500)}`);
       };
+      if (hasMcp) {
+        console.log(`[sdk] session=${sessionId} mcpServers: ${Object.keys(mcpServers).join(", ")}`);
+      }
 
       const queryOptions: Record<string, any> = {
         // On Windows, child_process.spawn("bun") fails with ENOENT — force node
         ...(process.platform === "win32" && { executable: "node" }),
-        sessionId: isFirstMessage && !shouldFork ? sessionId : undefined,
-        resume: (isFirstMessage && !shouldFork) ? undefined : sdkId,
+        sessionId: effectiveIsFirst && !shouldFork ? sessionId : undefined,
+        resume: (effectiveIsFirst && !shouldFork) ? undefined : sdkId,
         ...(shouldFork && { forkSession: true }),
         cwd: effectiveCwd,
         systemPrompt: systemPromptOpt,
@@ -786,6 +798,7 @@ export class ClaudeAgentSdkProvider implements AIProvider {
       this.streamingSessions.set(sessionId, { meta, query: initialQuery, controller: initialCtrl });
       this.activeQueries.set(sessionId, initialQuery);
       let eventSource: AsyncIterable<any> = initialQuery;
+      console.log(`[sdk] session=${sessionId} query() created, waiting for first SDK event...`);
 
       // Helper: close the CURRENT streaming session (not stale closure refs).
       // All retry paths must use this instead of closing captured variables directly.