lint-suite 1.0.10 → 1.2.0

package/README.md

@@ -5,7 +5,7 @@ A comprehensive collection of ESLint Flat configurations for modern web applicat
 ## Features
 
 - **TypeScript Linting**: Strict typing rules, consistent imports, and code organization
-- **Angular Support**: Component best practices and template accessibility rules
+- **Angular Support**: Component best practices (including Signals), template accessibility rules, and modern control flow
 - **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
@@ -51,6 +51,7 @@ You can selectively include configurations you need.
 - `jest`: Testing configurations
 - `json`: JSON file linting
 - `storybook`: Storybook support
+- `boundaries`: Enforce module boundaries (feature, data-access, ui, etc.)
 
 ## Customization
 

package/index.cjs

@@ -33,23 +33,29 @@ __export(index_exports, {
   angular: () => angular,
   angularTemplate: () => angularTemplate,
   base: () => base,
+  boundaries: () => boundaries,
   javascript: () => javascript,
   jest: () => jest,
   json: () => json,
+  playwright: () => playwright,
   recommended: () => recommended,
   rxjs: () => rxjs,
   storybook: () => storybook,
-  typescript: () => typescript
+  typescript: () => typescript,
+  vitest: () => vitest
 });
 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_config = require("eslint/config");
+var angularTemplate = (0, import_config.defineConfig)([
   {
     files: ["**/*.html"],
-    extends: [...import_angular_eslint.configs.templateAccessibility, ...import_angular_eslint.configs.templateRecommended],
+    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",
@@ -67,6 +73,14 @@ var angularTemplate = (0, import_typescript_eslint.config)([
       "@angular-eslint/template/prefer-control-flow": "error",
       "@angular-eslint/template/prefer-self-closing-tags": "error",
       "@angular-eslint/template/prefer-ngsrc": "error",
+      "@angular-eslint/template/prefer-empty-for": "error",
+      "@angular-eslint/template/prefer-built-in-pipes": "error",
+      "@angular-eslint/template/no-call-expression": "error",
+      "@angular-eslint/template/no-interpolation-in-attributes": "error",
+      "@angular-eslint/template/no-empty-control-flow-blocks": "error",
+      "@angular-eslint/template/no-nested-tags": "error",
+      "@angular-eslint/template/use-track-by-function": "error",
+      "@angular-eslint/template/label-has-associated-control": "error",
       "@angular-eslint/template/attributes-order": [
         "error",
         {
@@ -94,8 +108,8 @@ 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_config2 = require("eslint/config");
+var angular = (0, import_config2.defineConfig)([
   {
     files: ["**/*.ts"],
     extends: [...import_angular_eslint2.configs.tsRecommended],
@@ -107,11 +121,13 @@ var angular = (0, import_typescript_eslint2.config)([
       }
     },
     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/use-injectable-provided-in": "error",
+      "@angular-eslint/prefer-signal-model": "error",
+      "@angular-eslint/no-uncalled-signals": "error",
+      "@angular-eslint/prefer-host-metadata-property": "error"
     }
   }
 ]);
@@ -120,8 +136,8 @@ var angular = (0, import_typescript_eslint2.config)([
 var import_eslint_plugin = require("@nx/eslint-plugin");
 var import_eslint_plugin2 = __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)([
+var import_config3 = require("eslint/config");
+var base = (0, import_config3.defineConfig)([
   { extends: [...import_eslint_plugin.configs["flat/base"]] },
   {
     files: [
@@ -187,7 +203,10 @@ var base = (0, import_typescript_eslint3.config)([
       // Warn if function complexity is high
       "max-depth": ["warn", 4],
       // Warn on deeply nested blocks
-      "max-lines-per-function": ["warn", 50],
+      "max-lines-per-function": [
+        "warn",
+        { max: 50, skipBlankLines: true, skipComments: true }
+      ],
       // Warn on long functions
       "max-params": ["warn", 4],
       // Warn on functions with many parameters
@@ -304,10 +323,17 @@ var base = (0, import_typescript_eslint3.config)([
     }
   },
   {
-    files: ["**/*.spec.ts", "**/*.test.ts", "**/*.spec.js", "**/*.test.js"],
+    files: [
+      "**/*.spec.ts",
+      "**/*.test.ts",
+      "**/*.spec.js",
+      "**/*.test.js",
+      "**/*.e2e.ts",
+      "**/*.e2e.js"
+    ],
     rules: {
       "max-lines-per-function": "off",
-      complexity: ["warn", { max: 1 }],
+      complexity: ["warn", { max: 3 }],
       "max-lines": [
         "error",
         {
@@ -334,19 +360,18 @@ var base = (0, import_typescript_eslint3.config)([
 
 // 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)({
+var import_config4 = require("eslint/config");
+var javascript = (0, import_config4.defineConfig)({
   files: ["**/*.js", "**/*.jsx", "**/*.cjs", "**/*.mjs"],
-  extends: [...import_eslint_plugin3.configs["flat/javascript"]],
-  rules: {}
+  extends: [...import_eslint_plugin3.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 import_config5 = require("eslint/config");
+var jest = (0, import_config5.defineConfig)([
   {
-    files: ["**/*.spec.ts", "**/*.test.ts", "**/*.spec.js", "**/*.test.js"],
+    files: ["**/*.spec.ts", "**/*.spec.js"],
     extends: [import_eslint_plugin_jest.default.configs["flat/all"]],
     rules: {
       "jest/no-done-callback": "off"
@@ -356,19 +381,18 @@ var jest = (0, import_typescript_eslint5.config)([
 
 // 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 import_config6 = require("eslint/config");
+var json = (0, import_config6.defineConfig)([
   {
     files: ["**/*.json"],
-    extends: [import_eslint_plugin_json.default.configs.recommended],
-    rules: {}
+    extends: [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 import_config7 = require("eslint/config");
+var rxjs = (0, import_config7.defineConfig)([
   {
     files: [
       "**/*.js",
@@ -437,13 +461,12 @@ var rxjs = (0, import_typescript_eslint7.config)([
 
 // 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 import_config8 = require("eslint/config");
+var storybook = (0, import_config8.defineConfig)([
   {
     ignores: ["!.storybook"],
     files: ["**/*.stories.ts"],
-    extends: [import_eslint_plugin_storybook.default.configs["flat/recommended"]],
-    rules: {}
+    extends: [import_eslint_plugin_storybook.default.configs["flat/recommended"]]
   }
 ]);
 
@@ -451,6 +474,7 @@ var storybook = (0, import_typescript_eslint8.config)([
 var import_eslint_plugin4 = require("@nx/eslint-plugin");
 var import_eslint_import_resolver_typescript = require("eslint-import-resolver-typescript");
 var import_eslint_plugin_import_x2 = __toESM(require("eslint-plugin-import-x"), 1);
+var import_config9 = require("eslint/config");
 
 // packages/lint-suite/src/lib/rules/explicit-accessibility.ts
 var import_utils = require("@typescript-eslint/utils");
@@ -487,9 +511,6 @@ var getNodeName = (node) => {
       }
       return "<destructured parameter>";
     }
-    if (param.type === import_utils.TSESTree.AST_NODE_TYPES.RestElement && param.argument.type === import_utils.TSESTree.AST_NODE_TYPES.Identifier) {
-      return `...${param.argument.name}`;
-    }
     return "<complex parameter>";
   }
   return "<unknown>";
@@ -589,13 +610,12 @@ 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 = (0, import_config9.defineConfig)([
   // NOTE: Checks for 'prettier' and 'eslint-plugin-prettier'
   {
     files: ["**/*.ts", "**/*.tsx", "**/*.cts", "**/*.mts"],
@@ -724,6 +744,21 @@ var typescript = (0, import_typescript_eslint9.config)([
               group: "builtin",
               position: "before"
             },
+            {
+              pattern: "@shared/**",
+              group: "internal",
+              position: "before"
+            },
+            {
+              pattern: "@backend/**",
+              group: "internal",
+              position: "before"
+            },
+            {
+              pattern: "@frontend/**",
+              group: "internal",
+              position: "before"
+            },
             {
               pattern: "@*/shared/**",
               group: "internal",
@@ -740,7 +775,7 @@ var typescript = (0, import_typescript_eslint9.config)([
               position: "after"
             }
           ],
-          pathGroupsExcludedImportTypes: ["@angular/**", "@nestjs/**"],
+          pathGroupsExcludedImportTypes: ["builtin"],
           "newlines-between": "always",
           alphabetize: { order: "asc", caseInsensitive: true }
         }
@@ -780,7 +815,14 @@ var typescript = (0, import_typescript_eslint9.config)([
     }
   },
   {
-    files: ["**/*.spec.ts", "**/*.test.ts", "**/*.spec.js", "**/*.test.js"],
+    files: [
+      "**/*.spec.ts",
+      "**/*.test.ts",
+      "**/*.spec.js",
+      "**/*.test.js",
+      "**/*.e2e.ts",
+      "**/*.e2e.js"
+    ],
     rules: {
       "@typescript-eslint/no-empty-function": "off",
       "@typescript-eslint/no-unbound-method": "off",
@@ -796,6 +838,130 @@ var typescript = (0, import_typescript_eslint9.config)([
   }
 ]);
 
+// packages/lint-suite/src/lib/vitest.ts
+var import_eslint_plugin5 = __toESM(require("@vitest/eslint-plugin"), 1);
+var import_config10 = require("eslint/config");
+var vitestPlugin = {
+  vitest: import_eslint_plugin5.default
+};
+var vitest = (0, import_config10.defineConfig)([
+  {
+    files: ["**/*.spec.ts", "**/*.spec.js", "**/*.test.ts", "**/*.test.js"],
+    plugins: vitestPlugin,
+    rules: {
+      ...import_eslint_plugin5.default.configs.recommended.rules,
+      "vitest/max-nested-describe": ["error", { max: 3 }]
+    }
+  }
+]);
+
+// packages/lint-suite/src/lib/playwright.ts
+var import_eslint_plugin_playwright = __toESM(require("eslint-plugin-playwright"), 1);
+var import_config11 = require("eslint/config");
+var playwright = (0, import_config11.defineConfig)([
+  {
+    ...import_eslint_plugin_playwright.default.configs["flat/recommended"],
+    files: ["**/*.e2e.ts", "**/*.e2e.js"],
+    rules: {
+      ...import_eslint_plugin_playwright.default.configs["flat/recommended"].rules
+    }
+  }
+]);
+
+// packages/lint-suite/src/lib/boundaries.ts
+var import_eslint_plugin_boundaries = __toESM(require("eslint-plugin-boundaries"), 1);
+var import_config12 = require("eslint/config");
+var boundaries = (0, import_config12.defineConfig)([
+  {
+    files: [
+      "**/*.js",
+      "**/*.jsx",
+      "**/*.ts",
+      "**/*.tsx",
+      "**/*.cts",
+      "**/*.mts"
+    ],
+    plugins: {
+      boundaries: import_eslint_plugin_boundaries.default
+    },
+    settings: {
+      "boundaries/elements": [
+        {
+          type: "feature",
+          pattern: "**/feature/**"
+        },
+        {
+          type: "data-access",
+          pattern: "**/data-access/**"
+        },
+        {
+          type: "domain-logic",
+          pattern: "**/domain-logic/**"
+        },
+        {
+          type: "ui",
+          pattern: "**/ui/**"
+        },
+        {
+          type: "common",
+          pattern: "**/common/**"
+        },
+        {
+          type: "utils",
+          pattern: "**/utils/**"
+        }
+      ],
+      "import/resolver": {
+        typescript: {
+          alwaysTryTypes: true
+        }
+      }
+    },
+    rules: {
+      ...import_eslint_plugin_boundaries.default.configs.recommended.rules,
+      "boundaries/element-types": [
+        "error",
+        {
+          default: "disallow",
+          message: "${dependency.type} is not allowed to be imported in ${file.type}",
+          rules: [
+            {
+              from: "feature",
+              allow: ["feature", "ui", "domain-logic", "utils", "common"]
+            },
+            {
+              from: "domain-logic",
+              allow: ["domain-logic", "data-access", "utils", "common"]
+            },
+            {
+              from: "data-access",
+              allow: ["data-access", "utils", "common"]
+            },
+            {
+              from: "ui",
+              allow: ["ui", "utils", "common"]
+            },
+            {
+              from: "utils",
+              allow: ["utils", "common"]
+            },
+            {
+              from: "common",
+              allow: ["common"]
+            }
+          ]
+        }
+      ]
+    }
+  },
+  {
+    files: ["**/*.stories.ts"],
+    rules: {
+      "boundaries/element-types": "off"
+    }
+  }
+]);
+
 // packages/lint-suite/src/index.ts
 var recommended = [
   ...base,
@@ -806,19 +972,25 @@ var recommended = [
   ...rxjs,
   ...jest,
   ...json,
-  ...storybook
+  ...boundaries,
+  ...storybook,
+  ...vitest,
+  ...playwright
 ];
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
   angular,
   angularTemplate,
   base,
+  boundaries,
   javascript,
   jest,
   json,
+  playwright,
   recommended,
   rxjs,
   storybook,
-  typescript
+  typescript,
+  vitest
 });
 //# sourceMappingURL=index.cjs.map