es-check 7.0.0 → 7.0.1

package/README.md

@@ -39,6 +39,7 @@ Thanks to Anders for this deeper fix, to [Pavel Starosek](https://github.com/Stu
   <a href="#debugging">Debugging</a>&nbsp;&nbsp;
   <a href="#contributing">Contributing</a>&nbsp;&nbsp;
   <a href="/issues">Issues</a>
+  <a href="#roadmap">Roadmap</a>
 </p>
 
 ---
@@ -213,3 +214,14 @@ ES Check has 3 main dependencies: [acorn](https://github.com/ternjs/acorn/), [gl
 - [Ben Junya](https://github.com/MrBenJ)
 - [Jeff Barczewski](https://github.com/jeffbski)
 - [Brandon Casey](https://github.com/BrandonOCasey)
+
+### Roadmap
+
+- Provide compilation step to support esm
+  - non-user-facing
+  - required to keep package dependencies up-to-date as more dependencies are ESM-only
+- Provide checks for _theoretical_ keywork words
+  - Things like `Map` and `Object.assign` are not keywords that fail ECMAScript
+    compilation depending on specific versions of ECMAScript. However, they hint at additions to ECMAScript that previous version did not support.
+  - This feature will enhance an already built-in confiration feature to provide more out-of-the-box support for ECMAScript checking.
+  - If enabled, this feature will warn (or fail) based on _theoretical_ ECMAScript keywords.

package/index.js

@@ -25,35 +25,31 @@ program
   .version(pkg.version)
   .argument(
     '[ecmaVersion]',
-    'ecmaVersion to check files against. Can be: es3, es4, es5, es6/es2015, es7/es2016, es8/es2017, es9/es2018, es10/es2019'
-  )
-  .argument(
-    '[files...]',
-    'a glob of files to to test the EcmaScript version against'
+    'ecmaVersion to check files against. Can be: es3, es4, es5, es6/es2015, es7/es2016, es8/es2017, es9/es2018, es10/es2019',
   )
+  .argument('[files...]', 'a glob of files to to test the EcmaScript version against')
   .option('--module', 'use ES modules')
-  .option(
-    '--allow-hash-bang',
-    'if the code starts with #! treat it as a comment'
-  )
+  .option('--allow-hash-bang', 'if the code starts with #! treat it as a comment')
   .option('--not <files>', 'folder or file names to skip')
   .option('--no-color', 'disable use of colors in output')
   .option('-v, --verbose', 'verbose mode: will also output debug messages')
   .option('--quiet', 'quiet mode: only displays warn and error messages')
   .option(
     '--silent',
-    'silent mode: does not output anything, giving no indication of success or failure other than the exit code'
+    'silent mode: does not output anything, giving no indication of success or failure other than the exit code',
   )
   .action((ecmaVersionArg, filesArg, options) => {
     const logger = winston.createLogger()
-    logger.add(new winston.transports.Console({
-      silent: options.silent,
-      level: options.verbose ? 'silly' : options.quiet ? 'warn' : 'info',
-      format: winston.format.combine(
-        ...supportsColor.stdout ? [winston.format.colorize()] : [],
-        winston.format.simple()
-      )
-    }))
+    logger.add(
+      new winston.transports.Console({
+        silent: options.silent,
+        level: options.verbose ? 'silly' : options.quiet ? 'warn' : 'info',
+        format: winston.format.combine(
+          ...(supportsColor.stdout ? [winston.format.colorize()] : []),
+          winston.format.simple(),
+        ),
+      }),
+    )
 
     const configFilePath = path.resolve(process.cwd(), '.escheckrc')
 
@@ -63,34 +59,22 @@ program
      * - If one exists, default to those options
      * - If no command line arguments are passed in
      */
-    const config = fs.existsSync(configFilePath)
-      ? JSON.parse(fs.readFileSync(configFilePath))
-      : {}
-    const expectedEcmaVersion = ecmaVersionArg
-      ? ecmaVersionArg
-      : config.ecmaVersion
-    const files =
-      filesArg && filesArg.length ? filesArg : [].concat(config.files)
+    const config = fs.existsSync(configFilePath) ? JSON.parse(fs.readFileSync(configFilePath)) : {}
+    const expectedEcmaVersion = ecmaVersionArg ? ecmaVersionArg : config.ecmaVersion
+    const files = filesArg && filesArg.length ? filesArg : [].concat(config.files)
     const esmodule = options.module ? options.module : config.module
-    const allowHashBang = options.allowHashBang
-      ? options.allowHashBang
-      : config.allowHashBang
-    const pathsToIgnore =
-      options.not
-        ? options.not.split(',')
-        : [].concat(config.not || [])
+    const allowHashBang = options.allowHashBang ? options.allowHashBang : config.allowHashBang
+    const pathsToIgnore = options.not ? options.not.split(',') : [].concat(config.not || [])
 
     if (!expectedEcmaVersion) {
       logger.error(
-        'No ecmaScript version passed in or found in .escheckrc. Please set your ecmaScript version in the CLI or in .escheckrc'
+        'No ecmaScript version passed in or found in .escheckrc. Please set your ecmaScript version in the CLI or in .escheckrc',
       )
       process.exit(1)
     }
 
     if (!files || !files.length) {
-      logger.error(
-        'No files were passed in please pass in a list of files to es-check!'
-      )
+      logger.error('No files were passed in please pass in a list of files to es-check!')
       process.exit(1)
     }
 
@@ -151,9 +135,7 @@ program
         ecmaVersion = '2021'
         break
       default:
-        logger.error(
-          'Invalid ecmaScript version, please pass a valid version, use --help for help'
-        )
+        logger.error('Invalid ecmaScript version, please pass a valid version, use --help for help')
         process.exit(1)
     }
 
@@ -172,10 +154,7 @@ program
     const filterForIgnore = (globbedFiles) => {
       if (expandedPathsToIgnore && expandedPathsToIgnore.length > 0) {
         const filtered = globbedFiles.filter(
-          (filePath) =>
-            !expandedPathsToIgnore.some((ignoreValue) =>
-              filePath.includes(ignoreValue)
-            )
+          (filePath) => !expandedPathsToIgnore.some((ignoreValue) => filePath.includes(ignoreValue)),
         )
         return filtered
       }
@@ -198,9 +177,7 @@ program
       const globbedFiles = glob.sync(pattern, globOpts)
 
       if (globbedFiles.length === 0) {
-        logger.error(
-          `ES-Check: Did not find any files to check for ${pattern}.`
-        )
+        logger.error(`ES-Check: Did not find any files to check for ${pattern}.`)
         process.exit(1)
       }
 
@@ -208,18 +185,15 @@ program
 
       filteredFiles.forEach((file) => {
         const code = fs.readFileSync(file, 'utf8')
-
         logger.debug(`ES-Check: checking ${file}`)
         try {
           acorn.parse(code, acornOpts)
         } catch (err) {
-          logger.debug(
-            `ES-Check: failed to parse file: ${file} \n - error: ${err}`
-          )
+          logger.debug(`ES-Check: failed to parse file: ${file} \n - error: ${err}`)
           const errorObj = {
             err,
             stack: err.stack,
-            file
+            file,
           }
           errArray.push(errorObj)
         }
@@ -227,9 +201,7 @@ program
     })
 
     if (errArray.length > 0) {
-      logger.error(
-        `ES-Check: there were ${errArray.length} ES version matching errors.`
-      )
+      logger.error(`ES-Check: there were ${errArray.length} ES version matching errors.`)
       errArray.forEach((o) => {
         logger.info(`
           ES-Check Error:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "es-check",
-  "version": "7.0.0",
+  "version": "7.0.1",
   "description": "Checks the ECMAScript version of .js glob against a specified version of ECMAScript with a shell command",
   "main": "index.js",
   "license": "MIT",
@@ -11,16 +11,20 @@
     "es-check": "index.js"
   },
   "scripts": {
-    "chore:delete-branch": "if git show-ref --quiet refs/heads/chore-changelog; then git branch -D chore-changelog; fi",
-    "chore:branch": "git checkout -b chore-changelog",
-    "chore:changelog": "conventional-changelog -p angular -i CHANGELOG.md -s -r 0",
-    "chore:pr": "git add . && git commit -m '[chore] updates changelog' --no-verify && git push origin chore-changelog -f",
-    "chore": "pnpm run chore:delete-branch && pnpm run chore:branch && pnpm run chore:changelog && pnpm run chore:pr",
+    "commit": "git-cz",
+    "commit-msg": "commitlint --edit $1",
+    "husky-setup": "path-exists .husky/commit-msg || (husky install && pnpm husky-setup:commit-msg && pnpm husky-setup:post-merge && pnpm husky-setup:pre-commit)",
+    "husky-setup:commit-msg": "npx husky add .husky/commit-msg 'pnpm run commit-msg'",
+    "husky-setup:post-merge": "npx husky add .husky/post-merge 'pnpm run setup'",
+    "husky-setup:pre-commit": "npx husky add .husky/pre-commit 'pnpm run pre-commit'",
     "lint": "eslint index.js --fix",
     "lint:ci": "eslint index.js",
-    "prepush": "pnpm run lint && pnpm test",
-    "postpublish": "git tag $npm_package_version && git push origin --tags && pnpm run chore",
+    "pre-commit": "pnpm lint && pnpm test",
+    "prepare": "is-ci || pnpm husky-setup",
+    "prepublishOnly": "pnpm test",
+    "release": "release-it",
     "report:coverage": "nyc report --reporter=lcov > coverage.lcov && codecov",
+    "setup": "pnpm install --reporter=silent",
     "test": "nyc mocha test.js --timeout 10s --coverage"
   },
   "repository": {
@@ -45,7 +49,9 @@
     "husky": "^8.0.1",
     "mocha": "^10.0.0",
     "nyc": "^15.1.0",
-    "prettier": "^2.5.1"
+    "path-exists-cli": "^2.0.0",
+    "prettier": "^2.5.1",
+    "release-it": "^15.1.3"
   },
   "dependencies": {
     "acorn": "^8.7.0",
@@ -87,13 +93,9 @@
       ]
     }
   },
-  "husky": {
-    "hooks": {
-      "commit-msg": "commitlint -E HUSKY_GIT_PARAMS",
-      "post-checkout": "if [[ $HUSKY_GIT_PARAMS =~ 1$ ]]; then pnpm i -r; fi",
-      "post-merge": "pnpm i -r",
-      "post-rewrite": "pnpm i -r",
-      "pre-commit": "pnpm test && pnpm lint"
+  "release-it": {
+    "github": {
+      "release": true
     }
   }
 }