feat: validate pckgs before building

packages/scripts/src/cli.ts

@@ -6,7 +6,7 @@ dotenv.config({
 import { Command } from "commander";
 
 import { runBuild } from "./commands/build";
-import { ALL_PACKAGES, CATEGORY_VM } from "./common/cli.constants";
+import { ALL_PACKAGES, CATEGORY_VM, PCKG } from "./common/cli.constants";
 import { startDeleteFlow } from "./commands/delete";
 import { log } from "./common/cli.utils";
 import { Options_Cli } from "./common/cli.types";
@@ -58,6 +58,21 @@ const getCliOptions = (program: Command): Options_Cli => {
   return options;
 };
 
+const validatePackages = (packages: string[] | undefined) => {
+  if (!packages) {
+    log.error("Packages must be defined");
+  }
+  if (!packages?.includes(PCKG.NODE) && !packages?.includes(PCKG.WEB)) {
+    log.error(
+      `One or more of these pckgs isn't supported: ${(
+        packages as string[]
+      )?.toString()}`
+    );
+
+    process.exit(1);
+  }
+};
+
 const runScript = async () => {
   const program = createProgram();
   program.parse(process.argv);
@@ -68,6 +83,7 @@ const runScript = async () => {
   const cmd = program.args[0];
   switch (true) {
     case cmd === "build": {
+      validatePackages(options.packages);
       await runBuild(options);
       break;
     }

packages/scripts/src/commands/build.ts

@@ -18,24 +18,21 @@ import {
 } from "@scripts/common/cli.utils";
 
 export const runBuild = async (options: Options_Cli) => {
-  const env = options["environment"];
-  const vmInfo = await getVmInfo(env);
+  const vmInfo = await getVmInfo(options.environment);
 
   const pckgs =
-    options["packages"] === undefined
+    options?.packages?.length === 0
       ? await getPckgsTo("build")
-      : options["packages"];
+      : (options.packages as string[]);
 
   if (pckgs.includes(PCKG.NODE)) {
-    await buildNodePckgs(vmInfo, options["skipEnv"]);
+    await buildNodePckgs(vmInfo, options.skipEnv);
   }
-
   if (pckgs.includes(PCKG.WEB)) {
     await buildWeb(vmInfo);
   }
 };
 
-// eslint-disable-next-line @typescript-eslint/require-await
 const buildNodePckgs = async (vmInfo: Info_VM, skipEnv?: boolean) => {
   removeOldBuildFor(PCKG.NODE);
   createNodeDirs();
@@ -78,10 +75,10 @@ const buildWeb = async (vmInfo: Info_VM) => {
   log.success(`Done building web files.`);
   log.tip(`
     Now you'll probably want to:
-      - zip the build/web dir
+      - zip the build dir
       - copy it to your ${destination} server
-      - unzip it
-      - run it`);
+      - unzip it and serve as the static assets
+      `);
   process.exit(0);
 };