fix: do not try to load configuration from files that are not checked out

.changeset/wet-cups-jog.md

@@ -0,0 +1,5 @@
+---
+'lint-staged': patch
+---
+
+_Lint-staged_ no longer tries to load configuration from files that are not checked out. This might happen when using sparse-checkout.

.github/workflows/push.yml

@@ -94,7 +94,7 @@ jobs:
         # Test with Ubuntu, macOS, and Windows
         os:
           - ubuntu-latest
-          - macos-latest
+          - macos-14 # macOS Sonoma M1 runners
           - windows-latest
     name: Node.js v${{ matrix.node }} on ${{ matrix.os }}
     runs-on: ${{ matrix.os }}

lib/searchConfigs.js

@@ -13,10 +13,9 @@ import { CONFIG_FILE_NAMES } from './configFiles.js'
 
 const debugLog = debug('lint-staged:searchConfigs')
 
-const EXEC_GIT = ['ls-files', '-z', '--full-name']
+const EXEC_GIT = ['ls-files', '-z', '--full-name', '-t']
 
-const filterPossibleConfigFiles = (files) =>
-  files.filter((file) => CONFIG_FILE_NAMES.includes(path.basename(file)))
+const filterPossibleConfigFiles = (file) => CONFIG_FILE_NAMES.includes(path.basename(file))
 
 const numberOfLevels = (file) => file.split('/').length
 
@@ -58,17 +57,23 @@ export const searchConfigs = async (
     return { [configPath]: validateConfig(config, filepath, logger) }
   }
 
-  const [cachedFiles, otherFiles] = await Promise.all([
+  const [cachedFilesWithStatus, otherFilesWithStatus] = await Promise.all([
     /** Get all possible config files known to git */
-    execGit(EXEC_GIT, { cwd: gitDir }).then(parseGitZOutput).then(filterPossibleConfigFiles),
+    execGit(EXEC_GIT, { cwd: gitDir }).then(parseGitZOutput),
     /** Get all possible config files from uncommitted files */
-    execGit([...EXEC_GIT, '--others', '--exclude-standard'], { cwd: gitDir })
-      .then(parseGitZOutput)
-      .then(filterPossibleConfigFiles),
+    execGit([...EXEC_GIT, '--others', '--exclude-standard'], { cwd: gitDir }).then(parseGitZOutput),
   ])
 
   /** Sort possible config files so that deepest is first */
-  const possibleConfigFiles = [...cachedFiles, ...otherFiles]
+  const possibleConfigFiles = [...cachedFilesWithStatus, ...otherFilesWithStatus]
+    .flatMap(
+      /**
+       * Leave out lines starting with "S " to ignore not-checked-out files in a sparse repo.
+       * The "S" status means a tracked file that is "skip-worktree"
+       * @see https://git-scm.com/docs/git-ls-files#Documentation/git-ls-files.txt--t
+       */ (line) => (line.startsWith('S ') ? [] : [line.replace(/^[HSMRCK?U] /, '')])
+    )
+    .filter(filterPossibleConfigFiles)
     .map((file) => normalizePath(path.join(gitDir, file)))
     .filter(isInsideDirectory(cwd))
     .sort(sortDeepestParth)

test/unit/searchConfigs.spec.js

@@ -101,7 +101,7 @@ describe('searchConfigs', () => {
     const config = { '*.js': 'eslint' }
 
     execGit.mockResolvedValueOnce(
-      `.lintstagedrc.json\u0000even/deeper/.lintstagedrc.json\u0000deeper/.lintstagedrc.json\u0000`
+      `H .lintstagedrc.json\u0000H even/deeper/.lintstagedrc.json\u0000H deeper/.lintstagedrc.json\u0000`
     )
 
     const topLevelConfig = normalizePath(path.join(process.cwd(), '.lintstagedrc.json'))
@@ -118,4 +118,22 @@ describe('searchConfigs', () => {
 
     expect(Object.keys(configs)).toEqual([evenDeeperConfig, deeperConfig, topLevelConfig])
   })
+
+  it('should ignore config files skipped from the worktree (sparse checkout)', async () => {
+    const config = { '*.js': 'eslint' }
+
+    execGit.mockResolvedValueOnce(`H .lintstagedrc.json\u0000S skipped/.lintstagedrc.json\u0000`)
+
+    const topLevelConfig = normalizePath(path.join(process.cwd(), '.lintstagedrc.json'))
+    const skippedConfig = normalizePath(path.join(process.cwd(), 'skipped/.lintstagedrc.json'))
+
+    loadConfig.mockResolvedValueOnce({ config, filepath: topLevelConfig })
+
+    // Mock will return config for skipped file, but it should not be read
+    loadConfig.mockResolvedValueOnce({ config, filepath: skippedConfig })
+
+    const configs = await searchConfigs({})
+
+    expect(Object.keys(configs)).toEqual([topLevelConfig])
+  })
 })