Fix exec.exec return value handling for git show command

- Correctly capture exit code from exec.exec with ignoreReturnCode
- Remove unnecessary .catch() that was never triggered
- This fixes CI tests to properly detect missing baseline schemas

Author: gfx

dist/index.js

@@ -166,8 +166,7 @@ async function getSchemaFromBranch(branch, schemaFile) {
     const tempFile = path.join(os.tmpdir(), `schema-${branch.replace(/\//g, "-")}-${Date.now()}.sql`);
     // Create empty file first to ensure it exists
     fs.writeFileSync(tempFile, "");
-    let exitCode = 0;
-    await exec.exec("git", ["show", `${branch}:${schemaFile}`], {
+    const exitCode = await exec.exec("git", ["show", `${branch}:${schemaFile}`], {
         silent: true,
         ignoreReturnCode: true,
         listeners: {
@@ -179,8 +178,6 @@ async function getSchemaFromBranch(branch, schemaFile) {
                 core.debug(`git show stderr: ${data.toString()}`);
             },
         },
-    }).catch((error) => {
-        exitCode = error.exitCode || 1;
     });
     if (exitCode !== 0) {
         // If the file doesn't exist in the base branch, return empty schema

dist/index.js.map

@@ -141,8 +141,7 @@ async function getSchemaFromBranch(branch: string, schemaFile: string): Promise<
     // Create empty file first to ensure it exists
     fs.writeFileSync(tempFile, "");
 
-    let exitCode = 0;
-    await exec.exec("git", ["show", `${branch}:${schemaFile}`], {
+    const exitCode = await exec.exec("git", ["show", `${branch}:${schemaFile}`], {
         silent: true,
         ignoreReturnCode: true,
         listeners: {
@@ -154,8 +153,6 @@ async function getSchemaFromBranch(branch: string, schemaFile: string): Promise<
                 core.debug(`git show stderr: ${data.toString()}`);
             },
         },
-    }).catch((error) => {
-        exitCode = error.exitCode || 1;
     });
 
     if (exitCode !== 0) {