Merge pull request #30 from yusuf-eren/bugfix/issues/28

bugfix: import type from OpenAI Node.js library

Author: yusuf-eren

package-lock.json

@@ -37,15 +37,15 @@
     "axios": "^1.9.0",
     "dotenv": "^16.4.7",
     "graphviz": "^0.0.9",
-    "openai": "^4.103.0",
+    "openai": "^5.0.1",
     "uuid": "^11.1.0",
     "zod": "^3.24.2"
   },
   "devDependencies": {
     "@types/dotenv": "^6.1.1",
     "@types/graphviz": "^0.0.39",
     "@types/jest": "^29.5.12",
-    "@types/node": "^22.13.17",
+    "@types/node": "^22.15.24",
     "@types/uuid": "^10.0.0",
     "jest": "^29.7.0",
     "ts-jest": "^29.1.2",

package.json

@@ -1,3 +1,4 @@
+import { Reasoning } from 'openai/resources/shared';
 /**
  * Specifies how tools should be chosen by the model.
  * - 'auto': Model decides when to use tools
@@ -6,38 +7,6 @@
  */
 type ToolChoice = 'auto' | 'required' | 'none';
 
-/**
- * Specifies the level of reasoning effort for o-series models.
- * - 'low': Minimal reasoning effort
- * - 'medium': Balanced reasoning effort
- * - 'high': Maximum reasoning effort
- */
-type ReasoningEffort = 'low' | 'medium' | 'high';
-
-/**
- * Specifies the type of reasoning summary for computer_use_preview.
- * - 'concise': Brief summary
- * - 'detailed': Comprehensive summary
- */
-type ReasoningSummary = 'concise' | 'detailed';
-
-/**
- * Configuration for model reasoning capabilities.
- */
-interface Reasoning {
-  /**
-   * Constrains effort on reasoning for o-series models.
-   * Reducing reasoning effort can result in faster responses and fewer tokens used.
-   */
-  effort?: ReasoningEffort;
-
-  /**
-   * For computer_use_preview only. Generates a summary of the model's reasoning process.
-   * Useful for debugging and understanding the model's decision making.
-   */
-  generate_summary?: ReasoningSummary;
-}
-
 /**
  * Settings to use when calling an LLM.
  *
@@ -102,24 +71,16 @@ export class ModelSettings {
     const changes: Partial<ModelSettings> = {};
 
     // Only copy non-undefined values from override
-    if (override.temperature !== undefined)
-      changes.temperature = override.temperature;
+    if (override.temperature !== undefined) changes.temperature = override.temperature;
     if (override.top_p !== undefined) changes.top_p = override.top_p;
-    if (override.frequency_penalty !== undefined)
-      changes.frequency_penalty = override.frequency_penalty;
-    if (override.presence_penalty !== undefined)
-      changes.presence_penalty = override.presence_penalty;
-    if (override.tool_choice !== undefined)
-      changes.tool_choice = override.tool_choice;
-    if (override.parallel_tool_calls !== undefined)
-      changes.parallel_tool_calls = override.parallel_tool_calls;
-    if (override.max_tokens !== undefined)
-      changes.max_tokens = override.max_tokens;
+    if (override.frequency_penalty !== undefined) changes.frequency_penalty = override.frequency_penalty;
+    if (override.presence_penalty !== undefined) changes.presence_penalty = override.presence_penalty;
+    if (override.tool_choice !== undefined) changes.tool_choice = override.tool_choice;
+    if (override.parallel_tool_calls !== undefined) changes.parallel_tool_calls = override.parallel_tool_calls;
+    if (override.max_tokens !== undefined) changes.max_tokens = override.max_tokens;
     if (override.store !== undefined) changes.store = override.store;
-    if (override.truncation !== undefined)
-      changes.truncation = override.truncation;
-    if (override.reasoning !== undefined)
-      changes.reasoning = override.reasoning;
+    if (override.truncation !== undefined) changes.truncation = override.truncation;
+    if (override.reasoning !== undefined) changes.reasoning = override.reasoning;
     if (override.metadata !== undefined) changes.metadata = override.metadata;
 
     return new ModelSettings({ ...this, ...changes });

src/agents/models/model-settings.ts

@@ -19,7 +19,7 @@ import {
   ResponseTextConfig,
   Tool as OpenAITool,
 } from 'openai/resources/responses/responses';
-import { APIPromise } from 'openai/core';
+import { APIPromise } from 'openai/core/api-promise';
 import { MCPTool } from '../mcp/types';
 import { logger } from '../logger';
 import { withResponseSpan, SpanError, ResponseSpanData, GLOBAL_TRACE_PROVIDER } from '../tracing';