gsd-pi 2.78.1-dev.0fdacd524 → 2.78.1-dev.2ff77b0ce

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

@@ -1 +1 @@
-{"version":3,"file":"runner.test.js","sourceRoot":"","sources":["../../../src/core/extensions/runner.test.ts"],"names":[],"mappings":"AAAA,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAQ,MAAM,WAAW,CAAC;AAC/C,OAAO,EAAE,eAAe,EAAE,MAAM,aAAa,CAAC;AAE9C,OAAO,EAAE,cAAc,EAAE,MAAM,uBAAuB,CAAC;AACvD,OAAO,EAAE,aAAa,EAAE,MAAM,sBAAsB,CAAC;AACrD,OAAO,EAAE,WAAW,EAAE,MAAM,EAAE,MAAM,SAAS,CAAC;AAC9C,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,MAAM,EAAE,MAAM,SAAS,CAAC;AACjC,OAAO,EAAE,WAAW,EAAE,MAAM,oBAAoB,CAAC;AAEjD,SAAS,kBAAkB;IAC1B,OAAO;QACN,WAAW,EAAE,KAAK,IAAI,EAAE,GAAE,CAAC;QAC3B,eAAe,EAAE,KAAK,IAAI,EAAE,GAAE,CAAC;QAC/B,WAAW,EAAE,GAAG,EAAE,GAAE,CAAC;QACrB,cAAc,EAAE,GAAG,EAAE,GAAE,CAAC;QACxB,cAAc,EAAE,GAAG,EAAE,CAAC,SAAS;QAC/B,QAAQ,EAAE,GAAG,EAAE,GAAE,CAAC;QAClB,cAAc,EAAE,GAAG,EAAE,CAAC,EAAE;QACxB,WAAW,EAAE,GAAG,EAAE,CAAC,EAAE;QACrB,cAAc,EAAE,GAAG,EAAE,GAAE,CAAC;QACxB,YAAY,EAAE,GAAG,EAAE,GAAE,CAAC;QACtB,WAAW,EAAE,GAAG,EAAE,CAAC,EAAE;QACrB,QAAQ,EAAE,KAAK,IAAI,EAAE,GAAE,CAAC;QACxB,gBAAgB,EAAE,GAAG,EAAE,CAAC,SAAS;QACjC,gBAAgB,EAAE,GAAG,EAAE,GAAE,CAAC;QAC1B,gBAAgB,EAAE,GAAG,EAAE,GAAE,CAAC;QAC1B,kBAAkB,EAAE,GAAG,EAAE,GAAE,CAAC;QAC5B,4BAA4B,EAAE,EAAE;KACD,CAAC;AAClC,CAAC;AAED,SAAS,qBAAqB,CAAC,SAAiB,EAAE,KAAY;IAC7D,MAAM,QAAQ,GAAG,IAAI,GAAG,EAAE,CAAC;IAC3B,QAAQ,CAAC,GAAG,CAAC,SAAS,EAAE;QACvB,KAAK,IAAI,EAAE;YACV,MAAM,KAAK,CAAC;QACb,CAAC;KACD,CAAC,CAAC;IACH,OAAO;QACN,IAAI,EAAE,oBAAoB;QAC1B,QAAQ;QACR,QAAQ,EAAE,EAAE;QACZ,SAAS,EAAE,EAAE;QACb,WAAW,EAAE,EAAE;KACS,CAAC;AAC3B,CAAC;AAED,QAAQ,CAAC,8BAA8B,EAAE,GAAG,EAAE;IAC7C,EAAE,CAAC,4DAA4D,EAAE,KAAK,EAAE,CAAC,EAAE,EAAE;QAC5E,MAAM,GAAG,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,cAAc,CAAC,CAAC,CAAC;QACxD,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE;YACZ,MAAM,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QAC/C,CAAC,CAAC,CAAC;QAEH,MAAM,cAAc,GAAG,cAAc,CAAC,MAAM,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC;QACvD,MAAM,WAAW,GAAG,WAAW,CAAC,MAAM,EAAE,CAAC;QACzC,MAAM,aAAa,GAAG,IAAI,aAAa,CAAC,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,aAAa,CAAC,CAAC,CAAC;QAE/E,MAAM,WAAW,GAAG,qBAAqB,CAAC,WAAW,EAAE,IAAI,KAAK,CAAC,iBAAiB,CAAC,CAAC,CAAC;QACrF,MAAM,OAAO,GAAG,kBAAkB,EAAE,CAAC;QACrC,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,CAAC,WAAW,CAAC,EAAE,OAAO,EAAE,GAAG,EAAE,cAAc,EAAE,aAAa,CAAC,CAAC;QAE/F,MAAM,MAAM,GAAU,EAAE,CAAC;QACzB,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC;QAE1C,MAAM,KAAK,GAAkB;YAC5B,IAAI,EAAE,WAAW;YACjB,UAAU,EAAE,UAAU;YACtB,QAAQ,EAAE,WAAW;YACrB,KAAK,EAAE,EAAE;SACQ,CAAC;QAEnB,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC;QAEhD,6DAA6D;QAC7D,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;QAChC,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QAC/B,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,EAAE,iBAAiB,CAAC,CAAC;QACjD,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,EAAE,WAAW,CAAC,CAAC;QAC3C,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,aAAa,EAAE,oBAAoB,CAAC,CAAC;IAC7D,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC","sourcesContent":["import assert from \"node:assert/strict\";\nimport { describe, it, mock } from \"node:test\";\nimport { ExtensionRunner } from \"./runner.js\";\nimport type { Extension, ExtensionRuntime, ToolCallEvent } from \"./index.js\";\nimport { SessionManager } from \"../session-manager.js\";\nimport { ModelRegistry } from \"../model-registry.js\";\nimport { mkdtempSync, rmSync } from \"node:fs\";\nimport { join } from \"node:path\";\nimport { tmpdir } from \"node:os\";\nimport { AuthStorage } from \"../auth-storage.js\";\n\nfunction makeMinimalRuntime(): ExtensionRuntime {\n\treturn {\n\t\tsendMessage: async () => {},\n\t\tsendUserMessage: async () => {},\n\t\tappendEntry: () => {},\n\t\tsetSessionName: () => {},\n\t\tgetSessionName: () => undefined,\n\t\tsetLabel: () => {},\n\t\tgetActiveTools: () => [],\n\t\tgetAllTools: () => [],\n\t\tsetActiveTools: () => {},\n\t\trefreshTools: () => {},\n\t\tgetCommands: () => [],\n\t\tsetModel: async () => {},\n\t\tgetThinkingLevel: () => undefined,\n\t\tsetThinkingLevel: () => {},\n\t\tregisterProvider: () => {},\n\t\tunregisterProvider: () => {},\n\t\tpendingProviderRegistrations: [],\n\t} as unknown as ExtensionRuntime;\n}\n\nfunction makeThrowingExtension(eventType: string, error: Error): Extension {\n\tconst handlers = new Map();\n\thandlers.set(eventType, [\n\t\tasync () => {\n\t\t\tthrow error;\n\t\t},\n\t]);\n\treturn {\n\t\tpath: \"/test/throwing-ext\",\n\t\thandlers,\n\t\tcommands: [],\n\t\tshortcuts: [],\n\t\tdiagnostics: [],\n\t} as unknown as Extension;\n}\n\ndescribe(\"ExtensionRunner.emitToolCall\", () => {\n\tit(\"catches throwing extension handler and routes to emitError\", async (t) => {\n\t\tconst dir = mkdtempSync(join(tmpdir(), \"runner-test-\"));\n\t\tt.after(() => {\n\t\t\trmSync(dir, { recursive: true, force: true });\n\t\t});\n\n\t\tconst sessionManager = SessionManager.create(dir, dir);\n\t\tconst authStorage = AuthStorage.create();\n\t\tconst modelRegistry = new ModelRegistry(authStorage, join(dir, \"models.json\"));\n\n\t\tconst throwingExt = makeThrowingExtension(\"tool_call\", new Error(\"handler crashed\"));\n\t\tconst runtime = makeMinimalRuntime();\n\t\tconst runner = new ExtensionRunner([throwingExt], runtime, dir, sessionManager, modelRegistry);\n\n\t\tconst errors: any[] = [];\n\t\trunner.onError((err) => errors.push(err));\n\n\t\tconst event: ToolCallEvent = {\n\t\t\ttype: \"tool_call\",\n\t\t\ttoolCallId: \"test-123\",\n\t\t\ttoolName: \"test_tool\",\n\t\t\tinput: {},\n\t\t} as ToolCallEvent;\n\n\t\tconst result = await runner.emitToolCall(event);\n\n\t\t// Should not throw — error is caught and routed to emitError\n\t\tassert.equal(result, undefined);\n\t\tassert.equal(errors.length, 1);\n\t\tassert.equal(errors[0].error, \"handler crashed\");\n\t\tassert.equal(errors[0].event, \"tool_call\");\n\t\tassert.equal(errors[0].extensionPath, \"/test/throwing-ext\");\n\t});\n});\n"]}
+{"version":3,"file":"runner.test.js","sourceRoot":"","sources":["../../../src/core/extensions/runner.test.ts"],"names":[],"mappings":"AAAA,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAQ,MAAM,WAAW,CAAC;AAC/C,OAAO,EAAE,eAAe,EAAE,MAAM,aAAa,CAAC;AAE9C,OAAO,EAAE,cAAc,EAAE,MAAM,uBAAuB,CAAC;AACvD,OAAO,EAAE,aAAa,EAAE,MAAM,sBAAsB,CAAC;AACrD,OAAO,EAAE,SAAS,EAAE,WAAW,EAAE,YAAY,EAAE,MAAM,EAAE,MAAM,SAAS,CAAC;AACvE,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,MAAM,EAAE,MAAM,SAAS,CAAC;AACjC,OAAO,EAAE,WAAW,EAAE,MAAM,oBAAoB,CAAC;AAEjD,SAAS,kBAAkB;IAC1B,OAAO;QACN,WAAW,EAAE,KAAK,IAAI,EAAE,GAAE,CAAC;QAC3B,eAAe,EAAE,KAAK,IAAI,EAAE,GAAE,CAAC;QAC/B,WAAW,EAAE,GAAG,EAAE,GAAE,CAAC;QACrB,cAAc,EAAE,GAAG,EAAE,GAAE,CAAC;QACxB,cAAc,EAAE,GAAG,EAAE,CAAC,SAAS;QAC/B,QAAQ,EAAE,GAAG,EAAE,GAAE,CAAC;QAClB,cAAc,EAAE,GAAG,EAAE,CAAC,EAAE;QACxB,WAAW,EAAE,GAAG,EAAE,CAAC,EAAE;QACrB,cAAc,EAAE,GAAG,EAAE,GAAE,CAAC;QACxB,YAAY,EAAE,GAAG,EAAE,GAAE,CAAC;QACtB,WAAW,EAAE,GAAG,EAAE,CAAC,EAAE;QACrB,QAAQ,EAAE,KAAK,IAAI,EAAE,GAAE,CAAC;QACxB,gBAAgB,EAAE,GAAG,EAAE,CAAC,SAAS;QACjC,gBAAgB,EAAE,GAAG,EAAE,GAAE,CAAC;QAC1B,gBAAgB,EAAE,GAAG,EAAE,GAAE,CAAC;QAC1B,kBAAkB,EAAE,GAAG,EAAE,GAAE,CAAC;QAC5B,4BAA4B,EAAE,EAAE;KACD,CAAC;AAClC,CAAC;AAED,SAAS,qBAAqB,CAAC,SAAiB,EAAE,KAAY;IAC7D,MAAM,QAAQ,GAAG,IAAI,GAAG,EAAE,CAAC;IAC3B,QAAQ,CAAC,GAAG,CAAC,SAAS,EAAE;QACvB,KAAK,IAAI,EAAE;YACV,MAAM,KAAK,CAAC;QACb,CAAC;KACD,CAAC,CAAC;IACH,OAAO;QACN,IAAI,EAAE,oBAAoB;QAC1B,QAAQ;QACR,QAAQ,EAAE,EAAE;QACZ,SAAS,EAAE,EAAE;QACb,WAAW,EAAE,EAAE;KACS,CAAC;AAC3B,CAAC;AAED,SAAS,oBAAoB,CAAC,IAAY,EAAE,WAAmB,EAAE,MAAc;IAC9E,OAAO;QACN,IAAI;QACJ,QAAQ,EAAE,IAAI,GAAG,CAAC;YACjB;gBACC,WAAW;gBACX;oBACC,IAAI,EAAE,WAAW;oBACjB,WAAW,EAAE,MAAM;oBACnB,OAAO,EAAE,KAAK,IAAI,EAAE,GAAE,CAAC;iBACvB;aACD;SACD,CAAC;QACF,QAAQ,EAAE,IAAI,GAAG,EAAE;QACnB,SAAS,EAAE,IAAI,GAAG,EAAE;QACpB,KAAK,EAAE,IAAI,GAAG,EAAE;QAChB,KAAK,EAAE,IAAI,GAAG,EAAE;QAChB,WAAW,EAAE,EAAE;KACS,CAAC;AAC3B,CAAC;AAED,QAAQ,CAAC,8BAA8B,EAAE,GAAG,EAAE;IAC7C,EAAE,CAAC,4DAA4D,EAAE,KAAK,EAAE,CAAC,EAAE,EAAE;QAC5E,MAAM,GAAG,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,cAAc,CAAC,CAAC,CAAC;QACxD,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE;YACZ,MAAM,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QAC/C,CAAC,CAAC,CAAC;QAEH,MAAM,cAAc,GAAG,cAAc,CAAC,MAAM,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC;QACvD,MAAM,WAAW,GAAG,WAAW,CAAC,MAAM,EAAE,CAAC;QACzC,MAAM,aAAa,GAAG,IAAI,aAAa,CAAC,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,aAAa,CAAC,CAAC,CAAC;QAE/E,MAAM,WAAW,GAAG,qBAAqB,CAAC,WAAW,EAAE,IAAI,KAAK,CAAC,iBAAiB,CAAC,CAAC,CAAC;QACrF,MAAM,OAAO,GAAG,kBAAkB,EAAE,CAAC;QACrC,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,CAAC,WAAW,CAAC,EAAE,OAAO,EAAE,GAAG,EAAE,cAAc,EAAE,aAAa,CAAC,CAAC;QAE/F,MAAM,MAAM,GAAU,EAAE,CAAC;QACzB,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC;QAE1C,MAAM,KAAK,GAAkB;YAC5B,IAAI,EAAE,WAAW;YACjB,UAAU,EAAE,UAAU;YACtB,QAAQ,EAAE,WAAW;YACrB,KAAK,EAAE,EAAE;SACQ,CAAC;QAEnB,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC;QAEhD,6DAA6D;QAC7D,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;QAChC,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QAC/B,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,EAAE,iBAAiB,CAAC,CAAC;QACjD,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,EAAE,WAAW,CAAC,CAAC;QAC3C,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,aAAa,EAAE,oBAAoB,CAAC,CAAC;IAC7D,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,+BAA+B,EAAE,GAAG,EAAE;IAC9C,EAAE,CAAC,0DAA0D,EAAE,CAAC,CAAC,EAAE,EAAE;QACpE,MAAM,WAAW,GAAG,OAAO,CAAC,GAAG,EAAE,CAAC;QAClC,MAAM,GAAG,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,cAAc,CAAC,CAAC,CAAC;QACxD,MAAM,UAAU,GAAG,IAAI,CAAC,GAAG,EAAE,SAAS,CAAC,CAAC;QACxC,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE;YACZ,OAAO,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC;YAC3B,MAAM,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QAC/C,CAAC,CAAC,CAAC;QAEH,MAAM,cAAc,GAAG,cAAc,CAAC,MAAM,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC;QACvD,MAAM,WAAW,GAAG,WAAW,CAAC,MAAM,EAAE,CAAC;QACzC,MAAM,aAAa,GAAG,IAAI,aAAa,CAAC,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,aAAa,CAAC,CAAC,CAAC;QAC/E,MAAM,OAAO,GAAG,kBAAkB,EAAE,CAAC;QACrC,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,EAAE,EAAE,OAAO,EAAE,WAAW,EAAE,cAAc,EAAE,aAAa,CAAC,CAAC;QAE5F,SAAS,CAAC,UAAU,CAAC,CAAC;QACtB,MAAM,cAAc,GAAG,YAAY,CAAC,UAAU,CAAC,CAAC;QAChD,OAAO,CAAC,KAAK,CAAC,cAAc,CAAC,CAAC;QAE9B,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,aAAa,EAAE,CAAC,GAAG,EAAE,cAAc,CAAC,CAAC;QACzD,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,oBAAoB,EAAE,CAAC,GAAG,EAAE,cAAc,CAAC,CAAC;IACjE,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,oCAAoC,EAAE,GAAG,EAAE;IACnD,EAAE,CAAC,oFAAoF,EAAE,GAAG,EAAE;QAC7F,MAAM,GAAG,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,cAAc,CAAC,CAAC,CAAC;QACxD,IAAI,CAAC;YACJ,MAAM,cAAc,GAAG,cAAc,CAAC,MAAM,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC;YACvD,MAAM,WAAW,GAAG,WAAW,CAAC,MAAM,EAAE,CAAC;YACzC,MAAM,aAAa,GAAG,IAAI,aAAa,CAAC,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,aAAa,CAAC,CAAC,CAAC;YAC/E,MAAM,OAAO,GAAG,kBAAkB,EAAE,CAAC;YACrC,MAAM,OAAO,GAAG,oBAAoB,CAAC,qCAAqC,EAAE,KAAK,EAAE,OAAO,CAAC,CAAC;YAC5F,MAAM,MAAM,GAAG,oBAAoB,CAAC,GAAG,GAAG,0BAA0B,EAAE,KAAK,EAAE,SAAS,CAAC,CAAC;YACxF,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,CAAC,OAAO,EAAE,MAAM,CAAC,EAAE,OAAO,EAAE,GAAG,EAAE,cAAc,EAAE,aAAa,CAAC,CAAC;YAEnG,MAAM,OAAO,GAAG,MAAM,CAAC,UAAU,CAAC,KAAK,CAAC,CAAC;YACzC,MAAM,CAAC,KAAK,CAAC,OAAO,EAAE,WAAW,EAAE,SAAS,CAAC,CAAC;QAC/C,CAAC;gBAAS,CAAC;YACV,MAAM,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QAC/C,CAAC;IACF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uDAAuD,EAAE,GAAG,EAAE;QAChE,MAAM,GAAG,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,cAAc,CAAC,CAAC,CAAC;QACxD,IAAI,CAAC;YACJ,MAAM,cAAc,GAAG,cAAc,CAAC,MAAM,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC;YACvD,MAAM,WAAW,GAAG,WAAW,CAAC,MAAM,EAAE,CAAC;YACzC,MAAM,aAAa,GAAG,IAAI,aAAa,CAAC,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,aAAa,CAAC,CAAC,CAAC;YAC/E,MAAM,OAAO,GAAG,kBAAkB,EAAE,CAAC;YACrC,MAAM,OAAO,GAAG,oBAAoB,CAAC,qCAAqC,EAAE,KAAK,EAAE,OAAO,CAAC,CAAC;YAC5F,MAAM,MAAM,GAAG,oBAAoB,CAAC,GAAG,GAAG,0BAA0B,EAAE,KAAK,EAAE,SAAS,CAAC,CAAC;YACxF,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,CAAC,OAAO,EAAE,MAAM,CAAC,EAAE,OAAO,EAAE,GAAG,EAAE,cAAc,EAAE,aAAa,CAAC,CAAC;YAEnG,MAAM,QAAQ,GAAG,MAAM,CAAC,qBAAqB,EAAE,CAAC;YAChD,MAAM,CAAC,SAAS,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,WAAW,CAAC,EAAE,CAAC,SAAS,CAAC,CAAC,CAAC;YAC9E,MAAM,CAAC,EAAE,CACR,MAAM,CAAC,qBAAqB,EAAE,CAAC,IAAI,CAAC,CAAC,UAAU,EAAE,EAAE,CAAC,UAAU,CAAC,OAAO,CAAC,QAAQ,CAAC,yBAAyB,CAAC,CAAC,EAC3G,0CAA0C,CAC1C,CAAC;QACH,CAAC;gBAAS,CAAC;YACV,MAAM,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QAC/C,CAAC;IACF,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC","sourcesContent":["import assert from \"node:assert/strict\";\nimport { describe, it, mock } from \"node:test\";\nimport { ExtensionRunner } from \"./runner.js\";\nimport type { Extension, ExtensionRuntime, ToolCallEvent } from \"./index.js\";\nimport { SessionManager } from \"../session-manager.js\";\nimport { ModelRegistry } from \"../model-registry.js\";\nimport { mkdirSync, mkdtempSync, realpathSync, rmSync } from \"node:fs\";\nimport { join } from \"node:path\";\nimport { tmpdir } from \"node:os\";\nimport { AuthStorage } from \"../auth-storage.js\";\n\nfunction makeMinimalRuntime(): ExtensionRuntime {\n\treturn {\n\t\tsendMessage: async () => {},\n\t\tsendUserMessage: async () => {},\n\t\tappendEntry: () => {},\n\t\tsetSessionName: () => {},\n\t\tgetSessionName: () => undefined,\n\t\tsetLabel: () => {},\n\t\tgetActiveTools: () => [],\n\t\tgetAllTools: () => [],\n\t\tsetActiveTools: () => {},\n\t\trefreshTools: () => {},\n\t\tgetCommands: () => [],\n\t\tsetModel: async () => {},\n\t\tgetThinkingLevel: () => undefined,\n\t\tsetThinkingLevel: () => {},\n\t\tregisterProvider: () => {},\n\t\tunregisterProvider: () => {},\n\t\tpendingProviderRegistrations: [],\n\t} as unknown as ExtensionRuntime;\n}\n\nfunction makeThrowingExtension(eventType: string, error: Error): Extension {\n\tconst handlers = new Map();\n\thandlers.set(eventType, [\n\t\tasync () => {\n\t\t\tthrow error;\n\t\t},\n\t]);\n\treturn {\n\t\tpath: \"/test/throwing-ext\",\n\t\thandlers,\n\t\tcommands: [],\n\t\tshortcuts: [],\n\t\tdiagnostics: [],\n\t} as unknown as Extension;\n}\n\nfunction makeCommandExtension(path: string, commandName: string, marker: string): Extension {\n\treturn {\n\t\tpath,\n\t\tcommands: new Map([\n\t\t\t[\n\t\t\t\tcommandName,\n\t\t\t\t{\n\t\t\t\t\tname: commandName,\n\t\t\t\t\tdescription: marker,\n\t\t\t\t\thandler: async () => {},\n\t\t\t\t},\n\t\t\t],\n\t\t]),\n\t\thandlers: new Map(),\n\t\tshortcuts: new Map(),\n\t\ttools: new Map(),\n\t\tflags: new Map(),\n\t\tdiagnostics: [],\n\t} as unknown as Extension;\n}\n\ndescribe(\"ExtensionRunner.emitToolCall\", () => {\n\tit(\"catches throwing extension handler and routes to emitError\", async (t) => {\n\t\tconst dir = mkdtempSync(join(tmpdir(), \"runner-test-\"));\n\t\tt.after(() => {\n\t\t\trmSync(dir, { recursive: true, force: true });\n\t\t});\n\n\t\tconst sessionManager = SessionManager.create(dir, dir);\n\t\tconst authStorage = AuthStorage.create();\n\t\tconst modelRegistry = new ModelRegistry(authStorage, join(dir, \"models.json\"));\n\n\t\tconst throwingExt = makeThrowingExtension(\"tool_call\", new Error(\"handler crashed\"));\n\t\tconst runtime = makeMinimalRuntime();\n\t\tconst runner = new ExtensionRunner([throwingExt], runtime, dir, sessionManager, modelRegistry);\n\n\t\tconst errors: any[] = [];\n\t\trunner.onError((err) => errors.push(err));\n\n\t\tconst event: ToolCallEvent = {\n\t\t\ttype: \"tool_call\",\n\t\t\ttoolCallId: \"test-123\",\n\t\t\ttoolName: \"test_tool\",\n\t\t\tinput: {},\n\t\t} as ToolCallEvent;\n\n\t\tconst result = await runner.emitToolCall(event);\n\n\t\t// Should not throw — error is caught and routed to emitError\n\t\tassert.equal(result, undefined);\n\t\tassert.equal(errors.length, 1);\n\t\tassert.equal(errors[0].error, \"handler crashed\");\n\t\tassert.equal(errors[0].event, \"tool_call\");\n\t\tassert.equal(errors[0].extensionPath, \"/test/throwing-ext\");\n\t});\n});\n\ndescribe(\"ExtensionRunner.createContext\", () => {\n\tit(\"uses the live process cwd instead of the constructor cwd\", (t) => {\n\t\tconst originalCwd = process.cwd();\n\t\tconst dir = mkdtempSync(join(tmpdir(), \"runner-test-\"));\n\t\tconst projectDir = join(dir, \"project\");\n\t\tt.after(() => {\n\t\t\tprocess.chdir(originalCwd);\n\t\t\trmSync(dir, { recursive: true, force: true });\n\t\t});\n\n\t\tconst sessionManager = SessionManager.create(dir, dir);\n\t\tconst authStorage = AuthStorage.create();\n\t\tconst modelRegistry = new ModelRegistry(authStorage, join(dir, \"models.json\"));\n\t\tconst runtime = makeMinimalRuntime();\n\t\tconst runner = new ExtensionRunner([], runtime, originalCwd, sessionManager, modelRegistry);\n\n\t\tmkdirSync(projectDir);\n\t\tconst realProjectDir = realpathSync(projectDir);\n\t\tprocess.chdir(realProjectDir);\n\n\t\tassert.equal(runner.createContext().cwd, realProjectDir);\n\t\tassert.equal(runner.createCommandContext().cwd, realProjectDir);\n\t});\n});\n\ndescribe(\"ExtensionRunner protected commands\", () => {\n\tit(\"resolves /gsd to the bundled GSD extension even when another extension loads first\", () => {\n\t\tconst dir = mkdtempSync(join(tmpdir(), \"runner-test-\"));\n\t\ttry {\n\t\t\tconst sessionManager = SessionManager.create(dir, dir);\n\t\t\tconst authStorage = AuthStorage.create();\n\t\t\tconst modelRegistry = new ModelRegistry(authStorage, join(dir, \"models.json\"));\n\t\t\tconst runtime = makeMinimalRuntime();\n\t\t\tconst userExt = makeCommandExtension(\"/tmp/extensions/user-spoof/index.ts\", \"gsd\", \"spoof\");\n\t\t\tconst gsdExt = makeCommandExtension(`${dir}/extensions/gsd/index.ts`, \"gsd\", \"bundled\");\n\t\t\tconst runner = new ExtensionRunner([userExt, gsdExt], runtime, dir, sessionManager, modelRegistry);\n\n\t\t\tconst command = runner.getCommand(\"gsd\");\n\t\t\tassert.equal(command?.description, \"bundled\");\n\t\t} finally {\n\t\t\trmSync(dir, { recursive: true, force: true });\n\t\t}\n\t});\n\n\tit(\"omits spoofed /gsd from registered extension commands\", () => {\n\t\tconst dir = mkdtempSync(join(tmpdir(), \"runner-test-\"));\n\t\ttry {\n\t\t\tconst sessionManager = SessionManager.create(dir, dir);\n\t\t\tconst authStorage = AuthStorage.create();\n\t\t\tconst modelRegistry = new ModelRegistry(authStorage, join(dir, \"models.json\"));\n\t\t\tconst runtime = makeMinimalRuntime();\n\t\t\tconst userExt = makeCommandExtension(\"/tmp/extensions/user-spoof/index.ts\", \"gsd\", \"spoof\");\n\t\t\tconst gsdExt = makeCommandExtension(`${dir}/extensions/gsd/index.ts`, \"gsd\", \"bundled\");\n\t\t\tconst runner = new ExtensionRunner([userExt, gsdExt], runtime, dir, sessionManager, modelRegistry);\n\n\t\t\tconst commands = runner.getRegisteredCommands();\n\t\t\tassert.deepEqual(commands.map((command) => command.description), [\"bundled\"]);\n\t\t\tassert.ok(\n\t\t\t\trunner.getCommandDiagnostics().some((diagnostic) => diagnostic.message.includes(\"protected command owner\")),\n\t\t\t\t\"spoofed /gsd conflict should be reported\",\n\t\t\t);\n\t\t} finally {\n\t\t\trmSync(dir, { recursive: true, force: true });\n\t\t}\n\t});\n});\n"]}

package/packages/pi-coding-agent/dist/core/tools/bash-spawn-windows.test.js

@@ -21,14 +21,32 @@ import { spawn } from "node:child_process";
 // Verify the spawn option pattern used across the codebase.
 // This is a static/structural test — it reads the source files and asserts
 // they use the platform-guarded detached flag.
-import { readFileSync } from "node:fs";
-import { join, dirname } from "node:path";
+import { existsSync, readFileSync } from "node:fs";
+import { dirname, join, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
 const __dirname = dirname(fileURLToPath(import.meta.url));
+function findRepoRoot() {
+    const requiredFiles = [
+        "packages/pi-coding-agent/src/core/tools/bash.ts",
+        "packages/pi-coding-agent/src/core/bash-executor.ts",
+        "packages/pi-coding-agent/src/utils/shell.ts",
+    ];
+    const candidates = [
+        resolve(__dirname, "../../../../../"),
+        resolve(__dirname, "../../../../../../"),
+    ];
+    for (const candidate of candidates) {
+        if (requiredFiles.every((file) => existsSync(join(candidate, file)))) {
+            return candidate;
+        }
+    }
+    throw new Error(`Unable to resolve repository root from ${__dirname}`);
+}
+const REPO_ROOT = findRepoRoot();
 const SPAWN_FILES = [
-    join(__dirname, "bash.ts"),
-    join(__dirname, "..", "bash-executor.ts"),
-    join(__dirname, "..", "..", "utils", "shell.ts"),
+    join(REPO_ROOT, "packages/pi-coding-agent/src/core/tools/bash.ts"),
+    join(REPO_ROOT, "packages/pi-coding-agent/src/core/bash-executor.ts"),
+    join(REPO_ROOT, "packages/pi-coding-agent/src/utils/shell.ts"),
 ];
 test("spawn calls use platform-guarded detached flag (no unconditional detached: true)", () => {
     for (const file of SPAWN_FILES) {
@@ -49,7 +67,7 @@ test("spawn calls use platform-guarded detached flag (no unconditional detached:
     }
 });
 test("killProcessTree does not use detached: true for taskkill on Windows", () => {
-    const shellFile = join(__dirname, "..", "..", "utils", "shell.ts");
+    const shellFile = join(REPO_ROOT, "packages/pi-coding-agent/src/utils/shell.ts");
     const content = readFileSync(shellFile, "utf-8");
     // Find the taskkill spawn call and ensure it doesn't have detached: true
     const taskkillRegion = content.match(/spawn\("taskkill"[\s\S]*?\}\)/);

package/packages/pi-coding-agent/dist/core/tools/bash-spawn-windows.test.js.map

@@ -1 +1 @@
-{"version":3,"file":"bash-spawn-windows.test.js","sourceRoot":"","sources":["../../../src/core/tools/bash-spawn-windows.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAEH,OAAO,IAAI,MAAM,WAAW,CAAC;AAC7B,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,EAAE,KAAK,EAAE,MAAM,oBAAoB,CAAC;AAE3C,4DAA4D;AAC5D,2EAA2E;AAC3E,+CAA+C;AAC/C,OAAO,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AACvC,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAC1C,OAAO,EAAE,aAAa,EAAE,MAAM,UAAU,CAAC;AAEzC,MAAM,SAAS,GAAG,OAAO,CAAC,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC;AAE1D,MAAM,WAAW,GAAG;IACnB,IAAI,CAAC,SAAS,EAAE,SAAS,CAAC;IAC1B,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,kBAAkB,CAAC;IACzC,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,IAAI,EAAE,OAAO,EAAE,UAAU,CAAC;CAChD,CAAC;AAEF,IAAI,CAAC,kFAAkF,EAAE,GAAG,EAAE;IAC7F,KAAK,MAAM,IAAI,IAAI,WAAW,EAAE,CAAC;QAChC,MAAM,OAAO,GAAG,YAAY,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC;QAC5C,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QAElC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;YACvC,MAAM,IAAI,GAAG,KAAK,CAAC,CAAC,CAAE,CAAC;YACvB,gBAAgB;YAChB,IAAI,IAAI,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,IAAI,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC;gBAAE,SAAS;YAC1E,2CAA2C;YAC3C,IAAI,oBAAoB,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;gBACrC,MAAM,CAAC,IAAI,CACV,GAAG,IAAI,IAAI,CAAC,GAAG,CAAC,wCAAwC;oBACxD,oDAAoD;oBACpD,yDAAyD,CACzD,CAAC;YACH,CAAC;QACF,CAAC;IACF,CAAC;AACF,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,qEAAqE,EAAE,GAAG,EAAE;IAChF,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,IAAI,EAAE,OAAO,EAAE,UAAU,CAAC,CAAC;IACnE,MAAM,OAAO,GAAG,YAAY,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC;IAEjD,yEAAyE;IACzE,MAAM,cAAc,GAAG,OAAO,CAAC,KAAK,CAAC,+BAA+B,CAAC,CAAC;IACtE,IAAI,cAAc,EAAE,CAAC;QACpB,MAAM,CAAC,EAAE,CACR,CAAC,kBAAkB,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC,CAAC,CAAC,EAC3C,iDAAiD;YACjD,yEAAyE,CACzE,CAAC;IACH,CAAC;AACF,CAAC,CAAC,CAAC;AAEH,uEAAuE;AACvE,IAAI,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;IAC7E,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,aAAa,EAAQ,CAAC;IAEnE,MAAM,KAAK,GAAG,KAAK,CAClB,OAAO,CAAC,QAAQ,KAAK,OAAO,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,EAC3C,OAAO,CAAC,QAAQ,KAAK,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,SAAS,CAAC,EACpE;QACC,QAAQ,EAAE,OAAO,CAAC,QAAQ,KAAK,OAAO;QACtC,KAAK,EAAE,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,CAAC;KACjC,CACD,CAAC;IAEF,IAAI,MAAM,GAAG,EAAE,CAAC;IAChB,KAAK,CAAC,MAAM,EAAE,EAAE,CAAC,MAAM,EAAE,CAAC,CAAS,EAAE,EAAE,GAAG,MAAM,IAAI,CAAC,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;IACrE,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;IAC1B,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,CAAC,IAAI,EAAE,EAAE;QAC1B,IAAI,CAAC;YACJ,MAAM,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,EAAE,sBAAsB,CAAC,CAAC;YAC9C,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,IAAI,EAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE,iCAAiC,MAAM,EAAE,CAAC,CAAC;YACnF,OAAO,EAAE,CAAC;QACX,CAAC;QAAC,OAAO,CAAC,EAAE,CAAC;YACZ,MAAM,CAAC,CAAC,CAAC,CAAC;QACX,CAAC;IACF,CAAC,CAAC,CAAC;IAEH,MAAM,OAAO,CAAC;AACf,CAAC,CAAC,CAAC","sourcesContent":["/**\n * bash-spawn-windows.test.ts — Regression test for Windows spawn EINVAL.\n *\n * Verifies that bash tool spawn options disable `detached: true` on Windows\n * to prevent EINVAL errors in ConPTY / VSCode terminal contexts.\n *\n * Background:\n *   On Windows, `spawn()` with `detached: true` sets the\n *   CREATE_NEW_PROCESS_GROUP flag in CreateProcess.  In certain terminal\n *   contexts (VSCode integrated terminal, ConPTY, Windows Terminal) this\n *   flag conflicts with the parent process group and causes a synchronous\n *   EINVAL from libuv.  The bg-shell extension already guards against this\n *   with `detached: process.platform !== \"win32\"` (process-manager.ts);\n *   this test ensures all other spawn sites are aligned.\n *\n * See: gsd-build/gsd-2#XXXX\n */\n\nimport test from \"node:test\";\nimport assert from \"node:assert/strict\";\nimport { spawn } from \"node:child_process\";\n\n// Verify the spawn option pattern used across the codebase.\n// This is a static/structural test — it reads the source files and asserts\n// they use the platform-guarded detached flag.\nimport { readFileSync } from \"node:fs\";\nimport { join, dirname } from \"node:path\";\nimport { fileURLToPath } from \"node:url\";\n\nconst __dirname = dirname(fileURLToPath(import.meta.url));\n\nconst SPAWN_FILES = [\n\tjoin(__dirname, \"bash.ts\"),\n\tjoin(__dirname, \"..\", \"bash-executor.ts\"),\n\tjoin(__dirname, \"..\", \"..\", \"utils\", \"shell.ts\"),\n];\n\ntest(\"spawn calls use platform-guarded detached flag (no unconditional detached: true)\", () => {\n\tfor (const file of SPAWN_FILES) {\n\t\tconst content = readFileSync(file, \"utf-8\");\n\t\tconst lines = content.split(\"\\n\");\n\n\t\tfor (let i = 0; i < lines.length; i++) {\n\t\t\tconst line = lines[i]!;\n\t\t\t// Skip comments\n\t\t\tif (line.trim().startsWith(\"//\") || line.trim().startsWith(\"*\")) continue;\n\t\t\t// Check for unconditional `detached: true`\n\t\t\tif (/detached:\\s*true\\b/.test(line)) {\n\t\t\t\tassert.fail(\n\t\t\t\t\t`${file}:${i + 1} has unconditional 'detached: true' — ` +\n\t\t\t\t\t`must use 'detached: process.platform !== \"win32\"' ` +\n\t\t\t\t\t`to prevent EINVAL on Windows (ConPTY / VSCode terminal)`,\n\t\t\t\t);\n\t\t\t}\n\t\t}\n\t}\n});\n\ntest(\"killProcessTree does not use detached: true for taskkill on Windows\", () => {\n\tconst shellFile = join(__dirname, \"..\", \"..\", \"utils\", \"shell.ts\");\n\tconst content = readFileSync(shellFile, \"utf-8\");\n\n\t// Find the taskkill spawn call and ensure it doesn't have detached: true\n\tconst taskkillRegion = content.match(/spawn\\(\"taskkill\"[\\s\\S]*?\\}\\)/);\n\tif (taskkillRegion) {\n\t\tassert.ok(\n\t\t\t!/detached:\\s*true/.test(taskkillRegion[0]),\n\t\t\t\"taskkill spawn should not use detached: true — \" +\n\t\t\t\"it can cause EINVAL on Windows and is unnecessary for a utility process\",\n\t\t);\n\t}\n});\n\n// Smoke test: spawn with platform-guarded detached flag actually works\ntest(\"spawn with detached: process.platform !== 'win32' succeeds\", async () => {\n\tconst { promise, resolve, reject } = Promise.withResolvers<void>();\n\n\tconst child = spawn(\n\t\tprocess.platform === \"win32\" ? \"cmd\" : \"sh\",\n\t\tprocess.platform === \"win32\" ? [\"/c\", \"echo ok\"] : [\"-c\", \"echo ok\"],\n\t\t{\n\t\t\tdetached: process.platform !== \"win32\",\n\t\t\tstdio: [\"ignore\", \"pipe\", \"pipe\"],\n\t\t},\n\t);\n\n\tlet output = \"\";\n\tchild.stdout?.on(\"data\", (d: Buffer) => { output += d.toString(); });\n\tchild.on(\"error\", reject);\n\tchild.on(\"close\", (code) => {\n\t\ttry {\n\t\t\tassert.equal(code, 0, \"spawn should succeed\");\n\t\t\tassert.ok(output.trim().includes(\"ok\"), `Expected 'ok' in output, got: ${output}`);\n\t\t\tresolve();\n\t\t} catch (e) {\n\t\t\treject(e);\n\t\t}\n\t});\n\n\tawait promise;\n});\n"]}
+{"version":3,"file":"bash-spawn-windows.test.js","sourceRoot":"","sources":["../../../src/core/tools/bash-spawn-windows.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAEH,OAAO,IAAI,MAAM,WAAW,CAAC;AAC7B,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,EAAE,KAAK,EAAE,MAAM,oBAAoB,CAAC;AAE3C,4DAA4D;AAC5D,2EAA2E;AAC3E,+CAA+C;AAC/C,OAAO,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AACnD,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AACnD,OAAO,EAAE,aAAa,EAAE,MAAM,UAAU,CAAC;AAEzC,MAAM,SAAS,GAAG,OAAO,CAAC,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC;AAE1D,SAAS,YAAY;IACpB,MAAM,aAAa,GAAG;QACrB,iDAAiD;QACjD,oDAAoD;QACpD,6CAA6C;KAC7C,CAAC;IACF,MAAM,UAAU,GAAG;QAClB,OAAO,CAAC,SAAS,EAAE,iBAAiB,CAAC;QACrC,OAAO,CAAC,SAAS,EAAE,oBAAoB,CAAC;KACxC,CAAC;IACF,KAAK,MAAM,SAAS,IAAI,UAAU,EAAE,CAAC;QACpC,IAAI,aAAa,CAAC,KAAK,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,SAAS,EAAE,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC;YACtE,OAAO,SAAS,CAAC;QAClB,CAAC;IACF,CAAC;IACD,MAAM,IAAI,KAAK,CAAC,0CAA0C,SAAS,EAAE,CAAC,CAAC;AACxE,CAAC;AAED,MAAM,SAAS,GAAG,YAAY,EAAE,CAAC;AAEjC,MAAM,WAAW,GAAG;IACnB,IAAI,CAAC,SAAS,EAAE,iDAAiD,CAAC;IAClE,IAAI,CAAC,SAAS,EAAE,oDAAoD,CAAC;IACrE,IAAI,CAAC,SAAS,EAAE,6CAA6C,CAAC;CAC9D,CAAC;AAEF,IAAI,CAAC,kFAAkF,EAAE,GAAG,EAAE;IAC7F,KAAK,MAAM,IAAI,IAAI,WAAW,EAAE,CAAC;QAChC,MAAM,OAAO,GAAG,YAAY,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC;QAC5C,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QAElC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;YACvC,MAAM,IAAI,GAAG,KAAK,CAAC,CAAC,CAAE,CAAC;YACvB,gBAAgB;YAChB,IAAI,IAAI,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,IAAI,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC;gBAAE,SAAS;YAC1E,2CAA2C;YAC3C,IAAI,oBAAoB,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;gBACrC,MAAM,CAAC,IAAI,CACV,GAAG,IAAI,IAAI,CAAC,GAAG,CAAC,wCAAwC;oBACxD,oDAAoD;oBACpD,yDAAyD,CACzD,CAAC;YACH,CAAC;QACF,CAAC;IACF,CAAC;AACF,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,qEAAqE,EAAE,GAAG,EAAE;IAChF,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,EAAE,6CAA6C,CAAC,CAAC;IACjF,MAAM,OAAO,GAAG,YAAY,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC;IAEjD,yEAAyE;IACzE,MAAM,cAAc,GAAG,OAAO,CAAC,KAAK,CAAC,+BAA+B,CAAC,CAAC;IACtE,IAAI,cAAc,EAAE,CAAC;QACpB,MAAM,CAAC,EAAE,CACR,CAAC,kBAAkB,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC,CAAC,CAAC,EAC3C,iDAAiD;YACjD,yEAAyE,CACzE,CAAC;IACH,CAAC;AACF,CAAC,CAAC,CAAC;AAEH,uEAAuE;AACvE,IAAI,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;IAC7E,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,aAAa,EAAQ,CAAC;IAEnE,MAAM,KAAK,GAAG,KAAK,CAClB,OAAO,CAAC,QAAQ,KAAK,OAAO,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,EAC3C,OAAO,CAAC,QAAQ,KAAK,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,SAAS,CAAC,EACpE;QACC,QAAQ,EAAE,OAAO,CAAC,QAAQ,KAAK,OAAO;QACtC,KAAK,EAAE,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,CAAC;KACjC,CACD,CAAC;IAEF,IAAI,MAAM,GAAG,EAAE,CAAC;IAChB,KAAK,CAAC,MAAM,EAAE,EAAE,CAAC,MAAM,EAAE,CAAC,CAAS,EAAE,EAAE,GAAG,MAAM,IAAI,CAAC,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;IACrE,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;IAC1B,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,CAAC,IAAI,EAAE,EAAE;QAC1B,IAAI,CAAC;YACJ,MAAM,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,EAAE,sBAAsB,CAAC,CAAC;YAC9C,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,IAAI,EAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE,iCAAiC,MAAM,EAAE,CAAC,CAAC;YACnF,OAAO,EAAE,CAAC;QACX,CAAC;QAAC,OAAO,CAAC,EAAE,CAAC;YACZ,MAAM,CAAC,CAAC,CAAC,CAAC;QACX,CAAC;IACF,CAAC,CAAC,CAAC;IAEH,MAAM,OAAO,CAAC;AACf,CAAC,CAAC,CAAC","sourcesContent":["/**\n * bash-spawn-windows.test.ts — Regression test for Windows spawn EINVAL.\n *\n * Verifies that bash tool spawn options disable `detached: true` on Windows\n * to prevent EINVAL errors in ConPTY / VSCode terminal contexts.\n *\n * Background:\n *   On Windows, `spawn()` with `detached: true` sets the\n *   CREATE_NEW_PROCESS_GROUP flag in CreateProcess.  In certain terminal\n *   contexts (VSCode integrated terminal, ConPTY, Windows Terminal) this\n *   flag conflicts with the parent process group and causes a synchronous\n *   EINVAL from libuv.  The bg-shell extension already guards against this\n *   with `detached: process.platform !== \"win32\"` (process-manager.ts);\n *   this test ensures all other spawn sites are aligned.\n *\n * See: gsd-build/gsd-2#XXXX\n */\n\nimport test from \"node:test\";\nimport assert from \"node:assert/strict\";\nimport { spawn } from \"node:child_process\";\n\n// Verify the spawn option pattern used across the codebase.\n// This is a static/structural test — it reads the source files and asserts\n// they use the platform-guarded detached flag.\nimport { existsSync, readFileSync } from \"node:fs\";\nimport { dirname, join, resolve } from \"node:path\";\nimport { fileURLToPath } from \"node:url\";\n\nconst __dirname = dirname(fileURLToPath(import.meta.url));\n\nfunction findRepoRoot(): string {\n\tconst requiredFiles = [\n\t\t\"packages/pi-coding-agent/src/core/tools/bash.ts\",\n\t\t\"packages/pi-coding-agent/src/core/bash-executor.ts\",\n\t\t\"packages/pi-coding-agent/src/utils/shell.ts\",\n\t];\n\tconst candidates = [\n\t\tresolve(__dirname, \"../../../../../\"),\n\t\tresolve(__dirname, \"../../../../../../\"),\n\t];\n\tfor (const candidate of candidates) {\n\t\tif (requiredFiles.every((file) => existsSync(join(candidate, file)))) {\n\t\t\treturn candidate;\n\t\t}\n\t}\n\tthrow new Error(`Unable to resolve repository root from ${__dirname}`);\n}\n\nconst REPO_ROOT = findRepoRoot();\n\nconst SPAWN_FILES = [\n\tjoin(REPO_ROOT, \"packages/pi-coding-agent/src/core/tools/bash.ts\"),\n\tjoin(REPO_ROOT, \"packages/pi-coding-agent/src/core/bash-executor.ts\"),\n\tjoin(REPO_ROOT, \"packages/pi-coding-agent/src/utils/shell.ts\"),\n];\n\ntest(\"spawn calls use platform-guarded detached flag (no unconditional detached: true)\", () => {\n\tfor (const file of SPAWN_FILES) {\n\t\tconst content = readFileSync(file, \"utf-8\");\n\t\tconst lines = content.split(\"\\n\");\n\n\t\tfor (let i = 0; i < lines.length; i++) {\n\t\t\tconst line = lines[i]!;\n\t\t\t// Skip comments\n\t\t\tif (line.trim().startsWith(\"//\") || line.trim().startsWith(\"*\")) continue;\n\t\t\t// Check for unconditional `detached: true`\n\t\t\tif (/detached:\\s*true\\b/.test(line)) {\n\t\t\t\tassert.fail(\n\t\t\t\t\t`${file}:${i + 1} has unconditional 'detached: true' — ` +\n\t\t\t\t\t`must use 'detached: process.platform !== \"win32\"' ` +\n\t\t\t\t\t`to prevent EINVAL on Windows (ConPTY / VSCode terminal)`,\n\t\t\t\t);\n\t\t\t}\n\t\t}\n\t}\n});\n\ntest(\"killProcessTree does not use detached: true for taskkill on Windows\", () => {\n\tconst shellFile = join(REPO_ROOT, \"packages/pi-coding-agent/src/utils/shell.ts\");\n\tconst content = readFileSync(shellFile, \"utf-8\");\n\n\t// Find the taskkill spawn call and ensure it doesn't have detached: true\n\tconst taskkillRegion = content.match(/spawn\\(\"taskkill\"[\\s\\S]*?\\}\\)/);\n\tif (taskkillRegion) {\n\t\tassert.ok(\n\t\t\t!/detached:\\s*true/.test(taskkillRegion[0]),\n\t\t\t\"taskkill spawn should not use detached: true — \" +\n\t\t\t\"it can cause EINVAL on Windows and is unnecessary for a utility process\",\n\t\t);\n\t}\n});\n\n// Smoke test: spawn with platform-guarded detached flag actually works\ntest(\"spawn with detached: process.platform !== 'win32' succeeds\", async () => {\n\tconst { promise, resolve, reject } = Promise.withResolvers<void>();\n\n\tconst child = spawn(\n\t\tprocess.platform === \"win32\" ? \"cmd\" : \"sh\",\n\t\tprocess.platform === \"win32\" ? [\"/c\", \"echo ok\"] : [\"-c\", \"echo ok\"],\n\t\t{\n\t\t\tdetached: process.platform !== \"win32\",\n\t\t\tstdio: [\"ignore\", \"pipe\", \"pipe\"],\n\t\t},\n\t);\n\n\tlet output = \"\";\n\tchild.stdout?.on(\"data\", (d: Buffer) => { output += d.toString(); });\n\tchild.on(\"error\", reject);\n\tchild.on(\"close\", (code) => {\n\t\ttry {\n\t\t\tassert.equal(code, 0, \"spawn should succeed\");\n\t\t\tassert.ok(output.trim().includes(\"ok\"), `Expected 'ok' in output, got: ${output}`);\n\t\t\tresolve();\n\t\t} catch (e) {\n\t\t\treject(e);\n\t\t}\n\t});\n\n\tawait promise;\n});\n"]}

package/packages/pi-coding-agent/src/core/agent-session-abort-order.test.ts

@@ -77,6 +77,38 @@ function recordCallOrder<O extends object>(
 	return order;
 }
 
+function makeAssistantMessage(text: string) {
+	return {
+		role: "assistant",
+		content: [{ type: "text", text }],
+		usage: {
+			input: 1,
+			output: 1,
+			cacheRead: 0,
+			cacheWrite: 0,
+			total: 2,
+			cost: { input: 0, output: 0, cacheRead: 0, cacheWrite: 0, total: 0 },
+		},
+		stopReason: "stop",
+		timestamp: Date.now(),
+	} as any;
+}
+
+function installAgentEndSessionTransition(
+	session: AgentSession,
+	transition: () => Promise<unknown>,
+): void {
+	(session as any)._extensionRunner = {
+		hasHandlers: () => false,
+		emit: async (event: any) => {
+			if (event.type === "agent_end") {
+				await transition();
+			}
+		},
+		emitStop: async () => {},
+	};
+}
+
 describe("#4243 — abort() must run before _disconnectFromAgent()", () => {
 	beforeEach(() => {
 		testDir = mkdtempSync(join(tmpdir(), "agent-session-abort-"));
@@ -106,6 +138,293 @@ describe("#4243 — abort() must run before _disconnectFromAgent()", () => {
 		);
 	});
 
+	it("newSession() waits instead of aborting while agent_end processing is still streaming", async () => {
+		const session = await createSession();
+		const order: string[] = [];
+		let releaseIdle!: () => void;
+		const idle = new Promise<void>((resolve) => {
+			releaseIdle = resolve;
+		});
+
+		(session as any)._processingAgentEnd = true;
+		(session as any).agent.state.isStreaming = true;
+		(session as any).agent.waitForIdle = () => {
+			order.push("waitForIdle");
+			return idle;
+		};
+		(session as any).abort = async () => {
+			order.push("abort");
+		};
+		const originalDisconnect = (session as any)._disconnectFromAgent.bind(session);
+		(session as any)._disconnectFromAgent = () => {
+			order.push("_disconnectFromAgent");
+			originalDisconnect();
+		};
+
+		const pendingNewSession = session.newSession();
+		await Promise.resolve();
+		assert.deepEqual(order, ["waitForIdle"]);
+		assert.equal(order.includes("abort"), false);
+
+		releaseIdle();
+		const ok = await pendingNewSession;
+		assert.equal(ok, true);
+		assert.deepEqual(order, ["waitForIdle", "_disconnectFromAgent"]);
+		assert.equal(order.includes("abort"), false);
+	});
+
+	it("newSession() waits during agent_end processing even once already idle", async () => {
+		const session = await createSession();
+		const order: string[] = [];
+
+		(session as any)._processingAgentEnd = true;
+		(session as any).agent.state.isStreaming = false;
+		(session as any).agent.waitForIdle = async () => {
+			order.push("waitForIdle");
+		};
+		(session as any).abort = async () => {
+			order.push("abort");
+		};
+		const originalDisconnect = (session as any)._disconnectFromAgent.bind(session);
+		(session as any)._disconnectFromAgent = () => {
+			order.push("_disconnectFromAgent");
+			originalDisconnect();
+		};
+
+		const ok = await session.newSession();
+		assert.equal(ok, true);
+		assert.deepEqual(order, ["waitForIdle", "_disconnectFromAgent"]);
+		assert.equal(order.includes("abort"), false);
+	});
+
+	it("abort() marks synthetic agent_end processing while extension handlers run", async () => {
+		const session = await createSession();
+		const observedProcessingStates: boolean[] = [];
+
+		(session as any).agent.abort = () => {};
+		(session as any).agent.waitForIdle = async () => {};
+		(session as any)._extensionRunner = {
+			emit: async (event: any) => {
+				if (event.type === "agent_end") {
+					observedProcessingStates.push((session as any)._processingAgentEnd);
+				}
+			},
+			emitStop: async () => {
+				observedProcessingStates.push((session as any)._processingAgentEnd);
+			},
+		};
+
+		await session.abort();
+
+		assert.deepEqual(observedProcessingStates, [true, true]);
+		assert.equal((session as any)._processingAgentEnd, false);
+	});
+
+	it("newSession() during agent_end preserves the previous session for resume", async () => {
+		const session = await createSession({ persistSessions: true });
+		const previousSessionFile = session.sessionFile;
+		assert.ok(previousSessionFile, "need a persisted session file");
+
+		session.sessionManager.appendMessage({
+			role: "user",
+			content: [{ type: "text", text: "persisted prompt" }],
+		} as any);
+		session.sessionManager.appendMessage({
+			role: "assistant",
+			content: [{ type: "text", text: "persisted response" }],
+			usage: {
+				input: 1,
+				output: 1,
+				cacheRead: 0,
+				cacheWrite: 0,
+				total: 2,
+				cost: { input: 0, output: 0, cacheRead: 0, cacheWrite: 0, total: 0 },
+			},
+			stopReason: "stop",
+			timestamp: Date.now(),
+		} as any);
+		session.agent.replaceMessages(session.sessionManager.buildSessionContext().messages);
+
+		(session as any)._processingAgentEnd = true;
+		(session as any).agent.waitForIdle = async () => {};
+
+		const ok = await session.newSession();
+		assert.equal(ok, true);
+		assert.notEqual(session.sessionFile, previousSessionFile);
+		assert.deepEqual(session.messages, []);
+
+		(session as any)._processingAgentEnd = false;
+		const switched = await session.switchSession(previousSessionFile);
+		assert.equal(switched, true);
+
+		const restoredText = session.messages
+			.flatMap((message: any) => message.content ?? [])
+			.filter((part: any) => part.type === "text")
+			.map((part: any) => part.text);
+		assert.deepEqual(restoredText, ["persisted prompt", "persisted response"]);
+	});
+
+	it("switchSession() waits instead of aborting while agent_end processing is still streaming", async () => {
+		const session = await createSession({ persistSessions: true });
+		const previousSessionFile = session.sessionFile;
+		assert.ok(previousSessionFile, "need a persisted session file");
+
+		session.sessionManager.appendMessage({
+			role: "user",
+			content: [{ type: "text", text: "switch persisted prompt" }],
+		} as any);
+		session.sessionManager.appendMessage({
+			role: "assistant",
+			content: [{ type: "text", text: "switch persisted response" }],
+			usage: {
+				input: 1,
+				output: 1,
+				cacheRead: 0,
+				cacheWrite: 0,
+				total: 2,
+				cost: { input: 0, output: 0, cacheRead: 0, cacheWrite: 0, total: 0 },
+			},
+			stopReason: "stop",
+			timestamp: Date.now(),
+		} as any);
+		session.agent.replaceMessages(session.sessionManager.buildSessionContext().messages);
+
+		const ok = await session.newSession();
+		assert.equal(ok, true);
+		const activeSessionFile = session.sessionFile;
+		assert.ok(activeSessionFile, "need an active session file");
+		assert.notEqual(activeSessionFile, previousSessionFile);
+		assert.deepEqual(session.messages, []);
+
+		const order: string[] = [];
+		let releaseIdle!: () => void;
+		const idle = new Promise<void>((resolve) => {
+			releaseIdle = resolve;
+		});
+
+		(session as any)._processingAgentEnd = true;
+		(session as any).agent.state.isStreaming = true;
+		(session as any).agent.waitForIdle = () => {
+			order.push("waitForIdle");
+			return idle;
+		};
+		(session as any).abort = async () => {
+			order.push("abort");
+		};
+		const originalDisconnect = (session as any)._disconnectFromAgent.bind(session);
+		(session as any)._disconnectFromAgent = () => {
+			order.push("_disconnectFromAgent");
+			originalDisconnect();
+		};
+
+		const pendingSwitch = session.switchSession(previousSessionFile);
+		await Promise.resolve();
+		assert.deepEqual(order, ["waitForIdle"]);
+		assert.equal(order.includes("abort"), false);
+		assert.equal(session.sessionFile, activeSessionFile);
+		assert.deepEqual(session.messages, []);
+
+		releaseIdle();
+		const switched = await pendingSwitch;
+		assert.equal(switched, true);
+		assert.deepEqual(order, ["waitForIdle", "_disconnectFromAgent"]);
+		assert.equal(order.includes("abort"), false);
+		assert.equal(session.sessionFile, previousSessionFile);
+
+		const restoredText = session.messages
+			.flatMap((message: any) => message.content ?? [])
+			.filter((part: any) => part.type === "text")
+			.map((part: any) => part.text);
+		assert.deepEqual(restoredText, ["switch persisted prompt", "switch persisted response"]);
+	});
+
+	it("newSession() during agent_end skips stale post-handlers after the transition starts", async () => {
+		const session = await createSession();
+		const assistantMessage = makeAssistantMessage("old response");
+		let compactionChecks = 0;
+		let listenerAgentEnds = 0;
+
+		(session as any)._lastAssistantMessage = assistantMessage;
+		(session as any)._compactionOrchestrator.checkCompaction = async () => {
+			compactionChecks++;
+		};
+		session.subscribe((event: any) => {
+			if (event.type === "agent_end") listenerAgentEnds++;
+		});
+		installAgentEndSessionTransition(session, () => session.newSession());
+
+		await (session as any)._processAgentEvent({
+			type: "agent_end",
+			messages: [assistantMessage],
+		});
+
+		assert.equal(compactionChecks, 0);
+		assert.equal(listenerAgentEnds, 0);
+		assert.equal((session as any)._lastAssistantMessage, undefined);
+		assert.equal((session as any)._sessionSwitchPending, false);
+		assert.equal((session as any)._sessionTransitionStartedDuringAgentEnd, false);
+	});
+
+	it("switchSession() during agent_end skips stale post-handlers after the transition starts", async () => {
+		const session = await createSession({ persistSessions: true });
+		const previousSessionFile = session.sessionFile;
+		assert.ok(previousSessionFile, "need a persisted session file");
+
+		const ok = await session.newSession();
+		assert.equal(ok, true);
+		assert.notEqual(session.sessionFile, previousSessionFile);
+
+		const assistantMessage = makeAssistantMessage("old switch response");
+		let compactionChecks = 0;
+		let listenerAgentEnds = 0;
+
+		(session as any)._lastAssistantMessage = assistantMessage;
+		(session as any)._compactionOrchestrator.checkCompaction = async () => {
+			compactionChecks++;
+		};
+		session.subscribe((event: any) => {
+			if (event.type === "agent_end") listenerAgentEnds++;
+		});
+		installAgentEndSessionTransition(session, () => session.switchSession(previousSessionFile));
+
+		await (session as any)._processAgentEvent({
+			type: "agent_end",
+			messages: [assistantMessage],
+		});
+
+		assert.equal(session.sessionFile, previousSessionFile);
+		assert.equal(compactionChecks, 0);
+		assert.equal(listenerAgentEnds, 0);
+		assert.equal((session as any)._lastAssistantMessage, undefined);
+		assert.equal((session as any)._sessionSwitchPending, false);
+		assert.equal((session as any)._sessionTransitionStartedDuringAgentEnd, false);
+	});
+
+	it("agent_end post-handlers bail while a session switch is pending", async () => {
+		const session = await createSession();
+		const assistantMessage = makeAssistantMessage("old pending response");
+		let compactionChecks = 0;
+		let listenerAgentEnds = 0;
+
+		(session as any)._lastAssistantMessage = assistantMessage;
+		(session as any)._sessionSwitchPending = true;
+		(session as any)._compactionOrchestrator.checkCompaction = async () => {
+			compactionChecks++;
+		};
+		session.subscribe((event: any) => {
+			if (event.type === "agent_end") listenerAgentEnds++;
+		});
+
+		await (session as any)._processAgentEvent({
+			type: "agent_end",
+			messages: [assistantMessage],
+		});
+
+		assert.equal(compactionChecks, 0);
+		assert.equal(listenerAgentEnds, 1);
+		assert.equal((session as any)._lastAssistantMessage, undefined);
+	});
+
 	it("switchSession() invokes abort() before _disconnectFromAgent()", async () => {
 		const session = await createSession({ persistSessions: true });
 		// Seed a session file to switch to (switchSession reads from the session manager).