gsd-pi 2.58.0-dev.778d6ac → 2.58.0-dev.d63175c

package/packages/pi-ai/dist/utils/repair-tool-json.js

@@ -1,75 +0,0 @@
-/**
- * Repair malformed JSON in LLM tool-call arguments.
- *
- * LLMs sometimes copy YAML template formatting into JSON tool arguments,
- * producing patterns like:
- *
- *   "keyDecisions": - Used Web Notification API...,
- *   "keyFiles": - src-tauri/src/lib.rs — Extended...
- *
- * instead of valid JSON arrays:
- *
- *   "keyDecisions": ["Used Web Notification API..."],
- *   "keyFiles": ["src-tauri/src/lib.rs — Extended..."]
- *
- * This module detects and repairs such patterns before JSON.parse is called.
- *
- * @see https://github.com/gsd-build/gsd-2/issues/2660
- */
-/**
- * Detect whether a JSON string contains YAML-style bullet-list values
- * (i.e. `"key": - item` instead of `"key": ["item"]`).
- */
-export function hasYamlBulletLists(json) {
-    // Match: "key": followed by whitespace then a dash-space pattern (YAML bullet)
-    // The negative lookahead excludes negative numbers (e.g. "key": -1)
-    return /"\s*:\s*-\s+(?!\d)/.test(json);
-}
-/**
- * Attempt to repair YAML-style bullet lists embedded in a JSON string.
- *
- * Converts patterns like:
- *   "keyDecisions": - Used Web Notification API..., "keyFiles": - file1
- *
- * Into:
- *   "keyDecisions": ["Used Web Notification API..."], "keyFiles": ["file1"]
- *
- * Returns the original string unchanged if no YAML patterns are detected
- * or if the repair itself would produce invalid JSON.
- */
-export function repairToolJson(json) {
-    if (!hasYamlBulletLists(json)) {
-        return json;
-    }
-    // Strategy: find each `"key": - item1\n  - item2\n  - item3` region and
-    // wrap items in a JSON array.
-    //
-    // We work on the raw string because the JSON is not parseable yet.
-    // The pattern we target:
-    //   "someKey":\s*- item text (possibly multiline)
-    //   optionally followed by more `- item` lines
-    //   terminated by the next `"key":` or `}` or end of string.
-    let repaired = json;
-    // Match a key followed by YAML-style bullet list.
-    // Capture: (1) the key portion including colon, (2) the bullet-list body,
-    // (3) the separator (comma or empty) before the next key/bracket.
-    // The bullet list body ends at the next `"key":` or `}` or `]` or end of string.
-    const keyBulletPattern = /("(?:[^"\\]|\\.)*"\s*:\s*)(- .+?)(,?\s*)(?="(?:[^"\\]|\\.)*"\s*:|[}\]]|$)/gs;
-    repaired = repaired.replace(keyBulletPattern, (_match, keyPart, bulletBody, separator) => {
-        // Split the bullet body into individual items on `- ` boundaries.
-        // Items may contain embedded newlines for multi-line values.
-        const items = bulletBody
-            .split(/\n?\s*- /)
-            .filter((s) => s.trim().length > 0)
-            .map((s) => s.replace(/,\s*$/, "").trim());
-        // JSON-encode each item as a string, then wrap in an array.
-        const jsonArray = "[" + items.map((item) => JSON.stringify(item)).join(", ") + "]";
-        // Re-emit the separator (comma) so the next key is properly delimited
-        const sep = separator.trim() ? separator : (/^\s*"/.test(separator + "x") ? ", " : "");
-        return keyPart + jsonArray + sep;
-    });
-    // Strip trailing commas before } or ] (common in repaired JSON)
-    repaired = repaired.replace(/,(\s*[}\]])/g, "$1");
-    return repaired;
-}
-//# sourceMappingURL=repair-tool-json.js.map

package/packages/pi-ai/dist/utils/repair-tool-json.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"repair-tool-json.js","sourceRoot":"","sources":["../../src/utils/repair-tool-json.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAEH;;;GAGG;AACH,MAAM,UAAU,kBAAkB,CAAC,IAAY;IAC9C,+EAA+E;IAC/E,oEAAoE;IACpE,OAAO,oBAAoB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACxC,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,cAAc,CAAC,IAAY;IAC1C,IAAI,CAAC,kBAAkB,CAAC,IAAI,CAAC,EAAE,CAAC;QAC/B,OAAO,IAAI,CAAC;IACb,CAAC;IAED,wEAAwE;IACxE,8BAA8B;IAC9B,EAAE;IACF,mEAAmE;IACnE,yBAAyB;IACzB,kDAAkD;IAClD,+CAA+C;IAC/C,6DAA6D;IAE7D,IAAI,QAAQ,GAAG,IAAI,CAAC;IAEpB,kDAAkD;IAClD,0EAA0E;IAC1E,kEAAkE;IAClE,iFAAiF;IACjF,MAAM,gBAAgB,GACrB,6EAA6E,CAAC;IAE/E,QAAQ,GAAG,QAAQ,CAAC,OAAO,CAC1B,gBAAgB,EAChB,CAAC,MAAM,EAAE,OAAe,EAAE,UAAkB,EAAE,SAAiB,EAAE,EAAE;QAClE,kEAAkE;QAClE,6DAA6D;QAC7D,MAAM,KAAK,GAAG,UAAU;aACtB,KAAK,CAAC,UAAU,CAAC;aACjB,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,MAAM,GAAG,CAAC,CAAC;aAClC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC;QAE5C,4DAA4D;QAC5D,MAAM,SAAS,GAAG,GAAG,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,GAAG,CAAC;QAEnF,sEAAsE;QACtE,MAAM,GAAG,GAAG,SAAS,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,SAAS,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;QACvF,OAAO,OAAO,GAAG,SAAS,GAAG,GAAG,CAAC;IAClC,CAAC,CACD,CAAC;IAEF,gEAAgE;IAChE,QAAQ,GAAG,QAAQ,CAAC,OAAO,CAAC,cAAc,EAAE,IAAI,CAAC,CAAC;IAElD,OAAO,QAAQ,CAAC;AACjB,CAAC","sourcesContent":["/**\n * Repair malformed JSON in LLM tool-call arguments.\n *\n * LLMs sometimes copy YAML template formatting into JSON tool arguments,\n * producing patterns like:\n *\n *   \"keyDecisions\": - Used Web Notification API...,\n *   \"keyFiles\": - src-tauri/src/lib.rs — Extended...\n *\n * instead of valid JSON arrays:\n *\n *   \"keyDecisions\": [\"Used Web Notification API...\"],\n *   \"keyFiles\": [\"src-tauri/src/lib.rs — Extended...\"]\n *\n * This module detects and repairs such patterns before JSON.parse is called.\n *\n * @see https://github.com/gsd-build/gsd-2/issues/2660\n */\n\n/**\n * Detect whether a JSON string contains YAML-style bullet-list values\n * (i.e. `\"key\": - item` instead of `\"key\": [\"item\"]`).\n */\nexport function hasYamlBulletLists(json: string): boolean {\n\t// Match: \"key\": followed by whitespace then a dash-space pattern (YAML bullet)\n\t// The negative lookahead excludes negative numbers (e.g. \"key\": -1)\n\treturn /\"\\s*:\\s*-\\s+(?!\\d)/.test(json);\n}\n\n/**\n * Attempt to repair YAML-style bullet lists embedded in a JSON string.\n *\n * Converts patterns like:\n *   \"keyDecisions\": - Used Web Notification API..., \"keyFiles\": - file1\n *\n * Into:\n *   \"keyDecisions\": [\"Used Web Notification API...\"], \"keyFiles\": [\"file1\"]\n *\n * Returns the original string unchanged if no YAML patterns are detected\n * or if the repair itself would produce invalid JSON.\n */\nexport function repairToolJson(json: string): string {\n\tif (!hasYamlBulletLists(json)) {\n\t\treturn json;\n\t}\n\n\t// Strategy: find each `\"key\": - item1\\n  - item2\\n  - item3` region and\n\t// wrap items in a JSON array.\n\t//\n\t// We work on the raw string because the JSON is not parseable yet.\n\t// The pattern we target:\n\t//   \"someKey\":\\s*- item text (possibly multiline)\n\t//   optionally followed by more `- item` lines\n\t//   terminated by the next `\"key\":` or `}` or end of string.\n\n\tlet repaired = json;\n\n\t// Match a key followed by YAML-style bullet list.\n\t// Capture: (1) the key portion including colon, (2) the bullet-list body,\n\t// (3) the separator (comma or empty) before the next key/bracket.\n\t// The bullet list body ends at the next `\"key\":` or `}` or `]` or end of string.\n\tconst keyBulletPattern =\n\t\t/(\"(?:[^\"\\\\]|\\\\.)*\"\\s*:\\s*)(- .+?)(,?\\s*)(?=\"(?:[^\"\\\\]|\\\\.)*\"\\s*:|[}\\]]|$)/gs;\n\n\trepaired = repaired.replace(\n\t\tkeyBulletPattern,\n\t\t(_match, keyPart: string, bulletBody: string, separator: string) => {\n\t\t\t// Split the bullet body into individual items on `- ` boundaries.\n\t\t\t// Items may contain embedded newlines for multi-line values.\n\t\t\tconst items = bulletBody\n\t\t\t\t.split(/\\n?\\s*- /)\n\t\t\t\t.filter((s) => s.trim().length > 0)\n\t\t\t\t.map((s) => s.replace(/,\\s*$/, \"\").trim());\n\n\t\t\t// JSON-encode each item as a string, then wrap in an array.\n\t\t\tconst jsonArray = \"[\" + items.map((item) => JSON.stringify(item)).join(\", \") + \"]\";\n\n\t\t\t// Re-emit the separator (comma) so the next key is properly delimited\n\t\t\tconst sep = separator.trim() ? separator : (/^\\s*\"/.test(separator + \"x\") ? \", \" : \"\");\n\t\t\treturn keyPart + jsonArray + sep;\n\t\t},\n\t);\n\n\t// Strip trailing commas before } or ] (common in repaired JSON)\n\trepaired = repaired.replace(/,(\\s*[}\\]])/g, \"$1\");\n\n\treturn repaired;\n}\n"]}

package/packages/pi-ai/dist/utils/tests/repair-tool-json.test.d.ts

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

package/packages/pi-ai/dist/utils/tests/repair-tool-json.test.d.ts.map

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

package/packages/pi-ai/dist/utils/tests/repair-tool-json.test.js

@@ -1,73 +0,0 @@
-import { describe, test } from "node:test";
-import assert from "node:assert/strict";
-import { repairToolJson, hasYamlBulletLists } from "../repair-tool-json.js";
-describe("repairToolJson — YAML bullet list repair (#2660)", () => {
-    // ── Detection ──────────────────────────────────────────────────────────
-    test("hasYamlBulletLists detects YAML-style bullets", () => {
-        assert.equal(hasYamlBulletLists('"keyDecisions": - Used Web Notification API'), true);
-    });
-    test("hasYamlBulletLists ignores negative numbers", () => {
-        assert.equal(hasYamlBulletLists('"offset": -1'), false, "negative number should not be detected as YAML bullet");
-    });
-    test("hasYamlBulletLists returns false for valid JSON", () => {
-        assert.equal(hasYamlBulletLists('{"keyDecisions": ["item1", "item2"]}'), false);
-    });
-    // ── Single bullet item ────────────────────────────────────────────────
-    test("repairs single YAML bullet to JSON array", () => {
-        const malformed = '{"keyDecisions": - Used Web Notification API}';
-        const repaired = repairToolJson(malformed);
-        const parsed = JSON.parse(repaired);
-        assert.deepEqual(parsed.keyDecisions, ["Used Web Notification API"]);
-    });
-    // ── Multiple bullet items (newline-separated) ─────────────────────────
-    test("repairs multiple YAML bullets separated by newlines", () => {
-        const malformed = '{"keyDecisions": - Used Web Notification API\n  - Chose Tauri over Electron\n  - Adopted SQLite for storage, "title": "M005"}';
-        const repaired = repairToolJson(malformed);
-        const parsed = JSON.parse(repaired);
-        assert.deepEqual(parsed.keyDecisions, [
-            "Used Web Notification API",
-            "Chose Tauri over Electron",
-            "Adopted SQLite for storage",
-        ]);
-        assert.equal(parsed.title, "M005");
-    });
-    // ── Multiple fields with YAML bullets ─────────────────────────────────
-    test("repairs multiple fields each with YAML bullet lists", () => {
-        const malformed = '{"keyDecisions": - decision one\n  - decision two, "keyFiles": - src/lib.rs — Extended menu\n  - src/main.ts — Entry point, "title": "done"}';
-        const repaired = repairToolJson(malformed);
-        const parsed = JSON.parse(repaired);
-        assert.deepEqual(parsed.keyDecisions, ["decision one", "decision two"]);
-        assert.deepEqual(parsed.keyFiles, [
-            "src/lib.rs \u2014 Extended menu",
-            "src/main.ts \u2014 Entry point",
-        ]);
-        assert.equal(parsed.title, "done");
-    });
-    // ── Exact reproduction from issue #2660 ───────────────────────────────
-    test("repairs the exact malformed JSON from issue #2660", () => {
-        const malformed = `{"milestoneId": "M005", "title": "Native Desktop Polish", "oneLiner": "summary", "narrative": "details", "successCriteriaResults": "all pass", "definitionOfDoneResults": "all done", "requirementOutcomes": "met", "keyDecisions": - Used Web Notification API (new window.Notification()) instead of Tauri sendNotification wrapper, "keyFiles": - src-tauri/src/lib.rs \u2014 Extended menu builder with notification toggle, "lessonsLearned": - Always test notification permissions before sending, "followUps": "none", "deviations": "none", "verificationPassed": true}`;
-        const repaired = repairToolJson(malformed);
-        const parsed = JSON.parse(repaired);
-        assert.equal(parsed.milestoneId, "M005");
-        assert.equal(parsed.title, "Native Desktop Polish");
-        assert.ok(Array.isArray(parsed.keyDecisions), "keyDecisions should be an array");
-        assert.ok(parsed.keyDecisions[0].includes("Web Notification API"));
-        assert.ok(Array.isArray(parsed.keyFiles), "keyFiles should be an array");
-        assert.ok(parsed.keyFiles[0].includes("src-tauri/src/lib.rs"));
-        assert.ok(Array.isArray(parsed.lessonsLearned), "lessonsLearned should be an array");
-        assert.equal(parsed.verificationPassed, true);
-    });
-    // ── Passthrough for valid JSON ────────────────────────────────────────
-    test("returns valid JSON unchanged", () => {
-        const valid = '{"keyDecisions": ["item1", "item2"], "count": -5}';
-        const result = repairToolJson(valid);
-        assert.equal(result, valid, "valid JSON should be returned unchanged");
-    });
-    // ── Negative numbers are preserved ────────────────────────────────────
-    test("does not mangle negative numbers", () => {
-        const valid = '{"offset": -1, "limit": -100}';
-        const result = repairToolJson(valid);
-        assert.equal(result, valid);
-    });
-});
-//# sourceMappingURL=repair-tool-json.test.js.map

package/packages/pi-ai/dist/utils/tests/repair-tool-json.test.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"repair-tool-json.test.js","sourceRoot":"","sources":["../../../src/utils/tests/repair-tool-json.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AAC3C,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,EAAE,cAAc,EAAE,kBAAkB,EAAE,MAAM,wBAAwB,CAAC;AAE5E,QAAQ,CAAC,kDAAkD,EAAE,GAAG,EAAE;IACjE,0EAA0E;IAE1E,IAAI,CAAC,+CAA+C,EAAE,GAAG,EAAE;QAC1D,MAAM,CAAC,KAAK,CACX,kBAAkB,CAAC,6CAA6C,CAAC,EACjE,IAAI,CACJ,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,6CAA6C,EAAE,GAAG,EAAE;QACxD,MAAM,CAAC,KAAK,CACX,kBAAkB,CAAC,cAAc,CAAC,EAClC,KAAK,EACL,uDAAuD,CACvD,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,iDAAiD,EAAE,GAAG,EAAE;QAC5D,MAAM,CAAC,KAAK,CACX,kBAAkB,CAAC,sCAAsC,CAAC,EAC1D,KAAK,CACL,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,yEAAyE;IAEzE,IAAI,CAAC,0CAA0C,EAAE,GAAG,EAAE;QACrD,MAAM,SAAS,GAAG,+CAA+C,CAAC;QAClE,MAAM,QAAQ,GAAG,cAAc,CAAC,SAAS,CAAC,CAAC;QAC3C,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;QACpC,MAAM,CAAC,SAAS,CAAC,MAAM,CAAC,YAAY,EAAE,CAAC,2BAA2B,CAAC,CAAC,CAAC;IACtE,CAAC,CAAC,CAAC;IAEH,yEAAyE;IAEzE,IAAI,CAAC,qDAAqD,EAAE,GAAG,EAAE;QAChE,MAAM,SAAS,GACd,+HAA+H,CAAC;QACjI,MAAM,QAAQ,GAAG,cAAc,CAAC,SAAS,CAAC,CAAC;QAC3C,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;QACpC,MAAM,CAAC,SAAS,CAAC,MAAM,CAAC,YAAY,EAAE;YACrC,2BAA2B;YAC3B,2BAA2B;YAC3B,4BAA4B;SAC5B,CAAC,CAAC;QACH,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,EAAE,MAAM,CAAC,CAAC;IACpC,CAAC,CAAC,CAAC;IAEH,yEAAyE;IAEzE,IAAI,CAAC,qDAAqD,EAAE,GAAG,EAAE;QAChE,MAAM,SAAS,GACd,8IAA8I,CAAC;QAChJ,MAAM,QAAQ,GAAG,cAAc,CAAC,SAAS,CAAC,CAAC;QAC3C,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;QACpC,MAAM,CAAC,SAAS,CAAC,MAAM,CAAC,YAAY,EAAE,CAAC,cAAc,EAAE,cAAc,CAAC,CAAC,CAAC;QACxE,MAAM,CAAC,SAAS,CAAC,MAAM,CAAC,QAAQ,EAAE;YACjC,iCAAiC;YACjC,gCAAgC;SAChC,CAAC,CAAC;QACH,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,EAAE,MAAM,CAAC,CAAC;IACpC,CAAC,CAAC,CAAC;IAEH,yEAAyE;IAEzE,IAAI,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC9D,MAAM,SAAS,GAAG,kjBAAkjB,CAAC;QAErkB,MAAM,QAAQ,GAAG,cAAc,CAAC,SAAS,CAAC,CAAC;QAC3C,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;QAEpC,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,WAAW,EAAE,MAAM,CAAC,CAAC;QACzC,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,EAAE,uBAAuB,CAAC,CAAC;QACpD,MAAM,CAAC,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,YAAY,CAAC,EAAE,iCAAiC,CAAC,CAAC;QACjF,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,sBAAsB,CAAC,CAAC,CAAC;QACnE,MAAM,CAAC,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,EAAE,6BAA6B,CAAC,CAAC;QACzE,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,sBAAsB,CAAC,CAAC,CAAC;QAC/D,MAAM,CAAC,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,cAAc,CAAC,EAAE,mCAAmC,CAAC,CAAC;QACrF,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,kBAAkB,EAAE,IAAI,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,yEAAyE;IAEzE,IAAI,CAAC,8BAA8B,EAAE,GAAG,EAAE;QACzC,MAAM,KAAK,GAAG,mDAAmD,CAAC;QAClE,MAAM,MAAM,GAAG,cAAc,CAAC,KAAK,CAAC,CAAC;QACrC,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,KAAK,EAAE,yCAAyC,CAAC,CAAC;IACxE,CAAC,CAAC,CAAC;IAEH,yEAAyE;IAEzE,IAAI,CAAC,kCAAkC,EAAE,GAAG,EAAE;QAC7C,MAAM,KAAK,GAAG,+BAA+B,CAAC;QAC9C,MAAM,MAAM,GAAG,cAAc,CAAC,KAAK,CAAC,CAAC;QACrC,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,KAAK,CAAC,CAAC;IAC7B,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC","sourcesContent":["import { describe, test } from \"node:test\";\nimport assert from \"node:assert/strict\";\nimport { repairToolJson, hasYamlBulletLists } from \"../repair-tool-json.js\";\n\ndescribe(\"repairToolJson — YAML bullet list repair (#2660)\", () => {\n\t// ── Detection ──────────────────────────────────────────────────────────\n\n\ttest(\"hasYamlBulletLists detects YAML-style bullets\", () => {\n\t\tassert.equal(\n\t\t\thasYamlBulletLists('\"keyDecisions\": - Used Web Notification API'),\n\t\t\ttrue,\n\t\t);\n\t});\n\n\ttest(\"hasYamlBulletLists ignores negative numbers\", () => {\n\t\tassert.equal(\n\t\t\thasYamlBulletLists('\"offset\": -1'),\n\t\t\tfalse,\n\t\t\t\"negative number should not be detected as YAML bullet\",\n\t\t);\n\t});\n\n\ttest(\"hasYamlBulletLists returns false for valid JSON\", () => {\n\t\tassert.equal(\n\t\t\thasYamlBulletLists('{\"keyDecisions\": [\"item1\", \"item2\"]}'),\n\t\t\tfalse,\n\t\t);\n\t});\n\n\t// ── Single bullet item ────────────────────────────────────────────────\n\n\ttest(\"repairs single YAML bullet to JSON array\", () => {\n\t\tconst malformed = '{\"keyDecisions\": - Used Web Notification API}';\n\t\tconst repaired = repairToolJson(malformed);\n\t\tconst parsed = JSON.parse(repaired);\n\t\tassert.deepEqual(parsed.keyDecisions, [\"Used Web Notification API\"]);\n\t});\n\n\t// ── Multiple bullet items (newline-separated) ─────────────────────────\n\n\ttest(\"repairs multiple YAML bullets separated by newlines\", () => {\n\t\tconst malformed =\n\t\t\t'{\"keyDecisions\": - Used Web Notification API\\n  - Chose Tauri over Electron\\n  - Adopted SQLite for storage, \"title\": \"M005\"}';\n\t\tconst repaired = repairToolJson(malformed);\n\t\tconst parsed = JSON.parse(repaired);\n\t\tassert.deepEqual(parsed.keyDecisions, [\n\t\t\t\"Used Web Notification API\",\n\t\t\t\"Chose Tauri over Electron\",\n\t\t\t\"Adopted SQLite for storage\",\n\t\t]);\n\t\tassert.equal(parsed.title, \"M005\");\n\t});\n\n\t// ── Multiple fields with YAML bullets ─────────────────────────────────\n\n\ttest(\"repairs multiple fields each with YAML bullet lists\", () => {\n\t\tconst malformed =\n\t\t\t'{\"keyDecisions\": - decision one\\n  - decision two, \"keyFiles\": - src/lib.rs — Extended menu\\n  - src/main.ts — Entry point, \"title\": \"done\"}';\n\t\tconst repaired = repairToolJson(malformed);\n\t\tconst parsed = JSON.parse(repaired);\n\t\tassert.deepEqual(parsed.keyDecisions, [\"decision one\", \"decision two\"]);\n\t\tassert.deepEqual(parsed.keyFiles, [\n\t\t\t\"src/lib.rs \\u2014 Extended menu\",\n\t\t\t\"src/main.ts \\u2014 Entry point\",\n\t\t]);\n\t\tassert.equal(parsed.title, \"done\");\n\t});\n\n\t// ── Exact reproduction from issue #2660 ───────────────────────────────\n\n\ttest(\"repairs the exact malformed JSON from issue #2660\", () => {\n\t\tconst malformed = `{\"milestoneId\": \"M005\", \"title\": \"Native Desktop Polish\", \"oneLiner\": \"summary\", \"narrative\": \"details\", \"successCriteriaResults\": \"all pass\", \"definitionOfDoneResults\": \"all done\", \"requirementOutcomes\": \"met\", \"keyDecisions\": - Used Web Notification API (new window.Notification()) instead of Tauri sendNotification wrapper, \"keyFiles\": - src-tauri/src/lib.rs \\u2014 Extended menu builder with notification toggle, \"lessonsLearned\": - Always test notification permissions before sending, \"followUps\": \"none\", \"deviations\": \"none\", \"verificationPassed\": true}`;\n\n\t\tconst repaired = repairToolJson(malformed);\n\t\tconst parsed = JSON.parse(repaired);\n\n\t\tassert.equal(parsed.milestoneId, \"M005\");\n\t\tassert.equal(parsed.title, \"Native Desktop Polish\");\n\t\tassert.ok(Array.isArray(parsed.keyDecisions), \"keyDecisions should be an array\");\n\t\tassert.ok(parsed.keyDecisions[0].includes(\"Web Notification API\"));\n\t\tassert.ok(Array.isArray(parsed.keyFiles), \"keyFiles should be an array\");\n\t\tassert.ok(parsed.keyFiles[0].includes(\"src-tauri/src/lib.rs\"));\n\t\tassert.ok(Array.isArray(parsed.lessonsLearned), \"lessonsLearned should be an array\");\n\t\tassert.equal(parsed.verificationPassed, true);\n\t});\n\n\t// ── Passthrough for valid JSON ────────────────────────────────────────\n\n\ttest(\"returns valid JSON unchanged\", () => {\n\t\tconst valid = '{\"keyDecisions\": [\"item1\", \"item2\"], \"count\": -5}';\n\t\tconst result = repairToolJson(valid);\n\t\tassert.equal(result, valid, \"valid JSON should be returned unchanged\");\n\t});\n\n\t// ── Negative numbers are preserved ────────────────────────────────────\n\n\ttest(\"does not mangle negative numbers\", () => {\n\t\tconst valid = '{\"offset\": -1, \"limit\": -100}';\n\t\tconst result = repairToolJson(valid);\n\t\tassert.equal(result, valid);\n\t});\n});\n"]}

package/packages/pi-ai/src/providers/anthropic-shared.test.ts

@@ -1,29 +0,0 @@
-import { describe, it } from "node:test";
-import assert from "node:assert/strict";
-import { mapStopReason } from "./anthropic-shared.js";
-
-describe("mapStopReason", () => {
-	it("maps end_turn to stop", () => {
-		assert.equal(mapStopReason("end_turn"), "stop");
-	});
-
-	it("maps max_tokens to length", () => {
-		assert.equal(mapStopReason("max_tokens"), "length");
-	});
-
-	it("maps tool_use to toolUse", () => {
-		assert.equal(mapStopReason("tool_use"), "toolUse");
-	});
-
-	it("maps pause_turn to pauseTurn (not stop)", () => {
-		// pause_turn means the server paused a long-running turn (e.g. native
-		// web search hit its iteration limit). Mapping it to "stop" causes the
-		// agent loop to exit, leaving an incomplete server_tool_use block in
-		// history which triggers a 400 on the next request.
-		assert.equal(mapStopReason("pause_turn"), "pauseTurn");
-	});
-
-	it("throws on unknown stop reason", () => {
-		assert.throws(() => mapStopReason("bogus"), /Unhandled stop reason/);
-	});
-});

package/packages/pi-ai/src/utils/repair-tool-json.ts

@@ -1,88 +0,0 @@
-/**
- * Repair malformed JSON in LLM tool-call arguments.
- *
- * LLMs sometimes copy YAML template formatting into JSON tool arguments,
- * producing patterns like:
- *
- *   "keyDecisions": - Used Web Notification API...,
- *   "keyFiles": - src-tauri/src/lib.rs — Extended...
- *
- * instead of valid JSON arrays:
- *
- *   "keyDecisions": ["Used Web Notification API..."],
- *   "keyFiles": ["src-tauri/src/lib.rs — Extended..."]
- *
- * This module detects and repairs such patterns before JSON.parse is called.
- *
- * @see https://github.com/gsd-build/gsd-2/issues/2660
- */
-
-/**
- * Detect whether a JSON string contains YAML-style bullet-list values
- * (i.e. `"key": - item` instead of `"key": ["item"]`).
- */
-export function hasYamlBulletLists(json: string): boolean {
-	// Match: "key": followed by whitespace then a dash-space pattern (YAML bullet)
-	// The negative lookahead excludes negative numbers (e.g. "key": -1)
-	return /"\s*:\s*-\s+(?!\d)/.test(json);
-}
-
-/**
- * Attempt to repair YAML-style bullet lists embedded in a JSON string.
- *
- * Converts patterns like:
- *   "keyDecisions": - Used Web Notification API..., "keyFiles": - file1
- *
- * Into:
- *   "keyDecisions": ["Used Web Notification API..."], "keyFiles": ["file1"]
- *
- * Returns the original string unchanged if no YAML patterns are detected
- * or if the repair itself would produce invalid JSON.
- */
-export function repairToolJson(json: string): string {
-	if (!hasYamlBulletLists(json)) {
-		return json;
-	}
-
-	// Strategy: find each `"key": - item1\n  - item2\n  - item3` region and
-	// wrap items in a JSON array.
-	//
-	// We work on the raw string because the JSON is not parseable yet.
-	// The pattern we target:
-	//   "someKey":\s*- item text (possibly multiline)
-	//   optionally followed by more `- item` lines
-	//   terminated by the next `"key":` or `}` or end of string.
-
-	let repaired = json;
-
-	// Match a key followed by YAML-style bullet list.
-	// Capture: (1) the key portion including colon, (2) the bullet-list body,
-	// (3) the separator (comma or empty) before the next key/bracket.
-	// The bullet list body ends at the next `"key":` or `}` or `]` or end of string.
-	const keyBulletPattern =
-		/("(?:[^"\\]|\\.)*"\s*:\s*)(- .+?)(,?\s*)(?="(?:[^"\\]|\\.)*"\s*:|[}\]]|$)/gs;
-
-	repaired = repaired.replace(
-		keyBulletPattern,
-		(_match, keyPart: string, bulletBody: string, separator: string) => {
-			// Split the bullet body into individual items on `- ` boundaries.
-			// Items may contain embedded newlines for multi-line values.
-			const items = bulletBody
-				.split(/\n?\s*- /)
-				.filter((s) => s.trim().length > 0)
-				.map((s) => s.replace(/,\s*$/, "").trim());
-
-			// JSON-encode each item as a string, then wrap in an array.
-			const jsonArray = "[" + items.map((item) => JSON.stringify(item)).join(", ") + "]";
-
-			// Re-emit the separator (comma) so the next key is properly delimited
-			const sep = separator.trim() ? separator : (/^\s*"/.test(separator + "x") ? ", " : "");
-			return keyPart + jsonArray + sep;
-		},
-	);
-
-	// Strip trailing commas before } or ] (common in repaired JSON)
-	repaired = repaired.replace(/,(\s*[}\]])/g, "$1");
-
-	return repaired;
-}

package/packages/pi-ai/src/utils/tests/repair-tool-json.test.ts

@@ -1,102 +0,0 @@
-import { describe, test } from "node:test";
-import assert from "node:assert/strict";
-import { repairToolJson, hasYamlBulletLists } from "../repair-tool-json.js";
-
-describe("repairToolJson — YAML bullet list repair (#2660)", () => {
-	// ── Detection ──────────────────────────────────────────────────────────
-
-	test("hasYamlBulletLists detects YAML-style bullets", () => {
-		assert.equal(
-			hasYamlBulletLists('"keyDecisions": - Used Web Notification API'),
-			true,
-		);
-	});
-
-	test("hasYamlBulletLists ignores negative numbers", () => {
-		assert.equal(
-			hasYamlBulletLists('"offset": -1'),
-			false,
-			"negative number should not be detected as YAML bullet",
-		);
-	});
-
-	test("hasYamlBulletLists returns false for valid JSON", () => {
-		assert.equal(
-			hasYamlBulletLists('{"keyDecisions": ["item1", "item2"]}'),
-			false,
-		);
-	});
-
-	// ── Single bullet item ────────────────────────────────────────────────
-
-	test("repairs single YAML bullet to JSON array", () => {
-		const malformed = '{"keyDecisions": - Used Web Notification API}';
-		const repaired = repairToolJson(malformed);
-		const parsed = JSON.parse(repaired);
-		assert.deepEqual(parsed.keyDecisions, ["Used Web Notification API"]);
-	});
-
-	// ── Multiple bullet items (newline-separated) ─────────────────────────
-
-	test("repairs multiple YAML bullets separated by newlines", () => {
-		const malformed =
-			'{"keyDecisions": - Used Web Notification API\n  - Chose Tauri over Electron\n  - Adopted SQLite for storage, "title": "M005"}';
-		const repaired = repairToolJson(malformed);
-		const parsed = JSON.parse(repaired);
-		assert.deepEqual(parsed.keyDecisions, [
-			"Used Web Notification API",
-			"Chose Tauri over Electron",
-			"Adopted SQLite for storage",
-		]);
-		assert.equal(parsed.title, "M005");
-	});
-
-	// ── Multiple fields with YAML bullets ─────────────────────────────────
-
-	test("repairs multiple fields each with YAML bullet lists", () => {
-		const malformed =
-			'{"keyDecisions": - decision one\n  - decision two, "keyFiles": - src/lib.rs — Extended menu\n  - src/main.ts — Entry point, "title": "done"}';
-		const repaired = repairToolJson(malformed);
-		const parsed = JSON.parse(repaired);
-		assert.deepEqual(parsed.keyDecisions, ["decision one", "decision two"]);
-		assert.deepEqual(parsed.keyFiles, [
-			"src/lib.rs \u2014 Extended menu",
-			"src/main.ts \u2014 Entry point",
-		]);
-		assert.equal(parsed.title, "done");
-	});
-
-	// ── Exact reproduction from issue #2660 ───────────────────────────────
-
-	test("repairs the exact malformed JSON from issue #2660", () => {
-		const malformed = `{"milestoneId": "M005", "title": "Native Desktop Polish", "oneLiner": "summary", "narrative": "details", "successCriteriaResults": "all pass", "definitionOfDoneResults": "all done", "requirementOutcomes": "met", "keyDecisions": - Used Web Notification API (new window.Notification()) instead of Tauri sendNotification wrapper, "keyFiles": - src-tauri/src/lib.rs \u2014 Extended menu builder with notification toggle, "lessonsLearned": - Always test notification permissions before sending, "followUps": "none", "deviations": "none", "verificationPassed": true}`;
-
-		const repaired = repairToolJson(malformed);
-		const parsed = JSON.parse(repaired);
-
-		assert.equal(parsed.milestoneId, "M005");
-		assert.equal(parsed.title, "Native Desktop Polish");
-		assert.ok(Array.isArray(parsed.keyDecisions), "keyDecisions should be an array");
-		assert.ok(parsed.keyDecisions[0].includes("Web Notification API"));
-		assert.ok(Array.isArray(parsed.keyFiles), "keyFiles should be an array");
-		assert.ok(parsed.keyFiles[0].includes("src-tauri/src/lib.rs"));
-		assert.ok(Array.isArray(parsed.lessonsLearned), "lessonsLearned should be an array");
-		assert.equal(parsed.verificationPassed, true);
-	});
-
-	// ── Passthrough for valid JSON ────────────────────────────────────────
-
-	test("returns valid JSON unchanged", () => {
-		const valid = '{"keyDecisions": ["item1", "item2"], "count": -5}';
-		const result = repairToolJson(valid);
-		assert.equal(result, valid, "valid JSON should be returned unchanged");
-	});
-
-	// ── Negative numbers are preserved ────────────────────────────────────
-
-	test("does not mangle negative numbers", () => {
-		const valid = '{"offset": -1, "limit": -100}';
-		const result = repairToolJson(valid);
-		assert.equal(result, valid);
-	});
-});

package/packages/pi-coding-agent/dist/core/compaction/compaction.test.d.ts

@@ -1,6 +0,0 @@
-/**
- * Tests for chunked compaction fallback when messages exceed model context window.
- * Regression test for #2932.
- */
-export {};
-//# sourceMappingURL=compaction.test.d.ts.map

package/packages/pi-coding-agent/dist/core/compaction/compaction.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"compaction.test.d.ts","sourceRoot":"","sources":["../../../src/core/compaction/compaction.test.ts"],"names":[],"mappings":"AAAA;;;GAGG"}

package/packages/pi-coding-agent/dist/core/compaction/compaction.test.js

@@ -1,176 +0,0 @@
-/**
- * Tests for chunked compaction fallback when messages exceed model context window.
- * Regression test for #2932.
- */
-import assert from "node:assert/strict";
-import { describe, it, mock } from "node:test";
-import { generateSummary, estimateTokens, chunkMessages } from "./compaction.js";
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-/** Create a user message with approximately `tokenCount` tokens (chars = tokens * 4). */
-function makeUserMessage(tokenCount) {
-    const text = "x".repeat(tokenCount * 4);
-    return { role: "user", content: text };
-}
-/** Create a mock model with a given context window. */
-function makeModel(contextWindow) {
-    return {
-        id: "test-model",
-        name: "Test Model",
-        api: "anthropic-messages",
-        provider: "anthropic",
-        baseUrl: "https://api.test",
-        reasoning: false,
-        input: ["text"],
-        cost: { input: 0, output: 0, cacheRead: 0, cacheWrite: 0 },
-        contextWindow,
-        maxTokens: 4096,
-    };
-}
-function makeFakeResponse(text) {
-    return {
-        content: [{ type: "text", text }],
-        stopReason: "end_turn",
-    };
-}
-// ---------------------------------------------------------------------------
-// chunkMessages tests
-// ---------------------------------------------------------------------------
-describe("chunkMessages", () => {
-    it("returns a single chunk when messages fit in budget", () => {
-        const messages = [
-            makeUserMessage(1_000),
-            makeUserMessage(1_000),
-        ];
-        const chunks = chunkMessages(messages, 100_000);
-        assert.equal(chunks.length, 1);
-        assert.equal(chunks[0].length, 2);
-    });
-    it("splits messages into multiple chunks when they exceed budget", () => {
-        const messages = [
-            makeUserMessage(50_000),
-            makeUserMessage(50_000),
-            makeUserMessage(50_000),
-        ];
-        // Budget of 80k tokens means each 50k message gets its own chunk
-        // (or two fit together if budget allows)
-        const chunks = chunkMessages(messages, 80_000);
-        assert.ok(chunks.length > 1, `Expected multiple chunks, got ${chunks.length}`);
-        // All messages should be present across chunks
-        const totalMessages = chunks.reduce((sum, c) => sum + c.length, 0);
-        assert.equal(totalMessages, 3);
-    });
-    it("puts a single oversized message in its own chunk", () => {
-        const messages = [
-            makeUserMessage(200_000), // Way over any reasonable budget
-        ];
-        const chunks = chunkMessages(messages, 80_000);
-        assert.equal(chunks.length, 1);
-        assert.equal(chunks[0].length, 1);
-    });
-    it("preserves message order across chunks", () => {
-        // Create messages with identifiable sizes
-        const messages = [
-            makeUserMessage(30_000), // ~30k tokens
-            makeUserMessage(30_000),
-            makeUserMessage(30_000),
-            makeUserMessage(30_000),
-        ];
-        const chunks = chunkMessages(messages, 50_000);
-        // Reconstruct original order
-        const flat = chunks.flat();
-        assert.equal(flat.length, 4);
-        for (let i = 0; i < flat.length; i++) {
-            assert.strictEqual(flat[i], messages[i], `Message ${i} should be in order`);
-        }
-    });
-});
-// ---------------------------------------------------------------------------
-// generateSummary chunked fallback tests
-// ---------------------------------------------------------------------------
-describe("generateSummary — chunked fallback (#2932)", () => {
-    it("calls _completeFn multiple times when messages exceed model context window", async () => {
-        // Arrange: 3 messages of ~80k tokens each = ~240k total, model has 200k window
-        const messages = [
-            makeUserMessage(80_000),
-            makeUserMessage(80_000),
-            makeUserMessage(80_000),
-        ];
-        const model = makeModel(200_000);
-        const reserveTokens = 16_384;
-        // Verify our test setup: messages really do exceed the model window
-        let totalTokens = 0;
-        for (const m of messages)
-            totalTokens += estimateTokens(m);
-        assert.ok(totalTokens > model.contextWindow, `Test setup: ${totalTokens} tokens should exceed ${model.contextWindow} context window`);
-        // Track calls
-        const calls = [];
-        const mockComplete = mock.fn(async (_model, context, _options) => {
-            const userMsg = context.messages?.[0];
-            const text = typeof userMsg?.content === "string"
-                ? userMsg.content
-                : userMsg?.content?.[0]?.text ?? "";
-            if (text.includes("<previous-summary>")) {
-                calls.push("update");
-            }
-            else {
-                calls.push("initial");
-            }
-            return makeFakeResponse("Summary of chunk");
-        });
-        const summary = await generateSummary(messages, model, reserveTokens, undefined, // apiKey
-        undefined, // signal
-        undefined, // customInstructions
-        undefined, // previousSummary
-        mockComplete);
-        // Assert: should have called completeSimple more than once (chunked)
-        assert.ok(mockComplete.mock.callCount() > 1, `Expected multiple calls for chunked summarization, got ${mockComplete.mock.callCount()}`);
-        // First call should be an initial summary, subsequent should be updates
-        assert.equal(calls[0], "initial", "First chunk should use initial summarization prompt");
-        for (let i = 1; i < calls.length; i++) {
-            assert.equal(calls[i], "update", `Chunk ${i + 1} should use update summarization prompt`);
-        }
-        // Should return a non-empty summary
-        assert.ok(summary.length > 0, "Summary should not be empty");
-    });
-    it("uses single-pass when messages fit within model context window", async () => {
-        const messages = [
-            makeUserMessage(10_000),
-            makeUserMessage(10_000),
-        ];
-        const model = makeModel(200_000);
-        const reserveTokens = 16_384;
-        // Verify test setup
-        let totalTokens = 0;
-        for (const m of messages)
-            totalTokens += estimateTokens(m);
-        assert.ok(totalTokens < model.contextWindow, `Test setup: ${totalTokens} tokens should fit in ${model.contextWindow} context window`);
-        const mockComplete = mock.fn(async () => makeFakeResponse("Single pass summary"));
-        await generateSummary(messages, model, reserveTokens, undefined, undefined, undefined, undefined, mockComplete);
-        assert.equal(mockComplete.mock.callCount(), 1, "Should use single-pass summarization when messages fit in context window");
-    });
-    it("passes previousSummary through chunked summarization", async () => {
-        const messages = [
-            makeUserMessage(80_000),
-            makeUserMessage(80_000),
-            makeUserMessage(80_000),
-        ];
-        const model = makeModel(200_000);
-        const reserveTokens = 16_384;
-        const previousSummary = "Previous session summary content";
-        const prompts = [];
-        const mockComplete = mock.fn(async (_model, context) => {
-            const userMsg = context.messages?.[0];
-            const text = typeof userMsg?.content === "string"
-                ? userMsg.content
-                : userMsg?.content?.[0]?.text ?? "";
-            prompts.push(text);
-            return makeFakeResponse("Chunk summary");
-        });
-        await generateSummary(messages, model, reserveTokens, undefined, undefined, undefined, previousSummary, mockComplete);
-        // First chunk should include the previousSummary
-        assert.ok(prompts[0].includes(previousSummary), "First chunk should incorporate the previousSummary");
-    });
-});
-//# sourceMappingURL=compaction.test.js.map

package/packages/pi-coding-agent/dist/core/compaction/compaction.test.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"compaction.test.js","sourceRoot":"","sources":["../../../src/core/compaction/compaction.test.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AAK/C,OAAO,EAAE,eAAe,EAAE,cAAc,EAAE,aAAa,EAAE,MAAM,iBAAiB,CAAC;AAEjF,8EAA8E;AAC9E,UAAU;AACV,8EAA8E;AAE9E,yFAAyF;AACzF,SAAS,eAAe,CAAC,UAAkB;IAC1C,MAAM,IAAI,GAAG,GAAG,CAAC,MAAM,CAAC,UAAU,GAAG,CAAC,CAAC,CAAC;IACxC,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,IAAI,EAA6B,CAAC;AACnE,CAAC;AAED,uDAAuD;AACvD,SAAS,SAAS,CAAC,aAAqB;IACvC,OAAO;QACN,EAAE,EAAE,YAAY;QAChB,IAAI,EAAE,YAAY;QAClB,GAAG,EAAE,oBAAoB;QACzB,QAAQ,EAAE,WAAW;QACrB,OAAO,EAAE,kBAAkB;QAC3B,SAAS,EAAE,KAAK;QAChB,KAAK,EAAE,CAAC,MAAM,CAAC;QACf,IAAI,EAAE,EAAE,KAAK,EAAE,CAAC,EAAE,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,UAAU,EAAE,CAAC,EAAE;QAC1D,aAAa;QACb,SAAS,EAAE,IAAI;KACD,CAAC;AACjB,CAAC;AAED,SAAS,gBAAgB,CAAC,IAAY;IACrC,OAAO;QACN,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;QACjC,UAAU,EAAE,UAAU;KACS,CAAC;AAClC,CAAC;AAED,8EAA8E;AAC9E,sBAAsB;AACtB,8EAA8E;AAE9E,QAAQ,CAAC,eAAe,EAAE,GAAG,EAAE;IAC9B,EAAE,CAAC,oDAAoD,EAAE,GAAG,EAAE;QAC7D,MAAM,QAAQ,GAAmB;YAChC,eAAe,CAAC,KAAK,CAAC;YACtB,eAAe,CAAC,KAAK,CAAC;SACtB,CAAC;QACF,MAAM,MAAM,GAAG,aAAa,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAC;QAChD,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QAC/B,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;QACvE,MAAM,QAAQ,GAAmB;YAChC,eAAe,CAAC,MAAM,CAAC;YACvB,eAAe,CAAC,MAAM,CAAC;YACvB,eAAe,CAAC,MAAM,CAAC;SACvB,CAAC;QACF,iEAAiE;QACjE,yCAAyC;QACzC,MAAM,MAAM,GAAG,aAAa,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;QAC/C,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,iCAAiC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC;QAC/E,+CAA+C;QAC/C,MAAM,aAAa,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,GAAG,GAAG,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QACnE,MAAM,CAAC,KAAK,CAAC,aAAa,EAAE,CAAC,CAAC,CAAC;IAChC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC3D,MAAM,QAAQ,GAAmB;YAChC,eAAe,CAAC,OAAO,CAAC,EAAE,iCAAiC;SAC3D,CAAC;QACF,MAAM,MAAM,GAAG,aAAa,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;QAC/C,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QAC/B,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;QAChD,0CAA0C;QAC1C,MAAM,QAAQ,GAAmB;YAChC,eAAe,CAAC,MAAM,CAAC,EAAE,cAAc;YACvC,eAAe,CAAC,MAAM,CAAC;YACvB,eAAe,CAAC,MAAM,CAAC;YACvB,eAAe,CAAC,MAAM,CAAC;SACvB,CAAC;QACF,MAAM,MAAM,GAAG,aAAa,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;QAC/C,6BAA6B;QAC7B,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,EAAE,CAAC;QAC3B,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QAC7B,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,IAAI,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;YACtC,MAAM,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,QAAQ,CAAC,CAAC,CAAC,EAAE,WAAW,CAAC,qBAAqB,CAAC,CAAC;QAC7E,CAAC;IACF,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,8EAA8E;AAC9E,yCAAyC;AACzC,8EAA8E;AAE9E,QAAQ,CAAC,4CAA4C,EAAE,GAAG,EAAE;IAC3D,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;QAC3F,+EAA+E;QAC/E,MAAM,QAAQ,GAAmB;YAChC,eAAe,CAAC,MAAM,CAAC;YACvB,eAAe,CAAC,MAAM,CAAC;YACvB,eAAe,CAAC,MAAM,CAAC;SACvB,CAAC;QACF,MAAM,KAAK,GAAG,SAAS,CAAC,OAAO,CAAC,CAAC;QACjC,MAAM,aAAa,GAAG,MAAM,CAAC;QAE7B,oEAAoE;QACpE,IAAI,WAAW,GAAG,CAAC,CAAC;QACpB,KAAK,MAAM,CAAC,IAAI,QAAQ;YAAE,WAAW,IAAI,cAAc,CAAC,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,EAAE,CACR,WAAW,GAAG,KAAK,CAAC,aAAa,EACjC,eAAe,WAAW,yBAAyB,KAAK,CAAC,aAAa,iBAAiB,CACvF,CAAC;QAEF,cAAc;QACd,MAAM,KAAK,GAAa,EAAE,CAAC;QAC3B,MAAM,YAAY,GAAG,IAAI,CAAC,EAAE,CAAC,KAAK,EAAE,MAAW,EAAE,OAAY,EAAE,QAAa,EAAE,EAAE;YAC/E,MAAM,OAAO,GAAG,OAAO,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC,CAAC;YACtC,MAAM,IAAI,GACT,OAAO,OAAO,EAAE,OAAO,KAAK,QAAQ;gBACnC,CAAC,CAAC,OAAO,CAAC,OAAO;gBACjB,CAAC,CAAC,OAAO,EAAE,OAAO,EAAE,CAAC,CAAC,CAAC,EAAE,IAAI,IAAI,EAAE,CAAC;YAEtC,IAAI,IAAI,CAAC,QAAQ,CAAC,oBAAoB,CAAC,EAAE,CAAC;gBACzC,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YACtB,CAAC;iBAAM,CAAC;gBACP,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;YACvB,CAAC;YACD,OAAO,gBAAgB,CAAC,kBAAkB,CAAC,CAAC;QAC7C,CAAC,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG,MAAM,eAAe,CACpC,QAAQ,EACR,KAAK,EACL,aAAa,EACb,SAAS,EAAE,SAAS;QACpB,SAAS,EAAE,SAAS;QACpB,SAAS,EAAE,qBAAqB;QAChC,SAAS,EAAE,kBAAkB;QAC7B,YAAY,CACZ,CAAC;QAEF,qEAAqE;QACrE,MAAM,CAAC,EAAE,CACR,YAAY,CAAC,IAAI,CAAC,SAAS,EAAE,GAAG,CAAC,EACjC,0DAA0D,YAAY,CAAC,IAAI,CAAC,SAAS,EAAE,EAAE,CACzF,CAAC;QAEF,wEAAwE;QACxE,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,SAAS,EAAE,qDAAqD,CAAC,CAAC;QACzF,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;YACvC,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,QAAQ,EAAE,SAAS,CAAC,GAAG,CAAC,yCAAyC,CAAC,CAAC;QAC3F,CAAC;QAED,oCAAoC;QACpC,MAAM,CAAC,EAAE,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC,EAAE,6BAA6B,CAAC,CAAC;IAC9D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gEAAgE,EAAE,KAAK,IAAI,EAAE;QAC/E,MAAM,QAAQ,GAAmB;YAChC,eAAe,CAAC,MAAM,CAAC;YACvB,eAAe,CAAC,MAAM,CAAC;SACvB,CAAC;QACF,MAAM,KAAK,GAAG,SAAS,CAAC,OAAO,CAAC,CAAC;QACjC,MAAM,aAAa,GAAG,MAAM,CAAC;QAE7B,oBAAoB;QACpB,IAAI,WAAW,GAAG,CAAC,CAAC;QACpB,KAAK,MAAM,CAAC,IAAI,QAAQ;YAAE,WAAW,IAAI,cAAc,CAAC,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,EAAE,CACR,WAAW,GAAG,KAAK,CAAC,aAAa,EACjC,eAAe,WAAW,yBAAyB,KAAK,CAAC,aAAa,iBAAiB,CACvF,CAAC;QAEF,MAAM,YAAY,GAAG,IAAI,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CAAC,gBAAgB,CAAC,qBAAqB,CAAC,CAAC,CAAC;QAElF,MAAM,eAAe,CAAC,QAAQ,EAAE,KAAK,EAAE,aAAa,EAAE,SAAS,EAAE,SAAS,EAAE,SAAS,EAAE,SAAS,EAAE,YAAY,CAAC,CAAC;QAEhH,MAAM,CAAC,KAAK,CACX,YAAY,CAAC,IAAI,CAAC,SAAS,EAAE,EAC7B,CAAC,EACD,0EAA0E,CAC1E,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;QACrE,MAAM,QAAQ,GAAmB;YAChC,eAAe,CAAC,MAAM,CAAC;YACvB,eAAe,CAAC,MAAM,CAAC;YACvB,eAAe,CAAC,MAAM,CAAC;SACvB,CAAC;QACF,MAAM,KAAK,GAAG,SAAS,CAAC,OAAO,CAAC,CAAC;QACjC,MAAM,aAAa,GAAG,MAAM,CAAC;QAC7B,MAAM,eAAe,GAAG,kCAAkC,CAAC;QAE3D,MAAM,OAAO,GAAa,EAAE,CAAC;QAC7B,MAAM,YAAY,GAAG,IAAI,CAAC,EAAE,CAAC,KAAK,EAAE,MAAW,EAAE,OAAY,EAAE,EAAE;YAChE,MAAM,OAAO,GAAG,OAAO,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC,CAAC;YACtC,MAAM,IAAI,GACT,OAAO,OAAO,EAAE,OAAO,KAAK,QAAQ;gBACnC,CAAC,CAAC,OAAO,CAAC,OAAO;gBACjB,CAAC,CAAC,OAAO,EAAE,OAAO,EAAE,CAAC,CAAC,CAAC,EAAE,IAAI,IAAI,EAAE,CAAC;YACtC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACnB,OAAO,gBAAgB,CAAC,eAAe,CAAC,CAAC;QAC1C,CAAC,CAAC,CAAC;QAEH,MAAM,eAAe,CACpB,QAAQ,EACR,KAAK,EACL,aAAa,EACb,SAAS,EACT,SAAS,EACT,SAAS,EACT,eAAe,EACf,YAAY,CACZ,CAAC;QAEF,iDAAiD;QACjD,MAAM,CAAC,EAAE,CACR,OAAO,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,eAAe,CAAC,EACpC,oDAAoD,CACpD,CAAC;IACH,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC","sourcesContent":["/**\n * Tests for chunked compaction fallback when messages exceed model context window.\n * Regression test for #2932.\n */\n\nimport assert from \"node:assert/strict\";\nimport { describe, it, mock } from \"node:test\";\n\nimport type { AgentMessage } from \"@gsd/pi-agent-core\";\nimport type { Model, AssistantMessage } from \"@gsd/pi-ai\";\n\nimport { generateSummary, estimateTokens, chunkMessages } from \"./compaction.js\";\n\n// ---------------------------------------------------------------------------\n// Helpers\n// ---------------------------------------------------------------------------\n\n/** Create a user message with approximately `tokenCount` tokens (chars = tokens * 4). */\nfunction makeUserMessage(tokenCount: number): AgentMessage {\n\tconst text = \"x\".repeat(tokenCount * 4);\n\treturn { role: \"user\", content: text } as unknown as AgentMessage;\n}\n\n/** Create a mock model with a given context window. */\nfunction makeModel(contextWindow: number): Model<any> {\n\treturn {\n\t\tid: \"test-model\",\n\t\tname: \"Test Model\",\n\t\tapi: \"anthropic-messages\",\n\t\tprovider: \"anthropic\",\n\t\tbaseUrl: \"https://api.test\",\n\t\treasoning: false,\n\t\tinput: [\"text\"],\n\t\tcost: { input: 0, output: 0, cacheRead: 0, cacheWrite: 0 },\n\t\tcontextWindow,\n\t\tmaxTokens: 4096,\n\t} as Model<any>;\n}\n\nfunction makeFakeResponse(text: string): AssistantMessage {\n\treturn {\n\t\tcontent: [{ type: \"text\", text }],\n\t\tstopReason: \"end_turn\",\n\t} as unknown as AssistantMessage;\n}\n\n// ---------------------------------------------------------------------------\n// chunkMessages tests\n// ---------------------------------------------------------------------------\n\ndescribe(\"chunkMessages\", () => {\n\tit(\"returns a single chunk when messages fit in budget\", () => {\n\t\tconst messages: AgentMessage[] = [\n\t\t\tmakeUserMessage(1_000),\n\t\t\tmakeUserMessage(1_000),\n\t\t];\n\t\tconst chunks = chunkMessages(messages, 100_000);\n\t\tassert.equal(chunks.length, 1);\n\t\tassert.equal(chunks[0].length, 2);\n\t});\n\n\tit(\"splits messages into multiple chunks when they exceed budget\", () => {\n\t\tconst messages: AgentMessage[] = [\n\t\t\tmakeUserMessage(50_000),\n\t\t\tmakeUserMessage(50_000),\n\t\t\tmakeUserMessage(50_000),\n\t\t];\n\t\t// Budget of 80k tokens means each 50k message gets its own chunk\n\t\t// (or two fit together if budget allows)\n\t\tconst chunks = chunkMessages(messages, 80_000);\n\t\tassert.ok(chunks.length > 1, `Expected multiple chunks, got ${chunks.length}`);\n\t\t// All messages should be present across chunks\n\t\tconst totalMessages = chunks.reduce((sum, c) => sum + c.length, 0);\n\t\tassert.equal(totalMessages, 3);\n\t});\n\n\tit(\"puts a single oversized message in its own chunk\", () => {\n\t\tconst messages: AgentMessage[] = [\n\t\t\tmakeUserMessage(200_000), // Way over any reasonable budget\n\t\t];\n\t\tconst chunks = chunkMessages(messages, 80_000);\n\t\tassert.equal(chunks.length, 1);\n\t\tassert.equal(chunks[0].length, 1);\n\t});\n\n\tit(\"preserves message order across chunks\", () => {\n\t\t// Create messages with identifiable sizes\n\t\tconst messages: AgentMessage[] = [\n\t\t\tmakeUserMessage(30_000), // ~30k tokens\n\t\t\tmakeUserMessage(30_000),\n\t\t\tmakeUserMessage(30_000),\n\t\t\tmakeUserMessage(30_000),\n\t\t];\n\t\tconst chunks = chunkMessages(messages, 50_000);\n\t\t// Reconstruct original order\n\t\tconst flat = chunks.flat();\n\t\tassert.equal(flat.length, 4);\n\t\tfor (let i = 0; i < flat.length; i++) {\n\t\t\tassert.strictEqual(flat[i], messages[i], `Message ${i} should be in order`);\n\t\t}\n\t});\n});\n\n// ---------------------------------------------------------------------------\n// generateSummary chunked fallback tests\n// ---------------------------------------------------------------------------\n\ndescribe(\"generateSummary — chunked fallback (#2932)\", () => {\n\tit(\"calls _completeFn multiple times when messages exceed model context window\", async () => {\n\t\t// Arrange: 3 messages of ~80k tokens each = ~240k total, model has 200k window\n\t\tconst messages: AgentMessage[] = [\n\t\t\tmakeUserMessage(80_000),\n\t\t\tmakeUserMessage(80_000),\n\t\t\tmakeUserMessage(80_000),\n\t\t];\n\t\tconst model = makeModel(200_000);\n\t\tconst reserveTokens = 16_384;\n\n\t\t// Verify our test setup: messages really do exceed the model window\n\t\tlet totalTokens = 0;\n\t\tfor (const m of messages) totalTokens += estimateTokens(m);\n\t\tassert.ok(\n\t\t\ttotalTokens > model.contextWindow,\n\t\t\t`Test setup: ${totalTokens} tokens should exceed ${model.contextWindow} context window`,\n\t\t);\n\n\t\t// Track calls\n\t\tconst calls: string[] = [];\n\t\tconst mockComplete = mock.fn(async (_model: any, context: any, _options: any) => {\n\t\t\tconst userMsg = context.messages?.[0];\n\t\t\tconst text =\n\t\t\t\ttypeof userMsg?.content === \"string\"\n\t\t\t\t\t? userMsg.content\n\t\t\t\t\t: userMsg?.content?.[0]?.text ?? \"\";\n\n\t\t\tif (text.includes(\"<previous-summary>\")) {\n\t\t\t\tcalls.push(\"update\");\n\t\t\t} else {\n\t\t\t\tcalls.push(\"initial\");\n\t\t\t}\n\t\t\treturn makeFakeResponse(\"Summary of chunk\");\n\t\t});\n\n\t\tconst summary = await generateSummary(\n\t\t\tmessages,\n\t\t\tmodel,\n\t\t\treserveTokens,\n\t\t\tundefined, // apiKey\n\t\t\tundefined, // signal\n\t\t\tundefined, // customInstructions\n\t\t\tundefined, // previousSummary\n\t\t\tmockComplete, // _completeFn override for testing\n\t\t);\n\n\t\t// Assert: should have called completeSimple more than once (chunked)\n\t\tassert.ok(\n\t\t\tmockComplete.mock.callCount() > 1,\n\t\t\t`Expected multiple calls for chunked summarization, got ${mockComplete.mock.callCount()}`,\n\t\t);\n\n\t\t// First call should be an initial summary, subsequent should be updates\n\t\tassert.equal(calls[0], \"initial\", \"First chunk should use initial summarization prompt\");\n\t\tfor (let i = 1; i < calls.length; i++) {\n\t\t\tassert.equal(calls[i], \"update\", `Chunk ${i + 1} should use update summarization prompt`);\n\t\t}\n\n\t\t// Should return a non-empty summary\n\t\tassert.ok(summary.length > 0, \"Summary should not be empty\");\n\t});\n\n\tit(\"uses single-pass when messages fit within model context window\", async () => {\n\t\tconst messages: AgentMessage[] = [\n\t\t\tmakeUserMessage(10_000),\n\t\t\tmakeUserMessage(10_000),\n\t\t];\n\t\tconst model = makeModel(200_000);\n\t\tconst reserveTokens = 16_384;\n\n\t\t// Verify test setup\n\t\tlet totalTokens = 0;\n\t\tfor (const m of messages) totalTokens += estimateTokens(m);\n\t\tassert.ok(\n\t\t\ttotalTokens < model.contextWindow,\n\t\t\t`Test setup: ${totalTokens} tokens should fit in ${model.contextWindow} context window`,\n\t\t);\n\n\t\tconst mockComplete = mock.fn(async () => makeFakeResponse(\"Single pass summary\"));\n\n\t\tawait generateSummary(messages, model, reserveTokens, undefined, undefined, undefined, undefined, mockComplete);\n\n\t\tassert.equal(\n\t\t\tmockComplete.mock.callCount(),\n\t\t\t1,\n\t\t\t\"Should use single-pass summarization when messages fit in context window\",\n\t\t);\n\t});\n\n\tit(\"passes previousSummary through chunked summarization\", async () => {\n\t\tconst messages: AgentMessage[] = [\n\t\t\tmakeUserMessage(80_000),\n\t\t\tmakeUserMessage(80_000),\n\t\t\tmakeUserMessage(80_000),\n\t\t];\n\t\tconst model = makeModel(200_000);\n\t\tconst reserveTokens = 16_384;\n\t\tconst previousSummary = \"Previous session summary content\";\n\n\t\tconst prompts: string[] = [];\n\t\tconst mockComplete = mock.fn(async (_model: any, context: any) => {\n\t\t\tconst userMsg = context.messages?.[0];\n\t\t\tconst text =\n\t\t\t\ttypeof userMsg?.content === \"string\"\n\t\t\t\t\t? userMsg.content\n\t\t\t\t\t: userMsg?.content?.[0]?.text ?? \"\";\n\t\t\tprompts.push(text);\n\t\t\treturn makeFakeResponse(\"Chunk summary\");\n\t\t});\n\n\t\tawait generateSummary(\n\t\t\tmessages,\n\t\t\tmodel,\n\t\t\treserveTokens,\n\t\t\tundefined,\n\t\t\tundefined,\n\t\t\tundefined,\n\t\t\tpreviousSummary,\n\t\t\tmockComplete,\n\t\t);\n\n\t\t// First chunk should include the previousSummary\n\t\tassert.ok(\n\t\t\tprompts[0].includes(previousSummary),\n\t\t\t\"First chunk should incorporate the previousSummary\",\n\t\t);\n\t});\n});\n"]}