fix(monorepo): Resolve issue with checking results in scripts tools

Author: sullivanpj

tools/scripts/src/bootstrap.mjs

@@ -41,13 +41,13 @@ try {
   //     echo`${data}`;
   //   });
   //   const result = await proc;
-  //   if (!result.ok) {
+  //   if (result.exitCode !== 0) {
   //     throw new Error(
   //       `An error occurred while resetting the Nx daemon process: \n\n${result.message}\n`
   //     );
   //   }
 
-  echo`${chalk.green("Completed monorepo bootstrapping successfully!")}`;
+  echo`${chalk.green(" ✅  Completed monorepo bootstrapping successfully!")}`;
 } catch (error) {
   echo`${chalk.red(
     error?.message

tools/scripts/src/build.mjs

@@ -40,7 +40,7 @@ try {
     echo`${data}`;
   });
   let result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while bootstrapping the monorepo: \n\n${
         result.message
@@ -56,7 +56,7 @@ try {
     echo`${data}`;
   });
   result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while building the monorepo in ${
         configuration
@@ -65,7 +65,7 @@ try {
   }
 
   echo`${chalk.green(
-    `Successfully built the monorepo in ${configuration} mode!`
+    ` ✅  Successfully built the monorepo in ${configuration} mode!`
   )}`;
 } catch (error) {
   echo`${chalk.red(error?.message ? error.message : "A failure occurred while building the monorepo")}`;

tools/scripts/src/format.mjs

@@ -35,7 +35,7 @@ try {
     echo`${data}`;
   });
   let result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while formatting the workspace's README file: \n\n${result.message}\n`
     );
@@ -49,7 +49,7 @@ try {
     echo`${data}`;
   });
   result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while formatting the monorepo: \n\n${result.message}\n`
     );
@@ -63,13 +63,13 @@ try {
     echo`${data}`;
   });
   result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while running \`nx format:write\` on the monorepo: \n\n${result.message}\n`
     );
   }
 
-  echo`${chalk.green("Successfully formatted the monorepo's files")}`;
+  echo`${chalk.green(" ✅  Successfully formatted the monorepo's files")}`;
 } catch (error) {
   echo`${chalk.red(error?.message ? error.message : "A failure occurred while formatting the monorepo")}`;
 

tools/scripts/src/lint.mjs

@@ -30,7 +30,7 @@ try {
     echo`${data}`;
   });
   let result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while linting the monorepo: \n\n${result.message}\n`
     );
@@ -43,13 +43,13 @@ try {
     echo`${data}`;
   });
   result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while running \`storm-lint\` on the monorepo: \n\n${result.message}\n`
     );
   }
 
-  echo`${chalk.green("Successfully linted the monorepo's files")}`;
+  echo`${chalk.green(" ✅  Successfully linted the monorepo's files")}`;
 } catch (error) {
   echo`${chalk.red(error?.message ? error.message : "A failure occurred while linting the monorepo")}`;
 

tools/scripts/src/nuke.mjs

@@ -30,7 +30,7 @@ try {
   //   echo`${data}`;
   // });
   // let result = await proc;
-  // if (!result.ok) {
+  // if (result.exitCode !== 0) {
   //   throw new Error(
   //     `An error occurred while cleaning the monorepo projects: \n\n${result.message}\n`
   //   );
@@ -41,7 +41,7 @@ try {
     echo`${data}`;
   });
   let result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while clearing Nx cache: \n\n${result.message}\n`
     );
@@ -54,7 +54,7 @@ try {
     echo`${data}`;
   });
   result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while removing cache directories: \n\n${result.message}\n`
     );
@@ -65,7 +65,7 @@ try {
     echo`${data}`;
   });
   result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while removing node modules and build directories from the monorepo's projects: \n\n${result.message}\n`
     );
@@ -76,7 +76,7 @@ try {
     echo`${data}`;
   });
   result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while removing node modules and build directories from the monorepo's projects: \n\n${result.message}\n`
     );
@@ -87,13 +87,13 @@ try {
     echo`${data}`;
   });
   result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while removing node modules and build directories from the monorepo's projects: \n\n${result.message}\n`
     );
   }
 
-  echo`${chalk.green("Successfully nuked the cache, node modules, and build folders \n\n")}`;
+  echo`${chalk.green(" ✅  Successfully nuked the cache, node modules, and build folders \n\n")}`;
 } catch (error) {
   echo`${chalk.red(error?.message ? error.message : "A failure occurred while nuking the monorepo")}`;
 }

tools/scripts/src/release.mjs

@@ -41,7 +41,7 @@ try {
     echo`${data}`;
   });
   let result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while building workspace packages: \n\n${result.message}\n`
     );
@@ -54,13 +54,13 @@ try {
     echo`${data}`;
   });
   result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while releasing workspace packages: \n\n${result.message}\n`
     );
   }
 
-  echo`${chalk.green("Successfully released workspace packages")}`;
+  echo`${chalk.green(" ✅  Successfully released workspace packages")}`;
 } catch (error) {
   echo`${chalk.red(error?.message ? error.message : "A failure occurred while releasing workspace packages")}`;
 

tools/scripts/src/update-storm.mjs

@@ -30,7 +30,7 @@ try {
     );
   proc.stdout.on("data", data => echo`${data}`);
   let result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while updating "@storm-software/*" packages:\n\n${result.message}\n`
     );
@@ -43,7 +43,7 @@ try {
   );
   proc.stdout.on("data", data => echo`${data}`);
   result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while updating "@stryke/*" packages:\n\n${result.message}\n`
     );
@@ -53,7 +53,7 @@ try {
   proc = $`pnpm dedupe`.timeout(`${8 * 60}s`);
   proc.stdout.on("data", data => echo`${data}`);
   result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while deduplicating workspace dependencies:\n\n${result.message}\n`
     );
@@ -63,13 +63,13 @@ try {
   proc = $`pnpm update --recursive --workspace`.timeout(`${8 * 60}s`);
   proc.stdout.on("data", data => echo`${data}`);
   result = await proc;
-  if (!result.ok) {
+  if (result.exitCode !== 0) {
     throw new Error(
       `An error occurred while refreshing workspace links:\n\n${result.message}\n`
     );
   }
 
-  echo`${chalk.green("  ✅ Successfully updated Storm Software package dependencies and re-linked workspace packages")}\n\n`;
+  echo`${chalk.green(" ✅  Successfully updated Storm Software package dependencies and re-linked workspace packages")}\n\n`;
 } catch (error) {
   echo`${chalk.red(
     error?.message ??