fix(onboard): omit empty provider request settings

src/commands/onboard-auth.config-shared.test.ts

@@ -101,6 +101,34 @@ describe("onboard auth provider config merges", () => {
     expect(next.models?.providers?.other?.timeoutSeconds).toBe(300);
   });
 
+  it("omits empty provider request settings", () => {
+    const next = applyOnboardAuthAgentModelsAndProviders(
+      {
+        models: {
+          providers: {
+            custom: {
+              api: "openai-completions",
+              baseUrl: "https://old.example.com/v1",
+              models: [makeModel("model-a")],
+            },
+          },
+        },
+      },
+      {
+        agentModels,
+        providers: {
+          custom: {
+            api: "openai-completions",
+            baseUrl: "https://new.example.com/v1",
+            models: [makeModel("model-b")],
+          },
+        },
+      },
+    );
+
+    expect(next.models?.providers?.custom).not.toHaveProperty("request");
+  });
+
   it("preserves settings without resurrecting a non-canonical provider key", () => {
     const next = applyOnboardAuthAgentModelsAndProviders(
       {

src/plugin-sdk/provider-onboard.ts

@@ -329,10 +329,15 @@ function mergeOnboardProviderConfigs(
       }
     }
     if (!("request" in providerConfig) || providerConfig.request) {
-      nextProvider.request = mergeOnboardProviderRequest(
+      const mergedRequest = mergeOnboardProviderRequest(
         existingProvider.request,
         providerConfig.request,
       );
+      if (mergedRequest) {
+        nextProvider.request = mergedRequest;
+      } else {
+        delete nextProvider.request;
+      }
     }
     merged[providerId] = nextProvider;
   }