@fullstacksjs/eslint-config 12.4.0 → 12.4.1

package/cjs/index.d.ts

@@ -97,10 +97,10 @@ export interface Options extends Linter.Config {
    */
   ignores?: string[];
   /**
-   * .gitignore file path relative to ESLint configuration file.
+   * .gitignore file path relative to ESLint configuration file. Set to `false` to disable.
    * @default './.gitignore'
    */
-  gitignorePath?: string;
+  gitignore?: string | false;
 }
 
 export declare function defineConfig(initOptions?: Options, ...extend: Linter.Config[]): Linter.Config[];

package/cjs/index.js

@@ -39,7 +39,7 @@ const defaultOptions = {
   disableExpensiveRules: false,
   esm: false,
   ignores: [],
-  gitignorePath: './.gitignore',
+  gitignore: './.gitignore',
   import: {},
   jest: (0, _localPkg.isPackageExists)('jest'),
   next: (0, _localPkg.isPackageExists)('next'),
@@ -79,7 +79,7 @@ function defineConfig(initOptions = {}, ...extend) {
     disableExpensiveRules,
     esm: enableEsm,
     ignores: enableIgnores,
-    gitignorePath,
+    gitignore,
     import: enableImport,
     jest: enableJest,
     next: enableNext,

package/cjs/modules/ignore.js

@@ -4,18 +4,23 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 
-var _compat = require("@eslint/compat");
 var _config = require("eslint/config");
-var _nodePath = _interopRequireDefault(require("node:path"));
+var _getGitignorePatterns = require("../utils/getGitignorePatterns.js");
 var _globs = require("../utils/globs.js");
-function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 /**
  * @param { import('../index.js').Options } options
  * @return { import('eslint').Linter.Config }
  */
 function ignores(options = {}) {
-  const gitignorePath = _nodePath.default.resolve(process.cwd(), options.gitignorePath);
-  const gitignore = (0, _compat.includeIgnoreFile)(gitignorePath);
-  return (0, _config.globalIgnores)([...gitignore.ignores, ..._globs.ignoreGlobs, ...(options.ignores ?? [])], 'ignores');
+  const ignorePatterns = [..._globs.ignoreGlobs, ...(options.ignores ?? [])];
+  const shouldUseGitignore = options.gitignore && options.gitignore.length > 0;
+  if (shouldUseGitignore) {
+    const gitignorePatterns = (0, _getGitignorePatterns.getGitignorePatterns)(options.gitignore);
+    if (gitignorePatterns.length > 0) {
+      ignorePatterns.push(...gitignorePatterns);
+    }
+  }
+  const uniqueIgnorePatterns = [...new Set(ignorePatterns)];
+  return (0, _config.globalIgnores)(uniqueIgnorePatterns, 'ignores');
 }
 module.exports = ignores;

package/cjs/utils/getGitignorePatterns.js

@@ -0,0 +1,35 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.getGitignorePatterns = getGitignorePatterns;
+var _compat = require("@eslint/compat");
+var _nodeFs = _interopRequireDefault(require("node:fs"));
+var _nodePath = _interopRequireDefault(require("node:path"));
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
+/* eslint 'no-console': ['warn', { allow: ['warn'] }] */
+
+const warningMessage = '\u001B[38;2;229;229;14mWarning:\u001B[0m';
+const fallbackMessage = 'Falling back to default ignore patterns. You can suppress this warning by setting \u001B[48;5;234m`gitignore: false`\u001B[0m in the config.';
+
+/**
+ * @param {string} gitignorePath relative path to the .gitignore file
+ * @returns {string[]}
+ */
+function getGitignorePatterns(gitignorePath) {
+  const gitignoreAbsolutePath = _nodePath.default.resolve(process.cwd(), gitignorePath);
+  const gitignoreExists = _nodeFs.default.existsSync(gitignoreAbsolutePath); // eslint-disable-line n/no-sync
+  const gitignorePatterns = [];
+  if (!gitignoreExists) {
+    console.warn(warningMessage, `No .gitignore file found at "${gitignoreAbsolutePath}".`, fallbackMessage);
+    return gitignorePatterns;
+  }
+  const ignorePatterns = (0, _compat.includeIgnoreFile)(gitignoreAbsolutePath).ignores ?? [];
+  if (ignorePatterns.length > 0) {
+    gitignorePatterns.push(...ignorePatterns);
+  } else {
+    console.warn(warningMessage, `No patterns found in .gitignore file at "${gitignoreAbsolutePath}".`, fallbackMessage);
+  }
+  return gitignorePatterns;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fullstacksjs/eslint-config",
-  "version": "12.4.0",
+  "version": "12.4.1",
   "license": "MIT",
   "author": "fullstacks <fullstacksjs@gmail.com>",
   "description": "fullstacks eslint config",
@@ -23,7 +23,7 @@
     "postbuild": "./postbuild",
     "prepublishOnly": "pinst --disable",
     "postpublish": "pinst --enable",
-    "pre-commit": "npm run test && lint-staged"
+    "pre-commit": "npm run test && lint-staged && npm run spell"
   },
   "main": "./cjs/index.js",
   "module": "./src/index.mjs",

package/src/index.d.ts

@@ -97,10 +97,10 @@ export interface Options extends Linter.Config {
    */
   ignores?: string[];
   /**
-   * .gitignore file path relative to ESLint configuration file.
+   * .gitignore file path relative to ESLint configuration file. Set to `false` to disable.
    * @default './.gitignore'
    */
-  gitignorePath?: string;
+  gitignore?: string | false;
 }
 
 export declare function defineConfig(initOptions?: Options, ...extend: Linter.Config[]): Linter.Config[];

package/src/index.mjs

@@ -33,7 +33,7 @@ const defaultOptions = {
   disableExpensiveRules: false,
   esm: false,
   ignores: [],
-  gitignorePath: './.gitignore',
+  gitignore: './.gitignore',
   import: {},
   jest: isPackageExists('jest'),
   next: isPackageExists('next'),
@@ -72,7 +72,7 @@ export function defineConfig(initOptions = {}, ...extend) {
     disableExpensiveRules,
     esm: enableEsm,
     ignores: enableIgnores,
-    gitignorePath,
+    gitignore,
     import: enableImport,
     jest: enableJest,
     next: enableNext,

package/src/modules/ignore.mjs

@@ -1,7 +1,6 @@
-import { includeIgnoreFile } from '@eslint/compat';
 import { globalIgnores } from 'eslint/config';
-import path from 'node:path';
 
+import { getGitignorePatterns } from '../utils/getGitignorePatterns.mjs';
 import { ignoreGlobs } from '../utils/globs.mjs';
 
 /**
@@ -9,9 +8,19 @@ import { ignoreGlobs } from '../utils/globs.mjs';
  * @return { import('eslint').Linter.Config }
  */
 function ignores(options = {}) {
-  const gitignorePath = path.resolve(process.cwd(), options.gitignorePath);
-  const gitignore = includeIgnoreFile(gitignorePath);
-  return globalIgnores([...gitignore.ignores, ...ignoreGlobs, ...(options.ignores ?? [])], 'ignores');
+  const ignorePatterns = [...ignoreGlobs, ...(options.ignores ?? [])];
+
+  const shouldUseGitignore = options.gitignore && options.gitignore.length > 0;
+  if (shouldUseGitignore) {
+    const gitignorePatterns = getGitignorePatterns(options.gitignore);
+    if (gitignorePatterns.length > 0) {
+      ignorePatterns.push(...gitignorePatterns);
+    }
+  }
+
+  const uniqueIgnorePatterns = [...new Set(ignorePatterns)];
+
+  return globalIgnores(uniqueIgnorePatterns, 'ignores');
 }
 
 export default ignores;

package/src/utils/getGitignorePatterns.mjs

@@ -0,0 +1,35 @@
+/* eslint 'no-console': ['warn', { allow: ['warn'] }] */
+
+import { includeIgnoreFile } from '@eslint/compat';
+import fs from 'node:fs';
+import path from 'node:path';
+
+const warningMessage = '\u001B[38;2;229;229;14mWarning:\u001B[0m';
+const fallbackMessage =
+  'Falling back to default ignore patterns. You can suppress this warning by setting \u001B[48;5;234m`gitignore: false`\u001B[0m in the config.';
+
+/**
+ * @param {string} gitignorePath relative path to the .gitignore file
+ * @returns {string[]}
+ */
+export function getGitignorePatterns(gitignorePath) {
+  const gitignoreAbsolutePath = path.resolve(process.cwd(), gitignorePath);
+  const gitignoreExists = fs.existsSync(gitignoreAbsolutePath); // eslint-disable-line n/no-sync
+  const gitignorePatterns = [];
+
+  if (!gitignoreExists) {
+    console.warn(warningMessage, `No .gitignore file found at "${gitignoreAbsolutePath}".`, fallbackMessage);
+
+    return gitignorePatterns;
+  }
+
+  const ignorePatterns = includeIgnoreFile(gitignoreAbsolutePath).ignores ?? [];
+
+  if (ignorePatterns.length > 0) {
+    gitignorePatterns.push(...ignorePatterns);
+  } else {
+    console.warn(warningMessage, `No patterns found in .gitignore file at "${gitignoreAbsolutePath}".`, fallbackMessage);
+  }
+
+  return gitignorePatterns;
+}