@codex-native/sdk 0.0.24 → 0.0.26

package/dist/index.d.mts

@@ -1190,7 +1190,7 @@ type ToolExecutorResult = string | NativeToolResult | {
     output?: string;
     error?: string;
     success?: boolean;
-} | void;
+} | Record<string, unknown> | unknown[] | number | boolean | null | void;
 type ToolExecutor = (context: ToolExecutionContext) => Promise<ToolExecutorResult> | ToolExecutorResult;
 declare function registerCodexToolExecutor(name: string, executor: ToolExecutor): void;
 declare function getCodexToolExecutor(name: string): ToolExecutor | undefined;

package/dist/index.d.ts

@@ -1190,7 +1190,7 @@ type ToolExecutorResult = string | NativeToolResult | {
     output?: string;
     error?: string;
     success?: boolean;
-} | void;
+} | Record<string, unknown> | unknown[] | number | boolean | null | void;
 type ToolExecutor = (context: ToolExecutionContext) => Promise<ToolExecutorResult> | ToolExecutorResult;
 declare function registerCodexToolExecutor(name: string, executor: ToolExecutor): void;
 declare function getCodexToolExecutor(name: string): ToolExecutor | undefined;

package/dist/index.mjs

@@ -163,6 +163,11 @@ var CodexModel = class {
   getThreadOptions() {
     return {
       model: this.modelName,
+      // IMPORTANT: ModelProvider must be forwarded to thread options, otherwise
+      // codex-rs will fall back to the default provider from ~/.codex/config.toml
+      // (or another discovered config). This was causing Agents runs to ignore
+      // `new CodexProvider({ modelProvider: "github" })` and hit the wrong backend.
+      modelProvider: this.options.modelProvider,
       // When a custom baseUrl is provided (e.g., test proxy), do not enable OSS mode,
       // since the backend is not Ollama in that case.
       oss: this.options.baseUrl ? false : this.options.oss,
@@ -264,10 +269,21 @@ var CodexModel = class {
         };
         this.codex.registerTool(nativeToolDef);
         this.registeredTools.add(tool2.name);
-        console.log(`Registered tool with Codex: ${tool2.name}`);
+        const DEBUG = process.env.DEBUG === "1" || process.env.DEBUG === "true";
+        if (DEBUG) {
+          try {
+            process.stderr.write(`[codex-native] Registered tool with Codex: ${tool2.name}
+`);
+          } catch {
+          }
+        }
       } catch (error) {
         const errorMessage = `Failed to register tool ${tool2.name}: ${error instanceof Error ? error.message : String(error)}`;
-        console.error(errorMessage);
+        try {
+          process.stderr.write(`[codex-native] ${errorMessage}
+`);
+        } catch {
+        }
       }
     }
   }
@@ -292,11 +308,21 @@ var CodexModel = class {
    * but the framework integration is not yet complete.
    */
   async executeToolViaFramework(invocation) {
-    console.log("[DEBUG executeToolViaFramework] invocation:", JSON.stringify(invocation, null, 2));
-    console.log("[DEBUG executeToolViaFramework] invocation type:", typeof invocation);
-    console.log("[DEBUG executeToolViaFramework] invocation keys:", invocation ? Object.keys(invocation) : "null/undefined");
+    const DEBUG = process.env.DEBUG === "1" || process.env.DEBUG === "true";
+    if (DEBUG) {
+      try {
+        process.stderr.write(
+          `[codex-native] executeToolViaFramework invocation: ${JSON.stringify(invocation)}
+`
+        );
+      } catch {
+      }
+    }
     if (!invocation) {
-      console.warn("Codex requested a tool execution without invocation data.");
+      try {
+        process.stderr.write("[codex-native] Codex requested a tool execution without invocation data.\n");
+      } catch {
+      }
       return {
         output: JSON.stringify({
           message: "Tool invocation payload missing",
@@ -306,13 +332,23 @@ var CodexModel = class {
         error: "Missing tool invocation data from Codex"
       };
     }
-    console.log(
-      `Tool execution requested by Codex: ${invocation.toolName} (callId: ${invocation.callId})`
-    );
+    if (DEBUG) {
+      try {
+        process.stderr.write(
+          `[codex-native] Tool execution requested: ${invocation.toolName} (callId: ${invocation.callId})
+`
+        );
+      } catch {
+      }
+    }
     const executor = this.toolExecutors.get(invocation.toolName) ?? getCodexToolExecutor(invocation.toolName);
     if (!executor) {
       const message = `No Codex executor registered for tool '${invocation.toolName}'. Use codexTool() or provide a codexExecute handler.`;
-      console.warn(message);
+      try {
+        process.stderr.write(`[codex-native] ${message}
+`);
+      } catch {
+      }
       return {
         success: false,
         error: message,
@@ -357,7 +393,15 @@ var CodexModel = class {
     if (typeof result === "string") {
       return { success: true, output: result };
     }
-    if (typeof result === "object" && ("output" in result || "error" in result || "success" in result)) {
+    const isNativeToolResultLike = (value) => {
+      if (!value || typeof value !== "object") return false;
+      if (!("output" in value) && !("error" in value) && !("success" in value)) return false;
+      if ("success" in value && value.success != null && typeof value.success !== "boolean") return false;
+      if ("output" in value && value.output != null && typeof value.output !== "string") return false;
+      if ("error" in value && value.error != null && typeof value.error !== "string") return false;
+      return true;
+    };
+    if (isNativeToolResultLike(result)) {
       return {
         success: result.success ?? !result.error,
         output: result.output,