lint-staged 17.0.8 → 17.1.1

package/lib/state.js

@@ -1,4 +1,4 @@
-import { GIT_ERROR, TASK_ERROR } from './messages.js'
+import { gitError, taskError } from './messages.js'
 import {
   FailOnChangesError,
   GitError,
@@ -56,31 +56,31 @@ export const updateIndexSkipped = (ctx) => {
 
   // Should be skipped in case of git errors
   if (ctx.errors.has(GitError)) {
-    return GIT_ERROR
+    return gitError()
   }
 
   // Should be skipped when tasks fail
   if (ctx.errors.has(TaskError)) {
-    return TASK_ERROR
+    return taskError()
   }
 }
 
 export const restoreUnstagedChangesSkipped = (ctx) => {
   // Should be skipped in case of git errors
   if (ctx.errors.has(GitError)) {
-    return GIT_ERROR
+    return gitError()
   }
 
   // When complete reverting to original state is skipped,
   // we can still restore unstaged changes to make it easier
   // to do manually.
   if (!ctx.shouldRevert) {
-    false
+    return false
   }
 
   // Should be skipped when tasks fail
   if (ctx.errors.has(TaskError)) {
-    return TASK_ERROR
+    return taskError()
   }
 }
 
@@ -94,7 +94,7 @@ export const restoreOriginalStateEnabled = (ctx) =>
 export const restoreOriginalStateSkipped = (ctx) => {
   // Should be skipped in case of unknown git errors
   if (ctx.errors.has(GitError) && !ctx.errors.has(RestoreUnstagedChangesError)) {
-    return GIT_ERROR
+    return gitError()
   }
 }
 
@@ -106,13 +106,18 @@ export const cleanupSkipped = (ctx) => {
     return true
   }
 
+  /** Failed to restore hidden unstaged changes, shouldn't drop stash, @see restoreUnstagedChangesSkipped */
+  if (ctx.errors.has(RestoreUnstagedChangesError) && !ctx.shouldRevert) {
+    return gitError()
+  }
+
   // Should be skipped in case of unknown git errors
   if (restoreOriginalStateSkipped(ctx)) {
-    return GIT_ERROR
+    return gitError()
   }
 
   // Should be skipped when reverting to original state fails
   if (ctx.errors.has(RestoreOriginalStateError)) {
-    return GIT_ERROR
+    return gitError()
   }
 }

package/lib/validateOptions.js

@@ -8,6 +8,9 @@ import { InvalidOptionsError } from './symbols.js'
 
 const debugLog = createDebug('lint-staged:validateOptions')
 
+/** @type {(value: number) => boolean} */
+const isValidIntegerValue = (value) => Number.isInteger(value) || value === Infinity
+
 /**
  * Validate lint-staged options, either from the Node.js API or the command line flags.
  * @param {*} options
@@ -17,17 +20,41 @@ const debugLog = createDebug('lint-staged:validateOptions')
 export const validateOptions = async (options = {}, logger) => {
   debugLog('Validating options...')
 
+  const { concurrent, cwd, maxArgLength } = options
+
+  if (
+    concurrent !== undefined &&
+    typeof concurrent !== 'boolean' &&
+    (!isValidIntegerValue(concurrent) || concurrent < 1)
+  ) {
+    logger.error(
+      invalidOption('concurrent', `${concurrent}`, 'Must be boolean, positive integer or Infinite')
+    )
+    throw InvalidOptionsError
+  }
+
   /** Ensure the passed cwd option exists; it might also be relative */
-  if (typeof options.cwd === 'string') {
+  if (typeof cwd === 'string') {
     try {
-      const resolved = path.resolve(options.cwd)
+      const resolved = path.resolve(cwd)
       await fs.access(resolved, constants.F_OK)
     } catch (error) {
       debugLog('Failed to validate options: %o', options)
-      logger.error(invalidOption('cwd', options.cwd, error.message))
+      logger.error(invalidOption('cwd', cwd, error.message))
       throw InvalidOptionsError
     }
   }
 
+  if (
+    maxArgLength !== undefined &&
+    maxArgLength !== null &&
+    (!isValidIntegerValue(maxArgLength) || maxArgLength < 1)
+  ) {
+    logger.error(
+      invalidOption('maxArgLength', `${maxArgLength}`, 'Must be positive integer, or Infinite')
+    )
+    throw InvalidOptionsError
+  }
+
   debugLog('Validated options: %o', options)
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lint-staged",
-  "version": "17.0.8",
+  "version": "17.1.1",
   "description": "Lint files staged by git",
   "license": "MIT",
   "repository": {
@@ -44,7 +44,7 @@
     "MIGRATION.md"
   ],
   "scripts": {
-    "lint": "eslint",
+    "lint": "oxfmt && oxlint",
     "test": "vitest",
     "typecheck": "tsc",
     "version": "npx changeset version",
@@ -52,8 +52,7 @@
     "tag": "npx changeset tag"
   },
   "dependencies": {
-    "listr2": "^10.2.1",
-    "picomatch": "^4.0.4",
+    "picomatch": "^4.0.5",
     "string-argv": "^0.3.2",
     "tinyexec": "^1.2.4"
   },
@@ -62,36 +61,32 @@
   },
   "devDependencies": {
     "@changesets/changelog-github": "0.7.0",
-    "@changesets/cli": "2.31.0",
-    "@commitlint/cli": "21.0.2",
-    "@commitlint/config-conventional": "21.0.2",
-    "@eslint/js": "10.0.1",
-    "@vitest/coverage-istanbul": "4.1.9",
-    "@vitest/eslint-plugin": "1.6.20",
+    "@changesets/cli": "2.31.1",
+    "@commitlint/cli": "21.2.1",
+    "@commitlint/config-conventional": "21.2.0",
+    "@vitest/coverage-istanbul": "4.1.10",
     "consolemock": "1.1.0",
     "cross-env": "10.1.0",
-    "eslint": "10.5.0",
-    "eslint-config-prettier": "10.1.8",
-    "eslint-plugin-n": "18.1.0",
-    "eslint-plugin-prettier": "5.5.6",
-    "eslint-plugin-simple-import-sort": "13.0.0",
     "husky": "9.1.7",
     "mock-stdin": "1.0.0",
-    "prettier": "3.8.4",
-    "semver": "7.8.4",
-    "vitest": "4.1.9"
+    "oxfmt": "0.59.0",
+    "oxlint": "1.74.0",
+    "semver": "7.8.5",
+    "vitest": "4.1.10"
   },
   "keywords": [
-    "lint",
-    "git",
-    "staged",
+    "check",
+    "code",
     "eslint",
+    "format",
+    "git",
+    "lint",
+    "oxlint",
+    "oxfmt",
     "prettier",
-    "stylelint",
-    "code",
     "quality",
-    "check",
-    "format",
+    "staged",
+    "stylelint",
     "validate"
   ]
 }

package/lib/getRenderer.js

@@ -1,63 +0,0 @@
-import { EOL } from 'node:os'
-import { Writable } from 'node:stream'
-
-import { ListrLogger, ProcessOutput } from 'listr2'
-
-const EOLRegex = new RegExp(EOL + '$')
-
-const bindLogger = (consoleLogMethod) =>
-  new Writable({
-    write: function (chunk, encoding, next) {
-      consoleLogMethod(chunk.toString().replace(EOLRegex, ''))
-      next()
-    },
-  })
-
-const getMainRendererOptions = ({ color, debug, quiet }, logger, env) => {
-  if (quiet) {
-    return {
-      renderer: 'silent',
-    }
-  }
-
-  if (env.NODE_ENV === 'test') {
-    return {
-      renderer: 'test',
-      rendererOptions: {
-        logger: new ListrLogger({
-          processOutput: new ProcessOutput(bindLogger(logger.log), bindLogger(logger.error)),
-        }),
-      },
-    }
-  }
-
-  if (debug || !color) {
-    return {
-      renderer: 'verbose',
-    }
-  }
-
-  return {
-    renderer: 'update',
-    rendererOptions: {
-      formatOutput: 'truncate',
-    },
-  }
-}
-
-const getFallbackRenderer = ({ renderer }, { color }) => {
-  if (renderer === 'silent' || renderer === 'test' || !color) {
-    return renderer
-  }
-
-  return 'verbose'
-}
-
-export const getRenderer = ({ color, debug, quiet }, logger, env = process.env) => {
-  const mainRendererOptions = getMainRendererOptions({ color, debug, quiet }, logger, env)
-
-  return {
-    ...mainRendererOptions,
-    fallbackRenderer: getFallbackRenderer(mainRendererOptions, { color }),
-  }
-}