lint-suite 2.0.0 → 2.0.2

package/README.md

@@ -207,7 +207,9 @@ literal class names inside `[class]="..."` expressions and `class="a {{ b }}"`
 interpolations. String literals, object-literal keys, array elements, and both
 branches of a ternary contribute names; identifiers, calls, pipes, and `+`
 concatenations contribute nothing, so a class the rule cannot see is never
-reported. `[ngClass]` is deliberately not analysed.
+reported. `[ngClass]` is deliberately not analysed. `routerLinkActive`,
+`animate.enter`, and `animate.leave` are also read as class lists, static or
+bound.
 
 Stylesheets come from the component beside the template: `styleUrl`,
 `styleUrls`, and inline `styles` read as string or template literals from the
@@ -278,13 +280,17 @@ Selectors resolve through the same parser as the ESLint rule, so `&__element`,
 `&--modifier`, `&.other`, `& > .child`, `.wrapper &`, `@media` blocks, and
 selector lists all report the resolved name on the rule that declares it: in
 `.panel { .inner {} }` only `inner` is checked on the inner rule, never `panel`
-twice. Arguments of `:host(.dark)` and `:host-context(.rtl)` are skipped, and
+twice. A rule that only wraps nested rules, like `.dialog` in
+`.dialog { &__name {} }`, emits no selector of its own and is never reported;
+`.shell` in `.shell { .inner {} }` still is, because `.shell .inner` reaches
+the output. Arguments of `:host(.dark)` and `:host-context(.rtl)` are skipped, and
 everything after `::ng-deep`, `/deep/`, or `>>>` is skipped too, because those
 classes live in other templates. A selector built with interpolation
 (`.icon-#{$size}`) is never reported, and `@extend .base` counts `base` as
 used.
 
-The template side reads the same sources as `no-unstyled-classes` plus
+The template side reads the same sources as `no-unstyled-classes` (including
+`routerLinkActive`, `animate.enter`, and `animate.leave`, static or bound) plus
 `[ngClass]`, and it does not skip custom elements: a class on
 `<app-child class="foo">` is written by this template, so `.foo` counts as
 used. When any template of the stylesheet holds a class source the rule cannot
@@ -457,7 +463,7 @@ per file.
 
 | Rule | Reports | Options |
 |---|---|---|
-| `local/type-placement` | An exported `type` outside a `common/*.type.ts` or `test/common/*.type.ts` file; a value exported from a `*.type.ts` file; a non-`const` export from a `*.const.ts` file; an `import type` from a relative or internal path that is neither a `*.type.ts` file nor a `common` barrel (`../common`, `./common/index.ts`, `@shared/common`). `.spec.ts`, `.stub.ts`, `.d.ts`, `state.type.ts` (any prefix), and fixtures are exempt; a `.spec.util.ts` file is not, so an exported type in one is reported. Never resolves imports. | `internalPatterns`: regex sources for alias prefixes that must resolve to a `*.type.ts` file. Default empty: a workspace alias (`@shared/common`) resolves to a library entry point, and module boundaries forbid deep imports, so alias type imports pass. |
+| `local/type-placement` | An exported `type` outside a `common/*.type.ts` or `test/common/*.type.ts` file; a value exported from a `*.type.ts` file; a non-`const` export from a `*.const.ts` file; an `import type` from a relative or internal path that is neither a `*.type.ts` / `*.schema.ts` file nor a `common` barrel (`../common`, `./common/index.ts`, `@shared/common`). `.spec.ts`, `.stub.ts`, `.schema.ts` (an inferred type lives beside its schema), `.d.ts`, `state.type.ts` (any prefix), and fixtures are exempt; a `.spec.util.ts` file is not, so an exported type in one is reported. Never resolves imports. | `internalPatterns`: regex sources for alias prefixes that must resolve to a `*.type.ts` file. Default empty: a workspace alias (`@shared/common`) resolves to a library entry point, and module boundaries forbid deep imports, so alias type imports pass. |
 | `local/util-purity` | Inside `utils/*.util.ts` (excluding paths under `test/` or `testing/`): imports of `node:fs`, `child_process`, `os`, `process`, `http`, `net`, `worker_threads`; a module-level `let`; a module-level `new Map/Set/WeakMap/WeakSet`; `process.*`, `globalThis`, `window`, `document`, `localStorage`, `console`; `Date.now`, `Math.random`, `performance.now`, `crypto.randomUUID`; `setTimeout`, `setInterval`, `fetch`, `inject`, `require`. | `bannedModules` |
 | `local/test-file-shape` | A file named `*.spec-support.ts`, `*.spec-helper.ts`, `*.test-utils.ts`, `*-fixture.ts`, or under `__mocks__/` / `helpers/` / `common/stubs/`; a `.stub.ts` outside `test/stubs/`, a `.mock.ts` outside `test/mocks/`, a `.spec.util.ts` outside `test/utils/`, or any `fixtures/` directory outside `test/fixtures/` (a dedicated `testing/` library is exempt from all of these); a `test/stubs/*.stub.ts` export not named `UPPER_SNAKE_STUB` or without a type annotation; a `test/mocks/*.mock.ts` export that is not a camelCase `...Mock` function with an explicit return type. | none |
 

package/eslint.cjs

@@ -687,7 +687,12 @@ var classExpressionLiterals = (source) => {
 };
 
 // packages/lint-suite/src/lib/rules/utils/template-classes.util.ts
-var CLASS_ATTRIBUTE = "class";
+var CLASS_ATTRIBUTES = [
+  "class",
+  "routerLinkActive",
+  "animate.enter",
+  "animate.leave"
+];
 var CLASS_BINDING_PREFIX = "class.";
 var TOKEN = /\S+/gu;
 var expressionSource = (value) => {
@@ -716,11 +721,11 @@ var boundClassName = (input) => {
   return input.name;
 };
 var isClassExpression = (input) => {
-  const isClassName = input.name === CLASS_ATTRIBUTE;
+  const isClassName = CLASS_ATTRIBUTES.includes(input.name);
   if (!isClassName) return false;
   const { details } = input.keySpan;
   if (details === null) return true;
-  return details === CLASS_ATTRIBUTE;
+  return details === input.name;
 };
 var expressionClasses = (input) => {
   const source = expressionSource(input.value);
@@ -745,7 +750,7 @@ var inputClasses = (input) => {
 var templateClasses = (node) => {
   const classes = [];
   for (const attribute of node.attributes) {
-    const isClassName = attribute.name === CLASS_ATTRIBUTE;
+    const isClassName = CLASS_ATTRIBUTES.includes(attribute.name);
     if (isClassName) {
       const found = staticClasses(attribute);
       classes.push(...found);
@@ -6811,13 +6816,13 @@ var import_utils35 = require("@typescript-eslint/utils");
 // packages/lint-suite/src/lib/rules/type-placement/common/type-placement.const.ts
 var DEFAULT_INTERNAL_PATTERNS = [];
 var docs19 = {
-  description: "Require types to live in common/*.type.ts files, *.const.ts files to export only consts, and type-only imports to come from *.type.ts files or a common barrel"
+  description: "Require types to live in common/*.type.ts files, *.const.ts files to export only consts, and type-only imports to come from *.type.ts or *.schema.ts files or a common barrel"
 };
 var messages19 = {
   typeOutsideTypeFile: "Exported type '{{ name }}' belongs in a common/*.type.ts or test/common/*.type.ts file, not here.",
   valueInTypeFile: "A *.type.ts file may only export type aliases and interfaces.",
   nonConstInConstFile: "A *.const.ts file may only export const variable declarations.",
-  typeImportNotFromTypeFile: "Type-only import from '{{ source }}' must come from a *.type.ts file or a common barrel."
+  typeImportNotFromTypeFile: "Type-only import from '{{ source }}' must come from a *.type.ts or *.schema.ts file, or a common barrel."
 };
 var stringItems2 = { type: "string" };
 var internalPatternsSchema = {
@@ -6848,10 +6853,10 @@ var defaultOptions10 = [
 var TYPE_FILE_SUFFIX = ".type.ts";
 var CONST_FILE_SUFFIX = ".const.ts";
 var COMMON_SEGMENT = "/common/";
-var EXEMPT_FILE = /\.(spec|stub|d)\.ts$/;
+var EXEMPT_FILE = /\.(spec|stub|schema|d)\.ts$/;
 var STATE_TYPE_FILE = /(^|[./])state\.type\.ts$/;
 var FIXTURES_SEGMENT2 = "/fixtures/";
-var TYPE_FILE_SEGMENT = /\.type(\.ts)?$/;
+var TYPE_SOURCE_SEGMENT = /\.(type|schema)(\.ts)?$/;
 var COMMON_BARREL = /(^|\/)common(\/index(\.ts)?)?$/;
 var createRule19 = import_utils35.ESLintUtils.RuleCreator(
   () => "https://github.com/F0rty-Tw0/lint-suite#type-placement"
@@ -6870,7 +6875,7 @@ var lastSegment = (source) => {
 var isTypeOnlySource = (source) => {
   const isCommonBarrel = COMMON_BARREL.test(source);
   if (isCommonBarrel) return true;
-  return TYPE_FILE_SEGMENT.test(lastSegment(source));
+  return TYPE_SOURCE_SEGMENT.test(lastSegment(source));
 };
 var isInternalSource = (source, patterns) => {
   const isRelative = source.startsWith(".");
@@ -7292,6 +7297,14 @@ var typescriptSafety = (0, import_config18.defineConfig)([
       // Class actions carry their payload as constructor parameter properties.
       "@typescript-eslint/parameter-properties": "off"
     }
+  },
+  {
+    files: ["**/*.stub.ts"],
+    rules: {
+      // local/test-file-shape requires STUB: Type; a literal stub would trip
+      // no-inferrable-types.
+      "@typescript-eslint/no-inferrable-types": "off"
+    }
   }
 ]);