lint-suite 1.0.7 → 1.0.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lint-suite",
-  "version": "1.0.7",
+  "version": "1.0.8",
   "description": "Reusable ESLint configurations",
   "repository": {
     "type": "git",
@@ -24,19 +24,16 @@
   "module": "./index.js",
   "types": "./index.d.ts",
   "dependencies": {
-    "globals": "^16.1.0",
-    "eslint": "^9.26.0",
-    "@nx/eslint-plugin": "^21.0.3",
-    "eslint-plugin-jest": "^28.11.0",
+    "eslint": "^9.38.0",
+    "@nx/eslint-plugin": "^21.6.5",
+    "eslint-plugin-jest": "29.0.1",
     "eslint-plugin-json": "^4.0.1",
-    "@smarttools/eslint-plugin-rxjs": "^1.0.19",
-    "eslint-plugin-storybook": "^0.12.0",
-    "eslint-plugin-import-x": "^4.11.1",
-    "angular-eslint": "^19.4.0",
-    "typescript-eslint": "^8.32.0",
-    "@stylistic/eslint-plugin": "^4.2.0",
-    "eslint-import-resolver-typescript": "^4.3.4",
-    "@typescript-eslint/utils": "8.32.0"
+    "@smarttools/eslint-plugin-rxjs": "1.0.22",
+    "eslint-plugin-storybook": "9.1.13",
+    "eslint-plugin-import-x": "4.16.1",
+    "@stylistic/eslint-plugin": "5.5.0",
+    "eslint-import-resolver-typescript": "4.4.4",
+    "@typescript-eslint/utils": "8.46.2"
   },
   "scripts": {
     "build": "nx build lint-suite",

package/{README.md → packages/lint-suite/src/README.md}

@@ -1,135 +1,121 @@
-# Lint Suite
-
-A comprehensive collection of ESLint Flat configurations for modern web applications.
-
-## Features
-
-- **TypeScript Linting**: Strict typing rules, consistent imports, and code organization
-- **Angular Support**: Component best practices and template accessibility rules
-- **RxJS Guidelines**: Observable patterns, Finnish notation, and operator safety
-- **Code Style**: Formatting rules, line limits, and structural consistency
-- **Accessibility**: ARIA validation, keyboard events, and semantic HTML
-- **Testing**: Jest configurations with appropriate relaxations for test files
-- **Additional Support**: JSON, Storybook, and more
-
-## Installation
-
-```bash
-npm install --save-dev lint-suite
-```
-
-## Dependencies
-
-```bash
-npm install --save-dev eslint typescript-eslint eslint-config-prettier
-```
-
-## Usage
-
-Create a `.eslintrc.config.mjs` file in your project root:
-
-```js
-import { base, javascript, typescript } from 'lint-suite';
-
-export default [
-  ...base,
-  ...javascript,
-  ...typescript
-  // Rest of the required configurations
-];
-```
-
-You can selectively include configurations you need.
-
-## Available Configurations
-
-- `base`: Core JavaScript rules and formatting
-- `typescript`: TypeScript-specific rules
-- `angular`: Angular component best practices
-- `angularTemplate`: HTML template rules with accessibility focus
-- `rxjs`: Observable patterns and operator safety
-- `jest`: Testing configurations
-- `json`: JSON file linting
-- `storybook`: Storybook support
-
-## Customization
-
-You can override any rules by adding a `rules` section to your ESLint config:
-
-```js
-import { typescript } from 'lint-suite';
-
-export default [
-  ...typescript,
-  {
-    rules: {
-      '@typescript-eslint/explicit-function-return-type': 'off'
-    }
-  }
-];
-```
-
-## Available Rules (you can add more as you prefer)
-
-### nx/eslint-plugin
-
-- `@nx/enforce-module-boundaries`: Enforces module boundary restrictions
-- `@nx/dependency-checks`: Validates dependencies in workspace projects
-- ...
-
-### eslint-plugin-jest
-
-- `jest/no-disabled-tests`: Disallows disabled tests
-- `jest/expect-expect`: Ensures test cases contain at least one expect statement
-- `jest/valid-expect`: Validates expect() usage
-- ...
-
-### eslint-plugin-json
-
-- `json/json`: Validates JSON syntax
-- `json/sort-package-json`: Enforces consistent ordering in package.json
-- ...
-
-### @smarttools/eslint-plugin-rxjs
-
-- `@rxjs/finnish`: Enforces Finnish notation for observables
-- `@rxjs/no-ignored-subscription`: Prevents ignoring returned subscriptions
-- `@rxjs/no-unsafe-takeuntil`: Ensures proper usage of takeUntil operator
-- ...
-
-### eslint-plugin-storybook
-
-- `storybook/no-redundant-story-name`: Prevents redundant story names
-- `storybook/csf-component`: Enforces Component Story Format standards
-- `storybook/hierarchy-separator`: Enforces story hierarchy separators
-- ...
-
-### eslint-plugin-import
-
-- `import/no-unresolved`: Ensures imports point to valid modules
-- `import/order`: Enforces a consistent order of import statements
-- `import/no-duplicates`: Prevents duplicate imports
-- ...
-
-### @stylistic/eslint-plugin
-
-- `@stylistic/max-len`: Enforces maximum line length
-- `@stylistic/indent`: Enforces consistent indentation
-- `@stylistic/quotes`: Enforces consistent quote style
-- ...
-
-## Contributing
-
-See [CONTRIBUTING.md](./CONTRIBUTING.md) for contribution guidelines.
-
-## License
-
-MIT
-
-```
-
-```
-
-```
-
-```
+# Lint Suite
+
+A comprehensive collection of ESLint Flat configurations for modern web applications.
+
+## Features
+
+- **TypeScript Linting**: Strict typing rules, consistent imports, and code organization
+- **Angular Support**: Component best practices and template accessibility rules
+- **RxJS Guidelines**: Observable patterns, Finnish notation, and operator safety
+- **Code Style**: Formatting rules, line limits, and structural consistency
+- **Accessibility**: ARIA validation, keyboard events, and semantic HTML
+- **Testing**: Jest configurations with appropriate relaxations for test files
+- **Additional Support**: JSON, Storybook, and more
+
+## Installation
+
+```bash
+npm install --save-dev lint-suite
+```
+
+## Usage
+
+Create a `.eslintrc.config.mjs` file in your project root:
+
+```js
+import { base, javascript, typescript } from 'lint-suite';
+
+export default [
+  ...base,
+  ...javascript,
+  ...typescript,
+  // Rest of the required configurations
+];
+```
+
+You can selectively include configurations you need.
+
+## Available Configurations
+
+- `base`: Core JavaScript rules and formatting
+- `typescript`: TypeScript-specific rules
+- `angular`: Angular component best practices
+- `angularTemplate`: HTML template rules with accessibility focus
+- `rxjs`: Observable patterns and operator safety
+- `jest`: Testing configurations
+- `json`: JSON file linting
+- `storybook`: Storybook support
+
+## Customization
+
+You can override any rules by adding a `rules` section to your ESLint config:
+
+```js
+import { typescript } from 'lint-suite';
+
+export default [
+  ...typescript,
+  {
+    rules: {
+      '@typescript-eslint/explicit-function-return-type': 'off',
+    },
+  },
+];
+```
+
+## Available Rules (you can add more as you prefer)
+
+### nx/eslint-plugin
+
+- `@nx/enforce-module-boundaries`: Enforces module boundary restrictions
+- `@nx/dependency-checks`: Validates dependencies in workspace projects
+- ...
+
+### eslint-plugin-jest
+
+- `jest/no-disabled-tests`: Disallows disabled tests
+- `jest/expect-expect`: Ensures test cases contain at least one expect statement
+- `jest/valid-expect`: Validates expect() usage
+- ...
+
+### eslint-plugin-json
+
+- `json/json`: Validates JSON syntax
+- `json/sort-package-json`: Enforces consistent ordering in package.json
+- ...
+
+### @smarttools/eslint-plugin-rxjs
+
+- `@rxjs/finnish`: Enforces Finnish notation for observables
+- `@rxjs/no-ignored-subscription`: Prevents ignoring returned subscriptions
+- `@rxjs/no-unsafe-takeuntil`: Ensures proper usage of takeUntil operator
+- ...
+
+### eslint-plugin-storybook
+
+- `storybook/no-redundant-story-name`: Prevents redundant story names
+- `storybook/csf-component`: Enforces Component Story Format standards
+- `storybook/hierarchy-separator`: Enforces story hierarchy separators
+- ...
+
+### eslint-plugin-import
+
+- `import/no-unresolved`: Ensures imports point to valid modules
+- `import/order`: Enforces a consistent order of import statements
+- `import/no-duplicates`: Prevents duplicate imports
+- ...
+
+### @stylistic/eslint-plugin
+
+- `@stylistic/max-len`: Enforces maximum line length
+- `@stylistic/indent`: Enforces consistent indentation
+- `@stylistic/quotes`: Enforces consistent quote style
+- ...
+
+## Contributing
+
+See [CONTRIBUTING.md](./CONTRIBUTING.md) for contribution guidelines.
+
+## License
+
+MIT

package/{index.cjs → packages/lint-suite/src/index.cjs}

@@ -28,8 +28,8 @@ var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__ge
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
 
 // packages/lint-suite/src/index.ts
-var src_exports = {};
-__export(src_exports, {
+var index_exports = {};
+__export(index_exports, {
   angular: () => angular,
   angularTemplate: () => angularTemplate,
   base: () => base,
@@ -41,17 +41,15 @@ __export(src_exports, {
   storybook: () => storybook,
   typescript: () => typescript
 });
-module.exports = __toCommonJS(src_exports);
+module.exports = __toCommonJS(index_exports);
 
 // packages/lint-suite/src/lib/angular-template.ts
-var import_angular_eslint = require("angular-eslint");
-var import_typescript_eslint = require("typescript-eslint");
-var angularTemplate = (0, import_typescript_eslint.config)([
+var import_eslint_plugin = __toESM(require("@nx/eslint-plugin"), 1);
+var angularTemplate = [
+  ...import_eslint_plugin.default.configs["flat/angular-template"],
   {
     files: ["**/*.html"],
-    extends: [...import_angular_eslint.configs.templateAccessibility, ...import_angular_eslint.configs.templateRecommended],
     rules: {
-      "@angular-eslint/template/prefer-template-literal": "error",
       "@angular-eslint/template/no-any": "error",
       "@angular-eslint/template/no-duplicate-attributes": "error",
       "@angular-eslint/template/button-has-type": "error",
@@ -89,40 +87,26 @@ var angularTemplate = (0, import_typescript_eslint.config)([
       ]
     }
   }
-]);
+];
 
 // packages/lint-suite/src/lib/angular.ts
-var import_globals = __toESM(require("globals"), 1);
-var import_angular_eslint2 = require("angular-eslint");
-var import_typescript_eslint2 = require("typescript-eslint");
-var angular = (0, import_typescript_eslint2.config)([
+var import_eslint_plugin2 = __toESM(require("@nx/eslint-plugin"), 1);
+var angular = [
+  ...import_eslint_plugin2.default.configs["flat/angular"],
   {
     files: ["**/*.ts"],
-    extends: [...import_angular_eslint2.configs.tsRecommended],
-    languageOptions: {
-      globals: {
-        ...import_globals.default.browser,
-        ...import_globals.default.es2025,
-        ...import_globals.default.node
-      }
-    },
-    processor: import_angular_eslint2.processInlineTemplates,
-    plugins: { "@angular-eslint": import_angular_eslint2.tsPlugin },
     rules: {
-      "@angular-eslint/prefer-on-push-component-change-detection": "error",
-      "@angular-eslint/prefer-signals": "error",
-      "@angular-eslint/use-injectable-provided-in": "error"
+      "@angular-eslint/prefer-on-push-component-change-detection": "error"
     }
   }
-]);
+];
 
 // packages/lint-suite/src/lib/base.ts
-var import_eslint_plugin = require("@nx/eslint-plugin");
-var import_eslint_plugin2 = __toESM(require("@stylistic/eslint-plugin"), 1);
+var import_eslint_plugin3 = __toESM(require("@nx/eslint-plugin"), 1);
+var import_eslint_plugin4 = __toESM(require("@stylistic/eslint-plugin"), 1);
 var import_eslint_plugin_import_x = __toESM(require("eslint-plugin-import-x"), 1);
-var import_typescript_eslint3 = require("typescript-eslint");
-var base = (0, import_typescript_eslint3.config)([
-  { extends: [...import_eslint_plugin.configs["flat/base"]] },
+var base = [
+  ...import_eslint_plugin3.default.configs["flat/base"],
   {
     files: [
       "**/*.js",
@@ -134,7 +118,7 @@ var base = (0, import_typescript_eslint3.config)([
       "**/*.cts",
       "**/*.mts"
     ],
-    extends: [import_eslint_plugin_import_x.default.flatConfigs.recommended]
+    ...import_eslint_plugin_import_x.default.flatConfigs.recommended
   },
   {
     files: [
@@ -147,7 +131,7 @@ var base = (0, import_typescript_eslint3.config)([
       "**/*.cts",
       "**/*.mts"
     ],
-    extends: [import_eslint_plugin2.default.configs.recommended]
+    ...import_eslint_plugin4.default.configs.recommended
   },
   {
     files: [
@@ -166,7 +150,7 @@ var base = (0, import_typescript_eslint3.config)([
       "no-console": ["warn", { allow: ["warn", "error"] }],
       "no-debugger": "error",
       eqeqeq: ["error", "always", { null: "ignore" }],
-      curly: ["error", "multi"],
+      curly: ["error", "all"],
       "prefer-const": ["error", { destructuring: "all" }],
       "no-var": "error",
       "object-shorthand": "error",
@@ -197,6 +181,24 @@ var base = (0, import_typescript_eslint3.config)([
       // Enforce consistency for getters/setters
       "class-methods-use-this": "error",
       "no-underscore-dangle": "error",
+      "padding-line-between-statements": [
+        "error",
+        {
+          blankLine: "always",
+          prev: "export",
+          next: ["export", "const"]
+        },
+        {
+          blankLine: "always",
+          prev: "*",
+          next: "return"
+        },
+        {
+          blankLine: "always",
+          prev: "*",
+          next: "if"
+        }
+      ],
       "max-len": [
         "error",
         {
@@ -213,38 +215,6 @@ var base = (0, import_typescript_eslint3.config)([
           }
         }
       ],
-      "@stylistic/padding-line-between-statements": [
-        "error",
-        {
-          blankLine: "always",
-          prev: "export",
-          next: ["export", "const"]
-        },
-        {
-          blankLine: "always",
-          prev: ["const", "let", "var"],
-          next: "*"
-        },
-        {
-          blankLine: "any",
-          prev: ["const", "let", "var"],
-          next: ["const", "let", "var"]
-        },
-        {
-          blankLine: "always",
-          prev: "*",
-          next: [
-            "for",
-            "if",
-            "return",
-            "throw",
-            "switch",
-            "try",
-            "class",
-            "function"
-          ]
-        }
-      ],
       "@stylistic/comma-dangle": [
         "error",
         {
@@ -330,45 +300,36 @@ var base = (0, import_typescript_eslint3.config)([
       "class-methods-use-this": "off"
     }
   }
-]);
+];
 
 // packages/lint-suite/src/lib/javascript.ts
-var import_eslint_plugin3 = require("@nx/eslint-plugin");
-var import_typescript_eslint4 = require("typescript-eslint");
-var javascript = (0, import_typescript_eslint4.config)({
-  files: ["**/*.js", "**/*.jsx", "**/*.cjs", "**/*.mjs"],
-  extends: [...import_eslint_plugin3.configs["flat/javascript"]],
-  rules: {}
-});
+var import_eslint_plugin5 = __toESM(require("@nx/eslint-plugin"), 1);
+var javascript = import_eslint_plugin5.default.configs["flat/javascript"];
 
 // packages/lint-suite/src/lib/jest.ts
 var import_eslint_plugin_jest = __toESM(require("eslint-plugin-jest"), 1);
-var import_typescript_eslint5 = require("typescript-eslint");
-var jest = (0, import_typescript_eslint5.config)([
+var jest = [
   {
     files: ["**/*.spec.ts", "**/*.test.ts", "**/*.spec.js", "**/*.test.js"],
-    extends: [import_eslint_plugin_jest.default.configs["flat/all"]],
+    ...import_eslint_plugin_jest.default.configs["flat/all"],
     rules: {
       "jest/no-done-callback": "off"
     }
   }
-]);
+];
 
 // packages/lint-suite/src/lib/json.ts
 var import_eslint_plugin_json = __toESM(require("eslint-plugin-json"), 1);
-var import_typescript_eslint6 = require("typescript-eslint");
-var json = (0, import_typescript_eslint6.config)([
+var json = [
   {
     files: ["**/*.json"],
-    extends: [import_eslint_plugin_json.default.configs.recommended],
-    rules: {}
+    ...import_eslint_plugin_json.default.configs.recommended
   }
-]);
+];
 
 // packages/lint-suite/src/lib/rxjs.ts
 var import_eslint_plugin_rxjs = __toESM(require("@smarttools/eslint-plugin-rxjs"), 1);
-var import_typescript_eslint7 = require("typescript-eslint");
-var rxjs = (0, import_typescript_eslint7.config)([
+var rxjs = [
   {
     files: [
       "**/*.js",
@@ -385,7 +346,7 @@ var rxjs = (0, import_typescript_eslint7.config)([
         projectService: true
       }
     },
-    extends: [import_eslint_plugin_rxjs.default.configs.recommended]
+    ...import_eslint_plugin_rxjs.default.configs.recommended
   },
   {
     files: [
@@ -433,22 +394,20 @@ var rxjs = (0, import_typescript_eslint7.config)([
       "rxjs/no-ignored-subscription": "off"
     }
   }
-]);
+];
 
 // packages/lint-suite/src/lib/storybook.ts
 var import_eslint_plugin_storybook = __toESM(require("eslint-plugin-storybook"), 1);
-var import_typescript_eslint8 = require("typescript-eslint");
-var storybook = (0, import_typescript_eslint8.config)([
+var storybook = [
+  ...import_eslint_plugin_storybook.default.configs["flat/recommended"],
   {
     ignores: ["!.storybook"],
-    files: ["**/*.stories.ts"],
-    extends: [import_eslint_plugin_storybook.default.configs["flat/recommended"]],
-    rules: {}
+    files: ["**/*.stories.ts"]
   }
-]);
+];
 
 // packages/lint-suite/src/lib/typescript.ts
-var import_eslint_plugin4 = require("@nx/eslint-plugin");
+var import_eslint_plugin6 = __toESM(require("@nx/eslint-plugin"), 1);
 var import_eslint_import_resolver_typescript = require("eslint-import-resolver-typescript");
 var import_eslint_plugin_import_x2 = __toESM(require("eslint-plugin-import-x"), 1);
 
@@ -558,8 +517,7 @@ var explicit_accessibility_default = createRule({
               tokenToInsertBefore = firstToken;
             }
           }
-          if (!tokenToInsertBefore)
-            return null;
+          if (!tokenToInsertBefore) return null;
           const tokenAfterInsertion = sourceCode.getTokenAfter(
             tokenToInsertBefore,
             { includeComments: false }
@@ -590,21 +548,15 @@ var explicit_accessibility_default = createRule({
 });
 
 // packages/lint-suite/src/lib/typescript.ts
-var import_typescript_eslint9 = require("typescript-eslint");
 var localTypescriptRules = {
   rules: {
     "explicit-accessibility": explicit_accessibility_default
   }
 };
-var typescript = (0, import_typescript_eslint9.config)([
+var typescript = [
   // NOTE: Checks for 'prettier' and 'eslint-plugin-prettier'
-  {
-    files: ["**/*.ts", "**/*.tsx", "**/*.cts", "**/*.mts"],
-    extends: [
-      import_eslint_plugin_import_x2.default.flatConfigs.typescript,
-      ...import_eslint_plugin4.configs["flat/typescript"]
-    ]
-  },
+  import_eslint_plugin_import_x2.default.flatConfigs.typescript,
+  ...import_eslint_plugin6.default.configs["flat/typescript"],
   {
     files: ["**/*.ts", "**/*.tsx", "**/*.cts", "**/*.mts"],
     plugins: { local: localTypescriptRules },
@@ -627,8 +579,6 @@ var typescript = (0, import_typescript_eslint9.config)([
       // Consider enabling if maximum explicitness is desired, even if verbose:
       // C. Null/Undefined Safety (Requires strictNullChecks)
       "@typescript-eslint/no-non-null-assertion": "error",
-      "@typescript-eslint/prefer-nullish-coalescing": "error",
-      "@typescript-eslint/prefer-optional-chain": "error",
       // D. Promise Handling (Supplementing plugin:promise/recommended)
       "@typescript-eslint/no-floating-promises": [
         "error",
@@ -795,7 +745,7 @@ var typescript = (0, import_typescript_eslint9.config)([
       "@typescript-eslint/naming-convention": "off"
     }
   }
-]);
+];
 
 // packages/lint-suite/src/index.ts
 var recommended = [