[SDK Validation]: handle rename scenario gracefully in findAllParentsWithFile to prevent crash

eng/tools/spec-gen-sdk-runner/src/index.ts

@@ -1,3 +1,3 @@
 import { existsSync, mkdirSync } from "node:fs";
 import path from "node:path";
 import { exit } from "node:process";
@@ -22,12 +22,18 @@
     mkdirSync(logFolder, { recursive: true });
   }
   let statusCode: number;
-  if (batchType) {
-    statusCode = await generateSdkForBatchSpecs(batchType);
-  } else if (pullRequestNumber) {
-    statusCode = await generateSdkForSpecPr();
-  } else {
-    statusCode = await generateSdkForSingleSpec();
+  try {
+    if (batchType) {
+      statusCode = await generateSdkForBatchSpecs(batchType);
+    } else if (pullRequestNumber) {
+      statusCode = await generateSdkForSpecPr();
+    } else {
+      statusCode = await generateSdkForSingleSpec();
+    }
+  } catch (error) {
+    console.error("Unhandled exception in spec-gen-sdk-runner:", error);
+    console.log("##vso[task.complete result=Failed;]");
+    exit(1);
   }
   if (statusCode !== 0) {
     console.log("##vso[task.complete result=Failed;]");

eng/tools/spec-gen-sdk-runner/src/utils.ts

@@ -1,3 +1,3 @@
 import { getChangedFiles as getChangedFilesShared } from "@azure-tools/specs-shared/changed-files";
 import { exec, spawn, spawnSync } from "node:child_process";
 import fs from "node:fs";
@@ -249,13 +249,8 @@
         results.push(currentPath);
       }
     } catch (error) {
-      logMessage(
-        `Error reading directory: ${currentPath}. ` +
-          `If this is a transient IO error, please re-run the pipeline. ` +
-          `Error details: ${inspect(error)}`,
-        LogLevel.Error,
-      );
-      throw error;
+      logMessage(`Error reading directory: ${currentPath} with ${inspect(error)}`, LogLevel.Warn);
+      return results;
     }
     currentPath = path.dirname(currentPath);
     // Check if we've reached the root of the path (stopAtFolder) or

eng/tools/spec-gen-sdk-runner/test/utils/findAllParentsWithFile.test.ts

@@ -1,3 +1,3 @@
 import path from "node:path";
 import { fileURLToPath } from "node:url";
 import { describe, expect, test } from "vitest";
@@ -66,10 +66,13 @@
     );
   });
 
-  test("throws error when directory does not exist", () => {
-    expect(() =>
-      findAllParentsWithFile("specification/nonexistent/path", typespecProjectRegex, repoRoot),
-    ).toThrow(/ENOENT/);
+  test("returns empty array when directory does not exist", () => {
+    const result = findAllParentsWithFile(
+      "specification/nonexistent/path",
+      typespecProjectRegex,
+      repoRoot,
+    );
+    expect(result).toHaveLength(0);
   });
 
   test("stops at specified boundary and finds files before it", () => {