@copilotkit/runtime 1.56.0 → 1.56.2-canary.pin-to-send
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/agent/index.cjs +2 -2
- package/dist/agent/index.cjs.map +1 -1
- package/dist/agent/index.d.cts.map +1 -1
- package/dist/agent/index.d.mts.map +1 -1
- package/dist/agent/index.mjs +2 -2
- package/dist/agent/index.mjs.map +1 -1
- package/dist/graphql/resolvers/copilot.resolver.cjs +2 -1
- package/dist/graphql/resolvers/copilot.resolver.cjs.map +1 -1
- package/dist/graphql/resolvers/copilot.resolver.mjs +2 -1
- package/dist/graphql/resolvers/copilot.resolver.mjs.map +1 -1
- package/dist/graphql/resolvers/resolve-message-id.cjs +19 -0
- package/dist/graphql/resolvers/resolve-message-id.cjs.map +1 -0
- package/dist/graphql/resolvers/resolve-message-id.mjs +18 -0
- package/dist/graphql/resolvers/resolve-message-id.mjs.map +1 -0
- package/dist/lib/integrations/node-http/index.cjs +4 -1
- package/dist/lib/integrations/node-http/index.cjs.map +1 -1
- package/dist/lib/integrations/node-http/index.d.cts.map +1 -1
- package/dist/lib/integrations/node-http/index.d.mts.map +1 -1
- package/dist/lib/integrations/node-http/index.mjs +4 -1
- package/dist/lib/integrations/node-http/index.mjs.map +1 -1
- package/dist/lib/runtime/copilot-runtime.cjs +15 -3
- package/dist/lib/runtime/copilot-runtime.cjs.map +1 -1
- package/dist/lib/runtime/copilot-runtime.d.cts.map +1 -1
- package/dist/lib/runtime/copilot-runtime.d.mts.map +1 -1
- package/dist/lib/runtime/copilot-runtime.mjs +15 -3
- package/dist/lib/runtime/copilot-runtime.mjs.map +1 -1
- package/dist/lib/runtime/mcp-tools-utils.cjs +21 -4
- package/dist/lib/runtime/mcp-tools-utils.cjs.map +1 -1
- package/dist/lib/runtime/mcp-tools-utils.d.cts.map +1 -1
- package/dist/lib/runtime/mcp-tools-utils.d.mts.map +1 -1
- package/dist/lib/runtime/mcp-tools-utils.mjs +21 -4
- package/dist/lib/runtime/mcp-tools-utils.mjs.map +1 -1
- package/dist/package.cjs +2 -2
- package/dist/package.mjs +2 -2
- package/dist/service-adapters/anthropic/anthropic-adapter.cjs +11 -3
- package/dist/service-adapters/anthropic/anthropic-adapter.cjs.map +1 -1
- package/dist/service-adapters/anthropic/anthropic-adapter.d.cts +6 -0
- package/dist/service-adapters/anthropic/anthropic-adapter.d.cts.map +1 -1
- package/dist/service-adapters/anthropic/anthropic-adapter.d.mts +6 -0
- package/dist/service-adapters/anthropic/anthropic-adapter.d.mts.map +1 -1
- package/dist/service-adapters/anthropic/anthropic-adapter.mjs +11 -3
- package/dist/service-adapters/anthropic/anthropic-adapter.mjs.map +1 -1
- package/dist/service-adapters/anthropic/utils.cjs +27 -1
- package/dist/service-adapters/anthropic/utils.cjs.map +1 -1
- package/dist/service-adapters/anthropic/utils.mjs +27 -1
- package/dist/service-adapters/anthropic/utils.mjs.map +1 -1
- package/dist/service-adapters/langchain/utils.cjs +1 -1
- package/dist/service-adapters/langchain/utils.cjs.map +1 -1
- package/dist/service-adapters/langchain/utils.mjs +1 -1
- package/dist/service-adapters/langchain/utils.mjs.map +1 -1
- package/dist/service-adapters/openai/openai-adapter.cjs +2 -1
- package/dist/service-adapters/openai/openai-adapter.cjs.map +1 -1
- package/dist/service-adapters/openai/openai-adapter.d.cts +6 -0
- package/dist/service-adapters/openai/openai-adapter.d.cts.map +1 -1
- package/dist/service-adapters/openai/openai-adapter.d.mts +6 -0
- package/dist/service-adapters/openai/openai-adapter.d.mts.map +1 -1
- package/dist/service-adapters/openai/openai-adapter.mjs +2 -1
- package/dist/service-adapters/openai/openai-adapter.mjs.map +1 -1
- package/dist/v2/runtime/core/debug-event-bus.cjs +36 -0
- package/dist/v2/runtime/core/debug-event-bus.cjs.map +1 -0
- package/dist/v2/runtime/core/debug-event-bus.d.cts +19 -0
- package/dist/v2/runtime/core/debug-event-bus.d.cts.map +1 -0
- package/dist/v2/runtime/core/debug-event-bus.d.mts +19 -0
- package/dist/v2/runtime/core/debug-event-bus.d.mts.map +1 -0
- package/dist/v2/runtime/core/debug-event-bus.mjs +35 -0
- package/dist/v2/runtime/core/debug-event-bus.mjs.map +1 -0
- package/dist/v2/runtime/core/fetch-handler.cjs +6 -0
- package/dist/v2/runtime/core/fetch-handler.cjs.map +1 -1
- package/dist/v2/runtime/core/fetch-handler.d.cts.map +1 -1
- package/dist/v2/runtime/core/fetch-handler.d.mts.map +1 -1
- package/dist/v2/runtime/core/fetch-handler.mjs +6 -0
- package/dist/v2/runtime/core/fetch-handler.mjs.map +1 -1
- package/dist/v2/runtime/core/fetch-router.cjs +1 -0
- package/dist/v2/runtime/core/fetch-router.cjs.map +1 -1
- package/dist/v2/runtime/core/fetch-router.mjs +1 -0
- package/dist/v2/runtime/core/fetch-router.mjs.map +1 -1
- package/dist/v2/runtime/core/hooks.cjs.map +1 -1
- package/dist/v2/runtime/core/hooks.d.cts +2 -0
- package/dist/v2/runtime/core/hooks.d.cts.map +1 -1
- package/dist/v2/runtime/core/hooks.d.mts +2 -0
- package/dist/v2/runtime/core/hooks.d.mts.map +1 -1
- package/dist/v2/runtime/core/hooks.mjs.map +1 -1
- package/dist/v2/runtime/core/middleware-sse-parser.cjs +5 -2
- package/dist/v2/runtime/core/middleware-sse-parser.cjs.map +1 -1
- package/dist/v2/runtime/core/middleware-sse-parser.mjs +5 -2
- package/dist/v2/runtime/core/middleware-sse-parser.mjs.map +1 -1
- package/dist/v2/runtime/core/runtime.cjs +5 -0
- package/dist/v2/runtime/core/runtime.cjs.map +1 -1
- package/dist/v2/runtime/core/runtime.d.cts +5 -0
- package/dist/v2/runtime/core/runtime.d.cts.map +1 -1
- package/dist/v2/runtime/core/runtime.d.mts +5 -0
- package/dist/v2/runtime/core/runtime.d.mts.map +1 -1
- package/dist/v2/runtime/core/runtime.mjs +5 -0
- package/dist/v2/runtime/core/runtime.mjs.map +1 -1
- package/dist/v2/runtime/handlers/handle-connect.cjs +2 -0
- package/dist/v2/runtime/handlers/handle-connect.cjs.map +1 -1
- package/dist/v2/runtime/handlers/handle-connect.mjs +2 -0
- package/dist/v2/runtime/handlers/handle-connect.mjs.map +1 -1
- package/dist/v2/runtime/handlers/handle-debug-events.cjs +33 -0
- package/dist/v2/runtime/handlers/handle-debug-events.cjs.map +1 -0
- package/dist/v2/runtime/handlers/handle-debug-events.mjs +32 -0
- package/dist/v2/runtime/handlers/handle-debug-events.mjs.map +1 -0
- package/dist/v2/runtime/handlers/handle-run.cjs +1 -0
- package/dist/v2/runtime/handlers/handle-run.cjs.map +1 -1
- package/dist/v2/runtime/handlers/handle-run.mjs +1 -0
- package/dist/v2/runtime/handlers/handle-run.mjs.map +1 -1
- package/dist/v2/runtime/handlers/intelligence/connect.cjs +32 -2
- package/dist/v2/runtime/handlers/intelligence/connect.cjs.map +1 -1
- package/dist/v2/runtime/handlers/intelligence/connect.mjs +31 -2
- package/dist/v2/runtime/handlers/intelligence/connect.mjs.map +1 -1
- package/dist/v2/runtime/handlers/shared/resolve-intelligence-user.cjs +5 -1
- package/dist/v2/runtime/handlers/shared/resolve-intelligence-user.cjs.map +1 -1
- package/dist/v2/runtime/handlers/shared/resolve-intelligence-user.mjs +5 -1
- package/dist/v2/runtime/handlers/shared/resolve-intelligence-user.mjs.map +1 -1
- package/dist/v2/runtime/handlers/shared/sse-response.cjs +21 -1
- package/dist/v2/runtime/handlers/shared/sse-response.cjs.map +1 -1
- package/dist/v2/runtime/handlers/shared/sse-response.mjs +21 -1
- package/dist/v2/runtime/handlers/shared/sse-response.mjs.map +1 -1
- package/dist/v2/runtime/handlers/sse/connect.cjs +3 -1
- package/dist/v2/runtime/handlers/sse/connect.cjs.map +1 -1
- package/dist/v2/runtime/handlers/sse/connect.mjs +3 -1
- package/dist/v2/runtime/handlers/sse/connect.mjs.map +1 -1
- package/dist/v2/runtime/handlers/sse/run.cjs +3 -1
- package/dist/v2/runtime/handlers/sse/run.cjs.map +1 -1
- package/dist/v2/runtime/handlers/sse/run.mjs +3 -1
- package/dist/v2/runtime/handlers/sse/run.mjs.map +1 -1
- package/dist/v2/runtime/intelligence-platform/client.cjs +2 -7
- package/dist/v2/runtime/intelligence-platform/client.cjs.map +1 -1
- package/dist/v2/runtime/intelligence-platform/client.d.cts +1 -4
- package/dist/v2/runtime/intelligence-platform/client.d.cts.map +1 -1
- package/dist/v2/runtime/intelligence-platform/client.d.mts +1 -4
- package/dist/v2/runtime/intelligence-platform/client.d.mts.map +1 -1
- package/dist/v2/runtime/intelligence-platform/client.mjs +2 -7
- package/dist/v2/runtime/intelligence-platform/client.mjs.map +1 -1
- package/dist/v2/runtime/runner/intelligence.cjs +17 -5
- package/dist/v2/runtime/runner/intelligence.cjs.map +1 -1
- package/dist/v2/runtime/runner/intelligence.d.cts +1 -0
- package/dist/v2/runtime/runner/intelligence.d.cts.map +1 -1
- package/dist/v2/runtime/runner/intelligence.d.mts +1 -0
- package/dist/v2/runtime/runner/intelligence.d.mts.map +1 -1
- package/dist/v2/runtime/runner/intelligence.mjs +17 -5
- package/dist/v2/runtime/runner/intelligence.mjs.map +1 -1
- package/package.json +3 -3
- package/src/agent/__tests__/provider-id-collision.test.ts +195 -0
- package/src/agent/index.ts +19 -11
- package/src/agents/langgraph/__tests__/event-source.test.ts +256 -0
- package/src/graphql/resolvers/__tests__/resolve-message-id.test.ts +25 -0
- package/src/graphql/resolvers/copilot.resolver.ts +2 -1
- package/src/graphql/resolvers/resolve-message-id.ts +14 -0
- package/src/lib/integrations/node-http/__tests__/request-duck-type.test.ts +66 -0
- package/src/lib/integrations/node-http/index.ts +15 -1
- package/src/lib/runtime/__tests__/handle-service-adapter.test.ts +108 -0
- package/src/lib/runtime/__tests__/mcp-tools-utils.test.ts +30 -1
- package/src/lib/runtime/__tests__/on-after-request.test.ts +122 -0
- package/src/lib/runtime/__tests__/retry-utils.test.ts +137 -0
- package/src/lib/runtime/agent-integrations/langgraph/__tests__/dispatch-event-filtering.test.ts +190 -0
- package/src/lib/runtime/copilot-runtime.ts +36 -7
- package/src/lib/runtime/mcp-tools-utils.ts +41 -6
- package/src/lib/runtime/retry-utils.ts +41 -1
- package/src/service-adapters/anthropic/anthropic-adapter.ts +22 -2
- package/src/service-adapters/anthropic/utils.ts +60 -1
- package/src/service-adapters/langchain/utils.ts +1 -1
- package/src/service-adapters/openai/openai-adapter.ts +14 -1
- package/src/v2/runtime/__tests__/fetch-router.test.ts +22 -0
- package/src/v2/runtime/__tests__/handle-connect.test.ts +58 -5
- package/src/v2/runtime/__tests__/handle-run.test.ts +31 -4
- package/src/v2/runtime/__tests__/handle-threads.test.ts +66 -4
- package/src/v2/runtime/__tests__/integration/node-servers.integration.test.ts +19 -0
- package/src/v2/runtime/__tests__/integration/suites/debug-events.suite.ts +253 -0
- package/src/v2/runtime/__tests__/middleware-sse-parser.test.ts +50 -0
- package/src/v2/runtime/__tests__/runtime.test.ts +3 -1
- package/src/v2/runtime/core/__tests__/debug-event-bus.test.ts +156 -0
- package/src/v2/runtime/core/debug-event-bus.ts +45 -0
- package/src/v2/runtime/core/fetch-handler.ts +4 -0
- package/src/v2/runtime/core/fetch-router.ts +11 -0
- package/src/v2/runtime/core/hooks.ts +2 -1
- package/src/v2/runtime/core/middleware-sse-parser.ts +12 -2
- package/src/v2/runtime/core/runtime.ts +12 -0
- package/src/v2/runtime/handlers/__tests__/handle-debug-events.test.ts +176 -0
- package/src/v2/runtime/handlers/handle-connect.ts +2 -0
- package/src/v2/runtime/handlers/handle-debug-events.ts +52 -0
- package/src/v2/runtime/handlers/handle-run.ts +1 -0
- package/src/v2/runtime/handlers/intelligence/connect.ts +58 -1
- package/src/v2/runtime/handlers/shared/resolve-intelligence-user.ts +4 -1
- package/src/v2/runtime/handlers/shared/sse-response.ts +46 -0
- package/src/v2/runtime/handlers/sse/__tests__/sse-connect-agent-id.test.ts +71 -0
- package/src/v2/runtime/handlers/sse/connect.ts +6 -0
- package/src/v2/runtime/handlers/sse/run.ts +4 -0
- package/src/v2/runtime/intelligence-platform/__tests__/client.test.ts +13 -11
- package/src/v2/runtime/intelligence-platform/client.ts +3 -11
- package/src/v2/runtime/runner/__tests__/intelligence-runner.test.ts +51 -1
- package/src/v2/runtime/runner/intelligence.ts +27 -9
- package/tests/service-adapters/anthropic/anthropic-adapter.test.ts +268 -0
- package/tests/service-adapters/anthropic/utils-token-trimming.test.ts +301 -0
|
@@ -0,0 +1,301 @@
|
|
|
1
|
+
import { describe, it, expect } from "vitest";
|
|
2
|
+
import { limitMessagesToTokenCount } from "../../../src/service-adapters/anthropic/utils";
|
|
3
|
+
|
|
4
|
+
// Helper to build messages for testing. The token counter is length/3,
|
|
5
|
+
// so we can control token counts via string length.
|
|
6
|
+
|
|
7
|
+
function textUser(text: string) {
|
|
8
|
+
return { role: "user", content: [{ type: "text", text }] };
|
|
9
|
+
}
|
|
10
|
+
|
|
11
|
+
function textAssistant(text: string) {
|
|
12
|
+
return { role: "assistant", content: [{ type: "text", text }] };
|
|
13
|
+
}
|
|
14
|
+
|
|
15
|
+
function toolUseAssistant(id: string, name = "my_tool", input = {}) {
|
|
16
|
+
return {
|
|
17
|
+
role: "assistant",
|
|
18
|
+
content: [{ type: "tool_use", id, name, input }],
|
|
19
|
+
};
|
|
20
|
+
}
|
|
21
|
+
|
|
22
|
+
function toolResultUser(toolUseId: string, content = "result") {
|
|
23
|
+
return {
|
|
24
|
+
role: "user",
|
|
25
|
+
content: [{ type: "tool_result", tool_use_id: toolUseId, content }],
|
|
26
|
+
};
|
|
27
|
+
}
|
|
28
|
+
|
|
29
|
+
function mixedAssistant(blocks: any[]) {
|
|
30
|
+
return { role: "assistant", content: blocks };
|
|
31
|
+
}
|
|
32
|
+
|
|
33
|
+
function mixedUser(blocks: any[]) {
|
|
34
|
+
return { role: "user", content: blocks };
|
|
35
|
+
}
|
|
36
|
+
|
|
37
|
+
describe("limitMessagesToTokenCount - orphan handling", () => {
|
|
38
|
+
// Use a high token limit so trimming doesn't kick in for these tests
|
|
39
|
+
const HIGH_LIMIT = 999999;
|
|
40
|
+
|
|
41
|
+
it("preserves matched tool_use / tool_result pairs", () => {
|
|
42
|
+
const messages = [
|
|
43
|
+
textUser("hello"),
|
|
44
|
+
toolUseAssistant("t1", "tool_a"),
|
|
45
|
+
toolResultUser("t1", "done"),
|
|
46
|
+
textAssistant("ok"),
|
|
47
|
+
];
|
|
48
|
+
|
|
49
|
+
const result = limitMessagesToTokenCount(
|
|
50
|
+
messages,
|
|
51
|
+
[],
|
|
52
|
+
"claude-3",
|
|
53
|
+
HIGH_LIMIT,
|
|
54
|
+
);
|
|
55
|
+
|
|
56
|
+
// All four messages should survive
|
|
57
|
+
expect(result).toHaveLength(4);
|
|
58
|
+
// The tool_use and tool_result should still be present
|
|
59
|
+
const toolUse = result.find(
|
|
60
|
+
(m: any) =>
|
|
61
|
+
m.role === "assistant" &&
|
|
62
|
+
Array.isArray(m.content) &&
|
|
63
|
+
m.content.some((b: any) => b.type === "tool_use"),
|
|
64
|
+
);
|
|
65
|
+
const toolResult = result.find(
|
|
66
|
+
(m: any) =>
|
|
67
|
+
m.role === "user" &&
|
|
68
|
+
Array.isArray(m.content) &&
|
|
69
|
+
m.content.some((b: any) => b.type === "tool_result"),
|
|
70
|
+
);
|
|
71
|
+
expect(toolUse).toBeDefined();
|
|
72
|
+
expect(toolResult).toBeDefined();
|
|
73
|
+
});
|
|
74
|
+
|
|
75
|
+
it("removes orphaned tool_result when tool_use was trimmed", () => {
|
|
76
|
+
// Simulate: tool_use message was removed by token trimming, leaving
|
|
77
|
+
// a tool_result without a matching tool_use.
|
|
78
|
+
const messages = [
|
|
79
|
+
textUser("hello"),
|
|
80
|
+
// no toolUseAssistant for "t1"
|
|
81
|
+
toolResultUser("t1", "orphaned result"),
|
|
82
|
+
textAssistant("ok"),
|
|
83
|
+
];
|
|
84
|
+
|
|
85
|
+
const result = limitMessagesToTokenCount(
|
|
86
|
+
messages,
|
|
87
|
+
[],
|
|
88
|
+
"claude-3",
|
|
89
|
+
HIGH_LIMIT,
|
|
90
|
+
);
|
|
91
|
+
|
|
92
|
+
// The orphaned tool_result message should be gone
|
|
93
|
+
const hasToolResult = result.some(
|
|
94
|
+
(m: any) =>
|
|
95
|
+
m.role === "user" &&
|
|
96
|
+
Array.isArray(m.content) &&
|
|
97
|
+
m.content.some((b: any) => b.type === "tool_result"),
|
|
98
|
+
);
|
|
99
|
+
expect(hasToolResult).toBe(false);
|
|
100
|
+
expect(result).toHaveLength(2); // textUser + textAssistant
|
|
101
|
+
});
|
|
102
|
+
|
|
103
|
+
it("removes orphaned tool_use when tool_result was trimmed", () => {
|
|
104
|
+
// Simulate: tool_result message was removed by token trimming, leaving
|
|
105
|
+
// a tool_use without a matching tool_result.
|
|
106
|
+
const messages = [
|
|
107
|
+
textUser("hello"),
|
|
108
|
+
toolUseAssistant("t1", "tool_a"),
|
|
109
|
+
// no toolResultUser for "t1"
|
|
110
|
+
textAssistant("ok"),
|
|
111
|
+
];
|
|
112
|
+
|
|
113
|
+
const result = limitMessagesToTokenCount(
|
|
114
|
+
messages,
|
|
115
|
+
[],
|
|
116
|
+
"claude-3",
|
|
117
|
+
HIGH_LIMIT,
|
|
118
|
+
);
|
|
119
|
+
|
|
120
|
+
// The orphaned tool_use message should be gone
|
|
121
|
+
const hasToolUse = result.some(
|
|
122
|
+
(m: any) =>
|
|
123
|
+
m.role === "assistant" &&
|
|
124
|
+
Array.isArray(m.content) &&
|
|
125
|
+
m.content.some((b: any) => b.type === "tool_use"),
|
|
126
|
+
);
|
|
127
|
+
expect(hasToolUse).toBe(false);
|
|
128
|
+
expect(result).toHaveLength(2); // textUser + textAssistant
|
|
129
|
+
});
|
|
130
|
+
|
|
131
|
+
it("retains non-orphaned blocks in mixed-content messages", () => {
|
|
132
|
+
// Assistant message has both a text block and an orphaned tool_use
|
|
133
|
+
const messages = [
|
|
134
|
+
textUser("hello"),
|
|
135
|
+
mixedAssistant([
|
|
136
|
+
{ type: "text", text: "thinking..." },
|
|
137
|
+
{ type: "tool_use", id: "t1", name: "tool_a", input: {} },
|
|
138
|
+
]),
|
|
139
|
+
// no tool_result for t1
|
|
140
|
+
textAssistant("done"),
|
|
141
|
+
];
|
|
142
|
+
|
|
143
|
+
const result = limitMessagesToTokenCount(
|
|
144
|
+
messages,
|
|
145
|
+
[],
|
|
146
|
+
"claude-3",
|
|
147
|
+
HIGH_LIMIT,
|
|
148
|
+
);
|
|
149
|
+
|
|
150
|
+
// The assistant message should survive with only the text block
|
|
151
|
+
const assistantMixed = result.find(
|
|
152
|
+
(m: any) =>
|
|
153
|
+
m.role === "assistant" &&
|
|
154
|
+
Array.isArray(m.content) &&
|
|
155
|
+
m.content.some(
|
|
156
|
+
(b: any) => b.type === "text" && b.text === "thinking...",
|
|
157
|
+
),
|
|
158
|
+
);
|
|
159
|
+
expect(assistantMixed).toBeDefined();
|
|
160
|
+
expect(assistantMixed.content).toHaveLength(1);
|
|
161
|
+
expect(assistantMixed.content[0].type).toBe("text");
|
|
162
|
+
});
|
|
163
|
+
|
|
164
|
+
it("retains non-orphaned blocks in mixed user messages", () => {
|
|
165
|
+
// User message has both a text block and an orphaned tool_result
|
|
166
|
+
const messages = [
|
|
167
|
+
textUser("hello"),
|
|
168
|
+
mixedUser([
|
|
169
|
+
{ type: "text", text: "here is context" },
|
|
170
|
+
{ type: "tool_result", tool_use_id: "t_missing", content: "orphan" },
|
|
171
|
+
]),
|
|
172
|
+
textAssistant("ok"),
|
|
173
|
+
];
|
|
174
|
+
|
|
175
|
+
const result = limitMessagesToTokenCount(
|
|
176
|
+
messages,
|
|
177
|
+
[],
|
|
178
|
+
"claude-3",
|
|
179
|
+
HIGH_LIMIT,
|
|
180
|
+
);
|
|
181
|
+
|
|
182
|
+
const userMixed = result.find(
|
|
183
|
+
(m: any) =>
|
|
184
|
+
m.role === "user" &&
|
|
185
|
+
Array.isArray(m.content) &&
|
|
186
|
+
m.content.some(
|
|
187
|
+
(b: any) => b.type === "text" && b.text === "here is context",
|
|
188
|
+
),
|
|
189
|
+
);
|
|
190
|
+
expect(userMixed).toBeDefined();
|
|
191
|
+
expect(userMixed.content).toHaveLength(1);
|
|
192
|
+
expect(userMixed.content[0].type).toBe("text");
|
|
193
|
+
});
|
|
194
|
+
|
|
195
|
+
it("drops message entirely when all blocks are orphaned", () => {
|
|
196
|
+
const messages = [
|
|
197
|
+
textUser("hello"),
|
|
198
|
+
mixedUser([
|
|
199
|
+
{ type: "tool_result", tool_use_id: "t_a", content: "orphan a" },
|
|
200
|
+
{ type: "tool_result", tool_use_id: "t_b", content: "orphan b" },
|
|
201
|
+
]),
|
|
202
|
+
textAssistant("ok"),
|
|
203
|
+
];
|
|
204
|
+
|
|
205
|
+
const result = limitMessagesToTokenCount(
|
|
206
|
+
messages,
|
|
207
|
+
[],
|
|
208
|
+
"claude-3",
|
|
209
|
+
HIGH_LIMIT,
|
|
210
|
+
);
|
|
211
|
+
|
|
212
|
+
expect(result).toHaveLength(2);
|
|
213
|
+
expect(result[0].role).toBe("user");
|
|
214
|
+
expect(result[1].role).toBe("assistant");
|
|
215
|
+
});
|
|
216
|
+
|
|
217
|
+
it("drops assistant message entirely when all tool_use blocks are orphaned", () => {
|
|
218
|
+
const messages = [
|
|
219
|
+
textUser("hello"),
|
|
220
|
+
mixedAssistant([
|
|
221
|
+
{ type: "tool_use", id: "t_x", name: "tool_x", input: {} },
|
|
222
|
+
{ type: "tool_use", id: "t_y", name: "tool_y", input: {} },
|
|
223
|
+
]),
|
|
224
|
+
// no tool_results for either
|
|
225
|
+
textAssistant("done"),
|
|
226
|
+
];
|
|
227
|
+
|
|
228
|
+
const result = limitMessagesToTokenCount(
|
|
229
|
+
messages,
|
|
230
|
+
[],
|
|
231
|
+
"claude-3",
|
|
232
|
+
HIGH_LIMIT,
|
|
233
|
+
);
|
|
234
|
+
|
|
235
|
+
expect(result).toHaveLength(2);
|
|
236
|
+
});
|
|
237
|
+
|
|
238
|
+
it("does not mutate the original messages array or message objects", () => {
|
|
239
|
+
const originalContent = [
|
|
240
|
+
{ type: "text", text: "context" },
|
|
241
|
+
{ type: "tool_result", tool_use_id: "t_orphan", content: "orphan" },
|
|
242
|
+
];
|
|
243
|
+
const userMsg = { role: "user", content: [...originalContent] };
|
|
244
|
+
const messages = [textUser("hello"), userMsg, textAssistant("ok")];
|
|
245
|
+
|
|
246
|
+
const result = limitMessagesToTokenCount(
|
|
247
|
+
messages,
|
|
248
|
+
[],
|
|
249
|
+
"claude-3",
|
|
250
|
+
HIGH_LIMIT,
|
|
251
|
+
);
|
|
252
|
+
|
|
253
|
+
// Original message should still have both blocks
|
|
254
|
+
expect(userMsg.content).toHaveLength(2);
|
|
255
|
+
expect(userMsg.content[1].type).toBe("tool_result");
|
|
256
|
+
|
|
257
|
+
// Original messages array should still have 3 entries
|
|
258
|
+
expect(messages).toHaveLength(3);
|
|
259
|
+
|
|
260
|
+
// Result should have the filtered version
|
|
261
|
+
const filtered = result.find(
|
|
262
|
+
(m: any) =>
|
|
263
|
+
m.role === "user" &&
|
|
264
|
+
Array.isArray(m.content) &&
|
|
265
|
+
m.content.some((b: any) => b.text === "context"),
|
|
266
|
+
);
|
|
267
|
+
expect(filtered).toBeDefined();
|
|
268
|
+
expect(filtered.content).toHaveLength(1);
|
|
269
|
+
});
|
|
270
|
+
|
|
271
|
+
it("handles token trimming that creates orphans via cutoff", () => {
|
|
272
|
+
// Build messages where token trimming will cut off early messages,
|
|
273
|
+
// leaving orphaned tool_result for a tool_use that got trimmed.
|
|
274
|
+
// Each char ~0.33 tokens, so 300 chars ~ 100 tokens
|
|
275
|
+
const longText = "x".repeat(300);
|
|
276
|
+
|
|
277
|
+
const messages = [
|
|
278
|
+
toolUseAssistant("t_old"),
|
|
279
|
+
toolResultUser("t_old", "old result"),
|
|
280
|
+
textUser(longText),
|
|
281
|
+
textAssistant(longText),
|
|
282
|
+
toolUseAssistant("t_new"),
|
|
283
|
+
toolResultUser("t_new", "new result"),
|
|
284
|
+
];
|
|
285
|
+
|
|
286
|
+
// Set a limit that keeps only the last few messages, trimming t_old's tool_use
|
|
287
|
+
const result = limitMessagesToTokenCount(messages, [], "claude-3", 300);
|
|
288
|
+
|
|
289
|
+
// t_old's tool_use should have been trimmed by the token limit,
|
|
290
|
+
// and then t_old's tool_result should be cleaned up as orphaned
|
|
291
|
+
const hasOldResult = result.some(
|
|
292
|
+
(m: any) =>
|
|
293
|
+
m.role === "user" &&
|
|
294
|
+
Array.isArray(m.content) &&
|
|
295
|
+
m.content.some(
|
|
296
|
+
(b: any) => b.type === "tool_result" && b.tool_use_id === "t_old",
|
|
297
|
+
),
|
|
298
|
+
);
|
|
299
|
+
expect(hasOldResult).toBe(false);
|
|
300
|
+
});
|
|
301
|
+
});
|