@janole/ai-sdk-provider-codex-asp 0.3.4 → 0.3.6

package/README.md

@@ -126,6 +126,11 @@ codex.chat(modelId)           // explicit alias
 codex.shutdown()              // clean up persistent workers
 ```
 
+Approval callback notes:
+
+- `approvals.onCommandApproval(request)` receives the raw generated Codex protocol payload: `CommandExecutionRequestApprovalParams`.
+- `approvals.onFileChangeApproval(request)` receives the raw generated Codex protocol payload: `FileChangeRequestApprovalParams`.
+
 Rich sandbox policy example (turn-level):
 
 ```ts

package/dist/index.cjs

@@ -8,13 +8,6 @@ var path = require('path');
 var url = require('url');
 var provider = require('@ai-sdk/provider');
 
-// src/utils/object.ts
-function stripUndefined(obj) {
-  return Object.fromEntries(
-    Object.entries(obj).filter(([, value]) => value !== void 0)
-  );
-}
-
 // src/approvals.ts
 var ApprovalsDispatcher = class {
   onCommandApproval;
@@ -27,37 +20,14 @@ var ApprovalsDispatcher = class {
     const unsubCommand = client.onRequest(
       "item/commandExecution/requestApproval",
       async (params, _request) => {
-        const p = params;
-        const request = stripUndefined({
-          threadId: p.threadId,
-          turnId: p.turnId,
-          itemId: p.itemId,
-          approvalId: p.approvalId,
-          reason: p.reason,
-          networkApprovalContext: p.networkApprovalContext,
-          command: p.command,
-          cwd: p.cwd,
-          commandActions: p.commandActions,
-          additionalPermissions: p.additionalPermissions,
-          proposedExecpolicyAmendment: p.proposedExecpolicyAmendment,
-          proposedNetworkPolicyAmendments: p.proposedNetworkPolicyAmendments
-        });
-        const decision = await this.onCommandApproval(request);
+        const decision = await this.onCommandApproval(params);
         return { decision };
       }
     );
     const unsubFileChange = client.onRequest(
       "item/fileChange/requestApproval",
       async (params, _request) => {
-        const p = params;
-        const request = stripUndefined({
-          threadId: p.threadId,
-          turnId: p.turnId,
-          itemId: p.itemId,
-          reason: p.reason,
-          grantRoot: p.grantRoot
-        });
-        const decision = await this.onFileChangeApproval(request);
+        const decision = await this.onFileChangeApproval(params);
         return { decision };
       }
     );
@@ -82,6 +52,13 @@ var CodexNotImplementedError = class extends CodexProviderError {
   }
 };
 
+// src/utils/object.ts
+function stripUndefined(obj) {
+  return Object.fromEntries(
+    Object.entries(obj).filter(([, value]) => value !== void 0)
+  );
+}
+
 // src/client/app-server-client.ts
 var JsonRpcError = class extends CodexProviderError {
   code;
@@ -958,7 +935,7 @@ var DynamicToolsDispatcher = class {
 // package.json
 var package_default = {
   name: "@janole/ai-sdk-provider-codex-asp",
-  version: "0.3.4"};
+  version: "0.3.6"};
 
 // src/package-info.ts
 var PACKAGE_NAME = package_default.name;
@@ -1030,7 +1007,7 @@ var CodexEventMapper = class {
       "item/reasoning/textDelta": (p) => this.handleReasoningDelta(p),
       "item/reasoning/summaryTextDelta": (p) => this.handleReasoningDelta(p),
       "item/plan/delta": (p) => this.handleReasoningDelta(p),
-      "item/fileChange/outputDelta": (p) => this.handleReasoningDelta(p),
+      "item/fileChange/outputDelta": (p) => this.handleFileChangeOutputDelta(p),
       "item/reasoning/summaryPartAdded": (p) => this.handleSummaryPartAdded(p),
       "turn/plan/updated": (p) => this.handlePlanUpdated(p),
       "item/commandExecution/outputDelta": (p) => this.handleCommandOutputDelta(p),
@@ -1166,12 +1143,38 @@ ${output}`;
         parts.push(...this.startDynamicToolCall(item));
         break;
       }
+      case "fileChange": {
+        this.ensureStreamStarted(parts);
+        const toolName = "codex_file_change";
+        this.openToolCalls.set(item.id, { toolName, output: "", droppedChars: 0 });
+        parts.push(this.withMeta({
+          type: "tool-call",
+          toolCallId: item.id,
+          toolName,
+          input: JSON.stringify({ changes: item.changes, status: item.status }),
+          providerExecuted: true,
+          dynamic: true
+        }));
+        break;
+      }
+      case "webSearch": {
+        this.ensureStreamStarted(parts);
+        const toolName = "codex_web_search";
+        this.openToolCalls.set(item.id, { toolName, output: "", droppedChars: 0 });
+        parts.push(this.withMeta({
+          type: "tool-call",
+          toolCallId: item.id,
+          toolName,
+          input: JSON.stringify({ query: item.query, action: item.action ?? void 0 }),
+          providerExecuted: true,
+          dynamic: true
+        }));
+        break;
+      }
       case "reasoning":
       case "plan":
-      case "fileChange":
       case "mcpToolCall":
       case "collabAgentToolCall":
-      case "webSearch":
       case "imageView":
       case "contextCompaction":
       case "enteredReviewMode":
@@ -1252,22 +1255,40 @@ ${output}`;
         }
       }));
       this.openToolCalls.delete(item.id);
-    } else if (item.type === "webSearch") {
+    } else if (item.type === "fileChange" && this.openToolCalls.has(item.id)) {
+      const tracked = this.openToolCalls.get(item.id);
+      const toolName = tracked?.toolName ?? "codex_file_change";
+      const output = tracked ? this.formatToolOutput(tracked.output, tracked.droppedChars) : "";
+      parts.push(this.withMeta({
+        type: "tool-result",
+        toolCallId: item.id,
+        toolName,
+        result: { output, status: item.status, changes: item.changes }
+      }));
+      this.openToolCalls.delete(item.id);
+    } else if (item.type === "webSearch" && this.openToolCalls.has(item.id)) {
       const webSearchSummary = this.formatWebSearchItemSummary(item);
-      if (webSearchSummary) {
-        this.emitReasoningDelta(parts, item.id, webSearchSummary);
-      }
-      if (this.openReasoningParts.has(item.id)) {
-        parts.push(this.withMeta({ type: "reasoning-end", id: item.id }));
-        this.openReasoningParts.delete(item.id);
-      }
+      const tracked = this.openToolCalls.get(item.id);
+      const toolName = tracked?.toolName ?? "codex_web_search";
+      parts.push(this.withMeta({
+        type: "tool-result",
+        toolCallId: item.id,
+        toolName,
+        result: {
+          output: webSearchSummary || "",
+          query: item.query,
+          action: item.action ?? void 0,
+          summary: webSearchSummary || void 0
+        }
+      }));
+      this.openToolCalls.delete(item.id);
     } else if (this.openReasoningParts.has(item.id)) {
       parts.push(this.withMeta({ type: "reasoning-end", id: item.id }));
       this.openReasoningParts.delete(item.id);
     }
     return parts;
   }
-  // item/reasoning/textDelta, item/reasoning/summaryTextDelta, item/plan/delta, item/fileChange/outputDelta
+  // item/reasoning/textDelta, item/reasoning/summaryTextDelta, item/plan/delta
   handleReasoningDelta(params) {
     const delta = params ?? {};
     if (!delta.itemId || !delta.delta) {
@@ -1277,6 +1298,22 @@ ${output}`;
     this.emitReasoningDelta(parts, delta.itemId, delta.delta);
     return parts;
   }
+  // item/fileChange/outputDelta
+  handleFileChangeOutputDelta(params) {
+    const delta = params ?? {};
+    if (!delta.itemId || !delta.delta || !this.openToolCalls.has(delta.itemId)) {
+      return [];
+    }
+    const tracked = this.openToolCalls.get(delta.itemId);
+    this.appendTrackedOutput(tracked, delta.delta);
+    return [this.withMeta({
+      type: "tool-result",
+      toolCallId: delta.itemId,
+      toolName: tracked.toolName,
+      result: { output: this.formatToolOutput(tracked.output, tracked.droppedChars) },
+      preliminary: true
+    })];
+  }
   // item/reasoning/summaryPartAdded
   handleSummaryPartAdded(params) {
     const p = params ?? {};