




















@@ -395,6 +395,20 @@ function lastRespondCall(respond: ReturnType<typeof vi.fn>) {
395395| undefined;
396396}
397397398+function responseErrorMessage(error: unknown): string {
399+if (error instanceof Error) {
400+return error.message;
401+}
402+if (error && typeof error === "object") {
403+const message = (error as { message?: unknown }).message;
404+if (typeof message === "string") {
405+return message;
406+}
407+return JSON.stringify(error);
408+}
409+return String(error);
410+}
411+398412function lastBroadcastPayload(context: ChatContext): Record<string, any> | undefined {
399413const chatCall = (context.broadcast as unknown as ReturnType<typeof vi.fn>).mock.calls.at(-1);
400414expect(chatCall?.[0]).toBe("chat");
@@ -590,7 +604,9 @@ async function runNonStreamingChatSend(params: {
590604).toBe(1);
591605});
592606593-const chatCall = (params.context.broadcast as unknown as ReturnType<typeof vi.fn>).mock.calls[0];
607+const chatCall = (params.context.broadcast as unknown as ReturnType<typeof vi.fn>).mock.calls.at(
608+0,
609+);
594610expect(chatCall?.[0]).toBe("chat");
595611return chatCall?.[1];
596612}
@@ -2957,13 +2973,14 @@ describe("chat directive tag stripping for non-streaming final payloads", () =>
29572973// so the client retries instead of treating it as a bad request.
29582974expect(mockState.lastDispatchCtx).toBeUndefined();
29592975expect(respond).toHaveBeenCalledTimes(1);
2960-const [ok, payload, error] = respond.mock.calls[0] ?? [];
2976+const [ok, payload, error] = lastRespondCall(respond) ?? [];
29612977expect(ok).toBe(false);
29622978expect(payload).toBeUndefined();
29632979expect(error?.code).toBe(ErrorCodes.UNAVAILABLE);
2964-expect(error?.message ?? String(error)).toMatch(/ENOSPC|non-image attachments/i);
2965-const unavailableLogCall = (context.logGateway.error as unknown as ReturnType<typeof vi.fn>)
2966-.mock.calls[0] as [string, Record<string, string>] | undefined;
2980+expect(responseErrorMessage(error)).toMatch(/ENOSPC|non-image attachments/i);
2981+const unavailableLogCall = (
2982+context.logGateway.error as unknown as ReturnType<typeof vi.fn>
2983+).mock.calls.at(0) as [string, Record<string, string>] | undefined;
29672984expect(unavailableLogCall?.[0]).toBe("chat.send attachment parse/stage failed");
29682985expect(unavailableLogCall?.[1].consoleMessage).toContain(
29692986"chat.send attachment parse/stage failed: MediaOffloadError",
@@ -3067,11 +3084,11 @@ describe("chat directive tag stripping for non-streaming final payloads", () =>
3067308430683085expect(mockState.lastDispatchCtx).toBeUndefined();
30693086expect(respond).toHaveBeenCalledTimes(1);
3070-const [ok, payload, error] = respond.mock.calls[0] ?? [];
3087+const [ok, payload, error] = lastRespondCall(respond) ?? [];
30713088expect(ok).toBe(false);
30723089expect(payload).toBeUndefined();
30733090expect(error?.code).toBe(ErrorCodes.UNAVAILABLE);
3074-expect(error?.message ?? String(error)).toMatch(/staging incomplete/i);
3091+expect(responseErrorMessage(error)).toMatch(/staging incomplete/i);
30753092// Both media-store entries are cleaned up before the 5xx surfaces.
30763093expect(mockState.deleteMediaBufferCalls.map((c) => c.id).toSorted()).toEqual([
30773094"saved-media",
@@ -3127,13 +3144,13 @@ describe("chat directive tag stripping for non-streaming final payloads", () =>
3127314431283145expect(mockState.lastDispatchCtx).toBeUndefined();
31293146expect(respond).toHaveBeenCalledTimes(1);
3130-const [ok, payload, error] = respond.mock.calls[0] ?? [];
3147+const [ok, payload, error] = lastRespondCall(respond) ?? [];
31313148expect(ok).toBe(false);
31323149expect(payload).toBeUndefined();
31333150// 4xx, not 5xx — retrying a file that exceeds the staging cap cannot
31343151// succeed, so the failure must be surfaced as a client-side rejection.
31353152expect(error?.code).toBe(ErrorCodes.INVALID_REQUEST);
3136-expect(error?.message ?? String(error)).toMatch(/sandbox staging limit/i);
3153+expect(responseErrorMessage(error)).toMatch(/sandbox staging limit/i);
31373154// Orphaned media-store entries are cleaned up before the 4xx surfaces.
31383155expect(mockState.deleteMediaBufferCalls).toEqual([{ id: "saved-media", subdir: "inbound" }]);
31393156});
此内容由惯性聚合(RSS阅读器)自动聚合整理,仅供阅读参考。 原文来自 — 版权归原作者所有。