feat: conditional requireFlag on require-unicode-regexp

CHANGES.md

@@ -1,5 +1,10 @@
 # eslint-config-ash-nazg CHANGES
 
+## 36.14.0
+
+- feat: conditional `requireFlag` on `require-unicode-regexp`
+- feat: update deps.
+
 ## 36.13.0
 
 - feat: switch to renamed markdown plugin

detectEnv.js

@@ -1,4 +1,7 @@
 // Todo: Move to own repo
+/* eslint-disable n/no-sync -- Not sure possible to use here */
+
+import browserslist from 'browserslist';
 
 /**
  * You can use this in your `eslint.config.js` as follows...
@@ -31,7 +34,6 @@
  *   3|5|6|7|8|9|10|11|12|13|14|15} EcmaVersion
  */
 
-/* eslint-disable n/no-sync -- Not sure possible to use here */
 import {readFileSync} from 'node:fs';
 import {join} from 'node:path';
 import semver from 'semver';
@@ -110,6 +112,39 @@ const detectNodeVersion = (packagePath) => {
   return nodeVersion;
 };
 
+/**
+ * @param {{
+ *   engines?: {node?: string}
+ * }} pkg
+ * @param {string} range
+ */
+export const pkgSatisfiesNodeVersion = (pkg, range) => {
+  const {engines: {node} = {}} = pkg;
+  return node
+    ? semver.satisfies(node, range)
+    : (() => {
+      throw new Error('Could not detect Node version from package.json');
+    });
+};
+
+/**
+ * @param {{
+ *   browserslist?: string[]|string
+ * }} pkg
+ * @param {string} range
+ */
+export const pkgSatisfiesBrowserVersion = (pkg, range) => {
+  if (!pkg?.browserslist) {
+    throw new Error('No `browserslist` found in `package.json`');
+  }
+  const desiredBrowsers = browserslist(pkg.browserslist);
+  const browsers = browserslist(range);
+  const badBrowsers = desiredBrowsers.filter(
+    (browser) => browsers.includes(browser)
+  ).join(', ');
+  return !badBrowsers;
+};
+
 /**
  * @param {string} cwd
  */

index.js

@@ -108,7 +108,7 @@ export default function index (types, config) {
         languageOptions
       });
     } else {
-      configs.push(...sauron(pkg));
+      configs.push(...sauron(pkg, types));
     }
   // basic config ("saruman") is the default
   } else if (types.includes('node')) {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-ash-nazg",
-  "version": "36.13.0",
+  "version": "36.14.0",
   "description": "An expansion and tweaking of the \"standard\" config style for ESLint",
   "main": "index.js",
   "type": "module",
@@ -75,6 +75,7 @@
     "@eslint/markdown": "^6.1.0",
     "@fintechstudios/eslint-plugin-chai-as-promised": "^3.1.0",
     "@stylistic/eslint-plugin": "^2.7.2",
+    "browserslist": "^4.23.3",
     "es-file-traverse": "^1.2.0",
     "eslint-plugin-array-func": "^5.0.2",
     "eslint-plugin-chai-expect": "^3.1.0",

sauron-node.js

@@ -9,7 +9,7 @@ import sauron from './sauron.js';
 export default function sauronNode (pkg) {
   return [
     ...node(pkg),
-    ...sauron(pkg),
+    ...sauron(pkg, ['node']),
     {
       name: 'ash-nazg/sauron-node',
       rules: {

sauron-overrides.js

@@ -12,5 +12,5 @@ import mocha from './mocha.js';
   * }
   */
 export default function sauronOverrides (types, pkg) {
-  return [...sauron(pkg), ...overrides(types, pkg), ...mocha];
+  return [...sauron(pkg, []), ...overrides(types, pkg), ...mocha];
 }

sauron-script.js

@@ -7,5 +7,5 @@ import script from './script.js';
 * }) => import('eslint').Linter.Config[]}
  */
 export default function sauronScript (pkg) {
-  return [...sauron(pkg), ...script];
+  return [...sauron(pkg, []), ...script];
 }

sauron.js

@@ -1,11 +1,16 @@
 import main from './main.js';
+import {
+  pkgSatisfiesNodeVersion, pkgSatisfiesBrowserVersion
+} from './detectEnv.js';
 
 /**
  * @type {(pkg: {
-*   type?: "module"|"commonjs"
-* }) => import('eslint').Linter.Config[]}
-*/
-export default function sauron (pkg) {
+ *   type?: "module"|"commonjs",
+ *   engines?: {node?: string},
+ *   browserslist?: string[]|string
+ * }, types: string[]) => import('eslint').Linter.Config[]}
+ */
+export default function sauron (pkg, types) {
   return [
     ...main(pkg),
     {
@@ -102,7 +107,23 @@ export default function sauron (pkg) {
         }],
         'prefer-named-capture-group': ['warn'],
         'prefer-numeric-literals': ['warn'],
-        'require-unicode-regexp': ['warn'],
+        'require-unicode-regexp': [
+          'warn',
+          (
+            (types.includes('node') &&
+              pkgSatisfiesNodeVersion(pkg, '>=20.0.0')) ||
+            (types.includes('browser') &&
+              pkgSatisfiesBrowserVersion(
+                pkg,
+                // Keep in sync with eslint-plugin-escompat
+                'edge > 0, safari < 17, firefox < 116, chrome < 112'
+              ))
+          )
+            ? {
+              requireFlag: 'v'
+            }
+            : {}
+        ],
         'vars-on-top': ['warn'],
 
         'no-implicit-globals': ['error', {lexicalBindings: true}],