@open-mercato/shared 0.6.8-develop.7000.1.0ae682bad1 → 0.6.8-develop.7003.2.24fef49d48

package/dist/lib/i18n/context.js.map

@@ -1,7 +1,7 @@
 {
   "version": 3,
   "sources": ["../../../src/lib/i18n/context.tsx"],
-  "sourcesContent": ["\"use client\"\nimport { createContext, useContext, useMemo, type ReactNode } from 'react'\nimport type { Locale } from './config'\n\nexport type Dict = Record<string, string>\n\nexport type TranslateParams = Record<string, string | number>\n\nexport type TranslateFn = (\n  key: string,\n  fallbackOrParams?: string | TranslateParams,\n  params?: TranslateParams\n) => string\n\nexport type I18nContextValue = {\n  locale: Locale\n  t: TranslateFn\n  /** True when the locale is pinned via `OM_FORCE_LOCALE`; UI should hide switchers. */\n  localeLocked: boolean\n}\n\nconst I18N_CONTEXT_KEY = '__openMercatoI18nContext'\n\ntype GlobalI18nContextStore = typeof globalThis & {\n  [I18N_CONTEXT_KEY]?: ReturnType<typeof createContext<I18nContextValue | null>>\n}\n\nfunction getI18nContext() {\n  const store = globalThis as GlobalI18nContextStore\n  if (!store[I18N_CONTEXT_KEY]) {\n    store[I18N_CONTEXT_KEY] = createContext<I18nContextValue | null>(null)\n  }\n  return store[I18N_CONTEXT_KEY]\n}\n\nconst I18nContext = getI18nContext()\n\nfunction format(template: string, params?: TranslateParams) {\n  if (!params) return template\n  return template.replace(/\\{\\{(\\w+)\\}\\}|\\{(\\w+)\\}/g, (_, doubleKey, singleKey) => {\n    const key = doubleKey ?? singleKey\n    if (!key) return _\n    const value = params[key]\n    if (value === undefined) {\n      return doubleKey ? `{{${key}}}` : `{${key}}`\n    }\n    return String(value)\n  })\n}\n\nexport function I18nProvider({ children, locale, dict, localeLocked = false }: { children: ReactNode; locale: Locale; dict: Dict; localeLocked?: boolean }) {\n  const value = useMemo<I18nContextValue>(() => ({\n    locale,\n    localeLocked,\n    t: (key, fallbackOrParams, params) => {\n      let fallback: string | undefined\n      let resolvedParams: TranslateParams | undefined\n\n      if (typeof fallbackOrParams === 'string') {\n        fallback = fallbackOrParams\n        resolvedParams = params\n      } else {\n        resolvedParams = fallbackOrParams ?? params\n      }\n\n      const template = dict[key] ?? fallback ?? key\n      return format(template, resolvedParams)\n    },\n  }), [locale, dict, localeLocked])\n  return <I18nContext.Provider value={value}>{children}</I18nContext.Provider>\n}\n\nexport function useT() {\n  const ctx = useContext(I18nContext)\n  if (!ctx) throw new Error('useT must be used within I18nProvider')\n  return ctx.t\n}\n\n/**\n * Like `useT`, but returns `undefined` instead of throwing when no\n * `I18nProvider` is in scope. Use where a translator is desirable but not\n * guaranteed (e.g. plumbing `t` into side-effect handlers that may run before\n * the provider mounts) \u2014 callers MUST provide a fallback.\n */\nexport function useOptionalT(): TranslateFn | undefined {\n  const ctx = useContext(I18nContext)\n  return ctx?.t\n}\n\nexport function useLocale() {\n  const ctx = useContext(I18nContext)\n  if (!ctx) throw new Error('useLocale must be used within I18nProvider')\n  return ctx.locale\n}\n\n/**\n * Like `useLocale`, but returns `undefined` instead of throwing when no\n * `I18nProvider` is in scope. Use in shared components that receive their\n * translator as a prop and may render outside a provider (galleries, tests).\n */\nexport function useOptionalLocale(): Locale | undefined {\n  const ctx = useContext(I18nContext)\n  return ctx?.locale\n}\n\n/**\n * True when the active locale is pinned via `OM_FORCE_LOCALE`. Returns `false`\n * when no provider is in scope so callers can render unconditionally.\n */\nexport function useLocaleLocked() {\n  const ctx = useContext(I18nContext)\n  return ctx?.localeLocked ?? false\n}\n"],
-  "mappings": ";AAqES;AApET,SAAS,eAAe,YAAY,eAA+B;AAoBnE,MAAM,mBAAmB;AAMzB,SAAS,iBAAiB;AACxB,QAAM,QAAQ;AACd,MAAI,CAAC,MAAM,gBAAgB,GAAG;AAC5B,UAAM,gBAAgB,IAAI,cAAuC,IAAI;AAAA,EACvE;AACA,SAAO,MAAM,gBAAgB;AAC/B;AAEA,MAAM,cAAc,eAAe;AAEnC,SAAS,OAAO,UAAkB,QAA0B;AAC1D,MAAI,CAAC,OAAQ,QAAO;AACpB,SAAO,SAAS,QAAQ,4BAA4B,CAAC,GAAG,WAAW,cAAc;AAC/E,UAAM,MAAM,aAAa;AACzB,QAAI,CAAC,IAAK,QAAO;AACjB,UAAM,QAAQ,OAAO,GAAG;AACxB,QAAI,UAAU,QAAW;AACvB,aAAO,YAAY,KAAK,GAAG,OAAO,IAAI,GAAG;AAAA,IAC3C;AACA,WAAO,OAAO,KAAK;AAAA,EACrB,CAAC;AACH;AAEO,SAAS,aAAa,EAAE,UAAU,QAAQ,MAAM,eAAe,MAAM,GAAgF;AAC1J,QAAM,QAAQ,QAA0B,OAAO;AAAA,IAC7C;AAAA,IACA;AAAA,IACA,GAAG,CAAC,KAAK,kBAAkB,WAAW;AACpC,UAAI;AACJ,UAAI;AAEJ,UAAI,OAAO,qBAAqB,UAAU;AACxC,mBAAW;AACX,yBAAiB;AAAA,MACnB,OAAO;AACL,yBAAiB,oBAAoB;AAAA,MACvC;AAEA,YAAM,WAAW,KAAK,GAAG,KAAK,YAAY;AAC1C,aAAO,OAAO,UAAU,cAAc;AAAA,IACxC;AAAA,EACF,IAAI,CAAC,QAAQ,MAAM,YAAY,CAAC;AAChC,SAAO,oBAAC,YAAY,UAAZ,EAAqB,OAAe,UAAS;AACvD;AAEO,SAAS,OAAO;AACrB,QAAM,MAAM,WAAW,WAAW;AAClC,MAAI,CAAC,IAAK,OAAM,IAAI,MAAM,uCAAuC;AACjE,SAAO,IAAI;AACb;AAQO,SAAS,eAAwC;AACtD,QAAM,MAAM,WAAW,WAAW;AAClC,SAAO,KAAK;AACd;AAEO,SAAS,YAAY;AAC1B,QAAM,MAAM,WAAW,WAAW;AAClC,MAAI,CAAC,IAAK,OAAM,IAAI,MAAM,4CAA4C;AACtE,SAAO,IAAI;AACb;AAOO,SAAS,oBAAwC;AACtD,QAAM,MAAM,WAAW,WAAW;AAClC,SAAO,KAAK;AACd;AAMO,SAAS,kBAAkB;AAChC,QAAM,MAAM,WAAW,WAAW;AAClC,SAAO,KAAK,gBAAgB;AAC9B;",
+  "sourcesContent": ["\"use client\"\nimport { createContext, useContext, useMemo, type PropsWithChildren } from 'react'\nimport type { Locale } from './config'\n\nexport type Dict = Record<string, string>\n\nexport type TranslateParams = Record<string, string | number>\n\nexport type TranslateFn = (\n  key: string,\n  fallbackOrParams?: string | TranslateParams,\n  params?: TranslateParams\n) => string\n\nexport type I18nContextValue = {\n  locale: Locale\n  t: TranslateFn\n  /** True when the locale is pinned via `OM_FORCE_LOCALE`; UI should hide switchers. */\n  localeLocked: boolean\n}\n\nconst I18N_CONTEXT_KEY = '__openMercatoI18nContext'\n\ntype GlobalI18nContextStore = typeof globalThis & {\n  [I18N_CONTEXT_KEY]?: ReturnType<typeof createContext<I18nContextValue | null>>\n}\n\nfunction getI18nContext() {\n  const store = globalThis as GlobalI18nContextStore\n  if (!store[I18N_CONTEXT_KEY]) {\n    store[I18N_CONTEXT_KEY] = createContext<I18nContextValue | null>(null)\n  }\n  return store[I18N_CONTEXT_KEY]\n}\n\nconst I18nContext = getI18nContext()\n\nfunction format(template: string, params?: TranslateParams) {\n  if (!params) return template\n  return template.replace(/\\{\\{(\\w+)\\}\\}|\\{(\\w+)\\}/g, (_, doubleKey, singleKey) => {\n    const key = doubleKey ?? singleKey\n    if (!key) return _\n    const value = params[key]\n    if (value === undefined) {\n      return doubleKey ? `{{${key}}}` : `{${key}}`\n    }\n    return String(value)\n  })\n}\n\nexport function I18nProvider({ children, locale, dict, localeLocked = false }: PropsWithChildren<{ locale: Locale; dict: Dict; localeLocked?: boolean }>) {\n  const value = useMemo<I18nContextValue>(() => ({\n    locale,\n    localeLocked,\n    t: (key, fallbackOrParams, params) => {\n      let fallback: string | undefined\n      let resolvedParams: TranslateParams | undefined\n\n      if (typeof fallbackOrParams === 'string') {\n        fallback = fallbackOrParams\n        resolvedParams = params\n      } else {\n        resolvedParams = fallbackOrParams ?? params\n      }\n\n      const template = dict[key] ?? fallback ?? key\n      return format(template, resolvedParams)\n    },\n  }), [locale, dict, localeLocked])\n  return <I18nContext.Provider value={value}>{children}</I18nContext.Provider>\n}\n\nexport function useT() {\n  const ctx = useContext(I18nContext)\n  if (!ctx) throw new Error('useT must be used within I18nProvider')\n  return ctx.t\n}\n\n/**\n * Like `useT`, but returns `undefined` instead of throwing when no\n * `I18nProvider` is in scope. Use where a translator is desirable but not\n * guaranteed (e.g. plumbing `t` into side-effect handlers that may run before\n * the provider mounts) \u2014 callers MUST provide a fallback.\n */\nexport function useOptionalT(): TranslateFn | undefined {\n  const ctx = useContext(I18nContext)\n  return ctx?.t\n}\n\nexport function useLocale() {\n  const ctx = useContext(I18nContext)\n  if (!ctx) throw new Error('useLocale must be used within I18nProvider')\n  return ctx.locale\n}\n\n/**\n * Like `useLocale`, but returns `undefined` instead of throwing when no\n * `I18nProvider` is in scope. Use in shared components that receive their\n * translator as a prop and may render outside a provider (galleries, tests).\n */\nexport function useOptionalLocale(): Locale | undefined {\n  const ctx = useContext(I18nContext)\n  return ctx?.locale\n}\n\n/**\n * True when the active locale is pinned via `OM_FORCE_LOCALE`. Returns `false`\n * when no provider is in scope so callers can render unconditionally.\n */\nexport function useLocaleLocked() {\n  const ctx = useContext(I18nContext)\n  return ctx?.localeLocked ?? false\n}\n"],
+  "mappings": ";AAqES;AApET,SAAS,eAAe,YAAY,eAAuC;AAoB3E,MAAM,mBAAmB;AAMzB,SAAS,iBAAiB;AACxB,QAAM,QAAQ;AACd,MAAI,CAAC,MAAM,gBAAgB,GAAG;AAC5B,UAAM,gBAAgB,IAAI,cAAuC,IAAI;AAAA,EACvE;AACA,SAAO,MAAM,gBAAgB;AAC/B;AAEA,MAAM,cAAc,eAAe;AAEnC,SAAS,OAAO,UAAkB,QAA0B;AAC1D,MAAI,CAAC,OAAQ,QAAO;AACpB,SAAO,SAAS,QAAQ,4BAA4B,CAAC,GAAG,WAAW,cAAc;AAC/E,UAAM,MAAM,aAAa;AACzB,QAAI,CAAC,IAAK,QAAO;AACjB,UAAM,QAAQ,OAAO,GAAG;AACxB,QAAI,UAAU,QAAW;AACvB,aAAO,YAAY,KAAK,GAAG,OAAO,IAAI,GAAG;AAAA,IAC3C;AACA,WAAO,OAAO,KAAK;AAAA,EACrB,CAAC;AACH;AAEO,SAAS,aAAa,EAAE,UAAU,QAAQ,MAAM,eAAe,MAAM,GAA8E;AACxJ,QAAM,QAAQ,QAA0B,OAAO;AAAA,IAC7C;AAAA,IACA;AAAA,IACA,GAAG,CAAC,KAAK,kBAAkB,WAAW;AACpC,UAAI;AACJ,UAAI;AAEJ,UAAI,OAAO,qBAAqB,UAAU;AACxC,mBAAW;AACX,yBAAiB;AAAA,MACnB,OAAO;AACL,yBAAiB,oBAAoB;AAAA,MACvC;AAEA,YAAM,WAAW,KAAK,GAAG,KAAK,YAAY;AAC1C,aAAO,OAAO,UAAU,cAAc;AAAA,IACxC;AAAA,EACF,IAAI,CAAC,QAAQ,MAAM,YAAY,CAAC;AAChC,SAAO,oBAAC,YAAY,UAAZ,EAAqB,OAAe,UAAS;AACvD;AAEO,SAAS,OAAO;AACrB,QAAM,MAAM,WAAW,WAAW;AAClC,MAAI,CAAC,IAAK,OAAM,IAAI,MAAM,uCAAuC;AACjE,SAAO,IAAI;AACb;AAQO,SAAS,eAAwC;AACtD,QAAM,MAAM,WAAW,WAAW;AAClC,SAAO,KAAK;AACd;AAEO,SAAS,YAAY;AAC1B,QAAM,MAAM,WAAW,WAAW;AAClC,MAAI,CAAC,IAAK,OAAM,IAAI,MAAM,4CAA4C;AACtE,SAAO,IAAI;AACb;AAOO,SAAS,oBAAwC;AACtD,QAAM,MAAM,WAAW,WAAW;AAClC,SAAO,KAAK;AACd;AAMO,SAAS,kBAAkB;AAChC,QAAM,MAAM,WAAW,WAAW;AAClC,SAAO,KAAK,gBAAgB;AAC9B;",
   "names": []
 }

package/dist/lib/version.js

@@ -1,4 +1,4 @@
-const APP_VERSION = "0.6.8-develop.7000.1.0ae682bad1";
+const APP_VERSION = "0.6.8-develop.7003.2.24fef49d48";
 const appVersion = APP_VERSION;
 export {
   APP_VERSION,

package/dist/lib/version.js.map

@@ -1,7 +1,7 @@
 {
   "version": 3,
   "sources": ["../../src/lib/version.ts"],
-  "sourcesContent": ["// Build-time generated version\nexport const APP_VERSION = '0.6.8-develop.7000.1.0ae682bad1';\nexport const appVersion = APP_VERSION;\n"],
+  "sourcesContent": ["// Build-time generated version\nexport const APP_VERSION = '0.6.8-develop.7003.2.24fef49d48';\nexport const appVersion = APP_VERSION;\n"],
   "mappings": "AACO,MAAM,cAAc;AACpB,MAAM,aAAa;",
   "names": []
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-mercato/shared",
-  "version": "0.6.8-develop.7000.1.0ae682bad1",
+  "version": "0.6.8-develop.7003.2.24fef49d48",
   "license": "MIT",
   "type": "module",
   "main": "./dist/index.js",
@@ -105,7 +105,7 @@
     "@mikro-orm/core": "^7.1.8",
     "@mikro-orm/decorators": "^7.1.8",
     "@mikro-orm/postgresql": "^7.1.8",
-    "@open-mercato/cache": "0.6.8-develop.7000.1.0ae682bad1",
+    "@open-mercato/cache": "0.6.8-develop.7003.2.24fef49d48",
     "@types/sanitize-html": "^2.16.1",
     "dotenv": "^17.4.2",
     "pino": "^10.3.1",

package/src/lib/i18n/__tests__/context.test.tsx

@@ -0,0 +1,26 @@
+import * as React from 'react'
+import { renderToStaticMarkup } from 'react-dom/server'
+import { I18nProvider, useT } from '../context'
+
+function Greeting() {
+  const t = useT()
+  return React.createElement('span', null, t('greeting', 'hello'))
+}
+
+describe('I18nProvider', () => {
+  it('accepts children positionally via React.createElement, matching props without a children key', () => {
+    const element = React.createElement(
+      I18nProvider,
+      { locale: 'en', dict: {} },
+      React.createElement(Greeting),
+    )
+
+    expect(renderToStaticMarkup(element)).toContain('hello')
+  })
+
+  it('renders with no children at all', () => {
+    const element = React.createElement(I18nProvider, { locale: 'en', dict: {} })
+
+    expect(renderToStaticMarkup(element)).toBe('')
+  })
+})

package/src/lib/i18n/context.tsx

@@ -1,5 +1,5 @@
 "use client"
-import { createContext, useContext, useMemo, type ReactNode } from 'react'
+import { createContext, useContext, useMemo, type PropsWithChildren } from 'react'
 import type { Locale } from './config'
 
 export type Dict = Record<string, string>
@@ -48,7 +48,7 @@ function format(template: string, params?: TranslateParams) {
   })
 }
 
-export function I18nProvider({ children, locale, dict, localeLocked = false }: { children: ReactNode; locale: Locale; dict: Dict; localeLocked?: boolean }) {
+export function I18nProvider({ children, locale, dict, localeLocked = false }: PropsWithChildren<{ locale: Locale; dict: Dict; localeLocked?: boolean }>) {
   const value = useMemo<I18nContextValue>(() => ({
     locale,
     localeLocked,

package/src/lib/i18n/context.typecheck.tsx

@@ -0,0 +1,36 @@
+// Compile-time-only regression guard for issue #5155. Never imported at
+// runtime: `yarn typecheck` (`tsc --noEmit`) is the only gate that can catch
+// a regression here, because this repo's Jest transform runs with
+// `isolatedModules: true` and therefore skips type diagnostics — a test in
+// `__tests__/context.test.tsx` would keep passing even if `children` became
+// required again.
+//
+// It sits outside `__tests__` because `packages/shared/tsconfig.json` excludes
+// that directory from `tsc --noEmit`, and it is a `.tsx` rather than a `.ts`
+// because the mutation gate mutates changed `src/lib/**/*.ts` files and runs
+// only their related Jest tests (`enableFindRelatedTests`). A file that exists
+// purely for the type checker has no related tests by construction, so Stryker
+// aborts the run with "No tests were executed"; `.tsx` is already out of that
+// scope by design — see `scripts/stryker/scope.mjs`.
+import * as React from 'react'
+import { I18nProvider } from './context'
+
+function acceptsChildrenPositionally() {
+  return React.createElement(I18nProvider, { locale: 'en', dict: {} }, <span />)
+}
+
+function acceptsNoChildren() {
+  return React.createElement(I18nProvider, { locale: 'en', dict: {} })
+}
+
+function acceptsChildrenAsJsx() {
+  return (
+    <I18nProvider locale="en" dict={{}}>
+      <span />
+    </I18nProvider>
+  )
+}
+
+void acceptsChildrenPositionally
+void acceptsNoChildren
+void acceptsChildrenAsJsx