eslint-config-beslogic 2.4.8 → 2.4.10

package/README.md

@@ -84,6 +84,7 @@ In the `additional_dependencies` field, you'll need to include _all_ the plugins
 As you'll easily bust the 250MiB limit on the free tier, omit the following dependencies as they're special-cased to always be ignored on `pre-commit.ci` anyway:
 
 - `dprint`
+- `@eslint-community/eslint-plugin-eslint-comments`
 - `eslint-plugin-total-functions`
 - `esling-plugin-no-autofix`
 - `eslint-plugin-testing-library`

package/javascript.js

@@ -2,7 +2,13 @@
 // Copyright 2024 Beslogic Inc.
 
 const restrictedGlobals = require("confusing-browser-globals")
-const { jestNoRestrictedSyntax, jestFilePatterns, resolveModuleLocation, idLenghtConfig } = require(
+const {
+  jestNoRestrictedSyntax,
+  jestFilePatterns,
+  resolveModuleLocation,
+  idLenghtConfig,
+  isPreCommitCI
+} = require(
   "./lib/utils"
 )
 
@@ -13,6 +19,10 @@ const hasEtc = !!resolveModuleLocation("eslint-plugin-etc")
 // https://github.com/aladdin-add/eslint-plugin/issues/98
 // Even if it gets fixed, `no-autofix` rules necessarily don't provide fixes anyway
 const hasNoAutofix = !!resolveModuleLocation("eslint-plugin-no-autofix")
+// @eslint-community/eslint-comments' only autofix is removing unused suppression comments
+// Some rules get globally disabled for pre-commit.ci, so finding unused comments is expected
+const hasEslintComments = !!resolveModuleLocation("@eslint-community/eslint-comments")
+console.log({ hasEtc, hasNoAutofix, hasEslintComments })
 
 const commaDangleDefault = "always-multiline"
 
@@ -55,7 +65,9 @@ module.exports = {
     "plugin:unicorn/all",
     "plugin:regexp/all",
     "plugin:import/recommended",
-    "plugin:@eslint-community/eslint-comments/recommended"
+    ...hasEslintComments
+      ? ["plugin:@eslint-community/eslint-comments/recommended"]
+      : []
   ],
   "parserOptions": {
     "requireConfigFile": false,
@@ -342,12 +354,14 @@ module.exports = {
          *  @eslint-community/eslint-comments/recommended overrides (https://eslint-community.github.io/eslint-plugin-eslint-comments/rules)
          */
         // TODO: Move back to "all files" once HTML parsing crash is fixed https://github.com/eslint-community/eslint-plugin-eslint-comments/issues/162
-        "@eslint-community/eslint-comments/disable-enable-pair": [
-          "error",
-          {
-            "allowWholeFile": true
-          }
-        ]
+        "@eslint-community/eslint-comments/disable-enable-pair": hasEslintComments
+          ? [
+            "error",
+            {
+              "allowWholeFile": true
+            }
+          ]
+          : "off"
       }
     },
     {
@@ -389,16 +403,20 @@ module.exports = {
     }
   ],
   "rules": {
-    /*
-     *  @eslint-community/eslint-comments/recommended overrides (https://eslint-community.github.io/eslint-plugin-eslint-comments/rules)
-     */
-    "@eslint-community/eslint-comments/no-unused-disable": "error",
-    "@eslint-community/eslint-comments/require-description": [
-      "error",
-      {
-        "ignore": ["eslint-enable"]
+    ...hasEslintComments
+      ? {
+        /*
+         *  @eslint-community/eslint-comments/recommended overrides (https://eslint-community.github.io/eslint-plugin-eslint-comments/rules)
+         */
+        "@eslint-community/eslint-comments/no-unused-disable": "error",
+        "@eslint-community/eslint-comments/require-description": [
+          "error",
+          {
+            "ignore": ["eslint-enable"]
+          }
+        ]
       }
-    ],
+      : {},
 
     /*
      *  sonarjs/recommended overrides (https://github.com/SonarSource/eslint-plugin-sonarjs#rules)

package/lib/utils.js

@@ -1,6 +1,6 @@
-// TEST const isPreCommitCI = __dirname.startsWith("/pc/clone/")
-// console.log(__dirname)
-// console.log(__filename)
+const isPreCommitCI = __dirname.startsWith("/pc/clone/")
+console.log(__dirname)
+console.log(__filename)
 
 const getModuleVersion = (
   /** @type {string} */ nodeModule,
@@ -24,10 +24,8 @@ const getModuleVersion = (
 const resolveModuleLocation = (/** @type {string} */ nodeModule) => {
   try {
     const resolved = require.resolve(nodeModule)
-    // DEBUG: console.log(resolved)
-    console.log(resolved)
-    console.log(resolved.startsWith("/pc/clone/"))
-    if (resolved.startsWith("/pc/clone/")) {
+    // equivalent to resolved.startsWith("/pc/clone/"))
+    if (isPreCommitCI) {
       // Assume it's omitted from the pre-commit.ci run
       return false
     }
@@ -87,6 +85,7 @@ module.exports = {
   getModuleVersion,
   idLenghtConfig,
   jestFilePatterns,
+  isPreCommitCI,
   jestNoRestrictedSyntax,
   resolveModuleLocation
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-beslogic",
-  "version": "2.4.8",
+  "version": "2.4.10",
   "description": "ESLint rules, plugins and configs used at Beslogic",
   "main": "index.js",
   "// dependencies": "Run for ourselves when we install dependencies w/o running npm i afterward",