@kernhq/module-inventory 0.1.2 → 0.2.0

package/src/client/components/AssetFormDialog.svelte

@@ -1,23 +1,37 @@
 <script lang="ts">
-import { Button, Dialog, Field, Input, Select, Textarea, toast } from '@kernhq/ui'
+import { Button, Dialog, Field, Input, messageLocale, Select, Textarea, toast } from '@kernhq/ui'
 import { createMutation, useQueryClient } from '@tanstack/svelte-query'
+import type { Asset } from '../../contract/index.js'
 import { getInventoryApi } from '../api-instance.js'
 import { t } from '../i18n.js'
 import { canInventory } from '../permissions.js'
+import { formatPrice, parsePrice } from '../price.js'
+import { inventoryKeys } from '../query.js'
 
 /**
- * Add an asset. The tag (`INV-…`) is the server's job — people never invent codes, they read them
- * off stickers — so this form asks only for what a human actually knows on day one.
+ * Add or change an asset.
+ *
+ * One dialog for both, because they ask for exactly the same nine things and a second component
+ * would be nine fields to keep in step. `asset` is what decides which: null creates, a row edits.
+ * `assets.update` had no entry point anywhere in the module before this — the procedure existed,
+ * was tested, and no screen could reach it.
+ *
+ * The tag (`INV-…`) is the server's job — people never invent codes, they read them off stickers —
+ * so this form asks only for what a human actually knows.
  */
 interface Props {
   open: boolean
   workspaceId: string
+  /** The row being changed, or null to add one. */
+  asset?: Asset | null
 }
-let { open = $bindable(false), workspaceId }: Props = $props()
+let { open = $bindable(false), workspaceId, asset = null }: Props = $props()
 
 const api = getInventoryApi()
 const queryClient = useQueryClient()
 
+const editing = $derived(asset !== null)
+
 let shown = $state(false)
 $effect(() => {
   if (open) shown = true
@@ -29,63 +43,123 @@ let serialNumber = $state('')
 let location = $state('')
 let purchasedOn = $state('')
 let purchasedFrom = $state('')
-let priceMinor = $state('')
+let priceText = $state('')
 let currency = $state('')
 let warrantyUntil = $state('')
 
+/**
+ * What the form was last filled from, so opening it on a different row re-seeds it.
+ *
+ * A plain variable rather than `$state`: the effect below writes it on every run, and a reactive
+ * flag it also reads is a dependency on its own write.
+ */
+let seededFrom: string | null = null
+
+$effect(() => {
+  const key = open ? (asset?.id ?? 'new') : null
+  if (key === null) {
+    seededFrom = null
+    return
+  }
+  if (seededFrom === key) return
+  seededFrom = key
+  fill(asset)
+})
+
+function fill(row: Asset | null) {
+  name = row?.name ?? ''
+  description = row?.description ?? ''
+  serialNumber = row?.serialNumber ?? ''
+  location = row?.location ?? ''
+  purchasedOn = row?.purchasedOn ?? ''
+  purchasedFrom = row?.purchasedFrom ?? ''
+  // Formatted for the reader's own locale, and `parsePrice` reads back exactly what this writes.
+  priceText = formatPrice(row?.priceMinor, messageLocale())
+  currency = row?.currency ?? ''
+  warrantyUntil = row?.warrantyUntil ?? ''
+}
+
 const close = () => {
   shown = false
   open = false
 }
 
-const reset = () => {
-  name = ''
-  description = ''
-  serialNumber = ''
-  location = ''
-  purchasedOn = ''
-  purchasedFrom = ''
-  priceMinor = ''
-  currency = ''
-  warrantyUntil = ''
-}
+/**
+ * The price, parsed the way the reader's own language writes one.
+ *
+ * `Number.parseFloat(raw.replace(',', '.'))` replaced the first comma only, so `1.234,56` — German,
+ * Turkish and Persian for twelve hundred — was stored as €1.23, and `abc` and `-5` were stored as
+ * no price at all. Neither said anything. Now an amount this locale cannot read is an error on the
+ * field, and there is no path where a wrong number is saved quietly.
+ */
+const priceResult = $derived(parsePrice(priceText, messageLocale()))
+const priceError = $derived(
+  priceResult.ok ? null : t('price_invalid', { example: formatPrice(123456, messageLocale()) }),
+)
 
-/** Minor units are the wire format; the form takes what somebody reads off the receipt. */
-function toPrice(raw: string): number | null {
-  if (!raw.trim()) return null
-  const parsed = Number.parseFloat(raw.replace(',', '.'))
-  return Number.isFinite(parsed) && parsed >= 0 ? Math.round(parsed * 100) : null
-}
+/**
+ * `disabled={save.isPending}` does not stop the second click.
+ *
+ * The attribute reaches the button on the next render and two quick clicks are one render apart, so
+ * a double-click files two assets — two rows, two tags, one of them a duplicate somebody has to go
+ * and find. A plain flag set in the same tick as the click is what actually guards it.
+ */
+let submitting = $state(false)
+
+/** What both procedures take; `update` adds the id and takes it as a patch. */
+const fields = () => ({
+  name: name.trim(),
+  description: description.trim(),
+  serialNumber: serialNumber.trim() || null,
+  location: location.trim() || null,
+  purchasedFrom: purchasedFrom.trim() || null,
+  purchasedOn: purchasedOn || null,
+  warrantyUntil: warrantyUntil || null,
+  priceMinor: priceResult.ok ? priceResult.minor : null,
+  currency: currency.trim().toUpperCase() || null,
+})
 
-const create = createMutation(() => ({
-  mutationFn: () =>
-    api.assets.create({
-      workspaceId,
-      name: name.trim(),
-      description: description.trim(),
-      serialNumber: serialNumber.trim() || null,
-      location: location.trim() || null,
-      purchasedFrom: purchasedFrom.trim() || null,
-      purchasedOn: purchasedOn || null,
-      warrantyUntil: warrantyUntil || null,
-      priceMinor: toPrice(priceMinor),
-      currency: currency.trim().toUpperCase() || null,
-    }),
-  onSuccess: () => {
-    toast.success(t('new'))
-    void queryClient.invalidateQueries({ queryKey: ['inventory', 'assets'] })
-    reset()
+const save = createMutation(() => ({
+  mutationFn: () => {
+    const row = asset
+    return row
+      ? api.assets.update({ workspaceId, assetId: row.id, ...fields() })
+      : api.assets.create({ workspaceId, ...fields() })
+  },
+  onSuccess: (saved: Asset) => {
+    // Not `t('new')`: that is the label on the button that opened this, so the toast used to
+    // congratulate somebody on the words "New asset" rather than telling them what happened.
+    toast.success(t(editing ? 'updated_toast' : 'created_toast', { name: saved.name }))
+    void queryClient.invalidateQueries({ queryKey: inventoryKeys.all })
     close()
   },
-  onError: (error: Error) => toast.error(error.message),
+  onError: (error: Error) => toast.error(error.message || t('common.error')),
+  onSettled: () => {
+    submitting = false
+  },
 }))
 
-const canSubmit = $derived(Boolean(name.trim()) && canInventory('manage') && !create.isPending)
+/**
+ * `submitting` is deliberately **not** in here.
+ *
+ * It is the guard `submit()` checks, and putting it in the disabled expression as well disables the
+ * button under the finger of whoever just pressed it — the browser blurs a focused element the
+ * moment it becomes disabled and hands focus nowhere, so a keyboard user loses their place on the
+ * page mid-save. `loading={submitting}` says the same thing to a screen reader through `aria-busy`
+ * and moves nothing.
+ */
+const canSubmit = $derived(Boolean(name.trim()) && priceResult.ok && canInventory('manage'))
+
+function submit() {
+  if (submitting || !canSubmit) return
+  submitting = true
+  save.mutate()
+}
 </script>
 
 <Dialog
   bind:open={shown}
-  title={t('new')}
+  title={editing ? t('edit') : t('new')}
   onOpenChange={(next) => {
     if (!next) close()
   }}
@@ -135,12 +209,17 @@ const canSubmit = $derived(Boolean(name.trim()) && canInventory('manage') && !cr
         {/snippet}
       </Field>
       <div class="price">
+        <!-- The error goes on the `Input`, not the `Field`: it is the half that also sets
+             `aria-invalid` and points `aria-describedby` at the message, and putting it on both
+             would print the sentence twice. -->
         <Field label={t('price')} id="inv-price">
           {#snippet children(id)}
-            <Input {id} type="text" inputmode="decimal" bind:value={priceMinor} />
+            <Input {id} type="text" inputmode="decimal" bind:value={priceText} error={priceError} />
           {/snippet}
         </Field>
-        <Field label="ISO" id="inv-currency" hint="USD">
+        <!-- Was `label="ISO" hint="USD"`: an untranslated abbreviation labelling a control on a
+             Persian, Arabic, German or Turkish form, explained by an example that is not a hint. -->
+        <Field label={t('currency')} id="inv-currency">
           {#snippet children(id)}
             <Select
               {id}
@@ -161,7 +240,7 @@ const canSubmit = $derived(Boolean(name.trim()) && canInventory('manage') && !cr
 
   {#snippet footer()}
     <Button variant="ghost" onclick={close}>{t('common.cancel')}</Button>
-    <Button onclick={() => create.mutate()} disabled={!canSubmit} loading={create.isPending}>
+    <Button onclick={submit} disabled={!canSubmit} loading={submitting}>
       {t('common.save')}
     </Button>
   {/snippet}
@@ -179,7 +258,11 @@ const canSubmit = $derived(Boolean(name.trim()) && canInventory('manage') && !cr
   }
   .price {
     display: grid;
-    grid-template-columns: 1fr 90px;
+    /* 90px fitted "ISO" and nothing else — "Para birimi" and «واحد پول» need room to sit on one
+       line, and a wrapped two-word label beside a one-line one knocks the two controls out of
+       alignment. */
+    grid-template-columns: minmax(0, 1fr) 124px;
     gap: 8px;
+    align-items: start;
   }
 </style>

package/src/client/i18n.ts

@@ -1,169 +1,14 @@
-import { type Message, scopedT } from '@kernhq/ui'
-
-/**
- * This module's own strings, in every locale the platform ships.
- *
- * A module ships separately from the app, so Paraglide — which compiles only the app's
- * `messages/*.json` — cannot see these. The shell merges them into the framework's message runtime
- * when it registers the module, and `t()` resolves against the merged map. Keys are namespaced by
- * module id, which is what keeps two modules from colliding in that one map.
- *
- * A **counted** message is not a string with `{count}` in it: give it a map of CLDR plural category
- * to string and `t(key, { n })` picks the form. English has two and Arabic has six, and which one
- * applies is `Intl.PluralRules`' answer rather than yours.
- *
- * Words every module needs and none owns — Save, Cancel, Retry — come from the framework's `common`
- * bundle (`t('common.save')`). Do not copy them here; six translations of "Save" drift apart.
- */
-
-export const en: Record<string, Message> = {
-  'inventory.nav': 'Inventory',
-  'inventory.title': 'Assets',
-  'inventory.empty': 'No assets yet',
-  'inventory.empty_desc': 'Add the first laptop, phone or desk — anything the company owns.',
-  'inventory.new': 'New asset',
-  'inventory.edit': 'Edit asset',
-  'inventory.code': 'Asset tag',
-  'inventory.name': 'Name',
-  'inventory.name_placeholder': 'MacBook Pro 14", office chair…',
-  'inventory.description': 'Description',
-  'inventory.category': 'Category',
-  'inventory.serial_number': 'Serial number',
-  'inventory.location': 'Location',
-  'inventory.purchased_on': 'Purchase date',
-  'inventory.purchased_from': 'Purchased from',
-  'inventory.price': 'Price',
-  'inventory.warranty_until': 'Warranty until',
-  'inventory.status_in_stock': 'In stock',
-  'inventory.status_assigned': 'Assigned',
-  'inventory.status_under_repair': 'Under repair',
-  'inventory.status_retired': 'Retired',
-  'inventory.count': { one: '{n} asset', other: '{n} assets' },
-  'inventory.search_placeholder': 'Search by name, tag or serial…',
-  'inventory.filter_all_statuses': 'All statuses',
-  'inventory.archived': 'Archived',
-  'inventory.archive': 'Archive',
-  'inventory.restore': 'Restore',
-  'inventory.widget_title': 'Recent assets',
-  'inventory.widget_desc': 'The most recently added assets in this workspace.',
-}
-
-export type InventoryMessageKey = keyof typeof en
-
-export const fa: Record<string, Message> = {
-  'inventory.nav': 'اموال',
-  'inventory.title': 'اقلام',
-  'inventory.empty': 'هنوز قلمی ثبت نشده است',
-  'inventory.empty_desc': 'اولین لپ‌تاپ، گوشی یا میز را اضافه کنید — هر چیزی که شرکت دارد.',
-  'inventory.new': 'قلم جدید',
-  'inventory.edit': 'ویرایش قلم',
-  'inventory.code': 'کد قلم',
-  'inventory.name': 'نام',
-  'inventory.name_placeholder': 'مک‌بوک پرو ۱۴ اینچ، صندلی اداری…',
-  'inventory.description': 'توضیحات',
-  'inventory.category': 'دسته',
-  'inventory.serial_number': 'شماره سریال',
-  'inventory.location': 'محل استقرار',
-  'inventory.purchased_on': 'تاریخ خرید',
-  'inventory.purchased_from': 'خریداری از',
-  'inventory.price': 'قیمت',
-  'inventory.warranty_until': 'گارانتی تا',
-  'inventory.status_in_stock': 'در انبار',
-  'inventory.status_assigned': 'تحویل‌شده',
-  'inventory.status_under_repair': 'در تعمیر',
-  'inventory.status_retired': 'اسقاط‌شده',
-  'inventory.count': { one: '{n} قلم', other: '{n} قلم' },
-  'inventory.search_placeholder': 'جستجو بر اساس نام، کد یا سریال…',
-  'inventory.filter_all_statuses': 'همه وضعیت‌ها',
-  'inventory.archived': 'بایگانی‌شده',
-  'inventory.archive': 'بایگانی',
-  'inventory.restore': 'بازگردانی',
-  'inventory.widget_title': 'اقلام اخیر',
-  'inventory.widget_desc': 'آخرین اقلامی که به این فضای کاری اضافه شده‌اند.',
-}
-
-export const ar: Record<string, Message> = {
-  'inventory.nav': 'الأصول',
-  'inventory.title': 'الأصول',
-  'inventory.empty': 'لا أصول بعد',
-  'inventory.empty_desc': 'أضف أول حاسوب أو هاتف أو مكتب — أي شيء تملكه الشركة.',
-  'inventory.new': 'أصل جديد',
-  'inventory.edit': 'تعديل الأصل',
-  'inventory.code': 'رمز الأصل',
-  'inventory.name': 'الاسم',
-  'inventory.name_placeholder': 'حاسوب ماك بوك برو ١٤، كرسي مكتبي…',
-  'inventory.description': 'الوصف',
-  'inventory.category': 'الفئة',
-  'inventory.serial_number': 'الرقم التسلسلي',
-  'inventory.location': 'الموقع',
-  'inventory.purchased_on': 'تاريخ الشراء',
-  'inventory.purchased_from': 'تم الشراء من',
-  'inventory.price': 'السعر',
-  'inventory.warranty_until': 'الضمان حتى',
-  'inventory.status_in_stock': 'في المخزن',
-  'inventory.status_assigned': 'مُسلَّم',
-  'inventory.status_under_repair': 'قيد الإصلاح',
-  'inventory.status_retired': 'مستبعد',
-  'inventory.count': {
-    zero: 'لا أصول',
-    one: 'أصل واحد',
-    two: 'أصلان',
-    few: '{n} أصول',
-    many: '{n} أصلاً',
-    other: '{n} أصل',
-  },
-  'inventory.search_placeholder': 'ابحث بالاسم أو الرمز أو الرقم التسلسلي…',
-  'inventory.filter_all_statuses': 'جميع الحالات',
-  'inventory.archived': 'مؤرشف',
-  'inventory.archive': 'أرشفة',
-  'inventory.restore': 'استعادة',
-  'inventory.widget_title': 'أصول حديثة',
-  'inventory.widget_desc': 'الأصول المضافة أخيرًا إلى مساحة العمل هذه.',
-}
-
-export const de: Record<string, Message> = {
-  'inventory.nav': 'Inventar',
-  'inventory.title': 'Gegenstände',
-  'inventory.empty': 'Noch keine Gegenstände',
-  'inventory.empty_desc':
-    'Fügen Sie den ersten Laptop, das erste Telefon oder den ersten Schreibtisch hinzu — alles, was die Firma besitzt.',
-  'inventory.new': 'Neuer Gegenstand',
-  'inventory.edit': 'Gegenstand bearbeiten',
-  'inventory.code': 'Inventarnummer',
-  'inventory.name': 'Name',
-  'inventory.name_placeholder': 'MacBook Pro 14", Bürostuhl…',
-  'inventory.description': 'Beschreibung',
-  'inventory.category': 'Kategorie',
-  'inventory.serial_number': 'Seriennummer',
-  'inventory.location': 'Standort',
-  'inventory.purchased_on': 'Kaufdatum',
-  'inventory.purchased_from': 'Gekauft bei',
-  'inventory.price': 'Preis',
-  'inventory.warranty_until': 'Garantie bis',
-  'inventory.status_in_stock': 'Auf Lager',
-  'inventory.status_assigned': 'Zugewiesen',
-  'inventory.status_under_repair': 'In Reparatur',
-  'inventory.status_retired': 'Ausgemustert',
-  'inventory.count': { one: '{n} Gegenstand', other: '{n} Gegenstände' },
-  'inventory.search_placeholder': 'Nach Name, Nummer oder Seriennummer suchen…',
-  'inventory.filter_all_statuses': 'Alle Zustände',
-  'inventory.archived': 'Archiviert',
-  'inventory.archive': 'Archivieren',
-  'inventory.restore': 'Wiederherstellen',
-  'inventory.widget_title': 'Neue Gegenstände',
-  'inventory.widget_desc': 'Die zuletzt hinzugefügten Gegenstände dieses Arbeitsbereichs.',
-}
-
-/**
- * Add a locale by adding a bundle here. Bundles are thunks so a locale is fetched only when it is
- * the one in use; English is the fallback and is therefore always loaded.
- */
-export const inventoryMessageBundles = {
-  ar: async () => ar,
-  de: async () => de,
-  en: async () => en,
-  fa: async () => fa,
-}
+import { scopedT } from '@kernhq/ui'
+
+export {
+  ar,
+  de,
+  en,
+  fa,
+  type InventoryMessageKey,
+  inventoryMessageBundles,
+  tr,
+} from './messages.js'
 
 /** `t('nav')` — the module id is implied. `t('common.save')` still reaches the shared bundle. */
 export const t = scopedT('inventory')

package/src/client/index.ts

@@ -17,7 +17,14 @@
  * `files` in package.json must cover every directory this entry reaches, contract source included.
  */
 
-export { type Asset, type AssetStatus, inventoryPermissions, MODULE_ID } from '../contract.js'
+export {
+  type Asset,
+  type AssetStatus,
+  InventorySettings,
+  inventoryCapabilities,
+  inventoryPermissions,
+  MODULE_ID,
+} from '../contract/index.js'
 export { createInventoryClient, type InventoryApi } from './api.js'
 export { __setInventoryApi, getInventoryApi } from './api-instance.js'
 export { type InventoryMessageKey, inventoryMessageBundles, t } from './i18n.js'

package/src/client/messages.test.ts

@@ -0,0 +1,169 @@
+import { describe, expect, it } from 'vitest'
+import { ar, de, en, fa, inventoryMessageBundles, tr } from './messages.js'
+
+/**
+ * The bundles, structurally — and the counted messages, behaviourally.
+ *
+ * Nothing else looks at these. A key present in English and missing in Persian type-checks, builds,
+ * lints and ships, and the first person to see it is the one reading `inventory.status_lost` on a
+ * screen. Adding a status to `AssetStatus` and forgetting four translations is the exact shape of
+ * that mistake, and this file is what makes it fail here instead.
+ */
+const BUNDLES = { en, ar, de, fa, tr } as const
+type Locale = keyof typeof BUNDLES
+
+/** The count placeholder: the runtime accepts `count` or `n`, and this module's catalogue uses `n`. */
+const COUNT = 'n'
+const placeholders = (s: string) => new Set([...s.matchAll(/\{(\w+)\}/g)].map((m) => m[1]!))
+const forms = (v: unknown): string[] => (typeof v === 'string' ? [v] : Object.values(v as object))
+
+describe('bundles', () => {
+  it('declares every locale the module claims to ship', () => {
+    expect(Object.keys(inventoryMessageBundles).sort()).toEqual(['ar', 'de', 'en', 'fa', 'tr'])
+  })
+
+  it('has the same key set in every locale', () => {
+    const keys = Object.keys(en).sort()
+    for (const [locale, bundle] of Object.entries(BUNDLES)) {
+      expect({ locale, keys: Object.keys(bundle).sort() }).toEqual({ locale, keys })
+    }
+  })
+
+  it('namespaces every key, so two modules cannot collide in the merged map', () => {
+    for (const key of Object.keys(en)) expect(key.startsWith('inventory.')).toBe(true)
+  })
+
+  /**
+   * A plural form may drop the numeral — "one asset" reads better than "1 asset" in some
+   * languages — but dropping any *other* placeholder loses information the sentence needed.
+   */
+  it('keeps every non-count placeholder in every plural form', () => {
+    for (const [key, value] of Object.entries(en)) {
+      const expected = placeholders(forms(value).join(' '))
+      expected.delete(COUNT)
+      for (const [locale, bundle] of Object.entries(BUNDLES)) {
+        for (const form of forms(bundle[key as keyof typeof bundle])) {
+          const got = placeholders(form)
+          got.delete(COUNT)
+          expect({ key, locale, form, got: [...got].sort() }).toEqual({
+            key,
+            locale,
+            form,
+            got: [...expected].sort(),
+          })
+        }
+      }
+    }
+  })
+
+  it('never invents a placeholder the English string does not have', () => {
+    for (const [key, value] of Object.entries(en)) {
+      const allowed = placeholders(forms(value).join(' '))
+      for (const [locale, bundle] of Object.entries(BUNDLES))
+        for (const form of forms(bundle[key as keyof typeof bundle]))
+          for (const name of placeholders(form))
+            expect({ key, locale, name, known: allowed.has(name) }).toEqual({
+              key,
+              locale,
+              name,
+              known: true,
+            })
+    }
+  })
+
+  /** Arabic inflects six ways. A bundle with only one/other picks `other` for two, and reads wrong. */
+  it('gives every Arabic plural all six CLDR categories', () => {
+    const wanted = new Intl.PluralRules('ar').resolvedOptions().pluralCategories.sort()
+    for (const [key, value] of Object.entries(ar)) {
+      if (typeof value === 'string') continue
+      expect({ key, cats: Object.keys(value).sort() }).toEqual({ key, cats: wanted })
+    }
+  })
+
+  it('makes a plural of a key wherever English has one', () => {
+    for (const [key, value] of Object.entries(en)) {
+      if (typeof value === 'string') continue
+      for (const [locale, bundle] of Object.entries(BUNDLES))
+        expect({ key, locale, plural: typeof bundle[key as keyof typeof bundle] }).toEqual({
+          key,
+          locale,
+          plural: 'object',
+        })
+    }
+  })
+})
+
+/**
+ * Every status the contract can hold has a label, in every language.
+ *
+ * The screens render a status with `t(`status_${status}`)`, which cannot be type-checked: a status
+ * added to the enum and not to the catalogue renders the literal key `inventory.status_lost` in the
+ * middle of a table, in every locale at once.
+ */
+describe('status labels', () => {
+  // Not imported from the contract: this file is data-only on purpose, and a copy of six strings
+  // that must match is exactly what a test is for.
+  const STATUSES = ['in_stock', 'assigned', 'reserved', 'under_repair', 'lost', 'retired']
+
+  it('labels every asset status in every locale', () => {
+    for (const status of STATUSES)
+      for (const [locale, bundle] of Object.entries(BUNDLES))
+        expect({ status, locale, has: `inventory.status_${status}` in bundle }).toEqual({
+          status,
+          locale,
+          has: true,
+        })
+  })
+})
+
+/**
+ * The counted messages, resolved the way the runtime resolves them.
+ *
+ * `t()` lives in `@kernhq/ui`, whose entry point pulls in Svelte components this package's test
+ * setup cannot transform. So these drive the *data* through the same `Intl.PluralRules` selection
+ * `selectPlural` performs, and `t()` itself is tested where it lives.
+ */
+describe('counted messages', () => {
+  const pick = (locale: Locale, key: string, count: number): string => {
+    const value = BUNDLES[locale][key as keyof (typeof BUNDLES)[Locale]] as
+      | string
+      | Partial<Record<Intl.LDMLPluralRule, string>>
+    if (typeof value === 'string') return value
+    const category = new Intl.PluralRules(locale).select(count)
+    const form = value[category] ?? value.other
+    if (form === undefined) throw new Error(`${locale} ${key} has no form for ${count}`)
+    return form.replace(/\{n\}/g, new Intl.NumberFormat(locale).format(count))
+  }
+
+  it('says "1 asset" and "2 assets" in English', () => {
+    expect(pick('en', 'inventory.count', 1)).toBe('1 asset')
+    expect(pick('en', 'inventory.count', 2)).toBe('2 assets')
+  })
+
+  it('says "1 Gegenstand" and "2 Gegenstände" in German', () => {
+    expect(pick('de', 'inventory.count', 1)).toBe('1 Gegenstand')
+    expect(pick('de', 'inventory.count', 2)).toBe('2 Gegenstände')
+  })
+
+  it('follows Arabic numeral agreement rather than one-or-many', () => {
+    expect(pick('ar', 'inventory.count', 0)).toBe('لا أصول')
+    expect(pick('ar', 'inventory.count', 1)).toBe('أصل واحد')
+    expect(pick('ar', 'inventory.count', 2)).toBe('أصلان')
+    expect(pick('ar', 'inventory.count', 5)).toContain('أصول')
+  })
+
+  it('does not inflect the noun after a numeral in Turkish or Persian', () => {
+    // "5 demirbaş", not "5 demirbaşlar" — the plural suffix is wrong once a number is present.
+    expect(pick('tr', 'inventory.count', 1).replace('1', '')).toBe(
+      pick('tr', 'inventory.count', 5).replace('5', ''),
+    )
+    expect(pick('fa', 'inventory.count', 1).replace('۱', '')).toBe(
+      pick('fa', 'inventory.count', 5).replace('۵', ''),
+    )
+  })
+
+  it('formats the number in the reader’s own digits', () => {
+    expect(pick('fa', 'inventory.count', 5)).toContain('۵')
+    expect(pick('en', 'inventory.count', 5)).toContain('5')
+  })
+})