small cleanup

node-src/tasks/build.ts

@@ -61,21 +61,21 @@ export const setBuildCommand = async (ctx: Context) => {
 const timeoutAfter = (ms) =>
   new Promise((resolve, reject) => setTimeout(reject, ms, new Error(`Operation timed out`)));
 
-function isE2EBuildCommandNotFoundError(err) {
+function isE2EBuildCommandNotFoundError(errorMessage: string) {
   // It's hard to know if this is the case as each package manager has a different type of
   // error for this, but we'll try to figure it out.
-  const msg = err.message as string;
   const errorRegexes = ['command not found', `[\\W]?${e2eBuildBinName}[\\W]? not found`, 'code E404', 'exit code 127', `command failed.*${e2eBuildBinName}.*$`];
-  return errorRegexes.some((regex) => msg.match(new RegExp(regex, 'gi')));
+  return errorRegexes.some((regex) => errorMessage.match(new RegExp(regex, 'gi')));
 }
 
-function e2eBuildErrorMessage(err, workingDir, ctx: Context): { exitCode: number, message: string } {
+function e2eBuildErrorMessage(err, workingDir: string, ctx: Context): { exitCode: number, message: string } {
   const flag = ctx.options.playwright ? 'playwright' : 'cypress';
+  const errorMessage = err.message;
 
   // If we tried to run the E2E package's bin directly (due to being in the action)
   // and it failed, that means we couldn't find it. This probably means they haven't
   // installed the right dependency or run from the right directory.
-  if (isE2EBuildCommandNotFoundError(err)) {
+  if (isE2EBuildCommandNotFoundError(errorMessage)) {
     const dependencyName = `@chromatic-com/${flag}`;
     return {
       exitCode: exitCodes.MISSING_DEPENDENCY,
@@ -85,7 +85,7 @@ function e2eBuildErrorMessage(err, workingDir, ctx: Context): { exitCode: number
 
   return {
     exitCode: exitCodes.E2E_BUILD_FAILED,
-    message: e2eBuildFailed({ flag, errorMessage: err.message }),
+    message: e2eBuildFailed({ flag, errorMessage }),
   };
 }