@ekanos/ui 0.1.0 → 0.1.1

package/README.md

@@ -119,6 +119,5 @@ pnpm --filter @ekanos/ui pack:test  # clean-room publish gate (see scripts/pack-
 into a temp project outside the workspace and typechecks
 (`skipLibCheck: false`) + esbuild-bundles a consumer importing every subpath.
 
-The package is `private: true` until the owner flips it for the first publish;
-`prepack` runs the build so a publish can never ship a stale or missing
+`prepack` runs the build, so a publish can never ship a stale or missing
 `dist/`.

package/dist/trans.d.ts

@@ -25,14 +25,52 @@ type TransProps = ComponentProps<typeof Trans$1>;
  *
  * Verified against react-i18next 16.5.4: when no instance is passed via the
  * `i18n` prop and none has been registered globally, the component logs a
- * one-time `NO_I18NEXT_INSTANCE` console warning and returns its children
- * unmodified. It does not throw.
- *
- * Note the sharp edge this implies: the fallback is **children**, not
- * `defaults`. `<Trans i18nKey="a.b" defaults="Continue" />` has no children, so
- * with no instance available it renders nothing at all. Supply children when a
- * string must survive an unconfigured host:
- * `<Trans i18nKey="a.b" defaults="Continue">Continue</Trans>`.
+ * one-time `NO_I18NEXT_INSTANCE` console warning and returns its **children**
+ * unmodified — not `defaults`. Every SDK call site passes `i18nKey` +
+ * `defaults` and no children, so unpatched that renders nothing at all in any
+ * host without i18next (a partner's Storybook, a bare unit test). Inside Fusion
+ * and the harness it never shows, which is how it reached 0.1.0.
+ *
+ * We therefore detect that fall-through and render `defaults` ourselves.
+ *
+ * ### How the no-instance case is detected
+ *
+ * Not by asking `getI18n()`, because from here we cannot. It is not exported
+ * from `react-i18next/TransWithoutContext`, and react-i18next's `exports` map
+ * lists only `.`, `./TransWithoutContext`, `./initReactI18next`, `./icu.macro`
+ * and `./package.json`, so a deep import is refused. Reaching it through the
+ * main entry would drag in `context.js`, which calls `createContext()` at
+ * module scope — and under the `react-server` condition React 19 exports
+ * `createContext: undefined`, which would cost this module the
+ * RSC-renderability that is its whole reason for using this entry point.
+ *
+ * What is available is the library's own signal: with no instance it returns
+ * the `children` prop BY REFERENCE, and with an instance it returns
+ * `renderNodes(...)`, which never is that reference. Reusing react-i18next's
+ * own guard rather than reimplementing its condition means the two cannot
+ * drift apart across upgrades, and its module-level warn-once still owns the
+ * warning, so nothing warns twice.
+ *
+ * ### Why not simply pass `defaults` as children
+ *
+ * Because it has a measurable regression in the no-instance branch: children
+ * are exactly what react-i18next returns there, so injected `defaults` would
+ * override children a caller actually supplied. There is a test for that.
+ *
+ * The reason usually given for avoiding it — that children flip
+ * `interpolationOverride` (react-i18next rewrites the interpolation delimiters
+ * to `#$?…?$#` when `values` and `count` are absent but children are present)
+ * and also feed `getValuesFromChildren` — is real in the source but, measured,
+ * is NOT observable: `renderNodes` re-interpolates afterwards, and injecting
+ * `defaults` as children produced byte-identical output across `values`,
+ * `tOptions`, `components`, plural-via-`count`, and
+ * missing-key-falls-back-to-`defaults`. That argument should not be repeated
+ * as fact.
+ *
+ * The approach below sidesteps the question entirely: it forwards the caller's
+ * props untouched and injects nothing, so the with-instance path is identical
+ * by construction rather than identical across whichever cases someone
+ * happened to try.
  */
 declare function Trans(props: TransProps): React.JSX.Element;
 

package/dist/trans.js

@@ -1,8 +1,14 @@
 // src/_impl/trans.tsx
 import { Trans as I18nTrans } from "react-i18next/TransWithoutContext";
-import { jsx } from "react/jsx-runtime";
+import { Fragment, jsx } from "react/jsx-runtime";
+var renderI18nTrans = I18nTrans;
 function Trans(props) {
-  return /* @__PURE__ */ jsx(I18nTrans, { ...props });
+  const rendered = renderI18nTrans(props);
+  const fellThroughToChildren = rendered === props.children && props.children == null;
+  if (fellThroughToChildren && typeof props.defaults === "string") {
+    return /* @__PURE__ */ jsx(Fragment, { children: props.defaults });
+  }
+  return /* @__PURE__ */ jsx(Fragment, { children: rendered });
 }
 export {
   Trans

package/package.json

@@ -1,17 +1,11 @@
 {
   "name": "@ekanos/ui",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "type": "module",
   "description": "Fusion-styled UI primitives for building Ekanos integrations outside the monorepo. A curated, published slice of the internal @kit/ui library.",
   "license": "MIT",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/companydotcom/fusion.git",
-    "directory": "packages/ekanos-ui"
-  },
-  "homepage": "https://github.com/companydotcom/fusion/tree/dev/packages/ekanos-ui#readme",
   "bugs": {
-    "url": "https://github.com/companydotcom/fusion/issues"
+    "email": "npm@govastly.com"
   },
   "sideEffects": false,
   "files": [
@@ -102,6 +96,7 @@
   },
   "dependencies": {
     "@base-ui/react": "1.6.0",
+    "@types/json-schema": "^7.0.15",
     "ai": "^6.0.197",
     "class-variance-authority": "^0.7.1",
     "clsx": "^2.1.1",
@@ -126,10 +121,11 @@
     "react-hook-form": "^7.68.0",
     "tsup": "8.5.1",
     "typescript": "^5.9.3",
-    "@kit/eslint-config": "0.2.0",
-    "@kit/ui": "0.1.0",
+    "vitest": "4.1.10",
     "@kit/prettier-config": "0.1.0",
-    "@kit/tsconfig": "0.1.0"
+    "@kit/tsconfig": "0.1.0",
+    "@kit/ui": "0.1.0",
+    "@kit/eslint-config": "0.2.0"
   },
   "prettier": "@kit/prettier-config",
   "typesVersions": {
@@ -145,6 +141,7 @@
     "format": "prettier --check \"**/*.{ts,tsx,mjs}\"",
     "lint": "eslint .",
     "typecheck": "tsc --noEmit",
+    "test": "vitest run --config vitest.config.ts",
     "pack:test": "node scripts/pack-test.mjs"
   }
 }