gsd-pi 2.78.1-dev.84a383f51 → 2.78.1

package/packages/pi-coding-agent/dist/core/token-telemetry.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"token-telemetry.js","sourceRoot":"","sources":["../../src/core/token-telemetry.ts"],"names":[],"mappings":"AAAA,+EAA+E;AAC/E,EAAE;AACF,gEAAgE;AAChE,0EAA0E;AAC1E,2EAA2E;AAC3E,yEAAyE;AACzE,oEAAoE;AACpE,EAAE;AACF,6EAA6E;AA8B7E,kEAAkE;AAClE,MAAM,UAAU,yBAAyB,CAAC,GAAqB;IAC9D,MAAM,KAAK,GAAG,GAAG,CAAC,KAAK,EAAE,KAAK,IAAI,CAAC,CAAC;IACpC,MAAM,MAAM,GAAG,GAAG,CAAC,KAAK,EAAE,MAAM,IAAI,CAAC,CAAC;IACtC,MAAM,SAAS,GAAG,GAAG,CAAC,KAAK,EAAE,SAAS,IAAI,CAAC,CAAC;IAC5C,MAAM,UAAU,GAAG,GAAG,CAAC,KAAK,EAAE,UAAU,IAAI,CAAC,CAAC;IAC9C,MAAM,SAAS,GAAG,GAAG,CAAC,KAAK,EAAE,IAAI,EAAE,KAAK,IAAI,CAAC,CAAC;IAC9C,MAAM,KAAK,GAAG,SAAS,GAAG,KAAK,CAAC;IAChC,MAAM,aAAa,GAAG,KAAK,GAAG,CAAC,CAAC,CAAC,CAAC,SAAS,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;IAExD,OAAO;QACN,EAAE,EAAE,GAAG,CAAC,SAAS;QACjB,KAAK,EAAE,GAAG,CAAC,KAAK;QAChB,UAAU,EAAE,GAAG,CAAC,UAAU;QAC1B,KAAK;QACL,MAAM;QACN,SAAS;QACT,UAAU;QACV,SAAS;QACT,aAAa;KACb,CAAC;AACH,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,kBAAkB,CAAC,GAAqB;IACvD,IAAI,OAAO,CAAC,GAAG,CAAC,kBAAkB,KAAK,GAAG;QAAE,OAAO;IACnD,IAAI,CAAC;QACJ,MAAM,MAAM,GAAG,yBAAyB,CAAC,GAAG,CAAC,CAAC;QAC9C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;IACrD,CAAC;IAAC,MAAM,CAAC;QACR,sDAAsD;IACvD,CAAC;AACF,CAAC","sourcesContent":["// @gsd/pi-coding-agent + token-telemetry — opt-in per-call token observability\n//\n// Emits a single JSON line per assistant message to stderr when\n// `PI_TOKEN_TELEMETRY=1` is set. Captures the cache_read_input_tokens and\n// cache_creation_input_tokens fields the providers already extract — so we\n// can empirically measure prompt-cache effectiveness (e.g. for #5019 and\n// future cache strategy work). Off by default — no behavior change.\n//\n// Capture pattern: `PI_TOKEN_TELEMETRY=1 npm start 2> token-telemetry.jsonl`\n\nimport type { AssistantMessage } from \"@gsd/pi-ai\";\n\n/** Schema of one telemetry line. JSON-stable for downstream ingestion. */\nexport interface TokenTelemetryRecord {\n\tts: number;\n\tmodel: string;\n\tstopReason: string;\n\tinput: number;\n\toutput: number;\n\tcacheRead: number;\n\tcacheWrite: number;\n\t/**\n\t * `usage.cost.total` from the provider. `0` when the provider's cost\n\t * registry has no rates for this model (e.g. unknown third-party providers)\n\t * — distinguish from a true zero-cost call by checking your model registry.\n\t */\n\tcostTotal: number;\n\t/**\n\t * Fraction of new prompt tokens served from cache:\n\t * `cacheRead / (cacheRead + input)`. Range [0, 1].\n\t * - `0` when neither cacheRead nor input is present (no division by zero).\n\t * - `1` on a full cache hit (input = 0, cacheRead > 0).\n\t * Note: `input` here is `input_tokens` from the API, which already excludes\n\t * cache reads/writes — the denominator is total prompt tokens consumed.\n\t */\n\tcacheHitRatio: number;\n}\n\n/** Build a telemetry record from a finished assistant message. */\nexport function buildTokenTelemetryRecord(msg: AssistantMessage): TokenTelemetryRecord {\n\tconst input = msg.usage?.input ?? 0;\n\tconst output = msg.usage?.output ?? 0;\n\tconst cacheRead = msg.usage?.cacheRead ?? 0;\n\tconst cacheWrite = msg.usage?.cacheWrite ?? 0;\n\tconst costTotal = msg.usage?.cost?.total ?? 0;\n\tconst denom = cacheRead + input;\n\tconst cacheHitRatio = denom > 0 ? cacheRead / denom : 0;\n\n\treturn {\n\t\tts: msg.timestamp,\n\t\tmodel: msg.model,\n\t\tstopReason: msg.stopReason,\n\t\tinput,\n\t\toutput,\n\t\tcacheRead,\n\t\tcacheWrite,\n\t\tcostTotal,\n\t\tcacheHitRatio,\n\t};\n}\n\n/**\n * Emit a token-telemetry line if `PI_TOKEN_TELEMETRY=1`. No-op otherwise.\n *\n * Writes to stderr so it doesn't interfere with TUI/stdout. One JSON object\n * per line. Errors during emission are swallowed — telemetry must never\n * break the agent loop.\n */\nexport function emitTokenTelemetry(msg: AssistantMessage): void {\n\tif (process.env.PI_TOKEN_TELEMETRY !== \"1\") return;\n\ttry {\n\t\tconst record = buildTokenTelemetryRecord(msg);\n\t\tprocess.stderr.write(`${JSON.stringify(record)}\\n`);\n\t} catch {\n\t\t// Telemetry must never break the agent loop. Swallow.\n\t}\n}\n"]}

package/packages/pi-coding-agent/dist/modes/interactive/components/tool-card-cleanup-and-success-runtime.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=tool-card-cleanup-and-success-runtime.test.d.ts.map

package/packages/pi-coding-agent/dist/modes/interactive/components/tool-card-cleanup-and-success-runtime.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"tool-card-cleanup-and-success-runtime.test.d.ts","sourceRoot":"","sources":["../../../../src/modes/interactive/components/tool-card-cleanup-and-success-runtime.test.ts"],"names":[],"mappings":""}

package/packages/pi-coding-agent/dist/modes/interactive/components/tool-card-cleanup-and-success-runtime.test.js

@@ -1,133 +0,0 @@
-// Runtime regression tests for the post-compaction tool-card cleanup and the
-// green-bordered success-notification rendering. Replaces the source-grep
-// `src/tests/tui-running-and-success-box.test.ts` that was deleted in #4875
-// (tracked as #4872).
-//
-// The previous tests asserted on identifier presence and method signature
-// shape via regex. A regression that routed `success` notifications through
-// `showStatus` (dim text) by accident would not have failed because the
-// `showSuccess` method would still exist and still match the regex. These
-// tests instead drive the components through the actual scenario and assert
-// on rendered output.
-import assert from "node:assert/strict";
-import { describe, it, before } from "node:test";
-import { Container, Text } from "@gsd/pi-tui";
-import stripAnsi from "strip-ansi";
-import { initTheme, theme } from "../theme/theme.js";
-import { DynamicBorder } from "./dynamic-border.js";
-import { ToolExecutionComponent } from "./tool-execution.js";
-// Theme is a globalThis-shared singleton that throws if not initialized.
-// Initialize once before any test that exercises themed rendering.
-before(() => {
-    initTheme("dark");
-});
-function makeMockTUI() {
-    return {
-        renderCount: 0,
-        requestRender() {
-            this.renderCount++;
-        },
-    };
-}
-// ─── Bug 1: tool cards stuck in "Running" after compaction ──────────────
-describe("ToolExecutionComponent post-compaction cleanup", () => {
-    it("renders 'Running' status while the tool call has no result", () => {
-        // Baseline: a freshly-constructed component (mid-stream) must show
-        // the running badge — this is the state we need to flip OUT of when
-        // compaction removes the result message.
-        const ui = makeMockTUI();
-        const c = new ToolExecutionComponent("read_file", { path: "/tmp/x.txt" }, {}, undefined, ui);
-        const rendered = c.render(60).map(stripAnsi).join("\n");
-        assert.ok(rendered.includes("Running"), "freshly constructed component should render 'Running' badge");
-    });
-    it("markHistoricalNoResult flips a stuck tool card OUT of 'Running'", () => {
-        // Real bug: after session-history replay (post-compaction or session
-        // switch), tool calls without matching tool_result messages stay in
-        // isPartial = true forever. markHistoricalNoResult must produce a
-        // rendered output that no longer reads "Running".
-        const ui = makeMockTUI();
-        const c = new ToolExecutionComponent("read_file", { path: "/tmp/x.txt" }, {}, undefined, ui);
-        c.markHistoricalNoResult();
-        const rendered = c.render(60).map(stripAnsi).join("\n");
-        assert.ok(!rendered.includes("Running"), "after markHistoricalNoResult, the tool card must NOT render 'Running' — got:\n" +
-            rendered);
-        assert.ok(rendered.includes("Done"), "flipped card should render 'Done' status (no-result success)");
-    });
-    it("markHistoricalNoResult is idempotent when a real result already exists", () => {
-        // Late-arriving stream events must not clobber legitimate results.
-        // We observe the no-clobber via behaviour: a card that completed
-        // with a real result keeps its "Done" badge and content even if
-        // markHistoricalNoResult fires again afterwards.
-        const ui = makeMockTUI();
-        const c = new ToolExecutionComponent("read_file", { path: "/tmp/x.txt" }, {}, undefined, ui);
-        // Use the public completion path the runtime calls.
-        c.updateResult({
-            content: [{ type: "text", text: "real-result-payload" }],
-            isError: false,
-        }, false);
-        const before = c.render(60).map(stripAnsi).join("\n");
-        assert.ok(before.includes("Done"), "after complete(), card shows 'Done'");
-        c.markHistoricalNoResult(); // must early-return
-        const after = c.render(60).map(stripAnsi).join("\n");
-        assert.equal(after, before, "markHistoricalNoResult must NOT mutate state when a real result is already present");
-    });
-});
-// ─── Bug 2: success notifications render as a green bordered box ────────
-describe("Success-notification rendering — DynamicBorder + success theme", () => {
-    // We drive the same component composition that interactive-mode.showSuccess
-    // builds (DynamicBorder + Text + DynamicBorder, all themed with
-    // theme.fg("success", …)) and assert on the rendered output, rather than
-    // trying to instantiate the full InteractiveMode (which requires a real
-    // session, bridge, and TUI runtime).
-    function buildSuccessNotification(message) {
-        const c = new Container();
-        const successColor = (text) => theme.fg("success", text);
-        c.addChild(new DynamicBorder(successColor));
-        c.addChild(new Text(theme.fg("success", message), 1, 0));
-        c.addChild(new DynamicBorder(successColor));
-        return c;
-    }
-    it("renders a top and bottom horizontal border framing the message", () => {
-        const c = buildSuccessNotification("Milestone M042 ready.");
-        const lines = c.render(40);
-        // First and last lines should be horizontal-rule borders
-        // (DynamicBorder renders a row of "─" characters).
-        const firstStripped = stripAnsi(lines[0]);
-        const lastStripped = stripAnsi(lines[lines.length - 1]);
-        assert.ok(firstStripped.includes("─"), `first line should be a border (got: ${JSON.stringify(firstStripped)})`);
-        assert.ok(lastStripped.includes("─"), `last line should be a border (got: ${JSON.stringify(lastStripped)})`);
-        assert.ok(lines.length >= 3, `success notification must have at least 3 lines (top border, message, bottom border) — got ${lines.length}`);
-    });
-    it("the message text appears between the two borders", () => {
-        const c = buildSuccessNotification("Milestone M042 ready.");
-        const lines = c.render(40);
-        const messageRow = lines.findIndex((l) => stripAnsi(l).includes("Milestone M042 ready."));
-        assert.ok(messageRow > 0, "message must appear after the top border");
-        assert.ok(messageRow < lines.length - 1, "message must appear before the bottom border");
-    });
-    it("borders carry ANSI styling (not plain dim text like showStatus)", () => {
-        // Goodhart-resistant: if a regression routed success through
-        // showStatus, the borders would be missing AND the rendered text
-        // would be plain (or only dim-styled, not success-colored). We
-        // observe the border lines carry ANSI escape codes — the literal
-        // foreground color depends on the theme (which is system-dependent
-        // in CI), so we don't pin a specific code but DO require styling
-        // to be present.
-        const c = buildSuccessNotification("ok");
-        const lines = c.render(40);
-        const top = lines[0];
-        assert.ok(top !== stripAnsi(top), "top border must contain ANSI styling — a plain unstyled border " +
-            "would indicate the rendering bypassed theme.fg('success', ...)");
-    });
-    it("plain Text status (the showStatus path) does NOT produce a bordered box", () => {
-        // Counter-test: this is what the bug looked like — a single dim Text
-        // with no surrounding border. If anyone ever "fixes" the showSuccess
-        // regression by also adding borders to showStatus, this test will
-        // surface the conflation.
-        const plain = new Text(theme.fg("dim", "Milestone M042 ready."), 1, 0);
-        const lines = plain.render(40);
-        const joined = lines.map(stripAnsi).join("\n");
-        assert.ok(!joined.includes("─"), "plain Text (showStatus path) must NOT contain border characters");
-    });
-});
-//# sourceMappingURL=tool-card-cleanup-and-success-runtime.test.js.map

package/packages/pi-coding-agent/dist/modes/interactive/components/tool-card-cleanup-and-success-runtime.test.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"tool-card-cleanup-and-success-runtime.test.js","sourceRoot":"","sources":["../../../../src/modes/interactive/components/tool-card-cleanup-and-success-runtime.test.ts"],"names":[],"mappings":"AAAA,6EAA6E;AAC7E,0EAA0E;AAC1E,4EAA4E;AAC5E,sBAAsB;AACtB,EAAE;AACF,0EAA0E;AAC1E,4EAA4E;AAC5E,wEAAwE;AACxE,0EAA0E;AAC1E,4EAA4E;AAC5E,sBAAsB;AAEtB,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,WAAW,CAAC;AAEjD,OAAO,EAAE,SAAS,EAAE,IAAI,EAAE,MAAM,aAAa,CAAC;AAC9C,OAAO,SAAS,MAAM,YAAY,CAAC;AAEnC,OAAO,EAAE,SAAS,EAAE,KAAK,EAAE,MAAM,mBAAmB,CAAC;AACrD,OAAO,EAAE,aAAa,EAAE,MAAM,qBAAqB,CAAC;AACpD,OAAO,EAAE,sBAAsB,EAAE,MAAM,qBAAqB,CAAC;AAE7D,yEAAyE;AACzE,mEAAmE;AACnE,MAAM,CAAC,GAAG,EAAE;IACX,SAAS,CAAC,MAAM,CAAC,CAAC;AACnB,CAAC,CAAC,CAAC;AAOH,SAAS,WAAW;IACnB,OAAO;QACN,WAAW,EAAE,CAAC;QACd,aAAa;YACZ,IAAI,CAAC,WAAW,EAAE,CAAC;QACpB,CAAC;KACD,CAAC;AACH,CAAC;AAED,2EAA2E;AAE3E,QAAQ,CAAC,gDAAgD,EAAE,GAAG,EAAE;IAC/D,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;QACrE,mEAAmE;QACnE,oEAAoE;QACpE,yCAAyC;QACzC,MAAM,EAAE,GAAG,WAAW,EAAE,CAAC;QACzB,MAAM,CAAC,GAAG,IAAI,sBAAsB,CACnC,WAAW,EACX,EAAE,IAAI,EAAE,YAAY,EAAE,EACtB,EAAE,EACF,SAAS,EACT,EAAW,CACX,CAAC;QACF,MAAM,QAAQ,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACxD,MAAM,CAAC,EAAE,CACR,QAAQ,CAAC,QAAQ,CAAC,SAAS,CAAC,EAC5B,6DAA6D,CAC7D,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iEAAiE,EAAE,GAAG,EAAE;QAC1E,qEAAqE;QACrE,oEAAoE;QACpE,kEAAkE;QAClE,kDAAkD;QAClD,MAAM,EAAE,GAAG,WAAW,EAAE,CAAC;QACzB,MAAM,CAAC,GAAG,IAAI,sBAAsB,CACnC,WAAW,EACX,EAAE,IAAI,EAAE,YAAY,EAAE,EACtB,EAAE,EACF,SAAS,EACT,EAAW,CACX,CAAC;QAEF,CAAC,CAAC,sBAAsB,EAAE,CAAC;QAE3B,MAAM,QAAQ,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACxD,MAAM,CAAC,EAAE,CACR,CAAC,QAAQ,CAAC,QAAQ,CAAC,SAAS,CAAC,EAC7B,gFAAgF;YAC/E,QAAQ,CACT,CAAC;QACF,MAAM,CAAC,EAAE,CACR,QAAQ,CAAC,QAAQ,CAAC,MAAM,CAAC,EACzB,8DAA8D,CAC9D,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wEAAwE,EAAE,GAAG,EAAE;QACjF,mEAAmE;QACnE,iEAAiE;QACjE,gEAAgE;QAChE,iDAAiD;QACjD,MAAM,EAAE,GAAG,WAAW,EAAE,CAAC;QACzB,MAAM,CAAC,GAAG,IAAI,sBAAsB,CACnC,WAAW,EACX,EAAE,IAAI,EAAE,YAAY,EAAE,EACtB,EAAE,EACF,SAAS,EACT,EAAW,CACX,CAAC;QAEF,oDAAoD;QACpD,CAAC,CAAC,YAAY,CACb;YACC,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,qBAAqB,EAAE,CAAC;YACxD,OAAO,EAAE,KAAK;SACd,EACD,KAAK,CACL,CAAC;QAEF,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACtD,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,qCAAqC,CAAC,CAAC;QAE1E,CAAC,CAAC,sBAAsB,EAAE,CAAC,CAAC,oBAAoB;QAChD,MAAM,KAAK,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACrD,MAAM,CAAC,KAAK,CACX,KAAK,EACL,MAAM,EACN,oFAAoF,CACpF,CAAC;IACH,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,2EAA2E;AAE3E,QAAQ,CAAC,gEAAgE,EAAE,GAAG,EAAE;IAC/E,4EAA4E;IAC5E,gEAAgE;IAChE,yEAAyE;IACzE,wEAAwE;IACxE,qCAAqC;IACrC,SAAS,wBAAwB,CAAC,OAAe;QAChD,MAAM,CAAC,GAAG,IAAI,SAAS,EAAE,CAAC;QAC1B,MAAM,YAAY,GAAG,CAAC,IAAY,EAAE,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,SAAS,EAAE,IAAI,CAAC,CAAC;QACjE,CAAC,CAAC,QAAQ,CAAC,IAAI,aAAa,CAAC,YAAY,CAAC,CAAC,CAAC;QAC5C,CAAC,CAAC,QAAQ,CAAC,IAAI,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC,SAAS,EAAE,OAAO,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC;QACzD,CAAC,CAAC,QAAQ,CAAC,IAAI,aAAa,CAAC,YAAY,CAAC,CAAC,CAAC;QAC5C,OAAO,CAAC,CAAC;IACV,CAAC;IAED,EAAE,CAAC,gEAAgE,EAAE,GAAG,EAAE;QACzE,MAAM,CAAC,GAAG,wBAAwB,CAAC,uBAAuB,CAAC,CAAC;QAC5D,MAAM,KAAK,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAE3B,yDAAyD;QACzD,mDAAmD;QACnD,MAAM,aAAa,GAAG,SAAS,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QAC1C,MAAM,YAAY,GAAG,SAAS,CAAC,KAAK,CAAC,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC;QAExD,MAAM,CAAC,EAAE,CACR,aAAa,CAAC,QAAQ,CAAC,GAAG,CAAC,EAC3B,uCAAuC,IAAI,CAAC,SAAS,CAAC,aAAa,CAAC,GAAG,CACvE,CAAC;QACF,MAAM,CAAC,EAAE,CACR,YAAY,CAAC,QAAQ,CAAC,GAAG,CAAC,EAC1B,sCAAsC,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,GAAG,CACrE,CAAC;QACF,MAAM,CAAC,EAAE,CACR,KAAK,CAAC,MAAM,IAAI,CAAC,EACjB,8FAA8F,KAAK,CAAC,MAAM,EAAE,CAC5G,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC3D,MAAM,CAAC,GAAG,wBAAwB,CAAC,uBAAuB,CAAC,CAAC;QAC5D,MAAM,KAAK,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAE3B,MAAM,UAAU,GAAG,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,uBAAuB,CAAC,CAAC,CAAC;QAC1F,MAAM,CAAC,EAAE,CAAC,UAAU,GAAG,CAAC,EAAE,0CAA0C,CAAC,CAAC;QACtE,MAAM,CAAC,EAAE,CACR,UAAU,GAAG,KAAK,CAAC,MAAM,GAAG,CAAC,EAC7B,8CAA8C,CAC9C,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iEAAiE,EAAE,GAAG,EAAE;QAC1E,6DAA6D;QAC7D,iEAAiE;QACjE,+DAA+D;QAC/D,iEAAiE;QACjE,mEAAmE;QACnE,iEAAiE;QACjE,iBAAiB;QACjB,MAAM,CAAC,GAAG,wBAAwB,CAAC,IAAI,CAAC,CAAC;QACzC,MAAM,KAAK,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAC3B,MAAM,GAAG,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;QACrB,MAAM,CAAC,EAAE,CACR,GAAG,KAAK,SAAS,CAAC,GAAG,CAAC,EACtB,iEAAiE;YAChE,gEAAgE,CACjE,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yEAAyE,EAAE,GAAG,EAAE;QAClF,qEAAqE;QACrE,qEAAqE;QACrE,kEAAkE;QAClE,0BAA0B;QAC1B,MAAM,KAAK,GAAG,IAAI,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC,KAAK,EAAE,uBAAuB,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACvE,MAAM,KAAK,GAAG,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAC/B,MAAM,MAAM,GAAG,KAAK,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC/C,MAAM,CAAC,EAAE,CACR,CAAC,MAAM,CAAC,QAAQ,CAAC,GAAG,CAAC,EACrB,iEAAiE,CACjE,CAAC;IACH,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC","sourcesContent":["// Runtime regression tests for the post-compaction tool-card cleanup and the\n// green-bordered success-notification rendering. Replaces the source-grep\n// `src/tests/tui-running-and-success-box.test.ts` that was deleted in #4875\n// (tracked as #4872).\n//\n// The previous tests asserted on identifier presence and method signature\n// shape via regex. A regression that routed `success` notifications through\n// `showStatus` (dim text) by accident would not have failed because the\n// `showSuccess` method would still exist and still match the regex. These\n// tests instead drive the components through the actual scenario and assert\n// on rendered output.\n\nimport assert from \"node:assert/strict\";\nimport { describe, it, before } from \"node:test\";\n\nimport { Container, Text } from \"@gsd/pi-tui\";\nimport stripAnsi from \"strip-ansi\";\n\nimport { initTheme, theme } from \"../theme/theme.js\";\nimport { DynamicBorder } from \"./dynamic-border.js\";\nimport { ToolExecutionComponent } from \"./tool-execution.js\";\n\n// Theme is a globalThis-shared singleton that throws if not initialized.\n// Initialize once before any test that exercises themed rendering.\nbefore(() => {\n\tinitTheme(\"dark\");\n});\n\ninterface MockTui {\n\trenderCount: number;\n\trequestRender(): void;\n}\n\nfunction makeMockTUI(): MockTui {\n\treturn {\n\t\trenderCount: 0,\n\t\trequestRender() {\n\t\t\tthis.renderCount++;\n\t\t},\n\t};\n}\n\n// ─── Bug 1: tool cards stuck in \"Running\" after compaction ──────────────\n\ndescribe(\"ToolExecutionComponent post-compaction cleanup\", () => {\n\tit(\"renders 'Running' status while the tool call has no result\", () => {\n\t\t// Baseline: a freshly-constructed component (mid-stream) must show\n\t\t// the running badge — this is the state we need to flip OUT of when\n\t\t// compaction removes the result message.\n\t\tconst ui = makeMockTUI();\n\t\tconst c = new ToolExecutionComponent(\n\t\t\t\"read_file\",\n\t\t\t{ path: \"/tmp/x.txt\" },\n\t\t\t{},\n\t\t\tundefined,\n\t\t\tui as never,\n\t\t);\n\t\tconst rendered = c.render(60).map(stripAnsi).join(\"\\n\");\n\t\tassert.ok(\n\t\t\trendered.includes(\"Running\"),\n\t\t\t\"freshly constructed component should render 'Running' badge\",\n\t\t);\n\t});\n\n\tit(\"markHistoricalNoResult flips a stuck tool card OUT of 'Running'\", () => {\n\t\t// Real bug: after session-history replay (post-compaction or session\n\t\t// switch), tool calls without matching tool_result messages stay in\n\t\t// isPartial = true forever. markHistoricalNoResult must produce a\n\t\t// rendered output that no longer reads \"Running\".\n\t\tconst ui = makeMockTUI();\n\t\tconst c = new ToolExecutionComponent(\n\t\t\t\"read_file\",\n\t\t\t{ path: \"/tmp/x.txt\" },\n\t\t\t{},\n\t\t\tundefined,\n\t\t\tui as never,\n\t\t);\n\n\t\tc.markHistoricalNoResult();\n\n\t\tconst rendered = c.render(60).map(stripAnsi).join(\"\\n\");\n\t\tassert.ok(\n\t\t\t!rendered.includes(\"Running\"),\n\t\t\t\"after markHistoricalNoResult, the tool card must NOT render 'Running' — got:\\n\" +\n\t\t\t\trendered,\n\t\t);\n\t\tassert.ok(\n\t\t\trendered.includes(\"Done\"),\n\t\t\t\"flipped card should render 'Done' status (no-result success)\",\n\t\t);\n\t});\n\n\tit(\"markHistoricalNoResult is idempotent when a real result already exists\", () => {\n\t\t// Late-arriving stream events must not clobber legitimate results.\n\t\t// We observe the no-clobber via behaviour: a card that completed\n\t\t// with a real result keeps its \"Done\" badge and content even if\n\t\t// markHistoricalNoResult fires again afterwards.\n\t\tconst ui = makeMockTUI();\n\t\tconst c = new ToolExecutionComponent(\n\t\t\t\"read_file\",\n\t\t\t{ path: \"/tmp/x.txt\" },\n\t\t\t{},\n\t\t\tundefined,\n\t\t\tui as never,\n\t\t);\n\n\t\t// Use the public completion path the runtime calls.\n\t\tc.updateResult(\n\t\t\t{\n\t\t\t\tcontent: [{ type: \"text\", text: \"real-result-payload\" }],\n\t\t\t\tisError: false,\n\t\t\t},\n\t\t\tfalse, // isPartial = false → \"Done\"\n\t\t);\n\n\t\tconst before = c.render(60).map(stripAnsi).join(\"\\n\");\n\t\tassert.ok(before.includes(\"Done\"), \"after complete(), card shows 'Done'\");\n\n\t\tc.markHistoricalNoResult(); // must early-return\n\t\tconst after = c.render(60).map(stripAnsi).join(\"\\n\");\n\t\tassert.equal(\n\t\t\tafter,\n\t\t\tbefore,\n\t\t\t\"markHistoricalNoResult must NOT mutate state when a real result is already present\",\n\t\t);\n\t});\n});\n\n// ─── Bug 2: success notifications render as a green bordered box ────────\n\ndescribe(\"Success-notification rendering — DynamicBorder + success theme\", () => {\n\t// We drive the same component composition that interactive-mode.showSuccess\n\t// builds (DynamicBorder + Text + DynamicBorder, all themed with\n\t// theme.fg(\"success\", …)) and assert on the rendered output, rather than\n\t// trying to instantiate the full InteractiveMode (which requires a real\n\t// session, bridge, and TUI runtime).\n\tfunction buildSuccessNotification(message: string): Container {\n\t\tconst c = new Container();\n\t\tconst successColor = (text: string) => theme.fg(\"success\", text);\n\t\tc.addChild(new DynamicBorder(successColor));\n\t\tc.addChild(new Text(theme.fg(\"success\", message), 1, 0));\n\t\tc.addChild(new DynamicBorder(successColor));\n\t\treturn c;\n\t}\n\n\tit(\"renders a top and bottom horizontal border framing the message\", () => {\n\t\tconst c = buildSuccessNotification(\"Milestone M042 ready.\");\n\t\tconst lines = c.render(40);\n\n\t\t// First and last lines should be horizontal-rule borders\n\t\t// (DynamicBorder renders a row of \"─\" characters).\n\t\tconst firstStripped = stripAnsi(lines[0]);\n\t\tconst lastStripped = stripAnsi(lines[lines.length - 1]);\n\n\t\tassert.ok(\n\t\t\tfirstStripped.includes(\"─\"),\n\t\t\t`first line should be a border (got: ${JSON.stringify(firstStripped)})`,\n\t\t);\n\t\tassert.ok(\n\t\t\tlastStripped.includes(\"─\"),\n\t\t\t`last line should be a border (got: ${JSON.stringify(lastStripped)})`,\n\t\t);\n\t\tassert.ok(\n\t\t\tlines.length >= 3,\n\t\t\t`success notification must have at least 3 lines (top border, message, bottom border) — got ${lines.length}`,\n\t\t);\n\t});\n\n\tit(\"the message text appears between the two borders\", () => {\n\t\tconst c = buildSuccessNotification(\"Milestone M042 ready.\");\n\t\tconst lines = c.render(40);\n\n\t\tconst messageRow = lines.findIndex((l) => stripAnsi(l).includes(\"Milestone M042 ready.\"));\n\t\tassert.ok(messageRow > 0, \"message must appear after the top border\");\n\t\tassert.ok(\n\t\t\tmessageRow < lines.length - 1,\n\t\t\t\"message must appear before the bottom border\",\n\t\t);\n\t});\n\n\tit(\"borders carry ANSI styling (not plain dim text like showStatus)\", () => {\n\t\t// Goodhart-resistant: if a regression routed success through\n\t\t// showStatus, the borders would be missing AND the rendered text\n\t\t// would be plain (or only dim-styled, not success-colored). We\n\t\t// observe the border lines carry ANSI escape codes — the literal\n\t\t// foreground color depends on the theme (which is system-dependent\n\t\t// in CI), so we don't pin a specific code but DO require styling\n\t\t// to be present.\n\t\tconst c = buildSuccessNotification(\"ok\");\n\t\tconst lines = c.render(40);\n\t\tconst top = lines[0];\n\t\tassert.ok(\n\t\t\ttop !== stripAnsi(top),\n\t\t\t\"top border must contain ANSI styling — a plain unstyled border \" +\n\t\t\t\t\"would indicate the rendering bypassed theme.fg('success', ...)\",\n\t\t);\n\t});\n\n\tit(\"plain Text status (the showStatus path) does NOT produce a bordered box\", () => {\n\t\t// Counter-test: this is what the bug looked like — a single dim Text\n\t\t// with no surrounding border. If anyone ever \"fixes\" the showSuccess\n\t\t// regression by also adding borders to showStatus, this test will\n\t\t// surface the conflation.\n\t\tconst plain = new Text(theme.fg(\"dim\", \"Milestone M042 ready.\"), 1, 0);\n\t\tconst lines = plain.render(40);\n\t\tconst joined = lines.map(stripAnsi).join(\"\\n\");\n\t\tassert.ok(\n\t\t\t!joined.includes(\"─\"),\n\t\t\t\"plain Text (showStatus path) must NOT contain border characters\",\n\t\t);\n\t});\n});\n"]}

package/packages/pi-coding-agent/dist/tests/system-prompt-cache-stability.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=system-prompt-cache-stability.test.d.ts.map

package/packages/pi-coding-agent/dist/tests/system-prompt-cache-stability.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"system-prompt-cache-stability.test.d.ts","sourceRoot":"","sources":["../../src/tests/system-prompt-cache-stability.test.ts"],"names":[],"mappings":""}

package/packages/pi-coding-agent/dist/tests/system-prompt-cache-stability.test.js

@@ -1,78 +0,0 @@
-// @gsd/pi-coding-agent + system-prompt-cache-stability.test — regression
-// coverage for #5019. The system prompt must NOT include a per-call timestamp
-// by default; embedding `Date.toLocaleString()` in the cached prefix
-// invalidates Anthropic prompt-cache hits on every request and incurs the
-// cache-write premium. The opt-in `includeDateTime` flag preserves the prior
-// behavior for callers that explicitly want clock awareness.
-import test from "node:test";
-import assert from "node:assert/strict";
-import { buildSystemPrompt } from "../core/system-prompt.js";
-// ─── Default branch (no customPrompt) ──────────────────────────────────────
-test("buildSystemPrompt: default omits 'Current date and time' line", () => {
-    const prompt = buildSystemPrompt({ selectedTools: ["read", "edit"] });
-    assert.ok(!prompt.includes("Current date and time:"), `prompt should not include the dateTime line by default. Got:\n${prompt}`);
-});
-test("buildSystemPrompt: includeDateTime=true emits 'Current date and time' line", () => {
-    const prompt = buildSystemPrompt({
-        selectedTools: ["read", "edit"],
-        includeDateTime: true,
-    });
-    assert.match(prompt, /Current date and time: /, "prompt should include the dateTime line when explicitly opted in");
-});
-test("buildSystemPrompt: includeDateTime=false explicit also omits the line", () => {
-    const prompt = buildSystemPrompt({
-        selectedTools: ["read", "edit"],
-        includeDateTime: false,
-    });
-    assert.ok(!prompt.includes("Current date and time:"));
-});
-// ─── Custom-prompt branch ──────────────────────────────────────────────────
-test("buildSystemPrompt (customPrompt): default omits 'Current date and time' line", () => {
-    const prompt = buildSystemPrompt({
-        customPrompt: "CUSTOM BASE",
-        selectedTools: ["read"],
-    });
-    assert.ok(!prompt.includes("Current date and time:"));
-});
-test("buildSystemPrompt (customPrompt): includeDateTime=true emits the line", () => {
-    const prompt = buildSystemPrompt({
-        customPrompt: "CUSTOM BASE",
-        selectedTools: ["read"],
-        includeDateTime: true,
-    });
-    assert.match(prompt, /Current date and time: /);
-});
-// ─── Cache-stability invariant ─────────────────────────────────────────────
-test("buildSystemPrompt: two back-to-back default calls produce identical prompts", async () => {
-    // The bug: the previous default-on `dateTime` line included `second: "2-digit"`,
-    // so two calls within the same second could match but any longer gap busted
-    // the cache. Asserting equality across a deliberate sub-second sleep proves
-    // the byte-for-byte stability that Anthropic prompt caching requires.
-    const first = buildSystemPrompt({ selectedTools: ["read", "edit"], cwd: "/tmp/example" });
-    await new Promise((resolve) => setTimeout(resolve, 1100));
-    const second = buildSystemPrompt({ selectedTools: ["read", "edit"], cwd: "/tmp/example" });
-    assert.equal(first, second, "system prompt must be byte-for-byte stable across calls so the prompt cache can hit");
-});
-test("buildSystemPrompt: includeDateTime=true intentionally produces different prompts across the second boundary", async () => {
-    // Inverse of the cache-stability test: when callers opt in, the dateTime
-    // line is expected to vary. This documents the trade-off the flag exists
-    // to surface.
-    const first = buildSystemPrompt({
-        selectedTools: ["read"],
-        cwd: "/tmp/example",
-        includeDateTime: true,
-    });
-    await new Promise((resolve) => setTimeout(resolve, 1100));
-    const second = buildSystemPrompt({
-        selectedTools: ["read"],
-        cwd: "/tmp/example",
-        includeDateTime: true,
-    });
-    assert.notEqual(first, second, "opt-in dateTime is expected to vary across the second boundary");
-});
-// ─── Cwd preservation ──────────────────────────────────────────────────────
-test("buildSystemPrompt: 'Current working directory' line is preserved (only dateTime is removed)", () => {
-    const prompt = buildSystemPrompt({ selectedTools: ["read"], cwd: "/tmp/example" });
-    assert.match(prompt, /Current working directory: \/tmp\/example/);
-});
-//# sourceMappingURL=system-prompt-cache-stability.test.js.map

package/packages/pi-coding-agent/dist/tests/system-prompt-cache-stability.test.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"system-prompt-cache-stability.test.js","sourceRoot":"","sources":["../../src/tests/system-prompt-cache-stability.test.ts"],"names":[],"mappings":"AAAA,yEAAyE;AACzE,8EAA8E;AAC9E,qEAAqE;AACrE,0EAA0E;AAC1E,6EAA6E;AAC7E,6DAA6D;AAE7D,OAAO,IAAI,MAAM,WAAW,CAAC;AAC7B,OAAO,MAAM,MAAM,oBAAoB,CAAC;AAExC,OAAO,EAAE,iBAAiB,EAAE,MAAM,0BAA0B,CAAC;AAE7D,8EAA8E;AAE9E,IAAI,CAAC,+DAA+D,EAAE,GAAG,EAAE;IAC1E,MAAM,MAAM,GAAG,iBAAiB,CAAC,EAAE,aAAa,EAAE,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,CAAC,CAAC;IACtE,MAAM,CAAC,EAAE,CACR,CAAC,MAAM,CAAC,QAAQ,CAAC,wBAAwB,CAAC,EAC1C,iEAAiE,MAAM,EAAE,CACzE,CAAC;AACH,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,4EAA4E,EAAE,GAAG,EAAE;IACvF,MAAM,MAAM,GAAG,iBAAiB,CAAC;QAChC,aAAa,EAAE,CAAC,MAAM,EAAE,MAAM,CAAC;QAC/B,eAAe,EAAE,IAAI;KACrB,CAAC,CAAC;IACH,MAAM,CAAC,KAAK,CACX,MAAM,EACN,yBAAyB,EACzB,kEAAkE,CAClE,CAAC;AACH,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,uEAAuE,EAAE,GAAG,EAAE;IAClF,MAAM,MAAM,GAAG,iBAAiB,CAAC;QAChC,aAAa,EAAE,CAAC,MAAM,EAAE,MAAM,CAAC;QAC/B,eAAe,EAAE,KAAK;KACtB,CAAC,CAAC;IACH,MAAM,CAAC,EAAE,CAAC,CAAC,MAAM,CAAC,QAAQ,CAAC,wBAAwB,CAAC,CAAC,CAAC;AACvD,CAAC,CAAC,CAAC;AAEH,8EAA8E;AAE9E,IAAI,CAAC,8EAA8E,EAAE,GAAG,EAAE;IACzF,MAAM,MAAM,GAAG,iBAAiB,CAAC;QAChC,YAAY,EAAE,aAAa;QAC3B,aAAa,EAAE,CAAC,MAAM,CAAC;KACvB,CAAC,CAAC;IACH,MAAM,CAAC,EAAE,CAAC,CAAC,MAAM,CAAC,QAAQ,CAAC,wBAAwB,CAAC,CAAC,CAAC;AACvD,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,uEAAuE,EAAE,GAAG,EAAE;IAClF,MAAM,MAAM,GAAG,iBAAiB,CAAC;QAChC,YAAY,EAAE,aAAa;QAC3B,aAAa,EAAE,CAAC,MAAM,CAAC;QACvB,eAAe,EAAE,IAAI;KACrB,CAAC,CAAC;IACH,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,yBAAyB,CAAC,CAAC;AACjD,CAAC,CAAC,CAAC;AAEH,8EAA8E;AAE9E,IAAI,CAAC,6EAA6E,EAAE,KAAK,IAAI,EAAE;IAC9F,iFAAiF;IACjF,4EAA4E;IAC5E,4EAA4E;IAC5E,sEAAsE;IACtE,MAAM,KAAK,GAAG,iBAAiB,CAAC,EAAE,aAAa,EAAE,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,GAAG,EAAE,cAAc,EAAE,CAAC,CAAC;IAC1F,MAAM,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,UAAU,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC,CAAC;IAC1D,MAAM,MAAM,GAAG,iBAAiB,CAAC,EAAE,aAAa,EAAE,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,GAAG,EAAE,cAAc,EAAE,CAAC,CAAC;IAC3F,MAAM,CAAC,KAAK,CACX,KAAK,EACL,MAAM,EACN,qFAAqF,CACrF,CAAC;AACH,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,6GAA6G,EAAE,KAAK,IAAI,EAAE;IAC9H,yEAAyE;IACzE,yEAAyE;IACzE,cAAc;IACd,MAAM,KAAK,GAAG,iBAAiB,CAAC;QAC/B,aAAa,EAAE,CAAC,MAAM,CAAC;QACvB,GAAG,EAAE,cAAc;QACnB,eAAe,EAAE,IAAI;KACrB,CAAC,CAAC;IACH,MAAM,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,UAAU,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC,CAAC;IAC1D,MAAM,MAAM,GAAG,iBAAiB,CAAC;QAChC,aAAa,EAAE,CAAC,MAAM,CAAC;QACvB,GAAG,EAAE,cAAc;QACnB,eAAe,EAAE,IAAI;KACrB,CAAC,CAAC;IACH,MAAM,CAAC,QAAQ,CAAC,KAAK,EAAE,MAAM,EAAE,gEAAgE,CAAC,CAAC;AAClG,CAAC,CAAC,CAAC;AAEH,8EAA8E;AAE9E,IAAI,CAAC,6FAA6F,EAAE,GAAG,EAAE;IACxG,MAAM,MAAM,GAAG,iBAAiB,CAAC,EAAE,aAAa,EAAE,CAAC,MAAM,CAAC,EAAE,GAAG,EAAE,cAAc,EAAE,CAAC,CAAC;IACnF,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,2CAA2C,CAAC,CAAC;AACnE,CAAC,CAAC,CAAC","sourcesContent":["// @gsd/pi-coding-agent + system-prompt-cache-stability.test — regression\n// coverage for #5019. The system prompt must NOT include a per-call timestamp\n// by default; embedding `Date.toLocaleString()` in the cached prefix\n// invalidates Anthropic prompt-cache hits on every request and incurs the\n// cache-write premium. The opt-in `includeDateTime` flag preserves the prior\n// behavior for callers that explicitly want clock awareness.\n\nimport test from \"node:test\";\nimport assert from \"node:assert/strict\";\n\nimport { buildSystemPrompt } from \"../core/system-prompt.js\";\n\n// ─── Default branch (no customPrompt) ──────────────────────────────────────\n\ntest(\"buildSystemPrompt: default omits 'Current date and time' line\", () => {\n\tconst prompt = buildSystemPrompt({ selectedTools: [\"read\", \"edit\"] });\n\tassert.ok(\n\t\t!prompt.includes(\"Current date and time:\"),\n\t\t`prompt should not include the dateTime line by default. Got:\\n${prompt}`,\n\t);\n});\n\ntest(\"buildSystemPrompt: includeDateTime=true emits 'Current date and time' line\", () => {\n\tconst prompt = buildSystemPrompt({\n\t\tselectedTools: [\"read\", \"edit\"],\n\t\tincludeDateTime: true,\n\t});\n\tassert.match(\n\t\tprompt,\n\t\t/Current date and time: /,\n\t\t\"prompt should include the dateTime line when explicitly opted in\",\n\t);\n});\n\ntest(\"buildSystemPrompt: includeDateTime=false explicit also omits the line\", () => {\n\tconst prompt = buildSystemPrompt({\n\t\tselectedTools: [\"read\", \"edit\"],\n\t\tincludeDateTime: false,\n\t});\n\tassert.ok(!prompt.includes(\"Current date and time:\"));\n});\n\n// ─── Custom-prompt branch ──────────────────────────────────────────────────\n\ntest(\"buildSystemPrompt (customPrompt): default omits 'Current date and time' line\", () => {\n\tconst prompt = buildSystemPrompt({\n\t\tcustomPrompt: \"CUSTOM BASE\",\n\t\tselectedTools: [\"read\"],\n\t});\n\tassert.ok(!prompt.includes(\"Current date and time:\"));\n});\n\ntest(\"buildSystemPrompt (customPrompt): includeDateTime=true emits the line\", () => {\n\tconst prompt = buildSystemPrompt({\n\t\tcustomPrompt: \"CUSTOM BASE\",\n\t\tselectedTools: [\"read\"],\n\t\tincludeDateTime: true,\n\t});\n\tassert.match(prompt, /Current date and time: /);\n});\n\n// ─── Cache-stability invariant ─────────────────────────────────────────────\n\ntest(\"buildSystemPrompt: two back-to-back default calls produce identical prompts\", async () => {\n\t// The bug: the previous default-on `dateTime` line included `second: \"2-digit\"`,\n\t// so two calls within the same second could match but any longer gap busted\n\t// the cache. Asserting equality across a deliberate sub-second sleep proves\n\t// the byte-for-byte stability that Anthropic prompt caching requires.\n\tconst first = buildSystemPrompt({ selectedTools: [\"read\", \"edit\"], cwd: \"/tmp/example\" });\n\tawait new Promise((resolve) => setTimeout(resolve, 1100));\n\tconst second = buildSystemPrompt({ selectedTools: [\"read\", \"edit\"], cwd: \"/tmp/example\" });\n\tassert.equal(\n\t\tfirst,\n\t\tsecond,\n\t\t\"system prompt must be byte-for-byte stable across calls so the prompt cache can hit\",\n\t);\n});\n\ntest(\"buildSystemPrompt: includeDateTime=true intentionally produces different prompts across the second boundary\", async () => {\n\t// Inverse of the cache-stability test: when callers opt in, the dateTime\n\t// line is expected to vary. This documents the trade-off the flag exists\n\t// to surface.\n\tconst first = buildSystemPrompt({\n\t\tselectedTools: [\"read\"],\n\t\tcwd: \"/tmp/example\",\n\t\tincludeDateTime: true,\n\t});\n\tawait new Promise((resolve) => setTimeout(resolve, 1100));\n\tconst second = buildSystemPrompt({\n\t\tselectedTools: [\"read\"],\n\t\tcwd: \"/tmp/example\",\n\t\tincludeDateTime: true,\n\t});\n\tassert.notEqual(first, second, \"opt-in dateTime is expected to vary across the second boundary\");\n});\n\n// ─── Cwd preservation ──────────────────────────────────────────────────────\n\ntest(\"buildSystemPrompt: 'Current working directory' line is preserved (only dateTime is removed)\", () => {\n\tconst prompt = buildSystemPrompt({ selectedTools: [\"read\"], cwd: \"/tmp/example\" });\n\tassert.match(prompt, /Current working directory: \\/tmp\\/example/);\n});\n"]}

package/packages/pi-coding-agent/dist/tests/token-telemetry.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=token-telemetry.test.d.ts.map

package/packages/pi-coding-agent/dist/tests/token-telemetry.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"token-telemetry.test.d.ts","sourceRoot":"","sources":["../../src/tests/token-telemetry.test.ts"],"names":[],"mappings":""}

package/packages/pi-coding-agent/dist/tests/token-telemetry.test.js

@@ -1,181 +0,0 @@
-// @gsd/pi-coding-agent + token-telemetry.test — coverage for #5023.
-// Verifies the env-gated emitter:
-//   - is silent by default (no behavior change for existing users)
-//   - emits a single valid JSON line when PI_TOKEN_TELEMETRY=1
-//   - record shape captures the cache breakdown the providers already extract
-//   - cacheHitRatio math is correct, including the no-input edge case
-import { describe, test, beforeEach, afterEach } from "node:test";
-import assert from "node:assert/strict";
-import { buildTokenTelemetryRecord, emitTokenTelemetry } from "../core/token-telemetry.js";
-function makeAssistantMessage(overrides = {}) {
-    return {
-        role: "assistant",
-        content: [],
-        api: "anthropic-messages",
-        provider: "anthropic",
-        model: "claude-sonnet-4-6",
-        usage: {
-            input: 100,
-            output: 50,
-            cacheRead: 0,
-            cacheWrite: 0,
-            totalTokens: 150,
-            cost: { input: 0.3, output: 0.75, cacheRead: 0, cacheWrite: 0, total: 1.05 },
-        },
-        stopReason: "stop",
-        timestamp: 1700000000000,
-        ...overrides,
-    };
-}
-// ─── buildTokenTelemetryRecord ─────────────────────────────────────────────
-describe("buildTokenTelemetryRecord", () => {
-    test("captures all fields from a typical message", () => {
-        const msg = makeAssistantMessage();
-        const record = buildTokenTelemetryRecord(msg);
-        assert.deepEqual(record, {
-            ts: 1700000000000,
-            model: "claude-sonnet-4-6",
-            stopReason: "stop",
-            input: 100,
-            output: 50,
-            cacheRead: 0,
-            cacheWrite: 0,
-            costTotal: 1.05,
-            cacheHitRatio: 0,
-        });
-    });
-    test("cacheHitRatio = read / (read + input) when both present", () => {
-        const msg = makeAssistantMessage({
-            usage: {
-                input: 200,
-                output: 50,
-                cacheRead: 800,
-                cacheWrite: 0,
-                totalTokens: 1050,
-                cost: { input: 0.6, output: 0.75, cacheRead: 0.24, cacheWrite: 0, total: 1.59 },
-            },
-        });
-        const record = buildTokenTelemetryRecord(msg);
-        assert.equal(record.cacheRead, 800);
-        assert.equal(record.input, 200);
-        assert.equal(record.cacheHitRatio, 0.8);
-    });
-    test("cacheHitRatio = 0 when both read and input are 0 (no division-by-zero)", () => {
-        const msg = makeAssistantMessage({
-            usage: {
-                input: 0,
-                output: 50,
-                cacheRead: 0,
-                cacheWrite: 0,
-                totalTokens: 50,
-                cost: { input: 0, output: 0.75, cacheRead: 0, cacheWrite: 0, total: 0.75 },
-            },
-        });
-        const record = buildTokenTelemetryRecord(msg);
-        assert.equal(record.cacheHitRatio, 0);
-    });
-    test("cacheHitRatio = 1 when only cacheRead present (full hit)", () => {
-        const msg = makeAssistantMessage({
-            usage: {
-                input: 0,
-                output: 50,
-                cacheRead: 5000,
-                cacheWrite: 0,
-                totalTokens: 5050,
-                cost: { input: 0, output: 0.75, cacheRead: 1.5, cacheWrite: 0, total: 2.25 },
-            },
-        });
-        const record = buildTokenTelemetryRecord(msg);
-        assert.equal(record.cacheHitRatio, 1);
-    });
-    test("cacheWrite is captured (the cache-miss-with-cache-control case from #5019)", () => {
-        const msg = makeAssistantMessage({
-            usage: {
-                input: 50,
-                output: 100,
-                cacheRead: 0,
-                cacheWrite: 5000,
-                totalTokens: 5150,
-                cost: { input: 0.15, output: 1.5, cacheRead: 0, cacheWrite: 18.75, total: 20.4 },
-            },
-        });
-        const record = buildTokenTelemetryRecord(msg);
-        assert.equal(record.cacheWrite, 5000);
-        assert.equal(record.cacheHitRatio, 0, "no read = ratio 0 even when write is large");
-    });
-    test("error stopReason is captured verbatim", () => {
-        const msg = makeAssistantMessage({ stopReason: "error", errorMessage: "rate_limit" });
-        assert.equal(buildTokenTelemetryRecord(msg).stopReason, "error");
-    });
-});
-// ─── emitTokenTelemetry ────────────────────────────────────────────────────
-describe("emitTokenTelemetry", () => {
-    let captured;
-    let originalWrite;
-    let originalEnv;
-    beforeEach(() => {
-        captured = [];
-        originalWrite = process.stderr.write.bind(process.stderr);
-        // Replace stderr.write with a capture; preserve the same return contract.
-        process.stderr.write = ((chunk) => {
-            captured.push(typeof chunk === "string" ? chunk : Buffer.from(chunk).toString());
-            return true;
-        });
-        originalEnv = process.env.PI_TOKEN_TELEMETRY;
-    });
-    afterEach(() => {
-        process.stderr.write = originalWrite;
-        if (originalEnv === undefined) {
-            delete process.env.PI_TOKEN_TELEMETRY;
-        }
-        else {
-            process.env.PI_TOKEN_TELEMETRY = originalEnv;
-        }
-    });
-    test("silent by default (env var unset)", () => {
-        delete process.env.PI_TOKEN_TELEMETRY;
-        emitTokenTelemetry(makeAssistantMessage());
-        assert.equal(captured.length, 0);
-    });
-    test("silent when env var has any non-'1' value", () => {
-        process.env.PI_TOKEN_TELEMETRY = "true"; // not literally "1"
-        emitTokenTelemetry(makeAssistantMessage());
-        assert.equal(captured.length, 0, "only literal '1' should enable telemetry");
-    });
-    test("emits a single JSON line when PI_TOKEN_TELEMETRY=1", () => {
-        process.env.PI_TOKEN_TELEMETRY = "1";
-        emitTokenTelemetry(makeAssistantMessage());
-        assert.equal(captured.length, 1);
-        assert.ok(captured[0].endsWith("\n"), "line must terminate with newline");
-        const parsed = JSON.parse(captured[0].trimEnd());
-        assert.equal(parsed.model, "claude-sonnet-4-6");
-        assert.equal(parsed.input, 100);
-        assert.equal(parsed.output, 50);
-    });
-    test("emitted JSON has the documented shape", () => {
-        process.env.PI_TOKEN_TELEMETRY = "1";
-        emitTokenTelemetry(makeAssistantMessage());
-        const parsed = JSON.parse(captured[0].trimEnd());
-        const keys = Object.keys(parsed).sort();
-        assert.deepEqual(keys, [
-            "cacheHitRatio",
-            "cacheRead",
-            "cacheWrite",
-            "costTotal",
-            "input",
-            "model",
-            "output",
-            "stopReason",
-            "ts",
-        ]);
-    });
-    test("never throws — telemetry must not break the agent loop", () => {
-        process.env.PI_TOKEN_TELEMETRY = "1";
-        // Force a write failure to exercise the swallow path.
-        process.stderr.write = (() => {
-            throw new Error("simulated stderr failure");
-        });
-        assert.doesNotThrow(() => emitTokenTelemetry(makeAssistantMessage()));
-    });
-});
-//# sourceMappingURL=token-telemetry.test.js.map

package/packages/pi-coding-agent/dist/tests/token-telemetry.test.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"token-telemetry.test.js","sourceRoot":"","sources":["../../src/tests/token-telemetry.test.ts"],"names":[],"mappings":"AAAA,oEAAoE;AACpE,kCAAkC;AAClC,mEAAmE;AACnE,+DAA+D;AAC/D,8EAA8E;AAC9E,sEAAsE;AAEtE,OAAO,EAAE,QAAQ,EAAE,IAAI,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,WAAW,CAAC;AAClE,OAAO,MAAM,MAAM,oBAAoB,CAAC;AAIxC,OAAO,EAAE,yBAAyB,EAAE,kBAAkB,EAAE,MAAM,4BAA4B,CAAC;AAE3F,SAAS,oBAAoB,CAAC,YAAuC,EAAE;IACtE,OAAO;QACN,IAAI,EAAE,WAAW;QACjB,OAAO,EAAE,EAAE;QACX,GAAG,EAAE,oBAAoB;QACzB,QAAQ,EAAE,WAAW;QACrB,KAAK,EAAE,mBAAmB;QAC1B,KAAK,EAAE;YACN,KAAK,EAAE,GAAG;YACV,MAAM,EAAE,EAAE;YACV,SAAS,EAAE,CAAC;YACZ,UAAU,EAAE,CAAC;YACb,WAAW,EAAE,GAAG;YAChB,IAAI,EAAE,EAAE,KAAK,EAAE,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,EAAE,UAAU,EAAE,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE;SAC5E;QACD,UAAU,EAAE,MAAM;QAClB,SAAS,EAAE,aAAa;QACxB,GAAG,SAAS;KACZ,CAAC;AACH,CAAC;AAED,8EAA8E;AAE9E,QAAQ,CAAC,2BAA2B,EAAE,GAAG,EAAE;IAC1C,IAAI,CAAC,4CAA4C,EAAE,GAAG,EAAE;QACvD,MAAM,GAAG,GAAG,oBAAoB,EAAE,CAAC;QACnC,MAAM,MAAM,GAAG,yBAAyB,CAAC,GAAG,CAAC,CAAC;QAC9C,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE;YACxB,EAAE,EAAE,aAAa;YACjB,KAAK,EAAE,mBAAmB;YAC1B,UAAU,EAAE,MAAM;YAClB,KAAK,EAAE,GAAG;YACV,MAAM,EAAE,EAAE;YACV,SAAS,EAAE,CAAC;YACZ,UAAU,EAAE,CAAC;YACb,SAAS,EAAE,IAAI;YACf,aAAa,EAAE,CAAC;SAChB,CAAC,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,yDAAyD,EAAE,GAAG,EAAE;QACpE,MAAM,GAAG,GAAG,oBAAoB,CAAC;YAChC,KAAK,EAAE;gBACN,KAAK,EAAE,GAAG;gBACV,MAAM,EAAE,EAAE;gBACV,SAAS,EAAE,GAAG;gBACd,UAAU,EAAE,CAAC;gBACb,WAAW,EAAE,IAAI;gBACjB,IAAI,EAAE,EAAE,KAAK,EAAE,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,SAAS,EAAE,IAAI,EAAE,UAAU,EAAE,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE;aAC/E;SACD,CAAC,CAAC;QACH,MAAM,MAAM,GAAG,yBAAyB,CAAC,GAAG,CAAC,CAAC;QAC9C,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,SAAS,EAAE,GAAG,CAAC,CAAC;QACpC,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC;QAChC,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,aAAa,EAAE,GAAG,CAAC,CAAC;IACzC,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,wEAAwE,EAAE,GAAG,EAAE;QACnF,MAAM,GAAG,GAAG,oBAAoB,CAAC;YAChC,KAAK,EAAE;gBACN,KAAK,EAAE,CAAC;gBACR,MAAM,EAAE,EAAE;gBACV,SAAS,EAAE,CAAC;gBACZ,UAAU,EAAE,CAAC;gBACb,WAAW,EAAE,EAAE;gBACf,IAAI,EAAE,EAAE,KAAK,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,EAAE,UAAU,EAAE,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE;aAC1E;SACD,CAAC,CAAC;QACH,MAAM,MAAM,GAAG,yBAAyB,CAAC,GAAG,CAAC,CAAC;QAC9C,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,aAAa,EAAE,CAAC,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,0DAA0D,EAAE,GAAG,EAAE;QACrE,MAAM,GAAG,GAAG,oBAAoB,CAAC;YAChC,KAAK,EAAE;gBACN,KAAK,EAAE,CAAC;gBACR,MAAM,EAAE,EAAE;gBACV,SAAS,EAAE,IAAI;gBACf,UAAU,EAAE,CAAC;gBACb,WAAW,EAAE,IAAI;gBACjB,IAAI,EAAE,EAAE,KAAK,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,SAAS,EAAE,GAAG,EAAE,UAAU,EAAE,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE;aAC5E;SACD,CAAC,CAAC;QACH,MAAM,MAAM,GAAG,yBAAyB,CAAC,GAAG,CAAC,CAAC;QAC9C,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,aAAa,EAAE,CAAC,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,4EAA4E,EAAE,GAAG,EAAE;QACvF,MAAM,GAAG,GAAG,oBAAoB,CAAC;YAChC,KAAK,EAAE;gBACN,KAAK,EAAE,EAAE;gBACT,MAAM,EAAE,GAAG;gBACX,SAAS,EAAE,CAAC;gBACZ,UAAU,EAAE,IAAI;gBAChB,WAAW,EAAE,IAAI;gBACjB,IAAI,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,MAAM,EAAE,GAAG,EAAE,SAAS,EAAE,CAAC,EAAE,UAAU,EAAE,KAAK,EAAE,KAAK,EAAE,IAAI,EAAE;aAChF;SACD,CAAC,CAAC;QACH,MAAM,MAAM,GAAG,yBAAyB,CAAC,GAAG,CAAC,CAAC;QAC9C,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,UAAU,EAAE,IAAI,CAAC,CAAC;QACtC,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,aAAa,EAAE,CAAC,EAAE,4CAA4C,CAAC,CAAC;IACrF,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,uCAAuC,EAAE,GAAG,EAAE;QAClD,MAAM,GAAG,GAAG,oBAAoB,CAAC,EAAE,UAAU,EAAE,OAAO,EAAE,YAAY,EAAE,YAAY,EAAE,CAAC,CAAC;QACtF,MAAM,CAAC,KAAK,CAAC,yBAAyB,CAAC,GAAG,CAAC,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC;IAClE,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,8EAA8E;AAE9E,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;IACnC,IAAI,QAAkB,CAAC;IACvB,IAAI,aAA0C,CAAC;IAC/C,IAAI,WAA+B,CAAC;IAEpC,UAAU,CAAC,GAAG,EAAE;QACf,QAAQ,GAAG,EAAE,CAAC;QACd,aAAa,GAAG,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC;QAC1D,0EAA0E;QAC1E,OAAO,CAAC,MAAM,CAAC,KAAK,GAAG,CAAC,CAAC,KAA0B,EAAW,EAAE;YAC/D,QAAQ,CAAC,IAAI,CAAC,OAAO,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,QAAQ,EAAE,CAAC,CAAC;YACjF,OAAO,IAAI,CAAC;QACb,CAAC,CAAgC,CAAC;QAClC,WAAW,GAAG,OAAO,CAAC,GAAG,CAAC,kBAAkB,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACd,OAAO,CAAC,MAAM,CAAC,KAAK,GAAG,aAAa,CAAC;QACrC,IAAI,WAAW,KAAK,SAAS,EAAE,CAAC;YAC/B,OAAO,OAAO,CAAC,GAAG,CAAC,kBAAkB,CAAC;QACvC,CAAC;aAAM,CAAC;YACP,OAAO,CAAC,GAAG,CAAC,kBAAkB,GAAG,WAAW,CAAC;QAC9C,CAAC;IACF,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC9C,OAAO,OAAO,CAAC,GAAG,CAAC,kBAAkB,CAAC;QACtC,kBAAkB,CAAC,oBAAoB,EAAE,CAAC,CAAC;QAC3C,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;IAClC,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACtD,OAAO,CAAC,GAAG,CAAC,kBAAkB,GAAG,MAAM,CAAC,CAAC,oBAAoB;QAC7D,kBAAkB,CAAC,oBAAoB,EAAE,CAAC,CAAC;QAC3C,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,MAAM,EAAE,CAAC,EAAE,0CAA0C,CAAC,CAAC;IAC9E,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,oDAAoD,EAAE,GAAG,EAAE;QAC/D,OAAO,CAAC,GAAG,CAAC,kBAAkB,GAAG,GAAG,CAAC;QACrC,kBAAkB,CAAC,oBAAoB,EAAE,CAAC,CAAC;QAC3C,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QACjC,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE,kCAAkC,CAAC,CAAC;QAC1E,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC,CAAC;QACjD,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,EAAE,mBAAmB,CAAC,CAAC;QAChD,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC;QAChC,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC;IACjC,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,uCAAuC,EAAE,GAAG,EAAE;QAClD,OAAO,CAAC,GAAG,CAAC,kBAAkB,GAAG,GAAG,CAAC;QACrC,kBAAkB,CAAC,oBAAoB,EAAE,CAAC,CAAC;QAC3C,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC,CAAC;QACjD,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,CAAC;QACxC,MAAM,CAAC,SAAS,CAAC,IAAI,EAAE;YACtB,eAAe;YACf,WAAW;YACX,YAAY;YACZ,WAAW;YACX,OAAO;YACP,OAAO;YACP,QAAQ;YACR,YAAY;YACZ,IAAI;SACJ,CAAC,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,wDAAwD,EAAE,GAAG,EAAE;QACnE,OAAO,CAAC,GAAG,CAAC,kBAAkB,GAAG,GAAG,CAAC;QACrC,sDAAsD;QACtD,OAAO,CAAC,MAAM,CAAC,KAAK,GAAG,CAAC,GAAG,EAAE;YAC5B,MAAM,IAAI,KAAK,CAAC,0BAA0B,CAAC,CAAC;QAC7C,CAAC,CAAgC,CAAC;QAClC,MAAM,CAAC,YAAY,CAAC,GAAG,EAAE,CAAC,kBAAkB,CAAC,oBAAoB,EAAE,CAAC,CAAC,CAAC;IACvE,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC","sourcesContent":["// @gsd/pi-coding-agent + token-telemetry.test — coverage for #5023.\n// Verifies the env-gated emitter:\n//   - is silent by default (no behavior change for existing users)\n//   - emits a single valid JSON line when PI_TOKEN_TELEMETRY=1\n//   - record shape captures the cache breakdown the providers already extract\n//   - cacheHitRatio math is correct, including the no-input edge case\n\nimport { describe, test, beforeEach, afterEach } from \"node:test\";\nimport assert from \"node:assert/strict\";\n\nimport type { AssistantMessage } from \"@gsd/pi-ai\";\n\nimport { buildTokenTelemetryRecord, emitTokenTelemetry } from \"../core/token-telemetry.js\";\n\nfunction makeAssistantMessage(overrides: Partial<AssistantMessage> = {}): AssistantMessage {\n\treturn {\n\t\trole: \"assistant\",\n\t\tcontent: [],\n\t\tapi: \"anthropic-messages\",\n\t\tprovider: \"anthropic\",\n\t\tmodel: \"claude-sonnet-4-6\",\n\t\tusage: {\n\t\t\tinput: 100,\n\t\t\toutput: 50,\n\t\t\tcacheRead: 0,\n\t\t\tcacheWrite: 0,\n\t\t\ttotalTokens: 150,\n\t\t\tcost: { input: 0.3, output: 0.75, cacheRead: 0, cacheWrite: 0, total: 1.05 },\n\t\t},\n\t\tstopReason: \"stop\",\n\t\ttimestamp: 1700000000000,\n\t\t...overrides,\n\t};\n}\n\n// ─── buildTokenTelemetryRecord ─────────────────────────────────────────────\n\ndescribe(\"buildTokenTelemetryRecord\", () => {\n\ttest(\"captures all fields from a typical message\", () => {\n\t\tconst msg = makeAssistantMessage();\n\t\tconst record = buildTokenTelemetryRecord(msg);\n\t\tassert.deepEqual(record, {\n\t\t\tts: 1700000000000,\n\t\t\tmodel: \"claude-sonnet-4-6\",\n\t\t\tstopReason: \"stop\",\n\t\t\tinput: 100,\n\t\t\toutput: 50,\n\t\t\tcacheRead: 0,\n\t\t\tcacheWrite: 0,\n\t\t\tcostTotal: 1.05,\n\t\t\tcacheHitRatio: 0,\n\t\t});\n\t});\n\n\ttest(\"cacheHitRatio = read / (read + input) when both present\", () => {\n\t\tconst msg = makeAssistantMessage({\n\t\t\tusage: {\n\t\t\t\tinput: 200,\n\t\t\t\toutput: 50,\n\t\t\t\tcacheRead: 800,\n\t\t\t\tcacheWrite: 0,\n\t\t\t\ttotalTokens: 1050,\n\t\t\t\tcost: { input: 0.6, output: 0.75, cacheRead: 0.24, cacheWrite: 0, total: 1.59 },\n\t\t\t},\n\t\t});\n\t\tconst record = buildTokenTelemetryRecord(msg);\n\t\tassert.equal(record.cacheRead, 800);\n\t\tassert.equal(record.input, 200);\n\t\tassert.equal(record.cacheHitRatio, 0.8);\n\t});\n\n\ttest(\"cacheHitRatio = 0 when both read and input are 0 (no division-by-zero)\", () => {\n\t\tconst msg = makeAssistantMessage({\n\t\t\tusage: {\n\t\t\t\tinput: 0,\n\t\t\t\toutput: 50,\n\t\t\t\tcacheRead: 0,\n\t\t\t\tcacheWrite: 0,\n\t\t\t\ttotalTokens: 50,\n\t\t\t\tcost: { input: 0, output: 0.75, cacheRead: 0, cacheWrite: 0, total: 0.75 },\n\t\t\t},\n\t\t});\n\t\tconst record = buildTokenTelemetryRecord(msg);\n\t\tassert.equal(record.cacheHitRatio, 0);\n\t});\n\n\ttest(\"cacheHitRatio = 1 when only cacheRead present (full hit)\", () => {\n\t\tconst msg = makeAssistantMessage({\n\t\t\tusage: {\n\t\t\t\tinput: 0,\n\t\t\t\toutput: 50,\n\t\t\t\tcacheRead: 5000,\n\t\t\t\tcacheWrite: 0,\n\t\t\t\ttotalTokens: 5050,\n\t\t\t\tcost: { input: 0, output: 0.75, cacheRead: 1.5, cacheWrite: 0, total: 2.25 },\n\t\t\t},\n\t\t});\n\t\tconst record = buildTokenTelemetryRecord(msg);\n\t\tassert.equal(record.cacheHitRatio, 1);\n\t});\n\n\ttest(\"cacheWrite is captured (the cache-miss-with-cache-control case from #5019)\", () => {\n\t\tconst msg = makeAssistantMessage({\n\t\t\tusage: {\n\t\t\t\tinput: 50,\n\t\t\t\toutput: 100,\n\t\t\t\tcacheRead: 0,\n\t\t\t\tcacheWrite: 5000,\n\t\t\t\ttotalTokens: 5150,\n\t\t\t\tcost: { input: 0.15, output: 1.5, cacheRead: 0, cacheWrite: 18.75, total: 20.4 },\n\t\t\t},\n\t\t});\n\t\tconst record = buildTokenTelemetryRecord(msg);\n\t\tassert.equal(record.cacheWrite, 5000);\n\t\tassert.equal(record.cacheHitRatio, 0, \"no read = ratio 0 even when write is large\");\n\t});\n\n\ttest(\"error stopReason is captured verbatim\", () => {\n\t\tconst msg = makeAssistantMessage({ stopReason: \"error\", errorMessage: \"rate_limit\" });\n\t\tassert.equal(buildTokenTelemetryRecord(msg).stopReason, \"error\");\n\t});\n});\n\n// ─── emitTokenTelemetry ────────────────────────────────────────────────────\n\ndescribe(\"emitTokenTelemetry\", () => {\n\tlet captured: string[];\n\tlet originalWrite: typeof process.stderr.write;\n\tlet originalEnv: string | undefined;\n\n\tbeforeEach(() => {\n\t\tcaptured = [];\n\t\toriginalWrite = process.stderr.write.bind(process.stderr);\n\t\t// Replace stderr.write with a capture; preserve the same return contract.\n\t\tprocess.stderr.write = ((chunk: string | Uint8Array): boolean => {\n\t\t\tcaptured.push(typeof chunk === \"string\" ? chunk : Buffer.from(chunk).toString());\n\t\t\treturn true;\n\t\t}) as typeof process.stderr.write;\n\t\toriginalEnv = process.env.PI_TOKEN_TELEMETRY;\n\t});\n\n\tafterEach(() => {\n\t\tprocess.stderr.write = originalWrite;\n\t\tif (originalEnv === undefined) {\n\t\t\tdelete process.env.PI_TOKEN_TELEMETRY;\n\t\t} else {\n\t\t\tprocess.env.PI_TOKEN_TELEMETRY = originalEnv;\n\t\t}\n\t});\n\n\ttest(\"silent by default (env var unset)\", () => {\n\t\tdelete process.env.PI_TOKEN_TELEMETRY;\n\t\temitTokenTelemetry(makeAssistantMessage());\n\t\tassert.equal(captured.length, 0);\n\t});\n\n\ttest(\"silent when env var has any non-'1' value\", () => {\n\t\tprocess.env.PI_TOKEN_TELEMETRY = \"true\"; // not literally \"1\"\n\t\temitTokenTelemetry(makeAssistantMessage());\n\t\tassert.equal(captured.length, 0, \"only literal '1' should enable telemetry\");\n\t});\n\n\ttest(\"emits a single JSON line when PI_TOKEN_TELEMETRY=1\", () => {\n\t\tprocess.env.PI_TOKEN_TELEMETRY = \"1\";\n\t\temitTokenTelemetry(makeAssistantMessage());\n\t\tassert.equal(captured.length, 1);\n\t\tassert.ok(captured[0].endsWith(\"\\n\"), \"line must terminate with newline\");\n\t\tconst parsed = JSON.parse(captured[0].trimEnd());\n\t\tassert.equal(parsed.model, \"claude-sonnet-4-6\");\n\t\tassert.equal(parsed.input, 100);\n\t\tassert.equal(parsed.output, 50);\n\t});\n\n\ttest(\"emitted JSON has the documented shape\", () => {\n\t\tprocess.env.PI_TOKEN_TELEMETRY = \"1\";\n\t\temitTokenTelemetry(makeAssistantMessage());\n\t\tconst parsed = JSON.parse(captured[0].trimEnd());\n\t\tconst keys = Object.keys(parsed).sort();\n\t\tassert.deepEqual(keys, [\n\t\t\t\"cacheHitRatio\",\n\t\t\t\"cacheRead\",\n\t\t\t\"cacheWrite\",\n\t\t\t\"costTotal\",\n\t\t\t\"input\",\n\t\t\t\"model\",\n\t\t\t\"output\",\n\t\t\t\"stopReason\",\n\t\t\t\"ts\",\n\t\t]);\n\t});\n\n\ttest(\"never throws — telemetry must not break the agent loop\", () => {\n\t\tprocess.env.PI_TOKEN_TELEMETRY = \"1\";\n\t\t// Force a write failure to exercise the swallow path.\n\t\tprocess.stderr.write = (() => {\n\t\t\tthrow new Error(\"simulated stderr failure\");\n\t\t}) as typeof process.stderr.write;\n\t\tassert.doesNotThrow(() => emitTokenTelemetry(makeAssistantMessage()));\n\t});\n});\n"]}

package/packages/pi-coding-agent/src/core/token-telemetry.ts

@@ -1,77 +0,0 @@
-// @gsd/pi-coding-agent + token-telemetry — opt-in per-call token observability
-//
-// Emits a single JSON line per assistant message to stderr when
-// `PI_TOKEN_TELEMETRY=1` is set. Captures the cache_read_input_tokens and
-// cache_creation_input_tokens fields the providers already extract — so we
-// can empirically measure prompt-cache effectiveness (e.g. for #5019 and
-// future cache strategy work). Off by default — no behavior change.
-//
-// Capture pattern: `PI_TOKEN_TELEMETRY=1 npm start 2> token-telemetry.jsonl`
-
-import type { AssistantMessage } from "@gsd/pi-ai";
-
-/** Schema of one telemetry line. JSON-stable for downstream ingestion. */
-export interface TokenTelemetryRecord {
-	ts: number;
-	model: string;
-	stopReason: string;
-	input: number;
-	output: number;
-	cacheRead: number;
-	cacheWrite: number;
-	/**
-	 * `usage.cost.total` from the provider. `0` when the provider's cost
-	 * registry has no rates for this model (e.g. unknown third-party providers)
-	 * — distinguish from a true zero-cost call by checking your model registry.
-	 */
-	costTotal: number;
-	/**
-	 * Fraction of new prompt tokens served from cache:
-	 * `cacheRead / (cacheRead + input)`. Range [0, 1].
-	 * - `0` when neither cacheRead nor input is present (no division by zero).
-	 * - `1` on a full cache hit (input = 0, cacheRead > 0).
-	 * Note: `input` here is `input_tokens` from the API, which already excludes
-	 * cache reads/writes — the denominator is total prompt tokens consumed.
-	 */
-	cacheHitRatio: number;
-}
-
-/** Build a telemetry record from a finished assistant message. */
-export function buildTokenTelemetryRecord(msg: AssistantMessage): TokenTelemetryRecord {
-	const input = msg.usage?.input ?? 0;
-	const output = msg.usage?.output ?? 0;
-	const cacheRead = msg.usage?.cacheRead ?? 0;
-	const cacheWrite = msg.usage?.cacheWrite ?? 0;
-	const costTotal = msg.usage?.cost?.total ?? 0;
-	const denom = cacheRead + input;
-	const cacheHitRatio = denom > 0 ? cacheRead / denom : 0;
-
-	return {
-		ts: msg.timestamp,
-		model: msg.model,
-		stopReason: msg.stopReason,
-		input,
-		output,
-		cacheRead,
-		cacheWrite,
-		costTotal,
-		cacheHitRatio,
-	};
-}
-
-/**
- * Emit a token-telemetry line if `PI_TOKEN_TELEMETRY=1`. No-op otherwise.
- *
- * Writes to stderr so it doesn't interfere with TUI/stdout. One JSON object
- * per line. Errors during emission are swallowed — telemetry must never
- * break the agent loop.
- */
-export function emitTokenTelemetry(msg: AssistantMessage): void {
-	if (process.env.PI_TOKEN_TELEMETRY !== "1") return;
-	try {
-		const record = buildTokenTelemetryRecord(msg);
-		process.stderr.write(`${JSON.stringify(record)}\n`);
-	} catch {
-		// Telemetry must never break the agent loop. Swallow.
-	}
-}