refactor(cli): improve error handling for remote script fetch (#1408)

Author: apoorvdarshan

build/cli.cjs

@@ -325,7 +325,8 @@ function readScriptFromHttp(remote) {
     const res = yield (0, import_index.fetch)(remote);
     if (!res.ok) {
       console.error(`Error: Can't get ${remote}`);
-      import_node_process2.default.exit(1);
+      import_node_process2.default.exitCode = 1;
+      throw new import_index.Fail(`Failed to fetch remote script: ${remote} (${res.status})`);
     }
     return res.text();
   });

src/cli.ts

@@ -252,7 +252,8 @@ async function readScriptFromHttp(remote: string): Promise<string> {
   const res = await fetch(remote)
   if (!res.ok) {
     console.error(`Error: Can't get ${remote}`)
-    process.exit(1)
+    process.exitCode = 1
+    throw new Fail(`Failed to fetch remote script: ${remote} (${res.status})`)
   }
   return res.text()
 }

test/cli.test.js

@@ -285,6 +285,8 @@ console.log(a);
     const server = await fakeServer([`HTTP/1.1 500\n\n500\n`]).listen(port)
     const out = await $`node build/cli.js http://127.0.0.1:${port}`.nothrow()
     assert.match(out.stderr, /Error: Can't get/)
+    assert.match(out.stderr, /Failed to fetch remote script/)
+    assert.equal(out.exitCode, 1)
     await server.stop()
   })