lint-staged 10.5.4 → 11.1.1

package/README.md

@@ -6,11 +6,11 @@ Run linters against staged git files and don't let :poop: slip into your code ba
 
 ## Why
 
-Linting makes more sense when run before committing your code. By doing so you can ensure no errors go into the repository and enforce code style. But running a lint process on a whole project is slow and linting results can be irrelevant. Ultimately you only want to lint files that will be committed.
+Linting makes more sense when run before committing your code. By doing so you can ensure no errors go into the repository and enforce code style. But running a lint process on a whole project is slow, and linting results can be irrelevant. Ultimately you only want to lint files that will be committed.
 
 This project contains a script that will run arbitrary shell tasks with a list of staged files as an argument, filtered by a specified glob pattern.
 
-## Related blogs posts and talks
+## Related blog posts and talks
 
 - [Introductory Medium post - Andrey Okonetchnikov, 2016](https://medium.com/@okonetchnikov/make-linting-great-again-f3890e1ad6b8#.8qepn2b5l)
 - [Running Jest Tests Before Each Git Commit - Ben McCormick, 2017](https://benmccormick.org/2017/02/26/running-jest-tests-before-each-git-commit/)
@@ -22,23 +22,23 @@ This project contains a script that will run arbitrary shell tasks with a list o
 
 ## Installation and setup
 
-The fastest way to start using lint-staged is to run following command in your terminal:
+The fastest way to start using lint-staged is to run the following command in your terminal:
 
 ```bash
-npx mrm lint-staged
+npx mrm@2 lint-staged
 ```
 
-It will install and configure [husky](https://github.com/typicode/husky) and lint-staged depending on code quality tools from `package.json` dependencies so please make sure you install (`npm install --save-dev`) and configure all code quality tools like [Prettier](https://prettier.io), [ESlint](https://eslint.org) prior that.
+This command will install and configure [husky](https://github.com/typicode/husky) and lint-staged depending on the code quality tools from your project's `package.json` dependencies, so please make sure you install (`npm install --save-dev`) and configure all code quality tools like [Prettier](https://prettier.io) and [ESLint](https://eslint.org) prior to that.
 
-Don't forget to commit changes to `package.json` to share this setup with your team!
+Don't forget to commit changes to `package.json` and `.husky` to share this setup with your team!
 
-Now change a few files, `git add` or `git add --patch` some of them to your commit and try to `git commit` them.
+Now change a few files, `git add` or `git add --patch` some of them to your commit, and try to `git commit` them.
 
 See [examples](#examples) and [configuration](#configuration) for more information.
 
 ## Changelog
 
-See [Releases](https://github.com/okonet/lint-staged/releases)
+See [Releases](https://github.com/okonet/lint-staged/releases).
 
 ### Migration
 
@@ -48,10 +48,10 @@ See [Releases](https://github.com/okonet/lint-staged/releases)
   If your task previously contained a `git add` step, please remove this.
   The automatic behaviour ensures there are less race-conditions,
   since trying to run multiple git operations at the same time usually results in an error.
-- From `v10.0.0` onwards _lint-staged_ uses git stashes to improve speed and provide backups while running.
-  Since git stashes require at least an initial commit, you shouldn't run _lint-staged_ in an empty repo.
-- From `v10.0.0` onwards _lint-staged_ requires Node.js version 10.13.0 or later.
-- From `v10.0.0` onwards _lint-staged_ will abort the commit if linter tasks undo all staged changes. To allow creating empty commit, please use the `--allow-empty` option.
+- From `v10.0.0` onwards, lint-staged uses git stashes to improve speed and provide backups while running.
+  Since git stashes require at least an initial commit, you shouldn't run lint-staged in an empty repo.
+- From `v10.0.0` onwards, lint-staged requires Node.js version 10.13.0 or later.
+- From `v10.0.0` onwards, lint-staged will abort the commit if linter tasks undo all staged changes. To allow creating an empty commit, please use the `--allow-empty` option.
 
 ## Command line flags
 
@@ -71,7 +71,7 @@ Options:
                                      tasks serially (default: true)
   -q, --quiet                        disable lint-staged’s own console output (default: false)
   -r, --relative                     pass relative filepaths to tasks (default: false)
-  -x, --shell                        skip parsing of tasks for better shell support (default:
+  -x, --shell [path]                 skip parsing of tasks for better shell support (default:
                                      false)
   -v, --verbose                      show task output even when tasks succeed; by default only
                                      failed output is shown (default: false)
@@ -79,7 +79,7 @@ Options:
 ```
 
 - **`--allow-empty`**: By default, when linter tasks undo all staged changes, lint-staged will exit with an error and abort the commit. Use this flag to allow creating empty git commits.
-- **`--config [path]`**: Manually specify a path to a config file or npm package name. Note: when used, lint-staged won't perform the config file search and print an error if the specified file cannot be found. If '-' is provided as the filename then the config will be read from stdin, allowing piping in the config like `cat my-config.json | npx lint-staged --config -`.
+- **`--config [path]`**: Manually specify a path to a config file or npm package name. Note: when used, lint-staged won't perform the config file search and will print an error if the specified file cannot be found. If '-' is provided as the filename then the config will be read from stdin, allowing piping in the config like `cat my-config.json | npx lint-staged --config -`.
 - **`--debug`**: Run in debug mode. When set, it does the following:
   - uses [debug](https://github.com/visionmedia/debug) internally to log additional information about staged files, commands being executed, location of binaries, etc. Debug logs, which are automatically enabled by passing the flag, can also be enabled by setting the environment variable `$DEBUG` to `lint-staged*`.
   - uses [`verbose` renderer](https://github.com/SamVerschueren/listr-verbose-renderer) for `listr`; this causes serial, uncoloured output to the terminal, instead of the default (beautified, dynamic) output.
@@ -90,12 +90,12 @@ Options:
 - **`--no-stash`**: By default a backup stash will be created before running the tasks, and all task modifications will be reverted in case of an error. This option will disable creating the stash, and instead leave all modifications in the index when aborting the commit.
 - **`--quiet`**: Supress all CLI output, except from tasks.
 - **`--relative`**: Pass filepaths relative to `process.cwd()` (where `lint-staged` runs) to tasks. Default is `false`.
-- **`--shell`**: By default linter commands will be parsed for speed and security. This has the side-effect that regular shell scripts might not work as expected. You can skip parsing of commands with this option.
+- **`--shell`**: By default linter commands will be parsed for speed and security. This has the side-effect that regular shell scripts might not work as expected. You can skip parsing of commands with this option. To use a specific shell, use a path like `--shell "/bin/bash"`.
 - **`--verbose`**: Show task output even when tasks succeed. By default only failed output is shown.
 
 ## Configuration
 
-Starting with v3.1 you can now use different ways of configuring it:
+Starting with v3.1 you can now use different ways of configuring lint-staged:
 
 - `lint-staged` object in your `package.json`
 - `.lintstagedrc` file in JSON or YML format, or you can be explicit with the file extension:
@@ -135,7 +135,7 @@ So, considering you did `git add file1.ext file2.ext`, lint-staged will run the
 
 ## Filtering files
 
-Linter commands work on a subset of all staged files, defined by a _glob pattern_. `lint-staged´ uses [micromatch](https://github.com/micromatch/micromatch) for matching files with the following rules:
+Linter commands work on a subset of all staged files, defined by a _glob pattern_. lint-staged uses [micromatch](https://github.com/micromatch/micromatch) for matching files with the following rules:
 
 - If the glob pattern contains no slashes (`/`), micromatch's `matchBase` option will enabled, so globs match a file's basename regardless of directory:
   - **`"*.js"`** will match all JS files, like `/test.js` and `/foo/bar/test.js`
@@ -144,7 +144,7 @@ Linter commands work on a subset of all staged files, defined by a _glob pattern
   - **`"./*.js"`** will match all JS files in the git repo root, so `/test.js` but not `/foo/bar/test.js`
   - **`"foo/**/\*.js"`** will match all JS files inside the`/foo`directory, so`/foo/bar/test.js`but not`/test.js`
 
-When matching, `lint-staged` will do the following
+When matching, lint-staged will do the following
 
 - Resolve the git root automatically, no configuration needed.
 - Pick the staged files which are present inside the project directory.
@@ -153,11 +153,11 @@ When matching, `lint-staged` will do the following
 
 **NOTE:** `lint-staged` will pass _absolute_ paths to the linters to avoid any confusion in case they're executed in a different working directory (i.e. when your `.git` directory isn't the same as your `package.json` directory).
 
-Also see [How to use `lint-staged` in a multi package monorepo?](#how-to-use-lint-staged-in-a-multi-package-monorepo)
+Also see [How to use `lint-staged` in a multi-package monorepo?](#how-to-use-lint-staged-in-a-multi-package-monorepo)
 
 ### Ignoring files
 
-The concept of `lint-staged` is to run configured linter (or other) tasks on files that are staged in git. `lint-staged` will always pass a list of all staged files to the task, and ignoring any files should be configured in the task itself.
+The concept of `lint-staged` is to run configured linter tasks (or other tasks) on files that are staged in git. `lint-staged` will always pass a list of all staged files to the task, and ignoring any files should be configured in the task itself.
 
 Consider a project that uses [`prettier`](https://prettier.io/) to keep code format consistent across all files. The project also stores minified 3rd-party vendor libraries in the `vendor/` directory. To keep `prettier` from throwing errors on these files, the vendor directory should be added to prettier's ignore configuration, the `.prettierignore` file. Running `npx prettier .` will ignore the entire vendor directory, throwing no errors. When `lint-staged` is added to the project and configured to run prettier, all modified and staged files in the vendor directory will be ignored by prettier, even though it receives them as input.
 
@@ -167,7 +167,7 @@ In advanced scenarios, where it is impossible to configure the linter task itsel
 
 Supported are any executables installed locally or globally via `npm` as well as any executable from your \$PATH.
 
-> Using globally installed scripts is discouraged, since lint-staged may not work for someone who doesn’t have it installed.
+> Using globally installed scripts is discouraged, since lint-staged may not work for someone who doesn't have it installed.
 
 `lint-staged` uses [execa](https://github.com/sindresorhus/execa#preferlocal) to locate locally installed scripts. So in your `.lintstagedrc` you can write:
 
@@ -195,11 +195,11 @@ going to execute `eslint` and if it exits with `0` code, it will execute `pretti
 
 ## Using JS configuration file
 
-Writing the configuration file in JavaScript is the most powerful way to configure _lint-staged_ (`lint-staged.config.js`, [similar](https://github.com/okonet/lint-staged/README.md#configuration), or passed via `--config`). From the configuration file, you can export either a single function, or an object.
+Writing the configuration file in JavaScript is the most powerful way to configure lint-staged (`lint-staged.config.js`, [similar](https://github.com/okonet/lint-staged/README.md#configuration), or passed via `--config`). From the configuration file, you can export either a single function or an object.
 
-If the `exports` value is a function, it will receive an array of all staged filenames. You can then build your own matchers for the files, and return a command string, or an array or command strings. These strings are considered complete and should include the filename arguments, if wanted.
+If the `exports` value is a function, it will receive an array of all staged filenames. You can then build your own matchers for the files and return a command string or an array of command strings. These strings are considered complete and should include the filename arguments, if wanted.
 
-If the `exports` value is an object, its keys should be glob matches (like in the normal non-js config format). The values can either be like in the normal config, or individual functions like described above. Instead of receiving all matched files, the functions in the exported object will only receive the staged files matching the corresponding glob key.
+If the `exports` value is an object, its keys should be glob matches (like in the normal non-js config format). The values can either be like in the normal config or individual functions like described above. Instead of receiving all matched files, the functions in the exported object will only receive the staged files matching the corresponding glob key.
 
 ### Function signature
 
@@ -259,7 +259,7 @@ module.exports = {
 
 </details>
 
-### Example: Run eslint on entire repo if more than 10 staged files
+### Example: Run ESLint on entire repo if more than 10 staged files
 
 <details>
   <summary>Click to expand</summary>
@@ -353,7 +353,7 @@ Prior to version 10, tasks had to manually include `git add` as the final step.
 
 ## Examples
 
-All examples assuming you’ve already set up lint-staged and husky in the `package.json`.
+All examples assume you've already set up lint-staged in the `package.json` file and [husky](https://github.com/typicode/husky) in its own config file.
 
 ```json
 {
@@ -362,16 +362,20 @@ All examples assuming you’ve already set up lint-staged and husky in the `pack
   "scripts": {
     "my-custom-script": "linter --arg1 --arg2"
   },
-  "husky": {
-    "hooks": {
-      "pre-commit": "lint-staged"
-    }
-  },
   "lint-staged": {}
 }
 ```
 
-_Note we don’t pass a path as an argument for the runners. This is important since lint-staged will do this for you._
+In `.husky/pre-commit`
+
+```shell
+#!/bin/sh
+. "$(dirname "$0")/_/husky.sh"
+
+npx lint-staged
+```
+
+_Note: we don't pass a path as an argument for the runners. This is important since lint-staged will do this for you._
 
 ### ESLint with default parameters for `*.js` and `*.jsx` running as a pre-commit hook
 
@@ -468,13 +472,15 @@ module.exports = (allStagedFiles) => {
     allStagedFiles,
     prettierSupportedExtensions.map((extension) => `**/*${extension}`)
   )
-  return [`prettier --write ${prettierFiles.map(addQuotes).join(' ')}`]
+  return prettierFiles.length > 0
+    ? [`prettier --write ${prettierFiles.map(addQuotes).join(' ')}`]
+    : []
 }
 ```
 
 </details>
 
-### Automatically fix code style with `prettier` for javascript, typescript, markdown, HTML, or CSS
+### Automatically fix code style with `prettier` for JavaScript, TypeScript, Markdown, HTML, or CSS
 
 <details>
   <summary>Click to expand</summary>
@@ -648,7 +654,7 @@ _Thanks to [this comment](https://youtrack.jetbrains.com/issue/IDEA-135454#comme
 
 </details>
 
-### How to use `lint-staged` in a multi package monorepo?
+### How to use `lint-staged` in a multi-package monorepo?
 
 <details>
   <summary>Click to expand</summary>
@@ -680,7 +686,7 @@ Example repo: [sudo-suhas/lint-staged-django-react-demo](https://github.com/sudo
 
 </details>
 
-### How can i ignore files from `.eslintignore` ?
+### How can I ignore files from `.eslintignore`?
 
 <details>
   <summary>Click to expand</summary>
@@ -711,16 +717,16 @@ module.exports = {
 
 </details>
 
-#### ESlint >= 7
+#### ESLint >= 7
 
 <details>
   <summary>Click to expand</summary>
 
-In versions of ESlint > 7, [isPathIgnored](https://eslint.org/docs/developer-guide/nodejs-api#-eslintispathignoredfilepath) is an async function and now returns a promise. The code below can be used to reinstate the above functionality.
+In versions of ESLint > 7, [isPathIgnored](https://eslint.org/docs/developer-guide/nodejs-api#-eslintispathignoredfilepath) is an async function and now returns a promise. The code below can be used to reinstate the above functionality.
 
 This particular code uses a tiny package, [node-filter-async](https://www.npmjs.com/package/node-filter-async), to filter the files array with an async function. If you prefer to not have an extra dependency, it is quite simple to write a similar function.
 
-Since [10.5.3](https://github.com/okonet/lint-staged/releases), any errors due to a bad eslint config will come through to the console.
+Since [10.5.3](https://github.com/okonet/lint-staged/releases), any errors due to a bad ESLint config will come through to the console.
 
 ```js
 const { ESLint } = require('eslint')

package/bin/lint-staged.js

@@ -17,7 +17,7 @@ const pkg = require('../package.json')
 require('please-upgrade-node')(
   Object.assign({}, pkg, {
     engines: {
-      node: '>=10.13.0', // First LTS release of 'Dubnium'
+      node: '>=12.13.0', // First LTS release of 'Erbium'
     },
   })
 )
@@ -42,7 +42,7 @@ cmdline
   )
   .option('-q, --quiet', 'disable lint-staged’s own console output', false)
   .option('-r, --relative', 'pass relative filepaths to tasks', false)
-  .option('-x, --shell', 'skip parsing of tasks for better shell support', false)
+  .option('-x, --shell [path]', 'skip parsing of tasks for better shell support', false)
   .option(
     '-v, --verbose',
     'show task output even when tasks succeed; by default only failed output is shown',
@@ -74,17 +74,19 @@ const getMaxArgLength = () => {
   }
 }
 
+const cmdlineOptions = cmdline.opts()
+
 const options = {
-  allowEmpty: !!cmdline.allowEmpty,
-  concurrent: JSON.parse(cmdline.concurrent),
-  configPath: cmdline.config,
-  debug: !!cmdline.debug,
+  allowEmpty: !!cmdlineOptions.allowEmpty,
+  concurrent: JSON.parse(cmdlineOptions.concurrent),
+  configPath: cmdlineOptions.config,
+  debug: !!cmdlineOptions.debug,
   maxArgLength: getMaxArgLength() / 2,
-  stash: !!cmdline.stash, // commander inverts `no-<x>` flags to `!x`
-  quiet: !!cmdline.quiet,
-  relative: !!cmdline.relative,
-  shell: !!cmdline.shell,
-  verbose: !!cmdline.verbose,
+  stash: !!cmdlineOptions.stash, // commander inverts `no-<x>` flags to `!x`
+  quiet: !!cmdlineOptions.quiet,
+  relative: !!cmdlineOptions.relative,
+  shell: cmdlineOptions.shell /* Either a boolean or a string pointing to the shell */,
+  verbose: !!cmdlineOptions.verbose,
 }
 
 debug('Options parsed from command-line:', options)

package/lib/index.js

@@ -8,13 +8,18 @@ const stringifyObject = require('stringify-object')
 const { PREVENTED_EMPTY_COMMIT, GIT_ERROR, RESTORE_STASH_EXAMPLE } = require('./messages')
 const printTaskOutput = require('./printTaskOutput')
 const runAll = require('./runAll')
-const { ApplyEmptyCommitError, GetBackupStashError, GitError } = require('./symbols')
+const {
+  ApplyEmptyCommitError,
+  ConfigNotFoundError,
+  GetBackupStashError,
+  GitError,
+  InvalidOptionsError,
+} = require('./symbols')
 const formatConfig = require('./formatConfig')
 const validateConfig = require('./validateConfig')
+const validateOptions = require('./validateOptions')
 
-const errConfigNotFound = new Error('Config could not be found')
-
-function resolveConfig(configPath) {
+const resolveConfig = (configPath) => {
   try {
     return require.resolve(configPath)
   } catch {
@@ -22,7 +27,7 @@ function resolveConfig(configPath) {
   }
 }
 
-function loadConfig(configPath) {
+const loadConfig = (configPath) => {
   const explorer = cosmiconfig('lint-staged', {
     searchPlaces: [
       'package.json',
@@ -56,14 +61,14 @@ function loadConfig(configPath) {
  * @param {number} [options.maxArgLength] - Maximum argument string length
  * @param {boolean} [options.quiet] - Disable lint-staged’s own console output
  * @param {boolean} [options.relative] - Pass relative filepaths to tasks
- * @param {boolean} [options.shell] - Skip parsing of tasks for better shell support
+ * @param {boolean|string} [options.shell] - Skip parsing of tasks for better shell support
  * @param {boolean} [options.stash] - Enable the backup stash, and revert in case of errors
  * @param {boolean} [options.verbose] - Show task output even when tasks succeed; by default only failed output is shown
  * @param {Logger} [logger]
  *
  * @returns {Promise<boolean>} Promise of whether the linting passed or failed
  */
-module.exports = async function lintStaged(
+const lintStaged = async (
   {
     allowEmpty = false,
     concurrent = true,
@@ -79,20 +84,27 @@ module.exports = async function lintStaged(
     verbose = false,
   } = {},
   logger = console
-) {
+) => {
   try {
+    await validateOptions({ shell }, logger)
+
     debugLog('Loading config using `cosmiconfig`')
 
     const resolved = configObject
       ? { config: configObject, filepath: '(input)' }
       : await loadConfig(configPath)
-    if (resolved == null) throw errConfigNotFound
+
+    if (resolved == null) {
+      throw ConfigNotFoundError
+    }
 
     debugLog('Successfully loaded config from `%s`:\n%O', resolved.filepath, resolved.config)
+
     // resolved.config is the parsed configuration object
     // resolved.filepath is the path to the config file that was found
     const formattedConfig = formatConfig(resolved.config)
     const config = validateConfig(formattedConfig)
+
     if (debug) {
       // Log using logger to be able to test through `consolemock`.
       logger.log('Running lint-staged with the following config:')
@@ -148,7 +160,13 @@ module.exports = async function lintStaged(
       throw runAllError
     }
   } catch (lintStagedError) {
-    if (lintStagedError === errConfigNotFound) {
+    /** throw early because `validateOptions` options contains own logging */
+    if (lintStagedError === InvalidOptionsError) {
+      throw InvalidOptionsError
+    }
+
+    /** @todo move logging to `validateConfig` and remove this try/catch block */
+    if (lintStagedError === ConfigNotFoundError) {
       logger.error(`${lintStagedError.message}.`)
     } else {
       // It was probably a parsing error
@@ -168,3 +186,5 @@ module.exports = async function lintStaged(
     throw lintStagedError
   }
 }
+
+module.exports = lintStaged

package/lib/messages.js

@@ -27,6 +27,14 @@ const SKIPPED_GIT_ERROR = 'Skipped because of previous git error.'
 
 const GIT_ERROR = `\n  ${error} ${chalk.red(`lint-staged failed due to a git error.`)}`
 
+const invalidOption = (name, value, message) => `${chalk.redBright(`${error} Validation Error:`)}
+
+  Invalid value for option ${chalk.bold(name)}: ${chalk.bold(value)}
+
+  ${message}
+
+See https://github.com/okonet/lint-staged#command-line-flags`
+
 const PREVENTED_EMPTY_COMMIT = `
   ${warning} ${chalk.yellow(`lint-staged prevented an empty git commit.
   Use the --allow-empty option to continue, or check your task configuration`)}
@@ -42,16 +50,17 @@ const RESTORE_STASH_EXAMPLE = `  Any lost modifications can be restored from a g
 const CONFIG_STDIN_ERROR = 'Error: Could not read config from stdin.'
 
 module.exports = {
-  NOT_GIT_REPO,
+  CONFIG_STDIN_ERROR,
+  DEPRECATED_GIT_ADD,
   FAILED_GET_STAGED_FILES,
+  GIT_ERROR,
+  invalidOption,
   NO_STAGED_FILES,
   NO_TASKS,
-  skippingBackup,
-  DEPRECATED_GIT_ADD,
-  TASK_ERROR,
-  SKIPPED_GIT_ERROR,
-  GIT_ERROR,
+  NOT_GIT_REPO,
   PREVENTED_EMPTY_COMMIT,
   RESTORE_STASH_EXAMPLE,
-  CONFIG_STDIN_ERROR,
+  SKIPPED_GIT_ERROR,
+  skippingBackup,
+  TASK_ERROR,
 }

package/lib/runAll.js

@@ -147,7 +147,8 @@ const runAll = async (
         matchedFiles.add(file)
       })
 
-      hasDeprecatedGitAdd = subTasks.some((subTask) => subTask.command === 'git add')
+      hasDeprecatedGitAdd =
+        hasDeprecatedGitAdd || subTasks.some((subTask) => subTask.command === 'git add')
 
       chunkListrTasks.push({
         title: `Running tasks for ${task.pattern}`,

package/lib/symbols.js

@@ -1,11 +1,13 @@
 'use strict'
 
 const ApplyEmptyCommitError = Symbol('ApplyEmptyCommitError')
+const ConfigNotFoundError = new Error('Config could not be found')
 const GetBackupStashError = Symbol('GetBackupStashError')
 const GetStagedFilesError = Symbol('GetStagedFilesError')
 const GitError = Symbol('GitError')
 const GitRepoError = Symbol('GitRepoError')
 const HideUnstagedChangesError = Symbol('HideUnstagedChangesError')
+const InvalidOptionsError = new Error('Invalid Options')
 const RestoreMergeStatusError = Symbol('RestoreMergeStatusError')
 const RestoreOriginalStateError = Symbol('RestoreOriginalStateError')
 const RestoreUnstagedChangesError = Symbol('RestoreUnstagedChangesError')
@@ -13,10 +15,12 @@ const TaskError = Symbol('TaskError')
 
 module.exports = {
   ApplyEmptyCommitError,
+  ConfigNotFoundError,
   GetBackupStashError,
   GetStagedFilesError,
   GitError,
   GitRepoError,
+  InvalidOptionsError,
   HideUnstagedChangesError,
   RestoreMergeStatusError,
   RestoreOriginalStateError,

package/lib/validateOptions.js

@@ -0,0 +1,31 @@
+const { promises: fs, constants } = require('fs')
+
+const { invalidOption } = require('./messages')
+const { InvalidOptionsError } = require('./symbols')
+
+const debug = require('debug')('lint-staged:options')
+
+/**
+ * Validate lint-staged options, either from the Node.js API or the command line flags.
+ * @param {*} options
+ * @param {boolean|string} [options.shell] - Skip parsing of tasks for better shell support
+ *
+ * @throws {InvalidOptionsError}
+ */
+const validateOptions = async (options = {}, logger) => {
+  debug('Validating options...')
+
+  /** Ensure the passed shell option is executable */
+  if (typeof options.shell === 'string') {
+    try {
+      await fs.access(options.shell, constants.X_OK)
+    } catch (error) {
+      logger.error(invalidOption('shell', options.shell, error.message))
+      throw InvalidOptionsError
+    }
+  }
+
+  debug('Validated options!')
+}
+
+module.exports = validateOptions

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lint-staged",
-  "version": "10.5.4",
+  "version": "11.1.1",
   "description": "Lint files staged by git",
   "license": "MIT",
   "repository": "https://github.com/okonet/lint-staged",
@@ -26,45 +26,40 @@
     "test": "jest --coverage",
     "test:watch": "jest --watch"
   },
-  "husky": {
-    "hooks": {
-      "pre-commit": "./bin/lint-staged.js"
-    }
-  },
   "dependencies": {
-    "chalk": "^4.1.0",
+    "chalk": "^4.1.1",
     "cli-truncate": "^2.1.0",
-    "commander": "^6.2.0",
+    "commander": "^7.2.0",
     "cosmiconfig": "^7.0.0",
-    "debug": "^4.2.0",
+    "debug": "^4.3.1",
     "dedent": "^0.7.0",
     "enquirer": "^2.3.6",
-    "execa": "^4.1.0",
-    "listr2": "^3.2.2",
-    "log-symbols": "^4.0.0",
-    "micromatch": "^4.0.2",
+    "execa": "^5.0.0",
+    "listr2": "^3.8.2",
+    "log-symbols": "^4.1.0",
+    "micromatch": "^4.0.4",
     "normalize-path": "^3.0.0",
     "please-upgrade-node": "^3.2.0",
     "string-argv": "0.3.1",
     "stringify-object": "^3.3.0"
   },
   "devDependencies": {
-    "@babel/core": "^7.12.3",
-    "@babel/plugin-proposal-object-rest-spread": "^7.12.1",
-    "@babel/preset-env": "^7.12.1",
+    "@babel/core": "^7.14.0",
+    "@babel/plugin-proposal-object-rest-spread": "^7.13.8",
+    "@babel/preset-env": "^7.14.1",
     "babel-eslint": "10.1.0",
-    "babel-jest": "^26.6.1",
+    "babel-jest": "^26.6.3",
     "consolemock": "^1.1.0",
-    "eslint": "^7.12.1",
-    "eslint-config-prettier": "^6.15.0",
+    "eslint": "^7.25.0",
+    "eslint-config-prettier": "^8.3.0",
     "eslint-plugin-import": "^2.22.1",
     "eslint-plugin-node": "^11.1.0",
-    "eslint-plugin-prettier": "^3.1.4",
-    "fs-extra": "^9.0.1",
-    "husky": "^4.3.0",
-    "jest": "^26.6.1",
+    "eslint-plugin-prettier": "^3.4.0",
+    "fs-extra": "^10.0.0",
+    "husky": "^6.0.0",
+    "jest": "^26.6.3",
     "jest-snapshot-serializer-ansi": "^1.0.0",
-    "prettier": "^2.1.2"
+    "prettier": "^2.2.1"
   },
   "config": {
     "commitizen": {