@multiplatform.one/config 7.5.0 → 7.6.0

package/lint.mjs

@@ -399,10 +399,24 @@ function readThemeKeys() {
   return keys;
 }
 
-/** Font-size steps the house fonts register (`defaults/fonts.ts`). */
+/**
+ * Font-size steps the house fonts register. MPO-19 X8/X9 retired the
+ * hand-mirrored `interBaseSizes` table — `defaults/fonts.ts` now READS its
+ * size tables from `@tamagui/config/v5`'s `defaultConfig.fonts`, so this
+ * registry reads the same source (the era pin is load-bearing). The source
+ * parse survives as a fallback for consumer trees that still carry the
+ * transcribed table.
+ */
 function readFontSizeSteps() {
   /** @type {Set<string>} */
   const steps = new Set();
+  const cfg = tryRequire("@tamagui/config/v5");
+  for (const font of Object.values(cfg?.defaultConfig?.fonts ?? {})) {
+    // Bare keys (`1`…`16`, `true`) — checkStyleTokens compares the captured
+    // token name without its `$`, same shape keysOfBlock returned.
+    for (const k of Object.keys(font?.size ?? {})) steps.add(k);
+  }
+  if (steps.size) return steps;
   const src = readPublicSource("theme/src/theme/defaults/fonts.ts");
   if (!src) return steps;
   for (const k of keysOfBlock(sliceBlock(src, "const interBaseSizes", "\n};"), 2)) steps.add(k);
@@ -566,7 +580,11 @@ function checkAnimationProps() {
     // `ctx.knobProps.transition` are caught too. Deliberately NOT "any value":
     // that also matches TS annotations (`animation: any`), css-in-js, and
     // unrelated config objects that have nothing to do with the tamagui prop.
-    for (const m of src.matchAll(/\banimation=\{|\banimation:\s*[\w$.[\]?]*\btransition\b/g)) {
+    // The lookbehind keeps `data-animation={…}` (probe attributes: Spinner,
+    // Pagination) out: `\b` alone matches between `-` and `animation`.
+    for (const m of src.matchAll(
+      /(?<![\w-])animation=\{|(?<![\w-])animation:\s*[\w$.[\]?]*\btransition\b/g,
+    )) {
       const line = src.slice(0, m.index).split("\n").length;
       legacyProp.push(`${rel}:${line} — \`${m[0]}\` (tamagui 1.x prop; use \`transition\`)`);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@multiplatform.one/config",
-  "version": "7.5.0",
+  "version": "7.6.0",
   "description": "Shared build and test configuration presets for multiplatform.one",
   "keywords": [
     "config",
@@ -100,7 +100,7 @@
     "vite-plugin-external": "^6.2.2",
     "vite-plugin-i18next-loader": "^3.1.3",
     "vitest": "^4.1.5",
-    "@multiplatform.one/utils": "7.5.0"
+    "@multiplatform.one/utils": "7.6.0"
   },
   "devDependencies": {
     "tsdown": "^0.21.10",

package/src/oxlint.spec.ts

@@ -59,3 +59,79 @@ describe("MPO-17 export surface", () => {
     expect(plugin.rules?.["no-chrome-props"]).toBeUndefined();
   });
 });
+
+/**
+ * AN-8 behavioural teeth (MPO-20 AC-4). The block above only asserts the rule
+ * is DEFINED — a rule that never reports would pass it. These cases drive the
+ * rule's own visitor and assert it reports on every raw typography hatch and
+ * stays silent on the house names, so "the lint fails on raw typography in app
+ * code" is a checked claim rather than a registration.
+ */
+describe("mpo-conventions/no-raw-typography (AN-8)", () => {
+  const rule = plugin.rules?.["no-raw-typography"];
+
+  type Report = { data?: { name?: string } };
+  type ImportVisitor = { ImportDeclaration: (node: unknown) => void };
+  type Creatable = { create: (context: unknown) => ImportVisitor };
+
+  /** Minimal ESTree ImportDeclaration — the only node shape the rule visits. */
+  function importDeclaration(source: string, names: string[]) {
+    return {
+      type: "ImportDeclaration",
+      source: { value: source },
+      specifiers: names.map((name) => ({ type: "ImportSpecifier", imported: { name } })),
+    };
+  }
+
+  function reportedNames(source: string, names: string[]): string[] {
+    const reports: Report[] = [];
+    const context = {
+      filename: "apps/one/app/index.tsx",
+      report: (report: Report) => reports.push(report),
+    };
+    const visitors = (rule as unknown as Creatable).create(context);
+    visitors.ImportDeclaration(importDeclaration(source, names));
+    return reports.map((report) => report.data?.name ?? "");
+  }
+
+  const hatches = [
+    "TamaguiText",
+    "TamaguiSizableText",
+    "TamaguiParagraph",
+    "TamaguiHeading",
+    "TamaguiH1",
+    "TamaguiH6",
+    "TamaguiAnchor",
+  ];
+
+  it.each(hatches)("reports %s imported from the components barrel", (name) => {
+    expect(reportedNames("@multiplatform.one/components", [name])).toEqual([name]);
+  });
+
+  it("stays silent on the house shadows — the bare names ARE the fix", () => {
+    expect(
+      reportedNames("@multiplatform.one/components", [
+        "Text",
+        "SizableText",
+        "Paragraph",
+        "Heading",
+        "H1",
+        "H6",
+        "Anchor",
+      ]),
+    ).toEqual([]);
+  });
+
+  it("stays silent on non-typography hatches and on other packages", () => {
+    expect(
+      reportedNames("@multiplatform.one/components", ["TamaguiButton", "TamaguiSeparator"]),
+    ).toEqual([]);
+    expect(reportedNames("some-other-package", ["TamaguiText"])).toEqual([]);
+  });
+
+  it("reports every offending specifier in one import, not just the first", () => {
+    expect(
+      reportedNames("@multiplatform.one/components", ["Text", "TamaguiText", "TamaguiHeading"]),
+    ).toEqual(["TamaguiText", "TamaguiHeading"]);
+  });
+});