@godxjp/ui 33.7.1 → 33.7.2

package/agent/START-HERE.md

@@ -3,7 +3,7 @@
 You are about to write code against a design system you did not author. This file is the whole
 contract. Read it before you write JSX.
 
-**This catalog describes `@godxjp/ui` 33.7.1.** If the project you are editing has a different
+**This catalog describes `@godxjp/ui` 33.7.2.** If the project you are editing has a different
 version in its `package.json`, read the pinned catalog for THAT version instead
 (`…/v<their-version>/agent/…`). A catalog newer than the installed package describes props that do
 not exist yet; older, and it hides props that do. Neither failure announces itself.

package/agent/index.json

@@ -54,7 +54,7 @@
   "note": "Pin to the tag that matches the @godxjp/ui version you installed. A catalog newer than your package describes props you do not have; older, and it hides props you do.",
   "read": {
     "live": "https://raw.githubusercontent.com/godx-jp/godxjp-ui/main/agent/index.json",
-    "pinned": "https://raw.githubusercontent.com/godx-jp/godxjp-ui/v33.7.1/agent/index.json"
+    "pinned": "https://raw.githubusercontent.com/godx-jp/godxjp-ui/v33.7.2/agent/index.json"
   },
   "source": "mcp/src/data — the same data @godxjp/ui-mcp serves — plus the foundation and semantic token tiers, read from src/tokens/*.css",
   "start": "https://raw.githubusercontent.com/godx-jp/godxjp-ui/main/agent/START-HERE.md",
@@ -68,5 +68,5 @@
     "foundation": "the seeds a consumer is invited to set — --primary, --background, --radius",
     "semantic": "named roles that follow the seeds — --ring, --text-link, --overlay-background"
   },
-  "version": "33.7.1"
+  "version": "33.7.2"
 }

package/agent/llms.txt

@@ -1,10 +1,10 @@
 # @godxjp/ui
 
 > A Japanese-enterprise React design system: 166 components, 2235 design tokens,
-> 44 cardinal rules. This file is the entry point for AI agents. Catalog version 33.7.1.
+> 44 cardinal rules. This file is the entry point for AI agents. Catalog version 33.7.2.
 
 If your client can run a process, do not read these files — run the MCP server instead
-(`npx @godxjp/ui-mcp@33.7.1`). It is searchable and version-locked. These files exist for agents
+(`npx @godxjp/ui-mcp@33.7.2`). It is searchable and version-locked. These files exist for agents
 that can only fetch URLs.
 
 ## Start
@@ -27,7 +27,7 @@ that can only fetch URLs.
 ## Pinning
 
 Every URL above tracks `main`. To pin to the release a project actually installed, swap `main` for
-the tag: `.../godx-jp/godxjp-ui/v33.7.1/agent/...`. A catalog that does not match the installed
+the tag: `.../godx-jp/godxjp-ui/v33.7.2/agent/...`. A catalog that does not match the installed
 package describes props that are absent, or hides props that are present, and says nothing either way.
 
 Pinned catalogs only exist for releases whose tag actually contains `agent/`. If `…/v<version>/agent/index.json` returns 404, that release predates this catalog: read `…/main/…` instead and compare `index.json` → `version` against the package you have, so you at least know which way it drifted.

package/dist/contracts/measurement.json

@@ -1,6 +1,6 @@
 {
   "$comment": "AUTO-GENERATED by scripts/gen-measurement-contract.mjs — do not edit. Read this instead of guessing: docs/MEASUREMENT-CONTRACT.md.",
-  "version": "33.7.1",
+  "version": "33.7.2",
   "targetSize": {
     "standard": "WCAG 2.2 SC 2.5.8 Target Size (Minimum), level AA — 24×24 CSS px",
     "min": 24,

package/dist/styles/layers.json

@@ -1,5 +1,5 @@
 {
-  "version": "33.7.1",
+  "version": "33.7.2",
   "generatedBy": "scripts/gen-style-layers.mjs — do not edit by hand; run `pnpm gen:style-layers`",
   "base": "base.css",
   "fonts": "fonts.css",

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@godxjp/ui",
-  "version": "33.7.1",
-  "godxUiMcp": "33.7.1",
+  "version": "33.7.2",
+  "godxUiMcp": "33.7.2",
   "license": "Apache-2.0",
   "repository": {
     "type": "git",

package/scripts/ui-audit.mjs

@@ -235,7 +235,8 @@ const PALETTE =
  * Followed by U+FE0F they are asking for emoji presentation on purpose, and then they ARE emoji —
  * so that spelling is still caught.
  */
-const EMOJI = /\p{Extended_Pictographic}\uFE0F|(?=\p{Extended_Pictographic})\p{Emoji_Presentation}/u;
+const EMOJI =
+  /\p{Extended_Pictographic}\uFE0F|(?=\p{Extended_Pictographic})\p{Emoji_Presentation}/u;
 /** Regional-indicator pairs = emoji flags (🇯🇵) — broken on Win/Linux; use Intl.DisplayNames. */
 const EMOJI_FLAG = /\p{Regional_Indicator}/u;
 
@@ -995,7 +996,9 @@ const RULES = [
     scope: "consumer",
     severity: "warn",
     spansElement: true,
-    matches: (source) => densityMatches(source, ["Card"], CARD_LIMIT),
+    // A dashboard's tiles ARE its surfaces — the same path test kpi-row-off-dashboard uses.
+    matches: (source, rel = "") =>
+      /dashboard/i.test(rel) ? [] : densityMatches(source, ["Card"], CARD_LIMIT),
     message: `${CARD_LIMIT}+ Cards in one file — a Card per section. Mark sections with a Heading and section spacing (PageContainer children, Flex gap="lg"); keep Card for a real boundary (a table surface, a selectable item).`,
   },
   {
@@ -1055,7 +1058,8 @@ const RULES = [
     scope: "consumer",
     severity: "warn",
     spansElement: true,
-    matches: (source) => densityMatches(source, ["Alert"], ALERT_LIMIT),
+    // An Alert behind a condition IS a state (what the message asks Alerts to be); count the standing ones.
+    matches: (source) => densityMatches(source, ["Alert"], ALERT_LIMIT, { skipConditional: true }),
     message: `${ALERT_LIMIT}+ Alerts in one file. An Alert is for a state that blocks or changes the task; field errors belong in FormField, results in a toast, and standing info in plain text.`,
   },
 ];
@@ -1810,9 +1814,45 @@ function* mixedButtonSizeMatches(source) {
 }
 
 /** One finding per file, at the `limit`-th occurrence of any tag in `tags`, naming the count. */
-function* densityMatches(source, tags, limit) {
+/**
+ * The JS token before a JSX element, as the expression that places it: `?` / `:` (a ternary
+ * branch), `&&` / `||` (a guard), `=>` (a callback, e.g. `.map`), `return`, or "" when the element
+ * is a plain JSX child. An opening `(` is skipped — `cond ? (\n  <Alert` reads as `?`.
+ */
+function placementBefore(source, index) {
+  let i = index - 1;
+  while (i >= 0 && (/\s/.test(source[i]) || source[i] === "(")) i -= 1;
+  if (source[i] === "?" || source[i] === ":") return source[i];
+  if (source.slice(i - 1, i + 1) === "&&" || source.slice(i - 1, i + 1) === "||") return "&&";
+  if (source.slice(i - 1, i + 1) === "=>") return "=>";
+  if (/\breturn$/.test(source.slice(Math.max(0, i - 5), i + 1))) return "return";
+  return "";
+}
+
+/**
+ * ≥limit elements of `tags` that can be on screen TOGETHER. Mutually exclusive branches of one state
+ * count once: the else branch of a ternary (`a ? <X/> : <X/>`) is the same slot as its `?` branch,
+ * and a function that returns the element from several `if`s returns ONE of them — gino-cloud's
+ * 実習生詳細 had 6 Badge tags and showed at most 3. `skipConditional` drops elements behind a
+ * condition altogether (a guard, a ternary, a `.map`).
+ */
+function* densityMatches(source, tags, limit, { skipConditional = false } = {}) {
   const hits = [];
-  for (const { tag, index } of jsxElements(source)) if (tags.includes(tag)) hits.push(index);
+  const returnedIn = new Set(); // function starts that already returned one of `tags`
+  for (const { tag, index } of jsxElements(source)) {
+    if (!tags.includes(tag)) continue;
+    const how = placementBefore(source, index);
+    if (skipConditional && (how === "?" || how === ":" || how === "&&" || how === "=>")) continue;
+    if (how === ":") continue;
+    if (how === "return" || how === "?") {
+      const head = source.slice(0, index);
+      const arrows = [...head.matchAll(/=>\s*\{/g)];
+      const fn = Math.max(head.lastIndexOf("function "), arrows.at(-1)?.index ?? -1);
+      if (returnedIn.has(fn)) continue;
+      returnedIn.add(fn);
+    }
+    hits.push(index);
+  }
   if (hits.length >= limit) yield { 0: `${hits.length} in this file`, index: hits[limit - 1] };
 }