@iloveagents/foundry-web-ui 0.3.0 → 0.4.0

package/CHANGELOG.md

@@ -1,203 +0,0 @@
-# @iloveagents/foundry-web-ui
-
-## 0.3.0
-
-### Patch Changes
-
-- @iloveagents/foundry-agent@0.3.0
-- @iloveagents/foundry-web-primitives@0.3.0
-
-## 0.2.2
-
-### Patch Changes
-
-- 395f0cd: web-ui: popover New Thread didn't actually start a new thread
-
-  When the user clicks the compose / SquarePen "New Thread" button in
-  the floating chat popover (`ChatBubble` on workspace/entity pages with
-  `navigateToChat: false`), the runtime was supposed to discard the
-  current thread and start fresh. In practice the popover header
-  subtitle flipped to "New conversation" but the message list stayed
-  populated with the prior chat's messages — the button looked like a
-  no-op.
-
-  Root cause: `useNewConversation` bumped the AGUIRuntime inner key to
-  remount, but the parent `RuntimeBody`'s `effectiveThreadId`
-  (`urlMatch ?? sticky ?? freshIdRef.current`) stayed pinned to the
-  sticky active-chat id. The history adapter on the fresh inner mount
-  immediately loaded the same chat's messages from the server →
-  nothing visibly changed.
-
-  The sidebar "New Thread" button avoided this only by side-effect —
-  it navigates to `/`, which triggers `useTrackActiveChatFromUrl` to
-  clear sticky. The popover preserves the user's workspace context and
-  doesn't navigate, so sticky never cleared.
-
-  Fix: add a tiny DI store `useChatLifecycleStore` exposing an
-  `onNewConversation` callback. `useNewConversation` invokes it BEFORE
-  `resetThread` so the host (spaces-web-ui in lastspace) can clear its
-  sticky id; the next render's `effectiveThreadId` has already advanced
-  to a freshly-minted UUID, and the history adapter binds to that
-  instead of re-hydrating the prior chat.
-
-  Customers that don't track sticky ids leave `onNewConversation`
-  unset; the call site no-ops gracefully.
-
-- 1ba01ef: web-ui(sidebar): align group-header labels (CHATS, WORKSPACES, ADMIN, …)
-  with the leftmost edge of their item labels
-
-  Previously the collapsible group header used `px-3` on the outer wrapper
-  and a `size-6` chevron, while item rows used `pl-5` (depth-0 indent) +
-  `size-3.5` icon + `gap-2`. Result: group labels landed at ~36px from the
-  sidebar edge while item labels landed at 42px — group labels drifted
-  right of the children they introduced and the visual rhythm broke.
-
-  Make the header's leading column a shared `SIDEBAR_ITEM_ICON_CLASS` +
-  `SIDEBAR_ITEM_GAP_CLASS` cell — the same constants item icons use — so
-  every group label (collapsible, link-style, or plain) lands at the same
-  x-coordinate as item labels. The chevron now sits in the icon column;
-  its hit-area is preserved at 28px via `-m-2 p-2` (negative-margin
-  absorbs the extra padding so the visual cell stays 14px and label
-  geometry doesn't shift).
-
-  Non-collapsible header branches get an explicit invisible spacer of the
-  same width so their labels align too.
-
-- 8184a8c: shell: URL is authoritative for the runtime threadId — fixes
-  resume-creates-new-conversation race
-
-  `ChatConversationAwareRuntime` previously computed
-  `effectiveThreadId = sticky ?? urlMatch ?? freshIdRef.current`.
-  When the user navigated from one chat to another via a Recents
-  click, the synchronous render that followed the URL change had:
-  - `urlMatch = B` (read from the now-updated `useLocation`)
-  - `sticky = A` (the active-chat-store hadn't been updated by
-    `useTrackActiveChatFromUrl`'s `useEffect` yet — effects run
-    AFTER the commit phase)
-
-  `sticky ?? urlMatch` picked `A` (the previous chat). The AG-UI
-  adapter was constructed with `threadId = A`. The first message the
-  user sent went to `/api/agent` with `thread_id = A`, the middleware
-  created a new row keyed by `A`... wait actually no, here's what
-  happens — the runtime mints a fresh runner UUID when given a
-  mismatched threadId; that fresh UUID became a new conversation row,
-  appearing as "Untitled chat" at the top of the sidebar while the
-  URL still said `/chat/B`. Two active-looking dots: one for `B`
-  (NavLink URL match), one for the new row (statusDot from the
-  updated active-chat-store after the lazy ensure ran).
-
-  Swap the priority: `urlMatch ?? sticky ?? freshIdRef.current`. The
-  URL is authoritative whenever it's set (i.e. on `/chat/<id>`
-  routes), eliminating the stale-sticky race entirely. `sticky` is
-  still consulted as the second-priority fallback for non-chat
-  routes (`/spaces`, `/tasks`) so the popout chat stays "live" while
-  the user browses workspaces — that's the original purpose of
-  `useStickyConversationId` and it's preserved.
-
-- Updated dependencies [8184a8c]
-  - @iloveagents/foundry-agent@0.2.2
-  - @iloveagents/foundry-web-primitives@0.2.2
-
-## 0.2.1
-
-### Patch Changes
-
-- No source changes. Bumped in lock-step with the fixed group so the
-  whole group can publish at a version number that's free across all
-  four packages. (`ui@0.2.0` already published cleanly; this is a
-  follow-the-group patch bump.)
-
-## 0.2.0
-
-### Minor Changes
-
-- UI: collapsible nav groups in `sidebar.tsx` with persisted state via
-  `useSyncExternalStore` + localStorage, plus stable group ordering via
-  new `priority` field in `nav-config.ts`.
-
-  `ag-ui-runtime-provider.tsx` accepts an external `threadId` and
-  `historyAdapterFactory` so the consuming app can wire its own
-  persistence (e.g. `ThreadHistoryAdapter` → REST `/api/conversations`).
-
-### Patch Changes
-
-- Move `@iloveagents/foundry-agent`, `-web-primitives` from
-  `peerDependencies` to `dependencies`. They always ship together as a
-  coordinated fixed group from this monorepo — they were never
-  independently-versioned peers. Declaring them as peerDeps caused
-  changesets' `shouldBumpMajor` cascade to promote the entire group to
-  a major version on every minor changeset. See the root `foundry-agent`
-  CHANGELOG entry for details.
-
-- Updated dependencies
-  - @iloveagents/foundry-agent@0.2.0
-  - @iloveagents/foundry-web-primitives@0.2.0
-
-### NOTE: 1.0.1 was accidental
-
-Version 1.0.1 was published briefly on 2026-05-27 due to the peerDep
-cascade bug described above (1.0.0 was reserved but not published —
-1.0.1 was the retry). It has been unpublished. Both 1.0.0 and 1.0.1
-are now permanently reserved on npm and CANNOT be re-published. Do
-not depend on 1.0.x.
-
-## 0.1.5
-
-### Patch Changes
-
-- f9144ca: Add `NavItem.statusIcon` slot for sidebar warning / status hints.
-
-  Renders a small tone-coloured icon next to the existing `statusDot`. Use for "this item needs attention" annotations (failed background jobs, permission drift, etc.) where a coloured dot isn't legible enough. Tones map to `warning` / `danger` / `info` / `neutral`. The `label` lands on both `aria-label` and the wrapping span's `title` attribute so screen-reader announcement + hover tooltip work without consumer wiring.
-  - @iloveagents/foundry-agent@0.1.5
-  - @iloveagents/foundry-web-primitives@0.1.5
-
-## 0.1.4
-
-### Patch Changes
-
-- Updated dependencies [07ace7e]
-  - @iloveagents/foundry-agent@0.1.4
-  - @iloveagents/foundry-web-primitives@0.1.4
-
-## 0.1.3
-
-### Patch Changes
-
-- Updated dependencies [d9b0460]
-  - @iloveagents/foundry-agent@0.1.3
-  - @iloveagents/foundry-web-primitives@0.1.3
-
-## 0.1.2
-
-### Patch Changes
-
-- Updated dependencies [30a4346]
-  - @iloveagents/foundry-agent@0.1.2
-  - @iloveagents/foundry-web-primitives@0.1.2
-
-## 0.1.1
-
-### Patch Changes
-
-- 689d3e9: feat(sidebar): distinct file-drop affordance for nav containers
-
-  `useNavItemDnd` now exposes `isFileDragOver` separately from `isDragOver`
-  so consuming nav items can render a prominent file-drop visual (dashed
-  primary outline + soft primary background + Upload icon) when the user
-  is dragging native files over the container. Previously file drags
-  shared the same subtle ring as entity-move drags, so users couldn't
-  tell that a folder accepted external files. Applies to all five nav-
-  item shapes (action-row leaf, button leaf, `NestedFolderItem`,
-  `CollapsibleNavItem` with children/actions, `NavLink` fallthrough) and
-  updates `ContainerDropZone` for visual parity.
-
-  Also fixes a related regression: the capture-phase
-  `onDragEnterCapture` / `onDragOverCapture` handlers were calling
-  `e.stopPropagation()`, which short-circuits React's synthetic dispatch
-  and prevented the bubble-phase `onDragEnter` (where state actually
-  mutates) from running. Removed — `preventDefault()` alone is enough to
-  mark the element as droppable.
-
-- Updated dependencies [689d3e9]
-  - @iloveagents/foundry-agent@0.1.1
-  - @iloveagents/foundry-web-primitives@0.1.1

package/CLAUDE.md

@@ -1 +0,0 @@
-@AGENTS.md

package/src/__tests__/no-spaces-imports.test.ts

@@ -1,59 +0,0 @@
-/**
- * Guard test: @iloveagents/foundry-web-ui must never import from @lastspace/spaces-web-ui.
- * This prevents SPACES code from leaking back into the React UI core package.
- */
-import { describe, expect, it } from "vitest";
-import { readFileSync, readdirSync, statSync } from "node:fs";
-import { join } from "node:path";
-
-function findFiles(dir: string, ext: string[]): string[] {
-  const results: string[] = [];
-  for (const entry of readdirSync(dir)) {
-    const full = join(dir, entry);
-    if (entry === "node_modules" || entry === "__tests__") continue;
-    if (statSync(full).isDirectory()) {
-      results.push(...findFiles(full, ext));
-    } else if (ext.some((e) => full.endsWith(e))) {
-      results.push(full);
-    }
-  }
-  return results;
-}
-
-describe("@iloveagents/foundry-web-ui import boundaries", () => {
-  const srcDir = join(import.meta.dirname!, "..");
-  const files = findFiles(srcDir, [".ts", ".tsx"]);
-
-  it("must not import from @lastspace/spaces-web-ui", () => {
-    const violations: string[] = [];
-    for (const file of files) {
-      const content = readFileSync(file, "utf-8");
-      if (content.includes("@lastspace/spaces-web-ui")) {
-        violations.push(file.replace(srcDir + "/", ""));
-      }
-    }
-    expect(violations).toEqual([]);
-  });
-
-  it("must not import from @content-editor/react (legacy name)", () => {
-    const violations: string[] = [];
-    for (const file of files) {
-      const content = readFileSync(file, "utf-8");
-      if (content.includes("@content-editor/react")) {
-        violations.push(file.replace(srcDir + "/", ""));
-      }
-    }
-    expect(violations).toEqual([]);
-  });
-
-  it("must not use @/ path aliases (uses relative imports)", () => {
-    const violations: string[] = [];
-    for (const file of files) {
-      const content = readFileSync(file, "utf-8");
-      if (/from ["']@\//.test(content)) {
-        violations.push(file.replace(srcDir + "/", ""));
-      }
-    }
-    expect(violations).toEqual([]);
-  });
-});

package/src/__tests__/open-core-guards.test.ts

@@ -1,307 +0,0 @@
-/**
- * Open-core packaging guards.
- *
- * Three data-driven guards (the customer-isolation + Python guards from the
- * lastspace-origin version were dropped — foundry-ui is TS-only, no
- * customer-specific code lives here):
- *
- *  1. Banned imports inside `packages/agent/src/` — no React, no DOM, no
- *     `@assistant-ui/*`, no bare `"zustand"`. The MSAL subpath is the
- *     single exception that may import `@azure/msal-browser`.
- *  2. Open-core import-direction graph — each `@iloveagents/foundry-*`
- *     package may only import from a declared subset of siblings.
- *  3. File-count ceiling on `packages/web-shell/src` — the shell stays
- *     thin (composition layer, not application surface).
- *  4. Version-pin parity — shared deps in `pnpm-workspace.yaml` overrides
- *     stay in sync across workspace `package.json` files.
- */
-import { describe, expect, it } from "vitest";
-import { existsSync, readFileSync, readdirSync, statSync } from "node:fs";
-import { join, relative } from "node:path";
-
-const ROOT = join(import.meta.dirname!, "../../../..");
-
-// ----------------------------------------------------------------------------
-// Configs
-// ----------------------------------------------------------------------------
-
-/**
- * Open-core package import graph. For each package, list which `@iloveagents/*`
- * packages it MAY import. Empty array = none.
- *
- * Packages that do not yet exist on disk are skipped at runtime by
- * `findPackageDir` so future packages can be wired in here ahead of their
- * extraction.
- */
-const OPEN_CORE_GRAPH: Record<string, string[]> = {
-  "@iloveagents/foundry-agent": [],
-  "@iloveagents/foundry-web-primitives": [],
-  "@iloveagents/foundry-web-ui": [
-    "@iloveagents/foundry-web-primitives",
-    "@iloveagents/foundry-agent",
-  ],
-  "@iloveagents/foundry-web-shell": [
-    "@iloveagents/foundry-web-primitives",
-    "@iloveagents/foundry-web-ui",
-    "@iloveagents/foundry-agent",
-  ],
-};
-
-/**
- * Banned bare-package imports inside `packages/agent/src/`.
- *
- * `@iloveagents/foundry-agent` is the cross-runtime transport — pure TS, zero React,
- * zero `@assistant-ui/*`. The MSAL subpath (`packages/agent/src/msal/`) may
- * pull in `@azure/msal-browser` (dynamic import keeps it optional). Bare
- * `"zustand"` is banned everywhere — must use `"zustand/vanilla"`.
- */
-const AGENT_BANNED_IMPORTS = [
-  { specifier: "react", scope: "all" as const },
-  { specifier: "react-dom", scope: "all" as const },
-  { specifier: "react/jsx-runtime", scope: "all" as const },
-  { specifierPrefix: "@assistant-ui/", scope: "all" as const },
-  { specifier: "@azure/msal-react", scope: "all" as const },
-  // bare "zustand" is the React entry — agent must use "zustand/vanilla".
-  { specifierExact: "zustand", scope: "all" as const },
-  // msal-browser is allowed only under src/msal/.
-  { specifier: "@azure/msal-browser", scope: "non-msal" as const },
-];
-
-/** File-count ceiling: `packages/web-shell/src` is hard-capped at 15 — the
- *  shell is a thin composition layer (bootstrapShell + ChatModule wiring +
- *  auth provider seam), not application surface. Growth indicates leakage.
- */
-const FILE_COUNT_CEILINGS = [
-  { path: "packages/web-shell/src", ceiling: 15, hardFail: true },
-];
-
-/**
- * Shared deps where all consumers must agree on the major version.
- * Rooted in `pnpm-workspace.yaml` overrides — once a customer's package.json
- * declares an incompatible major, pnpm overrides hide the drift at install
- * time, but the public package metadata is wrong.
- */
-const SHARED_DEPS_REQUIRED_MAJOR = [
-  { name: "react-router", expectedMajor: "7" },
-  { name: "@assistant-ui/react", expectedMajor: "0" },
-  { name: "@ag-ui/client", expectedMajor: "0" },
-  { name: "@ag-ui/core", expectedMajor: "0" },
-];
-
-// ----------------------------------------------------------------------------
-// Helpers
-// ----------------------------------------------------------------------------
-
-function findFiles(dir: string, exts: string[], skip: Set<string>): string[] {
-  if (!existsSync(dir)) return [];
-  const out: string[] = [];
-  for (const entry of readdirSync(dir)) {
-    if (skip.has(entry)) continue;
-    const full = join(dir, entry);
-    const stat = statSync(full);
-    if (stat.isDirectory()) out.push(...findFiles(full, exts, skip));
-    else if (exts.some((e) => full.endsWith(e))) out.push(full);
-  }
-  return out;
-}
-
-function findPackageDir(name: string): string | null {
-  const pkgDir = join(ROOT, "packages");
-  if (!existsSync(pkgDir)) return null;
-  for (const entry of readdirSync(pkgDir)) {
-    const candidate = join(pkgDir, entry);
-    const pkgJsonPath = join(candidate, "package.json");
-    if (existsSync(pkgJsonPath)) {
-      try {
-        const json = JSON.parse(readFileSync(pkgJsonPath, "utf-8"));
-        if (json.name === name) return candidate;
-      } catch {
-        /* ignore malformed */
-      }
-    }
-  }
-  return null;
-}
-
-function importsOf(content: string): string[] {
-  const out: string[] = [];
-  // ES imports + dynamic imports + re-exports
-  const re = /(?:import|export|require)\s*[^"';]*?["']([^"']+)["']/g;
-  let m: RegExpExecArray | null;
-  while ((m = re.exec(content)) !== null) out.push(m[1]);
-  return out;
-}
-
-// ----------------------------------------------------------------------------
-// Tests
-// ----------------------------------------------------------------------------
-
-describe("open-core guards", () => {
-  // 1. Agent package: no React / no assistant-ui / no bare zustand
-  describe("@iloveagents/foundry-agent banned-import guard", () => {
-    const agentRoot = findPackageDir("@iloveagents/foundry-agent");
-
-    if (!agentRoot) {
-      it.skip("agent package not yet extracted", () => undefined);
-    } else {
-      const files = findFiles(
-        join(agentRoot, "src"),
-        [".ts", ".tsx"],
-        new Set(["node_modules", "dist", "__tests__"]),
-      );
-
-      function isUnderMsal(filePath: string): boolean {
-        const rel = relative(agentRoot!, filePath);
-        return rel.startsWith(`src/msal/`) || rel.startsWith(`src${"/"}msal${"/"}`);
-      }
-
-      for (const rule of AGENT_BANNED_IMPORTS) {
-        const label =
-          (rule as { specifier?: string }).specifier ??
-          (rule as { specifierPrefix?: string }).specifierPrefix ??
-          (rule as { specifierExact?: string }).specifierExact!;
-
-        it(`no source file imports "${label}" (scope: ${rule.scope})`, () => {
-          const violations: string[] = [];
-          for (const file of files) {
-            if (rule.scope === "non-msal" && isUnderMsal(file)) continue;
-            const content = readFileSync(file, "utf-8");
-            for (const imp of importsOf(content)) {
-              const matches =
-                ("specifierExact" in rule && imp === rule.specifierExact) ||
-                ("specifier" in rule && imp === rule.specifier) ||
-                ("specifierPrefix" in rule &&
-                  rule.specifierPrefix !== undefined &&
-                  imp.startsWith(rule.specifierPrefix));
-              if (matches) violations.push(`${relative(ROOT, file)} -> ${imp}`);
-            }
-          }
-          expect(
-            violations,
-            `${label} import leaked into packages/agent/src/${
-              rule.scope === "non-msal" ? " (outside msal/)" : ""
-            }`,
-          ).toEqual([]);
-        });
-      }
-    }
-  });
-
-  // 2. Open-core import graph
-  describe("open-core package import graph", () => {
-    for (const [pkg, allowed] of Object.entries(OPEN_CORE_GRAPH)) {
-      it(`${pkg} only imports from [${allowed.join(", ") || "none"}]`, () => {
-        const dir = findPackageDir(pkg);
-        if (!dir) return; // package not yet extracted — skip
-        const files = findFiles(
-          join(dir, "src"),
-          [".ts", ".tsx"],
-          new Set(["node_modules", "dist", "__tests__"]),
-        );
-        const violations: string[] = [];
-        for (const file of files) {
-          const content = readFileSync(file, "utf-8");
-          for (const imp of importsOf(content)) {
-            // Foundry-ui packages must never import from @lastspace/* (commercial
-            // downstream); allow-listing is enforced for sibling @iloveagents/* deps.
-            if (imp.startsWith("@lastspace/")) {
-              violations.push(`${relative(ROOT, file)} -> ${imp} (lastspace import forbidden)`);
-              continue;
-            }
-            if (!imp.startsWith("@iloveagents/")) continue;
-            // Self-imports allowed (subpath exports etc).
-            if (imp === pkg || imp.startsWith(`${pkg}/`)) continue;
-            // Strip subpath off the imported package.
-            const importedPkg = imp.split("/").slice(0, 2).join("/");
-            if (!allowed.includes(importedPkg)) {
-              violations.push(`${relative(ROOT, file)} -> ${imp}`);
-            }
-          }
-        }
-        expect(violations, `${pkg} imported a forbidden package`).toEqual([]);
-      });
-    }
-  });
-
-  // 3. File-count ceiling (warning + optional hard-fail)
-  describe("file-count ceilings on host directories", () => {
-    for (const { path, ceiling, hardFail } of FILE_COUNT_CEILINGS) {
-      it(`${path} ${hardFail ? "must" : "should"} stay ≤ ${ceiling} files`, () => {
-        const dir = join(ROOT, path);
-        if (!existsSync(dir)) return;
-        const files = findFiles(
-          dir,
-          [".ts", ".tsx", ".py"],
-          new Set(["node_modules", ".venv", "__pycache__", "dist", "__tests__"]),
-        );
-        if (hardFail) {
-          expect(
-            files.length,
-            `${path} has ${files.length} files; ceiling is ${ceiling}. Move features into packages.`,
-          ).toBeLessThanOrEqual(ceiling);
-        } else {
-          // Warning-mode: log if over ceiling but don't fail. Flip to hardFail
-          // once host-shell extraction (#98) lands.
-          if (files.length > ceiling) {
-            // eslint-disable-next-line no-console
-            console.warn(
-              `[file-count guard] ${path} has ${files.length} files (ceiling ${ceiling}). Hard-fail enabled once #98 lands.`,
-            );
-          }
-          expect(files.length).toBeGreaterThanOrEqual(0); // always passes; just records
-        }
-      });
-    }
-  });
-
-  // 4. Version-pin parity
-  describe("shared deps stay in sync across workspace package.jsons", () => {
-    function workspacePackageJsons(): { dir: string; json: Record<string, unknown> }[] {
-      const out: { dir: string; json: Record<string, unknown> }[] = [];
-      for (const base of ["packages"]) {
-        const baseDir = join(ROOT, base);
-        if (!existsSync(baseDir)) continue;
-        for (const entry of readdirSync(baseDir)) {
-          const dir = join(baseDir, entry);
-          const pkgJson = join(dir, "package.json");
-          if (existsSync(pkgJson)) {
-            try {
-              out.push({ dir, json: JSON.parse(readFileSync(pkgJson, "utf-8")) });
-            } catch {
-              /* ignore */
-            }
-          }
-        }
-      }
-      return out;
-    }
-
-    function depMajor(spec: string): string | null {
-      // Strip leading ^, ~, >=, =, etc.
-      const m = spec.match(/(\d+)/);
-      return m ? m[1] : null;
-    }
-
-    for (const { name, expectedMajor } of SHARED_DEPS_REQUIRED_MAJOR) {
-      it(`${name} declared at major ${expectedMajor} in every package.json that declares it`, () => {
-        const violations: string[] = [];
-        for (const { dir, json } of workspacePackageJsons()) {
-          const buckets = ["dependencies", "peerDependencies", "devDependencies"] as const;
-          for (const bucket of buckets) {
-            const deps = (json[bucket] ?? {}) as Record<string, string>;
-            const spec = deps[name];
-            if (!spec) continue;
-            const major = depMajor(spec);
-            if (major !== expectedMajor) {
-              violations.push(`${relative(ROOT, dir)} ${bucket}.${name}=${spec}`);
-            }
-          }
-        }
-        expect(
-          violations,
-          `${name} version drift detected; pnpm-workspace.yaml override hides it at install time but published metadata is wrong`,
-        ).toEqual([]);
-      });
-    }
-  });
-});

package/src/__tests__/theme-runtime.test.ts

@@ -1,81 +0,0 @@
-import { describe, expect, it } from "vitest";
-import {
-  getThemePreset,
-  mergeThemeDefinitions,
-  resolveThemeRuntime,
-} from "../lib/theme-runtime.ts";
-
-describe("theme runtime", () => {
-  it("merges sparse definitions in precedence order", () => {
-    const merged = mergeThemeDefinitions(
-      {
-        light: {
-          primary: "oklch(0.4 0.2 280)",
-          foreground: "oklch(0.2 0.01 280)",
-        },
-      },
-      {
-        light: {
-          primary: "oklch(0.5 0.19 260)",
-        },
-        status: {
-          light: {
-            success: "oklch(0.6 0.15 150)",
-          },
-        },
-      },
-    );
-
-    expect(merged.light?.primary).toBe("oklch(0.5 0.19 260)");
-    expect(merged.light?.foreground).toBe("oklch(0.2 0.01 280)");
-    expect(merged.status?.light?.success).toBe("oklch(0.6 0.15 150)");
-  });
-
-  it("resolves scoped layers on top of the preset", () => {
-    const runtime = resolveThemeRuntime(
-      [
-        {
-          id: "global-theme",
-          basePreset: "foundry-violet",
-          definition: {
-            light: {
-              primary: "oklch(0.45 0.18 300)",
-            },
-            sidebar: {
-              light: {
-                sidebar: "oklch(0.98 0.01 300)",
-              },
-            },
-          },
-        },
-        {
-          id: "workspace-theme",
-          definition: {
-            light: {
-              primary: "oklch(0.52 0.16 240)",
-            },
-          },
-        },
-      ],
-      "light",
-    );
-
-    expect(runtime.mode).toBe("light");
-    expect(runtime.sources.map((layer) => layer.id)).toEqual([
-      "global-theme",
-      "workspace-theme",
-    ]);
-    expect(runtime.definition.light?.primary).toBe("oklch(0.52 0.16 240)");
-    expect(runtime.variables["--primary" as keyof typeof runtime.variables]).toBe(
-      "oklch(0.52 0.16 240)",
-    );
-    expect(runtime.variables["--sidebar" as keyof typeof runtime.variables]).toBe(
-      "oklch(0.98 0.01 300)",
-    );
-    expect(runtime.variables["--success" as keyof typeof runtime.variables]).toBeTruthy();
-  });
-
-  it("falls back to the default preset for unknown names", () => {
-    expect(getThemePreset("unknown-preset")).toEqual(getThemePreset("foundry-violet"));
-  });
-});