@urbicon-ui/i18n 8.25.0 → 8.26.0

package/README.md

@@ -344,7 +344,7 @@ import type {
 } from '@urbicon-ui/i18n';
 ```
 
-> **Breaking (major):** the pre-WP2 module singleton (`i18n`, the free `t` / `plural`, `I18nService`) was **removed** — it leaked the locale across SSR requests. Mount `<I18nProvider>` / `provideI18n` and read through the hooks; replace `i18n.t(k)` → `useI18n().t(k)`, `bt(k)` → `const bt = useBlocksI18n()`.
+> **Breaking (major):** the module-level singleton (`i18n`, the free `t` / `plural`, `I18nService`) was **removed** — it leaked the locale across SSR requests. Mount `<I18nProvider>` / `provideI18n` and read through the hooks; replace `i18n.t(k)` → `useI18n().t(k)`, `bt(k)` → `const bt = useBlocksI18n()`.
 
 ## Supported Locales (Core)
 

package/dist/i18n/context.svelte.js

@@ -108,7 +108,7 @@ export function provideI18n(locale, fallbackLocale = BASE_LOCALE) {
             }
         });
     }
-    // Ensure the initial active + fallback locales' lazy bundles are loaded (WP4) —
+    // Ensure the initial active + fallback locales' lazy bundles are loaded —
     // a no-op when everything is eager. Client-only (effects don't run during SSR),
     // so a lazy non-base initial locale renders the fallback on the server and the
     // first client paint, then re-resolves once the chunk lands. Subsequent switches

package/dist/i18n/package-integration.d.ts

@@ -1,7 +1,7 @@
 import type { CreatePackageTypes, Locale, PackageI18n, PackageTranslations, TranslationParams, TranslationSchema, Translations, TypedTranslationFunction } from './types.js';
 export interface CreatePackageI18nOptions {
     /**
-     * Per-locale lazy loaders (WP4 code-splitting). Each returns the package's
+     * Per-locale lazy loaders (code-splitting). Each returns the package's
      * bundle for that locale as a dynamic-import chunk, e.g.
      * `() => import('./translations/de').then((m) => m.default)`. Listed locales
      * stay out of the initial bundle until activated by the provider / `setLocale`;
@@ -21,7 +21,7 @@ export interface CreatePackageI18nOptions {
  * `TranslationSchema<T>` (T's structure with its string values widened), which
  * enforces key parity at compile time while allowing locale-specific strings.
  *
- * Opt-in code-splitting (WP4): pass `options.loaders` to keep non-base locales
+ * Opt-in code-splitting: pass `options.loaders` to keep non-base locales
  * out of the initial bundle as dynamic-import chunks, loaded only when activated.
  */
 export declare function createPackageI18n<const T extends Translations>(packageName: string, translations: {

package/dist/i18n/package-integration.js

@@ -12,11 +12,11 @@ import { isLocaleSupported, SUPPORTED_LOCALES } from './types.js';
  * `TranslationSchema<T>` (T's structure with its string values widened), which
  * enforces key parity at compile time while allowing locale-specific strings.
  *
- * Opt-in code-splitting (WP4): pass `options.loaders` to keep non-base locales
+ * Opt-in code-splitting: pass `options.loaders` to keep non-base locales
  * out of the initial bundle as dynamic-import chunks, loaded only when activated.
  */
 export function createPackageI18n(packageName, translations, options) {
-    // Lazy, first-use registration — deliberately NOT at module-eval (Codeberg #22).
+    // Lazy, first-use registration — deliberately NOT at module-eval.
     // A consumer's top-level `export const x = createPackageI18n(...)` must not call
     // getRegistry() during module initialisation: in a reordered *production* chunk
     // (Rollup, `sideEffects: false`) that call can run before the registry module's
@@ -40,7 +40,7 @@ export function createPackageI18n(packageName, translations, options) {
         registered = true;
         const registry = getRegistry();
         registry.registerPackage(packageName, translations);
-        // Opt-in lazy locales (WP4): dynamic-import loaders alongside the eager base
+        // Opt-in lazy locales: dynamic-import loaders alongside the eager base
         // bundle, loaded on demand by the provider / setLocale.
         if (options?.loaders) {
             for (const [locale, loader] of Object.entries(options.loaders)) {

package/dist/i18n/registry.svelte.d.ts

@@ -47,7 +47,7 @@ export declare class I18nRegistry {
     registerPackage(packageName: string, translations: PackageTranslations): void;
     registerTranslationLoader(locale: Locale, loader: TranslationLoader): void;
     /**
-     * Register a per-package lazy loader for one locale (WP4 code-splitting). The
+     * Register a per-package lazy loader for one locale (code-splitting). The
      * loader returns that package's bundle for `locale` (typically
      * `() => import('./translations/de').then((m) => m.default)`), kept out of the
      * initial chunk until the locale is activated.

package/dist/i18n/registry.svelte.js

@@ -27,7 +27,7 @@ export class I18nRegistry {
     loadingLocales = new SvelteSet();
     // Non-reactive per-instance cache; SvelteMap is unnecessary here.
     translationLoaders = new Map();
-    // Per-package lazy loaders (WP4 code-splitting), keyed `${packageName}::${locale}`.
+    // Per-package lazy loaders (code-splitting), keyed `${packageName}::${locale}`.
     // Registered at module-eval, read in setLocale/loadLocale (not in a $derived), so
     // a plain Map suffices. The *loaded data* lands in the reactive packageTranslations,
     // which is what re-resolves `$derived` reads when a chunk arrives.
@@ -101,7 +101,7 @@ export class I18nRegistry {
         this.translationLoaders.set(locale, loader);
     }
     /**
-     * Register a per-package lazy loader for one locale (WP4 code-splitting). The
+     * Register a per-package lazy loader for one locale (code-splitting). The
      * loader returns that package's bundle for `locale` (typically
      * `() => import('./translations/de').then((m) => m.default)`), kept out of the
      * initial chunk until the locale is activated.
@@ -491,13 +491,15 @@ export class I18nRegistry {
 }
 // Lazy, hoisted accessor for the process-wide registry. Holding the registry at
 // module scope is correct *because* it carries no per-request mutable locale —
-// only static, request-identical translation data. But it is built on first
-// touch (not at module top-level) and reached through this hoisted function so
-// that a consumer chunk's top-level `createPackageI18n` side-effect — which
-// Vite 8 / Rolldown may order before this module's body — still finds a callable
-// binding and an initialised instance, instead of a value in the temporal dead
-// zone (the `Cannot read properties of undefined (reading 'registerPackage')`
-// class of bug the old lazy singleton guarded against).
+// only static, request-identical translation data. The instance is built on the
+// first call, not at module top-level. Hoisting keeps the *function* callable from
+// a module that runs before this module's body, but it does not make an early call
+// safe: natively, the first read of the `let _registry` binding below throws
+// ("Cannot access '_registry' before initialization"); in a reordered bundled
+// chunk the class binding is still unassigned, so it is `new (undefined)()`.
+// Nothing may therefore call this during module initialisation — which is why
+// `createPackageI18n` registers lazily, on first use (`ensureRegistered()`),
+// rather than at module-eval.
 let _registry;
 export function getRegistry() {
     _registry ??= new I18nRegistry();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@urbicon-ui/i18n",
-  "version": "8.25.0",
+  "version": "8.26.0",
   "description": "Runes-based localization for Svelte 5 apps and the Urbicon UI design system",
   "license": "MIT",
   "repository": {