chore: use globby to improve git ignore file handling

Author: pnodet

Diff for: bun.lockb

@@ -1,6 +1,6 @@
 {
   "name": "@nivalis/linter",
-  "version": "0.0.8",
+  "version": "0.0.10",
   "module": "src/index.ts",
   "type": "module",
   "publishConfig": {
@@ -59,8 +59,7 @@
   "dependencies": {
     "@biomejs/js-api": "^0.6.2",
     "eslint": "^9.11.0",
-    "fast-glob": "^3.3.2",
-    "ignore": "^6.0.2",
+    "globby": "^14.0.2",
     "yargs": "^17.7.2"
   }
 }

Diff for: package.json

@@ -268,21 +268,11 @@ export const lintWithBiome = async (
     performance.mark("biome-start");
   }
 
-  const {stream, filter} = getFilesToLint(patterns);
-
-  const biomeResults: ESLint.LintResult[] = [];
-
-  for await (const file of stream) {
-    const filePath = file as string;
-
-    if (!filter(path.relative(process.cwd(), filePath))) {
-      continue;
-    }
-
-    const result = await biomeLintFile(biome, config, filePath, fix, unsafe);
-    biomeResults.push(result);
-  }
+  const files = await getFilesToLint(patterns);
 
+  const biomeResults: ESLint.LintResult[] = await Promise.all(
+    files.map(async (file) => await biomeLintFile(biome, config, file, fix, unsafe)),
+  );
 
   if (debug) {
     performance.mark("biome-end");

Diff for: src/biome.ts

@@ -1,46 +1,11 @@
-import path from "node:path";
-import ignore from 'ignore'
-import fs from "node:fs";
-import fg from "fast-glob";
-
-export const getFilesToLint =  (patterns: string[]) => {
-
-  const gitignore = path.join(process.cwd(), ".gitignore");
-  const ignorePatterns: string[] = [".git"];
-
-  if (fs.existsSync(gitignore)) {
-    const gitignoreContent = fs.readFileSync(gitignore, "utf8");
-    const lines = gitignoreContent
-    .split("\n")
-    .map((line) => line.trim())
-    .filter((line) => line && !line.startsWith("#") && !line.startsWith("!"));
-
-    ignorePatterns.push(...lines);
-  }
-
-  const ig = ignore().add(ignorePatterns);
-
-  const globPatterns = patterns.flatMap((pattern) => {
-    if (!fs.existsSync(pattern)) {
-      return pattern;
-    }
-
-    const stats = fs.lstatSync(pattern);
-
-    if (stats.isDirectory()) {
-      return path.join(pattern, "**/*.{js,jsx,ts,tsx}");
-    }
-
-    if (stats.isFile()) {
-      return pattern;
-    }
-
-    return [];
-  });
-
-  return {stream: fg.stream(globPatterns, { dot: false, absolute: true, ignore:
-     ignorePatterns
-   }),
-   filter: ig.createFilter()
-  };
+import {globby} from 'globby';
+
+export const getFilesToLint =  async (patterns: string[]) => {
+ return await globby(patterns, {
+  gitignore: true,
+  absolute: true,
+  expandDirectories: {
+    extensions: ['js', 'jsx', 'ts', 'tsx'],
+  },
+});
 };