getpatter 0.6.3 → 0.6.5

package/dist/cli.js

@@ -185,14 +185,49 @@ var MetricsStore = class extends import_events.EventEmitter {
     } else {
       for (let i = this.calls.length - 1; i >= 0; i--) {
         if (this.calls[i].call_id === callId) {
-          this.calls[i].status = status;
-          Object.assign(this.calls[i], extra);
+          this.calls[i] = { ...this.calls[i], status, ...extra };
           break;
         }
       }
     }
     this.publish("call_status", { call_id: callId, status, ...extra });
   }
+  /**
+   * Record a single transcript line (user/assistant) as it becomes known.
+   *
+   * FIX-5 (issue #154): the live forward path for the dashboard transcript.
+   * The Realtime stream handler calls this the moment each line is known — the
+   * user line right after the hallucination filter accepts it, the assistant
+   * line when its turn flushes — keyed by the monotonic ``turnIndex`` reserved
+   * at turn-open (``reserveTurnIndex``). Each line is appended to the active
+   * call's ``transcript`` array and broadcast over SSE as a ``transcript_line``
+   * event so the dashboard can render lines as they arrive and re-sort by
+   * ``(turnIndex, user<assistant)`` — making a late-arriving user line land
+   * ABOVE its agent line. ``recordTurn`` de-dups against the lines pushed here
+   * by ``(turnIndex, role)`` so the metrics path never double-pushes the same
+   * text. Parity with Python ``record_transcript_line``.
+   */
+  recordTranscriptLine(data) {
+    const callId = data.call_id || "";
+    const { role, text, turnIndex } = data;
+    if (!callId || role !== "user" && role !== "assistant" || !text) return;
+    const active = this.activeCalls.get(callId);
+    if (active) {
+      if (!active.transcript) active.transcript = [];
+      active.transcript.push({
+        role,
+        text,
+        timestamp: Date.now() / 1e3,
+        turnIndex
+      });
+    }
+    this.publish("transcript_line", {
+      call_id: callId,
+      turnIndex,
+      role,
+      text
+    });
+  }
   /** Append a single conversation turn to an active call and broadcast it via SSE. */
   recordTurn(data) {
     const callId = data.call_id || "";
@@ -207,14 +242,19 @@ var MetricsStore = class extends import_events.EventEmitter {
       const userText = typeof turnRecord.user_text === "string" ? turnRecord.user_text : "";
       const agentText = typeof turnRecord.agent_text === "string" ? turnRecord.agent_text : "";
       const ts = typeof turnRecord.timestamp === "number" ? turnRecord.timestamp : Date.now() / 1e3;
-      if (userText.length > 0) {
-        active.transcript.push({ role: "user", text: userText, timestamp: ts });
+      const turnIndex = typeof turnRecord.turn_index === "number" ? turnRecord.turn_index : void 0;
+      const alreadyLive = (role) => turnIndex !== void 0 && (active.transcript ?? []).some(
+        (e) => e.turnIndex === turnIndex && e.role === role
+      );
+      if (userText.length > 0 && !alreadyLive("user")) {
+        active.transcript.push({ role: "user", text: userText, timestamp: ts, turnIndex });
       }
-      if (agentText.length > 0 && agentText !== "[interrupted]") {
+      if (agentText.length > 0 && agentText !== "[interrupted]" && !alreadyLive("assistant")) {
         active.transcript.push({
           role: "assistant",
           text: agentText,
-          timestamp: ts
+          timestamp: ts,
+          turnIndex
         });
       }
     }
@@ -287,7 +327,7 @@ var MetricsStore = class extends import_events.EventEmitter {
   getCall(callId) {
     if (this.deletedCallIds.has(callId)) return null;
     for (let i = this.calls.length - 1; i >= 0; i--) {
-      if (this.calls[i].call_id === callId) return this.calls[i];
+      if (this.calls[i].call_id === callId) return { ...this.calls[i] };
     }
     return null;
   }
@@ -329,7 +369,9 @@ var MetricsStore = class extends import_events.EventEmitter {
     }
     if (accepted.length === 0) return [];
     accepted.sort();
-    this.persistDeletedIds();
+    this.persistDeletedIds().catch(
+      (err) => getLogger().debug(`MetricsStore.deleteCalls: persistDeletedIds failed: ${String(err)}`)
+    );
     this.publish("calls_deleted", { call_ids: accepted });
     return accepted;
   }
@@ -341,19 +383,19 @@ var MetricsStore = class extends import_events.EventEmitter {
   getDeletedCallIds() {
     return Array.from(this.deletedCallIds).sort();
   }
-  /** Atomically persist the deleted-ids set to disk. Best-effort. */
-  persistDeletedIds() {
+  /** Atomically persist the deleted-ids set to disk. Best-effort async. */
+  async persistDeletedIds() {
     if (this.deletedIdsPath === null) return;
     try {
       const dir = path2.dirname(this.deletedIdsPath);
-      fs2.mkdirSync(dir, { recursive: true });
+      await fs2.promises.mkdir(dir, { recursive: true });
       const tmp = this.deletedIdsPath + ".tmp";
       const payload = {
         version: 1,
         deleted_call_ids: Array.from(this.deletedCallIds).sort()
       };
-      fs2.writeFileSync(tmp, JSON.stringify(payload, null, 2), "utf8");
-      fs2.renameSync(tmp, this.deletedIdsPath);
+      await fs2.promises.writeFile(tmp, JSON.stringify(payload, null, 2), "utf8");
+      await fs2.promises.rename(tmp, this.deletedIdsPath);
     } catch (err) {
       getLogger().debug(
         `MetricsStore.persistDeletedIds: ${String(err)}`
@@ -362,7 +404,8 @@ var MetricsStore = class extends import_events.EventEmitter {
   }
   /** Look up an active call by id (returns undefined if not active or unknown). */
   getActive(callId) {
-    return this.activeCalls.get(callId);
+    const rec = this.activeCalls.get(callId);
+    return rec !== void 0 ? { ...rec } : void 0;
   }
   /** Return all currently active (not yet ended) calls. */
   getActiveCalls() {
@@ -607,8 +650,8 @@ function loadTranscriptJsonl(filePath) {
       } catch {
         continue;
       }
-      const tsIso = typeof row.ts === "string" ? Date.parse(row.ts) : NaN;
-      const tsNumeric = typeof row.timestamp === "number" ? row.timestamp * 1e3 : NaN;
+      const tsIso = typeof row.ts === "string" ? Date.parse(row.ts) / 1e3 : NaN;
+      const tsNumeric = typeof row.timestamp === "number" ? row.timestamp : NaN;
       const timestamp = Number.isFinite(tsIso) ? tsIso : Number.isFinite(tsNumeric) ? tsNumeric : 0;
       const userText = typeof row.user_text === "string" ? row.user_text : "";
       const agentText = typeof row.agent_text === "string" ? row.agent_text : "";
@@ -759,8 +802,8 @@ function mountDashboard(app, store, token = "") {
     res.type("text/html").send(DASHBOARD_HTML);
   });
   app.get("/api/dashboard/calls", auth, (req, res) => {
-    const limit = Math.min(parseInt(req.query.limit || "50", 10) || 50, 1e3);
-    const offset = parseInt(req.query.offset || "0", 10) || 0;
+    const limit = Math.min(Math.max(0, parseInt(req.query.limit || "50", 10) || 50), 1e3);
+    const offset = Math.max(0, parseInt(req.query.offset || "0", 10) || 0);
     res.json(store.getCalls(limit, offset));
   });
   app.get("/api/dashboard/calls/:callId", auth, (req, res) => {
@@ -850,8 +893,8 @@ data: ${data}
 function mountApi(app, store, token = "") {
   const auth = makeAuthMiddleware(token);
   app.get("/api/v1/calls", auth, (req, res) => {
-    const limit = Math.min(parseInt(req.query.limit || "50", 10) || 50, 1e3);
-    const offset = parseInt(req.query.offset || "0", 10) || 0;
+    const limit = Math.min(Math.max(0, parseInt(req.query.limit || "50", 10) || 50), 1e3);
+    const offset = Math.max(0, parseInt(req.query.offset || "0", 10) || 0);
     const calls = store.getCalls(limit, offset);
     res.json({
       data: calls,