@godxjp/ui 16.9.2 → 16.9.4

package/dist/styles/control.css

@@ -198,6 +198,22 @@
     padding: 0;
   }
 
+  /* Icon-only sizes must be square at the SAME height as their text-size counterpart, so an
+   * icon button can sit flush beside a labelled one (split buttons, toolbars). Derive them from
+   * the density tokens — never from a hand-written offset off the base --control-height, which
+   * is how icon-sm drifted a step down to the xs height (issue #133). */
+  .ui-button--icon-sm {
+    width: var(--control-height-sm);
+    height: var(--control-height-sm);
+    padding: 0;
+  }
+
+  .ui-button--icon-lg {
+    width: var(--control-height-lg);
+    height: var(--control-height-lg);
+    padding: 0;
+  }
+
   .ui-control {
     height: var(--control-height);
     padding-inline-start: var(--control-padding-x);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@godxjp/ui",
-  "version": "16.9.2",
+  "version": "16.9.4",
   "type": "module",
   "packageManager": "pnpm@10.29.1",
   "sideEffects": false,
@@ -294,7 +294,7 @@
     "docs:sync-primitives": "node preview/scripts/sync-primitive-docs.mjs && prettier --write \"docs/primitives/**/*.tsx\"",
     "audit": "node scripts/ui-audit.mjs",
     "audit:examples": "node scripts/ui-audit.mjs docs",
-    "build": "tsup && tsc -p tsconfig.build.json && node scripts/copy-styles.mjs && node scripts/add-use-client.mjs",
+    "build": "tsup && tsc -p tsconfig.build.json && node scripts/copy-styles.mjs && node scripts/fix-esm-extensions.mjs && node scripts/add-use-client.mjs",
     "dev": "node scripts/dev.mjs",
     "prepublishOnly": "pnpm run build",
     "check:audit-sync": "node scripts/check-audit-sync.mjs",

package/scripts/ui-audit.mjs

@@ -259,7 +259,12 @@ const RULES = [
   {
     id: "raw-intl-date",
     severity: "warn",
-    test: /\.toLocale(?:Date|Time)?String\(\s*\)|new Date\([^)]*\)\.(?:getMonth|getDate|getFullYear)\(\)\s*\+/,
+    // Only the unambiguously-Date methods: `.toLocaleDateString()` / `.toLocaleTimeString()`.
+    // Bare `.toLocaleString()` is excluded on purpose — it is overwhelmingly
+    // `Number.prototype.toLocaleString()` (legit locale number formatting), and matching it
+    // false-flagged every formatted count/amount. A date+time `Date.toLocaleString()` is rare and
+    // still partly covered by the hand-built `new Date(...).getMonth()+` alternative below.
+    test: /\.toLocale(?:Date|Time)String\(\s*\)|new Date\([^)]*\)\.(?:getMonth|getDate|getFullYear)\(\)\s*\+/,
     standard: "ISO 8601 · IANA tz database · ECMA-402 Intl.DateTimeFormat",
     message:
       "Don't hand-build or locale-default dates. Use formatDate from @godxjp/ui/datetime (Intl.DateTimeFormat + IANA timezone + ISO-8601), which respects the AppProvider locale/timezone.",
@@ -404,7 +409,7 @@ function walk(dir, acc = []) {
 // sub-component sits FLUSH (no padding). Per-line regexes can't see across lines, so this is a
 // whole-file pass. The body must be wrapped in <CardContent> (titles in <CardHeader>).
 const CARD_FLUSH =
-  /<Card(?![^>]*\bp-0\b)(?:\s[^>]*)?>\s*<(?!CardContent|CardHeader|CardCover|CardFooter|\/Card)/g;
+  /<Card(?![^>]*\bp-0\b)(?:\s[^>]*)?>\s*<(?!CardContent|CardHeader|CardCover|CardFooter|CardBar|\/Card)/g;
 
 // Structural: a bare <label>/<Label> paired with a TEXT control (its sibling) instead of a
 // <FormField>. FormField OWNS the label↔control association (htmlFor/id), aria-describedby/