@janole/ai-sdk-provider-codex-asp 0.4.1 → 0.4.3

package/README.md

@@ -2,7 +2,7 @@
 
 `@janole/ai-sdk-provider-codex-asp` is a [Vercel AI SDK](https://ai-sdk.dev/) v6 custom provider for the Codex App Server Protocol.
 
-Status: POC feature-complete for language model usage. Currently tested with [codex-cli](https://github.com/openai/codex/releases/tag/rust-v0.107.0) 0.107.0.
+Status: POC feature-complete for language model usage. Currently tested with [codex-cli](https://github.com/openai/codex/releases/tag/rust-v0.117.0) 0.117.0.
 
 - `LanguageModelV3` provider implementation
 - Streaming (`streamText`) and non-streaming (`generateText`)
@@ -113,8 +113,8 @@ const codex = createCodexAppServer({
   persistent?: { poolSize?, idleTimeoutMs?, scope?, key? },
   compaction?: { shouldCompactOnResume?, strict? }, // optional thread/compact/start before resumed turns
   debug?: { logPackets?, logger? },         // packet-level JSON-RPC debug logging
-  defaultThreadSettings?: { cwd?, approvalPolicy?, sandbox? },
-  defaultTurnSettings?: { cwd?, approvalPolicy?, sandboxPolicy?, model?, effort?, summary? },
+  defaultThreadSettings?: { cwd?, approvalPolicy?, approvalsReviewer?, sandbox? },
+  defaultTurnSettings?: { cwd?, approvalPolicy?, approvalsReviewer?, sandboxPolicy?, model?, effort?, summary? },
   approvals?: { onCommandApproval?, onFileChangeApproval? },
   toolTimeoutMs?: number,                  // default: 30000
   interruptTimeoutMs?: number,             // default: 10000
@@ -145,6 +145,29 @@ const codex = createCodexAppServer({
 });
 ```
 
+Approval reviewer example:
+
+Set `approvalsReviewer` to `"guardian_subagent"` to let Codex route approval decisions to its built-in reviewer agent instead of prompting the human user.
+
+```ts
+const codex = createCodexAppServer({
+  defaultThreadSettings: {
+    approvalsReviewer: "guardian_subagent",
+  },
+});
+
+await streamText({
+  model: codex("gpt-5.3-codex"),
+  prompt: "Delete the old generated protocol files under src/protocol/app-server-protocol if they are no longer referenced, then regenerate the current ones.",
+  providerOptions: codexCallOptions({
+    approvalsReviewer: "user",
+    approvals: {
+      onCommandApproval: async () => "decline",
+    },
+  }),
+});
+```
+
 See [`src/provider.ts`](src/provider.ts) for full type definitions.
 
 ## Examples

package/dist/index.cjs

@@ -935,7 +935,7 @@ var DynamicToolsDispatcher = class {
 // package.json
 var package_default = {
   name: "@janole/ai-sdk-provider-codex-asp",
-  version: "0.4.1"};
+  version: "0.4.3"};
 
 // src/package-info.ts
 var PACKAGE_NAME = package_default.name;
@@ -1769,6 +1769,12 @@ function sdkToolsToCodexDynamicTools(tools) {
     inputSchema: t.inputSchema
   }));
 }
+function resolveApprovalHandlers(providerSettings, callOptions) {
+  return stripUndefined({
+    onCommandApproval: callOptions?.approvals?.onCommandApproval ?? providerSettings.approvals?.onCommandApproval,
+    onFileChangeApproval: callOptions?.approvals?.onFileChangeApproval ?? providerSettings.approvals?.onFileChangeApproval
+  });
+}
 function isPassThroughContentPart(part) {
   switch (part.type) {
     case "tool-call":
@@ -1924,6 +1930,8 @@ var CodexLanguageModel = class {
     let activeThreadId;
     let activeTurnId;
     let session;
+    let detachApprovals;
+    let detachDynamicTools;
     const interruptTimeoutMs = this.config.providerSettings.interruptTimeoutMs ?? 1e4;
     const interruptTurnIfPossible = async () => {
       if (!activeThreadId || !activeTurnId) {
@@ -1950,6 +1958,10 @@ var CodexLanguageModel = class {
           try {
             controller.close();
           } finally {
+            detachDynamicTools?.();
+            detachDynamicTools = void 0;
+            detachApprovals?.();
+            detachApprovals = void 0;
             await fileResolver.cleanup();
             await client.disconnect();
           }
@@ -1963,6 +1975,10 @@ var CodexLanguageModel = class {
           try {
             controller.close();
           } finally {
+            detachDynamicTools?.();
+            detachDynamicTools = void 0;
+            detachApprovals?.();
+            detachApprovals = void 0;
             await fileResolver.cleanup();
             await client.disconnect();
           }
@@ -2024,11 +2040,10 @@ var CodexLanguageModel = class {
                 pendingToolCall.threadId,
                 closeSuccessfully
               );
-              const approvalsDispatcher2 = new ApprovalsDispatcher(stripUndefined({
-                onCommandApproval: this.config.providerSettings.approvals?.onCommandApproval,
-                onFileChangeApproval: this.config.providerSettings.approvals?.onFileChangeApproval
-              }));
-              approvalsDispatcher2.attach(client);
+              const approvalsDispatcher2 = new ApprovalsDispatcher(
+                resolveApprovalHandlers(this.config.providerSettings, callOptions)
+              );
+              detachApprovals = approvalsDispatcher2.attach(client);
               const result = toolResult ?? {
                 success: false,
                 contentItems: [{
@@ -2056,13 +2071,12 @@ var CodexLanguageModel = class {
                 timeoutMs: this.config.providerSettings.toolTimeoutMs,
                 onDebugEvent: toolLogger
               }));
-              dispatcher.attach(client);
+              detachDynamicTools = dispatcher.attach(client);
             }
-            const approvalsDispatcher = new ApprovalsDispatcher(stripUndefined({
-              onCommandApproval: this.config.providerSettings.approvals?.onCommandApproval,
-              onFileChangeApproval: this.config.providerSettings.approvals?.onFileChangeApproval
-            }));
-            approvalsDispatcher.attach(client);
+            const approvalsDispatcher = new ApprovalsDispatcher(
+              resolveApprovalHandlers(this.config.providerSettings, callOptions)
+            );
+            detachApprovals = approvalsDispatcher.attach(client);
             client.onAnyNotification((method, params) => {
               const parts = mapper.map({ method, params });
               const mappedTurnId = mapper.getTurnId();
@@ -2116,6 +2130,7 @@ var CodexLanguageModel = class {
                 developerInstructions,
                 cwd: callOptions?.cwd ?? this.config.providerSettings.defaultThreadSettings?.cwd,
                 approvalPolicy: callOptions?.approvalPolicy ?? this.config.providerSettings.defaultThreadSettings?.approvalPolicy,
+                approvalsReviewer: callOptions?.approvalsReviewer ?? this.config.providerSettings.defaultThreadSettings?.approvalsReviewer,
                 sandbox: callOptions?.sandbox ?? this.config.providerSettings.defaultThreadSettings?.sandbox,
                 model: callOptions?.model ?? this.config.providerSettings.defaultModel
               });
@@ -2179,6 +2194,7 @@ var CodexLanguageModel = class {
                 config,
                 cwd: callOptions?.cwd ?? this.config.providerSettings.defaultThreadSettings?.cwd,
                 approvalPolicy: callOptions?.approvalPolicy ?? this.config.providerSettings.defaultThreadSettings?.approvalPolicy,
+                approvalsReviewer: callOptions?.approvalsReviewer ?? this.config.providerSettings.defaultThreadSettings?.approvalsReviewer,
                 sandbox: callOptions?.sandbox ?? this.config.providerSettings.defaultThreadSettings?.sandbox
               });
               debugLog?.("outbound", "thread/start", threadStartParams);
@@ -2205,6 +2221,7 @@ var CodexLanguageModel = class {
               input: turnInput,
               cwd: callOptions?.cwd ?? this.config.providerSettings.defaultTurnSettings?.cwd,
               approvalPolicy: callOptions?.approvalPolicy ?? this.config.providerSettings.defaultTurnSettings?.approvalPolicy,
+              approvalsReviewer: callOptions?.approvalsReviewer ?? this.config.providerSettings.defaultTurnSettings?.approvalsReviewer,
               sandboxPolicy: callOptions?.sandboxPolicy ?? this.config.providerSettings.defaultTurnSettings?.sandboxPolicy,
               model: callOptions?.model ?? this.config.providerSettings.defaultTurnSettings?.model,
               effort: callOptions?.effort ?? this.config.providerSettings.defaultTurnSettings?.effort,