@jcoreio/toolchain 4.9.0 → 4.9.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jcoreio/toolchain",
-  "version": "4.9.0",
+  "version": "4.9.1",
   "description": "base JS build toolchain",
   "repository": {
     "type": "git",

package/plugins/smokeTestBuild.cjs

@@ -0,0 +1,26 @@
+const getModules = require('../util/getModules.cjs')
+const execa = require('../util/execa.cjs')
+const path = require('path')
+
+module.exports = [
+  async function smokeTestBuild() {
+    const { cjs, esm } = await getModules('dist/package.json')
+
+    function relpath(file) {
+      const result = path.relative(process.cwd(), file)
+      return result.startsWith('.') ? result : `./${result}`
+    }
+
+    for (const file of cjs) {
+      await execa('node', ['-e', `require(${JSON.stringify(relpath(file))})`])
+    }
+    for (const file of esm) {
+      await execa('node', [
+        '--input-type',
+        'module',
+        '-e',
+        `import ${JSON.stringify(relpath(file))}`,
+      ])
+    }
+  },
+]

package/scripts/smokeTestBuild.cjs

@@ -1,26 +1,7 @@
-const getModules = require('../util/getModules.cjs')
-const execa = require('../util/execa.cjs')
-const path = require('path')
+const getPluginsAsyncFunction = require('../util/getPluginsAsyncFunction.cjs')
 
-exports.run = async function smokeTestBuild() {
-  const { cjs, esm } = await getModules('dist/package.json')
-
-  function relpath(file) {
-    const result = path.relative(process.cwd(), file)
-    return result.startsWith('.') ? result : `./${result}`
-  }
-
-  for (const file of cjs) {
-    await execa('node', ['-e', `require(${JSON.stringify(relpath(file))})`])
-  }
-  for (const file of esm) {
-    await execa('node', [
-      '--input-type',
-      'module',
-      '-e',
-      `import ${JSON.stringify(relpath(file))}`,
-    ])
-  }
+exports.run = async function smokeTestBuild(args) {
+  await getPluginsAsyncFunction('smokeTestBuild')(args)
 }
 
 exports.description = 'smoke test that build output can be required/imported'

package/scripts/toolchain.cjs

@@ -42,12 +42,17 @@ const scripts = toolchainConfig
       ...Object.fromEntries(
         Object.entries(toolchainConfig.scripts || {}).map(([name, script]) => [
           name,
-          typeof script === 'string'
+          typeof script === 'string' && script.trim()
             ? {
                 run: (args = []) =>
                   execa([script, ...args].join(' '), { shell: true }),
                 description: script,
               }
+            : !script || typeof script === 'string'
+            ? {
+                run: () => {},
+                description: '(no-op)',
+              }
             : script,
         ])
       ),