@jterrazz/test 14.3.0 → 15.0.0

package/dist/oxlint.cjs

@@ -198,21 +198,10 @@ function isTestCallee(callee) {
 	if (callee.type === "CallExpression") return isTestCallee(callee.callee);
 	return false;
 }
-/** Does the test callee chain carry a modifier member with the given name? */
-function testCalleeHasModifier(callee, modifier) {
-	if (callee === void 0) return false;
-	if (callee.type === "MemberExpression") return memberPropertyName(callee) === modifier || testCalleeHasModifier(callee.object, modifier);
-	if (callee.type === "CallExpression") return testCalleeHasModifier(callee.callee, modifier);
-	return false;
-}
 /** The callback argument of a test call (arrow or function expression), if any. */
 function findTestCallback(args) {
 	return args.find((arg) => arg.type === "ArrowFunctionExpression" || arg.type === "FunctionExpression");
 }
-/** The static string name of a test call (`test('name', …)`), if a literal. */
-function testName(callNode) {
-	return stringValue((callNode.arguments ?? [])[0]);
-}
 /**
 * Visitor fragment collecting every import-like source: static imports,
 * dynamic `import()`, and re-exports. Rules spread this into their visitor.
@@ -239,6 +228,26 @@ function importSourceVisitor(onSource) {
 * The **statique** channel — one entry per shipped `jterrazz/*` rule. Each rule
 * file attaches its entry as `meta.docs`, so a rule and its normative text can
 * never drift (the completeness meta-test asserts every rule carries one).
+*
+* **What this channel does NOT own.** A convention oxlint's own `vitest` plugin
+* already enforces belongs to that plugin, not here — `@jterrazz/typescript` v10
+* turns the plugin on over the test globs in every profile, so a consumer of the
+* preset gets it without wiring. Four hygiene rules left the channel on that
+* ground (J1, J3, J4, J5 → `vitest/no-focused-tests` + `vitest/no-disabled-tests`,
+* `vitest/expect-expect`, `vitest/no-identical-title`, `vitest/prefer-lowercase-title`).
+* `prefer-lowercase-title` is stricter than J5 was: a title opening on an all-caps
+* identifier (`HTTP 404 …`, `DI …`) now fails, and the rulebook carries no
+* `allowedPrefixes` — a title starts lowercase, whatever its first word. The
+* rest are behaviour-identical on this package's own fixtures. Only J2 (the
+* arbitrary-sleep ban, which has no upstream counterpart) stays statique.
+*
+* **Severity.** The estate's rulebook has no warn tier: a rule is on at `error`
+* or off with a recorded reason, and that law binds `@jterrazz/typescript`'s
+* rulebook. This channel is its one documented exception, and it is narrow — an
+* id shaped `<family><n>w-…` is a REDUNDANCY heuristic (a `root` the walk would
+* have found, a probe cluster that wants a golden), a reading of taste that
+* would block a legitimate shape if it failed the build. Every hard convention
+* here is an `error`; `plugin.test.ts` holds the split.
 */
 const RULE_DOCS = {
 	"a1-specification-file": {
@@ -514,13 +523,6 @@ const RULE_DOCS = {
 		id: "I4",
 		rationale: "Dans les tests de module, mocks et données sont du CODE (`mockOf`, `*.fixtures.ts`) ; un vrai fichier appelle une spec."
 	},
-	"j1-no-only-skip": {
-		channel: "statique",
-		convention: "Aucun `.only` / `.skip` committé (`describe.only`, `test.only`, `test.skip`).",
-		family: "J",
-		id: "J1",
-		rationale: "Un `.only`/`.skip` oublié désactive silencieusement une partie de la suite."
-	},
 	"j2-no-sleep-in-specs": {
 		channel: "statique",
 		convention: "Aucun sleep arbitraire (`setTimeout`/`setInterval`/`Atomics.wait`) sous `specs/**` — la synchronisation passe par `waitFor`.",
@@ -528,27 +530,6 @@ const RULE_DOCS = {
 		id: "J2",
 		rationale: "Un sleep fixe rend les tests lents et instables ; attendre une condition est déterministe."
 	},
-	"j3-no-expectless-test": {
-		channel: "statique",
-		convention: "Un `test(...)` avec callback contient au moins un `expect(…)` ; `test.todo` (sans callback) est ignoré.",
-		family: "J",
-		id: "J3",
-		rationale: "Un test sans assertion est mort ou muet — il passe toujours sans rien vérifier."
-	},
-	"j4-unique-test-names": {
-		channel: "statique",
-		convention: "Deux tests d’un même fichier ne partagent pas un nom littéral (`.each` ignoré).",
-		family: "J",
-		id: "J4",
-		rationale: "Le nom du test est son unique description — deux noms identiques rendent un échec ambigu."
-	},
-	"j5-lowercase-title": {
-		channel: "statique",
-		convention: "La première lettre d’un titre `test()`/`describe()`/`it()` littéral est en minuscule. Exemptés : les titres dont le premier MOT est un identifiant tout en majuscules/underscores (`VALID_CATEGORIES`, `HTTP`, `DI`) et ceux démarrant sur un non-lettre — seuls les premiers mots de prose minusculisables sont contraints.",
-		family: "J",
-		id: "J5",
-		rationale: "Un titre est un fragment de prose, pas une phrase — la casse minuscule le garde fragmentaire ; minusculiser un symbole nommé le mal-orthographierait."
-	},
 	"w1-scenario-pure": {
 		channel: "statique",
 		convention: "Un scénario (`.visit()` website, `.open()` mobile) est le When : le visiteur agit, la capture reflète l’état final ; aucun `expect()` dans le callback — les assertions vivent dans le Then, sur le résultat retourné.",
@@ -907,7 +888,7 @@ const a1SpecificationFile = {
 	},
 	meta: {
 		docs: RULE_DOCS["a1-specification-file"],
-		messages: { outsideSpecification: "specification.{{member}}() must be called from a `*.specification.ts` file under specs/ (A1 — see docs/10-linting.md)." },
+		messages: { outsideSpecification: "specification.{{member}}() must be called from a `*.specification.ts` file under specs/ (A1 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -984,13 +965,13 @@ const a10DuplicateBinding = {
 	},
 	meta: {
 		docs: RULE_DOCS["a10-duplicate-binding"],
-		messages: { duplicate: "Service keys \"{{first}}\" and \"{{key}}\" both bind to compose service \"{{binding}}\" — one shadows the other; rename a key or set distinct composeService values (A10 — see docs/10-linting.md)." },
+		messages: { duplicate: "Service keys \"{{first}}\" and \"{{key}}\" both bind to compose service \"{{binding}}\" — one shadows the other; rename a key or set distinct composeService values (A10 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
 //#endregion
 //#region src/lint/rules/a2-known-constructors.ts
-/** The five constructors, and only five (A2 — see docs/10-linting.md). */
+/** The five constructors, and only five (A2 — see docs/13-linting.md). */
 const KNOWN_CONSTRUCTORS = /* @__PURE__ */ new Set([
 	"api",
 	"cli",
@@ -1019,7 +1000,7 @@ const a2KnownConstructors = {
 	},
 	meta: {
 		docs: RULE_DOCS["a2-known-constructors"],
-		messages: { unknownConstructor: "specification.{{member}} does not exist — the only constructors are specification.api(), specification.jobs(), specification.cli(), specification.website() and specification.mobile() (A2 — see docs/10-linting.md)." },
+		messages: { unknownConstructor: "specification.{{member}} does not exist — the only constructors are specification.api(), specification.jobs(), specification.cli(), specification.website() and specification.mobile() (A2 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -1064,7 +1045,7 @@ const a3NoDestructureAlias = {
 	},
 	meta: {
 		docs: RULE_DOCS["a3-no-destructure-alias"],
-		messages: { aliased: "Destructure the specification result with its canonical name: `{{key}}`, not `{{key}}: {{alias}}` (A3 — see docs/10-linting.md). Rename at the import site if a different local name is needed." },
+		messages: { aliased: "Destructure the specification result with its canonical name: `{{key}}`, not `{{key}}: {{alias}}` (A3 — see docs/13-linting.md). Rename at the import site if a different local name is needed." },
 		type: "problem"
 	}
 };
@@ -1116,7 +1097,7 @@ const a4CleanupAfterall = {
 	},
 	meta: {
 		docs: RULE_DOCS["a4-cleanup-afterall"],
-		messages: { unregistered: "`cleanup` is destructured but never passed to afterAll — add `afterAll(cleanup)` to the specification file (A4 — see docs/10-linting.md)." },
+		messages: { unregistered: "`cleanup` is destructured but never passed to afterAll — add `afterAll(cleanup)` to the specification file (A4 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -1146,7 +1127,7 @@ const a5ModeWithServer = {
 	},
 	meta: {
 		docs: RULE_DOCS["a5-mode-with-server"],
-		messages: { hardcodedMode: "`mode` must not be hardcoded when `server` is defined — the node/compose switch lives in vitest.config.ts via `env: { TEST_MODE: \"compose\" }` (A5 — see docs/10-linting.md)." },
+		messages: { hardcodedMode: "`mode` must not be hardcoded when `server` is defined — the node/compose switch lives in vitest.config.ts via `env: { TEST_MODE: \"compose\" }` (A5 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -1187,7 +1168,7 @@ const a6wRedundantComposeService = {
 	},
 	meta: {
 		docs: RULE_DOCS["a6w-redundant-compose-service"],
-		messages: { redundant: "composeService: \"{{name}}\" is redundant — the key \"{{key}}\" already binds to it (exact name or kebab-case derivation, A6 — see docs/10-linting.md)." },
+		messages: { redundant: "composeService: \"{{name}}\" is redundant — the key \"{{key}}\" already binds to it (exact name or kebab-case derivation, A6 — see docs/13-linting.md)." },
 		type: "suggestion"
 	}
 };
@@ -1222,7 +1203,7 @@ function findRoot(startDir, exists) {
 //#endregion
 //#region src/lint/rules/a9w-redundant-root.ts
 /**
-* The root the convention would derive (A9 — see docs/10-linting.md): the
+* The root the convention would derive (A9 — see docs/13-linting.md): the
 * framework's own walk-up, run against this layer's cached fs probe. The rule
 * flags a `root` that the runner would have found by itself, so the two MUST
 * be the same function — a private copy of the walk drifted from it before.
@@ -1254,7 +1235,7 @@ const a9wRedundantRoot = {
 	},
 	meta: {
 		docs: RULE_DOCS["a9w-redundant-root"],
-		messages: { redundant: "root: \"{{root}}\" is redundant — walking up from the specification file already resolves to that directory (A9 — see docs/10-linting.md)." },
+		messages: { redundant: "root: \"{{root}}\" is redundant — walking up from the specification file already resolves to that directory (A9 — see docs/13-linting.md)." },
 		type: "suggestion"
 	}
 };
@@ -1360,7 +1341,7 @@ const b2KnownFixtureMarker = {
 	},
 	meta: {
 		docs: RULE_DOCS["b2-known-fixture-marker"],
-		messages: { unknownMarker: "Unknown fixture marker \"{{marker}}\" — known markers: {{known}}. A path without a marker is feature-local (B2 — see docs/10-linting.md)." },
+		messages: { unknownMarker: "Unknown fixture marker \"{{marker}}\" — known markers: {{known}}. A path without a marker is feature-local (B2 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -1425,8 +1406,8 @@ const b4GivenThen = {
 	meta: {
 		docs: RULE_DOCS["b4-given-then"],
 		messages: {
-			givenAfterThen: "The `// Given -` marker comes after `// Then -` — Given describes the setup and must precede Then (B4 — see docs/10-linting.md).",
-			missing: "Test is missing a `// {{marker}} -` comment (B4 — see docs/10-linting.md — every test needs both `// Given -` and `// Then -`)."
+			givenAfterThen: "The `// Given -` marker comes after `// Then -` — Given describes the setup and must precede Then (B4 — see docs/13-linting.md).",
+			missing: "Test is missing a `// {{marker}} -` comment (B4 — see docs/13-linting.md — every test needs both `// Given -` and `// Then -`)."
 		},
 		type: "problem"
 	}
@@ -1467,7 +1448,7 @@ const b5AwaitUsing = {
 	meta: {
 		docs: RULE_DOCS["b5-await-using"],
 		defaultOptions: [{ runners: [] }],
-		messages: { requireAwaitUsing: "The result of docker-aware runner \"{{runner}}\" must be bound with `await using` so its containers are disposed (B5 — see docs/10-linting.md)." },
+		messages: { requireAwaitUsing: "The result of docker-aware runner \"{{runner}}\" must be bound with `await using` so its containers are disposed (B5 — see docs/13-linting.md)." },
 		schema: [{
 			additionalProperties: false,
 			properties: { runners: {
@@ -1510,7 +1491,7 @@ const b6wRedundantEnvUrl = {
 	},
 	meta: {
 		docs: RULE_DOCS["b6w-redundant-env-url"],
-		messages: { redundant: ".env({ {{key}}: ….connectionString }) is redundant — the framework already injects <KEY>_URL (and the DATABASE_URL/REDIS_URL aliases) for every service (B6 — see docs/10-linting.md)." },
+		messages: { redundant: ".env({ {{key}}: ….connectionString }) is redundant — the framework already injects <KEY>_URL (and the DATABASE_URL/REDIS_URL aliases) for every service (B6 — see docs/13-linting.md)." },
 		type: "suggestion"
 	}
 };
@@ -1539,7 +1520,7 @@ const b8KebabTrigger = {
 	},
 	meta: {
 		docs: RULE_DOCS["b8-kebab-trigger"],
-		messages: { notKebab: "Job name \"{{name}}\" must be a stable kebab-case identifier (e.g. \"nightly-report\") — it is a contract between the app and its tests (B8 — see docs/10-linting.md)." },
+		messages: { notKebab: "Job name \"{{name}}\" must be a stable kebab-case identifier (e.g. \"nightly-report\") — it is a contract between the app and its tests (B8 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -1594,7 +1575,7 @@ const b9wProductCommand = {
 	},
 	meta: {
 		docs: RULE_DOCS["b9w-product-command"],
-		messages: { thirdPartyBinary: "specification.cli() points at a third-party binary in node_modules/.bin — a spec should exercise your product command (cli.exec(\"check\"), …), not the tool underneath (B9 — see docs/10-linting.md). Suppress with a reason if the product genuinely is this binary." },
+		messages: { thirdPartyBinary: "specification.cli() points at a third-party binary in node_modules/.bin — a spec should exercise your product command (cli.exec(\"check\"), …), not the tool underneath (B9 — see docs/13-linting.md). Suppress with a reason if the product genuinely is this binary." },
 		type: "suggestion"
 	}
 };
@@ -1747,14 +1728,14 @@ const c1DomainStructure = {
 		defaultOptions: [{ depth: DEFAULT_DEPTH }],
 		docs: RULE_DOCS["c1-domain-structure"],
 		messages: {
-			specInGround: "A spec must not live under \"{{ground}}/\" — a leading underscore marks GROUND (what the specs of a row stand on: _fixtures/, _expected/, _requests/, _seeds/), never a domain (C1 — see docs/10-linting.md). Ground that is CODE keeps its own unit test: `<module>.test.ts` NEXT to the `<module>.ts` it tests (I2), and nothing else.",
-			specNotAtFacetRoot: "A `*.specification.ts` must sit at the facet root: `specs/<facet>/<name>.specification.ts` (C1 — see docs/10-linting.md).",
-			testAtFacetRoot: "A `*.test.ts` must live in a domain folder: `specs/<facet>/<domain>/<aspect>.test.ts` — tests directly at the facet root are forbidden (C1 — see docs/10-linting.md).",
-			testAtSpecsRoot: "A `*.test.ts` must live in a directory under specs/ — with `depth: \"mirror\"` the tree mirrors a structure, and the specs root mirrors nothing (C1 — see docs/10-linting.md).",
-			testNotMirroringDirectory: "A `*.test.ts` must be named after the directory holding it — `{{directory}}/{{directory}}.test.ts` — with `depth: \"mirror\"` (C1 — see docs/10-linting.md).",
-			testOutsideFacet: "A `*.test.ts` must live in a facet folder — `specs/<facet>/<aspect>.test.ts`, or one domain deeper: with `depth: \"facet\"` the specs root holds facets, not tests (C1 — see docs/10-linting.md).",
-			testTooDeep: "A `*.test.ts` must sit at facet/domain depth: `specs/<facet>/<domain>/<aspect>.test.ts` — no deeper nesting (C1 — see docs/10-linting.md).",
-			testTooDeepForFacet: "A `*.test.ts` must sit at the facet root or one domain folder down — `specs/<facet>/<aspect>.test.ts` or `specs/<facet>/<domain>/<aspect>.test.ts`: with `depth: \"facet\"` nothing nests deeper (C1 — see docs/10-linting.md)."
+			specInGround: "A spec must not live under \"{{ground}}/\" — a leading underscore marks GROUND (what the specs of a row stand on: _fixtures/, _expected/, _requests/, _seeds/), never a domain (C1 — see docs/13-linting.md). Ground that is CODE keeps its own unit test: `<module>.test.ts` NEXT to the `<module>.ts` it tests (I2), and nothing else.",
+			specNotAtFacetRoot: "A `*.specification.ts` must sit at the facet root: `specs/<facet>/<name>.specification.ts` (C1 — see docs/13-linting.md).",
+			testAtFacetRoot: "A `*.test.ts` must live in a domain folder: `specs/<facet>/<domain>/<aspect>.test.ts` — tests directly at the facet root are forbidden (C1 — see docs/13-linting.md).",
+			testAtSpecsRoot: "A `*.test.ts` must live in a directory under specs/ — with `depth: \"mirror\"` the tree mirrors a structure, and the specs root mirrors nothing (C1 — see docs/13-linting.md).",
+			testNotMirroringDirectory: "A `*.test.ts` must be named after the directory holding it — `{{directory}}/{{directory}}.test.ts` — with `depth: \"mirror\"` (C1 — see docs/13-linting.md).",
+			testOutsideFacet: "A `*.test.ts` must live in a facet folder — `specs/<facet>/<aspect>.test.ts`, or one domain deeper: with `depth: \"facet\"` the specs root holds facets, not tests (C1 — see docs/13-linting.md).",
+			testTooDeep: "A `*.test.ts` must sit at facet/domain depth: `specs/<facet>/<domain>/<aspect>.test.ts` — no deeper nesting (C1 — see docs/13-linting.md).",
+			testTooDeepForFacet: "A `*.test.ts` must sit at the facet root or one domain folder down — `specs/<facet>/<aspect>.test.ts` or `specs/<facet>/<domain>/<aspect>.test.ts`: with `depth: \"facet\"` nothing nests deeper (C1 — see docs/13-linting.md)."
 		},
 		schema: [{
 			additionalProperties: false,
@@ -1799,7 +1780,7 @@ const c10ContractsBoundary = {
 	},
 	meta: {
 		docs: RULE_DOCS["c10-contracts-boundary"],
-		messages: { internal: "Import \"{{source}}\" reaches into a provider folder — only contracts/*.contracts.ts is importable from a test; add a named scenario export there instead (C10 — see docs/10-linting.md)." },
+		messages: { internal: "Import \"{{source}}\" reaches into a provider folder — only contracts/*.contracts.ts is importable from a test; add a named scenario export there instead (C10 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -1854,7 +1835,7 @@ const c11ContractDataPairing = {
 	},
 	meta: {
 		docs: RULE_DOCS["c11-contract-data-pairing"],
-		messages: { orphan: "contracts/{{provider}}/{{entry}} has no owning contract — data pairs with the sibling {{owner}} (stem = name up to the first dot) (C11 — see docs/10-linting.md)." },
+		messages: { orphan: "contracts/{{provider}}/{{entry}} has no owning contract — data pairs with the sibling {{owner}} (stem = name up to the first dot) (C11 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -1907,7 +1888,7 @@ const c13UnderscoredGround = {
 	},
 	meta: {
 		docs: RULE_DOCS["c13-underscored-ground"],
-		messages: { notUnderscored: "The directory \"{{legacy}}/\" is a pre-14 ground name — rename it to \"{{ground}}/\" (`git mv {{legacy}} {{ground}}`). What a spec stands on carries the underscore; a spec's own folder never does, and no resolver reads the un-underscored name (C13 — see docs/10-linting.md)." },
+		messages: { notUnderscored: "The directory \"{{legacy}}/\" is a pre-14 ground name — rename it to \"{{ground}}/\" (`git mv {{legacy}} {{ground}}`). What a spec stands on carries the underscore; a spec's own folder never does, and no resolver reads the un-underscored name (C13 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -1936,7 +1917,7 @@ const c2HttpOnlyRequests = {
 	},
 	meta: {
 		docs: RULE_DOCS["c2-http-only-requests"],
-		messages: { notHttp: "_requests/{{entry}} is not a .http file — _requests/ contains only complete .http request files (C2 — see docs/10-linting.md)." },
+		messages: { notHttp: "_requests/{{entry}} is not a .http file — _requests/ contains only complete .http request files (C2 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -2160,18 +2141,18 @@ const c4ContractShape = {
 	meta: {
 		docs: RULE_DOCS["c4-contract-shape"],
 		messages: {
-			badName: "Contract unit \"{{base}}\" must be named <kebab-name>.ts — the folder already carries the provider (C4 — see docs/10-linting.md).",
-			badProviderDir: "contracts/{{entry}}/ is not a provider directory — a contracts/ root holds only http/, openai/, anthropic/ and *.contracts.ts files (C4 — see docs/10-linting.md).",
-			missingComposite: "Contract facade \"{{base}}\" has no default export — it must default-export the composed world (C4 — see docs/10-linting.md).",
-			missingDefault: "Contract unit \"{{base}}\" has no default export — a unit default-exports `defineContract(...)` or a factory returning one (C4 — see docs/10-linting.md).",
-			notDefineContract: "The default export of a contract unit must be `defineContract(...)` or a factory returning a contract (C4 — see docs/10-linting.md).",
-			notDefineContracts: "The default export of \"{{base}}\" must be built from `defineContracts(...)` (or a composition re-export) (C4 — see docs/10-linting.md).",
-			providerEntry: "contracts/{{provider}}/{{child}} is neither a *.ts contract nor a *.response.json payload (C4 — see docs/10-linting.md).",
-			providerMismatch: "This contract lives in contracts/{{provider}}/ but its request is a `{{declared}}.*` builder — the folder carries the provider (C4 — see docs/10-linting.md).",
-			providerSubfolder: "contracts/{{provider}}/{{child}}/ is nested — a provider folder is flat (C4 — see docs/10-linting.md).",
-			rootEntry: "contracts/{{entry}} is not a *.contracts.ts facade — data files live in a provider folder next to their contract (C4 — see docs/10-linting.md).",
-			rootFile: "Contract file \"{{base}}\" sits at the contracts/ root, which holds only *.contracts.ts facades — a unit contract belongs in http/, openai/ or anthropic/ (C4 — see docs/10-linting.md).",
-			tooDeep: "Contract file \"{{base}}\" is nested deeper than contracts/<provider>/ (C4 — see docs/10-linting.md)."
+			badName: "Contract unit \"{{base}}\" must be named <kebab-name>.ts — the folder already carries the provider (C4 — see docs/13-linting.md).",
+			badProviderDir: "contracts/{{entry}}/ is not a provider directory — a contracts/ root holds only http/, openai/, anthropic/ and *.contracts.ts files (C4 — see docs/13-linting.md).",
+			missingComposite: "Contract facade \"{{base}}\" has no default export — it must default-export the composed world (C4 — see docs/13-linting.md).",
+			missingDefault: "Contract unit \"{{base}}\" has no default export — a unit default-exports `defineContract(...)` or a factory returning one (C4 — see docs/13-linting.md).",
+			notDefineContract: "The default export of a contract unit must be `defineContract(...)` or a factory returning a contract (C4 — see docs/13-linting.md).",
+			notDefineContracts: "The default export of \"{{base}}\" must be built from `defineContracts(...)` (or a composition re-export) (C4 — see docs/13-linting.md).",
+			providerEntry: "contracts/{{provider}}/{{child}} is neither a *.ts contract nor a *.response.json payload (C4 — see docs/13-linting.md).",
+			providerMismatch: "This contract lives in contracts/{{provider}}/ but its request is a `{{declared}}.*` builder — the folder carries the provider (C4 — see docs/13-linting.md).",
+			providerSubfolder: "contracts/{{provider}}/{{child}}/ is nested — a provider folder is flat (C4 — see docs/13-linting.md).",
+			rootEntry: "contracts/{{entry}} is not a *.contracts.ts facade — data files live in a provider folder next to their contract (C4 — see docs/13-linting.md).",
+			rootFile: "Contract file \"{{base}}\" sits at the contracts/ root, which holds only *.contracts.ts facades — a unit contract belongs in http/, openai/ or anthropic/ (C4 — see docs/13-linting.md).",
+			tooDeep: "Contract file \"{{base}}\" is nested deeper than contracts/<provider>/ (C4 — see docs/13-linting.md)."
 		},
 		type: "problem"
 	}
@@ -2222,7 +2203,7 @@ const c6ToMatchExtension = {
 	},
 	meta: {
 		docs: RULE_DOCS["c6-tomatch-extension"],
-		messages: { missingExtension: "toMatch(\"{{name}}\") is missing its extension — the extension is part of the fixture name (\"help.txt\", never \"help\"); only directory-tree snapshots (_expected/<name>/) omit it (C6 — see docs/10-linting.md)." },
+		messages: { missingExtension: "toMatch(\"{{name}}\") is missing its extension — the extension is part of the fixture name (\"help.txt\", never \"help\"); only directory-tree snapshots (_expected/<name>/) omit it (C6 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -2250,7 +2231,7 @@ const c7SeedsSqlOnly = {
 	},
 	meta: {
 		docs: RULE_DOCS["c7-seeds-sql-only"],
-		messages: { notSql: "_seeds/{{entry}} is not a .sql file — _seeds/ carries database state only; file state goes through .fixture() (C7 — see docs/10-linting.md)." },
+		messages: { notSql: "_seeds/{{entry}} is not a .sql file — _seeds/ carries database state only; file state goes through .fixture() (C7 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -2327,7 +2308,7 @@ const c8ReferencedFixtureExists = {
 	},
 	meta: {
 		docs: RULE_DOCS["c8-referenced-fixture-exists"],
-		messages: { missing: "Referenced fixture \"{{path}}\" does not exist on disk under its ground {{root}}/ root (C8 — see docs/10-linting.md). Create it or fix the reference." },
+		messages: { missing: "Referenced fixture \"{{path}}\" does not exist on disk under its ground {{root}}/ root (C8 — see docs/13-linting.md). Create it or fix the reference." },
 		type: "problem"
 	}
 };
@@ -2396,7 +2377,7 @@ const d12wResponseBodyProbe = {
 	meta: {
 		docs: RULE_DOCS["d12w-response-body-probe"],
 		defaultOptions: [{ threshold: DEFAULT_THRESHOLD }],
-		messages: { bodyProbeCluster: "{{count}} body probes in one test — this wants a full golden: expect(result.response).toMatch('x.http') (D12 — see docs/10-linting.md)." },
+		messages: { bodyProbeCluster: "{{count}} body probes in one test — this wants a full golden: expect(result.response).toMatch('x.http') (D12 — see docs/13-linting.md)." },
 		schema: [{
 			additionalProperties: false,
 			properties: { threshold: {
@@ -2479,7 +2460,7 @@ const d13wUnfrozenNegativeFixture = {
 	},
 	meta: {
 		docs: RULE_DOCS["d13w-unfrozen-negative-fixture"],
-		messages: { unfrozenNegativeFixture: "This toMatch asserts a mismatch (wrapped in expect(…).toThrow/.rejects) — pass { frozen: true } so TEST_UPDATE=1 never overwrites the deliberately-wrong fixture (D13 — see docs/10-linting.md)." },
+		messages: { unfrozenNegativeFixture: "This toMatch asserts a mismatch (wrapped in expect(…).toThrow/.rejects) — pass { frozen: true } so TEST_UPDATE=1 never overwrites the deliberately-wrong fixture (D13 — see docs/13-linting.md)." },
 		type: "suggestion"
 	}
 };
@@ -2555,7 +2536,7 @@ const d15wStatusOnlyProbe = {
 	},
 	meta: {
 		docs: RULE_DOCS["d15w-status-only-probe"],
-		messages: { statusOnlyProbe: "The test's only assertion(s) are status probes — pin the full response: expect(result.response).toMatch('case.http') (D11 — see docs/10-linting.md)." },
+		messages: { statusOnlyProbe: "The test's only assertion(s) are status probes — pin the full response: expect(result.response).toMatch('case.http') (D11 — see docs/13-linting.md)." },
 		type: "suggestion"
 	}
 };
@@ -2610,7 +2591,7 @@ const d2AwaitIoMatcher = {
 	},
 	meta: {
 		docs: RULE_DOCS["d2-await-io-matcher"],
-		messages: { mustAwait: "expect(…).{{matcher}}(…) is IO-backed and returns a promise — it must be awaited or returned, else the assertion never runs (D2 — see docs/10-linting.md)." },
+		messages: { mustAwait: "expect(…).{{matcher}}(…) is IO-backed and returns a promise — it must be awaited or returned, else the assertion never runs (D2 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -2664,7 +2645,7 @@ const d2wAwaitSyncMatcher = {
 	},
 	meta: {
 		docs: RULE_DOCS["d2w-await-sync-matcher"],
-		messages: { redundantAwait: "await on expect(…).{{matcher}}(…) is redundant — that matcher is synchronous; only IO-backed matchers (toMatchRows/toBeEmpty/toBeRunning) are awaited (D2 — see docs/10-linting.md)." },
+		messages: { redundantAwait: "await on expect(…).{{matcher}}(…) is redundant — that matcher is synchronous; only IO-backed matchers (toMatchRows/toBeEmpty/toBeRunning) are awaited (D2 — see docs/13-linting.md)." },
 		type: "suggestion"
 	}
 };
@@ -2741,7 +2722,7 @@ const d6wTransformTokenEquivalent = {
 	},
 	meta: {
 		docs: RULE_DOCS["d6w-transform-token-equivalent"],
-		messages: { tokenEquivalent: "This transform only rewrites output into known token literals — write the tokens in the _expected/ fixture instead; transform is an escape hatch for uncovered noise (D6 — see docs/10-linting.md)." },
+		messages: { tokenEquivalent: "This transform only rewrites output into known token literals — write the tokens in the _expected/ fixture instead; transform is an escape hatch for uncovered noise (D6 — see docs/13-linting.md)." },
 		type: "suggestion"
 	}
 };
@@ -2775,7 +2756,7 @@ const d8wTextBypass = {
 	},
 	meta: {
 		docs: RULE_DOCS["d8w-text-bypass"],
-		messages: { textBypass: "expect(x.text).{{matcher}}(…) asserts on the raw stream — prefer the typed accessor subject (expect(x)) so the token grammar and fixture resolution apply (D8 — see docs/10-linting.md)." },
+		messages: { textBypass: "expect(x.text).{{matcher}}(…) asserts on the raw stream — prefer the typed accessor subject (expect(x)) so the token grammar and fixture resolution apply (D8 — see docs/13-linting.md)." },
 		type: "suggestion"
 	}
 };
@@ -2858,7 +2839,7 @@ const d9wSingleUseRef = {
 	},
 	meta: {
 		docs: RULE_DOCS["d9w-single-use-ref"],
-		messages: { singleUse: "Capture ref \"{{ref}}\" is used only once in this spec — a ref earns its name by asserting equality across occurrences; use a plain matcher instead (D9 — see docs/10-linting.md)." },
+		messages: { singleUse: "Capture ref \"{{ref}}\" is used only once in this spec — a ref earns its name by asserting equality across occurrences; use a plain matcher instead (D9 — see docs/13-linting.md)." },
 		type: "suggestion"
 	}
 };
@@ -2952,7 +2933,7 @@ const f1NoSubpathImport = {
 	},
 	meta: {
 		docs: RULE_DOCS["f1-no-subpath-import"],
-		messages: { subpath: "Import from \"@jterrazz/test\", not \"{{source}}\" — internal subpaths do not exist (F1 — see docs/10-linting.md). Published subpaths, exempt everywhere: {{published}}." },
+		messages: { subpath: "Import from \"@jterrazz/test\", not \"{{source}}\" — internal subpaths do not exist (F1 — see docs/13-linting.md). Published subpaths, exempt everywhere: {{published}}." },
 		type: "problem"
 	}
 };
@@ -3000,7 +2981,7 @@ const f2NoTestImportsInProd = {
 	},
 	meta: {
 		docs: RULE_DOCS["f2-no-test-imports-in-prod"],
-		messages: { testImport: "Production code must not import the test artefact \"{{source}}\" — test imports are only legal from specs/, *.test.ts, *.fixtures.ts or *.specification.ts (F2 — see docs/10-linting.md)." },
+		messages: { testImport: "Production code must not import the test artefact \"{{source}}\" — test imports are only legal from specs/, *.test.ts, *.fixtures.ts or *.specification.ts (F2 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -3063,7 +3044,7 @@ const f3SpecsPublicEntry = {
 	},
 	meta: {
 		docs: RULE_DOCS["f3-specs-public-entry"],
-		messages: { deepImport: "specs/ must not deep-import framework internals — reach the framework via its public entry (@jterrazz/test, or src/index.js in this repo), not \"{{source}}\" (F3 — see docs/10-linting.md; only @jterrazz/test/oxlint, and src/integrations/** from specs/integrations/, are exempt)." },
+		messages: { deepImport: "specs/ must not deep-import framework internals — reach the framework via its public entry (@jterrazz/test, or src/index.js in this repo), not \"{{source}}\" (F3 — see docs/13-linting.md; only @jterrazz/test/oxlint, and src/integrations/** from specs/integrations/, are exempt)." },
 		type: "problem"
 	}
 };
@@ -3089,7 +3070,7 @@ const f4NoTestToTestImport = {
 	},
 	meta: {
 		docs: RULE_DOCS["f4-no-test-to-test-import"],
-		messages: { testToTest: "A test file must not import another test file (\"{{source}}\") — share data via a *.fixtures.ts neighbour or a *.specification.ts runner (F4 — see docs/10-linting.md)." },
+		messages: { testToTest: "A test file must not import another test file (\"{{source}}\") — share data via a *.fixtures.ts neighbour or a *.specification.ts runner (F4 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -3115,7 +3096,7 @@ const f5FixturesOnlyFromTests = {
 	},
 	meta: {
 		docs: RULE_DOCS["f5-fixtures-only-from-tests"],
-		messages: { fixturesImport: "A *.fixtures.ts module (\"{{source}}\") is only importable from *.test.ts files (F5 — see docs/10-linting.md)." },
+		messages: { fixturesImport: "A *.fixtures.ts module (\"{{source}}\") is only importable from *.test.ts files (F5 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -3207,8 +3188,8 @@ const i1LayerBoundaries = {
 		defaultOptions: [{ layers: {} }],
 		docs: RULE_DOCS["i1-layer-boundaries"],
 		messages: {
-			crossLayer: "Layer \"{{layer}}\" must not import \"{{source}}\" — outside the edges its layer map declares (I1 — see docs/10-linting.md).",
-			foreignDependency: "\"{{source}}\" is not a dependency layer \"{{layer}}\" declares — one layer states the packages it may import (I1 — see docs/10-linting.md)."
+			crossLayer: "Layer \"{{layer}}\" must not import \"{{source}}\" — outside the edges its layer map declares (I1 — see docs/13-linting.md).",
+			foreignDependency: "\"{{source}}\" is not a dependency layer \"{{layer}}\" declares — one layer states the packages it may import (I1 — see docs/13-linting.md)."
 		},
 		schema: [{
 			additionalProperties: false,
@@ -3296,9 +3277,9 @@ const i2SiblingTestNaming = {
 	meta: {
 		docs: RULE_DOCS["i2-sibling-test-naming"],
 		messages: {
-			orphanTest: "A src/ module test must sit NEXT to the module it tests — no neighbour \"{{expected}}\" found (I2 — see docs/10-linting.md). A test needing more than its module is a specification: move it to specs/.",
-			rootTests: "A root-level tests/ directory is banned — module tests are siblings under src/, product specifications live in specs/ (I2 — see docs/10-linting.md).",
-			testsDir: "__tests__/ directories are banned under src/ — the test of <file>.ts is its neighbour <file>.test.ts (I2 — see docs/10-linting.md)."
+			orphanTest: "A src/ module test must sit NEXT to the module it tests — no neighbour \"{{expected}}\" found (I2 — see docs/13-linting.md). A test needing more than its module is a specification: move it to specs/.",
+			rootTests: "A root-level tests/ directory is banned — module tests are siblings under src/, product specifications live in specs/ (I2 — see docs/13-linting.md).",
+			testsDir: "__tests__/ directories are banned under src/ — the test of <file>.ts is its neighbour <file>.test.ts (I2 — see docs/13-linting.md)."
 		},
 		type: "problem"
 	}
@@ -3398,57 +3379,14 @@ const i4NoViMockInSrc = {
 	meta: {
 		docs: RULE_DOCS["i4-no-vi-mock-in-src"],
 		messages: {
-			assetImport: "A src/ module test must not import the data asset \"{{source}}\" — inline it as code or move the test to specs/ (I4 — see docs/10-linting.md).",
-			bannedDir: "`{{dir}}/` directories are banned under src/ — mocks and data are code: mockOf/mockOfDate inline, payloads in a *.fixtures.ts neighbour (I4 — see docs/10-linting.md).",
-			viMock: "`vi.mock` is banned under src/ — use mockOf/mockOfDate (I4 — see docs/10-linting.md)."
+			assetImport: "A src/ module test must not import the data asset \"{{source}}\" — inline it as code or move the test to specs/ (I4 — see docs/13-linting.md).",
+			bannedDir: "`{{dir}}/` directories are banned under src/ — mocks and data are code: mockOf/mockOfDate inline, payloads in a *.fixtures.ts neighbour (I4 — see docs/13-linting.md).",
+			viMock: "`vi.mock` is banned under src/ — use mockOf/mockOfDate (I4 — see docs/13-linting.md)."
 		},
 		type: "problem"
 	}
 };
 //#endregion
-//#region src/lint/rules/j1-no-only-skip.ts
-/** Test runners whose `.only` / `.skip` modifiers must never be committed. */
-const RUNNERS = /* @__PURE__ */ new Set([
-	"describe",
-	"it",
-	"test"
-]);
-/** The focus/skip modifiers CONVENTIONS J1 forbids. */
-const FORBIDDEN = /* @__PURE__ */ new Set(["only", "skip"]);
-/**
-* CONVENTIONS J1 — no committed `.only` / `.skip`.
-*
-* Flags the member form used across the specs: `describe.only`, `test.only`,
-* `it.only` and their `.skip` counterparts. Modifiers that legitimately gate a
-* test (`test.skipIf`, `test.runIf`) are untouched — only the bare `only`/`skip`
-* that pin or disable a test in a commit are reported.
-*/
-const j1NoOnlySkip = {
-	create(context) {
-		return { MemberExpression(node) {
-			if (node.computed === true) return;
-			const object = node.object;
-			const property = node.property;
-			if (object?.type !== "Identifier" || property?.type !== "Identifier") return;
-			const runner = object.name;
-			const modifier = property.name;
-			if (RUNNERS.has(runner) && FORBIDDEN.has(modifier)) context.report({
-				data: {
-					modifier,
-					runner
-				},
-				messageId: "forbidden",
-				node
-			});
-		} };
-	},
-	meta: {
-		docs: RULE_DOCS["j1-no-only-skip"],
-		messages: { forbidden: "Committed `{{runner}}.{{modifier}}` is not allowed (J1 — see docs/10-linting.md). Remove it before committing." },
-		type: "problem"
-	}
-};
-//#endregion
 //#region src/lint/rules/j2-no-sleep-in-specs.ts
 /**
 * CONVENTIONS J2 — spec files (`specs/**`) contain no arbitrary sleeps:
@@ -3487,138 +3425,13 @@ const j2NoSleepInSpecs = {
 	meta: {
 		docs: RULE_DOCS["j2-no-sleep-in-specs"],
 		messages: {
-			sleep: "No arbitrary sleeps in specs — synchronise via `waitFor` or the framework (J2 — see docs/10-linting.md).",
-			timersImport: "No timer-based sleeps in specs — synchronise via `waitFor` or the framework (J2 — see docs/10-linting.md)."
+			sleep: "No arbitrary sleeps in specs — synchronise via `waitFor` or the framework (J2 — see docs/13-linting.md).",
+			timersImport: "No timer-based sleeps in specs — synchronise via `waitFor` or the framework (J2 — see docs/13-linting.md)."
 		},
 		type: "problem"
 	}
 };
 //#endregion
-//#region src/lint/rules/j3-no-expectless-test.ts
-/** Does `callback` contain at least one `expect(…)` call? */
-function hasExpect(callback) {
-	let found = false;
-	walk(callback, (node) => {
-		if (found || node.type !== "CallExpression") return;
-		const callee = node.callee;
-		if (callee?.type === "Identifier" && callee.name === "expect") found = true;
-	});
-	return found;
-}
-/**
-* CONVENTIONS J3 — a test asserts something. A `test('…', () => { … })` whose
-* body contains no `expect(…)` is either dead or a silent no-op.
-*
-* Reuses the shared B4 test-shape helpers. `test.todo(...)` (and any other
-* callback-less form) is skipped — there is nothing to assert yet.
-*/
-const j3NoExpectlessTest = {
-	create(context) {
-		return { CallExpression(node) {
-			const callee = node.callee;
-			if (!isTestCallee(callee)) return;
-			if (testCalleeHasModifier(callee, "todo")) return;
-			const callback = findTestCallback(node.arguments ?? []);
-			if (callback === void 0) return;
-			if (!hasExpect(callback)) context.report({
-				messageId: "noExpect",
-				node
-			});
-		} };
-	},
-	meta: {
-		docs: RULE_DOCS["j3-no-expectless-test"],
-		messages: { noExpect: "Test has no `expect(…)` — a test must assert something (J3 — see docs/10-linting.md). Use `test.todo` for a pending test." },
-		type: "problem"
-	}
-};
-//#endregion
-//#region src/lint/rules/j4-unique-test-names.ts
-/**
-* CONVENTIONS J4 — test names are the sole description of behaviour (B3), so two
-* tests sharing a literal name in one file are a copy-paste smell: one shadows
-* the other in the reporter. Flags the second (and later) occurrence.
-*
-* `.each` / parametrized tests are skipped — their name is a template expanded
-* per row, collisions there are by construction.
-*/
-const j4UniqueTestNames = {
-	create(context) {
-		const seen = /* @__PURE__ */ new Set();
-		return { CallExpression(node) {
-			const callee = node.callee;
-			if (!isTestCallee(callee)) return;
-			if (testCalleeHasModifier(callee, "each")) return;
-			const name = testName(node);
-			if (name === void 0) return;
-			if (seen.has(name)) {
-				context.report({
-					data: { name },
-					messageId: "duplicate",
-					node
-				});
-				return;
-			}
-			seen.add(name);
-		} };
-	},
-	meta: {
-		docs: RULE_DOCS["j4-unique-test-names"],
-		messages: { duplicate: "Duplicate test name \"{{name}}\" in this file — the test name is the sole description of behaviour, so it must be unique (J4 — see docs/10-linting.md)." },
-		type: "problem"
-	}
-};
-//#endregion
-//#region src/lint/rules/j5-lowercase-title.ts
-/** The block/test introducers whose title literal C1' constrains. */
-const TITLED = /* @__PURE__ */ new Set([
-	"describe",
-	"it",
-	"test"
-]);
-/** An identifier-shaped, all-caps/underscored first word: `VALID_CATEGORIES`,
-* `HTTP`, `DI`. Such a word names a symbol verbatim — lowercasing it would
-* misspell it — so it is exempt. A prose word (`Rejects`, `Builds`) is not. */
-const ALL_CAPS_IDENTIFIER = /^[A-Z][A-Z0-9_]*$/u;
-/**
-* CONVENTIONS J5 — the first character of a `test('…')` / `describe('…')` title
-* literal must be lowercase.
-*
-* The test name is the sole description of behaviour (B3): a sentence fragment,
-* lowercase-led like prose (`'rejects an unknown job'`). Exempt are titles
-* whose first WORD is an identifier-shaped all-caps/underscored token
-* (`'VALID_CATEGORIES is rejected'`, `'HTTP is upgraded'`, `'DI wires …'`) —
-* lowercasing a named symbol would misspell it — and titles opening on a
-* non-letter (a `$`, a `{`, a `%s` placeholder). Only lowercase-able prose
-* first words are enforced. Covers the modifier and parametrized forms
-* (`test.only`, `describe.each(...)(...)`) via the chain root.
-*/
-const j5LowercaseTitle = {
-	create(context) {
-		return { CallExpression(node) {
-			const callee = node.callee;
-			const root = chainRootName(callee);
-			if (root === void 0 || !TITLED.has(root)) return;
-			const title = stringValue(node.arguments?.[0]);
-			if (title === void 0) return;
-			const first = [...title][0];
-			if (first === void 0 || !/\p{L}/u.test(first)) return;
-			const firstWord = title.split(/\s+/u)[0];
-			if (ALL_CAPS_IDENTIFIER.test(firstWord)) return;
-			if (first !== first.toLocaleLowerCase()) context.report({
-				data: { runner: root },
-				messageId: "uppercase",
-				node
-			});
-		} };
-	},
-	meta: {
-		docs: RULE_DOCS["j5-lowercase-title"],
-		messages: { uppercase: "A {{runner}}() title must start lowercase — the test name is a prose fragment, not a sentence (J5 — see docs/10-linting.md)." },
-		type: "problem"
-	}
-};
-//#endregion
 //#region src/lint/rules/w1-scenario-pure.ts
 /** The scenario-carrying terminal actions: `.visit()` (website) and `.open()` (mobile). */
 const SCENARIO_ACTIONS$1 = /* @__PURE__ */ new Set(["open", "visit"]);
@@ -3648,7 +3461,7 @@ const w1ScenarioPure = {
 	},
 	meta: {
 		docs: RULE_DOCS["w1-scenario-pure"],
-		messages: { expectInScenario: "No expect() inside a scenario — the scenario is the When; assert the final state on the returned result in the Then (W1 — see docs/10-linting.md)." },
+		messages: { expectInScenario: "No expect() inside a scenario — the scenario is the When; assert the final state on the returned result in the Then (W1 — see docs/13-linting.md)." },
 		type: "problem"
 	}
 };
@@ -3664,7 +3477,7 @@ const SCENARIO_ACTIONS = /* @__PURE__ */ new Set(["open", "visit"]);
 *
 * Registered in a consumer's (or this repo's own) `oxlint.config.ts` via
 * `jsPlugins: ['@jterrazz/test/oxlint']` and referenced as `jterrazz/<rule>` in
-* the `rules` map, e.g. `'jterrazz/j1-no-only-skip': 'error'` — or enabled
+* the `rules` map, e.g. `'jterrazz/j2-no-sleep-in-specs': 'error'` — or enabled
 * wholesale by spreading {@link recommendedRules}.
 *
 * This entry is bundled by tsdown (`dist/oxlint.js`); rules import nothing from
@@ -3714,11 +3527,7 @@ const plugin = {
 		"i1-layer-boundaries": i1LayerBoundaries,
 		"i2-sibling-test-naming": i2SiblingTestNaming,
 		"i4-no-vi-mock-in-src": i4NoViMockInSrc,
-		"j1-no-only-skip": j1NoOnlySkip,
 		"j2-no-sleep-in-specs": j2NoSleepInSpecs,
-		"j3-no-expectless-test": j3NoExpectlessTest,
-		"j4-unique-test-names": j4UniqueTestNames,
-		"j5-lowercase-title": j5LowercaseTitle,
 		"w1-scenario-pure": w1ScenarioPure,
 		"w2w-user-facing-elements": {
 			create(context) {
@@ -3740,7 +3549,7 @@ const plugin = {
 			},
 			meta: {
 				docs: RULE_DOCS["w2w-user-facing-elements"],
-				messages: { testIdElement: "Prefer a user-facing element (button, link, field, heading, content) over testId() — the escape hatch hides what the user actually sees (W2 — see docs/10-linting.md)." },
+				messages: { testIdElement: "Prefer a user-facing element (button, link, field, heading, content) over testId() — the escape hatch hides what the user actually sees (W2 — see docs/13-linting.md)." },
 				type: "suggestion"
 			}
 		}