@zentauri-ui/zentauri-components 1.8.1 → 1.8.3

package/src/ui/empty-state/empty-state.test.tsx

@@ -0,0 +1,88 @@
+import { createRef } from "react";
+import { render, screen } from "@testing-library/react";
+import { describe, expect, it } from "vitest";
+
+import { EmptyState } from "./empty-state";
+import {
+  EmptyStateAction,
+  EmptyStateDescription,
+  EmptyStateIcon,
+  EmptyStateTitle,
+} from "./empty-state-base";
+
+const EMPTY_STATE_SLOT = '[data-slot="empty-state"]';
+
+function getEmptyStateRoot(container: HTMLElement = document.body) {
+  const elements = container.querySelectorAll(EMPTY_STATE_SLOT);
+  expect(elements.length).toBe(1);
+  return elements[0] as HTMLElement;
+}
+
+describe("EmptyState", () => {
+  it("should set displayName on compound parts", () => {
+    expect(EmptyState.displayName).toBe("EmptyState");
+    expect(EmptyStateIcon.displayName).toBe("EmptyStateIcon");
+    expect(EmptyStateTitle.displayName).toBe("EmptyStateTitle");
+    expect(EmptyStateDescription.displayName).toBe("EmptyStateDescription");
+    expect(EmptyStateAction.displayName).toBe("EmptyStateAction");
+  });
+
+  it("should stamp data-slot on the root section", () => {
+    render(<EmptyState>Nothing here</EmptyState>);
+    const root = getEmptyStateRoot();
+    expect(root.tagName).toBe("SECTION");
+    expect(root).toHaveAttribute("data-slot", "empty-state");
+  });
+
+  it("should render title, description, icon, and action slots", () => {
+    render(
+      <EmptyState>
+        <EmptyStateIcon>!</EmptyStateIcon>
+        <EmptyStateTitle>No results</EmptyStateTitle>
+        <EmptyStateDescription>Try another filter.</EmptyStateDescription>
+        <EmptyStateAction>
+          <button type="button">Reset</button>
+        </EmptyStateAction>
+      </EmptyState>,
+    );
+
+    expect(
+      screen.getByRole("heading", { level: 2, name: "No results" }),
+    ).toHaveAttribute("data-slot", "empty-state-title");
+    expect(screen.getByText("Try another filter.")).toHaveAttribute(
+      "data-slot",
+      "empty-state-description",
+    );
+    expect(screen.getByText("!")).toHaveAttribute(
+      "data-slot",
+      "empty-state-icon",
+    );
+    expect(screen.getByRole("button", { name: "Reset" })).toBeVisible();
+  });
+
+  it("should apply live region state when requested", () => {
+    render(<EmptyState liveRegion="assertive">Updated</EmptyState>);
+    expect(getEmptyStateRoot()).toHaveAttribute("aria-live", "assertive");
+  });
+
+  it("should apply size classes to nested text slots through context", () => {
+    render(
+      <EmptyState size="lg">
+        <EmptyStateTitle>Large title</EmptyStateTitle>
+        <EmptyStateDescription>Large description</EmptyStateDescription>
+      </EmptyState>,
+    );
+
+    expect(screen.getByText("Large title").className).toMatch(/text-xl/);
+    expect(screen.getByText("Large description").className).toMatch(
+      /text-base/,
+    );
+  });
+
+  it("should forward refs to the root element", () => {
+    const ref = createRef<HTMLElement>();
+    render(<EmptyState ref={ref}>Empty</EmptyState>);
+    expect(ref.current).toBeInstanceOf(HTMLElement);
+    expect(ref.current?.getAttribute("data-slot")).toBe("empty-state");
+  });
+});

package/src/ui/marquee/marquee.test.tsx

@@ -1,9 +1,13 @@
-import { render, screen } from "@testing-library/react";
-import { describe, expect, it } from "vitest";
+import { render, screen, waitFor } from "@testing-library/react";
+import { afterEach, describe, expect, it, vi } from "vitest";
 
 import { Marquee } from "./marquee";
 
 describe("Marquee", () => {
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
   it("exposes a display name", () => {
     expect(Marquee.displayName).toBe("Marquee");
   });
@@ -17,13 +21,14 @@ describe("Marquee", () => {
 
     expect(document.querySelector('[data-slot="marquee"]')).toBeTruthy();
     expect(document.querySelector('[data-slot="marquee-track"]')).toBeTruthy();
-    expect(screen.getAllByText("Acme")).toHaveLength(2);
-
     const groups = document.querySelectorAll(
       '[data-slot="marquee-item-group"]',
     );
+    expect(groups).toHaveLength(2);
+    expect(groups[0]?.textContent).toContain("Acme");
     expect(groups[1]).toHaveAttribute("aria-hidden", "true");
     expect(groups[1]).toHaveAttribute("inert");
+    expect(groups[1]?.textContent).toContain("Acme");
   });
 
   it("applies horizontal metadata by default", () => {
@@ -84,6 +89,42 @@ describe("Marquee", () => {
     });
   });
 
+  it("hides filler copies from assistive technology", async () => {
+    vi.spyOn(HTMLElement.prototype, "offsetWidth", "get").mockImplementation(
+      function (this: HTMLElement) {
+        return this.getAttribute("data-slot") === "marquee" ? 300 : 0;
+      },
+    );
+    vi.spyOn(HTMLElement.prototype, "scrollWidth", "get").mockImplementation(
+      function (this: HTMLElement) {
+        return this.getAttribute("data-slot") === "marquee-measure" ? 100 : 0;
+      },
+    );
+
+    render(
+      <Marquee>
+        <button type="button">Focusable item</button>
+      </Marquee>,
+    );
+
+    const firstGroup = document.querySelector(
+      '[data-slot="marquee-item-group"]',
+    );
+
+    await waitFor(() => {
+      expect(firstGroup?.textContent).toBe(
+        "Focusable itemFocusable itemFocusable item",
+      );
+    });
+
+    expect(
+      screen.getAllByRole("button", { name: "Focusable item" }),
+    ).toHaveLength(1);
+    expect(
+      firstGroup?.querySelectorAll('[aria-hidden="true"][inert]'),
+    ).toHaveLength(2);
+  });
+
   it("accepts string gap values", () => {
     render(
       <Marquee gap="2rem" data-testid="marquee">

package/src/ui/marquee/marquee.tsx

@@ -1,7 +1,9 @@
 "use client";
 
-import type { CSSProperties } from "react";
+import { Children, useCallback, useMemo, useRef, useState } from "react";
+import type { CSSProperties, Ref } from "react";
 
+import { useIsomorphicLayoutEffect } from "../../hooks/useIsomorphicLayoutEffect";
 import { cn } from "../../lib/utils";
 
 import type { MarqueeProps } from "./types";
@@ -16,6 +18,17 @@ function toCssLength(value: number | string | undefined) {
   return typeof value === "number" ? `${value}px` : value;
 }
 
+function assignRef<TElement>(
+  ref: Ref<TElement> | undefined,
+  value: TElement | null,
+) {
+  if (typeof ref === "function") {
+    ref(value);
+  } else if (ref) {
+    (ref as { current: TElement | null }).current = value;
+  }
+}
+
 export function Marquee(props: MarqueeProps) {
   const {
     appearance,
@@ -44,14 +57,81 @@ export function Marquee(props: MarqueeProps) {
     resolvedDirection === "right" || resolvedDirection === "down";
   const animationName =
     resolvedOrientation === "vertical" ? "zui-marquee-y" : "zui-marquee-x";
+  const rootRef = useRef<HTMLDivElement | null>(null);
+  const measureRef = useRef<HTMLDivElement | null>(null);
+  const [copyCount, setCopyCount] = useState(1);
+  const childArray = useMemo(() => Children.toArray(children), [children]);
+  const setRootRef = useCallback(
+    (node: HTMLDivElement | null) => {
+      rootRef.current = node;
+      assignRef(ref, node);
+    },
+    [ref],
+  );
   const marqueeStyle = {
     ...(gap !== undefined ? { "--zui-marquee-gap": toCssLength(gap) } : null),
     ...style,
   } as CSSProperties;
+  const groupClassName = cn(
+    "flex shrink-0 items-center justify-around gap-(--zui-marquee-gap)",
+    resolvedOrientation === "vertical" ? "flex-col" : "flex-row",
+    itemClassName,
+  );
+  const fillerChildren = Array.from(
+    { length: Math.max(0, copyCount - 1) },
+    (_, index) => (
+      <div key={index} aria-hidden="true" inert className="contents">
+        {childArray}
+      </div>
+    ),
+  );
+
+  useIsomorphicLayoutEffect(() => {
+    const updateCopyCount = () => {
+      const root = rootRef.current;
+      const measure = measureRef.current;
+
+      if (!root || !measure) {
+        return;
+      }
+
+      const rootSize =
+        resolvedOrientation === "vertical"
+          ? root.offsetHeight
+          : root.offsetWidth;
+      const contentSize =
+        resolvedOrientation === "vertical"
+          ? measure.scrollHeight
+          : measure.scrollWidth;
+
+      if (!rootSize || !contentSize) {
+        setCopyCount(1);
+        return;
+      }
+
+      setCopyCount(Math.max(1, Math.ceil(rootSize / contentSize)));
+    };
+
+    updateCopyCount();
+
+    if (typeof ResizeObserver === "undefined") {
+      return;
+    }
+
+    const observer = new ResizeObserver(updateCopyCount);
+    if (rootRef.current) {
+      observer.observe(rootRef.current);
+    }
+    if (measureRef.current) {
+      observer.observe(measureRef.current);
+    }
+
+    return () => observer.disconnect();
+  }, [childArray, gap, resolvedOrientation]);
 
   return (
     <div
-      ref={ref}
+      ref={setRootRef}
       data-direction={resolvedDirection}
       data-orientation={resolvedOrientation}
       data-slot="marquee"
@@ -68,10 +148,21 @@ export function Marquee(props: MarqueeProps) {
       {...rest}
     >
       <style>{marqueeKeyframes}</style>
+      <div
+        aria-hidden="true"
+        data-slot="marquee-measure"
+        ref={measureRef}
+        className={cn(
+          "pointer-events-none invisible absolute -z-10",
+          groupClassName,
+        )}
+      >
+        {childArray}
+      </div>
       <div
         data-slot="marquee-track"
         className={cn(
-          "flex shrink-0 gap-[var(--zui-marquee-gap)] will-change-transform [animation-iteration-count:infinite] [animation-timing-function:linear] motion-reduce:[animation-play-state:paused]",
+          "flex shrink-0 gap-(--zui-marquee-gap) will-change-transform [animation-iteration-count:infinite] [animation-timing-function:linear] motion-reduce:[animation-play-state:paused]",
           resolvedOrientation === "vertical" ? "flex-col" : "w-max flex-row",
           pauseOnHover && "group-hover/marquee:[animation-play-state:paused]",
           isReverse && "[animation-direction:reverse]",
@@ -84,27 +175,18 @@ export function Marquee(props: MarqueeProps) {
           } as CSSProperties
         }
       >
-        <div
-          data-slot="marquee-item-group"
-          className={cn(
-            "flex shrink-0 items-center justify-around gap-[var(--zui-marquee-gap)]",
-            resolvedOrientation === "vertical" ? "flex-col" : "flex-row",
-            itemClassName,
-          )}
-        >
-          {children}
+        <div data-slot="marquee-item-group" className={groupClassName}>
+          {childArray}
+          {fillerChildren}
         </div>
         <div
           aria-hidden="true"
           inert
           data-slot="marquee-item-group"
-          className={cn(
-            "flex shrink-0 items-center justify-around gap-[var(--zui-marquee-gap)]",
-            resolvedOrientation === "vertical" ? "flex-col" : "flex-row",
-            itemClassName,
-          )}
+          className={groupClassName}
         >
-          {children}
+          {childArray}
+          {fillerChildren}
         </div>
       </div>
     </div>

package/src/ui/modal/modal.test.tsx

@@ -166,4 +166,28 @@ describe("Modal", () => {
     );
     expect(trigger).toHaveFocus();
   });
+
+  it("should expose aria-modal and wire labelledby/describedby to its title and description", async () => {
+    render(
+      <Modal defaultOpen>
+        <ModalContent>
+          <ModalTitle>Confirm</ModalTitle>
+          <ModalDescription>Please review</ModalDescription>
+        </ModalContent>
+      </Modal>,
+    );
+    const dialog = await screen.findByRole("dialog");
+    expect(dialog).toHaveAttribute("aria-modal", "true");
+
+    const labelledby = dialog.getAttribute("aria-labelledby");
+    const describedby = dialog.getAttribute("aria-describedby");
+    expect(labelledby).toBeTruthy();
+    expect(describedby).toBeTruthy();
+    expect(document.getElementById(labelledby as string)).toHaveTextContent(
+      "Confirm",
+    );
+    expect(document.getElementById(describedby as string)).toHaveTextContent(
+      "Please review",
+    );
+  });
 });

package/src/ui/peer-isolation.test.ts

@@ -0,0 +1,81 @@
+import { existsSync, readdirSync, readFileSync } from "node:fs";
+import { dirname, join } from "node:path";
+import { fileURLToPath } from "node:url";
+import { describe, expect, it } from "vitest";
+
+/**
+ * Guards the package's optional-peer isolation contract:
+ *
+ * - A component that ships BOTH a static entry and an `animated/` entry must keep
+ *   `framer-motion` out of its static files, so importing the static entry never
+ *   forces the optional `framer-motion` peer onto the consumer. (Components that
+ *   are themselves motion primitives — e.g. `animated-number`, `marquee` — have no
+ *   `animated/` split, so this rule does not apply to them.)
+ * - No file under `src/ui` may import `recharts`; charts live only in `src/charts/*`.
+ */
+
+const uiRoot = dirname(fileURLToPath(import.meta.url));
+
+const componentDirs = readdirSync(uiRoot, { withFileTypes: true })
+  .filter((e) => e.isDirectory())
+  .map((e) => e.name);
+
+/** `.ts`/`.tsx` files directly involved in the static entry (excludes `animated/` and tests). */
+function staticFiles(dir: string): string[] {
+  const out: string[] = [];
+  for (const entry of readdirSync(dir, { withFileTypes: true })) {
+    const full = join(dir, entry.name);
+    if (entry.isDirectory()) {
+      if (entry.name === "animated") continue;
+      out.push(...staticFiles(full));
+    } else if (
+      /\.(tsx?|jsx?)$/.test(entry.name) &&
+      !entry.name.includes(".test.")
+    ) {
+      out.push(full);
+    }
+  }
+  return out;
+}
+
+const FRAMER = /from\s+["']framer-motion/;
+const RECHARTS = /from\s+["']recharts/;
+
+describe("static UI entries — optional peer isolation", () => {
+  it("scans every UI component directory", () => {
+    expect(componentDirs.length).toBeGreaterThan(40);
+  });
+
+  for (const name of componentDirs) {
+    const dir = join(uiRoot, name);
+    const hasAnimatedEntry = existsSync(join(dir, "animated"));
+    if (!hasAnimatedEntry) continue;
+
+    it(`${name}: static files do not import framer-motion`, () => {
+      const offenders = staticFiles(dir).filter((file) =>
+        FRAMER.test(readFileSync(file, "utf8")),
+      );
+      expect(offenders).toEqual([]);
+    });
+  }
+
+  it("no UI file imports recharts (charts live in src/charts)", () => {
+    const offenders = componentDirs
+      .flatMap((name) => {
+        const all: string[] = [];
+        const dir = join(uiRoot, name);
+        // include animated files here too — recharts must never appear in ui/.
+        const walk = (d: string) => {
+          for (const entry of readdirSync(d, { withFileTypes: true })) {
+            const full = join(d, entry.name);
+            if (entry.isDirectory()) walk(full);
+            else if (/\.(tsx?|jsx?)$/.test(entry.name)) all.push(full);
+          }
+        };
+        walk(dir);
+        return all;
+      })
+      .filter((file) => RECHARTS.test(readFileSync(file, "utf8")));
+    expect(offenders).toEqual([]);
+  });
+});

package/src/ui/select/select.test.tsx

@@ -1,6 +1,6 @@
-import { render, screen } from "@testing-library/react";
+import { render, screen, waitFor } from "@testing-library/react";
 import userEvent from "@testing-library/user-event";
-import { describe, expect, it } from "vitest";
+import { describe, expect, it, vi } from "vitest";
 
 import {
   Select,
@@ -126,3 +126,99 @@ describe("Select", () => {
     expect(beta).toHaveAttribute("aria-selected", "false");
   });
 });
+
+describe("Select — keyboard and a11y", () => {
+  it("wires trigger aria attributes to the listbox", async () => {
+    const user = userEvent.setup();
+    render(
+      <Select multiple defaultValue={[]}>
+        <SelectTrigger>
+          <SelectValue placeholder="Pick" />
+        </SelectTrigger>
+        <SelectContent>
+          <SelectItem value="a">Alpha</SelectItem>
+        </SelectContent>
+      </Select>,
+    );
+    const trigger = screen.getByRole("button");
+    expect(trigger).toHaveAttribute("aria-haspopup", "listbox");
+    expect(trigger).toHaveAttribute("aria-expanded", "false");
+    const controls = trigger.getAttribute("aria-controls");
+
+    await user.click(trigger);
+    expect(trigger).toHaveAttribute("aria-expanded", "true");
+    const listbox = screen.getByRole("listbox");
+    expect(listbox).toHaveAttribute("id", controls);
+    expect(listbox).toHaveAttribute("aria-multiselectable", "true");
+  });
+
+  it("moves focus across enabled options with arrow keys", async () => {
+    const user = userEvent.setup();
+    render(
+      <Select defaultValue={[]}>
+        <SelectTrigger>
+          <SelectValue placeholder="Pick" />
+        </SelectTrigger>
+        <SelectContent>
+          <SelectItem value="a">Alpha</SelectItem>
+          <SelectItem value="b">Beta</SelectItem>
+        </SelectContent>
+      </Select>,
+    );
+    await user.click(screen.getByRole("button"));
+    const alpha = await screen.findByRole("option", { name: /alpha/i });
+    const beta = screen.getByRole("option", { name: /beta/i });
+    await waitFor(() => expect(alpha).toHaveFocus());
+
+    await user.keyboard("{ArrowDown}");
+    expect(beta).toHaveFocus();
+    await user.keyboard("{ArrowUp}");
+    expect(alpha).toHaveFocus();
+  });
+
+  it("selects the focused option with the Enter key", async () => {
+    const user = userEvent.setup();
+    const onChange = vi.fn();
+    render(
+      <Select multiple defaultValue={[]} onChange={onChange}>
+        <SelectTrigger>
+          <SelectValue placeholder="Pick" />
+        </SelectTrigger>
+        <SelectContent>
+          <SelectItem value="a">Alpha</SelectItem>
+          <SelectItem value="b">Beta</SelectItem>
+        </SelectContent>
+      </Select>,
+    );
+    await user.click(screen.getByRole("button"));
+    const alpha = await screen.findByRole("option", { name: /alpha/i });
+    await waitFor(() => expect(alpha).toHaveFocus());
+
+    await user.keyboard("{Enter}");
+    expect(onChange).toHaveBeenCalledWith(["a"]);
+  });
+
+  it("ignores a disabled option for pointer and marks it aria-disabled", async () => {
+    const user = userEvent.setup();
+    const onChange = vi.fn();
+    render(
+      <Select multiple defaultValue={[]} onChange={onChange}>
+        <SelectTrigger>
+          <SelectValue placeholder="Pick" />
+        </SelectTrigger>
+        <SelectContent>
+          <SelectItem value="a" disabled>
+            Alpha
+          </SelectItem>
+          <SelectItem value="b">Beta</SelectItem>
+        </SelectContent>
+      </Select>,
+    );
+    await user.click(screen.getByRole("button"));
+    const alpha = await screen.findByRole("option", { name: /alpha/i });
+    expect(alpha).toHaveAttribute("aria-disabled", "true");
+
+    await user.click(alpha);
+    expect(onChange).not.toHaveBeenCalled();
+  });
+});

package/src/ui/skeleton/skeleton.test.tsx

@@ -0,0 +1,85 @@
+import { createRef } from "react";
+import { render } from "@testing-library/react";
+import { describe, expect, it } from "vitest";
+
+import {
+  Skeleton,
+  SkeletonAvatar,
+  SkeletonButton,
+  SkeletonCard,
+  SkeletonText,
+} from "./skeleton";
+
+describe("Skeleton", () => {
+  it("should set displayName on skeleton primitives", () => {
+    expect(Skeleton.displayName).toBe("Skeleton");
+    expect(SkeletonText.displayName).toBe("SkeletonText");
+    expect(SkeletonAvatar.displayName).toBe("SkeletonAvatar");
+    expect(SkeletonCard.displayName).toBe("SkeletonCard");
+    expect(SkeletonButton.displayName).toBe("SkeletonButton");
+  });
+
+  it("should stamp data-slot and hide the placeholder from assistive tech", () => {
+    const { container } = render(<Skeleton busy />);
+    const root = container.querySelector('[data-slot="skeleton"]');
+    expect(root).toHaveAttribute("aria-hidden", "true");
+    expect(root).toHaveAttribute("aria-busy", "true");
+  });
+
+  it("should apply pulse motion when animation is not none", () => {
+    const { container } = render(<Skeleton animation="pulse" />);
+    expect(
+      container.querySelector('[data-slot="skeleton"]')?.className,
+    ).toMatch(/animate-pulse/);
+  });
+
+  it("should omit motion classes when animation is none", () => {
+    const { container } = render(<Skeleton animation="none" />);
+    expect(
+      container.querySelector('[data-slot="skeleton"]')?.className,
+    ).not.toMatch(/animate-pulse/);
+  });
+
+  it("should render the requested number of text lines", () => {
+    const { container } = render(<SkeletonText lines={4} />);
+    const root = container.querySelector('[data-slot="skeleton-text"]');
+    expect(root).toHaveAttribute("aria-hidden", "true");
+    expect(root?.children.length).toBe(4);
+    expect(root?.lastElementChild?.className).toMatch(/w-3\/5/);
+  });
+
+  it("should share parent animation with nested text placeholders", () => {
+    const { container } = render(
+      <Skeleton animation="none">
+        <SkeletonText lines={2} />
+      </Skeleton>,
+    );
+    const text = container.querySelector('[data-slot="skeleton-text"]');
+    expect(text?.firstElementChild?.className).not.toMatch(/animate-pulse/);
+  });
+
+  it("should render avatar, button, and card slots", () => {
+    const { container } = render(
+      <>
+        <SkeletonAvatar avatarSize="xl" />
+        <SkeletonButton buttonSize="lg" />
+        <SkeletonCard />
+      </>,
+    );
+
+    expect(
+      container.querySelector('[data-slot="skeleton-avatar"]'),
+    ).toHaveClass("size-14");
+    expect(
+      container.querySelector('[data-slot="skeleton-button"]'),
+    ).toHaveClass("h-12");
+    expect(container.querySelector('[data-slot="skeleton-card"]')).toBeTruthy();
+  });
+
+  it("should forward refs to the base skeleton element", () => {
+    const ref = createRef<HTMLDivElement>();
+    render(<Skeleton ref={ref} />);
+    expect(ref.current).toBeInstanceOf(HTMLElement);
+    expect(ref.current?.getAttribute("data-slot")).toBe("skeleton");
+  });
+});

package/src/ui/table/index.ts

@@ -18,5 +18,8 @@ export type {
   TableHeadCellProps,
   TableProps,
   TableSectionProps,
+  TableSortChangeHandler,
+  TableSortDirection,
+  TableSortState,
 } from "./types";
 export { tableVariants, tableRowVariants, tableCellVariants } from "./variants";