@hienlh/ppm 0.9.37 → 0.9.39

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [0.9.39] - 2026-04-06
+
+### Fixed
+- **Stream hangs 3 minutes after AI finishes**: `done` event was received but the `for-await` loop didn't break — `break` inside `switch` only exits the switch, not the loop. Used labeled `break eventLoop` to properly terminate on `done`.
+- **Identity never saved to memory**: Memory extraction only ran on session end. Now saves identity directly when user responds to onboarding prompt, and runs AI extraction every 5 messages.
+
 ## [0.9.36] - 2026-04-06
 
 ### Fixed

package/package.json

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

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

@@ -36,6 +36,15 @@ class PPMBotService {
   /** Sessions that already had their title set */
   private titledSessions = new Set<string>();
 
+  /** Chat IDs that just received identity onboarding prompt */
+  private identityPending = new Set<string>();
+
+  /** Message count per session for periodic memory save */
+  private messageCount = new Map<string, number>();
+
+  /** Interval (messages) between automatic memory saves */
+  private readonly MEMORY_SAVE_INTERVAL = 5;
+
   // ── Lifecycle ─────────────────────────────────────────────────
 
   async start(): Promise<void> {
@@ -77,6 +86,8 @@ class PPMBotService {
     this.debouncedTexts.clear();
     this.processing.clear();
     this.messageQueue.clear();
+    this.identityPending.clear();
+    this.messageCount.clear();
 
     console.log("[ppmbot] Stopped");
   }
@@ -192,6 +203,7 @@ class PPMBotService {
     // Identity onboarding: if no identity memories exist, ask user
     const identityMemories = this.memory.recall("_global", "user identity name role");
     if (identityMemories.length === 0) {
+      this.identityPending.add(chatId);
       await this.telegram!.sendMessage(
         Number(chatId),
         "📝 <b>Quick intro?</b>\n\n" +
@@ -430,6 +442,22 @@ class PPMBotService {
         },
       );
 
+      // Capture identity if onboarding was just shown
+      if (this.identityPending.has(chatId)) {
+        this.identityPending.delete(chatId);
+        this.memory.saveOne("_global", `User identity: ${text}`, "preference", session.sessionId);
+        console.log("[ppmbot] Saved identity memory from onboarding");
+      }
+
+      // Periodic memory extraction — fire-and-forget every N messages
+      const count = (this.messageCount.get(session.sessionId) ?? 0) + 1;
+      this.messageCount.set(session.sessionId, count);
+      if (count % this.MEMORY_SAVE_INTERVAL === 0) {
+        this.saveSessionMemory(chatId).catch((err) =>
+          console.warn("[ppmbot] Periodic memory save failed:", (err as Error).message),
+        );
+      }
+
       // Check context window — auto-rotate if near limit
       if (
         result.contextWindowPct != null &&

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

@@ -151,7 +151,7 @@ export async function streamToTelegram(
   // Process event stream with per-event timeout
   let eventCount = 0;
   try {
-    for await (const event of withEventTimeout(events, EVENT_TIMEOUT_MS)) {
+    eventLoop: for await (const event of withEventTimeout(events, EVENT_TIMEOUT_MS)) {
       eventCount++;
       // Debug: log each event type to help diagnose streaming issues
       if (event.type !== "text") {
@@ -207,7 +207,7 @@ export async function streamToTelegram(
         case "done": {
           result.contextWindowPct = event.contextWindowPct;
           result.resultSubtype = event.resultSubtype;
-          break;
+          break eventLoop; // break the for-await, not just the switch
         }
 
         case "session_migrated": {