fix: optimize TypeScript project disable search

Author: aminya

src/typescript.cts

@@ -2,12 +2,12 @@ import { eslintRulesExtra } from "./official-eslint-rules.cjs"
 import { pluginImportRulesExtra, pluginImportTypeScriptRulesExtra } from "./plugin-import-rules.cjs"
 import { pluginNodeRules } from "./plugin-node-rules.cjs"
 import makeSynchronous from "make-synchronous"
-import { findOneFile } from "./utils.cjs"
+import { findFilesForGroups } from "./utils.cjs"
 import type { GlobifiedEntry } from "globify-gitignore"
 import { Linter } from "eslint"
 
 const tsFiles = ["**/*.tsx", "**/*.ts", "**/*.mts", "**/*.cts"]
-const project = ["**/tsconfig.json", "!**/node_modules/**/tsconfig.json"]
+const tscConfigFiles = ["**/tsconfig.json", "!**/node_modules/**/tsconfig.json"]
 
 async function globifyGitIgnoreFileWithDeps(cwd: string, include: boolean) {
   try {
@@ -54,18 +54,15 @@ function disableProjectBasedRules() {
   )
 
   // check if there are any ts files
-  const hasTsFile = findOneFile(cwd, tsFiles, ignore)
+  const [hasTscConfig, hasTsFile] = findFilesForGroups(cwd, [tscConfigFiles, tsFiles], ignore)
 
   // return if there are no ts files
   if (!hasTsFile) {
     return true
   }
 
-  // check if there is a tsconfig.json file
-  const hasTsConfig = findOneFile(cwd, project, ignore)
-
   // if there is no tsconfig.json file, but there are ts files, disable the project-based rules
-  const disable = !hasTsConfig && hasTsFile
+  const disable = !hasTscConfig && hasTsFile
 
   if (disable) {
     console.warn(
@@ -132,7 +129,7 @@ export const tsConfig: Linter.ConfigOverride<Linter.RulesRecord> = {
   files: tsFiles,
   parser: "@typescript-eslint/parser",
   parserOptions: {
-    project,
+    project: tscConfigFiles,
     createDefaultProgram: true, // otherwise Eslint will error if a ts file is not covered by one of the tsconfig.json files
   },
   plugins: ["@typescript-eslint", "node", "import", "only-warn"],

src/utils.cts

@@ -2,23 +2,30 @@ import { readdirSync } from "fs"
 import { join } from "path"
 import { default as anymatch } from "anymatch"
 
-export function findOneFile(cwd: string, search: string[], ignored: string[]) {
-  // recursively search the current folder for a file with the given fileEnding, ignoring the given folders, and return true as soon as one is found
-  const files = readdirSync(cwd, { withFileTypes: true, recursive: false })
+export function findFilesForGroups(cwd: string, searchGroups: string[][], ignored: string[]) {
+  const status = searchGroups.map(() => false);
+  searchDirectory(cwd, status, searchGroups, ignored);
+
+  return status
+}
+
+function searchDirectory(directory: string, status: boolean[], searchGroups: string[][], ignored: string[]) {
+    // recursively search the current folder for a file with the given fileEnding, ignoring the given folders, and return true as soon as one is found
+  const files = readdirSync(directory, { withFileTypes: true, recursive: false });
   for (const file of files) {
-    const path = join(cwd, file.name)
+    const path = join(directory, file.name);
     if (file.isDirectory()) {
       if (!anymatch(ignored, path)) {
         // if the folder is not ignored, search it recursively
-        const found = findOneFile(path, search, ignored)
-        if (found) {
-          return true
+        searchDirectory(path, status, searchGroups, ignored);
+      }
+    } else {
+      // for each search group, check if the file matches any of the patterns
+      for (const [index, searchGroup] of searchGroups.entries()) {
+        if (!status[index] && anymatch(searchGroup, path)) {
+          status[index] = true;
         }
       }
-    } else if (anymatch(search, path)) {
-      // if the file ends with the given fileEnding, return true
-      return true
     }
   }
-  return false
 }