Added a step to check advanced minify using terser

Author: skymen

.vscode/settings.json

@@ -1,6 +1,6 @@
 {
   "javascript.updateImportsOnFileMove.enabled": "never",
-  "files.exclude": {
+  ".files.exclude": {
     "build": true,
     "generated": true,
     "template": true,

README.md

@@ -1,4 +1,4 @@
-<img src="./src/icon.svg?rand=8697" width="100" /><br>
+<img src="./src/icon.svg" width="100" /><br>
 # Sample Addon
 <i>Description</i> <br>
 ### Version 1.0.0.0
@@ -46,26 +46,26 @@ npm run dev
 ## Actions
 | Action | Description | Params
 | --- | --- | --- |
+| Sample Action Combo | This is a sample action | Param1             *(combo)* <br> |
 | Sample Action | This is a sample action | Param1             *(string)* <br> |
 | Sample Action Async | This is a sample action |  |
-| Sample Action Combo | This is a sample action | Param1             *(combo)* <br> |
 | Sample Action | This is a sample action | Param1             *(string)* <br> |
 
 
 ---
 ## Conditions
 | Condition | Description | Params
 | --- | --- | --- |
+| Sample Condition | This is a sample condition | Param1 *(combo)* <br> |
 | Sample Condition | This is a sample condition | Param1 *(string)* <br> |
 | Sample Trigger | This is a sample trigger |  |
-| Sample Condition | This is a sample condition | Param1 *(combo)* <br> |
 | Sample Condition | This is a sample condition |  |
 
 
 ---
 ## Expressions
 | Expression | Description | Return Type | Params
 | --- | --- | --- | --- |
-| Expression | Sample Expression | number |  | 
 | Expression2 | Sample Expression | string |  | 
+| Expression | Sample Expression | number |  | 
 | SampleExpression | This is a sample expression | string |  | 

build/build.js

@@ -17,6 +17,7 @@ const buildSteps = [
   "./generateAddonJSON.js",
   "./generateLangJSON.js",
   "./exportWebpack.js",
+  "./validateTerser.js",
   "./buildDomside.js",
   "./generateWrapperExtension.js",
   "./processDependencies.js",

build/doDev.js

@@ -13,6 +13,7 @@ build([
   "./generateAddonJSON.js",
   "./generateLangJSON.js",
   "./exportWebpack.js",
+  "./validateTerser.js",
   "./buildDomside.js",
   "./generateWrapperExtensionDev.js",
   "./processDependencies.js",

build/validateTerser.js

@@ -0,0 +1,71 @@
+import path from "path";
+import fs from "fs";
+import { fileURLToPath } from "url";
+import { minify } from "terser";
+import * as chalkUtils from "./chalkUtils.js";
+import fromConsole from "./fromConsole.js";
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+
+export default async function validateTerser() {
+  chalkUtils.step("Validating Terser build (mangle-props keep_quoted)");
+
+  const filesToCheck = [
+    "../dist/export/c3runtime/main.js",
+    "../dist/export/editor.js",
+  ];
+
+  let hadError = false;
+
+  for (const file of filesToCheck) {
+    const absolutePath = path.resolve(__dirname, file);
+    if (!fs.existsSync(absolutePath)) {
+      chalkUtils.info(`Skipping ${file} as it does not exist.`);
+      continue;
+    }
+
+    try {
+      const code = fs.readFileSync(absolutePath, "utf8");
+      const result = await minify(code, {
+        mangle: {
+          properties: {
+            keep_quoted: true,
+          },
+        },
+        compress: {
+          dead_code: true,
+          drop_console: false,
+          drop_debugger: true,
+          keep_classnames: false,
+          keep_fargs: true,
+          keep_fnames: false,
+          keep_infinity: false,
+        },
+      });
+
+      if (result.error) {
+        throw result.error;
+      }
+      chalkUtils.success(`Terser validation passed for ${file}`);
+    } catch (error) {
+      chalkUtils.error(`Terser validation failed for ${file}`);
+      chalkUtils.error(error.message || error);
+      hadError = true;
+    }
+  }
+
+  if (hadError) {
+    chalkUtils.failed("Terser validation failed.");
+  } else {
+    chalkUtils.success("Terser validation successful!");
+  }
+
+  return hadError;
+}
+
+// if is being called from the command line
+if (fromConsole(import.meta.url)) {
+  chalkUtils.fromCommandLine();
+  validateTerser();
+}

package-lock.json

@@ -7,10 +7,11 @@
     "cors": "^2.8.5",
     "express": "^4.21.2",
     "joi": "^17.13.3",
+    "open": "^10.1.0",
+    "terser": "^5.44.1",
     "vite": "^6.0.7",
     "webpack-cli": "^6.0.1",
-    "yocto-spinner": "^0.1.2",
-    "open": "^10.1.0"
+    "yocto-spinner": "^0.1.2"
   },
   "type": "module",
   "scripts": {
@@ -25,6 +26,5 @@
     "init": "npm install && node build/init.js",
     "generateDocs": "cd build && node generateDocumentation.js",
     "publish": "cd build && node publish.js"
-  },
-  "dependencies": {}
-}
+  }
+}