Files
openclaw/ui/src/app-navigation.test.ts
Peter Steinberger ea06d72e85 feat(secrets): manage team secrets in Control UI (#121724)
* feat(secrets): add gateway store settings

* perf(control-ui): trim secrets startup copy

* perf(control-ui): reduce secrets startup payload

* fix(secrets): harden store mutation refresh

* perf(control-ui): meet secrets startup budget

* test(control-ui): update secrets navigation copy

* fix(ui): pluralize secret-detection count and drop duplicated dialog hint

* chore(protocol): regenerate gateway clients and SDK baseline after rebase

* fix(gateway): merge secrets store methods after project RPCs in advertised order

* chore: leave changelog to release generation

* test(gateway): retain desktop launch train coverage
2026-08-11 07:20:38 +00:00

572 lines
22 KiB
TypeScript

// @vitest-environment node
// Control UI tests cover navigation behavior.
import { describe, expect, it } from "vitest";
import {
SETTINGS_NAVIGATION_GROUPS,
SIDEBAR_NAV_ROUTES,
formatDocumentTitle,
isPluginsHubRoute,
navigationIconForRoute,
settingsSearchTextMatches,
subtitleForRoute,
titleForRoute,
} from "./app-navigation.ts";
import {
inferBasePathFromPathname,
normalizeBasePath,
pathForRoute,
pathForWorkboardBoard,
workboardBoardIdFromPath,
} from "./app-route-paths.ts";
import { createApplicationRouter, routeIdFromPath, type RouteId } from "./app-routes.ts";
import { sessionRefFromPath } from "./app-session-route-paths.ts";
import { sessionNavigationTarget } from "./lib/sessions/route-navigation.ts";
import { pluginTabKey, pluginTabRefFromSearch, pluginTabSearch } from "./pages/plugin/route.ts";
/**
* All route identifiers derived from sidebar nav routes plus routed settings
* slices and the Plugins hub tabs, which route without their own sidebar item.
*/
const ALL_ROUTES: RouteId[] = Array.from(
new Set<RouteId>([
"chat",
"custodian",
...SIDEBAR_NAV_ROUTES,
"skills",
"skill-workshop",
// Hub tabs and settings subpages route without their own nav entry.
"worktrees",
"memory-import",
"ai-agents",
"model-setup",
"lobsterdex",
...SETTINGS_NAVIGATION_GROUPS.flatMap((group) => group.routes),
]),
);
const SETTINGS_ROUTE_PATHS = [
{ routeId: "config", path: "/settings/general", alias: "/config" },
{ routeId: "profile", path: "/settings/profile", alias: "/profile" },
{ routeId: "channels", path: "/settings/channels", alias: "/channels" },
{
routeId: "communications",
path: "/settings/communications",
alias: "/communications",
},
{ routeId: "appearance", path: "/settings/appearance", alias: "/appearance" },
{ routeId: "lobsterdex", path: "/settings/lobsterdex", alias: "/lobsterdex" },
{ routeId: "automation", path: "/settings/automation", alias: "/automation" },
{ routeId: "mcp", path: "/settings/mcp", alias: "/mcp" },
{
routeId: "infrastructure",
path: "/settings/infrastructure",
alias: "/infrastructure",
},
{ routeId: "worktrees", path: "/worktrees", alias: "/settings/worktrees" },
{ routeId: "sessions", path: "/sessions", alias: "/settings/sessions" },
{ routeId: "devices", path: "/settings/devices", alias: "/nodes" },
{ routeId: "cron", path: "/automations", alias: "/cron" },
{ routeId: "agents", path: "/settings/agents", alias: "/agents" },
{
routeId: "memory-import",
path: "/memory-import",
alias: "/settings/memory-import",
},
{ routeId: "ai-agents", path: "/settings/ai-agents", alias: "/ai-agents" },
{
routeId: "model-setup",
path: "/settings/model-setup",
alias: "/model-setup",
},
{
routeId: "model-providers",
path: "/settings/model-providers",
alias: "/model-providers",
},
] as const satisfies readonly { routeId: RouteId; path: string; alias: string }[];
describe("navigationIconForRoute", () => {
it("returns stable icons for every route", () => {
expect(
Object.fromEntries(ALL_ROUTES.map((routeId) => [routeId, navigationIconForRoute(routeId)])),
).toEqual({
chat: "messageSquare",
custodian: "lobster",
activity: "activity",
apps: "layoutGrid",
approvals: "badgeCheck",
workboard: "kanban",
dashboards: "layoutDashboard",
worktrees: "folder",
channels: "link",
connection: "radio",
sessions: "fileText",
usage: "coins",
cron: "calendarClock",
tasks: "listChecks",
agents: "bot",
skills: "zap",
plugins: "puzzle",
"skill-workshop": "wrench",
devices: "monitorSmartphone",
profile: "circleUser",
communications: "send",
appearance: "palette",
lobsterdex: "bug",
automation: "terminal",
mcp: "wrench",
memory: "book",
talk: "mic",
infrastructure: "globe",
labs: "flaskConical",
updates: "download",
about: "fileText",
"ai-agents": "brain",
"model-setup": "spark",
"model-providers": "plug",
"memory-import": "download",
notifications: "bell",
security: "shieldCheck",
secrets: "key",
advanced: "fileCode",
debug: "bug",
logs: "scrollText",
});
});
it("returns a fallback icon for unknown route", () => {
// TypeScript won't allow this normally, but runtime could receive unexpected values
const unknownRouteId = "unknown" as RouteId;
expect(navigationIconForRoute(unknownRouteId)).toBe("folder");
});
});
describe("settingsSearchTextMatches", () => {
it("uses locale-aware word prefixes for short queries", () => {
expect(settingsSearchTextMatches("CPU usage", "cp")).toBe(true);
expect(settingsSearchTextMatches("MCP", "cp")).toBe(false);
expect(settingsSearchTextMatches("外観設定", "設定")).toBe(true);
});
it.each([
["Cámara", "Ca\u0301mara"],
["Ca\u0301mara", "Cámara"],
["Notificación", "Notificacio\u0301n"],
["Notificacio\u0301n", "Notificación"],
])("matches canonically equivalent setting text %j against %j", (value, query) => {
expect(settingsSearchTextMatches(value, query)).toBe(true);
});
});
describe("formatDocumentTitle", () => {
it("suffixes the brand after a plain context", () => {
expect(formatDocumentTitle({ context: "Usage" })).toBe("Usage — OpenClaw");
});
it("does not duplicate a context ending in the brand", () => {
expect(formatDocumentTitle({ context: "Ask OpenClaw" })).toBe("Ask OpenClaw");
expect(formatDocumentTitle({ context: "OpenClaw" })).toBe("OpenClaw");
});
it("prefixes a positive attention count", () => {
expect(formatDocumentTitle({ context: "Usage", attentionCount: 2 })).toBe(
"(2) Usage — OpenClaw",
);
});
it("does not add a queued count for an empty offline outbox", () => {
expect(formatDocumentTitle({ context: "Usage", offline: true, queuedCount: 0 })).toBe(
"(Offline) Usage — OpenClaw",
);
});
it("includes the queued outbox count in the offline marker", () => {
expect(formatDocumentTitle({ context: "Usage", offline: true, queuedCount: 3 })).toBe(
"(Offline · 3 queued) Usage — OpenClaw",
);
});
it("ignores a queued count while online", () => {
expect(formatDocumentTitle({ context: "Usage", queuedCount: 3 })).toBe("Usage — OpenClaw");
});
it("suppresses the attention count while offline", () => {
expect(formatDocumentTitle({ context: "Usage", attentionCount: 2, offline: true })).toBe(
"(Offline) Usage — OpenClaw",
);
});
});
describe("titleForRoute", () => {
it("keeps every navigation title and subtitle backed by an English translation", () => {
// t() returns the raw dotted key (e.g. "tabs.advanced") when a catalog
// entry is missing; resolved copy is Title/Sentence case and never matches.
const rawI18nKey = /^[a-z][a-zA-Z0-9]*\.[a-zA-Z]/;
for (const routeId of ALL_ROUTES) {
expect(titleForRoute(routeId), routeId).not.toMatch(rawI18nKey);
expect(subtitleForRoute(routeId), routeId).not.toMatch(rawI18nKey);
}
});
it("returns expected titles for every route", () => {
expect(
Object.fromEntries(ALL_ROUTES.map((routeId) => [routeId, titleForRoute(routeId)])),
).toEqual({
chat: "Chat",
custodian: "OpenClaw",
activity: "Activity",
apps: "Apps",
approvals: "Approvals",
workboard: "Workboard",
dashboards: "Dashboards",
worktrees: "Worktrees",
channels: "Channels",
connection: "Gateway",
sessions: "Sessions",
usage: "Usage",
cron: "Automations",
tasks: "Tasks",
agents: "Agents",
skills: "Skills",
plugins: "Plugins",
"skill-workshop": "Skill Workshop",
devices: "Devices",
profile: "Profile",
communications: "Communications",
appearance: "Appearance",
lobsterdex: "Lobsterdex",
automation: "Automation",
mcp: "MCP",
memory: "Memory",
talk: "Talk",
infrastructure: "Infrastructure",
labs: "Labs",
updates: "Updates",
about: "About",
"ai-agents": "Agent Defaults",
"model-setup": "Model Setup",
"model-providers": "Models",
"memory-import": "Import Memory",
notifications: "Notifications",
security: "Privacy & Security",
secrets: "Secrets",
advanced: "Advanced",
debug: "Debug",
logs: "Logs",
});
});
});
describe("subtitleForRoute", () => {
it("returns expected subtitles for every route", () => {
expect(
Object.fromEntries(ALL_ROUTES.map((routeId) => [routeId, subtitleForRoute(routeId)])),
).toEqual({
chat: "Gateway chat for quick interventions.",
custodian: "System setup and care.",
activity: "Browser-local tool activity summaries.",
apps: "Companion apps for phone, watch, desktop, and browser.",
approvals: "Recent exec, plugin, and system-agent approvals.",
workboard: "Agent work queue and session handoff.",
dashboards: "Sessions that open on their dashboard face.",
worktrees: "Isolated agent task checkouts and recovery snapshots.",
channels: "Channels and settings.",
connection: "Gateway endpoint, credentials, and handshake status.",
sessions: "Active sessions and defaults.",
usage: "API usage and costs.",
cron: "Scheduled tasks and recurring agent runs.",
tasks: "Background tasks: subagents, automation runs, CLI.",
agents: "Workspaces, tools, identities.",
skills: "Skills and API keys.",
plugins: "Install and manage optional capabilities.",
"skill-workshop": "Review, refine, and apply proposals before they become live skills.",
devices: "Paired devices, pairing approvals, and exec bindings.",
profile: "Your display name, avatar, and identity on this gateway.",
communications: "Messages and text-to-speech settings.",
appearance: "Theme, UI, and setup wizard settings.",
lobsterdex: "Every lobster palette that has visited this browser.",
automation: "Commands, hooks, automations, and plugins.",
mcp: "MCP servers, auth, tools, and diagnostics.",
memory: "Memory engine, search, and dreaming.",
talk: "Realtime voice: provider, model, and speaker voice.",
infrastructure: "Gateway, browser, node host, discovery, and ACP settings.",
labs: "Experimental agent and tool capabilities.",
updates: "Release channel, automatic updates, and current update status.",
about: "Control UI and connected Gateway build identity.",
"ai-agents": "Global agent defaults: skills, tools, and session.",
"model-setup": "Connect a verified AI model",
"model-providers": "Default models, behavior, provider access, usage, and cost.",
"memory-import": "Bring Codex and Claude Code memory into an agent workspace.",
notifications: "Browser push notifications from your gateway.",
security: "Gateway auth, exec policy, tool profile, and approvals.",
secrets: "Secret values are hidden after saving. Env var values stay visible here.",
advanced: "Every remaining config section, plus the raw file editor.",
debug: "Snapshots, events, RPC.",
logs: "Live gateway logs.",
});
});
});
describe("pathForRoute", () => {
it("returns correct path without base", () => {
expect(pathForRoute("chat")).toBe("/chat");
expect(pathForRoute("apps")).toBe("/apps");
expect(pathForRoute("dashboards")).toBe("/dashboards");
expect(pathForRoute("custodian")).toBe("/custodian");
expect(pathForRoute("connection")).toBe("/settings/connection");
expect(pathForRoute("debug")).toBe("/debug");
expect(pathForRoute("logs")).toBe("/logs");
expect(pathForRoute("plugins")).toBe("/settings/plugins");
expect(pathForRoute("approvals")).toBe("/settings/approvals");
expect(pathForRoute("labs")).toBe("/settings/labs");
expect(pathForRoute("secrets")).toBe("/settings/secrets");
});
it("prepends base path", () => {
expect(pathForRoute("chat", "/ui")).toBe("/ui/chat");
expect(pathForRoute("sessions", "/apps/openclaw")).toBe("/apps/openclaw/sessions");
});
});
describe("route path normalization", () => {
it("normalizes base paths and trailing route slashes", () => {
expect(normalizeBasePath("")).toBe("");
expect(normalizeBasePath("/")).toBe("");
expect(normalizeBasePath("ui")).toBe("/ui");
expect(normalizeBasePath("/apps/openclaw/")).toBe("/apps/openclaw");
expect(routeIdFromPath("/chat/")).toBe("chat");
expect(routeIdFromPath("/ui/chat/", "/ui/")).toBe("chat");
});
});
describe("routeIdFromPath", () => {
it("returns tab for valid path", () => {
expect(routeIdFromPath("/chat")).toBe("chat");
expect(routeIdFromPath("/custodian")).toBe("custodian");
expect(routeIdFromPath("/new")).toBe("new-session");
expect(routeIdFromPath("/overview")).toBeNull();
expect(routeIdFromPath("/settings/connection")).toBe("connection");
expect(routeIdFromPath("/connection")).toBeNull();
expect(routeIdFromPath("/activity")).toBe("activity");
expect(routeIdFromPath("/apps")).toBe("apps");
expect(routeIdFromPath("/dashboards")).toBe("dashboards");
expect(routeIdFromPath("/sessions")).toBe("sessions");
expect(routeIdFromPath("/debug")).toBe("debug");
expect(routeIdFromPath("/logs")).toBe("logs");
expect(routeIdFromPath("/dreaming")).toBeNull();
expect(routeIdFromPath("/dreams")).toBeNull();
expect(routeIdFromPath("/settings/plugins")).toBe("plugins");
expect(routeIdFromPath("/plugins")).toBeNull();
expect(routeIdFromPath("/settings/about")).toBe("about");
expect(routeIdFromPath("/settings/labs")).toBe("labs");
expect(routeIdFromPath("/labs")).toBeNull();
expect(routeIdFromPath("/about")).toBeNull();
});
it("leaves root fallback to application startup", () => {
expect(routeIdFromPath("/")).toBeNull();
});
it("handles base paths", () => {
expect(routeIdFromPath("/ui/chat", "/ui")).toBe("chat");
expect(routeIdFromPath("/apps/openclaw/sessions", "/apps/openclaw")).toBe("sessions");
expect(routeIdFromPath("/ui/settings/plugins", "/ui")).toBe("plugins");
expect(routeIdFromPath("/xx/chat/main", "/ui")).toBeNull();
});
it("round-trips Workboard board paths", () => {
expect(pathForWorkboardBoard("ops.v2")).toBe("/workboard/ops%2Ev2");
expect(workboardBoardIdFromPath("/workboard/ops%2Ev2")).toBe("ops.v2");
expect(routeIdFromPath("/workboard/ops%2Ev2")).toBe("workboard");
expect(createApplicationRouter().routeIdFromPath("/workboard/ops%2Ev2")).toBe("workboard");
expect(pathForWorkboardBoard("ops", "/ui")).toBe("/ui/workboard/ops");
expect(workboardBoardIdFromPath("/ui/workboard/ops", "/ui")).toBe("ops");
expect(inferBasePathFromPathname("/ui/workboard/ops")).toBe("/ui");
});
it("round-trips session navigation through the lazy contract seam", () => {
const pathname = sessionNavigationTarget({
face: "chat",
sessionKey: "telegram:12345",
fallbackAgentId: "research",
basePath: "/ui",
}).options.pathname;
expect(pathname).toBe("/ui/chat/research/telegram/12345");
expect(sessionRefFromPath(pathname, "/ui")).toMatchObject({
kind: "literal",
sessionKey: "agent:research:telegram:12345",
});
expect(routeIdFromPath(pathname, "/ui")).toBe("chat");
expect(inferBasePathFromPathname(pathname)).toBe("/ui");
});
it("keeps dotted board IDs from resembling static asset paths", () => {
expect(pathForWorkboardBoard("release.js")).toBe("/workboard/release%2Ejs");
expect(workboardBoardIdFromPath("/workboard/release%2Ejs")).toBe("release.js");
});
it("rejects malformed Workboard board paths", () => {
expect(workboardBoardIdFromPath("/workboard/ops/extra")).toBeNull();
expect(workboardBoardIdFromPath("/workboard/%2F")).toBeNull();
expect(routeIdFromPath("/workboard/ops/extra")).toBeNull();
});
it("rejects route-shaped paths outside the configured base path", () => {
expect(routeIdFromPath("/xx/chat", "/ui")).toBeNull();
expect(routeIdFromPath("/other/sessions", "/apps/openclaw")).toBeNull();
});
it("returns null for unknown path", () => {
expect(routeIdFromPath("/unknown")).toBeNull();
expect(routeIdFromPath("/instances")).toBeNull();
});
it("matches canonical route casing exactly", () => {
expect(routeIdFromPath("/CHAT")).toBeNull();
expect(routeIdFromPath("/Sessions")).toBeNull();
});
});
describe("compiled settings routes", () => {
const router = createApplicationRouter();
it.each(SETTINGS_ROUTE_PATHS)(
"routes $routeId through its canonical path and legacy alias",
({ routeId, path, alias }) => {
expect(pathForRoute(routeId)).toBe(path);
expect(routeIdFromPath(path)).toBe(routeId);
expect(routeIdFromPath(alias)).toBe(routeId);
expect(router.pathForRoute(routeId)).toBe(path);
expect(router.routeIdFromPath(path)).toBe(routeId);
expect(router.routeIdFromPath(alias)).toBe(routeId);
},
);
it.each(SETTINGS_ROUTE_PATHS)(
"routes $routeId under a configured mount path",
({ routeId, path, alias }) => {
expect(pathForRoute(routeId, "/settings")).toBe(`/settings${path}`);
expect(routeIdFromPath(`/settings${path}`, "/settings")).toBe(routeId);
expect(routeIdFromPath(`/settings${alias}`, "/settings")).toBe(routeId);
expect(router.pathForRoute(routeId, "/settings")).toBe(`/settings${path}`);
expect(router.routeIdFromPath(`/settings${path}`, "/settings")).toBe(routeId);
expect(router.routeIdFromPath(`/settings${alias}`, "/settings")).toBe(routeId);
},
);
});
describe("inferBasePathFromPathname", () => {
it("handles direct routes, nested mounts, mount roots, and index.html", () => {
expect(inferBasePathFromPathname("/")).toBe("");
expect(inferBasePathFromPathname("/chat")).toBe("");
expect(inferBasePathFromPathname("/custodian")).toBe("");
expect(inferBasePathFromPathname("/settings/connection")).toBe("");
expect(inferBasePathFromPathname("/ui/chat")).toBe("/ui");
expect(inferBasePathFromPathname("/apps/openclaw/sessions")).toBe("/apps/openclaw");
expect(inferBasePathFromPathname("/__openclaw__/")).toBe("/__openclaw__");
expect(inferBasePathFromPathname("/apps/openclaw/")).toBe("/apps/openclaw");
expect(inferBasePathFromPathname("/typo")).toBe("");
expect(inferBasePathFromPathname("/index.html")).toBe("");
expect(inferBasePathFromPathname("/ui/index.html")).toBe("/ui");
});
it("never infers a route namespace as a mount base", () => {
// "/settings/config" is not a route; matching the "/config" alias must not
// rescope the page to base "/settings" or reconnect state and assets break.
expect(inferBasePathFromPathname("/settings/config")).toBe("");
expect(inferBasePathFromPathname("/settings/config/")).toBe("");
expect(inferBasePathFromPathname("/settings/chat/main")).toBe("");
// A leaf route is equally not a mount directory.
expect(inferBasePathFromPathname("/custodian/config")).toBe("");
// Nested unknown segments below a route namespace stay root-mounted too.
expect(inferBasePathFromPathname("/settings/other/config")).toBe("");
expect(inferBasePathFromPathname("/settings/")).toBe("");
expect(inferBasePathFromPathname("/skills/")).toBe("");
// Real mount directories that merely contain a route-suffix keep working.
expect(inferBasePathFromPathname("/ui/config")).toBe("/ui");
expect(inferBasePathFromPathname("/ui/settings/appearance")).toBe("/ui");
});
});
describe("plugin tabs route", () => {
it("round-trips the shared /plugin route", () => {
expect(pathForRoute("plugin", "")).toBe("/plugin");
expect(routeIdFromPath("/plugin", "")).toBe("plugin");
// The tab id travels in the search, not the pathname.
expect(routeIdFromPath("/plugin/logbook", "")).toBeNull();
});
it("round-trips a namespaced tab reference through the search", () => {
const ref = { pluginId: "logbook", id: "logbook" };
expect(pluginTabRefFromSearch(pluginTabSearch(ref))).toEqual(ref);
expect(pluginTabKey(ref)).toBe("logbook/logbook");
// Distinct plugins with the same local tab id stay distinct.
expect(pluginTabKey({ pluginId: "other", id: "logbook" })).not.toBe(pluginTabKey(ref));
});
it("stays out of the customizable static sidebar routes", () => {
expect(SIDEBAR_NAV_ROUTES).not.toContain("plugin");
expect(SIDEBAR_NAV_ROUTES).toContain("plugins");
expect(routeIdFromPath("/settings/plugins")).toBe("plugins");
expect(routeIdFromPath("/plugins")).toBeNull();
});
});
describe("SIDEBAR_NAV_ROUTES", () => {
it("all routes are unique", () => {
expect(new Set(SIDEBAR_NAV_ROUTES).size).toBe(SIDEBAR_NAV_ROUTES.length);
});
it("collapses the plugins hub to a single sidebar entry", () => {
expect(SIDEBAR_NAV_ROUTES).not.toContain("skills");
expect(SIDEBAR_NAV_ROUTES).not.toContain("skill-workshop");
expect(isPluginsHubRoute("plugins")).toBe(true);
expect(isPluginsHubRoute("skills")).toBe(true);
expect(isPluginsHubRoute("skill-workshop")).toBe(true);
expect(isPluginsHubRoute("sessions")).toBe(false);
});
it("keeps detailed settings slices routed but out of the customizable sidebar", () => {
const settingsRoutes = SETTINGS_NAVIGATION_GROUPS.flatMap((group) => group.routes);
expect(SIDEBAR_NAV_ROUTES).not.toContain("config");
expect(settingsRoutes).toEqual([
"custodian",
"profile",
"appearance",
"notifications",
"connection",
"channels",
"communications",
"talk",
"devices",
"agents",
"labs",
"model-providers",
"mcp",
"memory",
"automation",
"security",
"secrets",
"approvals",
"infrastructure",
"advanced",
"debug",
"logs",
"updates",
"about",
]);
});
it("keeps settings sidebar groups unique with personal settings first", () => {
const settingsRoutes = SETTINGS_NAVIGATION_GROUPS.flatMap((group) => group.routes);
expect(new Set(settingsRoutes).size).toBe(settingsRoutes.length);
const [firstGroup] = SETTINGS_NAVIGATION_GROUPS;
expect(firstGroup?.labelKey).toBeNull();
expect(firstGroup?.routes).toEqual(["custodian", "profile", "appearance", "notifications"]);
for (const group of SETTINGS_NAVIGATION_GROUPS.slice(1)) {
expect(group.labelKey).toBeTruthy();
}
});
});