gsd-pi 2.78.1-dev.a7b6e59b7 → 2.78.1-dev.b0759e59b

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/extensions/runner.test.ts

@@ -4,7 +4,7 @@ import { ExtensionRunner } from "./runner.js";
 import type { Extension, ExtensionRuntime, ToolCallEvent } from "./index.js";
 import { SessionManager } from "../session-manager.js";
 import { ModelRegistry } from "../model-registry.js";
-import { mkdtempSync, rmSync } from "node:fs";
+import { mkdirSync, mkdtempSync, realpathSync, rmSync } from "node:fs";
 import { join } from "node:path";
 import { tmpdir } from "node:os";
 import { AuthStorage } from "../auth-storage.js";
@@ -47,6 +47,27 @@ function makeThrowingExtension(eventType: string, error: Error): Extension {
 	} as unknown as Extension;
 }
 
+function makeCommandExtension(path: string, commandName: string, marker: string): Extension {
+	return {
+		path,
+		commands: new Map([
+			[
+				commandName,
+				{
+					name: commandName,
+					description: marker,
+					handler: async () => {},
+				},
+			],
+		]),
+		handlers: new Map(),
+		shortcuts: new Map(),
+		tools: new Map(),
+		flags: new Map(),
+		diagnostics: [],
+	} as unknown as Extension;
+}
+
 describe("ExtensionRunner.emitToolCall", () => {
 	it("catches throwing extension handler and routes to emitError", async (t) => {
 		const dir = mkdtempSync(join(tmpdir(), "runner-test-"));
@@ -82,3 +103,70 @@ describe("ExtensionRunner.emitToolCall", () => {
 		assert.equal(errors[0].extensionPath, "/test/throwing-ext");
 	});
 });
+
+describe("ExtensionRunner.createContext", () => {
+	it("uses the live process cwd instead of the constructor cwd", (t) => {
+		const originalCwd = process.cwd();
+		const dir = mkdtempSync(join(tmpdir(), "runner-test-"));
+		const projectDir = join(dir, "project");
+		t.after(() => {
+			process.chdir(originalCwd);
+			rmSync(dir, { recursive: true, force: true });
+		});
+
+		const sessionManager = SessionManager.create(dir, dir);
+		const authStorage = AuthStorage.create();
+		const modelRegistry = new ModelRegistry(authStorage, join(dir, "models.json"));
+		const runtime = makeMinimalRuntime();
+		const runner = new ExtensionRunner([], runtime, originalCwd, sessionManager, modelRegistry);
+
+		mkdirSync(projectDir);
+		const realProjectDir = realpathSync(projectDir);
+		process.chdir(realProjectDir);
+
+		assert.equal(runner.createContext().cwd, realProjectDir);
+		assert.equal(runner.createCommandContext().cwd, realProjectDir);
+	});
+});
+
+describe("ExtensionRunner protected commands", () => {
+	it("resolves /gsd to the bundled GSD extension even when another extension loads first", () => {
+		const dir = mkdtempSync(join(tmpdir(), "runner-test-"));
+		try {
+			const sessionManager = SessionManager.create(dir, dir);
+			const authStorage = AuthStorage.create();
+			const modelRegistry = new ModelRegistry(authStorage, join(dir, "models.json"));
+			const runtime = makeMinimalRuntime();
+			const userExt = makeCommandExtension("/tmp/extensions/user-spoof/index.ts", "gsd", "spoof");
+			const gsdExt = makeCommandExtension(`${dir}/extensions/gsd/index.ts`, "gsd", "bundled");
+			const runner = new ExtensionRunner([userExt, gsdExt], runtime, dir, sessionManager, modelRegistry);
+
+			const command = runner.getCommand("gsd");
+			assert.equal(command?.description, "bundled");
+		} finally {
+			rmSync(dir, { recursive: true, force: true });
+		}
+	});
+
+	it("omits spoofed /gsd from registered extension commands", () => {
+		const dir = mkdtempSync(join(tmpdir(), "runner-test-"));
+		try {
+			const sessionManager = SessionManager.create(dir, dir);
+			const authStorage = AuthStorage.create();
+			const modelRegistry = new ModelRegistry(authStorage, join(dir, "models.json"));
+			const runtime = makeMinimalRuntime();
+			const userExt = makeCommandExtension("/tmp/extensions/user-spoof/index.ts", "gsd", "spoof");
+			const gsdExt = makeCommandExtension(`${dir}/extensions/gsd/index.ts`, "gsd", "bundled");
+			const runner = new ExtensionRunner([userExt, gsdExt], runtime, dir, sessionManager, modelRegistry);
+
+			const commands = runner.getRegisteredCommands();
+			assert.deepEqual(commands.map((command) => command.description), ["bundled"]);
+			assert.ok(
+				runner.getCommandDiagnostics().some((diagnostic) => diagnostic.message.includes("protected command owner")),
+				"spoofed /gsd conflict should be reported",
+			);
+		} finally {
+			rmSync(dir, { recursive: true, force: true });
+		}
+	});
+});

package/packages/pi-coding-agent/src/core/extensions/runner.ts

@@ -116,6 +116,15 @@ const buildBuiltinKeybindings = (effectiveKeybindings: Required<KeybindingsConfi
 	return builtinKeybindings;
 };
 
+const PROTECTED_EXTENSION_COMMANDS = new Set(["gsd"]);
+
+function isProtectedCommandOwner(commandName: string, extensionPath: string): boolean {
+	if (!PROTECTED_EXTENSION_COMMANDS.has(commandName)) return false;
+	const normalized = extensionPath.replace(/\\/g, "/");
+	return /\/extensions\/gsd\/(?:index\.[cm]?[jt]s|dist\/.*)$/.test(normalized)
+		|| /\/extensions\/gsd\/?$/.test(normalized);
+}
+
 /** Combined result from all before_agent_start handlers */
 interface BeforeAgentStartCombinedResult {
 	messages?: NonNullable<BeforeAgentStartEventResult["message"]>[];
@@ -264,6 +273,14 @@ export class ExtensionRunner {
 		this.runtime.emitExtensionEvent = (event) => this.emitExtensionEventDynamic(event);
 	}
 
+	private currentCwd(): string {
+		try {
+			return process.cwd();
+		} catch {
+			return this.cwd;
+		}
+	}
+
 	/**
 	 * Dispatch an ExtensionEvent by type. Used by extensions to emit the
 	 * post-plan Layer 2 events (git lifecycle, verify, budget, milestone,
@@ -588,6 +605,15 @@ export class ExtensionRunner {
 
 		const commands: RegisteredCommand[] = [];
 		const commandOwners = new Map<string, string>();
+		const protectedOwners = new Map<string, string>();
+		for (const ext of this.extensions) {
+			for (const command of ext.commands.values()) {
+				if (isProtectedCommandOwner(command.name, ext.path)) {
+					protectedOwners.set(command.name, ext.path);
+				}
+			}
+		}
+
 		for (const ext of this.extensions) {
 			for (const command of ext.commands.values()) {
 				if (reserved?.has(command.name)) {
@@ -599,6 +625,16 @@ export class ExtensionRunner {
 					continue;
 				}
 
+				const protectedOwner = protectedOwners.get(command.name);
+				if (protectedOwner && protectedOwner !== ext.path) {
+					const message = `Extension command '${command.name}' from ${ext.path} conflicts with protected command owner ${protectedOwner}. Skipping.`;
+					this.commandDiagnostics.push({ type: "warning", message, path: ext.path });
+					if (!this.hasUI()) {
+						console.warn(message);
+					}
+					continue;
+				}
+
 				const existingOwner = commandOwners.get(command.name);
 				if (existingOwner) {
 					const message = `Extension command '${command.name}' from ${ext.path} conflicts with ${existingOwner}. Skipping.`;
@@ -631,13 +667,21 @@ export class ExtensionRunner {
 	}
 
 	getCommand(name: string): RegisteredCommand | undefined {
+		let protectedCommand: RegisteredCommand | undefined;
 		for (const ext of this.extensions) {
 			const command = ext.commands.get(name);
 			if (command) {
+				if (isProtectedCommandOwner(name, ext.path)) {
+					protectedCommand = command;
+					break;
+				}
+				if (PROTECTED_EXTENSION_COMMANDS.has(name)) {
+					continue;
+				}
 				return command;
 			}
 		}
-		return undefined;
+		return protectedCommand;
 	}
 
 	/**
@@ -657,7 +701,7 @@ export class ExtensionRunner {
 		return {
 			ui: this.uiContext,
 			hasUI: this.hasUI(),
-			cwd: this.cwd,
+			cwd: this.currentCwd(),
 			sessionManager: this.sessionManager,
 			modelRegistry: this.modelRegistry,
 			get model() {

package/packages/pi-coding-agent/src/core/tools/bash-spawn-windows.test.ts

@@ -23,16 +23,36 @@ 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(): string {
+	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)", () => {
@@ -57,7 +77,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