Preserve base URL variable when OpenAPI servers are omitted (#585)

plugins/importer-openapi/src/index.ts

@@ -63,9 +63,10 @@ export async function convertOpenApi(contents: string): Promise<ImportPluginResp
     httpRequests: [],
   };
   const baseUrl = importBaseUrl(spec);
-  const requestBaseUrl = baseUrl.length > 0 ? "${[baseUrl]}" : "";
+  // A local spec has no document URL against which OpenAPI's implicit "/"
-
+  // server can resolve. Keep the shared variable even when its initial value
-  if (baseUrl.length > 0) {
+  // is empty so users can configure the host once instead of editing requests.
+  const requestBaseUrl = "${[baseUrl]}";
   resources.environments.push({
     model: "environment",
     id: importState.generateId("environment"),
@@ -76,7 +77,6 @@ export async function convertOpenApi(contents: string): Promise<ImportPluginResp
     parentId: null,
     sortPriority: importState.nextSortPriority(),
   });
-  }
 
   const folderIdsByTag = new Map<string, string>();
   const routeLabels = new Map<string, string>();

plugins/importer-openapi/tests/index.test.ts

@@ -229,6 +229,26 @@ describe("importer-openapi", () => {
     expect(imported).toBeUndefined();
   });
 
+  test("Creates an editable baseUrl variable when OpenAPI omits servers", async () => {
+    const imported = await convertOpenApi(
+      JSON.stringify({
+        openapi: "3.0.4",
+        info: { title: "Serverless OpenAPI Test", version: "1.0.0" },
+        paths: {
+          "/api/widgets": { get: { responses: {} } },
+        },
+      }),
+    );
+
+    expect(imported?.resources.environments).toEqual([
+      expect.objectContaining({
+        name: "Global Variables",
+        variables: [{ name: "baseUrl", value: "" }],
+      }),
+    ]);
+    expect(imported?.resources.httpRequests[0]?.url).toBe("${[baseUrl]}/api/widgets");
+  });
+
   test("Prefers operation and path servers over the spec base URL", async () => {
     const imported = await convertOpenApi(
       JSON.stringify({