|
| 1 | +import assert from "node:assert/strict"; |
| 2 | +import { describe, it } from "node:test"; |
| 3 | + |
| 4 | +import { |
| 5 | + PROMPT_TEMPLATE_SUBAGENT_CANCEL_EVENT, |
| 6 | + PROMPT_TEMPLATE_SUBAGENT_REQUEST_EVENT, |
| 7 | + PROMPT_TEMPLATE_SUBAGENT_RESPONSE_EVENT, |
| 8 | + PROMPT_TEMPLATE_SUBAGENT_STARTED_EVENT, |
| 9 | + PROMPT_TEMPLATE_SUBAGENT_UPDATE_EVENT, |
| 10 | + registerPromptTemplateDelegationBridge, |
| 11 | + type PromptTemplateBridgeEvents, |
| 12 | +} from "./prompt-template-bridge.ts"; |
| 13 | + |
| 14 | +class FakeEvents implements PromptTemplateBridgeEvents { |
| 15 | + private handlers = new Map<string, Array<(data: unknown) => void>>(); |
| 16 | + |
| 17 | + on(event: string, handler: (data: unknown) => void): () => void { |
| 18 | + const list = this.handlers.get(event) ?? []; |
| 19 | + list.push(handler); |
| 20 | + this.handlers.set(event, list); |
| 21 | + return () => { |
| 22 | + const current = this.handlers.get(event) ?? []; |
| 23 | + this.handlers.set(event, current.filter((h) => h !== handler)); |
| 24 | + }; |
| 25 | + } |
| 26 | + |
| 27 | + emit(event: string, data: unknown): void { |
| 28 | + const list = this.handlers.get(event) ?? []; |
| 29 | + for (const handler of [...list]) handler(data); |
| 30 | + } |
| 31 | +} |
| 32 | + |
| 33 | +function once(events: FakeEvents, event: string): Promise<unknown> { |
| 34 | + return new Promise((resolve) => { |
| 35 | + const unsubscribe = events.on(event, (payload) => { |
| 36 | + unsubscribe(); |
| 37 | + resolve(payload); |
| 38 | + }); |
| 39 | + }); |
| 40 | +} |
| 41 | + |
| 42 | +describe("prompt-template delegation bridge", () => { |
| 43 | + it("emits started/update/response on successful request", async () => { |
| 44 | + const events = new FakeEvents(); |
| 45 | + let executeCalls = 0; |
| 46 | + const bridge = registerPromptTemplateDelegationBridge({ |
| 47 | + events, |
| 48 | + getContext: () => ({ cwd: "/repo" }), |
| 49 | + execute: async (_requestId, _request, _signal, _ctx, onUpdate) => { |
| 50 | + executeCalls++; |
| 51 | + onUpdate({ |
| 52 | + details: { |
| 53 | + progress: [{ currentTool: "read", currentToolArgs: "index.ts", recentOutput: ["line 1"], toolCount: 1, durationMs: 10, tokens: 42 }], |
| 54 | + }, |
| 55 | + }); |
| 56 | + return { |
| 57 | + details: { |
| 58 | + results: [{ messages: [{ role: "assistant", content: [{ type: "text", text: "ok" }] }] }], |
| 59 | + }, |
| 60 | + }; |
| 61 | + }, |
| 62 | + }); |
| 63 | + |
| 64 | + const startedPromise = once(events, PROMPT_TEMPLATE_SUBAGENT_STARTED_EVENT); |
| 65 | + const updatePromise = once(events, PROMPT_TEMPLATE_SUBAGENT_UPDATE_EVENT); |
| 66 | + const responsePromise = once(events, PROMPT_TEMPLATE_SUBAGENT_RESPONSE_EVENT); |
| 67 | + |
| 68 | + events.emit(PROMPT_TEMPLATE_SUBAGENT_REQUEST_EVENT, { |
| 69 | + requestId: "r1", |
| 70 | + agent: "worker", |
| 71 | + task: "do work", |
| 72 | + context: "fresh", |
| 73 | + model: "openai/gpt-5", |
| 74 | + cwd: "/repo", |
| 75 | + }); |
| 76 | + |
| 77 | + const started = await startedPromise as { requestId: string }; |
| 78 | + assert.equal(started.requestId, "r1"); |
| 79 | + |
| 80 | + const update = await updatePromise as { requestId: string; currentTool?: string; toolCount?: number }; |
| 81 | + assert.equal(update.requestId, "r1"); |
| 82 | + assert.equal(update.currentTool, "read"); |
| 83 | + assert.equal(update.toolCount, 1); |
| 84 | + |
| 85 | + const response = await responsePromise as { requestId: string; isError: boolean; messages: unknown[] }; |
| 86 | + assert.equal(response.requestId, "r1"); |
| 87 | + assert.equal(response.isError, false); |
| 88 | + assert.equal(Array.isArray(response.messages), true); |
| 89 | + assert.equal(executeCalls, 1); |
| 90 | + |
| 91 | + bridge.dispose(); |
| 92 | + }); |
| 93 | + |
| 94 | + it("returns structured error when no active context", async () => { |
| 95 | + const events = new FakeEvents(); |
| 96 | + const bridge = registerPromptTemplateDelegationBridge({ |
| 97 | + events, |
| 98 | + getContext: () => null, |
| 99 | + execute: async () => ({ details: { results: [{ messages: [] }] } }), |
| 100 | + }); |
| 101 | + |
| 102 | + const responsePromise = once(events, PROMPT_TEMPLATE_SUBAGENT_RESPONSE_EVENT); |
| 103 | + events.emit(PROMPT_TEMPLATE_SUBAGENT_REQUEST_EVENT, { |
| 104 | + requestId: "r2", |
| 105 | + agent: "worker", |
| 106 | + task: "do work", |
| 107 | + context: "fresh", |
| 108 | + model: "openai/gpt-5", |
| 109 | + cwd: "/repo", |
| 110 | + }); |
| 111 | + |
| 112 | + const response = await responsePromise as { isError: boolean; errorText?: string }; |
| 113 | + assert.equal(response.isError, true); |
| 114 | + assert.match(response.errorText ?? "", /No active extension context/); |
| 115 | + |
| 116 | + bridge.dispose(); |
| 117 | + }); |
| 118 | + |
| 119 | + it("rejects cwd mismatch", async () => { |
| 120 | + const events = new FakeEvents(); |
| 121 | + const bridge = registerPromptTemplateDelegationBridge({ |
| 122 | + events, |
| 123 | + getContext: () => ({ cwd: "/actual" }), |
| 124 | + execute: async () => ({ details: { results: [{ messages: [] }] } }), |
| 125 | + }); |
| 126 | + |
| 127 | + const responsePromise = once(events, PROMPT_TEMPLATE_SUBAGENT_RESPONSE_EVENT); |
| 128 | + events.emit(PROMPT_TEMPLATE_SUBAGENT_REQUEST_EVENT, { |
| 129 | + requestId: "r3", |
| 130 | + agent: "worker", |
| 131 | + task: "do work", |
| 132 | + context: "fresh", |
| 133 | + model: "openai/gpt-5", |
| 134 | + cwd: "/repo", |
| 135 | + }); |
| 136 | + |
| 137 | + const response = await responsePromise as { isError: boolean; errorText?: string }; |
| 138 | + assert.equal(response.isError, true); |
| 139 | + assert.match(response.errorText ?? "", /cwd mismatch/); |
| 140 | + |
| 141 | + bridge.dispose(); |
| 142 | + }); |
| 143 | + |
| 144 | + it("applies pending cancel when cancel arrives before request", async () => { |
| 145 | + const events = new FakeEvents(); |
| 146 | + let executeCalls = 0; |
| 147 | + const bridge = registerPromptTemplateDelegationBridge({ |
| 148 | + events, |
| 149 | + getContext: () => ({ cwd: "/repo" }), |
| 150 | + execute: async () => { |
| 151 | + executeCalls++; |
| 152 | + return { details: { results: [{ messages: [] }] } }; |
| 153 | + }, |
| 154 | + }); |
| 155 | + |
| 156 | + events.emit(PROMPT_TEMPLATE_SUBAGENT_CANCEL_EVENT, { requestId: "r4" }); |
| 157 | + const responsePromise = once(events, PROMPT_TEMPLATE_SUBAGENT_RESPONSE_EVENT); |
| 158 | + |
| 159 | + events.emit(PROMPT_TEMPLATE_SUBAGENT_REQUEST_EVENT, { |
| 160 | + requestId: "r4", |
| 161 | + agent: "worker", |
| 162 | + task: "do work", |
| 163 | + context: "fresh", |
| 164 | + model: "openai/gpt-5", |
| 165 | + cwd: "/repo", |
| 166 | + }); |
| 167 | + |
| 168 | + const response = await responsePromise as { isError: boolean; errorText?: string }; |
| 169 | + assert.equal(response.isError, true); |
| 170 | + assert.equal(response.errorText, "Delegated prompt cancelled."); |
| 171 | + assert.equal(executeCalls, 0); |
| 172 | + |
| 173 | + bridge.dispose(); |
| 174 | + }); |
| 175 | + |
| 176 | + it("cancels in-flight delegated execution", async () => { |
| 177 | + const events = new FakeEvents(); |
| 178 | + const bridge = registerPromptTemplateDelegationBridge({ |
| 179 | + events, |
| 180 | + getContext: () => ({ cwd: "/repo" }), |
| 181 | + execute: async (_requestId, _request, signal) => |
| 182 | + await new Promise((_resolve, reject) => { |
| 183 | + signal.addEventListener("abort", () => reject(new Error("aborted")), { once: true }); |
| 184 | + }), |
| 185 | + }); |
| 186 | + |
| 187 | + const startedPromise = once(events, PROMPT_TEMPLATE_SUBAGENT_STARTED_EVENT); |
| 188 | + const responsePromise = once(events, PROMPT_TEMPLATE_SUBAGENT_RESPONSE_EVENT); |
| 189 | + |
| 190 | + events.emit(PROMPT_TEMPLATE_SUBAGENT_REQUEST_EVENT, { |
| 191 | + requestId: "r5", |
| 192 | + agent: "worker", |
| 193 | + task: "do work", |
| 194 | + context: "fresh", |
| 195 | + model: "openai/gpt-5", |
| 196 | + cwd: "/repo", |
| 197 | + }); |
| 198 | + |
| 199 | + await startedPromise; |
| 200 | + events.emit(PROMPT_TEMPLATE_SUBAGENT_CANCEL_EVENT, { requestId: "r5" }); |
| 201 | + |
| 202 | + const response = await responsePromise as { isError: boolean; errorText?: string }; |
| 203 | + assert.equal(response.isError, true); |
| 204 | + assert.match(response.errorText ?? "", /aborted/i); |
| 205 | + |
| 206 | + bridge.dispose(); |
| 207 | + }); |
| 208 | +}); |
0 commit comments