@the-stranger/eslint-plugin 2.1.0 → 2.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@the-stranger/eslint-plugin",
-  "version": "2.1.0",
+  "version": "2.1.2",
   "repository": {
     "type": "git",
     "url": "https://github.com/the-chris-strange/the-stranger.git",
@@ -22,31 +22,28 @@
   "module": "./src/index.js",
   "types": "./src/types.d.ts",
   "dependencies": {
-    "@the-stranger/eslint-config": "^1.0.0",
-    "@the-stranger/eslint-utils": "^1.0.0",
-    "tslib": "^2.8.1"
+    "@the-stranger/eslint-config": "^1.2.0",
+    "@the-stranger/eslint-utils": "^1.1.0",
+    "tslib": "^2.8.1",
+    "typescript-eslint": "^8.58.2"
   },
   "devDependencies": {
-    "@nx/eslint": "22.5.2",
-    "@nx/eslint-plugin": "22.5.2",
-    "@nx/vitest": "22.5.2",
-    "@types/eslint-plugin-jsx-a11y": "^6.10.1",
     "@types/node": "^24.12.2",
-    "@typescript-eslint/parser": "^8.58.2",
     "@vitest/coverage-v8": "4.1.4",
     "eslint": "^9.39.4",
-    "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-eslint-plugin": "^7.3.2",
-    "nx": "22.5.2",
-    "prettier": "^3.8.3",
-    "prettier-plugin-packagejson": "^3.0.2",
+    "jsonc-eslint-parser": "^3.1.0",
     "typescript": "^5.9.3",
-    "vite": "7.3.2",
     "vitest": "4.1.4"
   },
   "peerDependencies": {
-    "eslint": "^9",
-    "typescript-eslint": "^8"
+    "@nx/eslint-plugin": ">=21 <23",
+    "eslint": "^9"
+  },
+  "peerDependenciesMeta": {
+    "@nx/eslint-plugin": {
+      "optional": true
+    }
   },
   "engines": {
     "node": "^24"

package/src/lib/configs.d.ts

@@ -1,7 +1,7 @@
 export declare const configs: {
     readonly disableTypeChecked: {
         name: string;
-        rules: {};
+        rules: object | undefined;
     }[];
     readonly json: import("eslint/config").Config[];
     readonly recommended: import("eslint/config").Config[];

package/src/lib/configs.js

@@ -1,16 +1,27 @@
-import { configure } from '@the-stranger/eslint-config';
+import { configure, } from '@the-stranger/eslint-config';
 import { disableTypeCheckedConfig } from './disable-type-checked.js';
 import { disableExcept } from './utils/options.js';
 const recommendedSourceOptions = {
     agentSkills: true,
-    js: true,
+    js: {
+        browser: true,
+        node: true,
+    },
     jsdoc: true,
     node: true,
     promise: true,
-    react: false,
+    react: {
+        astro: false,
+        typeChecked: false,
+        typescript: false,
+    },
     regexp: true,
     sort: true,
-    ts: { strict: false, typeChecked: true },
+    ts: {
+        strict: false,
+        typeChecked: true,
+        typescript: true,
+    },
     unicorn: true,
 };
 const recommendedReactOptions = {

package/src/lib/disable-type-checked.d.ts

@@ -1,4 +1,4 @@
 export declare const disableTypeCheckedConfig: {
     name: string;
-    rules: {};
+    rules: object | undefined;
 }[];

package/src/lib/disable-type-checked.js

@@ -1,8 +1,8 @@
+import { namer } from '@the-stranger/eslint-utils';
 import tseslint from 'typescript-eslint';
-import { namer } from './namer.js';
 export const disableTypeCheckedConfig = [
     {
         name: namer('disable-type-checked'),
-        rules: { ...tseslint.configs.disableTypeChecked.rules },
+        rules: tseslint.configs.disableTypeChecked.rules,
     },
 ];

package/src/lib/utils/optional-import-resolver.js

@@ -30,7 +30,7 @@ export class ImportResolver {
         try {
             const pending = this.loadModule(specifier);
             this.cache.set(specifier, pending);
-            return pending;
+            return await pending;
         }
         catch (error) {
             this.cache.delete(specifier);
@@ -72,7 +72,7 @@ export class ImportResolver {
      * @returns the target, or undefined if the message doesn't match the expected pattern
      */
     extractCannotFindModuleTarget(message) {
-        const match = message.match(/Cannot find module ['"]([^'"]+)['"]/i);
+        const match = /Cannot find module ['"]([^'"]+)['"]/i.exec(message);
         return match?.[1];
     }
     /**
@@ -83,7 +83,7 @@ export class ImportResolver {
      * @returns the target, or undefined if the message doesn't match the expected pattern
      */
     extractQuotedMissingTarget(message) {
-        const match = message.match(/Cannot find (?:package|module) ['"]([^'"]+)['"]/i);
+        const match = /Cannot find (?:package|module) ['"]([^'"]+)['"]/i.exec(message);
         return match?.[1];
     }
     /**

package/src/utils.d.ts

@@ -1,2 +1,2 @@
-export { configure, type ConfigOptions } from '@the-stranger/eslint-config';
+export { configure, type ConfigOptions, type Options, } from '@the-stranger/eslint-config';
 export { FilePatterns, getFilePatterns, setSeverity } from '@the-stranger/eslint-utils';

package/src/utils.js

@@ -1,2 +1,2 @@
-export { configure } from '@the-stranger/eslint-config';
+export { configure, } from '@the-stranger/eslint-config';
 export { FilePatterns, getFilePatterns, setSeverity } from '@the-stranger/eslint-utils';

package/src/lib/namer.d.ts

@@ -1,26 +0,0 @@
-import type { Linter } from 'eslint';
-/**
- * Prepend the workspace's namespace or full name to a string. Useful for naming eslint configuration objects.
- * @internal
- * @param value the value to add to the name
- * @returns prefixed value
- */
-export declare function namer(value?: string): string;
-/**
- * Set the `name` property of an ESLint config.
- * @internal
- * @param config the object
- * @param defaultName override the default behavior of this function
- * @returns the named config(s)
- */
-export declare function objectNamer(config: Config, defaultName?: string): Named<Config>;
-/**
- * Make the name property of an object required. Preserves JSDoc comments for objects that already have a name property.
- * @internal
- */
-export type Named<T extends object> = Required<T extends N ? Pick<T, 'name'> : N> & T;
-type Config = Linter.Config;
-type N = {
-    name?: string;
-};
-export {};

package/src/lib/namer.js

@@ -1,38 +0,0 @@
-import pkg from '../../package.json' with { type: 'json' };
-/**
- * Prepend the workspace's namespace or full name to a string. Useful for naming eslint configuration objects.
- * @internal
- * @param value the value to add to the name
- * @returns prefixed value
- */
-export function namer(value) {
-    const ws = /@[a-z-]+\/.+/i.exec(pkg.name)?.[0] ?? pkg.name;
-    if (value) {
-        return value.startsWith(ws) ? value : `${ws}/${value}`;
-    }
-    return ws;
-}
-/**
- * Set the `name` property of an ESLint config.
- * @internal
- * @param config the object
- * @param defaultName override the default behavior of this function
- * @returns the named config(s)
- */
-export function objectNamer(config, defaultName) {
-    if (defaultName) {
-        return { ...config, name: defaultName };
-    }
-    else if (config.name) {
-        return config;
-    }
-    else if (config.plugins) {
-        const plugins = Object.keys(config.plugins);
-        return { ...config, name: namer(plugins[0]) };
-    }
-    else {
-        const rules = Object.keys(config.rules ?? {}).map(e => e.split('/')[0]);
-        const name = [...new Set(rules)].join('/');
-        return { ...config, name };
-    }
-}