@astryxdesign/core 0.1.4-canary.37c19d3 → 0.1.4-canary.39ffd21

package/src/Table/BaseTable.tsx

@@ -284,12 +284,6 @@ function areRowPropsEqual<T extends Record<string, unknown>>(
   const nextItem = nextProps.item;
   const keys = Object.keys(nextItem);
 
-  // A key deleted from nextItem would never be visited by the loop below,
-  // and the row would keep rendering the removed field's stale value.
-  if (Object.keys(prevItem).length !== keys.length) {
-    return false;
-  }
-
   for (const key of keys) {
     if (prevItem[key] !== nextItem[key]) {
       return false;
@@ -371,13 +365,9 @@ function BaseTableInner<T extends Record<string, unknown>>({
   const headerCells = resolvedColumns.map((col, columnIndex) => {
     const headerContent = col.header ?? col.key;
 
-    // Build initial htmlProps with column alignment if specified.
-    // `scope: 'col'` is the default so every column header associates its
-    // data cells with the correct column for screen readers; a plugin's
-    // transformHeaderCell can override it via htmlProps.scope.
+    // Build initial htmlProps with column alignment if specified
     const initialHeaderHtmlProps: Record<string, unknown> = {
       'data-column-key': col.key,
-      scope: 'col',
     };
     if (col.align) {
       initialHeaderHtmlProps.style = {textAlign: col.align};

package/src/Table/Table.test.tsx

@@ -229,34 +229,6 @@ describe('BaseTable', () => {
     expect(headers[2]).toHaveTextContent('Email');
   });
 
-  it('renders every column header with scope="col"', () => {
-    render(<BaseTable data={users} columns={columns} />);
-    const headers = screen.getAllByRole('columnheader');
-    expect(headers).toHaveLength(3);
-    for (const header of headers) {
-      expect(header).toHaveAttribute('scope', 'col');
-    }
-  });
-
-  it('lets a consumer override scope via header html props', () => {
-    // A `<th scope="row">` maps to the `rowheader` role, so query the DOM
-    // directly to assert the attribute regardless of the resolved ARIA role.
-    const plugin: TablePlugin<User> = {
-      transformHeaderCell: (props, column) =>
-        column.key === 'name'
-          ? {...props, htmlProps: {...props.htmlProps, scope: 'row'}}
-          : props,
-    };
-    const {container} = render(
-      <BaseTable data={users} columns={columns} plugins={[plugin]} />,
-    );
-    const headerCells = container.querySelectorAll('thead th');
-    // columns fixture order: name, age, email — plugin overrides name → 'row'
-    expect(headerCells[0]).toHaveAttribute('scope', 'row');
-    expect(headerCells[1]).toHaveAttribute('scope', 'col');
-    expect(headerCells[2]).toHaveAttribute('scope', 'col');
-  });
-
   it('renders data cells', () => {
     render(<BaseTable data={users} columns={columns} />);
     const cells = screen.getAllByRole('cell');
@@ -645,29 +617,6 @@ describe('BaseTable', () => {
 // =============================================================================
 
 describe('Table', () => {
-  it('clears a cell when the field is removed from the row object (#3595)', () => {
-    const cols: TableColumn<Record<string, unknown>>[] = [
-      {key: 'name', header: 'Name'},
-      {key: 'status', header: 'Status'},
-    ];
-    const {rerender} = render(
-      <Table
-        data={[{id: '1', name: 'Alice', status: 'active'}]}
-        columns={cols}
-        idKey="id"
-      />,
-    );
-    expect(screen.getByText('active')).toBeInTheDocument();
-
-    // Clearing a field by omitting it (optimistic update / server response)
-    // must re-render the row — the memo previously only compared the keys of
-    // the NEW item, so the deleted field's stale value kept rendering.
-    rerender(
-      <Table data={[{id: '1', name: 'Alice'}]} columns={cols} idKey="id" />,
-    );
-    expect(screen.queryByText('active')).not.toBeInTheDocument();
-  });
-
   it('renders a table with correct structure', () => {
     render(<Table data={users} columns={columns} />);
     expect(screen.getByRole('table')).toBeInTheDocument();

package/src/Table/plugins/columnResize/useTableColumnResize.test.tsx

@@ -551,19 +551,6 @@ describe('useTableColumnResize', () => {
       expect(handle).toHaveAttribute('aria-valuenow', '200');
     });
 
-    it('handle has a numeric aria-valuenow on initial render (before any width is measured)', () => {
-      // A focusable role="separator" requires aria-valuenow per the ARIA spec.
-      // Before the column width has been measured there is no override width, so
-      // the handle must fall back to a defined numeric value (the column
-      // minWidth) rather than omitting the attribute.
-      render(<ResizeTable minWidth={80} />);
-      const handle = getResizeHandles()[0];
-      const valueNow = handle.getAttribute('aria-valuenow');
-      expect(valueNow).not.toBeNull();
-      expect(Number.isNaN(Number(valueNow))).toBe(false);
-      expect(valueNow).toBe('80');
-    });
-
     it('handle has aria-label with column header text', () => {
       render(<ResizeTable />);
       const handle = getResizeHandles()[0];

package/src/Table/plugins/columnResize/useTableColumnResize.tsx

@@ -676,9 +676,7 @@ function ResizeHandle({
     <div
       role="separator"
       aria-orientation="vertical"
-      // A focusable role="separator" requires a numeric aria-valuenow, so fall
-      // back to the column's minWidth before the width has been measured.
-      aria-valuenow={currentWidth ?? minWidth}
+      aria-valuenow={currentWidth ?? undefined}
       aria-valuemin={minWidth}
       aria-valuemax={maxWidth === Infinity ? undefined : maxWidth}
       aria-label={ariaLabel}

package/src/Table/plugins/pagination/paginateData.ts

@@ -38,10 +38,6 @@ export function paginateData<T>(
   const size = Number.isFinite(pageSize)
     ? Math.max(1, Math.floor(pageSize))
     : 10;
-  // page needs the same coercion: a negative start index makes Array.slice
-  // count from the END of the data (page -1 returns the tail dressed up as a
-  // page), and a fractional page returns a slice straddling two pages.
-  const p = Number.isFinite(page) ? Math.max(1, Math.floor(page)) : 1;
-  const start = (p - 1) * size;
+  const start = (page - 1) * size;
   return data.slice(start, start + size);
 }

package/src/Table/plugins/pagination/useTablePagination.test.tsx

@@ -132,19 +132,6 @@ describe('paginateData', () => {
     expect(paginateData([], 1, 10)).toEqual([]);
   });
 
-  it('clamps invalid page numbers to the first page (#3593)', () => {
-    const data = generateItems(30);
-    // Negative pages previously fed a negative index to Array.slice, which
-    // counts from the END of the data — the tail dressed up as a page.
-    expect(paginateData(data, -1, 10)[0].id).toBe('1');
-    expect(paginateData(data, 0, 10)[0].id).toBe('1');
-    expect(paginateData(data, NaN, 10)[0].id).toBe('1');
-    // Fractional pages floor to the containing page instead of straddling two.
-    expect(paginateData(data, 1.5, 10).map(i => i.id)).toEqual(
-      paginateData(data, 1, 10).map(i => i.id),
-    );
-  });
-
   it('returns empty array when page exceeds data', () => {
     const data = generateItems(10);
     expect(paginateData(data, 5, 10)).toEqual([]);

package/src/Table/plugins/selection/useTableSelectionState.test.tsx

@@ -302,29 +302,6 @@ describe('useTableSelectionState with filtered data', () => {
     expect(screen.getByTestId('selected-keys')).toHaveTextContent('1,3');
   });
 
-  it('select-all is unchecked when the filter matches nothing (#3591)', async () => {
-    const user = userEvent.setup();
-    render(<FilteredSelectionTable />);
-
-    // Select Bob, then filter to an empty result set.
-    const checkboxes = screen.getAllByLabelText('Select row');
-    await user.click(checkboxes[1]);
-    expect(screen.getByTestId('selected-keys')).toHaveTextContent('2');
-
-    const input = screen.getByTestId('filter-input');
-    await user.clear(input);
-    await user.type(input, 'zzz');
-
-    // Zero visible rows must not read as "all selected" — previously the
-    // header checkbox rendered checked here and deselect-all was a no-op.
-    const selectAll = screen.getByLabelText('Select all rows');
-    expect(selectAll).not.toBeChecked();
-
-    // The hidden selection itself is preserved (frozen), as designed.
-    await user.clear(input);
-    expect(screen.getByTestId('selected-keys')).toHaveTextContent('2');
-  });
-
   it('selections persist when filter changes', async () => {
     const user = userEvent.setup();
     render(<FilteredSelectionTable />);

package/src/Table/plugins/selection/useTableSelectionState.tsx

@@ -125,17 +125,12 @@ export function useTableSelectionState<T extends Record<string, unknown>>(
     [selectedKeys, selectableIDs],
   );
 
-  const getAllSelected = useCallback(() => {
-    // Require at least one actionable row: with zero visible selectable
-    // rows (e.g. a filter with no matches while a selection exists), the
-    // union-based size comparison would read any non-empty selection as
-    // "all selected" — a checked header checkbox over an empty table that
-    // deselect-all can't clear, since the invisible keys count as frozen.
-    if (selectableIDs.size === 0) {
-      return false;
-    }
-    return allSelectableIDs.size === selectedKeys.size;
-  }, [allSelectableIDs, selectableIDs, selectedKeys]);
+  const getAllSelected = useCallback(
+    () =>
+      allSelectableIDs.size === selectedKeys.size &&
+      allSelectableIDs.size !== 0,
+    [allSelectableIDs, selectedKeys],
+  );
 
   // Use refs to keep onSelectAll stable
   const frozenSelectedIDsRef = useRef(frozenSelectedIDs);

package/src/Table/plugins/sortable/useTableSortableState.test.tsx

@@ -8,7 +8,7 @@
  */
 
 import {describe, it, expect, vi} from 'vitest';
-import {render, renderHook, screen, within} from '@testing-library/react';
+import {render, screen, within} from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import {useState} from 'react';
 import {Table} from '../../Table';
@@ -421,40 +421,6 @@ describe('useTableSortableState', () => {
     });
   });
 
-  describe('NaN handling', () => {
-    it('keeps valid numbers sorted when a NaN cell is present', () => {
-      const data: Employee[] = [
-        {id: '1', name: 'A', age: 5, department: 'X', salary: 1},
-        {id: '2', name: 'B', age: NaN, department: 'X', salary: 1},
-        {id: '3', name: 'C', age: 1, department: 'X', salary: 1},
-        {id: '4', name: 'D', age: 3, department: 'X', salary: 1},
-      ];
-      const {result} = renderHook(() =>
-        useTableSortableState({
-          data,
-          defaultSort: [{sortKey: 'age', direction: 'ascending'}],
-        }),
-      );
-      // NaN groups at the end like null; the valid numbers stay ordered.
-      expect(result.current.sortedData.map(e => e.age)).toEqual([1, 3, 5, NaN]);
-    });
-
-    it('sorts NaN to the start in descending order, like null', () => {
-      const data: Employee[] = [
-        {id: '1', name: 'A', age: 5, department: 'X', salary: 1},
-        {id: '2', name: 'B', age: NaN, department: 'X', salary: 1},
-        {id: '3', name: 'C', age: 1, department: 'X', salary: 1},
-      ];
-      const {result} = renderHook(() =>
-        useTableSortableState({
-          data,
-          defaultSort: [{sortKey: 'age', direction: 'descending'}],
-        }),
-      );
-      expect(result.current.sortedData.map(e => e.age)).toEqual([NaN, 5, 1]);
-    });
-  });
-
   describe('null/undefined handling', () => {
     it('sorts null values to the end', () => {
       const data: Employee[] = [

package/src/Table/plugins/sortable/useTableSortableState.tsx

@@ -168,18 +168,14 @@ function defaultCompare<T extends Record<string, unknown>>(
   const aVal = a[sortKey];
   const bVal = b[sortKey];
 
-  // null/undefined/NaN sort to the end. NaN must not reach the numeric fast
-  // path: a NaN comparator result reads as "equal" to Array.sort, which makes
-  // the comparator inconsistent and corrupts the order of the other rows.
-  const aMissing = aVal == null || (typeof aVal === 'number' && Number.isNaN(aVal));
-  const bMissing = bVal == null || (typeof bVal === 'number' && Number.isNaN(bVal));
-  if (aMissing && bMissing) {
+  // null/undefined sort to the end
+  if (aVal == null && bVal == null) {
     return 0;
   }
-  if (aMissing) {
+  if (aVal == null) {
     return 1;
   }
-  if (bMissing) {
+  if (bVal == null) {
     return -1;
   }
 

package/src/TextArea/TextArea.test.tsx

@@ -13,7 +13,7 @@ import {useState} from 'react';
 import {describe, it, expect, vi, beforeEach} from 'vitest';
 import {render, screen, fireEvent, waitFor} from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
-import {TestIcon} from '../__tests__/TestIcon';
+import {MagnifyingGlassIcon} from '@heroicons/react/24/outline';
 import {TextArea} from './TextArea';
 
 // Mock showPopover/hidePopover since jsdom does not implement them. Used by the
@@ -193,7 +193,7 @@ describe('TextArea', () => {
         label="Description"
         value=""
         onChange={() => {}}
-        startIcon={TestIcon}
+        startIcon={MagnifyingGlassIcon}
       />,
     );
     expect(screen.getByRole('textbox')).toBeInTheDocument();
@@ -577,7 +577,7 @@ describe('TextArea', () => {
           label="Notes"
           value=""
           onChange={() => {}}
-          startIcon={<TestIcon />}
+          startIcon={<MagnifyingGlassIcon />}
         />,
       );
 

package/src/TextInput/TextInput.test.tsx

@@ -12,7 +12,7 @@
 import {describe, it, expect, vi, beforeEach} from 'vitest';
 import {render, screen, fireEvent, waitFor} from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
-import {TestIcon} from '../__tests__/TestIcon';
+import {MagnifyingGlassIcon} from '@heroicons/react/24/outline';
 import {TextInput} from './TextInput';
 
 // Mock showPopover/hidePopover since jsdom does not implement them. Used by the
@@ -163,7 +163,7 @@ describe('TextInput', () => {
         label="Search"
         value=""
         onChange={() => {}}
-        startIcon={TestIcon}
+        startIcon={MagnifyingGlassIcon}
       />,
     );
     expect(screen.getByRole('textbox')).toBeInTheDocument();
@@ -482,7 +482,7 @@ describe('TextInput', () => {
           label="Search"
           value=""
           onChange={() => {}}
-          startIcon={<TestIcon />}
+          startIcon={<MagnifyingGlassIcon />}
         />,
       );
 

package/src/Thumbnail/Thumbnail.test.tsx

@@ -37,45 +37,11 @@ describe('Thumbnail', () => {
     expect(screen.getByRole('status')).toBeInTheDocument();
   });
 
-  it('exposes the label as an accessible name via a valid group role', () => {
+  it('uses label as accessible name on root', () => {
     render(<Thumbnail label="photo.png" data-testid="thumb" />);
-    // The accessible name must be carried by a valid named role (group),
-    // not by a bare aria-label on a generic div (aria-prohibited-attr).
-    const group = screen.getByRole('group', {name: 'photo.png'});
-    expect(group).toBe(screen.getByTestId('thumb'));
+    expect(screen.getByTestId('thumb')).toHaveAttribute('aria-label', 'photo.png');
   });
 
-  it('does not put aria-label on a generic (roleless) element', () => {
-    render(<Thumbnail label="photo.png" data-testid="thumb" />);
-    const thumb = screen.getByTestId('thumb');
-    // The labeled element must declare a role so the name is legitimate.
-    expect(thumb).toHaveAttribute('role', 'group');
-  });
-
-  it('keeps interactive children accessible while exposing the group name', () => {
-    const onRemove = vi.fn();
-    render(
-      <Thumbnail
-        src="/img.jpg"
-        alt="Clickable"
-        label="file.png"
-        onClick={vi.fn()}
-        onRemove={onRemove}
-      />,
-    );
-    // A group role (unlike img) must not hide descendant controls.
-    expect(
-      screen.getByRole('group', {name: 'file.png — Clickable'}),
-    ).toBeInTheDocument();
-    expect(
-      screen.getByRole('button', {name: 'Open file.png — Clickable'}),
-    ).toBeInTheDocument();
-    expect(
-      screen.getByRole('button', {name: 'Remove file.png — Clickable'}),
-    ).toBeInTheDocument();
-  });
-
-
   it('label is shown via tooltip, not as inline text', () => {
     render(<Thumbnail label="photo.png" data-testid="thumb" />);
     // Label should exist in DOM (tooltip) but not as a direct child text node

package/src/Thumbnail/Thumbnail.tsx

@@ -58,9 +58,8 @@ export interface ThumbnailProps extends BaseProps<HTMLDivElement> {
   alt?: string;
   /**
    * Accessible label for the thumbnail (e.g. file name).
-   * Not rendered visually — shown in a tooltip on hover, exposed as the
-   * accessible name of the thumbnail group, and used as the accessible
-   * name for the remove button.
+   * Not rendered visually — shown in a tooltip on hover and used
+   * as accessible name for the remove button.
    */
   label?: string;
   /**
@@ -297,7 +296,6 @@ export function Thumbnail({
     <div
       ref={ref}
       data-testid={testId}
-      role="group"
       aria-label={accessibleName}
       {...mergeProps(
         themeProps('thumbnail'),

package/src/TimeInput/TimeInput.test.tsx

@@ -160,43 +160,6 @@ describe('TimeInput', () => {
     expect(onChange).not.toHaveBeenCalled();
   });
 
-  it('sets aria-invalid="true" when typed input is out of range', () => {
-    render(<TimeInput label="Time" onChange={() => {}} />);
-
-    const input = screen.getByRole('textbox');
-    fireEvent.change(input, {target: {value: '25:99'}});
-
-    expect(input).toHaveAttribute('aria-invalid', 'true');
-  });
-
-  it('does not set aria-invalid when typed input is a valid time', () => {
-    render(<TimeInput label="Time" onChange={() => {}} />);
-
-    const input = screen.getByRole('textbox');
-    fireEvent.change(input, {target: {value: '3:45 pm'}});
-
-    expect(input).not.toHaveAttribute('aria-invalid');
-  });
-
-  it('announces an alert message when typed input is invalid', () => {
-    render(<TimeInput label="Time" onChange={() => {}} />);
-
-    const input = screen.getByRole('textbox');
-    fireEvent.change(input, {target: {value: '25:99'}});
-
-    expect(screen.getByRole('alert')).toHaveTextContent('Invalid time');
-  });
-
-  it('does not announce an alert message when input is valid', () => {
-    render(<TimeInput label="Time" onChange={() => {}} />);
-
-    const input = screen.getByRole('textbox');
-    fireEvent.change(input, {target: {value: '3:45 pm'}});
-
-    expect(screen.getByRole('alert')).toHaveTextContent('');
-    expect(screen.queryByText('Invalid time')).not.toBeInTheDocument();
-  });
-
   it('calls onChange on blur when input is valid', async () => {
     const user = userEvent.setup();
     const onChange = vi.fn();

package/src/TimeInput/TimeInput.tsx

@@ -635,9 +635,7 @@ export function TimeInput({
         data-autofocus={hasAutoFocus || undefined}
         aria-describedby={ariaDescribedBy}
         aria-required={isRequired === true ? 'true' : undefined}
-        aria-invalid={
-          status?.type === 'error' || !isInputValid ? 'true' : undefined
-        }
+        aria-invalid={status?.type === 'error' ? 'true' : undefined}
         aria-busy={isBusy || undefined}
         aria-labelledby={ariaLabelledBy}
         {...stylex.props(
@@ -646,14 +644,6 @@ export function TimeInput({
           !isInputValid && styles.inputInvalid,
         )}
       />
-      {/*
-          Live region announcing invalid typed input to assistive technology.
-          The value silently reverts on blur, so without this a screen-reader
-          user would get no feedback that their entry was rejected (WCAG 3.3.1).
-        */}
-      <VisuallyHidden as="div" role="alert" aria-live="assertive">
-        {!isInputValid ? 'Invalid time' : ''}
-      </VisuallyHidden>
       {isBusy && <Spinner size="sm" />}
       {hasClear && value && !isDisabled && (
         <button

package/src/Timestamp/Timestamp.doc.mjs

@@ -84,7 +84,7 @@ export const docs = {
   ],
   theming: {
     targets: [
-      {className: 'astryx-timestamp', visualProps: ['type', 'color', 'format']},
+      {className: 'astryx-timestamp', visualProps: ['type', 'color']},
     ],
   },
   usage: {

package/src/Toast/Toast.tsx

@@ -121,14 +121,8 @@ export function Toast({
   // Will be initialized by startTimer when actually used
   const startTimeRef = useRef<number | null>(null);
 
-  // Read onDismiss through a ref: the viewport re-creates it on every render
-  // (another toast arriving/exiting), and a startTimer that depends on it
-  // would restart — and un-pause — this toast's timer on unrelated renders.
-  const onDismissRef = useRef(onDismiss);
-  onDismissRef.current = onDismiss;
-
   const startTimer = useCallback(() => {
-    if (!isAutoHide || isPausedRef.current) {
+    if (!isAutoHide) {
       return;
     }
     if (timerRef.current) {
@@ -136,9 +130,9 @@ export function Toast({
     }
     startTimeRef.current = Date.now();
     timerRef.current = setTimeout(() => {
-      onDismissRef.current('auto');
+      onDismiss('auto');
     }, remainingRef.current);
-  }, [isAutoHide]);
+  }, [isAutoHide, onDismiss]);
 
   const pauseTimer = useCallback(() => {
     if (!isAutoHide || isPausedRef.current) {
@@ -171,8 +165,6 @@ export function Toast({
         clearTimeout(timerRef.current);
       }
     };
-    // startTimer's identity is stable per isAutoHide, so this runs on mount
-    // and on a genuine duration change — not on unrelated viewport renders.
   }, [autoHideDuration, startTimer]);
 
   // Pause the auto-hide timer while the window is not focused, so a toast

package/src/Toast/ToastViewport.test.tsx

@@ -212,67 +212,3 @@ describe('ToastViewport region ARIA', () => {
     expect(region).not.toHaveAttribute('aria-modal');
   });
 });
-
-describe('toast timer lifecycle (#3589)', () => {
-  it('fires onHide exactly once when dismissed twice during the exit window', () => {
-    const onHide = vi.fn();
-    renderViewport(
-      <ShowToastButton options={{body: 'Once', onHide}} triggerLabel="Show" />,
-    );
-    act(() => {
-      fireEvent.click(screen.getByText('Show'));
-    });
-    const dismiss = screen.getByRole('button', {name: 'Dismiss notification'});
-    act(() => {
-      fireEvent.click(dismiss);
-    });
-    // The toast stays mounted during its exit transition; a second click
-    // lands on the same still-mounted button.
-    act(() => {
-      fireEvent.click(dismiss);
-    });
-    expect(onHide).toHaveBeenCalledTimes(1);
-  });
-
-  it('keeps a window-blur pause alive when another toast arrives', () => {
-    vi.useFakeTimers();
-    try {
-      const onHide = vi.fn();
-      renderViewport(
-        <>
-          <ShowToastButton
-            options={{body: 'Paused', autoHideDuration: 3000, onHide}}
-            triggerLabel="Show A"
-          />
-          <ShowToastButton options={INFO_B} triggerLabel="Show B" />
-        </>,
-      );
-      act(() => {
-        fireEvent.click(screen.getByText('Show A'));
-      });
-      act(() => {
-        fireEvent.blur(window);
-      });
-      // A second toast arriving re-renders the viewport; the paused timer
-      // must not silently restart.
-      act(() => {
-        fireEvent.click(screen.getByText('Show B'));
-      });
-      act(() => {
-        vi.advanceTimersByTime(60_000);
-      });
-      expect(onHide).not.toHaveBeenCalled();
-      // Focus returns: the remaining time resumes and completes normally.
-      act(() => {
-        fireEvent.focus(window);
-      });
-      act(() => {
-        vi.advanceTimersByTime(60_000);
-      });
-      expect(onHide).toHaveBeenCalledTimes(1);
-      expect(onHide).toHaveBeenCalledWith('auto');
-    } finally {
-      vi.useRealTimers();
-    }
-  });
-});

package/src/Toast/ToastViewport.tsx

@@ -114,9 +114,6 @@ export function ToastViewport({
 
   // Show the popover on mount so it enters the top layer.
   const viewportRef = useRef<HTMLDivElement>(null);
-  // Toast ids whose exit has begun — guards onHide from double-firing (see
-  // removeToast). Mirrors exitingIds state, readable synchronously.
-  const exitingIdsRef = useRef<Set<string>>(new Set());
   // The element that was focused before the user jumped into the viewport
   // (via F6). Used to restore focus once all toasts are dismissed so focus
   // never falls to <body>.
@@ -174,14 +171,6 @@ export function ToastViewport({
   }, []);
 
   const removeToast = useCallback((id: string, reason: ToastDismissReason) => {
-    // An exiting toast stays in toastsRef until its exit transition ends, so
-    // a second dismissal inside that window (double-click, auto-timer plus
-    // manual dismiss()) would re-fire onHide. Track exiting ids in a ref —
-    // the exitingIds state dedupe below runs too late to protect onHide.
-    if (exitingIdsRef.current.has(id)) {
-      return;
-    }
-    exitingIdsRef.current.add(id);
     const entry = toastsRef.current.find(t => t.id === id);
     if (entry) {
       entry.options.onHide?.(reason);
@@ -221,7 +210,6 @@ export function ToastViewport({
   }, []);
 
   const handleExited = useCallback((id: string) => {
-    exitingIdsRef.current.delete(id);
     setExitingIds(prev => {
       if (!prev.has(id)) {
         return prev;

package/src/ToggleButton/ToggleButton.doc.mjs

@@ -17,7 +17,6 @@ export const docs = {
   theming: {
     targets: [
       {className: 'astryx-toggle-button-group'},
-      {className: 'astryx-toggle-button', states: ['isPressed']},
     ],
   },
   description: 'A button that toggles between pressed and unpressed states. Thin wrapper over Button with controlled toggle pattern, icon swap, and font weight emphasis.',

package/src/Tokenizer/Tokenizer.doc.mjs

@@ -189,7 +189,7 @@ export const docs = {
   ],
   theming: {
     targets: [
-      {className: 'astryx-tokenizer', visualProps: ['size', 'status']},
+      {className: 'astryx-tokenizer', visualProps: ['size']},
     ],
   },
   usage: {
@@ -390,7 +390,7 @@ export const docsZh = {
   ],
   theming: {
     targets: [
-      {className: 'astryx-tokenizer', visualProps: ['size', 'status']},
+      {className: 'astryx-tokenizer', visualProps: ['size']},
     ],
   },
   usage: {