wushenghua/hapi
1
0
Fork 0
mirror of https://github.com/wu736139669/hapi.git synced 2026-11-02 23:51:34 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(codex): preserve reasoning effort across mode switches

Browse Source
This commit is contained in:
weishu
2026-07-28 12:20:54 +08:00
parent faf70c64dd
commit f0c7d0b7e7
6 changed files with 110 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files
cli/src/codex/runCodex.test.ts
+25 -2
View File
@@ -211,6 +211,13 @@ describe('runCodex', () => {
expect(mockCodexSession.setServiceTier).not.toHaveBeenCalled()
})
it('does not collapse inherited Codex reasoning effort into explicit default on startup', async () => {
await runCodexImpl({ workingDirectory: '/tmp/project' })
expect(mockCodexSession.setModelReasoningEffort).not.toHaveBeenCalled()
expect(harness.loopArgs[0]?.modelReasoningEffort).toBeUndefined()
})
it('uses lazy bootstrap for a fresh terminal launch', async () => {
await runCodexImpl({ workingDirectory: '/tmp/project' })
@@ -256,7 +263,23 @@ describe('runCodex', () => {
await handler?.({ modelReasoningEffort: 'max' })
await handler?.({ modelReasoningEffort: ' EXTREME ' })
expect(mockCodexSession.setModelReasoningEffort).toHaveBeenNthCalledWith(2, 'max')
expect(mockCodexSession.setModelReasoningEffort).toHaveBeenNthCalledWith(3, 'extreme')
expect(mockCodexSession.setModelReasoningEffort).toHaveBeenNthCalledWith(1, 'max')
expect(mockCodexSession.setModelReasoningEffort).toHaveBeenNthCalledWith(2, 'extreme')
})
it('still persists an explicit reasoning effort reset as null', async () => {
await runCodexImpl({ workingDirectory: '/tmp/project' })
const registration = harness.session.rpcHandlerManager.registerHandler.mock.calls.find(
([method]) => method === RPC_METHODS.SetSessionConfig
)
const handler = registration?.[1] as ((payload: unknown) => Promise<unknown>) | undefined
const result = await handler?.({ modelReasoningEffort: null })
expect(mockCodexSession.setModelReasoningEffort).toHaveBeenCalledTimes(1)
expect(mockCodexSession.setModelReasoningEffort).toHaveBeenCalledWith(null)
expect(result).toEqual({
applied: expect.objectContaining({ modelReasoningEffort: null })
})
})
})