fix: skip process.exit(1) in dry-run mode

The deploy e2e CI uses --dry-run with a dummy token, which causes
StaticSitesClient to fail with BadRequest. In dry-run mode, binary
failures should be logged but not cause process.exit(1).

Added test: 'should not call process.exit(1) in dry-run mode'

Author: LongOddCode

src/cli/commands/deploy/deploy.spec.ts

@@ -260,5 +260,14 @@ describe("deploy", () => {
 
       expect(exitSpy).not.toHaveBeenCalled();
     });
+
+    it("should not call process.exit(1) on non-zero exit code in dry-run mode", async () => {
+      await deploy({ outputLocation: "/test-output", dryRun: true });
+
+      mockChild.emit("close", 1);
+
+      expect(logger.error).toHaveBeenCalledWith("The deployment binary exited with code 1.");
+      expect(exitSpy).not.toHaveBeenCalled();
+    });
   });
 });

src/cli/commands/deploy/deploy.ts

@@ -337,7 +337,9 @@ export async function deploy(options: SWACLIConfig) {
           logger.error(`The deployment binary exited with code ${code}.`);
           logger.error(`If you are running in a minimal container image, ensure native dependencies are installed.`);
           logger.error(`Run ${chalk.cyan(`ldd ${binary}`)} to check for missing shared libraries.`);
-          process.exit(1);
+          if (!dryRun) {
+            process.exit(1);
+          }
         }
       });
     }