@starci/eslint-canon-fe 6.0.2 → 7.1.0

package/lib/why.mjs

@@ -46,10 +46,10 @@ export const why = {
     vi: "Client ở `<file>` không có nhánh so sánh `response.status` với 401 và 403 rồi trả `{ kind: \"refused\" }`, nên trạng thái \"cần đăng nhập\" không bao giờ đạt được.",
     fixVi: "Thêm nhánh `response.status === 401 || response.status === 403` trả `{ ok: false, kind: \"refused\" }` ngay trong client.",
   },
-  "no-http-status-collapse": {
+  "no-failure-collapse": {
     code: "FE_HTTP_STATUS_COLLAPSE",
-    vi: "`<file>` gộp mọi mã HTTP thành một nhánh (hoặc trả null khi phản hồi lỗi). 401/403 phải thành `refused`.",
-    fixVi: "Trả `Outcome` theo mã: `refused` (401/403), `not-found`, `invalid`, `unavailable`; không trả nguyên văn lỗi của server làm lý do.",
+    vi: "`<file>` gộp mọi thất bại (phản hồi HTTP lỗi hoặc `Outcome` thất bại) thành một nhánh hoặc một giá trị rỗng, nên lý do bị mất.",
+    fixVi: "Giữ lý do: với `Outcome` thì rẽ nhánh theo mã lý do; với phản hồi HTTP thì đổi mã trạng thái thành `Outcome` (`refused` cho 401/403, `not-found`, `invalid`, `unavailable`). Không dùng nguyên văn lỗi của server làm lý do.",
   },
   "no-hand-typed-wire": {
     code: "FE_WIRE_GENERATED",
@@ -121,71 +121,6 @@ export const why = {
     vi: "`<unit>` tự viết vòng poll (`setInterval` hoặc `setTimeout` tự gọi lại).",
     fixVi: "Làm tươi qua hook dữ liệu (`refreshInterval`) hoặc socket, mỗi resource đúng một cơ chế.",
   },
-  "e2e-spec-location": {
-    code: "FE_E2E_SHAPE",
-    vi: "Spec e2e `<file>` không nằm ở `e2e/<area>/<name>.e2e-spec.ts`.",
-    fixVi: "Đưa spec vào `e2e/<area>/` với hậu tố `.e2e-spec.ts`; helper vào `e2e/support/` hoặc `e2e/fixtures/`.",
-  },
-  "e2e-no-absolute-path": {
-    code: "FE_E2E_SHAPE",
-    vi: "Spec e2e `<file>` dùng đường dẫn tuyệt đối.",
-    fixVi: "Dựng đường dẫn từ cấu hình hoặc thư mục của chính spec.",
-  },
-  "e2e-no-docker": {
-    code: "FE_E2E_SHAPE",
-    vi: "Spec e2e `<file>` phụ thuộc vào docker.",
-    fixVi: "Trỏ suite tới một URL từ cấu hình; môi trường tự cung cấp stack.",
-  },
-  "e2e-no-cross-repo-write": {
-    code: "FE_E2E_SHAPE",
-    vi: "Spec e2e `<file>` ghi ra ngoài repo của app (đường dẫn tuyệt đối hoặc `..`).",
-    fixVi: "Chỉ ghi vào thư mục kết quả của chính repo này.",
-  },
-  "e2e-no-skip": {
-    code: "FE_E2E_SHAPE",
-    vi: "Spec e2e `<file>` bỏ qua test (`test.skip`/`fixme`) — thiếu môi trường phải làm suite đỏ, không được xanh.",
-    fixVi: "Sửa môi trường hoặc xóa test; không bỏ qua có điều kiện.",
-  },
-  "e2e-typed-helpers": {
-    code: "FE_E2E_SHAPE",
-    vi: "Helper e2e ở `<file>` có tham số không kiểu hoặc dùng `any`.",
-    fixVi: "Khai kiểu cho mọi tham số của helper; thay `any` bằng kiểu thật hoặc `unknown` kèm kiểm tra.",
-  },
-  "playwright-viewports": {
-    code: "FE_E2E_SHAPE",
-    vi: "`playwright.config` không khai đúng ba viewport 1440x900, 768x1024, 390x844.",
-    fixVi: "Mỗi project khai một viewport; đủ ba, không thêm cỡ thứ tư.",
-  },
-  "no-class-string-in-spec": {
-    code: "FE_SPEC_QUALITY",
-    vi: "Spec `<file>` ghim chuỗi class. Spec kiểm hành vi, không kiểm stylesheet.",
-    fixVi: "Assert theo role, tên, trạng thái hoặc chữ hiển thị.",
-  },
-  "spec-tests-its-neighbour": {
-    code: "FE_SPEC_QUALITY",
-    vi: "Spec `<file>` không import chủ thể nằm cạnh nó.",
-    fixVi: "Đưa spec về cạnh chủ thể, hoặc test đúng đơn vị nằm cạnh.",
-  },
-  "no-barrel-spec": {
-    code: "FE_SPEC_QUALITY",
-    vi: "Spec `<file>` đặt cạnh một `index` chỉ có import và re-export (barrel). Barrel không có hành vi để test.",
-    fixVi: "Xóa spec; test từng đơn vị mà barrel xuất lại, cạnh chính đơn vị đó.",
-  },
-  "no-double-cast-in-spec": {
-    code: "FE_SPEC_QUALITY",
-    vi: "Spec `<file>` ép kiểu hai lần (`as unknown as`).",
-    fixVi: "Dựng giá trị bằng factory có kiểu (`mock<T>()` hoặc fixture builder).",
-  },
-  "connected-spec-has-axe": {
-    code: "FE_SPEC_QUALITY",
-    vi: "Spec `<file>` của màn hình kết nối không có assert axe.",
-    fixVi: "Thêm `expect(await axe(container)).toHaveNoViolations()`.",
-  },
-  "no-mocked-translations": {
-    code: "FE_I18N_CATALOG",
-    vi: "Spec `<file>` giả lập `next-intl`, nên chỉ kiểm khóa chứ không kiểm catalog thật.",
-    fixVi: "Component client: render trong `NextIntlClientProvider` với `messages/<locale>.json` thật. Helper server (`next-intl/server`): factory của mock dựng translator từ catalog thật (`createTranslator` với `modules/i18n/messages/<locale>.json`).",
-  },
   "no-inline-lint-config": {
     code: "HFS_INLINE_SUPPRESSION",
     vi: "Có chú thích tắt luật ở `<file>:<line>`. HFS không cho tắt tại chỗ — sửa code, hoặc đề xuất đổi luật.",

package/lint-escape-hatch.mjs

@@ -19,10 +19,10 @@
  * nothing is dead weight that reads as if it did.
  */
 
-import { fileOf, isE2eSource, isProductSource, isSpecFile } from "./lib/scope.mjs"
+import { isProductSource } from "./lib/scope.mjs"
 
-/** Product source, its specs and the e2e tree are governed; canon tests deliberately build forbidden directives. */
-const isGoverned = (context) => isProductSource(context) || isSpecFile(fileOf(context)) || isE2eSource(context)
+/** Product source is governed; canon tests deliberately build forbidden directives. */
+const isGoverned = (context) => isProductSource(context)
 
 /**
  * Any directive that changes ESLint's active rule set inside a source file.

package/lists.mjs

@@ -16,7 +16,6 @@
  */
 
 import { attribute, attributeValue, calleeName, isFunction, mentions } from "./lib/ast.mjs"
-import { isSpecFile } from "./lib/scope.mjs"
 
 /** `.map(callback)` where the callback is written inline; the call node, else null. */
 const mapCallback = (call) => {
@@ -86,7 +85,6 @@ export const listItemHasKey = {
     },
   },
   create(context) {
-    if (isSpecFile(context.filename || context.getFilename())) return {}
     return {
       CallExpression(call) {
         const callback = mapCallback(call)
@@ -116,7 +114,6 @@ export const noIndexKey = {
     },
   },
   create(context) {
-    if (isSpecFile(context.filename || context.getFilename())) return {}
     const source = context.sourceCode ?? context.getSourceCode()
     return {
       CallExpression(call) {
@@ -170,7 +167,6 @@ export const noInlineLiteralPropInList = {
     },
   },
   create(context) {
-    if (isSpecFile(context.filename || context.getFilename())) return {}
     return {
       JSXAttribute(node) {
         const opening = node.parent

package/loading.mjs

@@ -12,10 +12,10 @@
  * lint rule is for - the one nothing else will ever report.
  */
 
-import { classOf, fileOf, isComponentFile, isSpecFile } from "./lib/scope.mjs"
+import { classOf, fileOf, isComponentFile } from "./lib/scope.mjs"
 
 /** Product component source these rules govern: a component owner's file, and not a spec (a placeholder in a test fixture is a fixture, not a second tree). */
-const isGoverned = (context) => isComponentFile(context) && !isSpecFile(fileOf(context))
+const isGoverned = (context) => isComponentFile(context)
 
 /** A flag that means "waiting", in the spellings this codebase uses. */
 const WAITING_FLAG = /\bis(?:Loading|Skeleton|Pending)\b/

package/native-controls.mjs

@@ -13,7 +13,6 @@
  */
 
 import { attribute } from "./lib/ast.mjs"
-import { isSpecFile } from "./lib/scope.mjs"
 
 /** Intrinsic elements a reader operates, that the grammar renders for the product. */
 const NATIVE = new Set(["select", "input", "textarea", "button"])
@@ -30,7 +29,6 @@ export const noNativeFormControl = {
     },
   },
   create(context) {
-    if (isSpecFile(context.filename || context.getFilename())) return {}
     return {
       JSXOpeningElement(node) {
         if (node.name.type !== "JSXIdentifier" || !NATIVE.has(node.name.name)) return
@@ -54,7 +52,6 @@ export const noNativeImg = {
     },
   },
   create(context) {
-    if (isSpecFile(context.filename || context.getFilename())) return {}
     return {
       JSXOpeningElement(node) {
         if (node.name.type === "JSXIdentifier" && node.name.name === "img") context.report({ node, messageId: "img" })
@@ -79,7 +76,6 @@ export const imageHasSize = {
     },
   },
   create(context) {
-    if (isSpecFile(context.filename || context.getFilename())) return {}
     const locals = new Set()
     return {
       ImportDeclaration(node) {

package/next-conventions.mjs

@@ -13,7 +13,7 @@
 
 import { attribute, attributeValue, calleeName, leadingTextOf, stringOf } from "./lib/ast.mjs"
 import { hfsOf } from "./lib/hfs.mjs"
-import { baseName, classOf, fileOf, inSlot, isSpecFile, roleOfFile, slotOfFile, stem } from "./lib/scope.mjs"
+import { baseName, classOf, fileOf, inSlot, roleOfFile, slotOfFile, stem } from "./lib/scope.mjs"
 
 /** Another i18n stack: a second place that decides which language a reader sees. */
 const OTHER_I18N = /^(?:react-i18next|i18next|next-i18next|react-intl|next-translate|@lingui\/.+|typesafe-i18n|rosetta)$/
@@ -35,7 +35,6 @@ export const noMiddlewareFile = {
   },
   create(context) {
     const file = fileOf(context)
-    if (isSpecFile(file)) return {}
     // The retired name has a slot of its own (`fe.source-root-retired`, forbidden); the interceptor is the `proxy` role of `fe.source-root-pinned`.
     if (inSlot(context, "fe.source-root-retired")) {
       return { Program: (node) => context.report({ node, messageId: "file", data: { name: baseName(file) } }) }
@@ -189,7 +188,6 @@ export const noNullSuspenseFallback = {
     },
   },
   create(context) {
-    if (isSpecFile(context.filename || context.getFilename())) return {}
     return {
       JSXOpeningElement(node) {
         const name = node.name
@@ -235,7 +233,7 @@ export const navigationFromIntl = {
     },
   },
   create(context) {
-    if (isSpecFile(fileOf(context)) || isNavigationOwner(context)) return {}
+    if (isNavigationOwner(context)) return {}
     return {
       ImportDeclaration(node) {
         if (node.importKind === "type") return
@@ -271,7 +269,6 @@ export const noNativeAnchor = {
     },
   },
   create(context) {
-    if (isSpecFile(context.filename || context.getFilename())) return {}
     return {
       JSXOpeningElement(node) {
         if (node.name.type !== "JSXIdentifier" || node.name.name !== "a") return
@@ -309,7 +306,7 @@ export const noHardcodedRoute = {
     },
   },
   create(context) {
-    if (isSpecFile(fileOf(context)) || isRoutesModule(context)) return {}
+    if (isRoutesModule(context)) return {}
     const source = context.sourceCode ?? context.getSourceCode()
     const report = (node, text) => context.report({ node, messageId: "route", data: { path: text } })
     return {
@@ -393,7 +390,6 @@ export const i18nStackInOneModule = {
     },
   },
   create(context) {
-    if (isSpecFile(context.filename || context.getFilename())) return {}
     const hfs = hfsOf(context)
     const slot = slotOfFile(context)
     const single = hfs.apps.length === 1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@starci/eslint-canon-fe",
-  "version": "6.0.2",
+  "version": "7.1.0",
   "description": "The StarCi front-end lint canon: every rule its laws publish, gathered as one flat-config plugin and attached by one factory.",
   "type": "module",
   "license": "UNLICENSED",

package/project-graph.mjs

@@ -0,0 +1,23 @@
+/**
+ * The rules that read the project graph: the findings of the architecture machine that sit on a TypeScript file.
+ *
+ * One graph per lint run: `scripts/lib/project-graph.mjs` (a byte copy in this package's runtime/) builds the machine's module
+ * graph and slot manifest once for the repository, caches it, and each rule below reports the findings of its own codes on
+ * the file ESLint is visiting, on the line the machine names. There is no second graph and no second judge: the list of
+ * rules and their codes is `LINT_ENFORCERS` of the machine (scripts/checks/architecture/surface.mjs); `hfs check` keeps the
+ * findings that have no TypeScript file to sit on. A graph that cannot be built stops the run with the machine's message.
+ */
+import { LINT_ENFORCERS } from "./runtime/scripts/checks/architecture/surface.mjs"
+import { projectRule } from "./runtime/scripts/lib/project-rule.mjs"
+import { hfsOf } from "./lib/hfs.mjs"
+import { dirname, join } from "node:path"
+import { fileURLToPath } from "node:url"
+
+const RUNTIME = join(dirname(fileURLToPath(import.meta.url)), "runtime")
+const mine = LINT_ENFORCERS.filter((enforcer) => enforcer.plugins.includes("fe"))
+
+/** The rules this law contributes to the plugin: one per graph enforcer of this profile. */
+export const rules = Object.fromEntries(mine.map((e) => [e.id, projectRule({ enforcer: e, runtimeRoot: RUNTIME, hfsOf })]))
+
+/** Every one at error. */
+export const recommended = Object.fromEntries(mine.map((e) => [`starci-fe/${e.id}`, "error"]))

package/props-and-slots.mjs

@@ -1,7 +1,7 @@
 /** Rules for named React props, ordinary children, and local visual ownership. */
 
 import { declarationsOf, fileOf as declarationFile } from "./lib/types.mjs"
-import { fileOf, isComponentFile, isComponentPath, isProductSource, isSpecFile, kindOfFile } from "./lib/scope.mjs"
+import { fileOf, isComponentFile, isComponentPath, isProductSource, kindOfFile } from "./lib/scope.mjs"
 
 const propertyName = (node) => {
   const key = node?.key ?? node?.property
@@ -70,7 +70,7 @@ export const publicComponentSignature = {
       const actual = type?.type === "TSTypeReference" && type.typeName?.type === "Identifier" ? type.typeName.name : null
       if (!actual || !expected.includes(actual)) context.report({ node: params[0], messageId: "type", data: { name, expected: expected.join(" or ") } })
     }
-    if (!isComponentFile(context) || isSpecFile(fileOf(context))) return {}
+    if (!isComponentFile(context)) return {}
     return {
       VariableDeclarator(node) {
         const name = componentNameOf(node)
@@ -93,7 +93,7 @@ export const noPerPartClassNameProp = {
     messages: { perPart: "Keep internal part styling owned by the component." },
   },
   create(context) {
-    if (!isComponentFile(context) || isSpecFile(fileOf(context))) return {}
+    if (!isComponentFile(context)) return {}
     return { TSPropertySignature(node) { const name = propertyName(node); if (name && name !== "className" && /^[a-z][A-Za-z0-9]*ClassName$/.test(name)) context.report({ node, messageId: "perPart" }) } }
   },
 }
@@ -107,7 +107,6 @@ export const noPublicClassNameProp = {
     messages: { declaration: "Component props must not expose {{prop}}.", usage: "Do not pass {{prop}} to house component {{component}}." },
   },
   create(context) {
-    if (isSpecFile(fileOf(context))) return {}
     const declared = isComponentFile(context)
     /** A house component: the identifier resolves (through its import) to a file of a component layer. */
     const isHouseComponent = (identifier) => declarationsOf(context, identifier).some((declaration) => isComponentPath(context, declarationFile(declaration)))
@@ -128,7 +127,7 @@ export const noPublicClassNameProp = {
 export const noPublicFrameCssProps = {
   meta: { type: "problem", docs: { description: "Non-leaf component props do not expose CSS-shaped frame decisions." }, schema: [], messages: { css: "Move {{prop}} into component-owned layout behavior." } },
   create(context) {
-    if (isSpecFile(fileOf(context)) || !isComponentFile(context) || kindOfFile(context) === "leaves") return {}
+    if (!isComponentFile(context) || kindOfFile(context) === "leaves") return {}
     const cssProps = new Set(["gap", "padding", "align", "justify", "className", "classNames", "style", "inline", "nested"])
     return { TSPropertySignature(node) { const name = propertyName(node); if (cssProps.has(name)) context.report({ node, messageId: "css", data: { prop: name } }) } }
   },