@kodelyth/codex 2026.5.42 → 2026.6.1
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/package.json +16 -1
- package/doctor-contract-api.test.ts +0 -44
- package/doctor-contract-api.ts +0 -68
- package/harness.ts +0 -72
- package/index.test.ts +0 -230
- package/index.ts +0 -66
- package/media-understanding-provider.test.ts +0 -486
- package/media-understanding-provider.ts +0 -521
- package/prompt-overlay-runtime-contract.test.ts +0 -48
- package/prompt-overlay.ts +0 -21
- package/provider-catalog.ts +0 -83
- package/provider-discovery.ts +0 -45
- package/provider.test.ts +0 -384
- package/provider.ts +0 -243
- package/src/app-server/app-inventory-cache.test.ts +0 -176
- package/src/app-server/app-inventory-cache.ts +0 -324
- package/src/app-server/approval-bridge.test.ts +0 -1471
- package/src/app-server/approval-bridge.ts +0 -1211
- package/src/app-server/auth-bridge.test.ts +0 -1449
- package/src/app-server/auth-bridge.ts +0 -614
- package/src/app-server/auth-profile-runtime-contract.test.ts +0 -239
- package/src/app-server/capabilities.ts +0 -27
- package/src/app-server/client-factory.ts +0 -24
- package/src/app-server/client.test.ts +0 -563
- package/src/app-server/client.ts +0 -715
- package/src/app-server/compact.test.ts +0 -710
- package/src/app-server/compact.ts +0 -500
- package/src/app-server/computer-use.test.ts +0 -788
- package/src/app-server/computer-use.ts +0 -683
- package/src/app-server/config.test.ts +0 -879
- package/src/app-server/config.ts +0 -1038
- package/src/app-server/context-engine-projection.test.ts +0 -252
- package/src/app-server/context-engine-projection.ts +0 -403
- package/src/app-server/delivery-no-reply-runtime-contract.test.ts +0 -80
- package/src/app-server/dynamic-tool-diagnostics.ts +0 -73
- package/src/app-server/dynamic-tool-profile.ts +0 -69
- package/src/app-server/dynamic-tools.test.ts +0 -1302
- package/src/app-server/dynamic-tools.ts +0 -623
- package/src/app-server/elicitation-bridge.test.ts +0 -1056
- package/src/app-server/elicitation-bridge.ts +0 -783
- package/src/app-server/event-projector.test.ts +0 -2668
- package/src/app-server/event-projector.ts +0 -2057
- package/src/app-server/image-payload-sanitizer.test.ts +0 -49
- package/src/app-server/image-payload-sanitizer.ts +0 -167
- package/src/app-server/klaw-owned-tool-runtime-contract.test.ts +0 -456
- package/src/app-server/local-runtime-attribution.ts +0 -39
- package/src/app-server/managed-binary.test.ts +0 -139
- package/src/app-server/managed-binary.ts +0 -193
- package/src/app-server/models.test.ts +0 -246
- package/src/app-server/models.ts +0 -172
- package/src/app-server/native-hook-relay.test.ts +0 -271
- package/src/app-server/native-hook-relay.ts +0 -150
- package/src/app-server/native-subagent-task-mirror.test.ts +0 -573
- package/src/app-server/native-subagent-task-mirror.ts +0 -497
- package/src/app-server/outcome-fallback-runtime-contract.test.ts +0 -404
- package/src/app-server/plugin-activation.test.ts +0 -336
- package/src/app-server/plugin-activation.ts +0 -283
- package/src/app-server/plugin-app-cache-key.ts +0 -74
- package/src/app-server/plugin-approval-roundtrip.ts +0 -122
- package/src/app-server/plugin-inventory.test.ts +0 -355
- package/src/app-server/plugin-inventory.ts +0 -357
- package/src/app-server/plugin-thread-config.test.ts +0 -865
- package/src/app-server/plugin-thread-config.ts +0 -455
- package/src/app-server/protocol-generated/json/DynamicToolCallParams.json +0 -33
- package/src/app-server/protocol-generated/json/v2/ErrorNotification.json +0 -199
- package/src/app-server/protocol-generated/json/v2/GetAccountResponse.json +0 -102
- package/src/app-server/protocol-generated/json/v2/ModelListResponse.json +0 -227
- package/src/app-server/protocol-generated/json/v2/ThreadResumeResponse.json +0 -2630
- package/src/app-server/protocol-generated/json/v2/ThreadStartResponse.json +0 -2630
- package/src/app-server/protocol-generated/json/v2/TurnCompletedNotification.json +0 -1659
- package/src/app-server/protocol-generated/json/v2/TurnStartResponse.json +0 -1655
- package/src/app-server/protocol-validators.test.ts +0 -75
- package/src/app-server/protocol-validators.ts +0 -203
- package/src/app-server/protocol.ts +0 -520
- package/src/app-server/rate-limit-cache.ts +0 -48
- package/src/app-server/rate-limits.test.ts +0 -202
- package/src/app-server/rate-limits.ts +0 -583
- package/src/app-server/request.ts +0 -73
- package/src/app-server/run-attempt.context-engine.test.ts +0 -1004
- package/src/app-server/run-attempt.test.ts +0 -9477
- package/src/app-server/run-attempt.ts +0 -4683
- package/src/app-server/run-attempt.vision-tools.test.ts +0 -35
- package/src/app-server/schema-normalization-runtime-contract.test.ts +0 -206
- package/src/app-server/session-binding.test.ts +0 -303
- package/src/app-server/session-binding.ts +0 -398
- package/src/app-server/session-history.ts +0 -44
- package/src/app-server/shared-client.test.ts +0 -589
- package/src/app-server/shared-client.ts +0 -289
- package/src/app-server/side-question.test.ts +0 -1175
- package/src/app-server/side-question.ts +0 -1007
- package/src/app-server/test-support.ts +0 -48
- package/src/app-server/thread-lifecycle.test.ts +0 -447
- package/src/app-server/thread-lifecycle.ts +0 -939
- package/src/app-server/thread-lifecycle.user-mcp-servers.test.ts +0 -442
- package/src/app-server/timeout.ts +0 -9
- package/src/app-server/tool-progress-normalization.ts +0 -77
- package/src/app-server/trajectory.test.ts +0 -205
- package/src/app-server/trajectory.ts +0 -365
- package/src/app-server/transcript-mirror.test.ts +0 -524
- package/src/app-server/transcript-mirror.ts +0 -208
- package/src/app-server/transcript-repair-runtime-contract.test.ts +0 -44
- package/src/app-server/transport-stdio.test.ts +0 -171
- package/src/app-server/transport-stdio.ts +0 -107
- package/src/app-server/transport-websocket.test.ts +0 -69
- package/src/app-server/transport-websocket.ts +0 -90
- package/src/app-server/transport.ts +0 -117
- package/src/app-server/user-input-bridge.test.ts +0 -249
- package/src/app-server/user-input-bridge.ts +0 -316
- package/src/app-server/version.ts +0 -4
- package/src/app-server/vision-tools.ts +0 -12
- package/src/command-account.ts +0 -544
- package/src/command-formatters.ts +0 -425
- package/src/command-handlers.ts +0 -2004
- package/src/command-rpc.test.ts +0 -16
- package/src/command-rpc.ts +0 -142
- package/src/commands.test.ts +0 -3312
- package/src/commands.ts +0 -65
- package/src/conversation-binding-data.ts +0 -124
- package/src/conversation-binding.test.ts +0 -599
- package/src/conversation-binding.ts +0 -561
- package/src/conversation-control.test.ts +0 -126
- package/src/conversation-control.ts +0 -303
- package/src/conversation-turn-collector.test.ts +0 -191
- package/src/conversation-turn-collector.ts +0 -186
- package/src/conversation-turn-input.test.ts +0 -141
- package/src/conversation-turn-input.ts +0 -106
- package/src/manifest.test.ts +0 -20
- package/src/migration/apply.ts +0 -501
- package/src/migration/helpers.ts +0 -55
- package/src/migration/plan.ts +0 -461
- package/src/migration/provider.test.ts +0 -1741
- package/src/migration/provider.ts +0 -41
- package/src/migration/source.ts +0 -643
- package/src/migration/targets.ts +0 -25
- package/src/node-cli-sessions.test.ts +0 -180
- package/src/node-cli-sessions.ts +0 -711
- package/test-api.ts +0 -82
- package/tsconfig.json +0 -16
|
@@ -1,710 +0,0 @@
|
|
|
1
|
-
import fs from "node:fs/promises";
|
|
2
|
-
import os from "node:os";
|
|
3
|
-
import path from "node:path";
|
|
4
|
-
import {
|
|
5
|
-
embeddedAgentLog,
|
|
6
|
-
type HarnessContextEngine as ContextEngine,
|
|
7
|
-
} from "klaw/plugin-sdk/agent-harness-runtime";
|
|
8
|
-
import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
|
|
9
|
-
import type { CodexAppServerClientFactory } from "./client-factory.js";
|
|
10
|
-
import type { CodexAppServerClient } from "./client.js";
|
|
11
|
-
import { maybeCompactCodexAppServerSession as maybeCompactCodexAppServerSessionImpl } from "./compact.js";
|
|
12
|
-
import type { CodexServerNotification } from "./protocol.js";
|
|
13
|
-
import { readCodexAppServerBinding, writeCodexAppServerBinding } from "./session-binding.js";
|
|
14
|
-
|
|
15
|
-
let tempDir: string;
|
|
16
|
-
let codexAppServerClientFactoryForTest: CodexAppServerClientFactory | undefined;
|
|
17
|
-
|
|
18
|
-
type MaybeCompactOptions = NonNullable<Parameters<typeof maybeCompactCodexAppServerSessionImpl>[1]>;
|
|
19
|
-
|
|
20
|
-
function setCodexAppServerClientFactoryForTest(factory: CodexAppServerClientFactory): void {
|
|
21
|
-
codexAppServerClientFactoryForTest = factory;
|
|
22
|
-
}
|
|
23
|
-
|
|
24
|
-
function resetCodexAppServerClientFactoryForTest(): void {
|
|
25
|
-
codexAppServerClientFactoryForTest = undefined;
|
|
26
|
-
}
|
|
27
|
-
|
|
28
|
-
function maybeCompactCodexAppServerSession(
|
|
29
|
-
params: Parameters<typeof maybeCompactCodexAppServerSessionImpl>[0],
|
|
30
|
-
options: MaybeCompactOptions = {},
|
|
31
|
-
) {
|
|
32
|
-
const clientFactory = options.clientFactory ?? codexAppServerClientFactoryForTest;
|
|
33
|
-
return maybeCompactCodexAppServerSessionImpl(
|
|
34
|
-
params,
|
|
35
|
-
clientFactory ? { ...options, clientFactory } : options,
|
|
36
|
-
);
|
|
37
|
-
}
|
|
38
|
-
|
|
39
|
-
async function writeTestBinding(options: { authProfileId?: string } = {}): Promise<string> {
|
|
40
|
-
const sessionFile = path.join(tempDir, "session.jsonl");
|
|
41
|
-
await writeCodexAppServerBinding(sessionFile, {
|
|
42
|
-
threadId: "thread-1",
|
|
43
|
-
cwd: tempDir,
|
|
44
|
-
...options,
|
|
45
|
-
});
|
|
46
|
-
return sessionFile;
|
|
47
|
-
}
|
|
48
|
-
|
|
49
|
-
function startCompaction(sessionFile: string, options: { currentTokenCount?: number } = {}) {
|
|
50
|
-
return maybeCompactCodexAppServerSession({
|
|
51
|
-
sessionId: "session-1",
|
|
52
|
-
sessionKey: "agent:main:session-1",
|
|
53
|
-
sessionFile,
|
|
54
|
-
workspaceDir: tempDir,
|
|
55
|
-
...options,
|
|
56
|
-
});
|
|
57
|
-
}
|
|
58
|
-
|
|
59
|
-
type CompactResult = NonNullable<Awaited<ReturnType<typeof maybeCompactCodexAppServerSession>>>;
|
|
60
|
-
|
|
61
|
-
function requireCompactResult(result: CompactResult | undefined): CompactResult {
|
|
62
|
-
if (!result) {
|
|
63
|
-
throw new Error("expected compaction result");
|
|
64
|
-
}
|
|
65
|
-
return result;
|
|
66
|
-
}
|
|
67
|
-
|
|
68
|
-
function compactDetails(result: CompactResult): Record<string, unknown> {
|
|
69
|
-
return (result.result?.details ?? {}) as Record<string, unknown>;
|
|
70
|
-
}
|
|
71
|
-
|
|
72
|
-
describe("maybeCompactCodexAppServerSession", () => {
|
|
73
|
-
beforeEach(async () => {
|
|
74
|
-
tempDir = await fs.mkdtemp(path.join(os.tmpdir(), "klaw-codex-compact-"));
|
|
75
|
-
});
|
|
76
|
-
|
|
77
|
-
afterEach(async () => {
|
|
78
|
-
resetCodexAppServerClientFactoryForTest();
|
|
79
|
-
await fs.rm(tempDir, { recursive: true, force: true });
|
|
80
|
-
});
|
|
81
|
-
|
|
82
|
-
it("waits for native app-server compaction before reporting success", async () => {
|
|
83
|
-
const fake = createFakeCodexClient();
|
|
84
|
-
setCodexAppServerClientFactoryForTest(async () => fake.client);
|
|
85
|
-
const sessionFile = await writeTestBinding();
|
|
86
|
-
|
|
87
|
-
const pendingResult = startCompaction(sessionFile, { currentTokenCount: 123 });
|
|
88
|
-
await vi.waitFor(() => {
|
|
89
|
-
expect(fake.request).toHaveBeenCalledWith("thread/compact/start", { threadId: "thread-1" });
|
|
90
|
-
});
|
|
91
|
-
|
|
92
|
-
let settled = false;
|
|
93
|
-
void pendingResult.then(() => {
|
|
94
|
-
settled = true;
|
|
95
|
-
});
|
|
96
|
-
await Promise.resolve();
|
|
97
|
-
expect(settled).toBe(false);
|
|
98
|
-
|
|
99
|
-
fake.emit({
|
|
100
|
-
method: "thread/compacted",
|
|
101
|
-
params: { threadId: "thread-1", turnId: "turn-1" },
|
|
102
|
-
});
|
|
103
|
-
const result = requireCompactResult(await pendingResult);
|
|
104
|
-
|
|
105
|
-
expect(result.ok).toBe(true);
|
|
106
|
-
expect(result.compacted).toBe(true);
|
|
107
|
-
expect(result.result?.tokensBefore).toBe(123);
|
|
108
|
-
const details = compactDetails(result);
|
|
109
|
-
expect(details.backend).toBe("codex-app-server");
|
|
110
|
-
expect(details.threadId).toBe("thread-1");
|
|
111
|
-
expect(details.signal).toBe("thread/compacted");
|
|
112
|
-
expect(details.turnId).toBe("turn-1");
|
|
113
|
-
});
|
|
114
|
-
|
|
115
|
-
it("accepts native context-compaction item completion as success", async () => {
|
|
116
|
-
const fake = createFakeCodexClient();
|
|
117
|
-
setCodexAppServerClientFactoryForTest(async () => fake.client);
|
|
118
|
-
const sessionFile = await writeTestBinding();
|
|
119
|
-
|
|
120
|
-
const pendingResult = startCompaction(sessionFile);
|
|
121
|
-
await vi.waitFor(() => {
|
|
122
|
-
expect(fake.request).toHaveBeenCalledWith("thread/compact/start", { threadId: "thread-1" });
|
|
123
|
-
});
|
|
124
|
-
fake.emit({
|
|
125
|
-
method: "item/completed",
|
|
126
|
-
params: {
|
|
127
|
-
threadId: "thread-1",
|
|
128
|
-
turnId: "turn-1",
|
|
129
|
-
item: { type: "contextCompaction", id: "compact-1" },
|
|
130
|
-
},
|
|
131
|
-
});
|
|
132
|
-
|
|
133
|
-
const result = requireCompactResult(await pendingResult);
|
|
134
|
-
expect(result.ok).toBe(true);
|
|
135
|
-
expect(result.compacted).toBe(true);
|
|
136
|
-
const details = compactDetails(result);
|
|
137
|
-
expect(details.signal).toBe("item/completed");
|
|
138
|
-
expect(details.itemId).toBe("compact-1");
|
|
139
|
-
});
|
|
140
|
-
|
|
141
|
-
it("reuses the bound auth profile for native compaction", async () => {
|
|
142
|
-
const fake = createFakeCodexClient();
|
|
143
|
-
let seenAuthProfileId: string | undefined;
|
|
144
|
-
setCodexAppServerClientFactoryForTest(async (_startOptions, authProfileId) => {
|
|
145
|
-
seenAuthProfileId = authProfileId;
|
|
146
|
-
return fake.client;
|
|
147
|
-
});
|
|
148
|
-
const sessionFile = await writeTestBinding({ authProfileId: "openai-codex:work" });
|
|
149
|
-
|
|
150
|
-
const pendingResult = startCompaction(sessionFile);
|
|
151
|
-
await vi.waitFor(() => {
|
|
152
|
-
expect(fake.request).toHaveBeenCalledWith("thread/compact/start", { threadId: "thread-1" });
|
|
153
|
-
});
|
|
154
|
-
fake.emit({
|
|
155
|
-
method: "thread/compacted",
|
|
156
|
-
params: { threadId: "thread-1", turnId: "turn-1" },
|
|
157
|
-
});
|
|
158
|
-
await pendingResult;
|
|
159
|
-
|
|
160
|
-
expect(seenAuthProfileId).toBe("openai-codex:work");
|
|
161
|
-
});
|
|
162
|
-
|
|
163
|
-
it("warns when stale Klaw compaction overrides are ignored", async () => {
|
|
164
|
-
const warn = vi.spyOn(embeddedAgentLog, "warn").mockImplementation(() => undefined);
|
|
165
|
-
const fake = createFakeCodexClient();
|
|
166
|
-
setCodexAppServerClientFactoryForTest(async () => fake.client);
|
|
167
|
-
const sessionFile = await writeTestBinding();
|
|
168
|
-
|
|
169
|
-
const pendingResult = maybeCompactCodexAppServerSession({
|
|
170
|
-
sessionId: "session-1",
|
|
171
|
-
sessionKey: "agent:main:session-1",
|
|
172
|
-
sessionFile,
|
|
173
|
-
workspaceDir: tempDir,
|
|
174
|
-
config: {
|
|
175
|
-
agents: {
|
|
176
|
-
defaults: {
|
|
177
|
-
compaction: {
|
|
178
|
-
model: "openai/gpt-5.4",
|
|
179
|
-
provider: "custom-summary",
|
|
180
|
-
},
|
|
181
|
-
},
|
|
182
|
-
},
|
|
183
|
-
},
|
|
184
|
-
});
|
|
185
|
-
await vi.waitFor(() => {
|
|
186
|
-
expect(fake.request).toHaveBeenCalledWith("thread/compact/start", { threadId: "thread-1" });
|
|
187
|
-
});
|
|
188
|
-
fake.emit({
|
|
189
|
-
method: "thread/compacted",
|
|
190
|
-
params: { threadId: "thread-1", turnId: "turn-1" },
|
|
191
|
-
});
|
|
192
|
-
await pendingResult;
|
|
193
|
-
|
|
194
|
-
expect(warn).toHaveBeenCalledWith(
|
|
195
|
-
"ignoring Klaw compaction overrides for Codex app-server compaction; Codex uses native server-side compaction",
|
|
196
|
-
{
|
|
197
|
-
sessionId: "session-1",
|
|
198
|
-
sessionKey: "agent:main:session-1",
|
|
199
|
-
ignoredConfig: ["agents.defaults.compaction.model", "agents.defaults.compaction.provider"],
|
|
200
|
-
},
|
|
201
|
-
);
|
|
202
|
-
warn.mockRestore();
|
|
203
|
-
});
|
|
204
|
-
|
|
205
|
-
it("warns when active agent compaction overrides are ignored", async () => {
|
|
206
|
-
const warn = vi.spyOn(embeddedAgentLog, "warn").mockImplementation(() => undefined);
|
|
207
|
-
const fake = createFakeCodexClient();
|
|
208
|
-
setCodexAppServerClientFactoryForTest(async () => fake.client);
|
|
209
|
-
const sessionFile = await writeTestBinding();
|
|
210
|
-
|
|
211
|
-
const pendingResult = maybeCompactCodexAppServerSession({
|
|
212
|
-
sessionId: "session-1",
|
|
213
|
-
sessionKey: "agent:nik:session-1",
|
|
214
|
-
sessionFile,
|
|
215
|
-
workspaceDir: tempDir,
|
|
216
|
-
config: {
|
|
217
|
-
agents: {
|
|
218
|
-
list: [
|
|
219
|
-
{
|
|
220
|
-
id: "nik",
|
|
221
|
-
compaction: {
|
|
222
|
-
model: "openai/gpt-5.4-mini",
|
|
223
|
-
provider: "openai",
|
|
224
|
-
},
|
|
225
|
-
},
|
|
226
|
-
],
|
|
227
|
-
},
|
|
228
|
-
},
|
|
229
|
-
});
|
|
230
|
-
await vi.waitFor(() => {
|
|
231
|
-
expect(fake.request).toHaveBeenCalledWith("thread/compact/start", { threadId: "thread-1" });
|
|
232
|
-
});
|
|
233
|
-
fake.emit({
|
|
234
|
-
method: "thread/compacted",
|
|
235
|
-
params: { threadId: "thread-1", turnId: "turn-1" },
|
|
236
|
-
});
|
|
237
|
-
await pendingResult;
|
|
238
|
-
|
|
239
|
-
expect(warn).toHaveBeenCalledWith(
|
|
240
|
-
"ignoring Klaw compaction overrides for Codex app-server compaction; Codex uses native server-side compaction",
|
|
241
|
-
{
|
|
242
|
-
sessionId: "session-1",
|
|
243
|
-
sessionKey: "agent:nik:session-1",
|
|
244
|
-
ignoredConfig: ["agents.list.nik.compaction.model", "agents.list.nik.compaction.provider"],
|
|
245
|
-
},
|
|
246
|
-
);
|
|
247
|
-
warn.mockRestore();
|
|
248
|
-
});
|
|
249
|
-
|
|
250
|
-
it("reports inherited compaction providers at the source path", async () => {
|
|
251
|
-
const warn = vi.spyOn(embeddedAgentLog, "warn").mockImplementation(() => undefined);
|
|
252
|
-
const fake = createFakeCodexClient();
|
|
253
|
-
setCodexAppServerClientFactoryForTest(async () => fake.client);
|
|
254
|
-
const sessionFile = await writeTestBinding();
|
|
255
|
-
|
|
256
|
-
const pendingResult = maybeCompactCodexAppServerSession({
|
|
257
|
-
sessionId: "session-1",
|
|
258
|
-
sessionKey: "agent:nik:session-1",
|
|
259
|
-
sessionFile,
|
|
260
|
-
workspaceDir: tempDir,
|
|
261
|
-
config: {
|
|
262
|
-
agents: {
|
|
263
|
-
defaults: {
|
|
264
|
-
compaction: {
|
|
265
|
-
provider: "custom-summary",
|
|
266
|
-
},
|
|
267
|
-
},
|
|
268
|
-
list: [
|
|
269
|
-
{
|
|
270
|
-
id: "nik",
|
|
271
|
-
compaction: {
|
|
272
|
-
model: "openai/gpt-5.4-mini",
|
|
273
|
-
},
|
|
274
|
-
},
|
|
275
|
-
],
|
|
276
|
-
},
|
|
277
|
-
},
|
|
278
|
-
});
|
|
279
|
-
await vi.waitFor(() => {
|
|
280
|
-
expect(fake.request).toHaveBeenCalledWith("thread/compact/start", { threadId: "thread-1" });
|
|
281
|
-
});
|
|
282
|
-
fake.emit({
|
|
283
|
-
method: "thread/compacted",
|
|
284
|
-
params: { threadId: "thread-1", turnId: "turn-1" },
|
|
285
|
-
});
|
|
286
|
-
await pendingResult;
|
|
287
|
-
|
|
288
|
-
expect(warn).toHaveBeenCalledWith(
|
|
289
|
-
"ignoring Klaw compaction overrides for Codex app-server compaction; Codex uses native server-side compaction",
|
|
290
|
-
{
|
|
291
|
-
sessionId: "session-1",
|
|
292
|
-
sessionKey: "agent:nik:session-1",
|
|
293
|
-
ignoredConfig: ["agents.defaults.compaction.provider", "agents.list.nik.compaction.model"],
|
|
294
|
-
},
|
|
295
|
-
);
|
|
296
|
-
warn.mockRestore();
|
|
297
|
-
});
|
|
298
|
-
|
|
299
|
-
it("does not warn for legacy Lossless config when the Lossless context engine slot is active", async () => {
|
|
300
|
-
const warn = vi.spyOn(embeddedAgentLog, "warn").mockImplementation(() => undefined);
|
|
301
|
-
const sessionFile = await writeTestBinding();
|
|
302
|
-
const contextEngine: ContextEngine = {
|
|
303
|
-
info: { id: "lcm", name: "Lossless Context Manager", ownsCompaction: true },
|
|
304
|
-
assemble: vi.fn() as never,
|
|
305
|
-
ingest: vi.fn() as never,
|
|
306
|
-
compact: vi.fn(async () => ({ ok: true, compacted: false, reason: "below threshold" })),
|
|
307
|
-
};
|
|
308
|
-
|
|
309
|
-
await maybeCompactCodexAppServerSession({
|
|
310
|
-
sessionId: "session-1",
|
|
311
|
-
sessionKey: "agent:main:session-1",
|
|
312
|
-
sessionFile,
|
|
313
|
-
workspaceDir: tempDir,
|
|
314
|
-
contextEngine,
|
|
315
|
-
config: {
|
|
316
|
-
plugins: {
|
|
317
|
-
slots: {
|
|
318
|
-
contextEngine: "lossless-claw",
|
|
319
|
-
},
|
|
320
|
-
},
|
|
321
|
-
agents: {
|
|
322
|
-
defaults: {
|
|
323
|
-
compaction: {
|
|
324
|
-
model: "openai/gpt-5.4",
|
|
325
|
-
provider: "lossless-claw",
|
|
326
|
-
},
|
|
327
|
-
},
|
|
328
|
-
},
|
|
329
|
-
},
|
|
330
|
-
});
|
|
331
|
-
|
|
332
|
-
expect(warn).not.toHaveBeenCalledWith(
|
|
333
|
-
"ignoring Klaw compaction overrides for Codex app-server compaction; Codex uses native server-side compaction",
|
|
334
|
-
expect.anything(),
|
|
335
|
-
);
|
|
336
|
-
warn.mockRestore();
|
|
337
|
-
});
|
|
338
|
-
|
|
339
|
-
it("does not warn for inherited legacy Lossless provider when the Lossless slot is active", async () => {
|
|
340
|
-
const warn = vi.spyOn(embeddedAgentLog, "warn").mockImplementation(() => undefined);
|
|
341
|
-
const sessionFile = await writeTestBinding();
|
|
342
|
-
const contextEngine: ContextEngine = {
|
|
343
|
-
info: { id: "lcm", name: "Lossless Context Manager", ownsCompaction: true },
|
|
344
|
-
assemble: vi.fn() as never,
|
|
345
|
-
ingest: vi.fn() as never,
|
|
346
|
-
compact: vi.fn(async () => ({ ok: true, compacted: false, reason: "below threshold" })),
|
|
347
|
-
};
|
|
348
|
-
|
|
349
|
-
await maybeCompactCodexAppServerSession({
|
|
350
|
-
sessionId: "session-1",
|
|
351
|
-
sessionKey: "agent:nik:session-1",
|
|
352
|
-
sessionFile,
|
|
353
|
-
workspaceDir: tempDir,
|
|
354
|
-
contextEngine,
|
|
355
|
-
config: {
|
|
356
|
-
plugins: {
|
|
357
|
-
slots: {
|
|
358
|
-
contextEngine: "lossless-claw",
|
|
359
|
-
},
|
|
360
|
-
},
|
|
361
|
-
agents: {
|
|
362
|
-
defaults: {
|
|
363
|
-
compaction: {
|
|
364
|
-
provider: "lossless-claw",
|
|
365
|
-
},
|
|
366
|
-
},
|
|
367
|
-
list: [
|
|
368
|
-
{
|
|
369
|
-
id: "nik",
|
|
370
|
-
compaction: {
|
|
371
|
-
model: "openai/gpt-5.4-mini",
|
|
372
|
-
},
|
|
373
|
-
},
|
|
374
|
-
],
|
|
375
|
-
},
|
|
376
|
-
},
|
|
377
|
-
});
|
|
378
|
-
|
|
379
|
-
expect(warn).not.toHaveBeenCalledWith(
|
|
380
|
-
"ignoring Klaw compaction overrides for Codex app-server compaction; Codex uses native server-side compaction",
|
|
381
|
-
expect.anything(),
|
|
382
|
-
);
|
|
383
|
-
warn.mockRestore();
|
|
384
|
-
});
|
|
385
|
-
|
|
386
|
-
it("fails closed when the persisted binding auth profile disagrees with the runtime request", async () => {
|
|
387
|
-
const fake = createFakeCodexClient();
|
|
388
|
-
const factory = vi.fn(async () => fake.client);
|
|
389
|
-
setCodexAppServerClientFactoryForTest(factory);
|
|
390
|
-
const sessionFile = path.join(tempDir, "session.jsonl");
|
|
391
|
-
await writeCodexAppServerBinding(sessionFile, {
|
|
392
|
-
threadId: "thread-1",
|
|
393
|
-
cwd: tempDir,
|
|
394
|
-
authProfileId: "openai-codex:binding",
|
|
395
|
-
});
|
|
396
|
-
|
|
397
|
-
const result = await maybeCompactCodexAppServerSession({
|
|
398
|
-
sessionId: "session-1",
|
|
399
|
-
sessionKey: "agent:main:session-1",
|
|
400
|
-
sessionFile,
|
|
401
|
-
workspaceDir: tempDir,
|
|
402
|
-
authProfileId: "openai-codex:runtime",
|
|
403
|
-
});
|
|
404
|
-
|
|
405
|
-
expect(result).toEqual({
|
|
406
|
-
ok: false,
|
|
407
|
-
compacted: false,
|
|
408
|
-
reason: "auth profile mismatch for session binding",
|
|
409
|
-
});
|
|
410
|
-
expect(factory).not.toHaveBeenCalled();
|
|
411
|
-
});
|
|
412
|
-
|
|
413
|
-
it("runs owning context-engine compaction and invalidates the Codex thread binding", async () => {
|
|
414
|
-
const info = vi.spyOn(embeddedAgentLog, "info").mockImplementation(() => undefined);
|
|
415
|
-
const sessionFile = await writeTestBinding();
|
|
416
|
-
const compact = vi.fn(async () => ({
|
|
417
|
-
ok: true,
|
|
418
|
-
compacted: true,
|
|
419
|
-
result: {
|
|
420
|
-
summary: "engine summary",
|
|
421
|
-
firstKeptEntryId: "entry-1",
|
|
422
|
-
tokensBefore: 55,
|
|
423
|
-
details: { engine: "lossless-claw" },
|
|
424
|
-
},
|
|
425
|
-
}));
|
|
426
|
-
const maintain = vi.fn(
|
|
427
|
-
async (_params: Parameters<NonNullable<ContextEngine["maintain"]>>[0]) => ({
|
|
428
|
-
changed: false,
|
|
429
|
-
bytesFreed: 0,
|
|
430
|
-
rewrittenEntries: 0,
|
|
431
|
-
}),
|
|
432
|
-
);
|
|
433
|
-
const contextEngine: ContextEngine = {
|
|
434
|
-
info: { id: "lossless-claw", name: "Lossless Claw", ownsCompaction: true },
|
|
435
|
-
assemble: vi.fn() as never,
|
|
436
|
-
ingest: vi.fn() as never,
|
|
437
|
-
compact,
|
|
438
|
-
maintain,
|
|
439
|
-
};
|
|
440
|
-
|
|
441
|
-
const pendingResult = maybeCompactCodexAppServerSession({
|
|
442
|
-
sessionId: "session-1",
|
|
443
|
-
sessionKey: "agent:main:session-1",
|
|
444
|
-
sessionFile,
|
|
445
|
-
workspaceDir: tempDir,
|
|
446
|
-
contextEngine,
|
|
447
|
-
contextTokenBudget: 777,
|
|
448
|
-
contextEngineRuntimeContext: { workspaceDir: tempDir, provider: "codex" },
|
|
449
|
-
currentTokenCount: 123,
|
|
450
|
-
trigger: "manual",
|
|
451
|
-
});
|
|
452
|
-
|
|
453
|
-
const result = requireCompactResult(await pendingResult);
|
|
454
|
-
expect(result.ok).toBe(true);
|
|
455
|
-
expect(result.compacted).toBe(true);
|
|
456
|
-
expect(result.result?.summary).toBe("engine summary");
|
|
457
|
-
expect(result.result?.firstKeptEntryId).toBe("entry-1");
|
|
458
|
-
expect(result.result?.tokensBefore).toBe(55);
|
|
459
|
-
const details = compactDetails(result);
|
|
460
|
-
expect(details.engine).toBe("lossless-claw");
|
|
461
|
-
expect(details.codexThreadBindingInvalidated).toBe(true);
|
|
462
|
-
expect(await readCodexAppServerBinding(sessionFile)).toBeUndefined();
|
|
463
|
-
expect(compact).toHaveBeenCalledTimes(1);
|
|
464
|
-
expect(compact).toHaveBeenCalledWith({
|
|
465
|
-
sessionId: "session-1",
|
|
466
|
-
sessionKey: "agent:main:session-1",
|
|
467
|
-
sessionFile,
|
|
468
|
-
tokenBudget: 777,
|
|
469
|
-
currentTokenCount: 123,
|
|
470
|
-
compactionTarget: "threshold",
|
|
471
|
-
customInstructions: undefined,
|
|
472
|
-
force: true,
|
|
473
|
-
runtimeContext: { workspaceDir: tempDir, provider: "codex" },
|
|
474
|
-
});
|
|
475
|
-
expect(maintain).toHaveBeenCalledTimes(1);
|
|
476
|
-
const [maintainCall] = maintain.mock.calls[0] ?? [];
|
|
477
|
-
const maintainParams = maintainCall as
|
|
478
|
-
| {
|
|
479
|
-
sessionId?: string;
|
|
480
|
-
sessionKey?: string;
|
|
481
|
-
sessionFile?: string;
|
|
482
|
-
runtimeContext?: { workspaceDir?: string; provider?: string };
|
|
483
|
-
}
|
|
484
|
-
| undefined;
|
|
485
|
-
expect(maintainParams?.sessionId).toBe("session-1");
|
|
486
|
-
expect(maintainParams?.sessionKey).toBe("agent:main:session-1");
|
|
487
|
-
expect(maintainParams?.sessionFile).toBe(sessionFile);
|
|
488
|
-
expect(maintainParams?.runtimeContext?.workspaceDir).toBe(tempDir);
|
|
489
|
-
expect(maintainParams?.runtimeContext?.provider).toBe("codex");
|
|
490
|
-
expect(info).toHaveBeenCalledWith(
|
|
491
|
-
"starting context-engine-owned Codex app-server compaction",
|
|
492
|
-
expect.objectContaining({
|
|
493
|
-
sessionId: "session-1",
|
|
494
|
-
sessionKey: "agent:main:session-1",
|
|
495
|
-
engineId: "lossless-claw",
|
|
496
|
-
tokenBudget: 777,
|
|
497
|
-
currentTokenCount: 123,
|
|
498
|
-
trigger: "manual",
|
|
499
|
-
compactionTarget: "threshold",
|
|
500
|
-
force: true,
|
|
501
|
-
}),
|
|
502
|
-
);
|
|
503
|
-
expect(info).toHaveBeenCalledWith(
|
|
504
|
-
"completed context-engine-owned Codex app-server compaction",
|
|
505
|
-
expect.objectContaining({
|
|
506
|
-
sessionId: "session-1",
|
|
507
|
-
sessionKey: "agent:main:session-1",
|
|
508
|
-
engineId: "lossless-claw",
|
|
509
|
-
ok: true,
|
|
510
|
-
compacted: true,
|
|
511
|
-
codexThreadBindingInvalidated: true,
|
|
512
|
-
}),
|
|
513
|
-
);
|
|
514
|
-
});
|
|
515
|
-
|
|
516
|
-
it("adopts successor transcript handles after owning context-engine compaction", async () => {
|
|
517
|
-
const sessionFile = await writeTestBinding();
|
|
518
|
-
const successorFile = path.join(tempDir, "session.compacted.jsonl");
|
|
519
|
-
await writeCodexAppServerBinding(successorFile, {
|
|
520
|
-
threadId: "thread-successor",
|
|
521
|
-
cwd: tempDir,
|
|
522
|
-
});
|
|
523
|
-
const compact = vi.fn(async () => ({
|
|
524
|
-
ok: true,
|
|
525
|
-
compacted: true,
|
|
526
|
-
result: {
|
|
527
|
-
summary: "engine summary",
|
|
528
|
-
firstKeptEntryId: "entry-1",
|
|
529
|
-
tokensBefore: 55,
|
|
530
|
-
sessionId: "session-1-compacted",
|
|
531
|
-
sessionFile: successorFile,
|
|
532
|
-
},
|
|
533
|
-
}));
|
|
534
|
-
const maintain = vi.fn(
|
|
535
|
-
async (_params: Parameters<NonNullable<ContextEngine["maintain"]>>[0]) => ({
|
|
536
|
-
changed: false,
|
|
537
|
-
bytesFreed: 0,
|
|
538
|
-
rewrittenEntries: 0,
|
|
539
|
-
}),
|
|
540
|
-
);
|
|
541
|
-
const contextEngine: ContextEngine = {
|
|
542
|
-
info: { id: "lossless-claw", name: "Lossless Claw", ownsCompaction: true },
|
|
543
|
-
assemble: vi.fn() as never,
|
|
544
|
-
ingest: vi.fn() as never,
|
|
545
|
-
compact,
|
|
546
|
-
maintain,
|
|
547
|
-
};
|
|
548
|
-
|
|
549
|
-
const result = requireCompactResult(
|
|
550
|
-
await maybeCompactCodexAppServerSession({
|
|
551
|
-
sessionId: "session-1",
|
|
552
|
-
sessionKey: "agent:main:session-1",
|
|
553
|
-
sessionFile,
|
|
554
|
-
workspaceDir: tempDir,
|
|
555
|
-
contextEngine,
|
|
556
|
-
}),
|
|
557
|
-
);
|
|
558
|
-
|
|
559
|
-
expect(result.ok).toBe(true);
|
|
560
|
-
expect(result.compacted).toBe(true);
|
|
561
|
-
expect(result.result?.sessionId).toBe("session-1-compacted");
|
|
562
|
-
expect(result.result?.sessionFile).toBe(successorFile);
|
|
563
|
-
expect(await readCodexAppServerBinding(sessionFile)).toBeUndefined();
|
|
564
|
-
expect(await readCodexAppServerBinding(successorFile)).toBeUndefined();
|
|
565
|
-
expect(maintain).toHaveBeenCalledTimes(1);
|
|
566
|
-
const [maintainCall] = maintain.mock.calls[0] ?? [];
|
|
567
|
-
const maintainParams = maintainCall as
|
|
568
|
-
| {
|
|
569
|
-
sessionId?: string;
|
|
570
|
-
sessionFile?: string;
|
|
571
|
-
}
|
|
572
|
-
| undefined;
|
|
573
|
-
expect(maintainParams?.sessionId).toBe("session-1-compacted");
|
|
574
|
-
expect(maintainParams?.sessionFile).toBe(successorFile);
|
|
575
|
-
});
|
|
576
|
-
|
|
577
|
-
it("returns context-engine compaction success when maintenance fails", async () => {
|
|
578
|
-
const sessionFile = await writeTestBinding();
|
|
579
|
-
const compact = vi.fn(async () => ({
|
|
580
|
-
ok: true,
|
|
581
|
-
compacted: true,
|
|
582
|
-
result: {
|
|
583
|
-
summary: "engine summary",
|
|
584
|
-
firstKeptEntryId: "entry-1",
|
|
585
|
-
tokensBefore: 55,
|
|
586
|
-
},
|
|
587
|
-
}));
|
|
588
|
-
const contextEngine: ContextEngine = {
|
|
589
|
-
info: { id: "lossless-claw", name: "Lossless Claw", ownsCompaction: true },
|
|
590
|
-
assemble: vi.fn() as never,
|
|
591
|
-
ingest: vi.fn() as never,
|
|
592
|
-
compact,
|
|
593
|
-
maintain: vi.fn(async () => {
|
|
594
|
-
throw new Error("maintenance boom");
|
|
595
|
-
}),
|
|
596
|
-
};
|
|
597
|
-
|
|
598
|
-
const pendingResult = maybeCompactCodexAppServerSession({
|
|
599
|
-
sessionId: "session-1",
|
|
600
|
-
sessionKey: "agent:main:session-1",
|
|
601
|
-
sessionFile,
|
|
602
|
-
workspaceDir: tempDir,
|
|
603
|
-
contextEngine,
|
|
604
|
-
});
|
|
605
|
-
|
|
606
|
-
const result = requireCompactResult(await pendingResult);
|
|
607
|
-
expect(result.ok).toBe(true);
|
|
608
|
-
expect(result.compacted).toBe(true);
|
|
609
|
-
expect(result.result?.summary).toBe("engine summary");
|
|
610
|
-
const details = compactDetails(result);
|
|
611
|
-
expect(details.codexThreadBindingInvalidated).toBe(true);
|
|
612
|
-
expect(compact).toHaveBeenCalledTimes(1);
|
|
613
|
-
});
|
|
614
|
-
|
|
615
|
-
it("does not require a Codex binding when the owning context engine compacts", async () => {
|
|
616
|
-
const compact = vi.fn(async () => ({
|
|
617
|
-
ok: true,
|
|
618
|
-
compacted: true,
|
|
619
|
-
result: {
|
|
620
|
-
summary: "engine summary",
|
|
621
|
-
firstKeptEntryId: "entry-1",
|
|
622
|
-
tokensBefore: 8,
|
|
623
|
-
},
|
|
624
|
-
}));
|
|
625
|
-
const maintain = vi.fn(async () => ({
|
|
626
|
-
changed: false,
|
|
627
|
-
bytesFreed: 0,
|
|
628
|
-
rewrittenEntries: 0,
|
|
629
|
-
}));
|
|
630
|
-
const contextEngine: ContextEngine = {
|
|
631
|
-
info: { id: "lossless-claw", name: "Lossless Claw", ownsCompaction: true },
|
|
632
|
-
assemble: vi.fn() as never,
|
|
633
|
-
ingest: vi.fn() as never,
|
|
634
|
-
compact,
|
|
635
|
-
maintain,
|
|
636
|
-
};
|
|
637
|
-
|
|
638
|
-
const result = await maybeCompactCodexAppServerSession({
|
|
639
|
-
sessionId: "session-1",
|
|
640
|
-
sessionKey: "agent:main:session-1",
|
|
641
|
-
sessionFile: path.join(tempDir, "missing-binding.jsonl"),
|
|
642
|
-
workspaceDir: tempDir,
|
|
643
|
-
contextEngine,
|
|
644
|
-
});
|
|
645
|
-
|
|
646
|
-
const compactResult = requireCompactResult(result);
|
|
647
|
-
expect(compactResult.ok).toBe(true);
|
|
648
|
-
expect(compactResult.compacted).toBe(true);
|
|
649
|
-
expect(compactResult.result?.summary).toBe("engine summary");
|
|
650
|
-
expect(compact).toHaveBeenCalledTimes(1);
|
|
651
|
-
expect(maintain).toHaveBeenCalledTimes(1);
|
|
652
|
-
});
|
|
653
|
-
|
|
654
|
-
it("does not run context-engine maintenance when owning compaction does not compact", async () => {
|
|
655
|
-
const maintain = vi.fn(async () => ({
|
|
656
|
-
changed: false,
|
|
657
|
-
bytesFreed: 0,
|
|
658
|
-
rewrittenEntries: 0,
|
|
659
|
-
}));
|
|
660
|
-
const contextEngine: ContextEngine = {
|
|
661
|
-
info: { id: "lossless-claw", name: "Lossless Claw", ownsCompaction: true },
|
|
662
|
-
assemble: vi.fn() as never,
|
|
663
|
-
ingest: vi.fn() as never,
|
|
664
|
-
compact: vi.fn(async () => ({
|
|
665
|
-
ok: true,
|
|
666
|
-
compacted: false,
|
|
667
|
-
reason: "below threshold",
|
|
668
|
-
})),
|
|
669
|
-
maintain,
|
|
670
|
-
};
|
|
671
|
-
|
|
672
|
-
const result = await maybeCompactCodexAppServerSession({
|
|
673
|
-
sessionId: "session-1",
|
|
674
|
-
sessionKey: "agent:main:session-1",
|
|
675
|
-
sessionFile: path.join(tempDir, "missing-binding.jsonl"),
|
|
676
|
-
workspaceDir: tempDir,
|
|
677
|
-
contextEngine,
|
|
678
|
-
});
|
|
679
|
-
|
|
680
|
-
const compactResult = requireCompactResult(result);
|
|
681
|
-
expect(compactResult.ok).toBe(true);
|
|
682
|
-
expect(compactResult.compacted).toBe(false);
|
|
683
|
-
expect(compactResult.reason).toBe("below threshold");
|
|
684
|
-
expect(maintain).not.toHaveBeenCalled();
|
|
685
|
-
});
|
|
686
|
-
});
|
|
687
|
-
|
|
688
|
-
function createFakeCodexClient(): {
|
|
689
|
-
client: CodexAppServerClient;
|
|
690
|
-
request: ReturnType<typeof vi.fn>;
|
|
691
|
-
emit: (notification: CodexServerNotification) => void;
|
|
692
|
-
} {
|
|
693
|
-
const handlers = new Set<(notification: CodexServerNotification) => void>();
|
|
694
|
-
const request = vi.fn(async () => ({}));
|
|
695
|
-
return {
|
|
696
|
-
client: {
|
|
697
|
-
request,
|
|
698
|
-
addNotificationHandler(handler: (notification: CodexServerNotification) => void) {
|
|
699
|
-
handlers.add(handler);
|
|
700
|
-
return () => handlers.delete(handler);
|
|
701
|
-
},
|
|
702
|
-
} as unknown as CodexAppServerClient,
|
|
703
|
-
request,
|
|
704
|
-
emit(notification: CodexServerNotification): void {
|
|
705
|
-
for (const handler of handlers) {
|
|
706
|
-
handler(notification);
|
|
707
|
-
}
|
|
708
|
-
},
|
|
709
|
-
};
|
|
710
|
-
}
|