lint-staged 12.1.2 → 12.1.6

package/bin/lint-staged.js

@@ -44,10 +44,13 @@ cmdline
   )
   .parse(process.argv)
 
-const debugLog = debug('lint-staged:bin')
-if (cmdline.debug) {
+const cmdlineOptions = cmdline.opts()
+
+if (cmdlineOptions.debug) {
   debug.enable('lint-staged*')
 }
+
+const debugLog = debug('lint-staged:bin')
 debugLog('Running `lint-staged@%s`', version)
 
 /**
@@ -68,8 +71,6 @@ const getMaxArgLength = () => {
   }
 }
 
-const cmdlineOptions = cmdline.opts()
-
 const options = {
   allowEmpty: !!cmdlineOptions.allowEmpty,
   concurrent: JSON.parse(cmdlineOptions.concurrent),

package/lib/index.js

@@ -58,7 +58,7 @@ const lintStaged = async (
 ) => {
   await validateOptions({ shell }, logger)
 
-  const inputConfig = configObject || (await loadConfig(configPath, logger))
+  const inputConfig = configObject || (await loadConfig({ configPath, cwd }, logger))
 
   if (!inputConfig) {
     logger.error(`${ConfigNotFoundError.message}.`)

package/lib/loadConfig.js

@@ -1,3 +1,5 @@
+/** @typedef {import('./index').Logger} Logger */
+
 import { pathToFileURL } from 'url'
 
 import debug from 'debug'
@@ -56,20 +58,23 @@ const resolveConfig = (configPath) => {
 }
 
 /**
- * @param {string} [configPath]
- * @param {Logger} [logger]
+ * @param {object} options
+ * @param {string} [options.configPath] - Explicit path to a config file
+ * @param {string} [options.cwd] - Current working directory
  */
-export const loadConfig = async (configPath, logger) => {
+export const loadConfig = async ({ configPath, cwd }, logger) => {
   try {
     if (configPath) {
       debugLog('Loading configuration from `%s`...', configPath)
     } else {
-      debugLog('Searching for configuration...')
+      debugLog('Searching for configuration from `%s`...', cwd)
     }
 
     const explorer = lilconfig('lint-staged', { searchPlaces, loaders })
 
-    const result = await (configPath ? explorer.load(resolveConfig(configPath)) : explorer.search())
+    const result = await (configPath
+      ? explorer.load(resolveConfig(configPath))
+      : explorer.search(cwd))
     if (!result) return null
 
     // config is a promise when using the `dynamicImport` loader
@@ -79,7 +84,7 @@ export const loadConfig = async (configPath, logger) => {
 
     return config
   } catch (error) {
-    debugLog('Failed to load configuration from `%s`', configPath)
+    debugLog('Failed to load configuration!')
     logger.error(error)
     return null
   }

package/lib/makeCmdTasks.js

@@ -28,13 +28,14 @@ const getTitleLength = (renderer, columns = process.stdout.columns) => {
  *
  * @param {object} options
  * @param {Array<string|Function>|string|Function} options.commands
+ * @param {string} options.cwd
  * @param {Array<string>} options.files
  * @param {string} options.gitDir
  * @param {string} options.renderer
  * @param {Boolean} shell
  * @param {Boolean} verbose
  */
-export const makeCmdTasks = async ({ commands, files, gitDir, renderer, shell, verbose }) => {
+export const makeCmdTasks = async ({ commands, cwd, files, gitDir, renderer, shell, verbose }) => {
   debugLog('Creating listr tasks for commands %o', commands)
   const commandArray = Array.isArray(commands) ? commands : [commands]
   const cmdTasks = []
@@ -61,7 +62,7 @@ export const makeCmdTasks = async ({ commands, files, gitDir, renderer, shell, v
 
       // Truncate title to single line based on renderer
       const title = cliTruncate(command, getTitleLength(renderer))
-      const task = resolveTaskFn({ command, files, gitDir, isFn, shell, verbose })
+      const task = resolveTaskFn({ command, cwd, files, gitDir, isFn, shell, verbose })
       cmdTasks.push({ title, command, task })
     }
   }

package/lib/resolveGitRepo.js

@@ -54,7 +54,7 @@ export const resolveGitRepo = async (cwd = process.cwd()) => {
 
     // read the path of the current directory relative to the top-level directory
     // don't read the toplevel directly, it will lead to an posix conform path on non posix systems (cygwin)
-    const gitRel = normalize(await execGit(['rev-parse', '--show-prefix']))
+    const gitRel = normalize(await execGit(['rev-parse', '--show-prefix'], { cwd }))
     const gitDir = determineGitDir(normalize(cwd), gitRel)
     const gitConfigDir = normalize(await resolveGitConfigDir(gitDir))
 

package/lib/resolveTaskFn.js

@@ -68,20 +68,20 @@ const makeErr = (command, result, ctx) => {
  *
  * @param {Object} options
  * @param {string} options.command — Linter task
+ * @param {string} [options.cwd]
  * @param {String} options.gitDir - Current git repo path
  * @param {Boolean} options.isFn - Whether the linter task is a function
  * @param {Array<string>} options.files — Filepaths to run the linter task against
- * @param {Boolean} [options.relative] — Whether the filepaths should be relative
  * @param {Boolean} [options.shell] — Whether to skip parsing linter task for better shell support
  * @param {Boolean} [options.verbose] — Always show task verbose
  * @returns {function(): Promise<Array<string>>}
  */
 export const resolveTaskFn = ({
   command,
+  cwd = process.cwd(),
   files,
   gitDir,
   isFn,
-  relative,
   shell = false,
   verbose = false,
 }) => {
@@ -89,14 +89,15 @@ export const resolveTaskFn = ({
   debugLog('cmd:', cmd)
   debugLog('args:', args)
 
-  const execaOptions = { preferLocal: true, reject: false, shell }
-  if (relative) {
-    execaOptions.cwd = process.cwd()
-  } else if (/^git(\.exe)?/i.test(cmd) && gitDir !== process.cwd()) {
+  const execaOptions = {
     // Only use gitDir as CWD if we are using the git binary
     // e.g `npm` should run tasks in the actual CWD
-    execaOptions.cwd = gitDir
+    cwd: /^git(\.exe)?/i.test(cmd) ? gitDir : cwd,
+    preferLocal: true,
+    reject: false,
+    shell,
   }
+
   debugLog('execaOptions:', execaOptions)
 
   return async (ctx = getInitialState()) => {

package/lib/runAll.js

@@ -135,6 +135,7 @@ export const runAll = async (
     for (const task of chunkTasks) {
       const subTasks = await makeCmdTasks({
         commands: task.commands,
+        cwd,
         files: task.fileList,
         gitDir,
         renderer: listrOptions.renderer,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lint-staged",
-  "version": "12.1.2",
+  "version": "12.1.6",
   "description": "Lint files staged by git",
   "license": "MIT",
   "repository": "https://github.com/okonet/lint-staged",
@@ -32,34 +32,33 @@
     "cli-truncate": "^3.1.0",
     "colorette": "^2.0.16",
     "commander": "^8.3.0",
-    "debug": "^4.3.2",
-    "enquirer": "^2.3.6",
+    "debug": "^4.3.3",
     "execa": "^5.1.1",
     "lilconfig": "2.0.4",
-    "listr2": "^3.13.3",
+    "listr2": "^3.13.5",
     "micromatch": "^4.0.4",
     "normalize-path": "^3.0.0",
-    "object-inspect": "^1.11.0",
+    "object-inspect": "^1.11.1",
     "string-argv": "^0.3.1",
-    "supports-color": "^9.0.2",
+    "supports-color": "^9.2.1",
     "yaml": "^1.10.2"
   },
   "devDependencies": {
-    "@babel/core": "^7.16.0",
-    "@babel/eslint-parser": "^7.16.3",
-    "@babel/preset-env": "^7.16.0",
-    "babel-jest": "^27.3.1",
+    "@babel/core": "^7.16.5",
+    "@babel/eslint-parser": "^7.16.5",
+    "@babel/preset-env": "^7.16.5",
+    "babel-jest": "^27.4.5",
     "consolemock": "^1.1.0",
-    "eslint": "^8.2.0",
+    "eslint": "^8.4.1",
     "eslint-config-prettier": "^8.3.0",
     "eslint-plugin-import": "^2.25.3",
     "eslint-plugin-node": "^11.1.0",
     "eslint-plugin-prettier": "^4.0.0",
     "fs-extra": "^10.0.0",
     "husky": "^7.0.4",
-    "jest": "^27.3.1",
+    "jest": "^27.4.5",
     "jest-snapshot-serializer-ansi": "^1.0.0",
-    "prettier": "^2.4.1"
+    "prettier": "^2.5.1"
   },
   "keywords": [
     "lint",