chore(tools): handle additional errors (renovatebot#42205)

chore(tools): handle additional error

In the case that `pdm` can't be executed, for instance if it's only
installed in a Virtualenv that's not currently active, this previously
would silently pass, even though the command had failed.

We can now make it more resilient by warning/erroring in the cases that:

- there is a timeout
- the process was killed
- the process failed to execute

Originally raised as renovatebot#40987, which was closed due to the migration to
`execa` and errors being flagged as an `unhandledRejection`.

However, after b35d7c9, we would no
longer reject, leading to an incomplete error handling case.

Author: jamietanna

tools/mkdocs.ts

@@ -79,6 +79,15 @@ function checkResult(res: ExecaSyncReturnValue<string>): void {
   } else if (res.exitCode) {
     logger.error(`Error occured:\n${res.stderr || res.stdout}`);
     process.exit(res.exitCode);
+  } else if (res.timedOut) {
+    logger.error({ res }, 'Process timed out');
+    process.exit(-1);
+  } else if (res.killed) {
+    logger.error({ res }, 'Process was killed');
+    process.exit(-1);
+  } else if (res.failed) {
+    logger.error({ res }, 'Process call failed');
+    process.exit(-1);
   } else {
     logger.debug(`Build completed:\n${res.stdout || res.stderr}`);
   }

tools/prepare-release.ts

@@ -50,6 +50,15 @@ async function build(): Promise<void> {
   } else if (res.exitCode) {
     logger.error(`Error occured:\n${res.stderr || res.stdout}`);
     process.exit(res.exitCode);
+  } else if (res.timedOut) {
+    logger.error({ res }, 'Process timed out');
+    process.exit(-1);
+  } else if (res.killed) {
+    logger.error({ res }, 'Process was killed');
+    process.exit(-1);
+  } else if (res.failed) {
+    logger.error({ res }, 'Process call failed');
+    process.exit(-1);
   } else {
     logger.debug(`Build succeeded:\n${res.stdout || res.stderr}`);
   }