fix(respect): error handling when servers are missing from both openapi and cli options

Author: DmitryAnansky

.changeset/sweet-pumpkins-wait.md

@@ -0,0 +1,6 @@
+---
+"@redocly/respect-core": patch
+"@redocly/cli": patch
+---
+
+Improved Respect's error handling when server URLs are missing from both OpenAPI descriptions and CLI options. Now provides a clearer error message when no server URL is available.

__tests__/smoke/run-smoke.sh

@@ -27,4 +27,4 @@ echo "✅ Docs built correctly."
 # Check for broken $refs (or other issues) in the split files, especially on Windows (it will fail on a difference)
 echo "Checking split files for issues..."
 diff -r pre-split output/split
-echo "✅ Files split correctly."
+echo "✅ Files split correctly."

packages/respect-core/src/modules/__tests__/flow-runner/get-server-url.test.ts

@@ -365,4 +365,15 @@ describe('getServerUrl', () => {
       ],
     });
   });
+
+  it('should return undefined when descriptionName is provided but servers are undefined in sourceDescription', () => {
+    const ctx = {
+      $sourceDescriptions: {
+        test: { servers: undefined },
+      },
+    } as unknown as TestContext;
+
+    const result = getServerUrl({ ctx, descriptionName: 'test' });
+    expect(result).toBeUndefined();
+  });
 });

packages/respect-core/src/modules/flow-runner/get-server-url.ts

@@ -112,7 +112,7 @@ export function getServerUrl({
     !descriptionName ||
     !ctx.$sourceDescriptions ||
     !ctx.$sourceDescriptions[descriptionName] ||
-    !ctx.$sourceDescriptions[descriptionName].servers.length
+    !ctx.$sourceDescriptions[descriptionName]?.servers?.length
   ) {
     return undefined;
   }

scripts/local-pack.sh

@@ -29,4 +29,4 @@ cd ../../
 # Restore original package.json files
 mv packages/core/package.json.bak packages/core/package.json
 mv packages/respect-core/package.json.bak packages/respect-core/package.json
-mv packages/cli/package.json.bak packages/cli/package.json
+mv packages/cli/package.json.bak packages/cli/package.json