gsd-pi 2.78.0 → 2.78.1-dev.0fdacd524

package/packages/pi-tui/dist/__tests__/autocomplete.test.js

@@ -23,9 +23,12 @@ describe("CombinedAutocompleteProvider — slash commands", () => {
         const provider = makeProvider(sampleCommands);
         const result = provider.getSuggestions(["/se"], 0, 3);
         assert.ok(result);
-        assert.equal(result.items.length, 2); // settings, session
+        // /se matches settings and session — name the values, then assert exact
+        // count as the contract for this prefix. Asserting count alone would
+        // pass even if the matches were the wrong commands.
         assert.ok(result.items.some((i) => i.value === "settings"));
         assert.ok(result.items.some((i) => i.value === "session"));
+        assert.equal(result.items.length, 2);
     });
     it("returns null when no commands match", () => {
         const provider = makeProvider(sampleCommands);
@@ -36,7 +39,11 @@ describe("CombinedAutocompleteProvider — slash commands", () => {
         const provider = makeProvider(sampleCommands);
         const result = provider.getSuggestions(["/mod"], 0, 4);
         assert.ok(result);
-        assert.equal(result.items[0]?.description, "Select model");
+        // Verify the matched command is present and every item carries a
+        // description — avoids `[0]` positional coupling that would silently
+        // pass if list ordering changed.
+        assert.ok(result.items.some((i) => i.value === "model"));
+        assert.ok(result.items.every((i) => typeof i.description === "string" && i.description.length > 0), "every suggestion must have a non-empty description");
     });
     it("does not offer slash command suggestions mid-line", () => {
         const sentinelCommands = [
@@ -77,8 +84,10 @@ describe("CombinedAutocompleteProvider — argument completions", () => {
         const provider = makeProvider(commands);
         const result = provider.getSuggestions(["/thinking m"], 0, 11);
         assert.ok(result);
+        // /thinking m matches only "medium" — verify the expected value is
+        // present (by name, not index) and then assert exact count.
+        assert.ok(result.items.some((i) => i.value === "medium"));
         assert.equal(result.items.length, 1);
-        assert.equal(result.items[0]?.value, "medium");
     });
     it("returns null for commands without argument completions", () => {
         const provider = makeProvider(sampleCommands);
@@ -102,6 +111,11 @@ describe("CombinedAutocompleteProvider — argument completions", () => {
         const provider = makeProvider(commands);
         const result = provider.getSuggestions(["/test "], 0, 6);
         assert.ok(result);
+        // Empty prefix returns all 3 subcommands — verify each is present
+        // by name. Bare count would pass for any 3-element list.
+        assert.ok(result.items.some((i) => i.value === "start"));
+        assert.ok(result.items.some((i) => i.value === "stop"));
+        assert.ok(result.items.some((i) => i.value === "status"));
         assert.equal(result.items.length, 3);
     });
 });

package/packages/pi-tui/dist/__tests__/autocomplete.test.js.map

@@ -1 +1 @@
-{"version":3,"file":"autocomplete.test.js","sourceRoot":"","sources":["../../src/__tests__/autocomplete.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,WAAW,CAAC;AACzC,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,EAAE,4BAA4B,EAAE,MAAM,oBAAoB,CAAC;AAGlE,SAAS,YAAY,CAAC,WAA2B,EAAE,EAAE,WAAmB,MAAM;IAC7E,OAAO,IAAI,4BAA4B,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAC;AAC7D,CAAC;AAED,MAAM,cAAc,GAAmB;IACtC,EAAE,IAAI,EAAE,UAAU,EAAE,WAAW,EAAE,oBAAoB,EAAE;IACvD,EAAE,IAAI,EAAE,OAAO,EAAE,WAAW,EAAE,cAAc,EAAE;IAC9C,EAAE,IAAI,EAAE,SAAS,EAAE,WAAW,EAAE,mBAAmB,EAAE;IACrD,EAAE,IAAI,EAAE,QAAQ,EAAE,WAAW,EAAE,gBAAgB,EAAE;IACjD,EAAE,IAAI,EAAE,UAAU,EAAE,WAAW,EAAE,oBAAoB,EAAE;CACvD,CAAC;AAEF,QAAQ,CAAC,+CAA+C,EAAE,GAAG,EAAE;IAC9D,EAAE,CAAC,iCAAiC,EAAE,GAAG,EAAE;QAC1C,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACpD,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,2BAA2B,CAAC,CAAC;QAC/C,MAAM,CAAC,KAAK,CAAC,MAAO,CAAC,KAAK,CAAC,MAAM,EAAE,cAAc,CAAC,MAAM,CAAC,CAAC;QAC1D,MAAM,CAAC,KAAK,CAAC,MAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kCAAkC,EAAE,GAAG,EAAE;QAC3C,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,KAAK,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACtD,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,CAAC;QAClB,MAAM,CAAC,KAAK,CAAC,MAAO,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC,CAAC,oBAAoB;QAC3D,MAAM,CAAC,EAAE,CAAC,MAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,UAAU,CAAC,CAAC,CAAC;QAC7D,MAAM,CAAC,EAAE,CAAC,MAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,SAAS,CAAC,CAAC,CAAC;IAC7D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC9C,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACvD,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;IAC5B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC9C,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACvD,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,CAAC;QAClB,MAAM,CAAC,KAAK,CAAC,MAAO,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,cAAc,CAAC,CAAC;IAC7D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC5D,MAAM,gBAAgB,GAAmB;YACxC,EAAE,IAAI,EAAE,qBAAqB,EAAE,WAAW,EAAE,wBAAwB,EAAE;SACtE,CAAC;QACF,MAAM,QAAQ,GAAG,YAAY,CAAC,gBAAgB,CAAC,CAAC;QAChD,MAAM,IAAI,GAAG,iBAAiB,CAAC;QAC/B,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,IAAI,CAAC,MAAM,CAAC,CAAC;QAE/D,IAAI,MAAM,KAAK,IAAI,EAAE,CAAC;YACrB,OAAO;QACR,CAAC;QAED,MAAM,CAAC,EAAE,CACR,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,KAAK,KAAK,qBAAqB,CAAC,EAClE,sEAAsE,CACtE,CAAC;QACF,MAAM,CAAC,EAAE,CACR,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,WAAW,KAAK,wBAAwB,CAAC,EAC3E,mEAAmE,CACnE,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC3D,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACxD,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,CAAC;QAClB,MAAM,CAAC,KAAK,CAAC,MAAO,CAAC,MAAM,EAAE,KAAK,CAAC,CAAC;QACpC,MAAM,CAAC,EAAE,CAAC,MAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,KAAK,KAAK,UAAU,CAAC,CAAC,CAAC;IACpE,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,qDAAqD,EAAE,GAAG,EAAE;IACpE,EAAE,CAAC,6DAA6D,EAAE,GAAG,EAAE;QACtE,MAAM,QAAQ,GAAmB;YAChC;gBACC,IAAI,EAAE,UAAU;gBAChB,WAAW,EAAE,oBAAoB;gBACjC,sBAAsB,EAAE,CAAC,MAAM,EAAE,EAAE;oBAClC,MAAM,MAAM,GAAG,CAAC,KAAK,EAAE,KAAK,EAAE,QAAQ,EAAE,MAAM,CAAC,CAAC;oBAChD,MAAM,QAAQ,GAAG,MAAM;yBACrB,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,MAAM,CAAC,IAAI,EAAE,CAAC,CAAC;yBAC1C,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;oBACvC,OAAO,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC;gBAC9C,CAAC;aACD;SACD,CAAC;QACF,MAAM,QAAQ,GAAG,YAAY,CAAC,QAAQ,CAAC,CAAC;QACxC,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,aAAa,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;QAC/D,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,CAAC;QAClB,MAAM,CAAC,KAAK,CAAC,MAAO,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QACtC,MAAM,CAAC,KAAK,CAAC,MAAO,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,KAAK,EAAE,QAAQ,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wDAAwD,EAAE,GAAG,EAAE;QACjE,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,eAAe,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;QACjE,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;IAC5B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0DAA0D,EAAE,GAAG,EAAE;QACnE,MAAM,QAAQ,GAAmB;YAChC;gBACC,IAAI,EAAE,MAAM;gBACZ,WAAW,EAAE,cAAc;gBAC3B,sBAAsB,EAAE,CAAC,MAAM,EAAE,EAAE;oBAClC,MAAM,IAAI,GAAG,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,CAAC,CAAC;oBACzC,MAAM,QAAQ,GAAG,IAAI;yBACnB,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,MAAM,CAAC,IAAI,EAAE,CAAC,CAAC;yBAC1C,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;oBACvC,OAAO,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC;gBAC9C,CAAC;aACD;SACD,CAAC;QACF,MAAM,QAAQ,GAAG,YAAY,CAAC,QAAQ,CAAC,CAAC;QACxC,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,QAAQ,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACzD,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,CAAC;QAClB,MAAM,CAAC,KAAK,CAAC,MAAO,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,yDAAyD,EAAE,GAAG,EAAE;IACxE,EAAE,CAAC,iEAAiE,EAAE,GAAG,EAAE;QAC1E,MAAM,QAAQ,GAAG,YAAY,EAAE,CAAC;QAChC,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,kBAAkB,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;QACpE,gFAAgF;QAChF,yEAAyE;QACzE,qEAAqE;QACrE,IAAI,MAAM,KAAK,IAAI,EAAE,CAAC;YACrB,MAAM,CAAC,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,+BAA+B,CAAC,CAAC;YACxE,kEAAkE;YAClE,8CAA8C;YAC9C,MAAM,CAAC,KAAK,CAAC,OAAO,MAAM,CAAC,MAAM,EAAE,QAAQ,EAAE,yBAAyB,CAAC,CAAC;YACxE,MAAM,CAAC,EAAE,CACR,CAAC,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,GAAG,CAAC,EAC9B,iDAAiD,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,MAAM,CAAC,EAAE,CAChF,CAAC;QACH,CAAC;IACF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;QACrE,MAAM,QAAQ,GAAG,YAAY,EAAE,CAAC;QAChC,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,wBAAwB,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;QAC1E,IAAI,MAAM,KAAK,IAAI,EAAE,CAAC;YACrB,MAAM,CAAC,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,+BAA+B,CAAC,CAAC;YACxE,MAAM,CAAC,KAAK,CAAC,OAAO,MAAM,CAAC,MAAM,EAAE,QAAQ,EAAE,yBAAyB,CAAC,CAAC;YACxE,uEAAuE;YACvE,MAAM,CAAC,EAAE,CACR,CAAC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,EAChC,mDAAmD,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,MAAM,CAAC,EAAE,CAClF,CAAC;QACH,CAAC;IACF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uEAAuE,EAAE,GAAG,EAAE;QAChF,MAAM,QAAQ,GAAG,YAAY,CAAC,EAAE,EAAE,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;QACjD,kEAAkE;QAClE,oEAAoE;QACpE,wEAAwE;QACxE,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACpD,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,IAAI,EAAE,6CAA6C,CAAC,CAAC;IAC3E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC/D,MAAM,QAAQ,GAAG,YAAY,CAAC,EAAE,EAAE,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;QACjD,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,WAAW,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QAC5D,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,IAAI,EAAE,kEAAkE,CAAC,CAAC;IAChG,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,gDAAgD,EAAE,GAAG,EAAE;IAC/D,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC/D,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,eAAe,CAAC,CAAC,KAAK,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,EAAE,KAAK,EAAE,UAAU,EAAE,KAAK,EAAE,UAAU,EAAE,EAAE,KAAK,CAAC,CAAC;QACxG,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,YAAY,CAAC,CAAC;QAC5C,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,SAAS,EAAE,EAAE,CAAC,CAAC,CAAC,qBAAqB;IAC1D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qEAAqE,EAAE,GAAG,EAAE;QAC9E,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,eAAe,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,EAAE,KAAK,EAAE,UAAU,EAAE,KAAK,EAAE,UAAU,EAAE,EAAE,KAAK,CAAC,CAAC;QAC1G,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,cAAc,CAAC,CAAC;QAC9C,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,SAAS,EAAE,EAAE,CAAC,CAAC;IACpC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACpD,MAAM,QAAQ,GAAG,YAAY,EAAE,CAAC;QAChC,MAAM,MAAM,GAAG,QAAQ,CAAC,eAAe,CACtC,CAAC,OAAO,CAAC,EACT,CAAC,EACD,CAAC,EACD,EAAE,KAAK,EAAE,eAAe,EAAE,KAAK,EAAE,UAAU,EAAE,EAC7C,OAAO,CACP,CAAC;QACF,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,gBAAgB,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,GAAG,EAAE;QAC9D,MAAM,QAAQ,GAAG,YAAY,EAAE,CAAC;QAChC,MAAM,MAAM,GAAG,QAAQ,CAAC,eAAe,CACtC,CAAC,KAAK,CAAC,EACP,CAAC,EACD,CAAC,EACD,EAAE,KAAK,EAAE,OAAO,EAAE,KAAK,EAAE,MAAM,EAAE,EACjC,KAAK,CACL,CAAC;QACF,wEAAwE;QACxE,MAAM,CAAC,EAAE,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAE,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6BAA6B,EAAE,GAAG,EAAE;QACtC,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,eAAe,CACtC,CAAC,mBAAmB,CAAC,EACrB,CAAC,EACD,CAAC,EACD,EAAE,KAAK,EAAE,UAAU,EAAE,KAAK,EAAE,UAAU,EAAE,EACxC,KAAK,CACL,CAAC;QACF,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAE,CAAC,QAAQ,CAAC,eAAe,CAAC,CAAC,CAAC;IACvD,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,uDAAuD,EAAE,GAAG,EAAE;IACtE,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC9C,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,uBAAuB,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QAChE,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;IAC5B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;QACvE,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,2BAA2B,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,EAAE,KAAK,CAAC,CAAC;IAC3E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;QACpE,MAAM,QAAQ,GAAG,YAAY,EAAE,CAAC;QAChC,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,2BAA2B,CAAC,CAAC,WAAW,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC;IAC/E,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC","sourcesContent":["import { describe, it } from \"node:test\";\nimport assert from \"node:assert/strict\";\nimport { CombinedAutocompleteProvider } from \"../autocomplete.js\";\nimport type { SlashCommand } from \"../autocomplete.js\";\n\nfunction makeProvider(commands: SlashCommand[] = [], basePath: string = \"/tmp\") {\n\treturn new CombinedAutocompleteProvider(commands, basePath);\n}\n\nconst sampleCommands: SlashCommand[] = [\n\t{ name: \"settings\", description: \"Open settings menu\" },\n\t{ name: \"model\", description: \"Select model\" },\n\t{ name: \"session\", description: \"Show session info\" },\n\t{ name: \"export\", description: \"Export session\" },\n\t{ name: \"thinking\", description: \"Set thinking level\" },\n];\n\ndescribe(\"CombinedAutocompleteProvider — slash commands\", () => {\n\tit(\"returns all commands for bare /\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.getSuggestions([\"/\"], 0, 1);\n\t\tassert.ok(result, \"should return suggestions\");\n\t\tassert.equal(result!.items.length, sampleCommands.length);\n\t\tassert.equal(result!.prefix, \"/\");\n\t});\n\n\tit(\"filters commands by typed prefix\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.getSuggestions([\"/se\"], 0, 3);\n\t\tassert.ok(result);\n\t\tassert.equal(result!.items.length, 2); // settings, session\n\t\tassert.ok(result!.items.some((i) => i.value === \"settings\"));\n\t\tassert.ok(result!.items.some((i) => i.value === \"session\"));\n\t});\n\n\tit(\"returns null when no commands match\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.getSuggestions([\"/zzz\"], 0, 4);\n\t\tassert.equal(result, null);\n\t});\n\n\tit(\"includes description in suggestions\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.getSuggestions([\"/mod\"], 0, 4);\n\t\tassert.ok(result);\n\t\tassert.equal(result!.items[0]?.description, \"Select model\");\n\t});\n\n\tit(\"does not offer slash command suggestions mid-line\", () => {\n\t\tconst sentinelCommands: SlashCommand[] = [\n\t\t\t{ name: \"codexmidlinecommand\", description: \"Sentinel slash command\" },\n\t\t];\n\t\tconst provider = makeProvider(sentinelCommands);\n\t\tconst line = \"hello /codexmid\";\n\t\tconst result = provider.getSuggestions([line], 0, line.length);\n\n\t\tif (result === null) {\n\t\t\treturn;\n\t\t}\n\n\t\tassert.ok(\n\t\t\tresult.items.every((item) => item.value !== \"codexmidlinecommand\"),\n\t\t\t\"mid-line slash-like text should not return slash command completions\",\n\t\t);\n\t\tassert.ok(\n\t\t\tresult.items.every((item) => item.description !== \"Sentinel slash command\"),\n\t\t\t\"mid-line slash-like text should not return slash command metadata\",\n\t\t);\n\t});\n\n\tit(\"triggers slash commands after leading whitespace\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.getSuggestions([\"  /se\"], 0, 5);\n\t\tassert.ok(result);\n\t\tassert.equal(result!.prefix, \"/se\");\n\t\tassert.ok(result!.items.some((item) => item.value === \"settings\"));\n\t});\n});\n\ndescribe(\"CombinedAutocompleteProvider — argument completions\", () => {\n\tit(\"returns argument completions for commands that support them\", () => {\n\t\tconst commands: SlashCommand[] = [\n\t\t\t{\n\t\t\t\tname: \"thinking\",\n\t\t\t\tdescription: \"Set thinking level\",\n\t\t\t\tgetArgumentCompletions: (prefix) => {\n\t\t\t\t\tconst levels = [\"off\", \"low\", \"medium\", \"high\"];\n\t\t\t\t\tconst filtered = levels\n\t\t\t\t\t\t.filter((l) => l.startsWith(prefix.trim()))\n\t\t\t\t\t\t.map((l) => ({ value: l, label: l }));\n\t\t\t\t\treturn filtered.length > 0 ? filtered : null;\n\t\t\t\t},\n\t\t\t},\n\t\t];\n\t\tconst provider = makeProvider(commands);\n\t\tconst result = provider.getSuggestions([\"/thinking m\"], 0, 11);\n\t\tassert.ok(result);\n\t\tassert.equal(result!.items.length, 1);\n\t\tassert.equal(result!.items[0]?.value, \"medium\");\n\t});\n\n\tit(\"returns null for commands without argument completions\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.getSuggestions([\"/settings foo\"], 0, 13);\n\t\tassert.equal(result, null);\n\t});\n\n\tit(\"returns all arg completions for empty prefix after space\", () => {\n\t\tconst commands: SlashCommand[] = [\n\t\t\t{\n\t\t\t\tname: \"test\",\n\t\t\t\tdescription: \"Test command\",\n\t\t\t\tgetArgumentCompletions: (prefix) => {\n\t\t\t\t\tconst subs = [\"start\", \"stop\", \"status\"];\n\t\t\t\t\tconst filtered = subs\n\t\t\t\t\t\t.filter((s) => s.startsWith(prefix.trim()))\n\t\t\t\t\t\t.map((s) => ({ value: s, label: s }));\n\t\t\t\t\treturn filtered.length > 0 ? filtered : null;\n\t\t\t\t},\n\t\t\t},\n\t\t];\n\t\tconst provider = makeProvider(commands);\n\t\tconst result = provider.getSuggestions([\"/test \"], 0, 6);\n\t\tassert.ok(result);\n\t\tassert.equal(result!.items.length, 3);\n\t});\n});\n\ndescribe(\"CombinedAutocompleteProvider — @ file prefix extraction\", () => {\n\tit(\"detects @ at start of line and returns a valid suggestion shape\", () => {\n\t\tconst provider = makeProvider();\n\t\tconst result = provider.getSuggestions([\"@nonexistent_xyz\"], 0, 16);\n\t\t// Either null (nothing matched) or a well-formed {items: Array, prefix: string}\n\t\t// shape. Previous version's `result.items.length >= 0` was a tautology —\n\t\t// array length is always ≥ 0; the whole expression could never fail.\n\t\tif (result !== null) {\n\t\t\tassert.ok(Array.isArray(result.items), \"result.items must be an array\");\n\t\t\t// The @-prefix extraction strips the leading @ — prefix should be\n\t\t\t// the raw text without the trigger character.\n\t\t\tassert.equal(typeof result.prefix, \"string\", \"prefix must be a string\");\n\t\t\tassert.ok(\n\t\t\t\t!result.prefix.startsWith(\"@\"),\n\t\t\t\t`prefix must have the @ trigger stripped, got: ${JSON.stringify(result.prefix)}`,\n\t\t\t);\n\t\t}\n\t});\n\n\tit(\"detects @ after space and returns a valid suggestion shape\", () => {\n\t\tconst provider = makeProvider();\n\t\tconst result = provider.getSuggestions([\"check @nonexistent_xyz\"], 0, 22);\n\t\tif (result !== null) {\n\t\t\tassert.ok(Array.isArray(result.items), \"result.items must be an array\");\n\t\t\tassert.equal(typeof result.prefix, \"string\", \"prefix must be a string\");\n\t\t\t// The prefix must NOT include the word \"check\" that came before the @.\n\t\t\tassert.ok(\n\t\t\t\t!result.prefix.includes(\"check\"),\n\t\t\t\t`prefix must not include text before the @, got: ${JSON.stringify(result.prefix)}`,\n\t\t\t);\n\t\t}\n\t});\n\n\tit(\"returns null for bare @ with no query to avoid full tree walk (#1824)\", () => {\n\t\tconst provider = makeProvider([], process.cwd());\n\t\t// A bare \"@\" produces an empty rawPrefix after stripping the \"@\".\n\t\t// This must return null to avoid a synchronous full filesystem walk\n\t\t// via the native fuzzyFind addon, which freezes the TUI on large repos.\n\t\tconst result = provider.getSuggestions([\"@\"], 0, 1);\n\t\tassert.equal(result, null, \"bare @ should not trigger fuzzy file search\");\n\t});\n\n\tit(\"returns null for @ after space with no query (#1824)\", () => {\n\t\tconst provider = makeProvider([], process.cwd());\n\t\tconst result = provider.getSuggestions([\"look at @\"], 0, 9);\n\t\tassert.equal(result, null, \"@ after space with no query should not trigger fuzzy file search\");\n\t});\n});\n\ndescribe(\"CombinedAutocompleteProvider — applyCompletion\", () => {\n\tit(\"applies slash command completion with trailing space\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.applyCompletion([\"/se\"], 0, 3, { value: \"settings\", label: \"settings\" }, \"/se\");\n\t\tassert.equal(result.lines[0], \"/settings \");\n\t\tassert.equal(result.cursorCol, 10); // after \"/settings \"\n\t});\n\n\tit(\"preserves leading whitespace when applying slash command completion\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.applyCompletion([\"  /se\"], 0, 5, { value: \"settings\", label: \"settings\" }, \"/se\");\n\t\tassert.equal(result.lines[0], \"  /settings \");\n\t\tassert.equal(result.cursorCol, 12);\n\t});\n\n\tit(\"applies file path completion for @ prefix\", () => {\n\t\tconst provider = makeProvider();\n\t\tconst result = provider.applyCompletion(\n\t\t\t[\"@src/\"],\n\t\t\t0,\n\t\t\t5,\n\t\t\t{ value: \"@src/index.ts\", label: \"index.ts\" },\n\t\t\t\"@src/\",\n\t\t);\n\t\tassert.equal(result.lines[0], \"@src/index.ts \");\n\t});\n\n\tit(\"applies directory completion without trailing space\", () => {\n\t\tconst provider = makeProvider();\n\t\tconst result = provider.applyCompletion(\n\t\t\t[\"@sr\"],\n\t\t\t0,\n\t\t\t3,\n\t\t\t{ value: \"@src/\", label: \"src/\" },\n\t\t\t\"@sr\",\n\t\t);\n\t\t// Directories should not get trailing space so user can continue typing\n\t\tassert.ok(!result.lines[0]!.endsWith(\" \"));\n\t});\n\n\tit(\"preserves text after cursor\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.applyCompletion(\n\t\t\t[\"/se and more text\"],\n\t\t\t0,\n\t\t\t3,\n\t\t\t{ value: \"settings\", label: \"settings\" },\n\t\t\t\"/se\",\n\t\t);\n\t\tassert.ok(result.lines[0]!.includes(\"and more text\"));\n\t});\n});\n\ndescribe(\"CombinedAutocompleteProvider — force file suggestions\", () => {\n\tit(\"does not trigger for slash commands\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.getForceFileSuggestions([\"/set\"], 0, 4);\n\t\tassert.equal(result, null);\n\t});\n\n\tit(\"shouldTriggerFileCompletion returns false for slash commands\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tassert.equal(provider.shouldTriggerFileCompletion([\"/set\"], 0, 4), false);\n\t});\n\n\tit(\"shouldTriggerFileCompletion returns true for regular text\", () => {\n\t\tconst provider = makeProvider();\n\t\tassert.equal(provider.shouldTriggerFileCompletion([\"some text\"], 0, 9), true);\n\t});\n});\n"]}
+{"version":3,"file":"autocomplete.test.js","sourceRoot":"","sources":["../../src/__tests__/autocomplete.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,WAAW,CAAC;AACzC,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,EAAE,4BAA4B,EAAE,MAAM,oBAAoB,CAAC;AAGlE,SAAS,YAAY,CAAC,WAA2B,EAAE,EAAE,WAAmB,MAAM;IAC7E,OAAO,IAAI,4BAA4B,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAC;AAC7D,CAAC;AAED,MAAM,cAAc,GAAmB;IACtC,EAAE,IAAI,EAAE,UAAU,EAAE,WAAW,EAAE,oBAAoB,EAAE;IACvD,EAAE,IAAI,EAAE,OAAO,EAAE,WAAW,EAAE,cAAc,EAAE;IAC9C,EAAE,IAAI,EAAE,SAAS,EAAE,WAAW,EAAE,mBAAmB,EAAE;IACrD,EAAE,IAAI,EAAE,QAAQ,EAAE,WAAW,EAAE,gBAAgB,EAAE;IACjD,EAAE,IAAI,EAAE,UAAU,EAAE,WAAW,EAAE,oBAAoB,EAAE;CACvD,CAAC;AAEF,QAAQ,CAAC,+CAA+C,EAAE,GAAG,EAAE;IAC9D,EAAE,CAAC,iCAAiC,EAAE,GAAG,EAAE;QAC1C,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACpD,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,2BAA2B,CAAC,CAAC;QAC/C,MAAM,CAAC,KAAK,CAAC,MAAO,CAAC,KAAK,CAAC,MAAM,EAAE,cAAc,CAAC,MAAM,CAAC,CAAC;QAC1D,MAAM,CAAC,KAAK,CAAC,MAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kCAAkC,EAAE,GAAG,EAAE;QAC3C,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,KAAK,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACtD,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,CAAC;QAClB,wEAAwE;QACxE,qEAAqE;QACrE,oDAAoD;QACpD,MAAM,CAAC,EAAE,CAAC,MAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,UAAU,CAAC,CAAC,CAAC;QAC7D,MAAM,CAAC,EAAE,CAAC,MAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,SAAS,CAAC,CAAC,CAAC;QAC5D,MAAM,CAAC,KAAK,CAAC,MAAO,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC9C,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACvD,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;IAC5B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC9C,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACvD,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,CAAC;QAClB,iEAAiE;QACjE,qEAAqE;QACrE,iCAAiC;QACjC,MAAM,CAAC,EAAE,CAAC,MAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,OAAO,CAAC,CAAC,CAAC;QAC1D,MAAM,CAAC,EAAE,CACR,MAAO,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,CAAC,WAAW,KAAK,QAAQ,IAAI,CAAC,CAAC,WAAW,CAAC,MAAM,GAAG,CAAC,CAAC,EACzF,oDAAoD,CACpD,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC5D,MAAM,gBAAgB,GAAmB;YACxC,EAAE,IAAI,EAAE,qBAAqB,EAAE,WAAW,EAAE,wBAAwB,EAAE;SACtE,CAAC;QACF,MAAM,QAAQ,GAAG,YAAY,CAAC,gBAAgB,CAAC,CAAC;QAChD,MAAM,IAAI,GAAG,iBAAiB,CAAC;QAC/B,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,IAAI,CAAC,MAAM,CAAC,CAAC;QAE/D,IAAI,MAAM,KAAK,IAAI,EAAE,CAAC;YACrB,OAAO;QACR,CAAC;QAED,MAAM,CAAC,EAAE,CACR,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,KAAK,KAAK,qBAAqB,CAAC,EAClE,sEAAsE,CACtE,CAAC;QACF,MAAM,CAAC,EAAE,CACR,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,WAAW,KAAK,wBAAwB,CAAC,EAC3E,mEAAmE,CACnE,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC3D,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACxD,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,CAAC;QAClB,MAAM,CAAC,KAAK,CAAC,MAAO,CAAC,MAAM,EAAE,KAAK,CAAC,CAAC;QACpC,MAAM,CAAC,EAAE,CAAC,MAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,KAAK,KAAK,UAAU,CAAC,CAAC,CAAC;IACpE,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,qDAAqD,EAAE,GAAG,EAAE;IACpE,EAAE,CAAC,6DAA6D,EAAE,GAAG,EAAE;QACtE,MAAM,QAAQ,GAAmB;YAChC;gBACC,IAAI,EAAE,UAAU;gBAChB,WAAW,EAAE,oBAAoB;gBACjC,sBAAsB,EAAE,CAAC,MAAM,EAAE,EAAE;oBAClC,MAAM,MAAM,GAAG,CAAC,KAAK,EAAE,KAAK,EAAE,QAAQ,EAAE,MAAM,CAAC,CAAC;oBAChD,MAAM,QAAQ,GAAG,MAAM;yBACrB,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,MAAM,CAAC,IAAI,EAAE,CAAC,CAAC;yBAC1C,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;oBACvC,OAAO,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC;gBAC9C,CAAC;aACD;SACD,CAAC;QACF,MAAM,QAAQ,GAAG,YAAY,CAAC,QAAQ,CAAC,CAAC;QACxC,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,aAAa,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;QAC/D,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,CAAC;QAClB,mEAAmE;QACnE,4DAA4D;QAC5D,MAAM,CAAC,EAAE,CAAC,MAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,KAAK,CAAC,MAAO,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wDAAwD,EAAE,GAAG,EAAE;QACjE,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,eAAe,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;QACjE,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;IAC5B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0DAA0D,EAAE,GAAG,EAAE;QACnE,MAAM,QAAQ,GAAmB;YAChC;gBACC,IAAI,EAAE,MAAM;gBACZ,WAAW,EAAE,cAAc;gBAC3B,sBAAsB,EAAE,CAAC,MAAM,EAAE,EAAE;oBAClC,MAAM,IAAI,GAAG,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,CAAC,CAAC;oBACzC,MAAM,QAAQ,GAAG,IAAI;yBACnB,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,MAAM,CAAC,IAAI,EAAE,CAAC,CAAC;yBAC1C,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;oBACvC,OAAO,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC;gBAC9C,CAAC;aACD;SACD,CAAC;QACF,MAAM,QAAQ,GAAG,YAAY,CAAC,QAAQ,CAAC,CAAC;QACxC,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,QAAQ,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACzD,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,CAAC;QAClB,kEAAkE;QAClE,yDAAyD;QACzD,MAAM,CAAC,EAAE,CAAC,MAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,OAAO,CAAC,CAAC,CAAC;QAC1D,MAAM,CAAC,EAAE,CAAC,MAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,MAAM,CAAC,CAAC,CAAC;QACzD,MAAM,CAAC,EAAE,CAAC,MAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,KAAK,CAAC,MAAO,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,yDAAyD,EAAE,GAAG,EAAE;IACxE,EAAE,CAAC,iEAAiE,EAAE,GAAG,EAAE;QAC1E,MAAM,QAAQ,GAAG,YAAY,EAAE,CAAC;QAChC,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,kBAAkB,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;QACpE,gFAAgF;QAChF,yEAAyE;QACzE,qEAAqE;QACrE,IAAI,MAAM,KAAK,IAAI,EAAE,CAAC;YACrB,MAAM,CAAC,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,+BAA+B,CAAC,CAAC;YACxE,kEAAkE;YAClE,8CAA8C;YAC9C,MAAM,CAAC,KAAK,CAAC,OAAO,MAAM,CAAC,MAAM,EAAE,QAAQ,EAAE,yBAAyB,CAAC,CAAC;YACxE,MAAM,CAAC,EAAE,CACR,CAAC,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,GAAG,CAAC,EAC9B,iDAAiD,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,MAAM,CAAC,EAAE,CAChF,CAAC;QACH,CAAC;IACF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;QACrE,MAAM,QAAQ,GAAG,YAAY,EAAE,CAAC;QAChC,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,wBAAwB,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;QAC1E,IAAI,MAAM,KAAK,IAAI,EAAE,CAAC;YACrB,MAAM,CAAC,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,+BAA+B,CAAC,CAAC;YACxE,MAAM,CAAC,KAAK,CAAC,OAAO,MAAM,CAAC,MAAM,EAAE,QAAQ,EAAE,yBAAyB,CAAC,CAAC;YACxE,uEAAuE;YACvE,MAAM,CAAC,EAAE,CACR,CAAC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,EAChC,mDAAmD,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,MAAM,CAAC,EAAE,CAClF,CAAC;QACH,CAAC;IACF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uEAAuE,EAAE,GAAG,EAAE;QAChF,MAAM,QAAQ,GAAG,YAAY,CAAC,EAAE,EAAE,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;QACjD,kEAAkE;QAClE,oEAAoE;QACpE,wEAAwE;QACxE,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACpD,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,IAAI,EAAE,6CAA6C,CAAC,CAAC;IAC3E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC/D,MAAM,QAAQ,GAAG,YAAY,CAAC,EAAE,EAAE,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;QACjD,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAC,WAAW,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QAC5D,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,IAAI,EAAE,kEAAkE,CAAC,CAAC;IAChG,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,gDAAgD,EAAE,GAAG,EAAE;IAC/D,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC/D,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,eAAe,CAAC,CAAC,KAAK,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,EAAE,KAAK,EAAE,UAAU,EAAE,KAAK,EAAE,UAAU,EAAE,EAAE,KAAK,CAAC,CAAC;QACxG,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,YAAY,CAAC,CAAC;QAC5C,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,SAAS,EAAE,EAAE,CAAC,CAAC,CAAC,qBAAqB;IAC1D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qEAAqE,EAAE,GAAG,EAAE;QAC9E,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,eAAe,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,EAAE,KAAK,EAAE,UAAU,EAAE,KAAK,EAAE,UAAU,EAAE,EAAE,KAAK,CAAC,CAAC;QAC1G,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,cAAc,CAAC,CAAC;QAC9C,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,SAAS,EAAE,EAAE,CAAC,CAAC;IACpC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACpD,MAAM,QAAQ,GAAG,YAAY,EAAE,CAAC;QAChC,MAAM,MAAM,GAAG,QAAQ,CAAC,eAAe,CACtC,CAAC,OAAO,CAAC,EACT,CAAC,EACD,CAAC,EACD,EAAE,KAAK,EAAE,eAAe,EAAE,KAAK,EAAE,UAAU,EAAE,EAC7C,OAAO,CACP,CAAC;QACF,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,gBAAgB,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,GAAG,EAAE;QAC9D,MAAM,QAAQ,GAAG,YAAY,EAAE,CAAC;QAChC,MAAM,MAAM,GAAG,QAAQ,CAAC,eAAe,CACtC,CAAC,KAAK,CAAC,EACP,CAAC,EACD,CAAC,EACD,EAAE,KAAK,EAAE,OAAO,EAAE,KAAK,EAAE,MAAM,EAAE,EACjC,KAAK,CACL,CAAC;QACF,wEAAwE;QACxE,MAAM,CAAC,EAAE,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAE,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6BAA6B,EAAE,GAAG,EAAE;QACtC,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,eAAe,CACtC,CAAC,mBAAmB,CAAC,EACrB,CAAC,EACD,CAAC,EACD,EAAE,KAAK,EAAE,UAAU,EAAE,KAAK,EAAE,UAAU,EAAE,EACxC,KAAK,CACL,CAAC;QACF,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAE,CAAC,QAAQ,CAAC,eAAe,CAAC,CAAC,CAAC;IACvD,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,uDAAuD,EAAE,GAAG,EAAE;IACtE,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC9C,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,uBAAuB,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QAChE,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;IAC5B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;QACvE,MAAM,QAAQ,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC9C,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,2BAA2B,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,EAAE,KAAK,CAAC,CAAC;IAC3E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;QACpE,MAAM,QAAQ,GAAG,YAAY,EAAE,CAAC;QAChC,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,2BAA2B,CAAC,CAAC,WAAW,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC;IAC/E,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC","sourcesContent":["import { describe, it } from \"node:test\";\nimport assert from \"node:assert/strict\";\nimport { CombinedAutocompleteProvider } from \"../autocomplete.js\";\nimport type { SlashCommand } from \"../autocomplete.js\";\n\nfunction makeProvider(commands: SlashCommand[] = [], basePath: string = \"/tmp\") {\n\treturn new CombinedAutocompleteProvider(commands, basePath);\n}\n\nconst sampleCommands: SlashCommand[] = [\n\t{ name: \"settings\", description: \"Open settings menu\" },\n\t{ name: \"model\", description: \"Select model\" },\n\t{ name: \"session\", description: \"Show session info\" },\n\t{ name: \"export\", description: \"Export session\" },\n\t{ name: \"thinking\", description: \"Set thinking level\" },\n];\n\ndescribe(\"CombinedAutocompleteProvider — slash commands\", () => {\n\tit(\"returns all commands for bare /\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.getSuggestions([\"/\"], 0, 1);\n\t\tassert.ok(result, \"should return suggestions\");\n\t\tassert.equal(result!.items.length, sampleCommands.length);\n\t\tassert.equal(result!.prefix, \"/\");\n\t});\n\n\tit(\"filters commands by typed prefix\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.getSuggestions([\"/se\"], 0, 3);\n\t\tassert.ok(result);\n\t\t// /se matches settings and session — name the values, then assert exact\n\t\t// count as the contract for this prefix. Asserting count alone would\n\t\t// pass even if the matches were the wrong commands.\n\t\tassert.ok(result!.items.some((i) => i.value === \"settings\"));\n\t\tassert.ok(result!.items.some((i) => i.value === \"session\"));\n\t\tassert.equal(result!.items.length, 2);\n\t});\n\n\tit(\"returns null when no commands match\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.getSuggestions([\"/zzz\"], 0, 4);\n\t\tassert.equal(result, null);\n\t});\n\n\tit(\"includes description in suggestions\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.getSuggestions([\"/mod\"], 0, 4);\n\t\tassert.ok(result);\n\t\t// Verify the matched command is present and every item carries a\n\t\t// description — avoids `[0]` positional coupling that would silently\n\t\t// pass if list ordering changed.\n\t\tassert.ok(result!.items.some((i) => i.value === \"model\"));\n\t\tassert.ok(\n\t\t\tresult!.items.every((i) => typeof i.description === \"string\" && i.description.length > 0),\n\t\t\t\"every suggestion must have a non-empty description\",\n\t\t);\n\t});\n\n\tit(\"does not offer slash command suggestions mid-line\", () => {\n\t\tconst sentinelCommands: SlashCommand[] = [\n\t\t\t{ name: \"codexmidlinecommand\", description: \"Sentinel slash command\" },\n\t\t];\n\t\tconst provider = makeProvider(sentinelCommands);\n\t\tconst line = \"hello /codexmid\";\n\t\tconst result = provider.getSuggestions([line], 0, line.length);\n\n\t\tif (result === null) {\n\t\t\treturn;\n\t\t}\n\n\t\tassert.ok(\n\t\t\tresult.items.every((item) => item.value !== \"codexmidlinecommand\"),\n\t\t\t\"mid-line slash-like text should not return slash command completions\",\n\t\t);\n\t\tassert.ok(\n\t\t\tresult.items.every((item) => item.description !== \"Sentinel slash command\"),\n\t\t\t\"mid-line slash-like text should not return slash command metadata\",\n\t\t);\n\t});\n\n\tit(\"triggers slash commands after leading whitespace\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.getSuggestions([\"  /se\"], 0, 5);\n\t\tassert.ok(result);\n\t\tassert.equal(result!.prefix, \"/se\");\n\t\tassert.ok(result!.items.some((item) => item.value === \"settings\"));\n\t});\n});\n\ndescribe(\"CombinedAutocompleteProvider — argument completions\", () => {\n\tit(\"returns argument completions for commands that support them\", () => {\n\t\tconst commands: SlashCommand[] = [\n\t\t\t{\n\t\t\t\tname: \"thinking\",\n\t\t\t\tdescription: \"Set thinking level\",\n\t\t\t\tgetArgumentCompletions: (prefix) => {\n\t\t\t\t\tconst levels = [\"off\", \"low\", \"medium\", \"high\"];\n\t\t\t\t\tconst filtered = levels\n\t\t\t\t\t\t.filter((l) => l.startsWith(prefix.trim()))\n\t\t\t\t\t\t.map((l) => ({ value: l, label: l }));\n\t\t\t\t\treturn filtered.length > 0 ? filtered : null;\n\t\t\t\t},\n\t\t\t},\n\t\t];\n\t\tconst provider = makeProvider(commands);\n\t\tconst result = provider.getSuggestions([\"/thinking m\"], 0, 11);\n\t\tassert.ok(result);\n\t\t// /thinking m matches only \"medium\" — verify the expected value is\n\t\t// present (by name, not index) and then assert exact count.\n\t\tassert.ok(result!.items.some((i) => i.value === \"medium\"));\n\t\tassert.equal(result!.items.length, 1);\n\t});\n\n\tit(\"returns null for commands without argument completions\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.getSuggestions([\"/settings foo\"], 0, 13);\n\t\tassert.equal(result, null);\n\t});\n\n\tit(\"returns all arg completions for empty prefix after space\", () => {\n\t\tconst commands: SlashCommand[] = [\n\t\t\t{\n\t\t\t\tname: \"test\",\n\t\t\t\tdescription: \"Test command\",\n\t\t\t\tgetArgumentCompletions: (prefix) => {\n\t\t\t\t\tconst subs = [\"start\", \"stop\", \"status\"];\n\t\t\t\t\tconst filtered = subs\n\t\t\t\t\t\t.filter((s) => s.startsWith(prefix.trim()))\n\t\t\t\t\t\t.map((s) => ({ value: s, label: s }));\n\t\t\t\t\treturn filtered.length > 0 ? filtered : null;\n\t\t\t\t},\n\t\t\t},\n\t\t];\n\t\tconst provider = makeProvider(commands);\n\t\tconst result = provider.getSuggestions([\"/test \"], 0, 6);\n\t\tassert.ok(result);\n\t\t// Empty prefix returns all 3 subcommands — verify each is present\n\t\t// by name. Bare count would pass for any 3-element list.\n\t\tassert.ok(result!.items.some((i) => i.value === \"start\"));\n\t\tassert.ok(result!.items.some((i) => i.value === \"stop\"));\n\t\tassert.ok(result!.items.some((i) => i.value === \"status\"));\n\t\tassert.equal(result!.items.length, 3);\n\t});\n});\n\ndescribe(\"CombinedAutocompleteProvider — @ file prefix extraction\", () => {\n\tit(\"detects @ at start of line and returns a valid suggestion shape\", () => {\n\t\tconst provider = makeProvider();\n\t\tconst result = provider.getSuggestions([\"@nonexistent_xyz\"], 0, 16);\n\t\t// Either null (nothing matched) or a well-formed {items: Array, prefix: string}\n\t\t// shape. Previous version's `result.items.length >= 0` was a tautology —\n\t\t// array length is always ≥ 0; the whole expression could never fail.\n\t\tif (result !== null) {\n\t\t\tassert.ok(Array.isArray(result.items), \"result.items must be an array\");\n\t\t\t// The @-prefix extraction strips the leading @ — prefix should be\n\t\t\t// the raw text without the trigger character.\n\t\t\tassert.equal(typeof result.prefix, \"string\", \"prefix must be a string\");\n\t\t\tassert.ok(\n\t\t\t\t!result.prefix.startsWith(\"@\"),\n\t\t\t\t`prefix must have the @ trigger stripped, got: ${JSON.stringify(result.prefix)}`,\n\t\t\t);\n\t\t}\n\t});\n\n\tit(\"detects @ after space and returns a valid suggestion shape\", () => {\n\t\tconst provider = makeProvider();\n\t\tconst result = provider.getSuggestions([\"check @nonexistent_xyz\"], 0, 22);\n\t\tif (result !== null) {\n\t\t\tassert.ok(Array.isArray(result.items), \"result.items must be an array\");\n\t\t\tassert.equal(typeof result.prefix, \"string\", \"prefix must be a string\");\n\t\t\t// The prefix must NOT include the word \"check\" that came before the @.\n\t\t\tassert.ok(\n\t\t\t\t!result.prefix.includes(\"check\"),\n\t\t\t\t`prefix must not include text before the @, got: ${JSON.stringify(result.prefix)}`,\n\t\t\t);\n\t\t}\n\t});\n\n\tit(\"returns null for bare @ with no query to avoid full tree walk (#1824)\", () => {\n\t\tconst provider = makeProvider([], process.cwd());\n\t\t// A bare \"@\" produces an empty rawPrefix after stripping the \"@\".\n\t\t// This must return null to avoid a synchronous full filesystem walk\n\t\t// via the native fuzzyFind addon, which freezes the TUI on large repos.\n\t\tconst result = provider.getSuggestions([\"@\"], 0, 1);\n\t\tassert.equal(result, null, \"bare @ should not trigger fuzzy file search\");\n\t});\n\n\tit(\"returns null for @ after space with no query (#1824)\", () => {\n\t\tconst provider = makeProvider([], process.cwd());\n\t\tconst result = provider.getSuggestions([\"look at @\"], 0, 9);\n\t\tassert.equal(result, null, \"@ after space with no query should not trigger fuzzy file search\");\n\t});\n});\n\ndescribe(\"CombinedAutocompleteProvider — applyCompletion\", () => {\n\tit(\"applies slash command completion with trailing space\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.applyCompletion([\"/se\"], 0, 3, { value: \"settings\", label: \"settings\" }, \"/se\");\n\t\tassert.equal(result.lines[0], \"/settings \");\n\t\tassert.equal(result.cursorCol, 10); // after \"/settings \"\n\t});\n\n\tit(\"preserves leading whitespace when applying slash command completion\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.applyCompletion([\"  /se\"], 0, 5, { value: \"settings\", label: \"settings\" }, \"/se\");\n\t\tassert.equal(result.lines[0], \"  /settings \");\n\t\tassert.equal(result.cursorCol, 12);\n\t});\n\n\tit(\"applies file path completion for @ prefix\", () => {\n\t\tconst provider = makeProvider();\n\t\tconst result = provider.applyCompletion(\n\t\t\t[\"@src/\"],\n\t\t\t0,\n\t\t\t5,\n\t\t\t{ value: \"@src/index.ts\", label: \"index.ts\" },\n\t\t\t\"@src/\",\n\t\t);\n\t\tassert.equal(result.lines[0], \"@src/index.ts \");\n\t});\n\n\tit(\"applies directory completion without trailing space\", () => {\n\t\tconst provider = makeProvider();\n\t\tconst result = provider.applyCompletion(\n\t\t\t[\"@sr\"],\n\t\t\t0,\n\t\t\t3,\n\t\t\t{ value: \"@src/\", label: \"src/\" },\n\t\t\t\"@sr\",\n\t\t);\n\t\t// Directories should not get trailing space so user can continue typing\n\t\tassert.ok(!result.lines[0]!.endsWith(\" \"));\n\t});\n\n\tit(\"preserves text after cursor\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.applyCompletion(\n\t\t\t[\"/se and more text\"],\n\t\t\t0,\n\t\t\t3,\n\t\t\t{ value: \"settings\", label: \"settings\" },\n\t\t\t\"/se\",\n\t\t);\n\t\tassert.ok(result.lines[0]!.includes(\"and more text\"));\n\t});\n});\n\ndescribe(\"CombinedAutocompleteProvider — force file suggestions\", () => {\n\tit(\"does not trigger for slash commands\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tconst result = provider.getForceFileSuggestions([\"/set\"], 0, 4);\n\t\tassert.equal(result, null);\n\t});\n\n\tit(\"shouldTriggerFileCompletion returns false for slash commands\", () => {\n\t\tconst provider = makeProvider(sampleCommands);\n\t\tassert.equal(provider.shouldTriggerFileCompletion([\"/set\"], 0, 4), false);\n\t});\n\n\tit(\"shouldTriggerFileCompletion returns true for regular text\", () => {\n\t\tconst provider = makeProvider();\n\t\tassert.equal(provider.shouldTriggerFileCompletion([\"some text\"], 0, 9), true);\n\t});\n});\n"]}

package/packages/pi-tui/dist/components/__tests__/leak-fixes-runtime.test.d.ts

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

package/packages/pi-tui/dist/components/__tests__/leak-fixes-runtime.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"leak-fixes-runtime.test.d.ts","sourceRoot":"","sources":["../../../src/components/__tests__/leak-fixes-runtime.test.ts"],"names":[],"mappings":""}

package/packages/pi-tui/dist/components/__tests__/leak-fixes-runtime.test.js

@@ -0,0 +1,161 @@
+// Runtime regression tests for the long-running-session leak fixes.
+//
+// Replaces the source-grep file `src/tests/session-memory-leaks.test.ts`
+// (deleted in #4875, tracked as #4873). The previous tests asserted on
+// identifier presence (`_prevRender`, `_lastMessage`, `MAX_CHAT_COMPONENTS`)
+// in source — a regression that set `MAX_CHAT_COMPONENTS = Number.MAX_SAFE_INTEGER`
+// or replaced `setText` with an inline mutation would still pass.
+//
+// Each test below drives the actual component through a long-running scenario
+// and asserts on observable behaviour.
+import { describe, it, mock, beforeEach, afterEach } from "node:test";
+import assert from "node:assert/strict";
+import { Container } from "../../tui.js";
+import { Loader } from "../loader.js";
+import { Text } from "../text.js";
+function makeMockTUI() {
+    return { requestRender: mock.fn() };
+}
+// ─── Container render-skip ──────────────────────────────────────────────
+describe("Container.render skips work when output is unchanged", () => {
+    it("returns the SAME array reference across two renders with no changes", () => {
+        // The container caches `_prevRender` so doRender() can short-circuit
+        // the post-processing (image-line scan, applyLineResets, line diffs).
+        // The contract for the optimization is reference equality of the
+        // returned array — that's what the consumer in tui.ts checks.
+        const c = new Container();
+        c.addChild(new Text("hello", 0, 0));
+        const first = c.render(20);
+        const second = c.render(20);
+        assert.strictEqual(second, first, "Container must return the same array reference when content is unchanged " +
+            "(reference equality is the consumer-visible contract for the skip)");
+    });
+    it("returns a DIFFERENT array reference after addChild invalidates the cache", () => {
+        // If a regression caused `_prevRender` to never reset, downstream
+        // rendering would skip post-processing for new components — visible
+        // as missing/stale UI. Behaviourally we observe that adding a child
+        // breaks the reference-equality contract on the next render.
+        const c = new Container();
+        c.addChild(new Text("a", 0, 0));
+        const first = c.render(20);
+        c.addChild(new Text("b", 0, 0));
+        const second = c.render(20);
+        assert.notStrictEqual(second, first, "adding a child must invalidate the cached render reference");
+    });
+    it("returns a different reference after content changes", () => {
+        const t = new Text("hello", 0, 0);
+        const c = new Container();
+        c.addChild(t);
+        const first = c.render(20);
+        t.setText("world");
+        const second = c.render(20);
+        assert.notStrictEqual(second, first, "changing a child's text must produce a fresh array (not the cached one)");
+    });
+});
+// ─── Loader frame isolation ─────────────────────────────────────────────
+describe("Loader does not mutate Text cache on every spinner tick", () => {
+    let tui;
+    let loader;
+    beforeEach(() => {
+        tui = makeMockTUI();
+        mock.timers.enable({ apis: ["setInterval"] });
+    });
+    afterEach(() => {
+        try {
+            loader?.stop();
+        }
+        catch {
+            /* best-effort */
+        }
+        mock.timers.reset();
+    });
+    it("does not invalidate Text's render cache across N spinner ticks", () => {
+        // Loader extends Text. Text caches its rendered lines keyed by
+        // (text, width). The leak was: the old Loader called setText()
+        // on every 80ms tick, which always cleared the cache, forcing a
+        // re-wrap of the message text.
+        // Behavioural test: render the loader, capture the cached array
+        // reference returned by Text.render, advance many ticks, and
+        // assert the underlying Text cache was NOT invalidated (we observe
+        // this via reference equality of the cached lines slice in the
+        // returned array — the second `result[*]` segment from super.render
+        // stays identity-stable when the cache survives).
+        loader = new Loader(tui, (s) => s, (s) => s, "the-message");
+        const before = loader.render(40);
+        // Run 50 frame intervals — the spinner should rotate, but the
+        // underlying message text wrapping must not change.
+        mock.timers.tick(80 * 50);
+        const after = loader.render(40);
+        // The message portion (everything after the first `""` padding line
+        // and after the spinner glyph + space prefix on the first content
+        // line) must be byte-identical across renders.
+        assert.equal(before.length, after.length, "render shape stable across ticks");
+        // Trailing message lines (index 2 onwards) come straight from Text's
+        // cache without any spinner mutation. They must be reference-equal
+        // substrings — same string contents byte-for-byte.
+        for (let i = 2; i < before.length; i++) {
+            assert.equal(after[i], before[i], `message line ${i} must remain byte-identical across spinner ticks ` +
+                `(cache must not be invalidated by frame rotation)`);
+        }
+    });
+    it("setMessage invalidates Text cache and a new message is reflected", () => {
+        // Quiescence is conditional on the message being unchanged — we
+        // must still see updates when it actually changes. This is the
+        // counter-test that prevents a false-positive "cache always stable"
+        // fix that would freeze the loader text.
+        loader = new Loader(tui, (s) => s, (s) => s, "first");
+        const beforeFirstRender = loader.render(40).join("\n");
+        loader.setMessage("second");
+        const afterChange = loader.render(40).join("\n");
+        assert.ok(afterChange.includes("second"), "new message must render");
+        assert.ok(!afterChange.includes("first") || beforeFirstRender !== afterChange, "render output must change when message changes");
+    });
+});
+// ─── Text.setText early-return guard ────────────────────────────────────
+describe("Text.setText returns early when value is unchanged", () => {
+    it("does not invalidate the cached render when setText receives the same value", () => {
+        // The setText early-return is the runtime guard that keeps the
+        // Loader-and-Text cache stable. We observe it via reference equality
+        // of the rendered output across a setText(SAME) call.
+        const t = new Text("identical", 0, 0);
+        const first = t.render(30);
+        t.setText("identical"); // same value — must NOT clear the cache
+        const second = t.render(30);
+        assert.strictEqual(second, first, "setText with an unchanged value must leave the render cache intact");
+    });
+    it("invalidates cache when the value actually changes", () => {
+        const t = new Text("one", 0, 0);
+        const first = t.render(30);
+        t.setText("two");
+        const second = t.render(30);
+        assert.notStrictEqual(second, first, "setText with a new value must produce a fresh render result");
+    });
+});
+// ─── Heap growth bound (gated on --expose-gc) ───────────────────────────
+describe("Long-running render loop does not leak heap (forced-GC bound)", () => {
+    const hasGc = typeof globalThis.gc === "function";
+    it("renders a Container 5000 times with stable heapUsed (within 2x baseline)", { skip: !hasGc ? "requires --expose-gc to drive deterministic GC between snapshots" : false }, () => {
+        // This is the on-the-wire memory-soak test. The naive version
+        // (no forced GC) is flaky because Node's GC is opportunistic;
+        // any background allocation can shift heapUsed past an absolute
+        // bound. We instead force GC between snapshots and assert a
+        // RELATIVE bound: post-iteration heap must be within 2x of the
+        // post-warmup baseline.
+        const gc = globalThis.gc;
+        const c = new Container();
+        const t = new Text("steady-state", 0, 0);
+        c.addChild(t);
+        // Warm up so the JIT and any lazy initial allocations settle.
+        for (let i = 0; i < 200; i++)
+            c.render(40);
+        gc();
+        const baseline = process.memoryUsage().heapUsed;
+        for (let i = 0; i < 5000; i++)
+            c.render(40);
+        gc();
+        const after = process.memoryUsage().heapUsed;
+        assert.ok(after < baseline * 2, `heapUsed must stay within 2x baseline after 5000 renders ` +
+            `(baseline=${baseline}B, after=${after}B, ratio=${(after / baseline).toFixed(2)})`);
+    });
+});
+//# sourceMappingURL=leak-fixes-runtime.test.js.map

package/packages/pi-tui/dist/components/__tests__/leak-fixes-runtime.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"leak-fixes-runtime.test.js","sourceRoot":"","sources":["../../../src/components/__tests__/leak-fixes-runtime.test.ts"],"names":[],"mappings":"AAAA,oEAAoE;AACpE,EAAE;AACF,yEAAyE;AACzE,uEAAuE;AACvE,6EAA6E;AAC7E,oFAAoF;AACpF,kEAAkE;AAClE,EAAE;AACF,8EAA8E;AAC9E,uCAAuC;AAEvC,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,WAAW,CAAC;AACtE,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,EAAE,SAAS,EAAE,MAAM,cAAc,CAAC;AACzC,OAAO,EAAE,MAAM,EAAE,MAAM,cAAc,CAAC;AACtC,OAAO,EAAE,IAAI,EAAE,MAAM,YAAY,CAAC;AAMlC,SAAS,WAAW;IACnB,OAAO,EAAE,aAAa,EAAE,IAAI,CAAC,EAAE,EAAE,EAAE,CAAC;AACrC,CAAC;AAED,2EAA2E;AAE3E,QAAQ,CAAC,sDAAsD,EAAE,GAAG,EAAE;IACrE,EAAE,CAAC,qEAAqE,EAAE,GAAG,EAAE;QAC9E,qEAAqE;QACrE,sEAAsE;QACtE,iEAAiE;QACjE,8DAA8D;QAC9D,MAAM,CAAC,GAAG,IAAI,SAAS,EAAE,CAAC;QAC1B,CAAC,CAAC,QAAQ,CAAC,IAAI,IAAI,CAAC,OAAO,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC;QAEpC,MAAM,KAAK,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAC3B,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAC5B,MAAM,CAAC,WAAW,CACjB,MAAM,EACN,KAAK,EACL,2EAA2E;YAC1E,oEAAoE,CACrE,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0EAA0E,EAAE,GAAG,EAAE;QACnF,kEAAkE;QAClE,oEAAoE;QACpE,oEAAoE;QACpE,6DAA6D;QAC7D,MAAM,CAAC,GAAG,IAAI,SAAS,EAAE,CAAC;QAC1B,CAAC,CAAC,QAAQ,CAAC,IAAI,IAAI,CAAC,GAAG,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC;QAChC,MAAM,KAAK,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAC3B,CAAC,CAAC,QAAQ,CAAC,IAAI,IAAI,CAAC,GAAG,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC;QAChC,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAC5B,MAAM,CAAC,cAAc,CACpB,MAAM,EACN,KAAK,EACL,4DAA4D,CAC5D,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,GAAG,EAAE;QAC9D,MAAM,CAAC,GAAG,IAAI,IAAI,CAAC,OAAO,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QAClC,MAAM,CAAC,GAAG,IAAI,SAAS,EAAE,CAAC;QAC1B,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC;QAEd,MAAM,KAAK,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAC3B,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;QACnB,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAC5B,MAAM,CAAC,cAAc,CACpB,MAAM,EACN,KAAK,EACL,yEAAyE,CACzE,CAAC;IACH,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,2EAA2E;AAE3E,QAAQ,CAAC,yDAAyD,EAAE,GAAG,EAAE;IACxE,IAAI,GAAY,CAAC;IACjB,IAAI,MAAc,CAAC;IAEnB,UAAU,CAAC,GAAG,EAAE;QACf,GAAG,GAAG,WAAW,EAAE,CAAC;QACpB,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,EAAE,IAAI,EAAE,CAAC,aAAa,CAAC,EAAE,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACd,IAAI,CAAC;YACJ,MAAM,EAAE,IAAI,EAAE,CAAC;QAChB,CAAC;QAAC,MAAM,CAAC;YACR,iBAAiB;QAClB,CAAC;QACD,IAAI,CAAC,MAAM,CAAC,KAAK,EAAE,CAAC;IACrB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gEAAgE,EAAE,GAAG,EAAE;QACzE,+DAA+D;QAC/D,+DAA+D;QAC/D,gEAAgE;QAChE,+BAA+B;QAC/B,gEAAgE;QAChE,6DAA6D;QAC7D,mEAAmE;QACnE,+DAA+D;QAC/D,oEAAoE;QACpE,kDAAkD;QAClD,MAAM,GAAG,IAAI,MAAM,CAAC,GAAY,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,EAAE,aAAa,CAAC,CAAC;QAErE,MAAM,MAAM,GAAG,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QACjC,8DAA8D;QAC9D,oDAAoD;QACpD,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,EAAE,GAAG,EAAE,CAAC,CAAC;QAC1B,MAAM,KAAK,GAAG,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAEhC,oEAAoE;QACpE,kEAAkE;QAClE,+CAA+C;QAC/C,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,MAAM,EAAE,KAAK,CAAC,MAAM,EAAE,kCAAkC,CAAC,CAAC;QAC9E,qEAAqE;QACrE,mEAAmE;QACnE,mDAAmD;QACnD,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,MAAM,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;YACxC,MAAM,CAAC,KAAK,CACX,KAAK,CAAC,CAAC,CAAC,EACR,MAAM,CAAC,CAAC,CAAC,EACT,gBAAgB,CAAC,mDAAmD;gBACnE,mDAAmD,CACpD,CAAC;QACH,CAAC;IACF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kEAAkE,EAAE,GAAG,EAAE;QAC3E,gEAAgE;QAChE,+DAA+D;QAC/D,oEAAoE;QACpE,yCAAyC;QACzC,MAAM,GAAG,IAAI,MAAM,CAAC,GAAY,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,EAAE,OAAO,CAAC,CAAC;QAC/D,MAAM,iBAAiB,GAAG,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACvD,MAAM,CAAC,UAAU,CAAC,QAAQ,CAAC,CAAC;QAC5B,MAAM,WAAW,GAAG,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACjD,MAAM,CAAC,EAAE,CAAC,WAAW,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,yBAAyB,CAAC,CAAC;QACrE,MAAM,CAAC,EAAE,CAAC,CAAC,WAAW,CAAC,QAAQ,CAAC,OAAO,CAAC,IAAI,iBAAiB,KAAK,WAAW,EAC5E,gDAAgD,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,2EAA2E;AAE3E,QAAQ,CAAC,oDAAoD,EAAE,GAAG,EAAE;IACnE,EAAE,CAAC,4EAA4E,EAAE,GAAG,EAAE;QACrF,+DAA+D;QAC/D,qEAAqE;QACrE,sDAAsD;QACtD,MAAM,CAAC,GAAG,IAAI,IAAI,CAAC,WAAW,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACtC,MAAM,KAAK,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAC3B,CAAC,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC,CAAC,wCAAwC;QAChE,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAC5B,MAAM,CAAC,WAAW,CACjB,MAAM,EACN,KAAK,EACL,oEAAoE,CACpE,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC5D,MAAM,CAAC,GAAG,IAAI,IAAI,CAAC,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QAChC,MAAM,KAAK,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAC3B,CAAC,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC;QACjB,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAC5B,MAAM,CAAC,cAAc,CACpB,MAAM,EACN,KAAK,EACL,6DAA6D,CAC7D,CAAC;IACH,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,2EAA2E;AAE3E,QAAQ,CAAC,+DAA+D,EAAE,GAAG,EAAE;IAC9E,MAAM,KAAK,GAAG,OAAQ,UAAkB,CAAC,EAAE,KAAK,UAAU,CAAC;IAE3D,EAAE,CACD,0EAA0E,EAC1E,EAAE,IAAI,EAAE,CAAC,KAAK,CAAC,CAAC,CAAC,kEAAkE,CAAC,CAAC,CAAC,KAAK,EAAE,EAC7F,GAAG,EAAE;QACJ,8DAA8D;QAC9D,8DAA8D;QAC9D,gEAAgE;QAChE,4DAA4D;QAC5D,+DAA+D;QAC/D,wBAAwB;QACxB,MAAM,EAAE,GAAI,UAAkB,CAAC,EAAgB,CAAC;QAEhD,MAAM,CAAC,GAAG,IAAI,SAAS,EAAE,CAAC;QAC1B,MAAM,CAAC,GAAG,IAAI,IAAI,CAAC,cAAc,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACzC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC;QAEd,8DAA8D;QAC9D,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,GAAG,EAAE,CAAC,EAAE;YAAE,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAC3C,EAAE,EAAE,CAAC;QACL,MAAM,QAAQ,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC;QAEhD,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,IAAI,EAAE,CAAC,EAAE;YAAE,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QAC5C,EAAE,EAAE,CAAC;QACL,MAAM,KAAK,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC;QAE7C,MAAM,CAAC,EAAE,CACR,KAAK,GAAG,QAAQ,GAAG,CAAC,EACpB,2DAA2D;YAC1D,aAAa,QAAQ,YAAY,KAAK,YAAY,CAAC,KAAK,GAAG,QAAQ,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,GAAG,CACnF,CAAC;IACH,CAAC,CACD,CAAC;AACH,CAAC,CAAC,CAAC","sourcesContent":["// Runtime regression tests for the long-running-session leak fixes.\n//\n// Replaces the source-grep file `src/tests/session-memory-leaks.test.ts`\n// (deleted in #4875, tracked as #4873). The previous tests asserted on\n// identifier presence (`_prevRender`, `_lastMessage`, `MAX_CHAT_COMPONENTS`)\n// in source — a regression that set `MAX_CHAT_COMPONENTS = Number.MAX_SAFE_INTEGER`\n// or replaced `setText` with an inline mutation would still pass.\n//\n// Each test below drives the actual component through a long-running scenario\n// and asserts on observable behaviour.\n\nimport { describe, it, mock, beforeEach, afterEach } from \"node:test\";\nimport assert from \"node:assert/strict\";\nimport { Container } from \"../../tui.js\";\nimport { Loader } from \"../loader.js\";\nimport { Text } from \"../text.js\";\n\ninterface MockTui {\n\trequestRender: ReturnType<typeof mock.fn>;\n}\n\nfunction makeMockTUI(): MockTui {\n\treturn { requestRender: mock.fn() };\n}\n\n// ─── Container render-skip ──────────────────────────────────────────────\n\ndescribe(\"Container.render skips work when output is unchanged\", () => {\n\tit(\"returns the SAME array reference across two renders with no changes\", () => {\n\t\t// The container caches `_prevRender` so doRender() can short-circuit\n\t\t// the post-processing (image-line scan, applyLineResets, line diffs).\n\t\t// The contract for the optimization is reference equality of the\n\t\t// returned array — that's what the consumer in tui.ts checks.\n\t\tconst c = new Container();\n\t\tc.addChild(new Text(\"hello\", 0, 0));\n\n\t\tconst first = c.render(20);\n\t\tconst second = c.render(20);\n\t\tassert.strictEqual(\n\t\t\tsecond,\n\t\t\tfirst,\n\t\t\t\"Container must return the same array reference when content is unchanged \" +\n\t\t\t\t\"(reference equality is the consumer-visible contract for the skip)\",\n\t\t);\n\t});\n\n\tit(\"returns a DIFFERENT array reference after addChild invalidates the cache\", () => {\n\t\t// If a regression caused `_prevRender` to never reset, downstream\n\t\t// rendering would skip post-processing for new components — visible\n\t\t// as missing/stale UI. Behaviourally we observe that adding a child\n\t\t// breaks the reference-equality contract on the next render.\n\t\tconst c = new Container();\n\t\tc.addChild(new Text(\"a\", 0, 0));\n\t\tconst first = c.render(20);\n\t\tc.addChild(new Text(\"b\", 0, 0));\n\t\tconst second = c.render(20);\n\t\tassert.notStrictEqual(\n\t\t\tsecond,\n\t\t\tfirst,\n\t\t\t\"adding a child must invalidate the cached render reference\",\n\t\t);\n\t});\n\n\tit(\"returns a different reference after content changes\", () => {\n\t\tconst t = new Text(\"hello\", 0, 0);\n\t\tconst c = new Container();\n\t\tc.addChild(t);\n\n\t\tconst first = c.render(20);\n\t\tt.setText(\"world\");\n\t\tconst second = c.render(20);\n\t\tassert.notStrictEqual(\n\t\t\tsecond,\n\t\t\tfirst,\n\t\t\t\"changing a child's text must produce a fresh array (not the cached one)\",\n\t\t);\n\t});\n});\n\n// ─── Loader frame isolation ─────────────────────────────────────────────\n\ndescribe(\"Loader does not mutate Text cache on every spinner tick\", () => {\n\tlet tui: MockTui;\n\tlet loader: Loader;\n\n\tbeforeEach(() => {\n\t\ttui = makeMockTUI();\n\t\tmock.timers.enable({ apis: [\"setInterval\"] });\n\t});\n\n\tafterEach(() => {\n\t\ttry {\n\t\t\tloader?.stop();\n\t\t} catch {\n\t\t\t/* best-effort */\n\t\t}\n\t\tmock.timers.reset();\n\t});\n\n\tit(\"does not invalidate Text's render cache across N spinner ticks\", () => {\n\t\t// Loader extends Text. Text caches its rendered lines keyed by\n\t\t// (text, width). The leak was: the old Loader called setText()\n\t\t// on every 80ms tick, which always cleared the cache, forcing a\n\t\t// re-wrap of the message text.\n\t\t// Behavioural test: render the loader, capture the cached array\n\t\t// reference returned by Text.render, advance many ticks, and\n\t\t// assert the underlying Text cache was NOT invalidated (we observe\n\t\t// this via reference equality of the cached lines slice in the\n\t\t// returned array — the second `result[*]` segment from super.render\n\t\t// stays identity-stable when the cache survives).\n\t\tloader = new Loader(tui as never, (s) => s, (s) => s, \"the-message\");\n\n\t\tconst before = loader.render(40);\n\t\t// Run 50 frame intervals — the spinner should rotate, but the\n\t\t// underlying message text wrapping must not change.\n\t\tmock.timers.tick(80 * 50);\n\t\tconst after = loader.render(40);\n\n\t\t// The message portion (everything after the first `\"\"` padding line\n\t\t// and after the spinner glyph + space prefix on the first content\n\t\t// line) must be byte-identical across renders.\n\t\tassert.equal(before.length, after.length, \"render shape stable across ticks\");\n\t\t// Trailing message lines (index 2 onwards) come straight from Text's\n\t\t// cache without any spinner mutation. They must be reference-equal\n\t\t// substrings — same string contents byte-for-byte.\n\t\tfor (let i = 2; i < before.length; i++) {\n\t\t\tassert.equal(\n\t\t\t\tafter[i],\n\t\t\t\tbefore[i],\n\t\t\t\t`message line ${i} must remain byte-identical across spinner ticks ` +\n\t\t\t\t\t`(cache must not be invalidated by frame rotation)`,\n\t\t\t);\n\t\t}\n\t});\n\n\tit(\"setMessage invalidates Text cache and a new message is reflected\", () => {\n\t\t// Quiescence is conditional on the message being unchanged — we\n\t\t// must still see updates when it actually changes. This is the\n\t\t// counter-test that prevents a false-positive \"cache always stable\"\n\t\t// fix that would freeze the loader text.\n\t\tloader = new Loader(tui as never, (s) => s, (s) => s, \"first\");\n\t\tconst beforeFirstRender = loader.render(40).join(\"\\n\");\n\t\tloader.setMessage(\"second\");\n\t\tconst afterChange = loader.render(40).join(\"\\n\");\n\t\tassert.ok(afterChange.includes(\"second\"), \"new message must render\");\n\t\tassert.ok(!afterChange.includes(\"first\") || beforeFirstRender !== afterChange,\n\t\t\t\"render output must change when message changes\");\n\t});\n});\n\n// ─── Text.setText early-return guard ────────────────────────────────────\n\ndescribe(\"Text.setText returns early when value is unchanged\", () => {\n\tit(\"does not invalidate the cached render when setText receives the same value\", () => {\n\t\t// The setText early-return is the runtime guard that keeps the\n\t\t// Loader-and-Text cache stable. We observe it via reference equality\n\t\t// of the rendered output across a setText(SAME) call.\n\t\tconst t = new Text(\"identical\", 0, 0);\n\t\tconst first = t.render(30);\n\t\tt.setText(\"identical\"); // same value — must NOT clear the cache\n\t\tconst second = t.render(30);\n\t\tassert.strictEqual(\n\t\t\tsecond,\n\t\t\tfirst,\n\t\t\t\"setText with an unchanged value must leave the render cache intact\",\n\t\t);\n\t});\n\n\tit(\"invalidates cache when the value actually changes\", () => {\n\t\tconst t = new Text(\"one\", 0, 0);\n\t\tconst first = t.render(30);\n\t\tt.setText(\"two\");\n\t\tconst second = t.render(30);\n\t\tassert.notStrictEqual(\n\t\t\tsecond,\n\t\t\tfirst,\n\t\t\t\"setText with a new value must produce a fresh render result\",\n\t\t);\n\t});\n});\n\n// ─── Heap growth bound (gated on --expose-gc) ───────────────────────────\n\ndescribe(\"Long-running render loop does not leak heap (forced-GC bound)\", () => {\n\tconst hasGc = typeof (globalThis as any).gc === \"function\";\n\n\tit(\n\t\t\"renders a Container 5000 times with stable heapUsed (within 2x baseline)\",\n\t\t{ skip: !hasGc ? \"requires --expose-gc to drive deterministic GC between snapshots\" : false },\n\t\t() => {\n\t\t\t// This is the on-the-wire memory-soak test. The naive version\n\t\t\t// (no forced GC) is flaky because Node's GC is opportunistic;\n\t\t\t// any background allocation can shift heapUsed past an absolute\n\t\t\t// bound. We instead force GC between snapshots and assert a\n\t\t\t// RELATIVE bound: post-iteration heap must be within 2x of the\n\t\t\t// post-warmup baseline.\n\t\t\tconst gc = (globalThis as any).gc as () => void;\n\n\t\t\tconst c = new Container();\n\t\t\tconst t = new Text(\"steady-state\", 0, 0);\n\t\t\tc.addChild(t);\n\n\t\t\t// Warm up so the JIT and any lazy initial allocations settle.\n\t\t\tfor (let i = 0; i < 200; i++) c.render(40);\n\t\t\tgc();\n\t\t\tconst baseline = process.memoryUsage().heapUsed;\n\n\t\t\tfor (let i = 0; i < 5000; i++) c.render(40);\n\t\t\tgc();\n\t\t\tconst after = process.memoryUsage().heapUsed;\n\n\t\t\tassert.ok(\n\t\t\t\tafter < baseline * 2,\n\t\t\t\t`heapUsed must stay within 2x baseline after 5000 renders ` +\n\t\t\t\t\t`(baseline=${baseline}B, after=${after}B, ratio=${(after / baseline).toFixed(2)})`,\n\t\t\t);\n\t\t},\n\t);\n});\n"]}

package/packages/pi-tui/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gsd/pi-tui",
-  "version": "2.78.0",
+  "version": "2.78.1",
   "description": "Terminal User Interface library (vendored from pi-mono)",
   "type": "module",
   "gsd": {

package/packages/pi-tui/src/__tests__/autocomplete.test.ts

@@ -28,9 +28,12 @@ describe("CombinedAutocompleteProvider — slash commands", () => {
 		const provider = makeProvider(sampleCommands);
 		const result = provider.getSuggestions(["/se"], 0, 3);
 		assert.ok(result);
-		assert.equal(result!.items.length, 2); // settings, session
+		// /se matches settings and session — name the values, then assert exact
+		// count as the contract for this prefix. Asserting count alone would
+		// pass even if the matches were the wrong commands.
 		assert.ok(result!.items.some((i) => i.value === "settings"));
 		assert.ok(result!.items.some((i) => i.value === "session"));
+		assert.equal(result!.items.length, 2);
 	});
 
 	it("returns null when no commands match", () => {
@@ -43,7 +46,14 @@ describe("CombinedAutocompleteProvider — slash commands", () => {
 		const provider = makeProvider(sampleCommands);
 		const result = provider.getSuggestions(["/mod"], 0, 4);
 		assert.ok(result);
-		assert.equal(result!.items[0]?.description, "Select model");
+		// Verify the matched command is present and every item carries a
+		// description — avoids `[0]` positional coupling that would silently
+		// pass if list ordering changed.
+		assert.ok(result!.items.some((i) => i.value === "model"));
+		assert.ok(
+			result!.items.every((i) => typeof i.description === "string" && i.description.length > 0),
+			"every suggestion must have a non-empty description",
+		);
 	});
 
 	it("does not offer slash command suggestions mid-line", () => {
@@ -95,8 +105,10 @@ describe("CombinedAutocompleteProvider — argument completions", () => {
 		const provider = makeProvider(commands);
 		const result = provider.getSuggestions(["/thinking m"], 0, 11);
 		assert.ok(result);
+		// /thinking m matches only "medium" — verify the expected value is
+		// present (by name, not index) and then assert exact count.
+		assert.ok(result!.items.some((i) => i.value === "medium"));
 		assert.equal(result!.items.length, 1);
-		assert.equal(result!.items[0]?.value, "medium");
 	});
 
 	it("returns null for commands without argument completions", () => {
@@ -122,6 +134,11 @@ describe("CombinedAutocompleteProvider — argument completions", () => {
 		const provider = makeProvider(commands);
 		const result = provider.getSuggestions(["/test "], 0, 6);
 		assert.ok(result);
+		// Empty prefix returns all 3 subcommands — verify each is present
+		// by name. Bare count would pass for any 3-element list.
+		assert.ok(result!.items.some((i) => i.value === "start"));
+		assert.ok(result!.items.some((i) => i.value === "stop"));
+		assert.ok(result!.items.some((i) => i.value === "status"));
 		assert.equal(result!.items.length, 3);
 	});
 });

package/packages/pi-tui/src/components/__tests__/leak-fixes-runtime.test.ts

@@ -0,0 +1,219 @@
+// Runtime regression tests for the long-running-session leak fixes.
+//
+// Replaces the source-grep file `src/tests/session-memory-leaks.test.ts`
+// (deleted in #4875, tracked as #4873). The previous tests asserted on
+// identifier presence (`_prevRender`, `_lastMessage`, `MAX_CHAT_COMPONENTS`)
+// in source — a regression that set `MAX_CHAT_COMPONENTS = Number.MAX_SAFE_INTEGER`
+// or replaced `setText` with an inline mutation would still pass.
+//
+// Each test below drives the actual component through a long-running scenario
+// and asserts on observable behaviour.
+
+import { describe, it, mock, beforeEach, afterEach } from "node:test";
+import assert from "node:assert/strict";
+import { Container } from "../../tui.js";
+import { Loader } from "../loader.js";
+import { Text } from "../text.js";
+
+interface MockTui {
+	requestRender: ReturnType<typeof mock.fn>;
+}
+
+function makeMockTUI(): MockTui {
+	return { requestRender: mock.fn() };
+}
+
+// ─── Container render-skip ──────────────────────────────────────────────
+
+describe("Container.render skips work when output is unchanged", () => {
+	it("returns the SAME array reference across two renders with no changes", () => {
+		// The container caches `_prevRender` so doRender() can short-circuit
+		// the post-processing (image-line scan, applyLineResets, line diffs).
+		// The contract for the optimization is reference equality of the
+		// returned array — that's what the consumer in tui.ts checks.
+		const c = new Container();
+		c.addChild(new Text("hello", 0, 0));
+
+		const first = c.render(20);
+		const second = c.render(20);
+		assert.strictEqual(
+			second,
+			first,
+			"Container must return the same array reference when content is unchanged " +
+				"(reference equality is the consumer-visible contract for the skip)",
+		);
+	});
+
+	it("returns a DIFFERENT array reference after addChild invalidates the cache", () => {
+		// If a regression caused `_prevRender` to never reset, downstream
+		// rendering would skip post-processing for new components — visible
+		// as missing/stale UI. Behaviourally we observe that adding a child
+		// breaks the reference-equality contract on the next render.
+		const c = new Container();
+		c.addChild(new Text("a", 0, 0));
+		const first = c.render(20);
+		c.addChild(new Text("b", 0, 0));
+		const second = c.render(20);
+		assert.notStrictEqual(
+			second,
+			first,
+			"adding a child must invalidate the cached render reference",
+		);
+	});
+
+	it("returns a different reference after content changes", () => {
+		const t = new Text("hello", 0, 0);
+		const c = new Container();
+		c.addChild(t);
+
+		const first = c.render(20);
+		t.setText("world");
+		const second = c.render(20);
+		assert.notStrictEqual(
+			second,
+			first,
+			"changing a child's text must produce a fresh array (not the cached one)",
+		);
+	});
+});
+
+// ─── Loader frame isolation ─────────────────────────────────────────────
+
+describe("Loader does not mutate Text cache on every spinner tick", () => {
+	let tui: MockTui;
+	let loader: Loader;
+
+	beforeEach(() => {
+		tui = makeMockTUI();
+		mock.timers.enable({ apis: ["setInterval"] });
+	});
+
+	afterEach(() => {
+		try {
+			loader?.stop();
+		} catch {
+			/* best-effort */
+		}
+		mock.timers.reset();
+	});
+
+	it("does not invalidate Text's render cache across N spinner ticks", () => {
+		// Loader extends Text. Text caches its rendered lines keyed by
+		// (text, width). The leak was: the old Loader called setText()
+		// on every 80ms tick, which always cleared the cache, forcing a
+		// re-wrap of the message text.
+		// Behavioural test: render the loader, capture the cached array
+		// reference returned by Text.render, advance many ticks, and
+		// assert the underlying Text cache was NOT invalidated (we observe
+		// this via reference equality of the cached lines slice in the
+		// returned array — the second `result[*]` segment from super.render
+		// stays identity-stable when the cache survives).
+		loader = new Loader(tui as never, (s) => s, (s) => s, "the-message");
+
+		const before = loader.render(40);
+		// Run 50 frame intervals — the spinner should rotate, but the
+		// underlying message text wrapping must not change.
+		mock.timers.tick(80 * 50);
+		const after = loader.render(40);
+
+		// The message portion (everything after the first `""` padding line
+		// and after the spinner glyph + space prefix on the first content
+		// line) must be byte-identical across renders.
+		assert.equal(before.length, after.length, "render shape stable across ticks");
+		// Trailing message lines (index 2 onwards) come straight from Text's
+		// cache without any spinner mutation. They must be reference-equal
+		// substrings — same string contents byte-for-byte.
+		for (let i = 2; i < before.length; i++) {
+			assert.equal(
+				after[i],
+				before[i],
+				`message line ${i} must remain byte-identical across spinner ticks ` +
+					`(cache must not be invalidated by frame rotation)`,
+			);
+		}
+	});
+
+	it("setMessage invalidates Text cache and a new message is reflected", () => {
+		// Quiescence is conditional on the message being unchanged — we
+		// must still see updates when it actually changes. This is the
+		// counter-test that prevents a false-positive "cache always stable"
+		// fix that would freeze the loader text.
+		loader = new Loader(tui as never, (s) => s, (s) => s, "first");
+		const beforeFirstRender = loader.render(40).join("\n");
+		loader.setMessage("second");
+		const afterChange = loader.render(40).join("\n");
+		assert.ok(afterChange.includes("second"), "new message must render");
+		assert.ok(!afterChange.includes("first") || beforeFirstRender !== afterChange,
+			"render output must change when message changes");
+	});
+});
+
+// ─── Text.setText early-return guard ────────────────────────────────────
+
+describe("Text.setText returns early when value is unchanged", () => {
+	it("does not invalidate the cached render when setText receives the same value", () => {
+		// The setText early-return is the runtime guard that keeps the
+		// Loader-and-Text cache stable. We observe it via reference equality
+		// of the rendered output across a setText(SAME) call.
+		const t = new Text("identical", 0, 0);
+		const first = t.render(30);
+		t.setText("identical"); // same value — must NOT clear the cache
+		const second = t.render(30);
+		assert.strictEqual(
+			second,
+			first,
+			"setText with an unchanged value must leave the render cache intact",
+		);
+	});
+
+	it("invalidates cache when the value actually changes", () => {
+		const t = new Text("one", 0, 0);
+		const first = t.render(30);
+		t.setText("two");
+		const second = t.render(30);
+		assert.notStrictEqual(
+			second,
+			first,
+			"setText with a new value must produce a fresh render result",
+		);
+	});
+});
+
+// ─── Heap growth bound (gated on --expose-gc) ───────────────────────────
+
+describe("Long-running render loop does not leak heap (forced-GC bound)", () => {
+	const hasGc = typeof (globalThis as any).gc === "function";
+
+	it(
+		"renders a Container 5000 times with stable heapUsed (within 2x baseline)",
+		{ skip: !hasGc ? "requires --expose-gc to drive deterministic GC between snapshots" : false },
+		() => {
+			// This is the on-the-wire memory-soak test. The naive version
+			// (no forced GC) is flaky because Node's GC is opportunistic;
+			// any background allocation can shift heapUsed past an absolute
+			// bound. We instead force GC between snapshots and assert a
+			// RELATIVE bound: post-iteration heap must be within 2x of the
+			// post-warmup baseline.
+			const gc = (globalThis as any).gc as () => void;
+
+			const c = new Container();
+			const t = new Text("steady-state", 0, 0);
+			c.addChild(t);
+
+			// Warm up so the JIT and any lazy initial allocations settle.
+			for (let i = 0; i < 200; i++) c.render(40);
+			gc();
+			const baseline = process.memoryUsage().heapUsed;
+
+			for (let i = 0; i < 5000; i++) c.render(40);
+			gc();
+			const after = process.memoryUsage().heapUsed;
+
+			assert.ok(
+				after < baseline * 2,
+				`heapUsed must stay within 2x baseline after 5000 renders ` +
+					`(baseline=${baseline}B, after=${after}B, ratio=${(after / baseline).toFixed(2)})`,
+			);
+		},
+	);
+});