@nextclaw/nextclaw-ncp-runtime-plugin-codex-sdk 0.1.7 → 0.1.8

package/dist/codex-model-provider.d.ts

@@ -0,0 +1,13 @@
+declare function resolveExternalModelProvider(params: {
+    explicitModelProvider?: unknown;
+    providerName?: string | null;
+    providerDisplayName?: string | null;
+    pluginId: string;
+}): string;
+declare function buildUserFacingModelRoute(params: {
+    externalModelProvider: string;
+    providerLocalModel: string;
+    resolvedModel: string;
+}): string;
+
+export { buildUserFacingModelRoute, resolveExternalModelProvider };

package/dist/codex-model-provider.js

@@ -0,0 +1,38 @@
+function readOptionalString(value) {
+  if (typeof value !== "string") {
+    return void 0;
+  }
+  const trimmed = value.trim();
+  return trimmed || void 0;
+}
+function isValidExternalModelProvider(value) {
+  return /^[A-Za-z0-9][A-Za-z0-9._-]*$/.test(value);
+}
+function resolveExternalModelProvider(params) {
+  const explicitModelProvider = readOptionalString(params.explicitModelProvider);
+  if (explicitModelProvider) {
+    return explicitModelProvider;
+  }
+  const providerName = readOptionalString(params.providerName);
+  if (providerName && !providerName.startsWith("custom-")) {
+    return providerName;
+  }
+  const providerDisplayName = readOptionalString(params.providerDisplayName);
+  if (providerDisplayName && isValidExternalModelProvider(providerDisplayName)) {
+    return providerDisplayName;
+  }
+  throw new Error(
+    `[codex] custom provider "${providerName ?? "unknown"}" requires an external model provider id. Set plugins.entries.${params.pluginId}.config.modelProvider or use a provider display name with only letters, numbers, ".", "_" or "-".`
+  );
+}
+function buildUserFacingModelRoute(params) {
+  const providerLocalModel = params.providerLocalModel.trim();
+  if (!providerLocalModel) {
+    return params.resolvedModel.trim();
+  }
+  return `${params.externalModelProvider}/${providerLocalModel}`;
+}
+export {
+  buildUserFacingModelRoute,
+  resolveExternalModelProvider
+};

package/dist/index.js

@@ -1,16 +1,18 @@
 import {
   findProviderByModel,
   findProviderByName,
-  getApiBase,
   buildRequestedSkillsUserPrompt,
-  getProvider,
-  getProviderName,
+  resolveProviderRuntime,
   getWorkspacePath,
   SkillsLoader
 } from "@nextclaw/core";
 import {
   CodexSdkNcpAgentRuntime
 } from "@nextclaw/nextclaw-ncp-runtime-codex-sdk";
+import {
+  buildUserFacingModelRoute,
+  resolveExternalModelProvider
+} from "./codex-model-provider.js";
 const PLUGIN_ID = "nextclaw-ncp-runtime-plugin-codex-sdk";
 const CODEX_RUNTIME_KIND = "codex";
 function readString(value) {
@@ -80,7 +82,12 @@ function resolveCodexExecutionOptions(params) {
 }
 function resolveCodexCliConfig(params) {
   const explicitConfig = readRecord(params.pluginConfig.config);
-  const modelProvider = readString(params.pluginConfig.modelProvider) ?? readString(params.providerName) ?? "openai";
+  const modelProvider = resolveExternalModelProvider({
+    explicitModelProvider: params.pluginConfig.modelProvider,
+    providerName: params.providerName,
+    providerDisplayName: params.providerDisplayName,
+    pluginId: PLUGIN_ID
+  });
   const preferredAuthMethod = readString(params.pluginConfig.preferredAuthMethod) ?? "apikey";
   const apiBase = readString(params.pluginConfig.apiBase) ?? readString(params.apiBase);
   const config = {
@@ -102,18 +109,6 @@ function resolveCodexCliConfig(params) {
     ...explicitConfig ?? {}
   };
 }
-function stripProviderPrefix(model, providerName) {
-  const normalizedModel = model.trim();
-  const normalizedProvider = providerName?.trim().toLowerCase();
-  if (!normalizedModel || !normalizedProvider) {
-    return normalizedModel;
-  }
-  const prefix = `${normalizedProvider}/`;
-  if (!normalizedModel.toLowerCase().startsWith(prefix)) {
-    return normalizedModel;
-  }
-  return normalizedModel.slice(prefix.length);
-}
 function readRequestedSkills(metadata) {
   const raw = metadata.requested_skills ?? metadata.requestedSkills;
   if (!Array.isArray(raw)) {
@@ -167,23 +162,34 @@ const plugin = {
           pluginConfig,
           sessionMetadata: runtimeParams.sessionMetadata
         });
-        const provider = getProvider(nextConfig, model);
-        const providerName = getProviderName(nextConfig, model);
+        const resolvedProviderRuntime = resolveProviderRuntime(nextConfig, model);
+        const providerName = resolvedProviderRuntime.providerName;
         const capabilitySpec = resolveCodexCapabilitySpec({
           model,
           providerName
         });
-        const apiBase = readString(pluginConfig.apiBase) ?? getApiBase(nextConfig, model) ?? void 0;
-        const apiKey = readString(pluginConfig.apiKey) ?? provider?.apiKey ?? void 0;
+        const externalModelProvider = resolveExternalModelProvider({
+          explicitModelProvider: pluginConfig.modelProvider,
+          providerName,
+          providerDisplayName: resolvedProviderRuntime.providerDisplayName,
+          pluginId: PLUGIN_ID
+        });
+        const userFacingModelRoute = buildUserFacingModelRoute({
+          externalModelProvider,
+          providerLocalModel: resolvedProviderRuntime.providerLocalModel,
+          resolvedModel: resolvedProviderRuntime.resolvedModel
+        });
+        const apiBase = readString(pluginConfig.apiBase) ?? resolvedProviderRuntime.apiBase ?? void 0;
+        const apiKey = readString(pluginConfig.apiKey) ?? resolvedProviderRuntime.apiKey ?? void 0;
         if (!apiKey) {
           throw new Error(
-            `[codex] missing apiKey. Set plugins.entries.${PLUGIN_ID}.config.apiKey or providers.*.apiKey for model "${model}".`
+            `[codex] missing apiKey. Set plugins.entries.${PLUGIN_ID}.config.apiKey or providers.*.apiKey for model "${userFacingModelRoute}".`
           );
         }
         if (capabilitySpec?.supportsResponsesApi === false) {
-          const capabilityProviderName = capabilitySpec.displayName ?? capabilitySpec.name ?? providerName ?? "provider";
+          const capabilityProviderName = capabilitySpec.displayName ?? capabilitySpec.name ?? resolvedProviderRuntime.providerDisplayName ?? externalModelProvider;
           throw new Error(
-            `[codex] model "${model}" is routed through "${capabilityProviderName}", which does not support the Responses API. Codex SDK currently only supports models available through the Responses API.`
+            `[codex] model "${userFacingModelRoute}" is routed through "${capabilityProviderName}", which does not support the Responses API. Codex SDK currently only supports models available through the Responses API.`
           );
         }
         const executionOptions = resolveCodexExecutionOptions({
@@ -195,14 +201,14 @@ const plugin = {
           sessionId: runtimeParams.sessionId,
           apiKey,
           apiBase,
-          model: stripProviderPrefix(model, providerName),
+          model: resolvedProviderRuntime.providerLocalModel,
           threadId: readString(runtimeParams.sessionMetadata.codex_thread_id) ?? null,
           codexPathOverride: readString(pluginConfig.codexPathOverride),
           env: readStringRecord(pluginConfig.env),
           cliConfig: resolveCodexCliConfig({
             pluginConfig,
-            model,
             providerName,
+            providerDisplayName: resolvedProviderRuntime.providerDisplayName,
             apiBase
           }),
           stateManager: runtimeParams.stateManager,

package/openclaw.plugin.json

@@ -17,6 +17,9 @@
       "model": {
         "type": "string"
       },
+      "modelProvider": {
+        "type": "string"
+      },
       "workingDirectory": {
         "type": "string"
       },
@@ -74,6 +77,11 @@
     "model": {
       "label": "Default Model"
     },
+    "modelProvider": {
+      "label": "External Model Provider",
+      "help": "Optional override for the provider id exposed to Codex SDK. Recommended for custom providers whose display name is not the actual upstream provider id.",
+      "advanced": true
+    },
     "workingDirectory": {
       "label": "Working Directory",
       "advanced": true

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nextclaw/nextclaw-ncp-runtime-plugin-codex-sdk",
-  "version": "0.1.7",
+  "version": "0.1.8",
   "private": false,
   "description": "NextClaw plugin that registers Codex SDK as an optional NCP runtime.",
   "type": "module",
@@ -21,10 +21,10 @@
     ]
   },
   "dependencies": {
-    "@nextclaw/core": "0.9.5",
+    "@nextclaw/core": "0.9.6",
     "@nextclaw/ncp": "0.3.1",
-    "@nextclaw/ncp-toolkit": "0.4.1",
-    "@nextclaw/nextclaw-ncp-runtime-codex-sdk": "0.1.1"
+    "@nextclaw/nextclaw-ncp-runtime-codex-sdk": "0.1.1",
+    "@nextclaw/ncp-toolkit": "0.4.1"
   },
   "devDependencies": {
     "@types/node": "^20.17.6",