orcs-oss/orcs-code
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test: align codexShim test with strict schema normalization

Browse Source 
Update the stale test expectation to match current behavior where
normalizeSchemaForOpenAI() promotes all properties into required[]
and marks the schema as strict: true.

Same fix as PR #72 — included here so PR #80 passes CI independently.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
gnanam1990
2026-04-02 08:16:51 +05:30
parent 4c9b9f0d5d
commit ac2ea6aeb2
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/services/api/codexShim.test.ts
View File

@@ -72,7 +72,7 @@ describe('Codex provider config', () => {
}) })
describe('Codex request translation', () => { describe('Codex request translation', () => {
test('disables strict mode for tools with optional parameters', () => { test('normalizes optional parameters into strict Responses schemas', () => {
const tools = convertToolsToResponsesTools([ const tools = convertToolsToResponsesTools([
{ {
name: 'Agent', name: 'Agent',
@@ -102,9 +102,10 @@ describe('Codex request translation', () => {
prompt: { type: 'string' }, prompt: { type: 'string' },
subagent_type: { type: 'string' }, subagent_type: { type: 'string' },
}, },
required: ['description', 'prompt'], required: ['description', 'prompt', 'subagent_type'],
additionalProperties: false, additionalProperties: false,
}, },
strict: true,
}, },
]) ])
}) })