@vanillaes/esmtk 1.2.0 → 1.2.2

package/bin/commands/bundle.js

@@ -12,14 +12,16 @@ export async function bundle (input, output, options) {
   if (!npmExists) {
     console.error('npm not found')
     console.error('is node installed?')
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   const esbuildExists = await installed('esbuild')
   if (!esbuildExists) {
     console.error('esbuild not found')
     console.error('esbuild can be installed with `npm i -g esbuild`')
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   const args = []
@@ -34,7 +36,8 @@ export async function bundle (input, output, options) {
   spawn('esbuild', args, {
     cwd: process.cwd(),
     stdio: ['pipe', process.stdout, process.stderr]
-  }).on('error', err => {
-    console.error(err)
+  }).on('error', error => {
+    console.error(error)
+    process.exitCode = 1
   })
 }

package/bin/commands/clean.js

@@ -10,7 +10,8 @@ export async function clean (cwd, options) {
   const exists = await fileExists(cwd)
   if (!exists) {
     console.error(`clean: ${cwd} No such file or directory`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   if (options?.bundle) {

package/bin/commands/commonjs.js

@@ -12,14 +12,16 @@ export async function commonjs (input, output, options) {
   if (!npmExists) {
     console.error('npm not found')
     console.error('is node installed?')
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   const esbuildExists = await installed('esbuild')
   if (!esbuildExists) {
     console.error('esbuild not found')
     console.error('esbuild can be installed with `npm i -g esbuild`')
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   const args = []
@@ -34,7 +36,8 @@ export async function commonjs (input, output, options) {
   spawn('esbuild', args, {
     cwd: process.cwd(),
     stdio: ['pipe', process.stdout, process.stderr]
-  }).on('error', err => {
-    console.error(err)
+  }).on('error', error => {
+    console.error(error)
+    process.exitCode = 1
   })
 }

package/bin/commands/cp.js

@@ -9,6 +9,8 @@ import { expand } from '../../src/index.js'
 export async function cp (paths, options) {
   if (paths.length < 2) {
     console.error('cp: Not enough arguments')
+    process.exitCode = 1
+    return
   }
 
   if (paths.length === 2) {

package/bin/commands/init.js

@@ -17,22 +17,37 @@ export async function init (options) {
   if (!npmExists) {
     console.error('npm not found')
     console.error('is node installed?')
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   const gitExists = await which('git')
   if (!gitExists) {
     console.error('git not found')
     console.error('is git installed?')
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   // defaults
   const DIR = process.cwd()
   const DIRNAME = basename(process.cwd())
-  const REPOSITORY = await getGitRepository()
-  const USERNAME = gitExists ? await fetchGitUser() : ''
-  const EMAIL = gitExists ? await fetchGitEmail() : ''
+  let REPOSITORY
+  let USERNAME
+  let EMAIL
+  try {
+    REPOSITORY = await fetchGitRepository()
+    USERNAME = gitExists ? await fetchGitUser() : ''
+    EMAIL = gitExists ? await fetchGitEmail() : ''
+  } catch (error) {
+    if (error instanceof Error) {
+      console.log(error.message)
+    } else {
+      console.error(`Unexpected error: ${error}`)
+    }
+    process.exitCode = 1
+    return
+  }
 
   const program = createInterface({ input: stdin, output: stdout })
 
@@ -116,8 +131,7 @@ async function ask (program, prompt, defaultValue) {
 async function fetchGitUser () {
   const { stdout, stderr } = await execAsync('git config --get user.name')
   if (stderr) {
-    console.error(`exec error: ${stderr}`)
-    process.exit(1)
+    throw new Error(`exec error: ${stderr}`)
   }
   console.log(`${stdout}`.trim())
   return `${stdout}`.trim()
@@ -130,18 +144,17 @@ async function fetchGitUser () {
 async function fetchGitEmail () {
   const { stdout, stderr } = await execAsync('git config --get user.email')
   if (stderr) {
-    console.error(`exec error: ${stderr}`)
-    process.exit(1)
+    throw new Error(`exec error: ${stderr}`)
   }
   console.log(`${stdout}`.trim())
   return `${stdout}`.trim()
 }
 
 /**
- * Get the repository name from .git/config
+ * Fetch the repository name from .git/config
  * @returns {Promise<string>} the repository name
  */
-async function getGitRepository () {
+async function fetchGitRepository () {
   const config = join(process.cwd(), '.git', 'config')
   const exists = await fileExists(config)
   if (!exists) {

package/bin/commands/lint.js

@@ -16,8 +16,9 @@ export async function lint (options) {
   const child = spawn(BIN_PATH, args, {
     cwd: process.cwd(),
     stdio: ['pipe', 'pipe', 'pipe']
-  }).on('error', err => {
-    console.error(`${err}`)
+  }).on('error', error => {
+    console.error(`${error}`)
+    process.exitCode = 1
   })
 
   child.stdout.on('data', (data) => {
@@ -30,5 +31,6 @@ export async function lint (options) {
     } else {
       process.stderr.write(`${data}`)
     }
+    process.exitCode = 1
   })
 }

package/bin/commands/minify.js

@@ -12,14 +12,16 @@ export async function minify (input, output, options) {
   if (!npmExists) {
     console.error('npm not found')
     console.error('is node installed?')
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   const esbuildExists = await installed('esbuild')
   if (!esbuildExists) {
     console.error('esbuild not found')
     console.error('esbuild can be installed with `npm i -g esbuild`')
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   const args = []
@@ -38,7 +40,8 @@ export async function minify (input, output, options) {
   spawn('esbuild', args, {
     cwd: process.cwd(),
     stdio: ['pipe', process.stdout, process.stderr]
-  }).on('error', err => {
-    console.error(err)
+  }).on('error', error => {
+    console.error(error)
+    process.exitCode = 1
   })
 }

package/bin/commands/preview.js

@@ -17,7 +17,7 @@ export async function preview (options) {
   let files = await match('**/*', options.cwd, ignore)
   if (files.length === 0) {
     console.log('preview: no files found')
-    process.exit(0)
+    return
   }
   files = files
     .filter(path => statSync(path).isFile())

package/bin/commands/test.js

@@ -21,10 +21,20 @@ export async function test (glob, options) {
     args.push('--watch')
   }
 
-  spawn('node', args, {
+  const child = spawn('node', args, {
     cwd: process.cwd(),
-    stdio: ['pipe', process.stdout, process.stderr]
-  }).on('error', err => {
-    console.error(err)
+    stdio: ['pipe', 'pipe', 'pipe']
+  }).on('error', error => {
+    console.error(error)
+    process.exitCode = 1
+  })
+
+  child.stdout.on('data', (data) => {
+    process.stdout.write(`${data}`)
+  })
+
+  child.stderr.on('data', (data) => {
+    process.stderr.write(`${data}`)
+    process.exitCode = 1
   })
 }

package/bin/commands/type.js

@@ -11,14 +11,16 @@ export async function type (entry, options) {
   if (!npmExists) {
     console.error('npm not found')
     console.error('is node installed?')
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   const esbuildExists = await installed('typescript')
   if (!esbuildExists) {
     console.error('typescript not found')
     console.error('typescript can be installed with `npm i -g typescript`')
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   const args = []
@@ -44,7 +46,8 @@ export async function type (entry, options) {
   spawn('tsc', args, {
     cwd: process.cwd(),
     stdio: ['pipe', process.stdout, process.stderr]
-  }).on('error', err => {
-    console.error(err)
+  }).on('error', error => {
+    console.error(error)
+    process.exitCode = 1
   })
 }

package/bin/commands/typings.js

@@ -10,20 +10,23 @@ export async function typings (entry) {
   if (!npmExists) {
     console.error('npm not found')
     console.error('is node installed?')
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   const esbuildExists = await installed('typescript')
   if (!esbuildExists) {
     console.error('typescript not found')
     console.error('typescript can be installed with `npm i -g typescript`')
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   spawn('tsc', [entry, '-t', 'esnext', '--allowJS', '--checkJS', '--skipLibCheck', '--declaration', '--emitDeclarationOnly', '--noEmitOnError'], {
     cwd: process.cwd(),
     stdio: ['pipe', process.stdout, process.stderr]
-  }).on('error', err => {
-    console.error(err)
+  }).on('error', error => {
+    console.error(error)
+    process.exitCode = 1
   })
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vanillaes/esmtk",
-  "version": "1.2.0",
+  "version": "1.2.2",
   "description": "ES Module Toolkit",
   "keywords": [
     "ecmascript",
@@ -54,7 +54,7 @@
   },
   "dependencies": {
     "@vanillaes/lint-es": "^1.0.0",
-    "@vanillaes/tape-es": "^4.0.3",
+    "@vanillaes/tape-es": "^4.0.5",
     "commander": "^14.0.3"
   },
   "devDependencies": {

package/src/cp.js

@@ -12,16 +12,19 @@ export async function copyAsync (source, target, force = false) {
   const sExists = await fileExists(source)
   if (!sExists) {
     console.error(`cp: ${source} No such file or directory`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
   const sStats = await stat(source)
   if (sStats.isSymbolicLink()) {
     console.error(`cp: ${source} is a sybolic link (not copied)`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
   if (!sStats.isFile()) {
     console.error(`cp: ${source} is a directory (not copied)`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   const tExists = await fileExists(target)
@@ -30,12 +33,14 @@ export async function copyAsync (source, target, force = false) {
   if (tIsDir) {
     if (!tExists) {
       console.error(`cp: ${target} No such file or directory`)
-      process.exit(1)
+      process.exitCode = 1
+      return
     }
     const tStats = await stat(target)
     if (tStats.isSymbolicLink()) {
       console.error(`cp: ${target} is a sybolic link (not copied)`)
-      process.exit(1)
+      process.exitCode = 1
+      return
     }
 
     // append source file name to target directory
@@ -46,8 +51,9 @@ export async function copyAsync (source, target, force = false) {
 
   try {
     await cp(source, target, { force: true })
-  } catch (err) {
-    console.error(`cp: error ${err.message}`)
+  } catch (error) {
+    console.error(`cp: error ${error.message}`)
+    process.exitCode = 1
   }
 }
 
@@ -61,12 +67,14 @@ export async function copyMultipleAsync (sources, target, force = false) {
   const tExists = await fileExists(target)
   if (!tExists) {
     console.error(`cp: ${target} No such file or directory`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
   const tStats = await stat(target)
   if (tStats.isSymbolicLink()) {
     console.error(`cp: ${target} is a sybolic link (not copied)`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   try {
@@ -74,8 +82,9 @@ export async function copyMultipleAsync (sources, target, force = false) {
     for (const source of sources) {
       await cp(source, `${target}${sep}${basename(source)}`, { force: true })
     }
-  } catch (err) {
-    console.error(`cp": error ${err.message}`)
+  } catch (error) {
+    console.error(`cp": error ${error.message}`)
+    process.exitCode = 1
   }
 }
 
@@ -89,32 +98,38 @@ export async function copyRecursiveAsync (source, target, force = false) {
   const sExists = await fileExists(source)
   if (!sExists) {
     console.error(`cp: ${source} No such file or directory`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
   const sStats = await stat(source)
   if (sStats.isSymbolicLink()) {
     console.error(`cp: ${source} is a sybolic link (not copied)`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   const tExists = await fileExists(target)
   if (!tExists) {
     console.error(`cp: ${target} No such file or directory`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
   const tStats = await stat(target)
   if (tStats.isSymbolicLink()) {
     console.error(`cp: ${target} is a sybolic link (not copied)`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
   if (!tStats.isDirectory()) {
     console.error(`cp: target ${target} Not a directory`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   try {
     await cp(source, target, { force: true, recursive: true })
-  } catch (err) {
-    console.error(`cp": error ${err.message}`)
+  } catch (error) {
+    console.error(`cp": error ${error.message}`)
+    process.exitCode = 1
   }
 }

package/src/rm.js

@@ -10,22 +10,25 @@ export async function removeAsync (path, force = false) {
   const exists = await fileExists(path)
   if (!exists) {
     console.error(`rm: ${path} No such file or directory`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
   const stats = await stat(path)
   if (stats.isSymbolicLink()) {
     console.error(`rm: ${path} is a sybolic link`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
   if (!stats.isFile()) {
     console.error(`rm: ${path} is a directory`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   try {
     await rm(path, { force: true })
-  } catch (err) {
-    console.error(`rm: error ${err.message}`)
+  } catch (error) {
+    console.error(`rm: error ${error.message}`)
   }
 }
 
@@ -39,8 +42,8 @@ export async function removeMultipleAsync (files, force = false) {
     for (const file of files) {
       await rm(file, { force: true })
     }
-  } catch (err) {
-    console.error(`cp": error ${err.message}`)
+  } catch (error) {
+    console.error(`cp": error ${error.message}`)
   }
 }
 
@@ -53,17 +56,19 @@ export async function removeRecursiveAsync (path, force = false) {
   const exists = await fileExists(path)
   if (!exists) {
     console.error(`rm: ${path} No such file or directory`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
   const stats = await stat(path)
   if (stats.isSymbolicLink()) {
     console.error(`rm: ${path} is a sybolic link`)
-    process.exit(1)
+    process.exitCode = 1
+    return
   }
 
   try {
     await rm(path, { force: true, recursive: true })
-  } catch (err) {
-    console.error(`rm": error ${err.message}`)
+  } catch (error) {
+    console.error(`rm": error ${error.message}`)
   }
 }

package/src/util.js

@@ -15,14 +15,16 @@ export async function expand (source) {
     const paths = await match(source)
     if (paths.length === 0) {
       console.error(`${source} no matches found`)
-      process.exit(1)
+      process.exitCode = 1
+      return
     }
     return paths
   } else {
     const exists = await fileExists(source)
     if (!exists) {
       console.error(`${source} No such file or directory`)
-      process.exit(1)
+      process.exitCode = 1
+      return
     }
     return [source]
   }
@@ -51,7 +53,7 @@ export async function installed (pkg) {
   try {
     await execAsync(`npm list -g --depth=0 ${pkg}`)
     return true
-  } catch (e) {
+  } catch (error) {
     return false
   }
 }
@@ -80,7 +82,7 @@ export async function which (program) {
   try {
     await execAsync(`which ${program}`)
     return true
-  } catch (e) {
+  } catch (error) {
     return false
   }
 }