Fixing issue with string templates importing vm2 which requires other prebuilts for JS in the CLI - no need for these to add to build size when JS is unused.

packages/cli/src/environment.js

@@ -0,0 +1 @@
+process.env.NO_JS = "1"

packages/cli/src/index.js

@@ -1,5 +1,6 @@
 #!/usr/bin/env node
 require("./prebuilds")
+require("./environment")
 const { getCommands } = require("./options")
 const { Command } = require("commander")
 const { getHelpDescription } = require("./utils")

packages/cli/src/prebuilds.js

@@ -1,6 +1,7 @@
 const os = require("os")
 const { join } = require("path")
 const fs = require("fs")
+const { error } = require("./utils")
 const PREBUILDS = "prebuilds"
 const ARCH = `${os.platform()}-${os.arch()}`
 const PREBUILD_DIR = join(process.execPath, "..", PREBUILDS, ARCH)
@@ -25,7 +26,15 @@ function checkForBinaries() {
   }
 }
 
-function cleanup() {
+function cleanup(evt) {
+  if (evt && evt.errno) {
+    console.error(
+      error(
+        "Failed to run CLI command - please report with the following message:"
+      )
+    )
+    console.error(error(evt))
+  }
   if (fs.existsSync(PREBUILD_DIR)) {
     fs.rmSync(PREBUILD_DIR, { recursive: true })
   }

packages/string-templates/src/helpers/javascript.js

@@ -36,6 +36,9 @@ const getContextValue = (path, context) => {
 
 // Evaluates JS code against a certain context
 module.exports.processJS = (handlebars, context) => {
+  if (process && process.env.NO_JS) {
+    throw new Error("JS disabled in environment.")
+  }
   try {
     // Wrap JS in a function and immediately invoke it.
     // This is required to allow the final `return` statement to be valid.

packages/string-templates/src/index.cjs

@@ -1,6 +1,4 @@
-const { VM } = require("vm2")
 const templates = require("./index.js")
-const { setJSRunner } = require("./helpers/javascript")
 
 /**
  * CJS entrypoint for rollup
@@ -21,13 +19,17 @@ module.exports.disableEscaping = templates.disableEscaping
 module.exports.findHBSBlocks = templates.findHBSBlocks
 module.exports.convertToJS = templates.convertToJS
 
-/**
+if (!process.env.NO_JS) {
- * Use vm2 to run JS scripts in a node env
+  const { VM } = require("vm2")
- */
+  const { setJSRunner } = require("./helpers/javascript")
-setJSRunner((js, context) => {
+  /**
-  const vm = new VM({
+   * Use vm2 to run JS scripts in a node env
-    sandbox: context,
+   */
-    timeout: 1000
+  setJSRunner((js, context) => {
+    const vm = new VM({
+      sandbox: context,
+      timeout: 1000
+    })
+    return vm.run(js)
   })
-  return vm.run(js)
+}
-})

packages/string-templates/src/index.mjs

@@ -21,16 +21,18 @@ export const disableEscaping = templates.disableEscaping
 export const findHBSBlocks = templates.findHBSBlocks
 export const convertToJS = templates.convertToJS
 
-/**
+if (process && !process.env.NO_JS) {
- * Use polyfilled vm to run JS scripts in a browser Env
+  /**
- */
+   * Use polyfilled vm to run JS scripts in a browser Env
-setJSRunner((js, context) => {
+   */
-  context = {
+  setJSRunner((js, context) => {
-    ...context,
+    context = {
-    alert: undefined,
+      ...context,
-    setInterval: undefined,
+      alert: undefined,
-    setTimeout: undefined,
+      setInterval: undefined,
-  }
+      setTimeout: undefined,
-  vm.createContext(context)
+    }
-  return vm.runInNewContext(js, context, { timeout: 1000 })
+    vm.createContext(context)
-})
+    return vm.runInNewContext(js, context, { timeout: 1000 })
+  })
+}