atom-agent 1.5.0 → 1.5.1

package/dist/telemetry.js

@@ -55,7 +55,7 @@ import { randomUUID } from "node:crypto";
 import { fileURLToPath } from "node:url";
 import { scrubSecrets } from "./policy.js";
 import { atomDir } from "./auth.js";
-export const TELEMETRY_VERSION = 1;
+export const TELEMETRY_VERSION = 2;
 export const TELEMETRY_DIRNAME = "telemetry";
 export const TELEMETRY_SESSIONS_DIRNAME = "sessions";
 export const TELEMETRY_DASHBOARD_FILENAME = "dashboard.html";
@@ -414,7 +414,8 @@ function isRecord(value) {
 function validateTelemetrySession(data) {
     if (!isRecord(data))
         return null;
-    if (data["version"] !== TELEMETRY_VERSION)
+    const ver = data["version"];
+    if (ver !== TELEMETRY_VERSION && ver !== 1)
         return null;
     const sessionId = data["sessionId"];
     const startedAt = data["startedAt"];
@@ -427,6 +428,17 @@ function validateTelemetrySession(data) {
         return null;
     const subagents = Array.isArray(data["subagents"]) ? data["subagents"] : [];
     const events = Array.isArray(data["events"]) ? data["events"] : [];
+    // v1 → v2 migration: inputTruncated defaults from preview length, loop
+    // gains additive phase fields only when the writer reported them.
+    const migratedTurns = turns.map((t) => {
+        const rec = t;
+        if (typeof rec["inputTruncated"] !== "boolean") {
+            const preview = typeof rec["inputPreview"] === "string" ? rec["inputPreview"] : "";
+            const chars = typeof rec["inputChars"] === "number" ? rec["inputChars"] : preview.length;
+            rec["inputTruncated"] = chars > preview.length;
+        }
+        return t;
+    });
     return {
         version: TELEMETRY_VERSION,
         sessionId,
@@ -436,7 +448,7 @@ function validateTelemetrySession(data) {
         project: typeof data["project"] === "string" ? data["project"] : null,
         provider: typeof data["provider"] === "string" ? data["provider"] : "unknown",
         model: typeof data["model"] === "string" ? data["model"] : "unknown",
-        turns: turns,
+        turns: migratedTurns,
         subagents,
         events,
         compactionUsage: isRecord(data["compactionUsage"]) ? data["compactionUsage"] : {},
@@ -704,6 +716,7 @@ export class TelemetryRecorder {
                 durationMs: null,
                 inputPreview: preview.preview,
                 inputChars: typeof input === "string" ? input.length : 0,
+                inputTruncated: preview.truncated,
                 provider: meta.provider,
                 model: meta.model,
                 effort: meta.effort,
@@ -798,7 +811,9 @@ export class TelemetryRecorder {
                     : 0,
                 usage: clean,
                 usageReported: info.usageReported === true && clean !== undefined,
-                reasoningLabel: typeof info.reasoningLabel === "string" ? info.reasoningLabel : undefined,
+                reasoningLabel: typeof info.reasoningLabel === "string" && info.reasoningLabel.trim().length > 2
+                    ? info.reasoningLabel
+                    : undefined,
                 toolCallCount: typeof info.toolCallCount === "number" ? info.toolCallCount : 0,
                 finishReason: info.finishReason === "tool_calls" || info.finishReason === "error" ? info.finishReason : "final",
                 error: typeof info.error === "string" ? info.error.slice(0, 500) : undefined,
@@ -895,6 +910,26 @@ export class TelemetryRecorder {
                 if (bMs !== undefined)
                     loop.bottleneckMs = bMs;
             }
+            const slowest = s["slowestModel"];
+            const sName = typeof slowest?.["id"] === "string" ? slowest["id"] : undefined;
+            const sMs = num(slowest?.["durationMs"]) ?? undefined;
+            if (sName && sName.length > 0) {
+                loop.slowestModelName = sName.slice(0, 80);
+                if (sMs !== undefined)
+                    loop.slowestModelMs = sMs;
+            }
+            const modelTotal = num(s["modelTotalMs"]) ?? undefined;
+            if (modelTotal !== undefined)
+                loop.modelTotalMs = modelTotal;
+            const toolTotal = num(s["toolTotalMs"]) ?? undefined;
+            if (toolTotal !== undefined)
+                loop.toolTotalMs = toolTotal;
+            const dom = s["dominantPhase"];
+            if (dom === "model" || dom === "tool")
+                loop.dominantPhase = dom;
+            else if (modelTotal !== undefined || toolTotal !== undefined) {
+                loop.dominantPhase = (modelTotal ?? 0) >= (toolTotal ?? 0) ? "model" : "tool";
+            }
             turn.loop = loop;
         }
         catch {
@@ -1031,7 +1066,21 @@ export class TelemetryRecorder {
                         : outcome === "cancelled"
                             ? "(cancelled)"
                             : "(failed)";
+                // Preserve any streamed partial for post-mortem instead of nulling
+                // the reply outright; dashboard renders partial distinctly.
+                if (typeof replyOrError === "string" && replyOrError.length > 0) {
+                    const scrubbed = this.scrub(replyOrError);
+                    turn.partialReplyPreview = truncatePreview(scrubbed, TELEMETRY_INPUT_PREVIEW_CHARS).preview;
+                }
+                else {
+                    turn.partialReplyPreview = null;
+                }
                 turn.replyPreview = null;
+                // Timeline entry for failed turns only: cancellations are
+                // user-initiated noise, and the turn record already keeps the error.
+                if (outcome === "failed") {
+                    this.recordEvent("info", `turn ${turn.id} ${outcome}: ${(turn.error ?? "").slice(0, 200)}`);
+                }
             }
             else {
                 const scrubbed = this.scrub(typeof replyOrError === "string" ? replyOrError : "");