From df204b1d8f824fd65f009ccf0405da91533bfce3 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Sat, 28 Mar 2026 06:18:35 +0000
Subject: [PATCH] fix: restore pi embedded runner transport typing

---
 src/agents/bedrock-discovery.test.ts   |  1 +
 src/agents/pi-embedded-runner/model.ts | 27 ++++++++++++++++++++++----
 2 files changed, 24 insertions(+), 4 deletions(-)

diff --git a/src/agents/bedrock-discovery.test.ts b/src/agents/bedrock-discovery.test.ts
index cab55e51678..a97c983c179 100644
--- a/src/agents/bedrock-discovery.test.ts
+++ b/src/agents/bedrock-discovery.test.ts
@@ -236,6 +236,7 @@ describe("bedrock discovery", () => {
           },
           providers: {
             "amazon-bedrock": {
+              baseUrl: "https://bedrock-runtime.us-east-1.amazonaws.com",
               headers: { "x-test-header": "1" },
               models: [],
             },
diff --git a/src/agents/pi-embedded-runner/model.ts b/src/agents/pi-embedded-runner/model.ts
index 8ae9158eda0..2f6c4dc47b2 100644
--- a/src/agents/pi-embedded-runner/model.ts
+++ b/src/agents/pi-embedded-runner/model.ts
@@ -59,6 +59,22 @@ const DEFAULT_PROVIDER_RUNTIME_HOOKS: ProviderRuntimeHooks = {
   normalizeProviderTransportWithPlugin,
 };
 
+function normalizeResolvedTransportApi(api: unknown): ModelDefinitionConfig["api"] | undefined {
+  switch (api) {
+    case "anthropic-messages":
+    case "bedrock-converse-stream":
+    case "github-copilot":
+    case "google-generative-ai":
+    case "ollama":
+    case "openai-codex-responses":
+    case "openai-completions":
+    case "openai-responses":
+      return api;
+    default:
+      return undefined;
+  }
+}
+
 function sanitizeModelHeaders(
   headers: unknown,
   opts?: { stripSecretRefMarkers?: boolean },
@@ -116,7 +132,7 @@ function normalizeResolvedModel(params: {
 
 function resolveProviderTransport(params: {
   provider: string;
-  api?: ModelDefinitionConfig["api"] | null;
+  api?: ModelDefinitionConfig["api"] | Api | null;
   baseUrl?: string;
   cfg?: OpenClawConfig;
   runtimeHooks?: ProviderRuntimeHooks;
@@ -136,7 +152,7 @@ function resolveProviderTransport(params: {
   }) as { api?: ModelDefinitionConfig["api"] | null; baseUrl?: string } | undefined;
 
   return {
-    api: normalized?.api ?? params.api ?? undefined,
+    api: normalizeResolvedTransportApi(normalized?.api ?? params.api),
     baseUrl: normalized?.baseUrl ?? params.baseUrl,
   };
 }
@@ -224,7 +240,10 @@ function applyConfiguredProviderOverrides(params: {
   });
   return {
     ...discoveredModel,
-    api: resolvedTransport.api,
+    api:
+      resolvedTransport.api ??
+      normalizeResolvedTransportApi(discoveredModel.api) ??
+      "openai-responses",
     baseUrl: resolvedTransport.baseUrl ?? discoveredModel.baseUrl,
     reasoning: configuredModel?.reasoning ?? discoveredModel.reasoning,
     input: normalizedInput,
@@ -432,7 +451,7 @@ function resolveConfiguredFallbackModel(params: {
     model: {
       id: modelId,
       name: modelId,
-      api: fallbackTransport.api,
+      api: fallbackTransport.api ?? "openai-responses",
       provider,
       baseUrl: fallbackTransport.baseUrl,
       reasoning: configuredModel?.reasoning ?? false,