@hienlh/ppm 0.8.72 → 0.8.74

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [0.8.74] - 2026-04-01
+
+### Fixed
+- **Cloud WS reconnect loop**: Stale WebSocket closure handlers from replaced connections no longer reset module state
+- **Cloud WS auth race**: Delay heartbeat/queue flush 500ms after auth to let server complete async DB auth — prevents 4002 rejection
+
 ## [0.8.72] - 2026-03-31
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hienlh/ppm",
-  "version": "0.8.72",
+  "version": "0.8.74",
   "description": "Personal Project Manager — mobile-first web IDE with AI assistance",
   "author": "hienlh",
   "license": "MIT",
@@ -11,7 +11,7 @@
   },
   "scripts": {
     "dev": "concurrently \"bun run dev:server\" \"bun run dev:web\"",
-    "dev:server": "bun run --hot src/index.ts start --profile dev -f",
+    "dev:server": "bun run --hot src/server/index.ts __serve__ 8081 0.0.0.0 '' dev",
     "dev:web": "bun run vite --config vite.config.ts",
     "build:web": "bun run vite build --config vite.config.ts",
     "build": "bun run build:web && bun build src/index.ts --compile --outfile dist/ppm",

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

@@ -13,7 +13,7 @@ import type {
 } from "./provider.interface.ts";
 import { configService } from "../services/config.service.ts";
 import { updateFromSdkEvent } from "../services/claude-usage.service.ts";
-import { getSessionMapping, setSessionMapping, getSessionTitles } from "../services/db.service.ts";
+import { getSessionMapping, setSessionMapping, getSessionTitles, getSessionTitle } 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";
@@ -176,10 +176,11 @@ export class ClaudeAgentSdkProvider implements AIProvider {
         (s) => s.sessionId === sessionId || s.sessionId === mappedSdkId,
       );
       if (found) {
+        const dbTitle = getSessionTitle(found.sessionId);
         const meta: Session = {
           id: sessionId,
           providerId: this.id,
-          title: found.customTitle ?? found.summary ?? "Resumed Chat",
+          title: dbTitle ?? found.customTitle ?? found.summary ?? "Resumed Chat",
           createdAt: new Date(found.lastModified).toISOString(),
         };
         this.activeSessions.set(sessionId, meta);

package/src/services/cloud-ws.service.ts

@@ -125,20 +125,26 @@ function doConnect(): void {
   if (!shouldConnect || reconnecting) return;
   reconnecting = true;
 
+  // Capture local ref — if a reconnect replaces `ws` before this socket's
+  // handlers fire, stale handlers must not reset module-level state.
+  let sock: WebSocket;
   try {
-    ws = new WebSocket(wsUrl);
+    sock = new WebSocket(wsUrl);
+    ws = sock;
   } catch {
     reconnecting = false;
-    scheduleReconnect();
+    scheduleReconnect("constructor");
     return;
   }
 
-  ws.onopen = () => {
+  sock.onopen = () => {
+    if (ws !== sock) return; // stale — newer connection replaced us
     reconnecting = false;
+    reconnectAttempt = 0;
     log("INFO", "Cloud WS connected, sending auth");
 
-    // Send auth as first message (not in URL)
-    ws!.send(JSON.stringify({
+    // Send auth as first message — server must process this before any other msg
+    sock.send(JSON.stringify({
       type: "auth",
       deviceId,
       secretKey,
@@ -146,26 +152,31 @@ function doConnect(): void {
       version: 1,
     }));
 
-    connected = true;
-    reconnectAttempt = 0;
-
-    // Flush queued messages
-    while (outboundQueue.length > 0 && connected) {
-      const msg = outboundQueue.shift()!;
-      ws!.send(JSON.stringify(msg));
-    }
+    // Delay setting connected + sending heartbeat to let server process auth.
+    // Server's authenticateDevice() is async (DB lookup), so messages sent
+    // immediately after auth arrive before authenticated=true → 4002 reject.
+    setTimeout(() => {
+      if (ws !== sock) return; // replaced during delay
+      connected = true;
+
+      // Flush queued messages
+      while (outboundQueue.length > 0 && connected) {
+        const msg = outboundQueue.shift()!;
+        sock.send(JSON.stringify(msg));
+      }
 
-    // Send immediate heartbeat
-    if (getHeartbeatData) send(getHeartbeatData());
+      // Send immediate heartbeat
+      if (getHeartbeatData) send(getHeartbeatData());
 
-    // Start periodic heartbeat
-    if (heartbeatTimer) clearInterval(heartbeatTimer);
-    heartbeatTimer = setInterval(() => {
-      if (getHeartbeatData && connected) send(getHeartbeatData());
-    }, HEARTBEAT_INTERVAL_MS);
+      // Start periodic heartbeat
+      if (heartbeatTimer) clearInterval(heartbeatTimer);
+      heartbeatTimer = setInterval(() => {
+        if (getHeartbeatData && connected) send(getHeartbeatData());
+      }, HEARTBEAT_INTERVAL_MS);
+    }, 500); // 500ms for DB auth round-trip
   };
 
-  ws.onmessage = (event) => {
+  sock.onmessage = (event) => {
     try {
       const msg = JSON.parse(String(event.data)) as CommandMsg;
       if (msg.type === "command" && commandHandler) {
@@ -174,27 +185,29 @@ function doConnect(): void {
     } catch {} // ignore malformed
   };
 
-  ws.onclose = () => {
+  sock.onclose = (event) => {
+    if (ws !== sock) return; // stale — ignore close from replaced connection
+    log("WARN", `Cloud WS closed: code=${event.code} reason=${event.reason || ""}`);
     connected = false;
     reconnecting = false;
     ws = null;
     if (heartbeatTimer) { clearInterval(heartbeatTimer); heartbeatTimer = null; }
-    if (shouldConnect) scheduleReconnect();
+    if (shouldConnect) scheduleReconnect("onclose");
   };
 
-  ws.onerror = () => {
-    // onclose will fire after onerror — reconnect handled there
+  sock.onerror = (event) => {
+    log("ERROR", `Cloud WS error: ${String(event)}`);
   };
 }
 
-function scheduleReconnect(): void {
+function scheduleReconnect(source = "unknown"): void {
   if (!shouldConnect || reconnectTimer) return;
   const base = BACKOFF_STEPS[Math.min(reconnectAttempt, BACKOFF_STEPS.length - 1)]!;
   // Add ±30% jitter to prevent thundering herd after Cloud deploy
   const jitter = base * (0.7 + Math.random() * 0.6);
   const delay = Math.round(jitter);
   reconnectAttempt++;
-  log("WARN", `Cloud WS reconnect in ${delay}ms (attempt #${reconnectAttempt})`);
+  log("WARN", `Cloud WS reconnect in ${delay}ms (attempt #${reconnectAttempt}) src=${source}`);
   reconnectTimer = setTimeout(() => {
     reconnectTimer = null;
     doConnect();