@astryxdesign/core 0.7.1 → 0.7.2-canary.374b53a

package/src/i18n/__tests__/InternationalizationProvider.test.tsx

@@ -0,0 +1,128 @@
+// Copyright (c) Meta Platforms, Inc. and affiliates.
+
+/**
+ * @file InternationalizationProvider.test.tsx
+ * @input packages/core/src/i18n/InternationalizationProvider.tsx, useTranslator,
+ *   useDirection, useLocale
+ * @output Unit tests for the provider's update and nesting contract
+ * @position Colocated tests; targets re-rendering when messages, overrides, or
+ *   dir change (locale is covered by useLocale/useTranslator tests) and a nested
+ *   provider replacing rather than merging its parent's context.
+ */
+
+import {describe, expect, test} from 'vitest';
+import {render} from '@testing-library/react';
+import {InternationalizationProvider} from '../InternationalizationProvider';
+import {useDirection} from '../useDirection';
+import {useLocale} from '../useLocale';
+import {useTranslator} from '../useTranslator';
+import type {MessagesByLocale, Overrides} from '../types';
+
+const NEXT = '@astryx.pagination.next';
+const APP_KEY = '@myapp.save';
+
+function Probe() {
+  const t = useTranslator();
+  return (
+    <>
+      <span data-testid="next">{t(NEXT)}</span>
+      <span data-testid="app">{t(APP_KEY)}</span>
+      <span data-testid="locale">{useLocale()}</span>
+      <span data-testid="dir">{useDirection()}</span>
+    </>
+  );
+}
+
+const FR_MESSAGES: MessagesByLocale = {
+  fr: {[APP_KEY]: {defaultMessage: 'Enregistrer'}},
+};
+const FR_MESSAGES_V2: MessagesByLocale = {
+  fr: {[APP_KEY]: {defaultMessage: 'Sauvegarder'}},
+};
+const FR_OVERRIDES: Overrides = {fr: {[NEXT]: 'Suivant'}};
+const FR_OVERRIDES_V2: Overrides = {fr: {[NEXT]: 'Page suivante'}};
+
+describe('InternationalizationProvider — re-rendering', () => {
+  test('re-renders consumers when messages change', () => {
+    const {getByTestId, rerender} = render(
+      <InternationalizationProvider locale="fr" messages={FR_MESSAGES}>
+        <Probe />
+      </InternationalizationProvider>,
+    );
+    expect(getByTestId('app').textContent).toBe('Enregistrer');
+
+    rerender(
+      <InternationalizationProvider locale="fr" messages={FR_MESSAGES_V2}>
+        <Probe />
+      </InternationalizationProvider>,
+    );
+    expect(getByTestId('app').textContent).toBe('Sauvegarder');
+  });
+
+  test('re-renders consumers when overrides change', () => {
+    const {getByTestId, rerender} = render(
+      <InternationalizationProvider locale="fr" overrides={FR_OVERRIDES}>
+        <Probe />
+      </InternationalizationProvider>,
+    );
+    expect(getByTestId('next').textContent).toBe('Suivant');
+
+    rerender(
+      <InternationalizationProvider locale="fr" overrides={FR_OVERRIDES_V2}>
+        <Probe />
+      </InternationalizationProvider>,
+    );
+    expect(getByTestId('next').textContent).toBe('Page suivante');
+  });
+
+  test('re-renders consumers when dir changes', () => {
+    const {getByTestId, rerender} = render(
+      <InternationalizationProvider locale="en" dir="rtl">
+        <Probe />
+      </InternationalizationProvider>,
+    );
+    expect(getByTestId('dir').textContent).toBe('rtl');
+
+    rerender(
+      <InternationalizationProvider locale="en" dir="ltr">
+        <Probe />
+      </InternationalizationProvider>,
+    );
+    expect(getByTestId('dir').textContent).toBe('ltr');
+  });
+});
+
+describe('InternationalizationProvider — nesting', () => {
+  test('a nested provider replaces the parent context instead of merging it', () => {
+    const {getByTestId} = render(
+      <InternationalizationProvider
+        locale="fr"
+        dir="rtl"
+        messages={FR_MESSAGES}
+        overrides={FR_OVERRIDES}>
+        <InternationalizationProvider locale="fr">
+          <Probe />
+        </InternationalizationProvider>
+      </InternationalizationProvider>,
+    );
+    // Locale is the child's own; direction derives from it, not the parent's
+    // explicit rtl; parent overrides and catalogs do not leak into the child.
+    expect(getByTestId('locale').textContent).toBe('fr');
+    expect(getByTestId('dir').textContent).toBe('ltr');
+    expect(getByTestId('next').textContent).toBe('Go to next page');
+    expect(getByTestId('app').textContent).toBe(APP_KEY);
+  });
+
+  test('a nested provider can narrow the locale for one subtree', () => {
+    const {getByTestId} = render(
+      <InternationalizationProvider locale="fr" overrides={FR_OVERRIDES}>
+        <InternationalizationProvider locale="de">
+          <Probe />
+        </InternationalizationProvider>
+      </InternationalizationProvider>,
+    );
+    expect(getByTestId('locale').textContent).toBe('de');
+    // No de override or catalog: shipped English wins.
+    expect(getByTestId('next').textContent).toBe('Go to next page');
+  });
+});

package/src/i18n/generated-locales/en.generated.ts

@@ -158,6 +158,27 @@ const runtimeCatalog: Readonly<Record<string, string>> = {
     '{itemName} dropped at position {position, number} of {total, number}.',
   '@astryx.listInput.announceAlreadyAtBoundary':
     'This {itemName} is already {boundary, select, first {first} last {last} other {}}.',
+  '@astryx.diffViewer.label': 'Diff',
+  '@astryx.diffViewer.renamedPath': '{oldPath} → {newPath}',
+  '@astryx.diffViewer.statusAdded': 'Added',
+  '@astryx.diffViewer.statusDeleted': 'Deleted',
+  '@astryx.diffViewer.statusRenamed': 'Renamed',
+  '@astryx.diffViewer.addedCount': '+{count, number}',
+  '@astryx.diffViewer.removedCount': '−{count, number}',
+  '@astryx.diffViewer.stats':
+    '{added, plural, one {# line added} other {# lines added}}, {removed, plural, one {# line removed} other {# lines removed}}',
+  '@astryx.diffViewer.added': 'Added',
+  '@astryx.diffViewer.removed': 'Removed',
+  '@astryx.diffViewer.lineNumberButton':
+    '{side, select, old {Original line {line, number}} other {Changed line {line, number}}}',
+  '@astryx.diffViewer.columnOldLine': 'Original line number',
+  '@astryx.diffViewer.columnNewLine': 'Changed line number',
+  '@astryx.diffViewer.columnChange': 'Change',
+  '@astryx.diffViewer.columnCode': 'Code',
+  '@astryx.diffViewer.columnOldCode': 'Original code',
+  '@astryx.diffViewer.columnNewCode': 'Changed code',
+  '@astryx.diffViewer.binary': 'Binary file not shown',
+  '@astryx.diffViewer.noChanges': 'No content changes',
   '@astryx.markdownHeadingLinks.copied': 'Link copied',
   '@astryx.markdownHeadingLinks.copy': 'Copy link to {heading}',
   '@astryx.richTextEditor.tabEscapeHint':

package/src/i18n/generated-locales/pseudo.generated.ts

@@ -163,6 +163,27 @@ const runtimeCatalog: Readonly<Record<string, string>> = {
     '⟦{itemName} ðřóþþéð àţ þóšíţíóñ {position, number} óƒ {total, number}.⟧',
   '@astryx.listInput.announceAlreadyAtBoundary':
     '⟦Ţĥíš {itemName} íš àłřéàðý {boundary, select, first {first} last {last} other {}}.⟧',
+  '@astryx.diffViewer.label': '⟦Ð탃⟧',
+  '@astryx.diffViewer.renamedPath': '⟦{oldPath} → {newPath}⟧',
+  '@astryx.diffViewer.statusAdded': '⟦Àððéð⟧',
+  '@astryx.diffViewer.statusDeleted': '⟦Ðéłéţéð⟧',
+  '@astryx.diffViewer.statusRenamed': '⟦Řéñàɱéð⟧',
+  '@astryx.diffViewer.addedCount': '⟦+{count, number}⟧',
+  '@astryx.diffViewer.removedCount': '⟦−{count, number}⟧',
+  '@astryx.diffViewer.stats':
+    '⟦{added, plural, one {# line added} other {# lines added}}, {removed, plural, one {# line removed} other {# lines removed}}⟧',
+  '@astryx.diffViewer.added': '⟦Àððéð⟧',
+  '@astryx.diffViewer.removed': '⟦Řéɱóṽéð⟧',
+  '@astryx.diffViewer.lineNumberButton':
+    '⟦{side, select, old {Original line {line, number}} other {Changed line {line, number}}}⟧',
+  '@astryx.diffViewer.columnOldLine': '⟦Óříĝíñàł łíñé ñúɱƀéř⟧',
+  '@astryx.diffViewer.columnNewLine': '⟦Çĥàñĝéð łíñé ñúɱƀéř⟧',
+  '@astryx.diffViewer.columnChange': '⟦Çĥàñĝé⟧',
+  '@astryx.diffViewer.columnCode': '⟦Çóðé⟧',
+  '@astryx.diffViewer.columnOldCode': '⟦Óříĝíñàł çóðé⟧',
+  '@astryx.diffViewer.columnNewCode': '⟦Çĥàñĝéð çóðé⟧',
+  '@astryx.diffViewer.binary': '⟦Ɓíñàřý ƒíłé ñóţ šĥóŵñ⟧',
+  '@astryx.diffViewer.noChanges': '⟦Ñó çóñţéñţ çĥàñĝéš⟧',
   '@astryx.markdownHeadingLinks.copied': '⟦Łíñķ çóþíéð⟧',
   '@astryx.markdownHeadingLinks.copy': '⟦Çóþý łíñķ ţó {heading}⟧',
   '@astryx.richTextEditor.tabEscapeHint':

package/src/i18n/useTranslator.test.tsx

@@ -32,6 +32,12 @@ const FR_MESSAGES: MessagesByLocale = {
 const FR_RUNTIME_MESSAGES: ProviderMessagesByLocale = {
   fr: {'@astryx.pagination.next': 'Page suivante compacte'},
 };
+// One messages value mixing shapes: a generated string map for the active
+// locale and a rich catalog for its parent.
+const MIXED_MESSAGES: ProviderMessagesByLocale = {
+  'fr-CA': {'@astryx.pagination.next': 'Page suivante (CA)'},
+  fr: {'@astryx.pagination.previous': {defaultMessage: 'Page précédente'}},
+};
 const FR_OVERRIDES: Overrides = {
   fr: {'@astryx.pagination.next': 'Suivant'},
 };
@@ -101,6 +107,37 @@ describe('useTranslator — resolution', () => {
     ).toEqual({defaultMessage: 'Page suivante compacte'});
   });
 
+  it('resolves rich and runtime catalogs mixed in one messages value', () => {
+    const {result} = renderHook(
+      () => ({
+        context: use(InternationalizationContext),
+        translate: useTranslator(),
+      }),
+      {
+        wrapper: ({children}) => (
+          <InternationalizationProvider
+            locale="fr-CA"
+            messages={MIXED_MESSAGES}>
+            {children}
+          </InternationalizationProvider>
+        ),
+      },
+    );
+
+    expect(result.current.translate('@astryx.pagination.next')).toBe(
+      'Page suivante (CA)',
+    );
+    expect(result.current.translate('@astryx.pagination.previous')).toBe(
+      'Page précédente',
+    );
+    expect(result.current.context.messages).toEqual({
+      'fr-CA': {
+        '@astryx.pagination.next': {defaultMessage: 'Page suivante (CA)'},
+      },
+      fr: {'@astryx.pagination.previous': {defaultMessage: 'Page précédente'}},
+    });
+  });
+
   it('lets a provider override beat the provider catalog', () => {
     const {result} = renderHook(() => useTranslator(), {
       wrapper: ({children}) => (

package/src/utils/sharedMutationObserver.test.ts

@@ -0,0 +1,53 @@
+// Copyright (c) Meta Platforms, Inc. and affiliates.
+
+import {describe, it, expect, vi} from 'vitest';
+import {observeContent} from './sharedMutationObserver';
+
+const flush = async () => new Promise(resolve => setTimeout(resolve, 0));
+
+describe('observeContent', () => {
+  it('calls a registration once per batch for changes anywhere inside its element', async () => {
+    const element = document.createElement('div');
+    element.innerHTML = '<span>first</span>';
+    const callback = vi.fn();
+    const unsubscribe = observeContent(element, callback);
+
+    element.querySelector('span')!.firstChild!.nodeValue = 'second';
+    element.append('third');
+    await flush();
+
+    expect(callback).toHaveBeenCalledTimes(1);
+    unsubscribe();
+  });
+
+  it('removes only its own registration', async () => {
+    const element = document.createElement('div');
+    const kept = vi.fn();
+    const removed = vi.fn();
+    const unsubscribeKept = observeContent(element, kept);
+    const unsubscribeRemoved = observeContent(element, removed);
+
+    unsubscribeRemoved();
+    element.textContent = 'changed';
+    await flush();
+
+    expect(kept).toHaveBeenCalledTimes(1);
+    expect(removed).not.toHaveBeenCalled();
+    unsubscribeKept();
+  });
+
+  it('ignores a released element', async () => {
+    const released = document.createElement('div');
+    const live = document.createElement('div');
+    const releasedCallback = vi.fn();
+    const unsubscribeReleased = observeContent(released, releasedCallback);
+    const unsubscribeLive = observeContent(live, vi.fn());
+
+    unsubscribeReleased();
+    released.textContent = 'changed';
+    await flush();
+
+    expect(releasedCallback).not.toHaveBeenCalled();
+    unsubscribeLive();
+  });
+});

package/src/utils/sharedMutationObserver.ts

@@ -0,0 +1,78 @@
+// Copyright (c) Meta Platforms, Inc. and affiliates.
+
+'use client';
+
+/**
+ * @file sharedMutationObserver.ts
+ * @input MutationObserver API
+ * @output Exports observeContent for shared content-change observation
+ * @position Internal utility; consumed by useTruncation. The content-change
+ *   counterpart of sharedResizeObserver: one observer for every element, so a
+ *   repeated cell or row path does not create one observer per instance.
+ */
+
+type ContentCallback = () => void;
+
+const OPTIONS: MutationObserverInit = {
+  characterData: true,
+  childList: true,
+  subtree: true,
+};
+
+let observer: MutationObserver | null = null;
+const callbacks = new Map<Node, Set<ContentCallback>>();
+
+function dispatch(records: MutationRecord[]): void {
+  // One call per callback per batch, however many records it received. A
+  // record's target can be any descendant, so walk up to the observed roots.
+  const pending = new Set<ContentCallback>();
+  for (const record of records) {
+    for (let node: Node | null = record.target; node; node = node.parentNode) {
+      callbacks.get(node)?.forEach(callback => pending.add(callback));
+    }
+  }
+  pending.forEach(callback => callback());
+}
+
+/**
+ * Call `callback` when the element's text or children change (including
+ * descendants). Returns an unsubscribe that removes only this registration.
+ *
+ * MutationObserver has no per-target unobserve, so a released element stays
+ * registered with the shared observer until no element remains; its records
+ * match no callback and are ignored. The observer disconnects and is released
+ * when the last registration goes.
+ */
+export function observeContent(
+  element: Element,
+  callback: ContentCallback,
+): () => void {
+  if (typeof MutationObserver === 'undefined') {
+    return () => {};
+  }
+  observer ??= new MutationObserver(dispatch);
+  const existing = callbacks.get(element);
+  if (existing) {
+    existing.add(callback);
+  } else {
+    callbacks.set(element, new Set([callback]));
+    observer.observe(element, OPTIONS);
+  }
+
+  let isSubscribed = true;
+  return () => {
+    if (!isSubscribed) {
+      return;
+    }
+    isSubscribed = false;
+    const targetCallbacks = callbacks.get(element);
+    targetCallbacks?.delete(callback);
+    if (targetCallbacks?.size === 0) {
+      callbacks.delete(element);
+    }
+    if (callbacks.size === 0 && observer) {
+      observer.disconnect();
+      observer = null;
+    }
+  };
+}