@ai-devkit/agent-manager 0.29.0 → 0.31.0
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/__tests__/adapters/ClaudeCodeAdapter.test.js +1 -342
- package/dist/__tests__/adapters/ClaudeCodeAdapter.test.js.map +1 -1
- package/dist/__tests__/database/DurableAgentsDatabase.test.js +31 -3
- package/dist/__tests__/database/DurableAgentsDatabase.test.js.map +1 -1
- package/dist/__tests__/durable/CodexCliProbe.test.js +129 -0
- package/dist/__tests__/durable/CodexCliProbe.test.js.map +1 -0
- package/dist/__tests__/durable/CodexDurableAgentRepository.test.js +84 -0
- package/dist/__tests__/durable/CodexDurableAgentRepository.test.js.map +1 -0
- package/dist/__tests__/durable/CodexPrintAgent.integration.test.js +126 -0
- package/dist/__tests__/durable/CodexPrintAgent.integration.test.js.map +1 -0
- package/dist/__tests__/durable/CodexPrintAgentService.test.js +271 -0
- package/dist/__tests__/durable/CodexPrintAgentService.test.js.map +1 -0
- package/dist/__tests__/durable/CodexPrintRunner.test.js +393 -0
- package/dist/__tests__/durable/CodexPrintRunner.test.js.map +1 -0
- package/dist/__tests__/print/ClaudeCliProbe.test.js +3 -7
- package/dist/__tests__/print/ClaudeCliProbe.test.js.map +1 -1
- package/dist/__tests__/print/ClaudePrintAgent.integration.test.js +25 -28
- package/dist/__tests__/print/ClaudePrintAgent.integration.test.js.map +1 -1
- package/dist/__tests__/print/ClaudePrintAgentService.test.js +65 -0
- package/dist/__tests__/print/ClaudePrintAgentService.test.js.map +1 -1
- package/dist/__tests__/print/ClaudePrintRunner.test.js +4 -10
- package/dist/__tests__/print/ClaudePrintRunner.test.js.map +1 -1
- package/dist/__tests__/providers/claude/ClaudeAgentMapper.test.js +79 -0
- package/dist/__tests__/providers/claude/ClaudeAgentMapper.test.js.map +1 -0
- package/dist/__tests__/providers/claude/ClaudeSessionLocator.test.js +436 -0
- package/dist/__tests__/providers/claude/ClaudeSessionLocator.test.js.map +1 -0
- package/dist/__tests__/providers/pi/durable/PiCliProbe.test.js +76 -0
- package/dist/__tests__/providers/pi/durable/PiCliProbe.test.js.map +1 -0
- package/dist/__tests__/providers/pi/durable/PiDurableAgentRepository.test.js +43 -0
- package/dist/__tests__/providers/pi/durable/PiDurableAgentRepository.test.js.map +1 -0
- package/dist/__tests__/providers/pi/durable/PiPrintAgentService.test.js +202 -0
- package/dist/__tests__/providers/pi/durable/PiPrintAgentService.test.js.map +1 -0
- package/dist/__tests__/providers/pi/durable/PiPrintProtocol.test.js +119 -0
- package/dist/__tests__/providers/pi/durable/PiPrintProtocol.test.js.map +1 -0
- package/dist/__tests__/providers/pi/durable/PiPrintRunner.test.js +386 -0
- package/dist/__tests__/providers/pi/durable/PiPrintRunner.test.js.map +1 -0
- package/dist/__tests__/providers/pi/durable/PiStreamParser.test.js +34 -0
- package/dist/__tests__/providers/pi/durable/PiStreamParser.test.js.map +1 -0
- package/dist/__tests__/utils/ClaudeSessionParser.test.js +2 -2
- package/dist/__tests__/utils/ClaudeSessionParser.test.js.map +1 -1
- package/dist/adapters/index.d.ts +1 -1
- package/dist/adapters/index.d.ts.map +1 -1
- package/dist/adapters/index.js +1 -1
- package/dist/adapters/index.js.map +1 -1
- package/dist/database/connection.d.ts.map +1 -1
- package/dist/database/connection.js +5 -1
- package/dist/database/connection.js.map +1 -1
- package/dist/database/migrations/004_codex_durable_sessions.sql +1 -0
- package/dist/durable/DurableAgent.d.ts +23 -3
- package/dist/durable/DurableAgent.d.ts.map +1 -1
- package/dist/durable/DurableAgent.js +12 -0
- package/dist/durable/DurableAgent.js.map +1 -1
- package/dist/durable/DurableAgentRepository.d.ts +6 -7
- package/dist/durable/DurableAgentRepository.d.ts.map +1 -1
- package/dist/durable/DurableAgentRepository.js +73 -42
- package/dist/durable/DurableAgentRepository.js.map +1 -1
- package/dist/durable/process.d.ts +8 -0
- package/dist/durable/process.d.ts.map +1 -0
- package/dist/durable/process.js +40 -0
- package/dist/durable/process.js.map +1 -0
- package/dist/durable/run.d.ts +32 -0
- package/dist/durable/run.d.ts.map +1 -0
- package/dist/durable/run.js +46 -0
- package/dist/durable/run.js.map +1 -0
- package/dist/durable/utils.d.ts +19 -0
- package/dist/durable/utils.d.ts.map +1 -0
- package/dist/durable/utils.js +52 -0
- package/dist/durable/utils.js.map +1 -0
- package/dist/index.d.ts +17 -13
- package/dist/index.d.ts.map +1 -1
- package/dist/index.js +9 -8
- package/dist/index.js.map +1 -1
- package/dist/providers/claude/ClaudeAgentMapper.d.ts +21 -0
- package/dist/providers/claude/ClaudeAgentMapper.d.ts.map +1 -0
- package/dist/providers/claude/ClaudeAgentMapper.js +42 -0
- package/dist/providers/claude/ClaudeAgentMapper.js.map +1 -0
- package/dist/providers/claude/ClaudeCodeAdapter.d.ts +29 -0
- package/dist/providers/claude/ClaudeCodeAdapter.d.ts.map +1 -0
- package/dist/providers/claude/ClaudeCodeAdapter.js +129 -0
- package/dist/providers/claude/ClaudeCodeAdapter.js.map +1 -0
- package/dist/{adapters/ClaudeCodeAdapter.d.ts → providers/claude/ClaudeSessionLocator.d.ts} +68 -52
- package/dist/providers/claude/ClaudeSessionLocator.d.ts.map +1 -0
- package/dist/{adapters/ClaudeCodeAdapter.js → providers/claude/ClaudeSessionLocator.js} +71 -189
- package/dist/providers/claude/ClaudeSessionLocator.js.map +1 -0
- package/dist/{utils → providers/claude}/ClaudeSessionParser.d.ts +2 -2
- package/dist/providers/claude/ClaudeSessionParser.d.ts.map +1 -0
- package/dist/{utils → providers/claude}/ClaudeSessionParser.js +1 -1
- package/dist/providers/claude/ClaudeSessionParser.js.map +1 -0
- package/dist/providers/claude/durable/ClaudeCliProbe.d.ts.map +1 -0
- package/dist/{durable → providers/claude/durable}/ClaudeCliProbe.js +4 -9
- package/dist/providers/claude/durable/ClaudeCliProbe.js.map +1 -0
- package/dist/{durable → providers/claude/durable}/ClaudePrintAgentService.d.ts +4 -9
- package/dist/providers/claude/durable/ClaudePrintAgentService.d.ts.map +1 -0
- package/dist/providers/claude/durable/ClaudePrintAgentService.js +45 -0
- package/dist/providers/claude/durable/ClaudePrintAgentService.js.map +1 -0
- package/dist/{durable → providers/claude/durable}/ClaudePrintRunner.d.ts +3 -3
- package/dist/providers/claude/durable/ClaudePrintRunner.d.ts.map +1 -0
- package/dist/providers/claude/durable/ClaudePrintRunner.js +75 -0
- package/dist/providers/claude/durable/ClaudePrintRunner.js.map +1 -0
- package/dist/providers/claude/durable/ClaudeStreamParser.d.ts +17 -0
- package/dist/providers/claude/durable/ClaudeStreamParser.d.ts.map +1 -0
- package/dist/providers/claude/durable/ClaudeStreamParser.js +69 -0
- package/dist/providers/claude/durable/ClaudeStreamParser.js.map +1 -0
- package/dist/providers/codex/durable/CodexCliProbe.d.ts +20 -0
- package/dist/providers/codex/durable/CodexCliProbe.d.ts.map +1 -0
- package/dist/providers/codex/durable/CodexCliProbe.js +61 -0
- package/dist/providers/codex/durable/CodexCliProbe.js.map +1 -0
- package/dist/providers/codex/durable/CodexPrintAgentService.d.ts +40 -0
- package/dist/providers/codex/durable/CodexPrintAgentService.d.ts.map +1 -0
- package/dist/providers/codex/durable/CodexPrintAgentService.js +50 -0
- package/dist/providers/codex/durable/CodexPrintAgentService.js.map +1 -0
- package/dist/providers/codex/durable/CodexPrintRunner.d.ts +34 -0
- package/dist/providers/codex/durable/CodexPrintRunner.d.ts.map +1 -0
- package/dist/providers/codex/durable/CodexPrintRunner.js +78 -0
- package/dist/providers/codex/durable/CodexPrintRunner.js.map +1 -0
- package/dist/providers/codex/durable/CodexStreamParser.d.ts +21 -0
- package/dist/providers/codex/durable/CodexStreamParser.d.ts.map +1 -0
- package/dist/providers/codex/durable/CodexStreamParser.js +86 -0
- package/dist/providers/codex/durable/CodexStreamParser.js.map +1 -0
- package/dist/providers/pi/durable/PiCliProbe.d.ts +20 -0
- package/dist/providers/pi/durable/PiCliProbe.d.ts.map +1 -0
- package/dist/providers/pi/durable/PiCliProbe.js +51 -0
- package/dist/providers/pi/durable/PiCliProbe.js.map +1 -0
- package/dist/providers/pi/durable/PiPrintAgentService.d.ts +39 -0
- package/dist/providers/pi/durable/PiPrintAgentService.d.ts.map +1 -0
- package/dist/providers/pi/durable/PiPrintAgentService.js +47 -0
- package/dist/providers/pi/durable/PiPrintAgentService.js.map +1 -0
- package/dist/providers/pi/durable/PiPrintProtocol.d.ts +4 -0
- package/dist/providers/pi/durable/PiPrintProtocol.d.ts.map +1 -0
- package/dist/providers/pi/durable/PiPrintProtocol.js +44 -0
- package/dist/providers/pi/durable/PiPrintProtocol.js.map +1 -0
- package/dist/providers/pi/durable/PiPrintRunner.d.ts +32 -0
- package/dist/providers/pi/durable/PiPrintRunner.d.ts.map +1 -0
- package/dist/providers/pi/durable/PiPrintRunner.js +61 -0
- package/dist/providers/pi/durable/PiPrintRunner.js.map +1 -0
- package/dist/providers/pi/durable/PiStreamParser.d.ts +21 -0
- package/dist/providers/pi/durable/PiStreamParser.d.ts.map +1 -0
- package/dist/providers/pi/durable/PiStreamParser.js +71 -0
- package/dist/providers/pi/durable/PiStreamParser.js.map +1 -0
- package/package.json +1 -1
- package/src/__tests__/adapters/ClaudeCodeAdapter.test.ts +1 -307
- package/src/__tests__/database/DurableAgentsDatabase.test.ts +28 -3
- package/src/__tests__/durable/CodexCliProbe.test.ts +63 -0
- package/src/__tests__/durable/CodexDurableAgentRepository.test.ts +54 -0
- package/src/__tests__/durable/CodexPrintAgent.integration.test.ts +79 -0
- package/src/__tests__/durable/CodexPrintAgentService.test.ts +139 -0
- package/src/__tests__/durable/CodexPrintRunner.test.ts +189 -0
- package/src/__tests__/fixtures/fake-codex.cjs +45 -0
- package/src/__tests__/print/ClaudeCliProbe.test.ts +3 -7
- package/src/__tests__/print/ClaudePrintAgent.integration.test.ts +11 -21
- package/src/__tests__/print/ClaudePrintAgentService.test.ts +33 -1
- package/src/__tests__/print/ClaudePrintRunner.test.ts +4 -10
- package/src/__tests__/providers/claude/ClaudeAgentMapper.test.ts +84 -0
- package/src/__tests__/providers/claude/ClaudeSessionLocator.test.ts +411 -0
- package/src/__tests__/providers/pi/durable/PiCliProbe.test.ts +38 -0
- package/src/__tests__/providers/pi/durable/PiDurableAgentRepository.test.ts +32 -0
- package/src/__tests__/providers/pi/durable/PiPrintAgentService.test.ts +69 -0
- package/src/__tests__/providers/pi/durable/PiPrintProtocol.test.ts +33 -0
- package/src/__tests__/providers/pi/durable/PiPrintRunner.test.ts +107 -0
- package/src/__tests__/providers/pi/durable/PiStreamParser.test.ts +24 -0
- package/src/__tests__/utils/ClaudeSessionParser.test.ts +2 -2
- package/src/adapters/index.ts +1 -1
- package/src/database/connection.ts +3 -1
- package/src/database/migrations/004_codex_durable_sessions.sql +1 -0
- package/src/durable/DurableAgent.ts +51 -3
- package/src/durable/DurableAgentRepository.ts +60 -31
- package/src/durable/process.ts +28 -0
- package/src/durable/run.ts +74 -0
- package/src/durable/utils.ts +61 -0
- package/src/index.ts +31 -15
- package/src/providers/claude/ClaudeAgentMapper.ts +62 -0
- package/src/providers/claude/ClaudeCodeAdapter.ts +160 -0
- package/src/{adapters/ClaudeCodeAdapter.ts → providers/claude/ClaudeSessionLocator.ts} +97 -239
- package/src/{utils → providers/claude}/ClaudeSessionParser.ts +2 -2
- package/src/{durable → providers/claude/durable}/ClaudeCliProbe.ts +4 -10
- package/src/providers/claude/durable/ClaudePrintAgentService.ts +66 -0
- package/src/providers/claude/durable/ClaudePrintRunner.ts +92 -0
- package/src/providers/claude/durable/ClaudeStreamParser.ts +79 -0
- package/src/providers/codex/durable/CodexCliProbe.ts +60 -0
- package/src/providers/codex/durable/CodexPrintAgentService.ts +65 -0
- package/src/providers/codex/durable/CodexPrintRunner.ts +94 -0
- package/src/providers/codex/durable/CodexStreamParser.ts +104 -0
- package/src/providers/pi/durable/PiCliProbe.ts +48 -0
- package/src/providers/pi/durable/PiPrintAgentService.ts +69 -0
- package/src/providers/pi/durable/PiPrintProtocol.ts +39 -0
- package/src/providers/pi/durable/PiPrintRunner.ts +88 -0
- package/src/providers/pi/durable/PiStreamParser.ts +84 -0
- package/dist/adapters/ClaudeCodeAdapter.d.ts.map +0 -1
- package/dist/adapters/ClaudeCodeAdapter.js.map +0 -1
- package/dist/durable/ClaudeCliProbe.d.ts.map +0 -1
- package/dist/durable/ClaudeCliProbe.js.map +0 -1
- package/dist/durable/ClaudePrintAgentService.d.ts.map +0 -1
- package/dist/durable/ClaudePrintAgentService.js +0 -66
- package/dist/durable/ClaudePrintAgentService.js.map +0 -1
- package/dist/durable/ClaudePrintRunner.d.ts.map +0 -1
- package/dist/durable/ClaudePrintRunner.js +0 -128
- package/dist/durable/ClaudePrintRunner.js.map +0 -1
- package/dist/utils/ClaudeSessionParser.d.ts.map +0 -1
- package/dist/utils/ClaudeSessionParser.js.map +0 -1
- package/src/durable/ClaudePrintAgentService.ts +0 -94
- package/src/durable/ClaudePrintRunner.ts +0 -139
- /package/dist/{durable → providers/claude/durable}/ClaudeCliProbe.d.ts +0 -0
|
@@ -0,0 +1,271 @@
|
|
|
1
|
+
import { describe, expect, it, vi } from 'vitest';
|
|
2
|
+
const SESSION = '22222222-2222-4222-8222-222222222222';
|
|
3
|
+
const base = {
|
|
4
|
+
id: 'id',
|
|
5
|
+
name: 'reviewer',
|
|
6
|
+
provider: 'codex',
|
|
7
|
+
providerSessionId: null,
|
|
8
|
+
sessionHealth: 'uninitialized'
|
|
9
|
+
};
|
|
10
|
+
describe('CodexPrintAgentService', ()=>{
|
|
11
|
+
it('constructs default non-billable dependencies without invoking them', async ()=>{
|
|
12
|
+
const api = await import('../../index.js');
|
|
13
|
+
const Service = api.CodexPrintAgentService;
|
|
14
|
+
expect(new Service().repository).toBeDefined();
|
|
15
|
+
});
|
|
16
|
+
it('validates before provider-aware create and never runs Codex', async ()=>{
|
|
17
|
+
const api = await import('../../index.js');
|
|
18
|
+
expect(api).toHaveProperty('CodexPrintAgentService');
|
|
19
|
+
const probe = {
|
|
20
|
+
validate: vi.fn().mockResolvedValue({
|
|
21
|
+
executable: 'codex',
|
|
22
|
+
version: '0.147.0'
|
|
23
|
+
})
|
|
24
|
+
};
|
|
25
|
+
const repository = {
|
|
26
|
+
create: vi.fn().mockResolvedValue(base),
|
|
27
|
+
list: vi.fn()
|
|
28
|
+
};
|
|
29
|
+
const runner = {
|
|
30
|
+
run: vi.fn()
|
|
31
|
+
};
|
|
32
|
+
const Service = api.CodexPrintAgentService;
|
|
33
|
+
await new Service({
|
|
34
|
+
repository,
|
|
35
|
+
probe,
|
|
36
|
+
runner
|
|
37
|
+
}).create({
|
|
38
|
+
name: 'reviewer',
|
|
39
|
+
cwd: '/project'
|
|
40
|
+
});
|
|
41
|
+
expect(repository.create).toHaveBeenCalledWith({
|
|
42
|
+
name: 'reviewer',
|
|
43
|
+
cwd: '/project',
|
|
44
|
+
provider: 'codex'
|
|
45
|
+
});
|
|
46
|
+
expect(runner.run).not.toHaveBeenCalled();
|
|
47
|
+
});
|
|
48
|
+
it('binds during first send and explicitly resumes later sends', async ()=>{
|
|
49
|
+
const api = await import('../../index.js');
|
|
50
|
+
const repository = {
|
|
51
|
+
list: vi.fn(),
|
|
52
|
+
resolve: vi.fn().mockResolvedValue(base),
|
|
53
|
+
acquireRun: vi.fn().mockResolvedValueOnce({
|
|
54
|
+
agent: base,
|
|
55
|
+
token: 'one'
|
|
56
|
+
}).mockResolvedValueOnce({
|
|
57
|
+
agent: {
|
|
58
|
+
...base,
|
|
59
|
+
providerSessionId: SESSION,
|
|
60
|
+
sessionHealth: 'healthy'
|
|
61
|
+
},
|
|
62
|
+
token: 'two'
|
|
63
|
+
}),
|
|
64
|
+
recordProviderProcess: vi.fn(),
|
|
65
|
+
bindProviderSession: vi.fn(),
|
|
66
|
+
completeRun: vi.fn()
|
|
67
|
+
};
|
|
68
|
+
const runner = {
|
|
69
|
+
run: vi.fn().mockImplementation(async (request)=>{
|
|
70
|
+
await request.onSpawn({
|
|
71
|
+
pid: 42,
|
|
72
|
+
startedAt: 'start'
|
|
73
|
+
});
|
|
74
|
+
await request.onSession(SESSION);
|
|
75
|
+
return {
|
|
76
|
+
sessionId: SESSION,
|
|
77
|
+
result: 'answer',
|
|
78
|
+
messages: [
|
|
79
|
+
'answer'
|
|
80
|
+
],
|
|
81
|
+
exitCode: 0
|
|
82
|
+
};
|
|
83
|
+
})
|
|
84
|
+
};
|
|
85
|
+
const Service = api.CodexPrintAgentService;
|
|
86
|
+
const service = new Service({
|
|
87
|
+
repository,
|
|
88
|
+
probe: {
|
|
89
|
+
validate: vi.fn()
|
|
90
|
+
},
|
|
91
|
+
runner,
|
|
92
|
+
executable: 'fake-codex'
|
|
93
|
+
});
|
|
94
|
+
await service.send('reviewer', 'first');
|
|
95
|
+
await service.send('reviewer', 'later');
|
|
96
|
+
expect(repository.bindProviderSession).toHaveBeenNthCalledWith(1, 'id', 'one', SESSION);
|
|
97
|
+
expect(repository.bindProviderSession).toHaveBeenNthCalledWith(2, 'id', 'two', SESSION);
|
|
98
|
+
expect(repository.completeRun).toHaveBeenCalledWith('id', 'one', expect.objectContaining({
|
|
99
|
+
status: 'succeeded',
|
|
100
|
+
sessionHealth: 'healthy'
|
|
101
|
+
}));
|
|
102
|
+
});
|
|
103
|
+
it('records mismatch separately from unknown failures', async ()=>{
|
|
104
|
+
const api = await import('../../index.js');
|
|
105
|
+
const ErrorType = api.CodexPrintError;
|
|
106
|
+
const completeRun = vi.fn();
|
|
107
|
+
const repository = {
|
|
108
|
+
list: vi.fn(),
|
|
109
|
+
resolve: vi.fn().mockResolvedValue(base),
|
|
110
|
+
acquireRun: vi.fn().mockResolvedValue({
|
|
111
|
+
agent: base,
|
|
112
|
+
token: 'one'
|
|
113
|
+
}),
|
|
114
|
+
recordProviderProcess: vi.fn(),
|
|
115
|
+
bindProviderSession: vi.fn(),
|
|
116
|
+
completeRun
|
|
117
|
+
};
|
|
118
|
+
const Service = api.CodexPrintAgentService;
|
|
119
|
+
const service = new Service({
|
|
120
|
+
repository,
|
|
121
|
+
probe: {
|
|
122
|
+
validate: vi.fn()
|
|
123
|
+
},
|
|
124
|
+
runner: {
|
|
125
|
+
run: vi.fn().mockRejectedValue(new ErrorType('mismatch', 'CODEX_SESSION_MISMATCH'))
|
|
126
|
+
}
|
|
127
|
+
});
|
|
128
|
+
await expect(service.send('reviewer', 'x')).rejects.toMatchObject({
|
|
129
|
+
code: 'CODEX_SESSION_MISMATCH'
|
|
130
|
+
});
|
|
131
|
+
expect(completeRun).toHaveBeenCalledWith('id', 'one', expect.objectContaining({
|
|
132
|
+
sessionHealth: 'mismatch'
|
|
133
|
+
}));
|
|
134
|
+
});
|
|
135
|
+
it('rejects a non-Codex target without acquiring or mutating it', async ()=>{
|
|
136
|
+
const api = await import('../../index.js');
|
|
137
|
+
const Service = api.CodexPrintAgentService;
|
|
138
|
+
const repository = {
|
|
139
|
+
resolve: vi.fn().mockResolvedValue({
|
|
140
|
+
...base,
|
|
141
|
+
provider: 'claude'
|
|
142
|
+
}),
|
|
143
|
+
acquireRun: vi.fn(),
|
|
144
|
+
recordProviderProcess: vi.fn(),
|
|
145
|
+
bindProviderSession: vi.fn(),
|
|
146
|
+
completeRun: vi.fn()
|
|
147
|
+
};
|
|
148
|
+
await expect(new Service({
|
|
149
|
+
repository,
|
|
150
|
+
probe: {
|
|
151
|
+
validate: vi.fn()
|
|
152
|
+
},
|
|
153
|
+
runner: {
|
|
154
|
+
run: vi.fn()
|
|
155
|
+
}
|
|
156
|
+
}).send('reviewer', 'x')).rejects.toMatchObject({
|
|
157
|
+
code: 'CODEX_UNSUPPORTED'
|
|
158
|
+
});
|
|
159
|
+
expect(repository.acquireRun).not.toHaveBeenCalled();
|
|
160
|
+
expect(repository.completeRun).not.toHaveBeenCalled();
|
|
161
|
+
});
|
|
162
|
+
it('does not turn a successful completion write failure into a second completion', async ()=>{
|
|
163
|
+
const api = await import('../../index.js');
|
|
164
|
+
const Service = api.CodexPrintAgentService;
|
|
165
|
+
const completionFailure = new Error('completion failed');
|
|
166
|
+
const repository = {
|
|
167
|
+
resolve: vi.fn().mockResolvedValue(base),
|
|
168
|
+
acquireRun: vi.fn().mockResolvedValue({
|
|
169
|
+
agent: base,
|
|
170
|
+
token: 'one'
|
|
171
|
+
}),
|
|
172
|
+
recordProviderProcess: vi.fn(),
|
|
173
|
+
bindProviderSession: vi.fn(),
|
|
174
|
+
completeRun: vi.fn().mockRejectedValue(completionFailure)
|
|
175
|
+
};
|
|
176
|
+
const runner = {
|
|
177
|
+
run: vi.fn().mockResolvedValue({
|
|
178
|
+
sessionId: SESSION,
|
|
179
|
+
result: 'answer',
|
|
180
|
+
messages: [
|
|
181
|
+
'answer'
|
|
182
|
+
],
|
|
183
|
+
exitCode: 0
|
|
184
|
+
})
|
|
185
|
+
};
|
|
186
|
+
await expect(new Service({
|
|
187
|
+
repository,
|
|
188
|
+
probe: {
|
|
189
|
+
validate: vi.fn()
|
|
190
|
+
},
|
|
191
|
+
runner
|
|
192
|
+
}).send('reviewer', 'x')).rejects.toBe(completionFailure);
|
|
193
|
+
expect(repository.completeRun).toHaveBeenCalledOnce();
|
|
194
|
+
});
|
|
195
|
+
it('records a repository binding conflict as a session mismatch', async ()=>{
|
|
196
|
+
const api = await import('../../index.js');
|
|
197
|
+
const BindingError = api.CodexPrintError;
|
|
198
|
+
const completeRun = vi.fn();
|
|
199
|
+
const repository = {
|
|
200
|
+
list: vi.fn(),
|
|
201
|
+
resolve: vi.fn().mockResolvedValue(base),
|
|
202
|
+
acquireRun: vi.fn().mockResolvedValue({
|
|
203
|
+
agent: base,
|
|
204
|
+
token: 'one'
|
|
205
|
+
}),
|
|
206
|
+
recordProviderProcess: vi.fn(),
|
|
207
|
+
bindProviderSession: vi.fn().mockRejectedValue(new BindingError('binding mismatch', 'CODEX_SESSION_MISMATCH')),
|
|
208
|
+
completeRun
|
|
209
|
+
};
|
|
210
|
+
const runner = {
|
|
211
|
+
run: vi.fn().mockImplementation(async (request)=>{
|
|
212
|
+
await request.onSession(SESSION);
|
|
213
|
+
return {
|
|
214
|
+
sessionId: SESSION,
|
|
215
|
+
result: 'x',
|
|
216
|
+
messages: [
|
|
217
|
+
'x'
|
|
218
|
+
],
|
|
219
|
+
exitCode: 0
|
|
220
|
+
};
|
|
221
|
+
})
|
|
222
|
+
};
|
|
223
|
+
const Service = api.CodexPrintAgentService;
|
|
224
|
+
await expect(new Service({
|
|
225
|
+
repository,
|
|
226
|
+
probe: {
|
|
227
|
+
validate: vi.fn()
|
|
228
|
+
},
|
|
229
|
+
runner
|
|
230
|
+
}).send('reviewer', 'x')).rejects.toMatchObject({
|
|
231
|
+
code: 'CODEX_SESSION_MISMATCH'
|
|
232
|
+
});
|
|
233
|
+
expect(completeRun).toHaveBeenCalledWith('id', 'one', expect.objectContaining({
|
|
234
|
+
sessionHealth: 'mismatch'
|
|
235
|
+
}));
|
|
236
|
+
});
|
|
237
|
+
it('rejects missing and ambiguous records before acquiring a run', async ()=>{
|
|
238
|
+
const api = await import('../../index.js');
|
|
239
|
+
const Service = api.CodexPrintAgentService;
|
|
240
|
+
const repository = {
|
|
241
|
+
list: vi.fn(),
|
|
242
|
+
create: vi.fn(),
|
|
243
|
+
resolve: vi.fn().mockResolvedValueOnce(null).mockResolvedValueOnce([
|
|
244
|
+
base,
|
|
245
|
+
base
|
|
246
|
+
]),
|
|
247
|
+
acquireRun: vi.fn(),
|
|
248
|
+
recordProviderProcess: vi.fn(),
|
|
249
|
+
bindProviderSession: vi.fn(),
|
|
250
|
+
completeRun: vi.fn()
|
|
251
|
+
};
|
|
252
|
+
const service = new Service({
|
|
253
|
+
repository,
|
|
254
|
+
probe: {
|
|
255
|
+
validate: vi.fn()
|
|
256
|
+
},
|
|
257
|
+
runner: {
|
|
258
|
+
run: vi.fn()
|
|
259
|
+
}
|
|
260
|
+
});
|
|
261
|
+
await expect(service.send('missing', 'x')).rejects.toMatchObject({
|
|
262
|
+
code: 'DURABLE_AGENT_NOT_FOUND'
|
|
263
|
+
});
|
|
264
|
+
await expect(service.send('ambiguous', 'x')).rejects.toMatchObject({
|
|
265
|
+
code: 'CODEX_UNSUPPORTED'
|
|
266
|
+
});
|
|
267
|
+
expect(repository.acquireRun).not.toHaveBeenCalled();
|
|
268
|
+
});
|
|
269
|
+
});
|
|
270
|
+
|
|
271
|
+
//# sourceMappingURL=CodexPrintAgentService.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"sources":["../../../src/__tests__/durable/CodexPrintAgentService.test.ts"],"sourcesContent":["import { describe, expect, it, vi } from 'vitest';\n\nconst SESSION = '22222222-2222-4222-8222-222222222222';\nconst base = {\n id: 'id', name: 'reviewer', provider: 'codex', providerSessionId: null, sessionHealth: 'uninitialized',\n};\n\ndescribe('CodexPrintAgentService', () => {\n it('constructs default non-billable dependencies without invoking them', async () => {\n const api = await import('../../index.js') as Record<string, unknown>;\n const Service = api.CodexPrintAgentService as new () => any;\n expect(new Service().repository).toBeDefined();\n });\n\n it('validates before provider-aware create and never runs Codex', async () => {\n const api = await import('../../index.js') as Record<string, unknown>;\n expect(api).toHaveProperty('CodexPrintAgentService');\n const probe = { validate: vi.fn().mockResolvedValue({ executable: 'codex', version: '0.147.0' }) };\n const repository = { create: vi.fn().mockResolvedValue(base), list: vi.fn() };\n const runner = { run: vi.fn() };\n const Service = api.CodexPrintAgentService as new (options: unknown) => any;\n await new Service({ repository, probe, runner }).create({ name: 'reviewer', cwd: '/project' });\n expect(repository.create).toHaveBeenCalledWith({ name: 'reviewer', cwd: '/project', provider: 'codex' });\n expect(runner.run).not.toHaveBeenCalled();\n });\n\n it('binds during first send and explicitly resumes later sends', async () => {\n const api = await import('../../index.js') as Record<string, unknown>;\n const repository = {\n list: vi.fn(),\n resolve: vi.fn().mockResolvedValue(base),\n acquireRun: vi.fn()\n .mockResolvedValueOnce({ agent: base, token: 'one' })\n .mockResolvedValueOnce({ agent: { ...base, providerSessionId: SESSION, sessionHealth: 'healthy' }, token: 'two' }),\n recordProviderProcess: vi.fn(), bindProviderSession: vi.fn(), completeRun: vi.fn(),\n };\n const runner = { run: vi.fn().mockImplementation(async (request) => {\n await request.onSpawn({ pid: 42, startedAt: 'start' });\n await request.onSession(SESSION);\n return { sessionId: SESSION, result: 'answer', messages: ['answer'], exitCode: 0 };\n }) };\n const Service = api.CodexPrintAgentService as new (options: unknown) => any;\n const service = new Service({ repository, probe: { validate: vi.fn() }, runner, executable: 'fake-codex' });\n\n await service.send('reviewer', 'first');\n await service.send('reviewer', 'later');\n\n expect(repository.bindProviderSession).toHaveBeenNthCalledWith(1, 'id', 'one', SESSION);\n expect(repository.bindProviderSession).toHaveBeenNthCalledWith(2, 'id', 'two', SESSION);\n expect(repository.completeRun).toHaveBeenCalledWith('id', 'one', expect.objectContaining({\n status: 'succeeded', sessionHealth: 'healthy',\n }));\n });\n\n it('records mismatch separately from unknown failures', async () => {\n const api = await import('../../index.js') as Record<string, unknown>;\n const ErrorType = api.CodexPrintError as new (message: string, code: string) => Error;\n const completeRun = vi.fn();\n const repository = {\n list: vi.fn(),\n resolve: vi.fn().mockResolvedValue(base), acquireRun: vi.fn().mockResolvedValue({ agent: base, token: 'one' }),\n recordProviderProcess: vi.fn(), bindProviderSession: vi.fn(), completeRun,\n };\n const Service = api.CodexPrintAgentService as new (options: unknown) => any;\n const service = new Service({ repository, probe: { validate: vi.fn() }, runner: {\n run: vi.fn().mockRejectedValue(new ErrorType('mismatch', 'CODEX_SESSION_MISMATCH')),\n } });\n await expect(service.send('reviewer', 'x')).rejects.toMatchObject({ code: 'CODEX_SESSION_MISMATCH' });\n expect(completeRun).toHaveBeenCalledWith('id', 'one', expect.objectContaining({ sessionHealth: 'mismatch' }));\n });\n\n it('rejects a non-Codex target without acquiring or mutating it', async () => {\n const api = await import('../../index.js') as Record<string, unknown>;\n const Service = api.CodexPrintAgentService as new (options: unknown) => any;\n const repository = {\n resolve: vi.fn().mockResolvedValue({ ...base, provider: 'claude' }),\n acquireRun: vi.fn(), recordProviderProcess: vi.fn(), bindProviderSession: vi.fn(), completeRun: vi.fn(),\n };\n\n await expect(new Service({ repository, probe: { validate: vi.fn() }, runner: { run: vi.fn() } })\n .send('reviewer', 'x')).rejects.toMatchObject({ code: 'CODEX_UNSUPPORTED' });\n expect(repository.acquireRun).not.toHaveBeenCalled();\n expect(repository.completeRun).not.toHaveBeenCalled();\n });\n\n it('does not turn a successful completion write failure into a second completion', async () => {\n const api = await import('../../index.js') as Record<string, unknown>;\n const Service = api.CodexPrintAgentService as new (options: unknown) => any;\n const completionFailure = new Error('completion failed');\n const repository = {\n resolve: vi.fn().mockResolvedValue(base),\n acquireRun: vi.fn().mockResolvedValue({ agent: base, token: 'one' }),\n recordProviderProcess: vi.fn(), bindProviderSession: vi.fn(),\n completeRun: vi.fn().mockRejectedValue(completionFailure),\n };\n const runner = { run: vi.fn().mockResolvedValue({\n sessionId: SESSION, result: 'answer', messages: ['answer'], exitCode: 0,\n }) };\n\n await expect(new Service({ repository, probe: { validate: vi.fn() }, runner })\n .send('reviewer', 'x')).rejects.toBe(completionFailure);\n expect(repository.completeRun).toHaveBeenCalledOnce();\n });\n\n it('records a repository binding conflict as a session mismatch', async () => {\n const api = await import('../../index.js') as Record<string, unknown>;\n const BindingError = api.CodexPrintError as new (message: string, code: string) => Error;\n const completeRun = vi.fn();\n const repository = {\n list: vi.fn(), resolve: vi.fn().mockResolvedValue(base),\n acquireRun: vi.fn().mockResolvedValue({ agent: base, token: 'one' }),\n recordProviderProcess: vi.fn(),\n bindProviderSession: vi.fn().mockRejectedValue(new BindingError('binding mismatch', 'CODEX_SESSION_MISMATCH')),\n completeRun,\n };\n const runner = { run: vi.fn().mockImplementation(async (request) => {\n await request.onSession(SESSION);\n return { sessionId: SESSION, result: 'x', messages: ['x'], exitCode: 0 };\n }) };\n const Service = api.CodexPrintAgentService as new (options: unknown) => any;\n\n await expect(new Service({ repository, probe: { validate: vi.fn() }, runner }).send('reviewer', 'x'))\n .rejects.toMatchObject({ code: 'CODEX_SESSION_MISMATCH' });\n expect(completeRun).toHaveBeenCalledWith('id', 'one', expect.objectContaining({ sessionHealth: 'mismatch' }));\n });\n\n it('rejects missing and ambiguous records before acquiring a run', async () => {\n const api = await import('../../index.js') as Record<string, unknown>;\n const Service = api.CodexPrintAgentService as new (options: unknown) => any;\n const repository = {\n list: vi.fn(), create: vi.fn(), resolve: vi.fn().mockResolvedValueOnce(null).mockResolvedValueOnce([base, base]),\n acquireRun: vi.fn(), recordProviderProcess: vi.fn(), bindProviderSession: vi.fn(), completeRun: vi.fn(),\n };\n const service = new Service({ repository, probe: { validate: vi.fn() }, runner: { run: vi.fn() } });\n await expect(service.send('missing', 'x')).rejects.toMatchObject({ code: 'DURABLE_AGENT_NOT_FOUND' });\n await expect(service.send('ambiguous', 'x')).rejects.toMatchObject({ code: 'CODEX_UNSUPPORTED' });\n expect(repository.acquireRun).not.toHaveBeenCalled();\n });\n});\n"],"names":["describe","expect","it","vi","SESSION","base","id","name","provider","providerSessionId","sessionHealth","api","Service","CodexPrintAgentService","repository","toBeDefined","toHaveProperty","probe","validate","fn","mockResolvedValue","executable","version","create","list","runner","run","cwd","toHaveBeenCalledWith","not","toHaveBeenCalled","resolve","acquireRun","mockResolvedValueOnce","agent","token","recordProviderProcess","bindProviderSession","completeRun","mockImplementation","request","onSpawn","pid","startedAt","onSession","sessionId","result","messages","exitCode","service","send","toHaveBeenNthCalledWith","objectContaining","status","ErrorType","CodexPrintError","mockRejectedValue","rejects","toMatchObject","code","completionFailure","Error","toBe","toHaveBeenCalledOnce","BindingError"],"mappings":"AAAA,SAASA,QAAQ,EAAEC,MAAM,EAAEC,EAAE,EAAEC,EAAE,QAAQ,SAAS;AAElD,MAAMC,UAAU;AAChB,MAAMC,OAAO;IACTC,IAAI;IAAMC,MAAM;IAAYC,UAAU;IAASC,mBAAmB;IAAMC,eAAe;AAC3F;AAEAV,SAAS,0BAA0B;IAC/BE,GAAG,sEAAsE;QACrE,MAAMS,MAAM,MAAM,MAAM,CAAC;QACzB,MAAMC,UAAUD,IAAIE,sBAAsB;QAC1CZ,OAAO,IAAIW,UAAUE,UAAU,EAAEC,WAAW;IAChD;IAEAb,GAAG,+DAA+D;QAC9D,MAAMS,MAAM,MAAM,MAAM,CAAC;QACzBV,OAAOU,KAAKK,cAAc,CAAC;QAC3B,MAAMC,QAAQ;YAAEC,UAAUf,GAAGgB,EAAE,GAAGC,iBAAiB,CAAC;gBAAEC,YAAY;gBAASC,SAAS;YAAU;QAAG;QACjG,MAAMR,aAAa;YAAES,QAAQpB,GAAGgB,EAAE,GAAGC,iBAAiB,CAACf;YAAOmB,MAAMrB,GAAGgB,EAAE;QAAG;QAC5E,MAAMM,SAAS;YAAEC,KAAKvB,GAAGgB,EAAE;QAAG;QAC9B,MAAMP,UAAUD,IAAIE,sBAAsB;QAC1C,MAAM,IAAID,QAAQ;YAAEE;YAAYG;YAAOQ;QAAO,GAAGF,MAAM,CAAC;YAAEhB,MAAM;YAAYoB,KAAK;QAAW;QAC5F1B,OAAOa,WAAWS,MAAM,EAAEK,oBAAoB,CAAC;YAAErB,MAAM;YAAYoB,KAAK;YAAYnB,UAAU;QAAQ;QACtGP,OAAOwB,OAAOC,GAAG,EAAEG,GAAG,CAACC,gBAAgB;IAC3C;IAEA5B,GAAG,8DAA8D;QAC7D,MAAMS,MAAM,MAAM,MAAM,CAAC;QACzB,MAAMG,aAAa;YACfU,MAAMrB,GAAGgB,EAAE;YACXY,SAAS5B,GAAGgB,EAAE,GAAGC,iBAAiB,CAACf;YACnC2B,YAAY7B,GAAGgB,EAAE,GACZc,qBAAqB,CAAC;gBAAEC,OAAO7B;gBAAM8B,OAAO;YAAM,GAClDF,qBAAqB,CAAC;gBAAEC,OAAO;oBAAE,GAAG7B,IAAI;oBAAEI,mBAAmBL;oBAASM,eAAe;gBAAU;gBAAGyB,OAAO;YAAM;YACpHC,uBAAuBjC,GAAGgB,EAAE;YAAIkB,qBAAqBlC,GAAGgB,EAAE;YAAImB,aAAanC,GAAGgB,EAAE;QACpF;QACA,MAAMM,SAAS;YAAEC,KAAKvB,GAAGgB,EAAE,GAAGoB,kBAAkB,CAAC,OAAOC;gBACpD,MAAMA,QAAQC,OAAO,CAAC;oBAAEC,KAAK;oBAAIC,WAAW;gBAAQ;gBACpD,MAAMH,QAAQI,SAAS,CAACxC;gBACxB,OAAO;oBAAEyC,WAAWzC;oBAAS0C,QAAQ;oBAAUC,UAAU;wBAAC;qBAAS;oBAAEC,UAAU;gBAAE;YACrF;QAAG;QACH,MAAMpC,UAAUD,IAAIE,sBAAsB;QAC1C,MAAMoC,UAAU,IAAIrC,QAAQ;YAAEE;YAAYG,OAAO;gBAAEC,UAAUf,GAAGgB,EAAE;YAAG;YAAGM;YAAQJ,YAAY;QAAa;QAEzG,MAAM4B,QAAQC,IAAI,CAAC,YAAY;QAC/B,MAAMD,QAAQC,IAAI,CAAC,YAAY;QAE/BjD,OAAOa,WAAWuB,mBAAmB,EAAEc,uBAAuB,CAAC,GAAG,MAAM,OAAO/C;QAC/EH,OAAOa,WAAWuB,mBAAmB,EAAEc,uBAAuB,CAAC,GAAG,MAAM,OAAO/C;QAC/EH,OAAOa,WAAWwB,WAAW,EAAEV,oBAAoB,CAAC,MAAM,OAAO3B,OAAOmD,gBAAgB,CAAC;YACrFC,QAAQ;YAAa3C,eAAe;QACxC;IACJ;IAEAR,GAAG,qDAAqD;QACpD,MAAMS,MAAM,MAAM,MAAM,CAAC;QACzB,MAAM2C,YAAY3C,IAAI4C,eAAe;QACrC,MAAMjB,cAAcnC,GAAGgB,EAAE;QACzB,MAAML,aAAa;YACfU,MAAMrB,GAAGgB,EAAE;YACXY,SAAS5B,GAAGgB,EAAE,GAAGC,iBAAiB,CAACf;YAAO2B,YAAY7B,GAAGgB,EAAE,GAAGC,iBAAiB,CAAC;gBAAEc,OAAO7B;gBAAM8B,OAAO;YAAM;YAC5GC,uBAAuBjC,GAAGgB,EAAE;YAAIkB,qBAAqBlC,GAAGgB,EAAE;YAAImB;QAClE;QACA,MAAM1B,UAAUD,IAAIE,sBAAsB;QAC1C,MAAMoC,UAAU,IAAIrC,QAAQ;YAAEE;YAAYG,OAAO;gBAAEC,UAAUf,GAAGgB,EAAE;YAAG;YAAGM,QAAQ;gBAC5EC,KAAKvB,GAAGgB,EAAE,GAAGqC,iBAAiB,CAAC,IAAIF,UAAU,YAAY;YAC7D;QAAE;QACF,MAAMrD,OAAOgD,QAAQC,IAAI,CAAC,YAAY,MAAMO,OAAO,CAACC,aAAa,CAAC;YAAEC,MAAM;QAAyB;QACnG1D,OAAOqC,aAAaV,oBAAoB,CAAC,MAAM,OAAO3B,OAAOmD,gBAAgB,CAAC;YAAE1C,eAAe;QAAW;IAC9G;IAEAR,GAAG,+DAA+D;QAC9D,MAAMS,MAAM,MAAM,MAAM,CAAC;QACzB,MAAMC,UAAUD,IAAIE,sBAAsB;QAC1C,MAAMC,aAAa;YACfiB,SAAS5B,GAAGgB,EAAE,GAAGC,iBAAiB,CAAC;gBAAE,GAAGf,IAAI;gBAAEG,UAAU;YAAS;YACjEwB,YAAY7B,GAAGgB,EAAE;YAAIiB,uBAAuBjC,GAAGgB,EAAE;YAAIkB,qBAAqBlC,GAAGgB,EAAE;YAAImB,aAAanC,GAAGgB,EAAE;QACzG;QAEA,MAAMlB,OAAO,IAAIW,QAAQ;YAAEE;YAAYG,OAAO;gBAAEC,UAAUf,GAAGgB,EAAE;YAAG;YAAGM,QAAQ;gBAAEC,KAAKvB,GAAGgB,EAAE;YAAG;QAAE,GACzF+B,IAAI,CAAC,YAAY,MAAMO,OAAO,CAACC,aAAa,CAAC;YAAEC,MAAM;QAAoB;QAC9E1D,OAAOa,WAAWkB,UAAU,EAAEH,GAAG,CAACC,gBAAgB;QAClD7B,OAAOa,WAAWwB,WAAW,EAAET,GAAG,CAACC,gBAAgB;IACvD;IAEA5B,GAAG,gFAAgF;QAC/E,MAAMS,MAAM,MAAM,MAAM,CAAC;QACzB,MAAMC,UAAUD,IAAIE,sBAAsB;QAC1C,MAAM+C,oBAAoB,IAAIC,MAAM;QACpC,MAAM/C,aAAa;YACfiB,SAAS5B,GAAGgB,EAAE,GAAGC,iBAAiB,CAACf;YACnC2B,YAAY7B,GAAGgB,EAAE,GAAGC,iBAAiB,CAAC;gBAAEc,OAAO7B;gBAAM8B,OAAO;YAAM;YAClEC,uBAAuBjC,GAAGgB,EAAE;YAAIkB,qBAAqBlC,GAAGgB,EAAE;YAC1DmB,aAAanC,GAAGgB,EAAE,GAAGqC,iBAAiB,CAACI;QAC3C;QACA,MAAMnC,SAAS;YAAEC,KAAKvB,GAAGgB,EAAE,GAAGC,iBAAiB,CAAC;gBAC5CyB,WAAWzC;gBAAS0C,QAAQ;gBAAUC,UAAU;oBAAC;iBAAS;gBAAEC,UAAU;YAC1E;QAAG;QAEH,MAAM/C,OAAO,IAAIW,QAAQ;YAAEE;YAAYG,OAAO;gBAAEC,UAAUf,GAAGgB,EAAE;YAAG;YAAGM;QAAO,GACvEyB,IAAI,CAAC,YAAY,MAAMO,OAAO,CAACK,IAAI,CAACF;QACzC3D,OAAOa,WAAWwB,WAAW,EAAEyB,oBAAoB;IACvD;IAEA7D,GAAG,+DAA+D;QAC9D,MAAMS,MAAM,MAAM,MAAM,CAAC;QACzB,MAAMqD,eAAerD,IAAI4C,eAAe;QACxC,MAAMjB,cAAcnC,GAAGgB,EAAE;QACzB,MAAML,aAAa;YACfU,MAAMrB,GAAGgB,EAAE;YAAIY,SAAS5B,GAAGgB,EAAE,GAAGC,iBAAiB,CAACf;YAClD2B,YAAY7B,GAAGgB,EAAE,GAAGC,iBAAiB,CAAC;gBAAEc,OAAO7B;gBAAM8B,OAAO;YAAM;YAClEC,uBAAuBjC,GAAGgB,EAAE;YAC5BkB,qBAAqBlC,GAAGgB,EAAE,GAAGqC,iBAAiB,CAAC,IAAIQ,aAAa,oBAAoB;YACpF1B;QACJ;QACA,MAAMb,SAAS;YAAEC,KAAKvB,GAAGgB,EAAE,GAAGoB,kBAAkB,CAAC,OAAOC;gBACpD,MAAMA,QAAQI,SAAS,CAACxC;gBACxB,OAAO;oBAAEyC,WAAWzC;oBAAS0C,QAAQ;oBAAKC,UAAU;wBAAC;qBAAI;oBAAEC,UAAU;gBAAE;YAC3E;QAAG;QACH,MAAMpC,UAAUD,IAAIE,sBAAsB;QAE1C,MAAMZ,OAAO,IAAIW,QAAQ;YAAEE;YAAYG,OAAO;gBAAEC,UAAUf,GAAGgB,EAAE;YAAG;YAAGM;QAAO,GAAGyB,IAAI,CAAC,YAAY,MAC3FO,OAAO,CAACC,aAAa,CAAC;YAAEC,MAAM;QAAyB;QAC5D1D,OAAOqC,aAAaV,oBAAoB,CAAC,MAAM,OAAO3B,OAAOmD,gBAAgB,CAAC;YAAE1C,eAAe;QAAW;IAC9G;IAEAR,GAAG,gEAAgE;QAC/D,MAAMS,MAAM,MAAM,MAAM,CAAC;QACzB,MAAMC,UAAUD,IAAIE,sBAAsB;QAC1C,MAAMC,aAAa;YACfU,MAAMrB,GAAGgB,EAAE;YAAII,QAAQpB,GAAGgB,EAAE;YAAIY,SAAS5B,GAAGgB,EAAE,GAAGc,qBAAqB,CAAC,MAAMA,qBAAqB,CAAC;gBAAC5B;gBAAMA;aAAK;YAC/G2B,YAAY7B,GAAGgB,EAAE;YAAIiB,uBAAuBjC,GAAGgB,EAAE;YAAIkB,qBAAqBlC,GAAGgB,EAAE;YAAImB,aAAanC,GAAGgB,EAAE;QACzG;QACA,MAAM8B,UAAU,IAAIrC,QAAQ;YAAEE;YAAYG,OAAO;gBAAEC,UAAUf,GAAGgB,EAAE;YAAG;YAAGM,QAAQ;gBAAEC,KAAKvB,GAAGgB,EAAE;YAAG;QAAE;QACjG,MAAMlB,OAAOgD,QAAQC,IAAI,CAAC,WAAW,MAAMO,OAAO,CAACC,aAAa,CAAC;YAAEC,MAAM;QAA0B;QACnG,MAAM1D,OAAOgD,QAAQC,IAAI,CAAC,aAAa,MAAMO,OAAO,CAACC,aAAa,CAAC;YAAEC,MAAM;QAAoB;QAC/F1D,OAAOa,WAAWkB,UAAU,EAAEH,GAAG,CAACC,gBAAgB;IACtD;AACJ"}
|
|
@@ -0,0 +1,393 @@
|
|
|
1
|
+
import { EventEmitter } from 'node:events';
|
|
2
|
+
import { PassThrough, Writable } from 'node:stream';
|
|
3
|
+
import { describe, expect, it, vi } from 'vitest';
|
|
4
|
+
const SESSION = '22222222-2222-4222-8222-222222222222';
|
|
5
|
+
function agent(providerSessionId = null) {
|
|
6
|
+
return {
|
|
7
|
+
id: '11111111-1111-4111-8111-111111111111',
|
|
8
|
+
name: 'reviewer',
|
|
9
|
+
provider: 'codex',
|
|
10
|
+
mode: 'durable',
|
|
11
|
+
cwd: '/project',
|
|
12
|
+
providerSessionId,
|
|
13
|
+
state: 'running',
|
|
14
|
+
sessionHealth: 'uninitialized',
|
|
15
|
+
createdAt: '',
|
|
16
|
+
updatedAt: '',
|
|
17
|
+
lastActiveAt: null,
|
|
18
|
+
lastResult: null,
|
|
19
|
+
activeRun: null
|
|
20
|
+
};
|
|
21
|
+
}
|
|
22
|
+
function fakeSpawn(lines, exitCode = 0, chunks = false) {
|
|
23
|
+
const promptChunks = [];
|
|
24
|
+
const child = new EventEmitter();
|
|
25
|
+
child.pid = 4242;
|
|
26
|
+
child.stdout = new PassThrough();
|
|
27
|
+
child.stderr = new PassThrough();
|
|
28
|
+
child.kill = vi.fn();
|
|
29
|
+
child.stdin = new Writable({
|
|
30
|
+
write (chunk, _encoding, callback) {
|
|
31
|
+
promptChunks.push(Buffer.from(chunk));
|
|
32
|
+
callback();
|
|
33
|
+
},
|
|
34
|
+
final (callback) {
|
|
35
|
+
const output = lines.join('\n');
|
|
36
|
+
if (chunks) {
|
|
37
|
+
const bytes = Buffer.from(output);
|
|
38
|
+
child.stdout.write(bytes.subarray(0, 7));
|
|
39
|
+
child.stdout.write(bytes.subarray(7));
|
|
40
|
+
} else child.stdout.write(output);
|
|
41
|
+
child.stdout.end();
|
|
42
|
+
queueMicrotask(()=>child.emit('close', exitCode, null));
|
|
43
|
+
callback();
|
|
44
|
+
}
|
|
45
|
+
});
|
|
46
|
+
const spawn = vi.fn(()=>child);
|
|
47
|
+
return {
|
|
48
|
+
child,
|
|
49
|
+
spawn,
|
|
50
|
+
promptChunks
|
|
51
|
+
};
|
|
52
|
+
}
|
|
53
|
+
function events(session = SESSION) {
|
|
54
|
+
return [
|
|
55
|
+
JSON.stringify({
|
|
56
|
+
type: 'thread.started',
|
|
57
|
+
thread_id: session
|
|
58
|
+
}),
|
|
59
|
+
JSON.stringify({
|
|
60
|
+
type: 'turn.started'
|
|
61
|
+
}),
|
|
62
|
+
JSON.stringify({
|
|
63
|
+
type: 'future.event',
|
|
64
|
+
anything: true
|
|
65
|
+
}),
|
|
66
|
+
JSON.stringify({
|
|
67
|
+
type: 'item.completed',
|
|
68
|
+
item: {
|
|
69
|
+
type: 'agent_message',
|
|
70
|
+
text: 'first'
|
|
71
|
+
}
|
|
72
|
+
}),
|
|
73
|
+
JSON.stringify({
|
|
74
|
+
type: 'item.completed',
|
|
75
|
+
item: {
|
|
76
|
+
type: 'agent_message',
|
|
77
|
+
text: 'final'
|
|
78
|
+
}
|
|
79
|
+
}),
|
|
80
|
+
JSON.stringify({
|
|
81
|
+
type: 'turn.completed'
|
|
82
|
+
}),
|
|
83
|
+
''
|
|
84
|
+
];
|
|
85
|
+
}
|
|
86
|
+
async function runner(fixture, maxLineBytes) {
|
|
87
|
+
const api = await import('../../index.js');
|
|
88
|
+
expect(api).toHaveProperty('CodexPrintRunner');
|
|
89
|
+
const Runner = api.CodexPrintRunner;
|
|
90
|
+
return new Runner({
|
|
91
|
+
spawn: fixture.spawn,
|
|
92
|
+
maxLineBytes,
|
|
93
|
+
processInspector: {
|
|
94
|
+
getIdentity: ()=>({
|
|
95
|
+
pid: 4242,
|
|
96
|
+
startedAt: 'provider-start'
|
|
97
|
+
})
|
|
98
|
+
}
|
|
99
|
+
});
|
|
100
|
+
}
|
|
101
|
+
describe('CodexPrintRunner', ()=>{
|
|
102
|
+
it('constructs default process dependencies without spawning', async ()=>{
|
|
103
|
+
const api = await import('../../index.js');
|
|
104
|
+
const Runner = api.CodexPrintRunner;
|
|
105
|
+
expect(new Runner()).toBeDefined();
|
|
106
|
+
});
|
|
107
|
+
it('binds an initial thread before returning ordered assistant output', async ()=>{
|
|
108
|
+
const fixture = fakeSpawn(events(), 0, true);
|
|
109
|
+
const instance = await runner(fixture);
|
|
110
|
+
const order = [];
|
|
111
|
+
const result = await instance.run({
|
|
112
|
+
agent: agent(),
|
|
113
|
+
prompt: 'secret prompt',
|
|
114
|
+
executable: 'fake-codex',
|
|
115
|
+
onSpawn: async ()=>{
|
|
116
|
+
expect(fixture.promptChunks).toHaveLength(0);
|
|
117
|
+
order.push('spawn');
|
|
118
|
+
},
|
|
119
|
+
onSession: async (id)=>{
|
|
120
|
+
expect(id).toBe(SESSION);
|
|
121
|
+
order.push('session');
|
|
122
|
+
}
|
|
123
|
+
});
|
|
124
|
+
expect(order).toEqual([
|
|
125
|
+
'spawn',
|
|
126
|
+
'session'
|
|
127
|
+
]);
|
|
128
|
+
expect(fixture.spawn).toHaveBeenCalledWith('fake-codex', [
|
|
129
|
+
'exec',
|
|
130
|
+
'--json',
|
|
131
|
+
'-'
|
|
132
|
+
], expect.objectContaining({
|
|
133
|
+
cwd: '/project',
|
|
134
|
+
shell: false,
|
|
135
|
+
stdio: [
|
|
136
|
+
'pipe',
|
|
137
|
+
'pipe',
|
|
138
|
+
'pipe'
|
|
139
|
+
]
|
|
140
|
+
}));
|
|
141
|
+
expect(JSON.stringify(fixture.spawn.mock.calls)).not.toContain('secret prompt');
|
|
142
|
+
expect(Buffer.concat(fixture.promptChunks).toString()).toBe('secret prompt');
|
|
143
|
+
expect(result).toEqual({
|
|
144
|
+
sessionId: SESSION,
|
|
145
|
+
result: 'final',
|
|
146
|
+
messages: [
|
|
147
|
+
'first',
|
|
148
|
+
'final'
|
|
149
|
+
],
|
|
150
|
+
exitCode: 0
|
|
151
|
+
});
|
|
152
|
+
});
|
|
153
|
+
it('resumes the exact stored UUID and rejects a mismatch', async ()=>{
|
|
154
|
+
const mismatch = '33333333-3333-4333-8333-333333333333';
|
|
155
|
+
const fixture = fakeSpawn(events(mismatch));
|
|
156
|
+
const instance = await runner(fixture);
|
|
157
|
+
await expect(instance.run({
|
|
158
|
+
agent: agent(SESSION),
|
|
159
|
+
prompt: 'later',
|
|
160
|
+
onSpawn: vi.fn(),
|
|
161
|
+
onSession: vi.fn()
|
|
162
|
+
})).rejects.toMatchObject({
|
|
163
|
+
code: 'CODEX_SESSION_MISMATCH'
|
|
164
|
+
});
|
|
165
|
+
expect(fixture.spawn.mock.calls[0][1]).toEqual([
|
|
166
|
+
'exec',
|
|
167
|
+
'resume',
|
|
168
|
+
'--json',
|
|
169
|
+
SESSION,
|
|
170
|
+
'-'
|
|
171
|
+
]);
|
|
172
|
+
});
|
|
173
|
+
it.each([
|
|
174
|
+
[
|
|
175
|
+
'malformed JSON',
|
|
176
|
+
[
|
|
177
|
+
'{bad\n'
|
|
178
|
+
],
|
|
179
|
+
'CODEX_PROTOCOL'
|
|
180
|
+
],
|
|
181
|
+
[
|
|
182
|
+
'non-object JSON',
|
|
183
|
+
[
|
|
184
|
+
'[]\n'
|
|
185
|
+
],
|
|
186
|
+
'CODEX_PROTOCOL'
|
|
187
|
+
],
|
|
188
|
+
[
|
|
189
|
+
'truncated JSON',
|
|
190
|
+
[
|
|
191
|
+
'{}'
|
|
192
|
+
],
|
|
193
|
+
'CODEX_PROTOCOL'
|
|
194
|
+
],
|
|
195
|
+
[
|
|
196
|
+
'missing thread',
|
|
197
|
+
[
|
|
198
|
+
JSON.stringify({
|
|
199
|
+
type: 'item.completed',
|
|
200
|
+
item: {
|
|
201
|
+
type: 'agent_message',
|
|
202
|
+
text: 'x'
|
|
203
|
+
}
|
|
204
|
+
}),
|
|
205
|
+
JSON.stringify({
|
|
206
|
+
type: 'turn.completed'
|
|
207
|
+
}),
|
|
208
|
+
''
|
|
209
|
+
],
|
|
210
|
+
'CODEX_PROTOCOL'
|
|
211
|
+
],
|
|
212
|
+
[
|
|
213
|
+
'missing assistant',
|
|
214
|
+
[
|
|
215
|
+
JSON.stringify({
|
|
216
|
+
type: 'thread.started',
|
|
217
|
+
thread_id: SESSION
|
|
218
|
+
}),
|
|
219
|
+
JSON.stringify({
|
|
220
|
+
type: 'turn.completed'
|
|
221
|
+
}),
|
|
222
|
+
''
|
|
223
|
+
],
|
|
224
|
+
'CODEX_RESULT_MISSING'
|
|
225
|
+
],
|
|
226
|
+
[
|
|
227
|
+
'missing completion',
|
|
228
|
+
[
|
|
229
|
+
JSON.stringify({
|
|
230
|
+
type: 'thread.started',
|
|
231
|
+
thread_id: SESSION
|
|
232
|
+
}),
|
|
233
|
+
JSON.stringify({
|
|
234
|
+
type: 'item.completed',
|
|
235
|
+
item: {
|
|
236
|
+
type: 'agent_message',
|
|
237
|
+
text: 'x'
|
|
238
|
+
}
|
|
239
|
+
}),
|
|
240
|
+
''
|
|
241
|
+
],
|
|
242
|
+
'CODEX_PROTOCOL'
|
|
243
|
+
]
|
|
244
|
+
])('rejects %s', async (_name, lines, code)=>{
|
|
245
|
+
const fixture = fakeSpawn(lines);
|
|
246
|
+
await expect((await runner(fixture)).run({
|
|
247
|
+
agent: agent(),
|
|
248
|
+
prompt: 'x',
|
|
249
|
+
onSpawn: vi.fn(),
|
|
250
|
+
onSession: vi.fn()
|
|
251
|
+
})).rejects.toMatchObject({
|
|
252
|
+
code
|
|
253
|
+
});
|
|
254
|
+
});
|
|
255
|
+
it('rejects oversized output, non-zero exit, and missing process identity without leaking stderr', async ()=>{
|
|
256
|
+
const oversized = fakeSpawn([
|
|
257
|
+
`${'x'.repeat(20)}\n`
|
|
258
|
+
]);
|
|
259
|
+
await expect((await runner(oversized, 10)).run({
|
|
260
|
+
agent: agent(),
|
|
261
|
+
prompt: 'x',
|
|
262
|
+
onSpawn: vi.fn(),
|
|
263
|
+
onSession: vi.fn()
|
|
264
|
+
})).rejects.toMatchObject({
|
|
265
|
+
code: 'CODEX_PROTOCOL'
|
|
266
|
+
});
|
|
267
|
+
const oversizedWithoutNewline = fakeSpawn([
|
|
268
|
+
'x'.repeat(20)
|
|
269
|
+
]);
|
|
270
|
+
await expect((await runner(oversizedWithoutNewline, 10)).run({
|
|
271
|
+
agent: agent(),
|
|
272
|
+
prompt: 'x',
|
|
273
|
+
onSpawn: vi.fn(),
|
|
274
|
+
onSession: vi.fn()
|
|
275
|
+
})).rejects.toMatchObject({
|
|
276
|
+
code: 'CODEX_PROTOCOL'
|
|
277
|
+
});
|
|
278
|
+
const failed = fakeSpawn(events(), 1);
|
|
279
|
+
failed.child.stderr.end('secret-looking provider diagnostic');
|
|
280
|
+
await expect((await runner(failed)).run({
|
|
281
|
+
agent: agent(),
|
|
282
|
+
prompt: 'x',
|
|
283
|
+
onSpawn: vi.fn(),
|
|
284
|
+
onSession: vi.fn()
|
|
285
|
+
})).rejects.toMatchObject({
|
|
286
|
+
code: 'CODEX_PROCESS'
|
|
287
|
+
});
|
|
288
|
+
const missing = fakeSpawn([]);
|
|
289
|
+
missing.child.pid = undefined;
|
|
290
|
+
await expect((await runner(missing)).run({
|
|
291
|
+
agent: agent(),
|
|
292
|
+
prompt: 'x',
|
|
293
|
+
onSpawn: vi.fn(),
|
|
294
|
+
onSession: vi.fn()
|
|
295
|
+
})).rejects.toMatchObject({
|
|
296
|
+
code: 'CODEX_PROCESS'
|
|
297
|
+
});
|
|
298
|
+
});
|
|
299
|
+
it('rejects invalid or duplicate thread identities', async ()=>{
|
|
300
|
+
for (const lines of [
|
|
301
|
+
[
|
|
302
|
+
JSON.stringify({
|
|
303
|
+
type: 'thread.started',
|
|
304
|
+
thread_id: 'bad'
|
|
305
|
+
}),
|
|
306
|
+
''
|
|
307
|
+
],
|
|
308
|
+
[
|
|
309
|
+
JSON.stringify({
|
|
310
|
+
type: 'thread.started',
|
|
311
|
+
thread_id: SESSION
|
|
312
|
+
}),
|
|
313
|
+
JSON.stringify({
|
|
314
|
+
type: 'thread.started',
|
|
315
|
+
thread_id: SESSION
|
|
316
|
+
}),
|
|
317
|
+
''
|
|
318
|
+
]
|
|
319
|
+
]){
|
|
320
|
+
const fixture = fakeSpawn(lines);
|
|
321
|
+
await expect((await runner(fixture)).run({
|
|
322
|
+
agent: agent(),
|
|
323
|
+
prompt: 'x',
|
|
324
|
+
onSpawn: vi.fn(),
|
|
325
|
+
onSession: vi.fn()
|
|
326
|
+
})).rejects.toMatchObject({
|
|
327
|
+
code: 'CODEX_PROTOCOL'
|
|
328
|
+
});
|
|
329
|
+
}
|
|
330
|
+
});
|
|
331
|
+
it('classifies callback processing failure and kills when spawn persistence fails', async ()=>{
|
|
332
|
+
const callbackFailure = fakeSpawn(events());
|
|
333
|
+
await expect((await runner(callbackFailure)).run({
|
|
334
|
+
agent: agent(),
|
|
335
|
+
prompt: 'x',
|
|
336
|
+
onSpawn: vi.fn(),
|
|
337
|
+
onSession: vi.fn().mockRejectedValue(new Error('storage unavailable'))
|
|
338
|
+
})).rejects.toMatchObject({
|
|
339
|
+
code: 'CODEX_PROTOCOL'
|
|
340
|
+
});
|
|
341
|
+
const spawnFailure = fakeSpawn([]);
|
|
342
|
+
const failure = new Error('cannot persist process');
|
|
343
|
+
await expect((await runner(spawnFailure)).run({
|
|
344
|
+
agent: agent(),
|
|
345
|
+
prompt: 'x',
|
|
346
|
+
onSpawn: vi.fn().mockRejectedValue(failure),
|
|
347
|
+
onSession: vi.fn()
|
|
348
|
+
})).rejects.toBe(failure);
|
|
349
|
+
expect(spawnFailure.child.kill).toHaveBeenCalledOnce();
|
|
350
|
+
});
|
|
351
|
+
it('rejects an unverifiable positive PID', async ()=>{
|
|
352
|
+
const api = await import('../../index.js');
|
|
353
|
+
const fixture = fakeSpawn([]);
|
|
354
|
+
const Runner = api.CodexPrintRunner;
|
|
355
|
+
const instance = new Runner({
|
|
356
|
+
spawn: fixture.spawn,
|
|
357
|
+
processInspector: {
|
|
358
|
+
getIdentity: ()=>null
|
|
359
|
+
}
|
|
360
|
+
});
|
|
361
|
+
await expect(instance.run({
|
|
362
|
+
agent: agent(),
|
|
363
|
+
prompt: 'x',
|
|
364
|
+
onSpawn: vi.fn(),
|
|
365
|
+
onSession: vi.fn()
|
|
366
|
+
})).rejects.toMatchObject({
|
|
367
|
+
code: 'CODEX_PROCESS'
|
|
368
|
+
});
|
|
369
|
+
expect(fixture.child.kill).toHaveBeenCalledOnce();
|
|
370
|
+
});
|
|
371
|
+
it('classifies a child spawn error as a process failure', async ()=>{
|
|
372
|
+
const fixture = fakeSpawn([]);
|
|
373
|
+
fixture.child.stdin = new Writable({
|
|
374
|
+
write (_chunk, _encoding, callback) {
|
|
375
|
+
callback();
|
|
376
|
+
},
|
|
377
|
+
final (callback) {
|
|
378
|
+
queueMicrotask(()=>fixture.child.emit('error', new Error('spawn failed')));
|
|
379
|
+
callback();
|
|
380
|
+
}
|
|
381
|
+
});
|
|
382
|
+
await expect((await runner(fixture)).run({
|
|
383
|
+
agent: agent(),
|
|
384
|
+
prompt: 'x',
|
|
385
|
+
onSpawn: vi.fn(),
|
|
386
|
+
onSession: vi.fn()
|
|
387
|
+
})).rejects.toMatchObject({
|
|
388
|
+
code: 'CODEX_PROCESS'
|
|
389
|
+
});
|
|
390
|
+
});
|
|
391
|
+
});
|
|
392
|
+
|
|
393
|
+
//# sourceMappingURL=CodexPrintRunner.test.js.map
|