lint-staged 16.2.1 → 16.2.2

package/bin/lint-staged.js

@@ -6,7 +6,7 @@ import { Option, program } from 'commander'
 
 import { createDebug, enableDebug } from '../lib/debug.js'
 import lintStaged from '../lib/index.js'
-import { CONFIG_STDIN_ERROR, RESTORE_STASH_EXAMPLE } from '../lib/messages.js'
+import { CONFIG_STDIN_ERROR, restoreStashExample } from '../lib/messages.js'
 import { readStdin } from '../lib/readStdin.js'
 import { getVersion } from '../lib/version.js'
 
@@ -120,7 +120,7 @@ program
     ).default(false)
   )
 
-  .addHelpText('afterAll', '\n' + RESTORE_STASH_EXAMPLE)
+  .addHelpText('afterAll', '\n' + restoreStashExample())
 
 const cliOptions = program.parse(process.argv).opts()
 

package/lib/gitWorkflow.js

@@ -9,7 +9,7 @@ import { getDiffCommand } from './getDiffCommand.js'
 import { parseGitZOutput } from './parseGitZOutput.js'
 import {
   ApplyEmptyCommitError,
-  ExitCodeError,
+  FailOnChangesError,
   GetBackupStashError,
   GitError,
   HideUnstagedChangesError,
@@ -317,7 +317,7 @@ export class GitWorkflow {
       const diffSha256 = calculateSha256(diff)
       debugLog('SHA-256 hash of changes after tasks is %S', this.diffSha256)
       if (this.unstagedDiffSha256 !== diffSha256) {
-        ctx.errors.add(ExitCodeError)
+        ctx.errors.add(FailOnChangesError)
         throw new Error('Tasks modified files and --fail-on-changes was used!')
       }
     }

package/lib/index.js

@@ -6,7 +6,7 @@ import {
   NO_CONFIGURATION,
   PREVENTED_EMPTY_COMMIT,
   PREVENTED_TASK_MODIFICATIONS,
-  RESTORE_STASH_EXAMPLE,
+  restoreStashExample,
   UNSTAGED_CHANGES_BACKUP_STASH_LOCATION,
 } from './messages.js'
 import { printTaskOutput } from './printTaskOutput.js'
@@ -15,7 +15,7 @@ import { cleanupSkipped } from './state.js'
 import {
   ApplyEmptyCommitError,
   ConfigNotFoundError,
-  ExitCodeError,
+  FailOnChangesError,
   GetBackupStashError,
   GitError,
   RestoreUnstagedChangesError,
@@ -155,8 +155,9 @@ const lintStaged = async (
         logger.error(NO_CONFIGURATION)
       } else if (ctx.errors.has(ApplyEmptyCommitError)) {
         logger.warn(PREVENTED_EMPTY_COMMIT)
-      } else if (ctx.errors.has(ExitCodeError)) {
-        logger.warn(PREVENTED_TASK_MODIFICATIONS)
+      } else if (ctx.errors.has(FailOnChangesError)) {
+        logger.warn(PREVENTED_TASK_MODIFICATIONS + '\n')
+        logger.warn(restoreStashExample(ctx.backupHash))
       } else if (ctx.errors.has(RestoreUnstagedChangesError)) {
         logger.warn(UNSTAGED_CHANGES_BACKUP_STASH_LOCATION)
         logger.warn(ctx.unstagedPatch)
@@ -167,7 +168,7 @@ const lintStaged = async (
         logger.error(GIT_ERROR)
         if (ctx.shouldBackup) {
           // No sense to show this if the backup stash itself is missing.
-          logger.error(RESTORE_STASH_EXAMPLE + '\n')
+          logger.error(restoreStashExample(ctx.backupHash) + '\n')
         }
       }
 

package/lib/messages.js

@@ -66,11 +66,13 @@ export const PREVENTED_EMPTY_COMMIT = `
   Use the --allow-empty option to continue, or check your task configuration`)}
 `
 
-export const RESTORE_STASH_EXAMPLE = `Any lost modifications can be restored from a git stash:
+export const restoreStashExample = (
+  hash = 'h0a0s0h0'
+) => `Any lost modifications can be restored from a git stash:
 
   > git stash list --format="%h %s"
-  h0a0s0h0 On main: lint-staged automatic backup
-  > git apply --index h0a0s0h0`
+  ${hash} On main: lint-staged automatic backup
+  > git apply --index ${hash}`
 
 export const CONFIG_STDIN_ERROR = red(`${error} Failed to read config from stdin.`)
 

package/lib/state.js

@@ -2,7 +2,7 @@ import EventEmitter from 'events'
 
 import { GIT_ERROR, TASK_ERROR } from './messages.js'
 import {
-  ExitCodeError,
+  FailOnChangesError,
   GitError,
   RestoreOriginalStateError,
   RestoreUnstagedChangesError,
@@ -18,6 +18,7 @@ export const getInitialState = ({
   backupHash: null,
   errors: new Set([]),
   events: new EventEmitter(),
+  shouldFailOnChanges: null,
   hasFilesToHide: null,
   output: [],
   quiet,
@@ -37,10 +38,12 @@ export const shouldRestoreUnstagedChanges = (ctx) =>
 export const applyModificationsSkipped = (ctx) => {
   // Always apply back unstaged modifications when skipping revert or backup
   if (!ctx.shouldRevert || !ctx.shouldBackup) return false
+
   // Should be skipped in case of git errors
   if (ctx.errors.has(GitError)) {
     return GIT_ERROR
   }
+
   // Should be skipped when tasks fail
   if (ctx.errors.has(TaskError)) {
     return TASK_ERROR
@@ -53,6 +56,13 @@ export const restoreUnstagedChangesSkipped = (ctx) => {
     return GIT_ERROR
   }
 
+  // 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
+  }
+
   // Should be skipped when tasks fail
   if (ctx.errors.has(TaskError)) {
     return TASK_ERROR
@@ -62,17 +72,13 @@ export const restoreUnstagedChangesSkipped = (ctx) => {
 export const restoreOriginalStateEnabled = (ctx) =>
   !!ctx.shouldRevert &&
   !!ctx.shouldBackup &&
-  (ctx.errors.has(ExitCodeError) ||
+  (ctx.errors.has(FailOnChangesError) ||
     ctx.errors.has(TaskError) ||
     ctx.errors.has(RestoreUnstagedChangesError))
 
 export const restoreOriginalStateSkipped = (ctx) => {
   // Should be skipped in case of unknown git errors
-  if (
-    ctx.errors.has(GitError) &&
-    !ctx.errors.has(RestoreUnstagedChangesError) &&
-    !ctx.errors.has(ExitCodeError)
-  ) {
+  if (ctx.errors.has(GitError) && !ctx.errors.has(RestoreUnstagedChangesError)) {
     return GIT_ERROR
   }
 }
@@ -80,6 +86,11 @@ export const restoreOriginalStateSkipped = (ctx) => {
 export const cleanupEnabled = (ctx) => ctx.shouldBackup
 
 export const cleanupSkipped = (ctx) => {
+  // "--fail-on-changes" was used, so we shouldn't drop the backup stash
+  if (ctx.errors.has(FailOnChangesError) && !ctx.shouldRevert) {
+    return true
+  }
+
   // Should be skipped in case of unknown git errors
   if (restoreOriginalStateSkipped(ctx)) {
     return GIT_ERROR

package/lib/symbols.js

@@ -26,4 +26,4 @@ export const RestoreUnstagedChangesError = Symbol('RestoreUnstagedChangesError')
 
 export const TaskError = Symbol('TaskError')
 
-export const ExitCodeError = Symbol('ExitCodeError')
+export const FailOnChangesError = Symbol('FailOnChangesError')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lint-staged",
-  "version": "16.2.1",
+  "version": "16.2.2",
   "description": "Lint files staged by git",
   "license": "MIT",
   "repository": {