@chipmobilesdk/rn-i18n 0.1.1 → 0.1.2

package/CHANGELOG.md

@@ -3,6 +3,18 @@
 All notable changes to `@chipmobilesdk/rn-i18n` are documented here. This project
 follows [Semantic Versioning](https://semver.org/).
 
+## 0.1.2 — 2026-11-01
+
+### Fixed
+
+- **A throwing locale listener no longer breaks a locale switch.** Listeners
+  were notified without isolation, so one that threw skipped every listener
+  after it (React subscribers included) and made `setLocale` reject after the
+  switch but before the choice was persisted: the app showed one locale and
+  stored another. Each listener is now isolated and reported with
+  `console.warn`, and a listener unsubscribed by an earlier one during the
+  same change is no longer called. No code change is required.
+
 ## 0.1.0 — 2026-06-13
 
 Initial release of the reusable mobile i18n and locale library.

package/README.md

@@ -4,30 +4,30 @@ Reusable localization for ChipMobileSdk-family React Native apps: per-locale
 configuration, app-owned namespaced translations, typed keys, interpolation,
 pluralization, fallback chains, runtime locale switching with app-injected
 persistence, device-locale detection, locale-aware number/date/currency/unit
-formatting, and a Node CLI that extracts and synchronizes translation keys.
-
-> **License:** `UNLICENSED`. Published publicly for use by the owner's
-> applications; no open-source license is granted.
+formatting, and a Node CLI that extracts and synchronizes translation keys.
+
+> **License:** `UNLICENSED`. Published publicly for use by the owner's
+> applications; no open-source license is granted.
 
 - **Zero runtime dependencies.** Formatting uses the `Intl` APIs built into
   Hermes; plural rules and unit conversion are package-owned.
-- **No native code.** Storage and (optionally) device-locale detection are
-  injected by the consuming app.
-- **Public package.** Published under the `@chipmobilesdk` npm organization.
-- **Version**: `0.1.0`
-- **Entrypoints**: `@chipmobilesdk/rn-i18n` and the Node-only
-  `@chipmobilesdk/rn-i18n/tooling`
+- **No native code.** Storage and (optionally) device-locale detection are
+  injected by the consuming app.
+- **Public package.** Published under the `@chipmobilesdk` npm organization.
+- **Version**: `0.1.2`
+- **Entrypoints**: `@chipmobilesdk/rn-i18n` and the Node-only
+  `@chipmobilesdk/rn-i18n/tooling`
 
 ## Installation
 
-This package ships TypeScript sources and is consumed through the workspace or
-the public npm registry:
-
-```bash
-npm install @chipmobilesdk/rn-i18n
-```
-
-Host frameworks are peer dependencies:
+This package ships TypeScript sources and is consumed through the workspace or
+the public npm registry:
+
+```bash
+npm install @chipmobilesdk/rn-i18n
+```
+
+Host frameworks are peer dependencies:
 
 ```jsonc
 // peerDependencies
@@ -104,12 +104,12 @@ t('cart.items', { count }, { count }); // plural variant by count
 | Unit conversion | `i18n.convertUnit`, `convertUnit`, `getUnitSymbol`, `SUPPORTED_UNITS`, `UnitConversionError` |
 | Runtime-only declarations | `i18n.registerRuntimeEntries` |
 | Persistence key | `LOCALE_STORAGE_KEY` |
-| React | `I18nProvider` |
-
-Public contract types also include the configuration, locale, translation,
-formatting, unit, provider/adapter, and hook result types exported by
-`src/index.ts`. Do not deep-import package internals. Node-only extraction and
-synchronization APIs must be imported from `@chipmobilesdk/rn-i18n/tooling`.
+| React | `I18nProvider` |
+
+Public contract types also include the configuration, locale, translation,
+formatting, unit, provider/adapter, and hook result types exported by
+`src/index.ts`. Do not deep-import package internals. Node-only extraction and
+synchronization APIs must be imported from `@chipmobilesdk/rn-i18n/tooling`.
 
 ### Storage adapter (required for persistence)
 
@@ -148,9 +148,9 @@ App-owned JSON, organized by locale and namespace; nested dot-path keys,
 }
 ```
 
-Plural categories deliberately come from package-owned rules rather than
-depending on `Intl.PluralRules`, which keeps behavior consistent across the
-supported Hermes environments. Built-ins cover `en` (one/other), `vi`/`ja`
+Plural categories deliberately come from package-owned rules rather than
+depending on `Intl.PluralRules`, which keeps behavior consistent across the
+supported Hermes environments. Built-ins cover `en` (one/other), `vi`/`ja`
 (other-only), and a default `count === 1 -> one` rule; supply
 `LocaleConfig.pluralRule` for richer languages.
 
@@ -209,10 +209,10 @@ locales; `report` only lists them.
 npm test                     # package + demo suites
 npm run typecheck:i18n       # tsc -p packages/rn-i18n/tsconfig.json --noEmit
 npm run pack:i18n            # npm pack --workspace @chipmobilesdk/rn-i18n --dry-run
-npm run i18n:sync            # extraction + synchronization
-```
-
-The demo integration lives in `src/screens/I18nDemoScreen.tsx` and uses
-`src/i18n/` for setup, translations, persistence, and generated keys.
+npm run i18n:sync            # extraction + synchronization
+```
+
+The demo integration lives in `src/screens/I18nDemoScreen.tsx` and uses
+`src/i18n/` for setup, translations, persistence, and generated keys.
 
 See [CHANGELOG.md](CHANGELOG.md) for release notes and adoption guidance.

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@chipmobilesdk/rn-i18n",
-	"version": "0.1.1",
+	"version": "0.1.2",
 	"private": false,
 	"license": "UNLICENSED",
 	"main": "src/index.ts",

package/skills/sdk-rn-i18n/SKILL.md

@@ -1,62 +1,62 @@
----
-name: sdk-rn-i18n
-description: "Localization via @chipmobilesdk/rn-i18n: createI18n, useTranslation, useLocale, locale persistence, unit formatting, and the extraction/sync CLI. Use when adding translated strings, switching locale, formatting units, or syncing translation keys."
-metadata:
-  package: "@chipmobilesdk/rn-i18n"
-  packageVersion: "0.1.0"
-user-invocable: true
-disable-model-invocation: false
----
-
-# @chipmobilesdk/rn-i18n
-
-Localization with device-locale detection, persisted locale choice, typed translation
-access, unit/number formatting, and a CLI that extracts and synchronizes keys.
-
-## Use this when
-
-- Adding or changing any user-facing string
-- Reading or switching the active locale
-- Formatting numbers, dates or units for a locale
-- Translation keys are missing, stale, or out of sync across locale files
-
-## Never hardcode a user-facing string
-
-Every string rendered to a user goes through `useTranslation`. A literal in a component
-is a defect — it cannot be translated and will not appear in extraction output.
-
-## Keys are generated, not hand-maintained
-
-Do not hand-edit locale JSON to add a key. Run the extraction/sync CLI so every locale
-file stays aligned. The monorepo exposes it as `npm run i18n:sync`. See
-`## Extraction & synchronization CLI`.
-
-## Public API (v0.1.0)
-
-Runtime:
-
-- `createI18n(...)` → `I18n`
-- `I18nProvider` (+ `I18nProviderProps`)
-- `useTranslation()` → `UseTranslationResult`
-- `useLocale()` → `UseLocaleResult`
-- `defaultDeviceLocaleProvider` — device locale detection
-- `LOCALE_STORAGE_KEY` — where the chosen locale persists
-- `I18nConfigError` — thrown on invalid configuration at startup
-
-Types: `I18n`, `UnitFormatOptions`, `I18nProviderProps`, `UseLocaleResult`,
-`UseTranslationResult`
-
-`I18nConfigError` throws at startup by design — a misconfigured i18n setup should fail
-loudly, not silently render keys.
-
-## Where the detail lives
-
-Read one section of `node_modules/@chipmobilesdk/rn-i18n/README.md`:
-
-| Need | Section |
-|---|---|
-| First integration | `## Installation`, `## Quick start` |
-| Full export list | `## Public API` |
-| Locale file layout, plurals | `## Translations` |
-| Adding/syncing keys | `## Extraction & synchronization CLI` |
-| Version/peer constraints | `## Compatibility` |
+---
+name: sdk-rn-i18n
+description: "Localization via @chipmobilesdk/rn-i18n: createI18n, useTranslation, useLocale, locale persistence, unit formatting, and the extraction/sync CLI. Use when adding translated strings, switching locale, formatting units, or syncing translation keys."
+metadata:
+  package: "@chipmobilesdk/rn-i18n"
+  packageVersion: "0.1.0"
+user-invocable: true
+disable-model-invocation: false
+---
+
+# @chipmobilesdk/rn-i18n
+
+Localization with device-locale detection, persisted locale choice, typed translation
+access, unit/number formatting, and a CLI that extracts and synchronizes keys.
+
+## Use this when
+
+- Adding or changing any user-facing string
+- Reading or switching the active locale
+- Formatting numbers, dates or units for a locale
+- Translation keys are missing, stale, or out of sync across locale files
+
+## Never hardcode a user-facing string
+
+Every string rendered to a user goes through `useTranslation`. A literal in a component
+is a defect — it cannot be translated and will not appear in extraction output.
+
+## Keys are generated, not hand-maintained
+
+Do not hand-edit locale JSON to add a key. Run the extraction/sync CLI so every locale
+file stays aligned. The monorepo exposes it as `npm run i18n:sync`. See
+`## Extraction & synchronization CLI`.
+
+## Public API (v0.1.2)
+
+Runtime:
+
+- `createI18n(...)` → `I18n`
+- `I18nProvider` (+ `I18nProviderProps`)
+- `useTranslation()` → `UseTranslationResult`
+- `useLocale()` → `UseLocaleResult`
+- `defaultDeviceLocaleProvider` — device locale detection
+- `LOCALE_STORAGE_KEY` — where the chosen locale persists
+- `I18nConfigError` — thrown on invalid configuration at startup
+
+Types: `I18n`, `UnitFormatOptions`, `I18nProviderProps`, `UseLocaleResult`,
+`UseTranslationResult`
+
+`I18nConfigError` throws at startup by design — a misconfigured i18n setup should fail
+loudly, not silently render keys.
+
+## Where the detail lives
+
+Read one section of `node_modules/@chipmobilesdk/rn-i18n/README.md`:
+
+| Need | Section |
+|---|---|
+| First integration | `## Installation`, `## Quick start` |
+| Full export list | `## Public API` |
+| Locale file layout, plurals | `## Translations` |
+| Adding/syncing keys | `## Extraction & synchronization CLI` |
+| Version/peer constraints | `## Compatibility` |

package/src/locale/activeLocale.ts

@@ -20,8 +20,18 @@ export function createActiveLocaleStore(
 		set(next) {
 			preference = next;
 			version += 1;
+			// Snapshot, and isolate each listener (CORE-03). One that threw used to
+			// skip every listener after it — useSyncExternalStore subscribers
+			// included — and made setLocale reject after the switch but before the
+			// choice was persisted, so the app showed one locale and stored another.
 			for (const listener of [...listeners]) {
-				listener();
+				// Unsubscribed by an earlier listener during this notification.
+				if (!listeners.has(listener)) continue;
+				try {
+					listener();
+				} catch (error) {
+					console.warn('[rn-i18n] A locale listener threw; the locale change still applied.', error);
+				}
 			}
 		},
 		subscribe(listener) {