Skip to content

Commit 8c60210

Browse files
committed
simplify isReasoningModel implementation
1 parent 1155090 commit 8c60210

File tree

2 files changed

+3
-13
lines changed

2 files changed

+3
-13
lines changed

packages/openai/src/openai-is-reasoning-model.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,4 +59,4 @@ describe('isReasoningModel', () => {
5959
])('%s supports reasoning: %s', (modelId, expectedStr) => {
6060
expect(isReasoningModel(modelId)).toBe(expectedStr === 'true');
6161
});
62-
});
62+
});

packages/openai/src/openai-is-reasoning-model.ts

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,7 @@
11
export function isReasoningModel(modelId: string) {
2-
const result = funk(modelId);
3-
console.log(`isReasoningModel(${modelId}) => ${result}`);
4-
return result;
5-
}
6-
7-
function funk(modelId: string) {
8-
if (modelId.startsWith('gpt-4o')) return false;
9-
if (modelId.startsWith('chatgpt-4o')) return false;
10-
if (modelId.startsWith('codex')) return true;
11-
if (modelId.startsWith('computer-use')) return true;
12-
13-
if (modelId.startsWith('gpt-4')) return false;
142
if (modelId.startsWith('gpt-3')) return false;
3+
if (modelId.startsWith('gpt-4')) return false;
4+
if (modelId.startsWith('chatgpt-4o')) return false;
155
if (modelId.startsWith('gpt-5-chat')) return false;
166

177
return true;

0 commit comments

Comments
 (0)