@hienlh/ppm 0.8.13 → 0.8.14

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## [0.8.14] - 2026-03-24
+
+### Fixed
+- **CLI error logging**: Always read stderr (even on exit code 0), log error event content to server logs for Windows CLI debugging
+
 ## [0.8.13] - 2026-03-24
 
 ### Added

package/package.json

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

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

@@ -231,6 +231,12 @@ export class ClaudeAgentSdkProvider implements AIProvider {
                 }
               }
             }
+            // Log error/result events for diagnostics
+            if (event.type === "error") {
+              console.error(`[sdk-cli] error event: ${JSON.stringify(event).slice(0, 1000)}`);
+            } else if (event.type === "result" && event.is_error) {
+              console.error(`[sdk-cli] result error: ${JSON.stringify(event).slice(0, 1000)}`);
+            }
             // Always yield the original event too (for init, result, rate_limit, etc.)
             yield event;
           } catch {
@@ -248,22 +254,20 @@ export class ClaudeAgentSdkProvider implements AIProvider {
       const exitCode = await proc.exited;
       console.log(`[sdk-cli] process exited: code=${exitCode}`);
 
-      // Read all stderr and surface error to frontend
-      if (exitCode !== 0) {
-        try {
-          const errReader = proc.stderr.getReader();
-          const stderrDecoder = new TextDecoder();
-          const errParts: string[] = [];
-          while (true) {
-            const { done, value } = await errReader.read();
-            if (done) break;
-            if (value) errParts.push(stderrDecoder.decode(value, { stream: true }));
-          }
-          const fullStderr = errParts.join("").trim();
-          if (fullStderr) {
-            // Actual error is at the END (minified source dump comes first)
-            console.error(`[sdk-cli] stderr (last 1500): ${fullStderr.slice(-1500)}`);
-            // Extract meaningful error line (e.g. "Error: ...", "TypeError: ...")
+      // Always read stderr for diagnostics (errors can occur even with exit code 0)
+      try {
+        const errReader = proc.stderr.getReader();
+        const stderrDecoder = new TextDecoder();
+        const errParts: string[] = [];
+        while (true) {
+          const { done, value } = await errReader.read();
+          if (done) break;
+          if (value) errParts.push(stderrDecoder.decode(value, { stream: true }));
+        }
+        const fullStderr = errParts.join("").trim();
+        if (fullStderr) {
+          console.error(`[sdk-cli] stderr (last 1500): ${fullStderr.slice(-1500)}`);
+          if (exitCode !== 0) {
             const errMatch = fullStderr.match(/\b(?:Error|TypeError|SyntaxError|ReferenceError|RangeError):\s*.+/);
             const errorMsg = errMatch ? errMatch[0].slice(0, 500) : fullStderr.slice(-300);
             yield {
@@ -272,8 +276,8 @@ export class ClaudeAgentSdkProvider implements AIProvider {
               error: `CLI exited with code ${exitCode}: ${errorMsg}`,
             };
           }
-        } catch {}
-      }
+        }
+      } catch {}
     } finally {
       this.activeQueries.delete(opts.sessionId);
       try { proc.kill(); } catch {}

package/src/server/ws/chat.ts

@@ -174,7 +174,9 @@ async function runStreamLoop(sessionId: string, providerId: string, content: str
       } else if (evType === "tool_result") {
         logSessionEvent(sessionId, "TOOL_RESULT", `error=${ev.isError ?? false} ${(ev.output ?? "").slice(0, 300)}`);
       } else if (evType === "error") {
-        logSessionEvent(sessionId, "ERROR", ev.message ?? JSON.stringify(ev).slice(0, 300));
+        const errorDetail = ev.message ?? JSON.stringify(ev).slice(0, 500);
+        console.error(`[chat] session=${sessionId} error: ${errorDetail}`);
+        logSessionEvent(sessionId, "ERROR", errorDetail);
       } else if (evType === "done") {
         logSessionEvent(sessionId, "DONE", `subtype=${ev.resultSubtype ?? "none"} turns=${ev.numTurns ?? "?"} ctx=${ev.contextWindowPct ?? "?"}%`);
         if (ev.contextWindowPct != null) lastContextWindowPct = ev.contextWindowPct;

package/src/services/file.service.ts

@@ -9,7 +9,7 @@ import {
   rmSync,
   renameSync,
 } from "node:fs";
-import { resolve, relative, basename, dirname, join, normalize } from "node:path";
+import { resolve, relative, basename, dirname, join, normalize, sep } from "node:path";
 import ignore, { type Ignore } from "ignore";
 import type { FileNode } from "../types/project.ts";
 
@@ -43,7 +43,7 @@ class FileService {
     const normalizedTarget = normalize(resolve(projectPath, targetPath));
     const normalizedProject = normalize(projectPath);
     if (
-      !normalizedTarget.startsWith(normalizedProject + "/") &&
+      !normalizedTarget.startsWith(normalizedProject + sep) &&
       normalizedTarget !== normalizedProject
     ) {
       throw new SecurityError("Path traversal not allowed");