@hienlh/ppm 0.8.93 → 0.8.95

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [0.8.95] - 2026-04-02
+
+### Fixed
+- **Streaming auth loop**: SDK auth errors in streaming mode don't emit result events, leaving the session alive with broken credentials — every follow-up message fails with 401 forever. Now breaks the loop, cooldowns the account, and tears down the session so the next message picks a different account.
+- **Streaming session resource leak**: `finally` block now properly closes SDK subprocess and generator instead of just removing from map.
+
+## [0.8.94] - 2026-04-02
+
+### Fixed
+- **Session JSONL "not found" after disconnect/restart**: Debug endpoint relied on in-memory `projectPath` which was lost when session cleanup timer expired or server restarted. Now persists `project_path` in `session_map` DB table (migration 11) and falls back to DB lookup when in-memory state is gone.
+- **Session mapping missing project info**: `setSessionMapping` now saves `projectName` and `projectPath` at both session creation and SDK init, ensuring resumed sessions can locate their JSONL files.
+
 ## [0.8.93] - 2026-04-02
 
 ### Removed

package/package.json

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

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

@@ -15,7 +15,7 @@ import type {
 import { configService } from "../services/config.service.ts";
 import { mcpConfigService } from "../services/mcp-config.service.ts";
 import { updateFromSdkEvent } from "../services/claude-usage.service.ts";
-import { getSessionMapping, setSessionMapping, getSessionTitles } from "../services/db.service.ts";
+import { getSessionMapping, getSessionProjectPath, setSessionMapping, getSessionTitles } from "../services/db.service.ts";
 import { accountSelector } from "../services/account-selector.service.ts";
 import { accountService } from "../services/account.service.ts";
 import { resolve } from "node:path";
@@ -244,6 +244,8 @@ export class ClaudeAgentSdkProvider implements AIProvider {
     };
     this.activeSessions.set(id, meta);
     this.messageCount.set(id, 0);
+    // Pre-persist mapping so project_path survives server restarts
+    setSessionMapping(id, id, config.projectName, config.projectPath);
     return meta;
   }
 
@@ -253,6 +255,8 @@ export class ClaudeAgentSdkProvider implements AIProvider {
 
     // Check if we have a mapped SDK session ID (from a previous query)
     const mappedSdkId = getSdkSessionId(sessionId);
+    // Restore project_path from DB so resumed sessions can find JSONL
+    const dbProjectPath = getSessionProjectPath(sessionId) ?? undefined;
 
     try {
       const sdkSessions = await sdkListSessions({ limit: 100 });
@@ -264,6 +268,7 @@ export class ClaudeAgentSdkProvider implements AIProvider {
           id: sessionId,
           providerId: this.id,
           title: found.customTitle ?? found.summary ?? "Resumed Chat",
+          projectPath: dbProjectPath,
           createdAt: new Date(found.lastModified).toISOString(),
         };
         this.activeSessions.set(sessionId, meta);
@@ -280,6 +285,7 @@ export class ClaudeAgentSdkProvider implements AIProvider {
       id: sessionId,
       providerId: this.id,
       title: "Resumed Chat",
+      projectPath: dbProjectPath,
       createdAt: new Date().toISOString(),
     };
     this.activeSessions.set(sessionId, meta);
@@ -730,7 +736,7 @@ export class ClaudeAgentSdkProvider implements AIProvider {
           if (subtype === "init") {
             const initMsg = msg as any;
             if (initMsg.session_id && initMsg.session_id !== sessionId) {
-              setSessionMapping(sessionId, initMsg.session_id);
+              setSessionMapping(sessionId, initMsg.session_id, meta.projectName, meta.projectPath);
               const oldMeta = this.activeSessions.get(sessionId);
               if (oldMeta) {
                 this.activeSessions.set(initMsg.session_id, { ...oldMeta, id: initMsg.session_id });
@@ -904,6 +910,18 @@ export class ClaudeAgentSdkProvider implements AIProvider {
               }
             }
 
+            // Auth failed permanently after retry — cooldown account and break loop.
+            // SDK doesn't send a result event after auth errors in streaming mode,
+            // so the streaming session would stay alive with broken credentials forever.
+            // Breaking here lets the finally block tear down the session, so the next
+            // user message creates a fresh session with a different account.
+            if (assistantError === "authentication_failed" && account && authRetried) {
+              accountSelector.onAuthError(account.id);
+              console.warn(`[sdk] session=${sessionId} auth permanently failed — tearing down streaming session`);
+              yield { type: "error", message: "API authentication failed. Check your account credentials in Settings → Accounts." };
+              break;
+            }
+
             const errorHints: Record<string, string> = {
               authentication_failed: "API authentication failed. Check your account credentials in Settings → Accounts.",
               billing_error: "Billing error on this account. Check your subscription status.",
@@ -1152,7 +1170,13 @@ export class ClaudeAgentSdkProvider implements AIProvider {
       }
     } finally {
       this.activeQueries.delete(sessionId);
-      this.streamingSessions.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`);
     }
 

package/src/server/routes/chat.ts

@@ -8,7 +8,7 @@ import { renameSession as sdkRenameSession } from "@anthropic-ai/claude-agent-sd
 import { listSlashItems } from "../../services/slash-items.service.ts";
 import { getCachedUsage, refreshUsageNow } from "../../services/claude-usage.service.ts";
 import { getSessionLog } from "../../services/session-log.service.ts";
-import { getSessionMapping, setSessionMapping, setSessionTitle, getPinnedSessionIds, pinSession, unpinSession, deleteSessionMapping, deleteSessionTitle } from "../../services/db.service.ts";
+import { getSessionMapping, getSessionProjectPath, setSessionMapping, setSessionTitle, getPinnedSessionIds, pinSession, unpinSession, deleteSessionMapping, deleteSessionTitle } from "../../services/db.service.ts";
 import { ok, err } from "../../types/api.ts";
 
 type Env = { Variables: { projectPath: string; projectName: string } };
@@ -236,11 +236,13 @@ chatRoutes.get("/sessions/:id/logs", (c) => {
 chatRoutes.get("/sessions/:id/debug", (c) => {
   const ppmId = c.req.param("id");
   const sdkId = getSessionMapping(ppmId) ?? ppmId;
-  const projectName = c.req.query("project") ?? "";
   // Resolve JSONL path: ~/.claude/projects/<encoded-cwd>/<sdkId>.jsonl
   const homedir = process.env.HOME ?? process.env.USERPROFILE ?? "";
   const provider = providerRegistry.get("claude") as any;
-  const projectPath = provider?.activeSessions?.get(ppmId)?.projectPath ?? "";
+  // Try in-memory first, fall back to DB-persisted project_path
+  const projectPath = provider?.activeSessions?.get(ppmId)?.projectPath
+    ?? getSessionProjectPath(ppmId)
+    ?? "";
   const encodedCwd = projectPath ? projectPath.replace(/\//g, "-") : "";
   const jsonlDir = encodedCwd ? resolve(homedir, ".claude", "projects", encodedCwd) : "";
   const jsonlPath = jsonlDir ? resolve(jsonlDir, `${sdkId}.jsonl`) : "";

package/src/services/db.service.ts

@@ -271,6 +271,21 @@ function runMigrations(database: Database): void {
       PRAGMA user_version = 10;
     `);
   }
+
+  if (current < 11) {
+    try {
+      database.exec(`ALTER TABLE session_map ADD COLUMN project_path TEXT`);
+    } catch {
+      // Column may already exist
+    }
+    // Backfill project_path from projects table where project_name matches
+    database.exec(`
+      UPDATE session_map SET project_path = (
+        SELECT path FROM projects WHERE projects.name = session_map.project_name
+      ) WHERE project_path IS NULL AND project_name IS NOT NULL
+    `);
+    database.exec(`PRAGMA user_version = 11`);
+  }
 }
 
 // ---------------------------------------------------------------------------
@@ -365,10 +380,15 @@ export function getSessionMapping(ppmId: string): string | null {
   return row?.sdk_id ?? null;
 }
 
-export function setSessionMapping(ppmId: string, sdkId: string, projectName?: string): void {
+export function getSessionProjectPath(ppmId: string): string | null {
+  const row = getDb().query("SELECT project_path FROM session_map WHERE ppm_id = ?").get(ppmId) as { project_path: string } | null;
+  return row?.project_path ?? null;
+}
+
+export function setSessionMapping(ppmId: string, sdkId: string, projectName?: string, projectPath?: string): void {
   getDb().query(
-    "INSERT INTO session_map (ppm_id, sdk_id, project_name) VALUES (?, ?, ?) ON CONFLICT(ppm_id) DO UPDATE SET sdk_id = excluded.sdk_id, project_name = excluded.project_name",
-  ).run(ppmId, sdkId, projectName ?? null);
+    "INSERT INTO session_map (ppm_id, sdk_id, project_name, project_path) VALUES (?, ?, ?, ?) ON CONFLICT(ppm_id) DO UPDATE SET sdk_id = excluded.sdk_id, project_name = COALESCE(excluded.project_name, session_map.project_name), project_path = COALESCE(excluded.project_path, session_map.project_path)",
+  ).run(ppmId, sdkId, projectName ?? null, projectPath ?? null);
 }
 
 export function getAllSessionMappings(): Record<string, string> {