prjct-cli 1.6.9 → 1.6.10

package/CHANGELOG.md

@@ -1,12 +1,44 @@
 # Changelog
 
+## [1.6.10] - 2026-02-07
+
+### Bug Fixes
+
+- resolve signal handler and EventBus listener accumulation leaks (PRJ-287) (#135)
+
+
+## [1.6.12] - 2026-02-07
+
+### Bug Fixes
+- **Fix signal handler and EventBus listener accumulation leaks (PRJ-287)**: WatchService signal handlers (`SIGINT`/`SIGTERM`) are now stored by reference and removed in `stop()`, preventing accumulation on restart cycles. `pendingChanges` Set is cleared on stop. EventBus gains `flush()` to clear history and stale once-listeners, and `removeAllListeners(event?)` for targeted cleanup.
+
+### Implementation Details
+Stored signal handler references as class properties (`sigintHandler`, `sigtermHandler`). In `start()`, old handlers are removed before new ones are added. In `stop()`, handlers are removed via `process.off()` and `pendingChanges` is cleared. EventBus `flush()` clears history array and all once-listeners. `removeAllListeners()` supports both targeted (single event) and global cleanup.
+
+### Learnings
+- Arrow functions passed to `process.on()` cannot be removed — must store named handler references for `process.off()`.
+- Cleanup code after `process.exit(0)` is unreachable — perform all cleanup before the exit call.
+
+### Test Plan
+
+#### For QA
+1. Start/stop watch mode 10 times — verify only 2 signal handlers (not 20)
+2. Trigger file changes, stop — verify `pendingChanges` cleared
+3. Emit 50 events, call `flush()` — verify history empty
+4. Register `once()` for unfired event, `flush()` — verify listener removed
+5. `removeAllListeners('event')` — verify only that event cleared
+6. `removeAllListeners()` — verify all cleared
+
+#### For Users
+**What changed:** WatchService no longer leaks signal handlers on restart. EventBus has `flush()` and `removeAllListeners()`.
+**Breaking changes:** None.
+
 ## [1.6.9] - 2026-02-07
 
 ### Bug Fixes
 
 - resolve SSE zombie connections and infinite promise leak (PRJ-286) (#134)
 
-
 ## [1.6.11] - 2026-02-07
 
 ### Bug Fixes

package/core/bus/bus.ts

@@ -237,6 +237,30 @@ class EventBus {
     this.onceListeners.clear()
   }
 
+  /**
+   * Flush event history and clean up stale once-listeners.
+   * Call on task completion, project switch, or periodically.
+   */
+  flush(): void {
+    this.history = []
+
+    // Remove once-listeners for events that were never fired
+    this.onceListeners.clear()
+  }
+
+  /**
+   * Remove all listeners for a specific event, or all events if none specified.
+   */
+  removeAllListeners(event?: string): void {
+    if (event) {
+      this.listeners.delete(event)
+      this.onceListeners.delete(event)
+    } else {
+      this.listeners.clear()
+      this.onceListeners.clear()
+    }
+  }
+
   /**
    * Get count of listeners for an event
    */

package/core/services/watch-service.ts

@@ -102,6 +102,8 @@ class WatchService {
   }
   private isRunning: boolean = false
   private syncCount: number = 0
+  private sigintHandler: (() => void) | null = null
+  private sigtermHandler: (() => void) | null = null
 
   /**
    * Start watching for file changes
@@ -151,9 +153,13 @@ class WatchService {
       .on('unlink', (filePath: string) => this.handleChange('unlink', filePath))
       .on('error', (error: unknown) => this.handleError(error as Error))
 
-    // Handle graceful shutdown
-    process.on('SIGINT', () => this.stop())
-    process.on('SIGTERM', () => this.stop())
+    // Handle graceful shutdown — store references for proper removal
+    if (this.sigintHandler) process.off('SIGINT', this.sigintHandler)
+    if (this.sigtermHandler) process.off('SIGTERM', this.sigtermHandler)
+    this.sigintHandler = () => this.stop()
+    this.sigtermHandler = () => this.stop()
+    process.on('SIGINT', this.sigintHandler)
+    process.on('SIGTERM', this.sigtermHandler)
 
     return { success: true }
   }
@@ -177,6 +183,17 @@ class WatchService {
       this.watcher = null
     }
 
+    // Remove signal handlers to prevent accumulation
+    if (this.sigintHandler) {
+      process.off('SIGINT', this.sigintHandler)
+      this.sigintHandler = null
+    }
+    if (this.sigtermHandler) {
+      process.off('SIGTERM', this.sigtermHandler)
+      this.sigtermHandler = null
+    }
+
+    this.pendingChanges.clear()
     this.isRunning = false
     process.exit(0)
   }

package/dist/bin/prjct.mjs

@@ -23753,6 +23753,8 @@ var init_watch_service = __esm({
       };
       isRunning = false;
       syncCount = 0;
+      sigintHandler = null;
+      sigtermHandler = null;
       /**
        * Start watching for file changes
        */
@@ -23783,8 +23785,12 @@ var init_watch_service = __esm({
           }
         });
         this.watcher.on("add", (filePath) => this.handleChange("add", filePath)).on("change", (filePath) => this.handleChange("change", filePath)).on("unlink", (filePath) => this.handleChange("unlink", filePath)).on("error", (error) => this.handleError(error));
-        process.on("SIGINT", () => this.stop());
-        process.on("SIGTERM", () => this.stop());
+        if (this.sigintHandler) process.off("SIGINT", this.sigintHandler);
+        if (this.sigtermHandler) process.off("SIGTERM", this.sigtermHandler);
+        this.sigintHandler = () => this.stop();
+        this.sigtermHandler = () => this.stop();
+        process.on("SIGINT", this.sigintHandler);
+        process.on("SIGTERM", this.sigtermHandler);
         return { success: true };
       }
       /**
@@ -23804,6 +23810,15 @@ var init_watch_service = __esm({
           await this.watcher.close();
           this.watcher = null;
         }
+        if (this.sigintHandler) {
+          process.off("SIGINT", this.sigintHandler);
+          this.sigintHandler = null;
+        }
+        if (this.sigtermHandler) {
+          process.off("SIGTERM", this.sigtermHandler);
+          this.sigtermHandler = null;
+        }
+        this.pendingChanges.clear();
         this.isRunning = false;
         process.exit(0);
       }
@@ -28649,7 +28664,7 @@ var require_package = __commonJS({
   "package.json"(exports, module) {
     module.exports = {
       name: "prjct-cli",
-      version: "1.6.9",
+      version: "1.6.10",
       description: "Context layer for AI agents. Project context for Claude Code, Gemini CLI, and more.",
       main: "core/index.ts",
       bin: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prjct-cli",
-  "version": "1.6.9",
+  "version": "1.6.10",
   "description": "Context layer for AI agents. Project context for Claude Code, Gemini CLI, and more.",
   "main": "core/index.ts",
   "bin": {