@portnet/ui 6.0.0 → 6.0.1

package/dist/v2/utils/mergeLabels.d.ts

@@ -0,0 +1,59 @@
+/**
+ * mergeLabels — Portnet UI v2
+ *
+ * Merges a consumer's partial label overrides onto a component's frozen
+ * defaults, IGNORING keys whose value is `undefined`.
+ *
+ * ─────────────────────────────────────────────────────────────────────────────
+ * WHY THIS EXISTS (capability gap G-14, latent runtime defect)
+ *
+ * Every localisable component in this library merged its labels like this:
+ *
+ *     const L = labels ? { ...DEFAULT_LABELS, ...labels } : DEFAULT_LABELS;
+ *
+ * A spread does NOT skip explicit `undefined`. `{ ...{a: 1}, ...{a: undefined} }`
+ * is `{ a: undefined }` — so a consumer writing
+ *
+ *     <FilterTab labels={{ countLabel: maybeFn }} />   // maybeFn possibly undefined
+ *
+ * silently DELETED the default and the component then threw
+ * `L.countLabel is not a function` at render time. Eleven components shipped
+ * that hazard.
+ *
+ * It stayed invisible because the label interfaces declared `?: T` rather than
+ * `?: T | undefined`: under that declaration TypeScript concluded the merged
+ * value could never be `undefined`, which is false at runtime. Widening the
+ * optional props (G-14) removed the blindfold and the compiler reported all
+ * seventeen call sites at once. The types were wrong; the code was wrong; only
+ * the combination of the two looked right.
+ *
+ * The fix restores what every call site already intended: an override REPLACES a
+ * default, an absent or `undefined` override LEAVES it alone.
+ * ─────────────────────────────────────────────────────────────────────────────
+ *
+ *     const L = mergeLabels(DEFAULT_LABELS, labels);
+ *     //    ^ every key guaranteed present, typed from the defaults
+ *
+ * The return type is the DEFAULTS' type, deliberately: defaults are the
+ * exhaustive, all-required shape, so the component can call `L.anything()`
+ * without a null check. That is the whole point.
+ *
+ * `overrides` is a mapped type over `keyof TDefaults` with `unknown` values
+ * rather than `Partial<TDefaults>`. Two reasons, both load-bearing:
+ *   - a mapped type accepts an INTERFACE argument, whereas `Record<string,
+ *     unknown>` would not (TypeScript grants implicit index signatures to type
+ *     aliases, not to interfaces), and most label contracts here are interfaces;
+ *   - some label interfaces legitimately widen a default's type (`React.ReactNode`
+ *     where the default is a `string`), which `Partial<TDefaults>` would reject.
+ *     Value types are already checked at the component's `labels` prop boundary,
+ *     so nothing is lost by not re-checking them here.
+ *
+ * Returns the defaults object ITSELF when there is nothing to override, so the
+ * common path allocates nothing and stays referentially stable across renders —
+ * which matters because these results are routinely wrapped in `useMemo`.
+ */
+export declare function mergeLabels<TDefaults extends object>(defaults: TDefaults, overrides?: {
+    readonly [K in keyof TDefaults]?: unknown;
+} | undefined): TDefaults;
+export default mergeLabels;
+//# sourceMappingURL=mergeLabels.d.ts.map

package/dist/v2/utils/mergeLabels.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"mergeLabels.d.ts","sourceRoot":"","sources":["../../../src/lib/v2/utils/mergeLabels.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAqDG;AACH,wBAAgB,WAAW,CAAC,SAAS,SAAS,MAAM,EAClD,QAAQ,EAAE,SAAS,EACnB,SAAS,CAAC,EAAE;IAAE,QAAQ,EAAE,CAAC,IAAI,MAAM,SAAS,CAAC,CAAC,EAAE,OAAO;CAAE,GAAG,SAAS,GACpE,SAAS,CAyBX;AAED,eAAe,WAAW,CAAC"}

package/dist/v2/utils/mergeLabels.js

@@ -0,0 +1,93 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports["default"] = void 0;
+exports.mergeLabels = mergeLabels;
+function _typeof(obj) { "@babel/helpers - typeof"; return _typeof = "function" == typeof Symbol && "symbol" == typeof Symbol.iterator ? function (obj) { return typeof obj; } : function (obj) { return obj && "function" == typeof Symbol && obj.constructor === Symbol && obj !== Symbol.prototype ? "symbol" : typeof obj; }, _typeof(obj); }
+function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (Object.getOwnPropertySymbols) { var symbols = Object.getOwnPropertySymbols(object); enumerableOnly && (symbols = symbols.filter(function (sym) { return Object.getOwnPropertyDescriptor(object, sym).enumerable; })), keys.push.apply(keys, symbols); } return keys; }
+function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? ownKeys(Object(source), !0).forEach(function (key) { _defineProperty(target, key, source[key]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)) : ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } return target; }
+function _defineProperty(obj, key, value) { key = _toPropertyKey(key); if (key in obj) { Object.defineProperty(obj, key, { value: value, enumerable: true, configurable: true, writable: true }); } else { obj[key] = value; } return obj; }
+function _toPropertyKey(arg) { var key = _toPrimitive(arg, "string"); return _typeof(key) === "symbol" ? key : String(key); }
+function _toPrimitive(input, hint) { if (_typeof(input) !== "object" || input === null) return input; var prim = input[Symbol.toPrimitive]; if (prim !== undefined) { var res = prim.call(input, hint || "default"); if (_typeof(res) !== "object") return res; throw new TypeError("@@toPrimitive must return a primitive value."); } return (hint === "string" ? String : Number)(input); }
+/**
+ * mergeLabels — Portnet UI v2
+ *
+ * Merges a consumer's partial label overrides onto a component's frozen
+ * defaults, IGNORING keys whose value is `undefined`.
+ *
+ * ─────────────────────────────────────────────────────────────────────────────
+ * WHY THIS EXISTS (capability gap G-14, latent runtime defect)
+ *
+ * Every localisable component in this library merged its labels like this:
+ *
+ *     const L = labels ? { ...DEFAULT_LABELS, ...labels } : DEFAULT_LABELS;
+ *
+ * A spread does NOT skip explicit `undefined`. `{ ...{a: 1}, ...{a: undefined} }`
+ * is `{ a: undefined }` — so a consumer writing
+ *
+ *     <FilterTab labels={{ countLabel: maybeFn }} />   // maybeFn possibly undefined
+ *
+ * silently DELETED the default and the component then threw
+ * `L.countLabel is not a function` at render time. Eleven components shipped
+ * that hazard.
+ *
+ * It stayed invisible because the label interfaces declared `?: T` rather than
+ * `?: T | undefined`: under that declaration TypeScript concluded the merged
+ * value could never be `undefined`, which is false at runtime. Widening the
+ * optional props (G-14) removed the blindfold and the compiler reported all
+ * seventeen call sites at once. The types were wrong; the code was wrong; only
+ * the combination of the two looked right.
+ *
+ * The fix restores what every call site already intended: an override REPLACES a
+ * default, an absent or `undefined` override LEAVES it alone.
+ * ─────────────────────────────────────────────────────────────────────────────
+ *
+ *     const L = mergeLabels(DEFAULT_LABELS, labels);
+ *     //    ^ every key guaranteed present, typed from the defaults
+ *
+ * The return type is the DEFAULTS' type, deliberately: defaults are the
+ * exhaustive, all-required shape, so the component can call `L.anything()`
+ * without a null check. That is the whole point.
+ *
+ * `overrides` is a mapped type over `keyof TDefaults` with `unknown` values
+ * rather than `Partial<TDefaults>`. Two reasons, both load-bearing:
+ *   - a mapped type accepts an INTERFACE argument, whereas `Record<string,
+ *     unknown>` would not (TypeScript grants implicit index signatures to type
+ *     aliases, not to interfaces), and most label contracts here are interfaces;
+ *   - some label interfaces legitimately widen a default's type (`React.ReactNode`
+ *     where the default is a `string`), which `Partial<TDefaults>` would reject.
+ *     Value types are already checked at the component's `labels` prop boundary,
+ *     so nothing is lost by not re-checking them here.
+ *
+ * Returns the defaults object ITSELF when there is nothing to override, so the
+ * common path allocates nothing and stays referentially stable across renders —
+ * which matters because these results are routinely wrapped in `useMemo`.
+ */
+function mergeLabels(defaults, overrides) {
+  if (!overrides) return defaults;
+
+  // Single documented assertion, guarded by the `!overrides` check above: the
+  // mapped parameter type deliberately says nothing about value types, so the
+  // only way to read them is through an index signature.
+  var source = overrides;
+  var out;
+  for (var _i = 0, _Object$keys = Object.keys(source); _i < _Object$keys.length; _i++) {
+    var key = _Object$keys[_i];
+    var value = source[key];
+    // THE point of this helper. `undefined` means "not specified", never
+    // "unset the default".
+    if (value === undefined) continue;
+    // Second assertion, same cause: `TDefaults extends object` carries no index
+    // signature, so spreading it into a writable record needs one. Allocated
+    // LAZILY — only once a real override is found — so the common paths (no
+    // overrides at all, or an object whose every value is undefined) return the
+    // defaults object itself and stay referentially stable across renders.
+    if (!out) out = _objectSpread({}, defaults);
+    out[key] = value;
+  }
+  return out === undefined ? defaults : out;
+}
+var _default = exports["default"] = mergeLabels;
+//# sourceMappingURL=mergeLabels.js.map

package/dist/v2/utils/mergeLabels.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"mergeLabels.js","names":["mergeLabels","defaults","overrides","source","out","_i","_Object$keys","Object","keys","length","key","value","undefined","_objectSpread","_default","exports"],"sources":["../../../src/lib/v2/utils/mergeLabels.ts"],"sourcesContent":["/**\r\n * mergeLabels — Portnet UI v2\r\n *\r\n * Merges a consumer's partial label overrides onto a component's frozen\r\n * defaults, IGNORING keys whose value is `undefined`.\r\n *\r\n * ─────────────────────────────────────────────────────────────────────────────\r\n * WHY THIS EXISTS (capability gap G-14, latent runtime defect)\r\n *\r\n * Every localisable component in this library merged its labels like this:\r\n *\r\n *     const L = labels ? { ...DEFAULT_LABELS, ...labels } : DEFAULT_LABELS;\r\n *\r\n * A spread does NOT skip explicit `undefined`. `{ ...{a: 1}, ...{a: undefined} }`\r\n * is `{ a: undefined }` — so a consumer writing\r\n *\r\n *     <FilterTab labels={{ countLabel: maybeFn }} />   // maybeFn possibly undefined\r\n *\r\n * silently DELETED the default and the component then threw\r\n * `L.countLabel is not a function` at render time. Eleven components shipped\r\n * that hazard.\r\n *\r\n * It stayed invisible because the label interfaces declared `?: T` rather than\r\n * `?: T | undefined`: under that declaration TypeScript concluded the merged\r\n * value could never be `undefined`, which is false at runtime. Widening the\r\n * optional props (G-14) removed the blindfold and the compiler reported all\r\n * seventeen call sites at once. The types were wrong; the code was wrong; only\r\n * the combination of the two looked right.\r\n *\r\n * The fix restores what every call site already intended: an override REPLACES a\r\n * default, an absent or `undefined` override LEAVES it alone.\r\n * ─────────────────────────────────────────────────────────────────────────────\r\n *\r\n *     const L = mergeLabels(DEFAULT_LABELS, labels);\r\n *     //    ^ every key guaranteed present, typed from the defaults\r\n *\r\n * The return type is the DEFAULTS' type, deliberately: defaults are the\r\n * exhaustive, all-required shape, so the component can call `L.anything()`\r\n * without a null check. That is the whole point.\r\n *\r\n * `overrides` is a mapped type over `keyof TDefaults` with `unknown` values\r\n * rather than `Partial<TDefaults>`. Two reasons, both load-bearing:\r\n *   - a mapped type accepts an INTERFACE argument, whereas `Record<string,\r\n *     unknown>` would not (TypeScript grants implicit index signatures to type\r\n *     aliases, not to interfaces), and most label contracts here are interfaces;\r\n *   - some label interfaces legitimately widen a default's type (`React.ReactNode`\r\n *     where the default is a `string`), which `Partial<TDefaults>` would reject.\r\n *     Value types are already checked at the component's `labels` prop boundary,\r\n *     so nothing is lost by not re-checking them here.\r\n *\r\n * Returns the defaults object ITSELF when there is nothing to override, so the\r\n * common path allocates nothing and stays referentially stable across renders —\r\n * which matters because these results are routinely wrapped in `useMemo`.\r\n */\r\nexport function mergeLabels<TDefaults extends object>(\r\n  defaults: TDefaults,\r\n  overrides?: { readonly [K in keyof TDefaults]?: unknown } | undefined\r\n): TDefaults {\r\n  if (!overrides) return defaults;\r\n\r\n  // Single documented assertion, guarded by the `!overrides` check above: the\r\n  // mapped parameter type deliberately says nothing about value types, so the\r\n  // only way to read them is through an index signature.\r\n  const source = overrides as Record<string, unknown>;\r\n\r\n  let out: Record<string, unknown> | undefined;\r\n\r\n  for (const key of Object.keys(source)) {\r\n    const value = source[key];\r\n    // THE point of this helper. `undefined` means \"not specified\", never\r\n    // \"unset the default\".\r\n    if (value === undefined) continue;\r\n    // Second assertion, same cause: `TDefaults extends object` carries no index\r\n    // signature, so spreading it into a writable record needs one. Allocated\r\n    // LAZILY — only once a real override is found — so the common paths (no\r\n    // overrides at all, or an object whose every value is undefined) return the\r\n    // defaults object itself and stay referentially stable across renders.\r\n    if (!out) out = { ...(defaults as Record<string, unknown>) };\r\n    out[key] = value;\r\n  }\r\n\r\n  return out === undefined ? defaults : (out as TDefaults);\r\n}\r\n\r\nexport default mergeLabels;\r\n"],"mappings":";;;;;;;;;;;;;AAAA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACO,SAASA,WAAWA,CACzBC,QAAmB,EACnBC,SAAqE,EAC1D;EACX,IAAI,CAACA,SAAS,EAAE,OAAOD,QAAQ;;EAE/B;EACA;EACA;EACA,IAAME,MAAM,GAAGD,SAAoC;EAEnD,IAAIE,GAAwC;EAE5C,SAAAC,EAAA,MAAAC,YAAA,GAAkBC,MAAM,CAACC,IAAI,CAACL,MAAM,CAAC,EAAAE,EAAA,GAAAC,YAAA,CAAAG,MAAA,EAAAJ,EAAA,IAAE;IAAlC,IAAMK,GAAG,GAAAJ,YAAA,CAAAD,EAAA;IACZ,IAAMM,KAAK,GAAGR,MAAM,CAACO,GAAG,CAAC;IACzB;IACA;IACA,IAAIC,KAAK,KAAKC,SAAS,EAAE;IACzB;IACA;IACA;IACA;IACA;IACA,IAAI,CAACR,GAAG,EAAEA,GAAG,GAAAS,aAAA,KAASZ,QAAQ,CAA8B;IAC5DG,GAAG,CAACM,GAAG,CAAC,GAAGC,KAAK;EAClB;EAEA,OAAOP,GAAG,KAAKQ,SAAS,GAAGX,QAAQ,GAAIG,GAAiB;AAC1D;AAAC,IAAAU,QAAA,GAAAC,OAAA,cAEcf,WAAW","ignoreList":[]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@portnet/ui",
-  "version": "6.0.0",
+  "version": "6.0.1",
   "description": "Portnet UI — Design System v1.0 components (MUI v9 baseline, React 19 ready). v1 Pui* legacy removed in 0.3.0.",
   "keywords": [
     "react",
@@ -12,11 +12,14 @@
     "mui9"
   ],
   "main": "dist/index.js",
+  "module": "dist/esm/index.js",
   "types": "dist/index.d.ts",
   "sideEffects": [
     "*.css",
     "./dist/v2/styles/index.js",
-    "./dist/v2/styles/base.css"
+    "./dist/v2/styles/base.css",
+    "./dist/esm/v2/styles/index.js",
+    "./dist/esm/v2/styles/base.css"
   ],
   "engines": {
     "node": ">=18.0.0",
@@ -25,73 +28,73 @@
   "exports": {
     ".": {
       "types": "./dist/index.d.ts",
-      "import": "./dist/index.js",
+      "import": "./dist/esm/index.js",
       "require": "./dist/index.js",
       "default": "./dist/index.js"
     },
     "./theme": {
       "types": "./dist/v2/theme/index.d.ts",
-      "import": "./dist/v2/theme/index.js",
+      "import": "./dist/esm/v2/theme/index.js",
       "require": "./dist/v2/theme/index.js",
       "default": "./dist/v2/theme/index.js"
     },
     "./styles": {
       "types": "./dist/v2/styles/index.d.ts",
-      "import": "./dist/v2/styles/index.js",
+      "import": "./dist/esm/v2/styles/index.js",
       "require": "./dist/v2/styles/index.js",
       "default": "./dist/v2/styles/index.js"
     },
     "./tokens": {
       "types": "./dist/v2/tokens/index.d.ts",
-      "import": "./dist/v2/tokens/index.js",
+      "import": "./dist/esm/v2/tokens/index.js",
       "require": "./dist/v2/tokens/index.js",
       "default": "./dist/v2/tokens/index.js"
     },
     "./hooks": {
       "types": "./dist/v2/hooks/index.d.ts",
-      "import": "./dist/v2/hooks/index.js",
+      "import": "./dist/esm/v2/hooks/index.js",
       "require": "./dist/v2/hooks/index.js",
       "default": "./dist/v2/hooks/index.js"
     },
     "./v2": {
       "types": "./dist/v2/index.d.ts",
-      "import": "./dist/v2/index.js",
+      "import": "./dist/esm/v2/index.js",
       "require": "./dist/v2/index.js",
       "default": "./dist/v2/index.js"
     },
     "./v2/theme": {
       "types": "./dist/v2/theme/index.d.ts",
-      "import": "./dist/v2/theme/index.js",
+      "import": "./dist/esm/v2/theme/index.js",
       "require": "./dist/v2/theme/index.js",
       "default": "./dist/v2/theme/index.js"
     },
     "./v2/styles": {
       "types": "./dist/v2/styles/index.d.ts",
-      "import": "./dist/v2/styles/index.js",
+      "import": "./dist/esm/v2/styles/index.js",
       "require": "./dist/v2/styles/index.js",
       "default": "./dist/v2/styles/index.js"
     },
     "./v2/tokens": {
       "types": "./dist/v2/tokens/index.d.ts",
-      "import": "./dist/v2/tokens/index.js",
+      "import": "./dist/esm/v2/tokens/index.js",
       "require": "./dist/v2/tokens/index.js",
       "default": "./dist/v2/tokens/index.js"
     },
     "./v2/hooks": {
       "types": "./dist/v2/hooks/index.d.ts",
-      "import": "./dist/v2/hooks/index.js",
+      "import": "./dist/esm/v2/hooks/index.js",
       "require": "./dist/v2/hooks/index.js",
       "default": "./dist/v2/hooks/index.js"
     },
     "./personalization": {
       "types": "./dist/v2/personalization/index.d.ts",
-      "import": "./dist/v2/personalization/index.js",
+      "import": "./dist/esm/v2/personalization/index.js",
       "require": "./dist/v2/personalization/index.js",
       "default": "./dist/v2/personalization/index.js"
     },
     "./v2/personalization": {
       "types": "./dist/v2/personalization/index.d.ts",
-      "import": "./dist/v2/personalization/index.js",
+      "import": "./dist/esm/v2/personalization/index.js",
       "require": "./dist/v2/personalization/index.js",
       "default": "./dist/v2/personalization/index.js"
     },
@@ -145,8 +148,9 @@
   },
   "scripts": {
     "start": "react-scripts start",
-    "build": "npm run build:js && npm run build:types",
+    "build": "npm run build:js && npm run build:esm && npm run build:types",
     "build:js": "cross-env NODE_ENV=production babel src/lib --out-dir dist --extensions \".js,.jsx,.ts,.tsx\" --copy-files --source-maps true",
+    "build:esm": "cross-env NODE_ENV=production BABEL_ENV=esm babel src/lib --out-dir dist/esm --extensions \".js,.jsx,.ts,.tsx\" --copy-files --source-maps true",
     "build:types": "tsc -p tsconfig.build.json",
     "build:clean": "rimraf dist && npm run build",
     "typecheck": "tsc --noEmit",
@@ -157,6 +161,10 @@
     "format:check": "prettier --check \"src/**/*.{js,jsx,ts,tsx,css,json,md}\"",
     "api:snapshot": "node scripts/check-public-api.mjs snapshot",
     "api:check": "node scripts/check-public-api.mjs check",
+    "props:widen": "node scripts/widen-optional-props.mjs",
+    "props:check": "node scripts/widen-optional-props.mjs --check",
+    "labels:fix": "node scripts/fix-label-merge.mjs",
+    "labels:check": "node scripts/fix-label-merge.mjs --check",
     "prepare": "npm run build",
     "prepublishOnly": "npm run build:clean",
     "test": "react-scripts test",