refactor: hide ui helper internals

steipete · steipete · commit 49e2992be5a9 · 2026-05-02T07:10:14.000+01:00
diff --git a/ui/src/ui/assistant-identity.ts b/ui/src/ui/assistant-identity.ts
@@ -14,7 +14,7 @@ const MAX_ASSISTANT_AVATAR_REASON = 200;
 // this module free of UI view imports (avoids an import cycle).
 const RENDERABLE_AVATAR_URL_RE = /^(data:image\/|\/(?!\/))/i;
 
-export const DEFAULT_ASSISTANT_NAME = "Assistant";
+const DEFAULT_ASSISTANT_NAME = "Assistant";
 export const DEFAULT_ASSISTANT_AVATAR = "A";
 
 export type AssistantIdentity = {
diff --git a/ui/src/ui/chat/attachment-payload-store.ts b/ui/src/ui/chat/attachment-payload-store.ts
@@ -50,7 +50,7 @@ export function getChatAttachmentPreviewUrl(attachment: ChatAttachment): string
   );
 }
 
-export function cloneChatAttachmentMetadata(attachment: ChatAttachment): ChatAttachment {
+function cloneChatAttachmentMetadata(attachment: ChatAttachment): ChatAttachment {
   const { dataUrl: _dataUrl, ...metadata } = attachment;
   return metadata;
 }
@@ -76,7 +76,7 @@ export function releaseChatAttachmentPayloads(attachments: readonly ChatAttachme
   }
 }
 
-export function discardChatAttachmentDataUrl(id: string): void {
+function discardChatAttachmentDataUrl(id: string): void {
   const payload = payloads.get(id);
   if (!payload) {
     return;
diff --git a/ui/src/ui/chat/session-cache.ts b/ui/src/ui/chat/session-cache.ts
@@ -1,4 +1,4 @@
-export const MAX_CACHED_CHAT_SESSIONS = 20;
+const MAX_CACHED_CHAT_SESSIONS = 20;
 
 export function getOrCreateSessionCacheValue<T>(
   map: Map<string, T>,
diff --git a/ui/src/ui/controllers/config.ts b/ui/src/ui/controllers/config.ts
@@ -78,7 +78,7 @@ export async function loadConfigSchema(state: ConfigState) {
   }
 }
 
-export function applyConfigSchema(state: ConfigState, res: ConfigSchemaResponse) {
+function applyConfigSchema(state: ConfigState, res: ConfigSchemaResponse) {
   state.configSchema = res.schema ?? null;
   state.configUiHints = res.uiHints ?? {};
   state.configSchemaVersion = res.version ?? null;
diff --git a/ui/src/ui/controllers/cron.ts b/ui/src/ui/controllers/cron.ts
@@ -89,9 +89,7 @@ export type CronModelSuggestionsState = {
   cronModelSuggestions: string[];
 };
 
-export function supportsAnnounceDelivery(
-  form: Pick<CronFormState, "sessionTarget" | "payloadKind">,
-) {
+function supportsAnnounceDelivery(form: Pick<CronFormState, "sessionTarget" | "payloadKind">) {
   return form.sessionTarget !== "main" && form.payloadKind === "agentTurn";
 }
 
@@ -523,7 +521,7 @@ function jobToForm(job: CronJob, prev: CronFormState): CronFormState {
   return normalizeCronFormState(next);
 }
 
-export function buildCronSchedule(form: CronFormState) {
+function buildCronSchedule(form: CronFormState) {
   if (form.scheduleKind === "at") {
     const ms = Date.parse(form.scheduleAt);
     if (!Number.isFinite(ms)) {
@@ -559,7 +557,7 @@ export function buildCronSchedule(form: CronFormState) {
   return { kind: "cron" as const, expr, tz: form.cronTz.trim() || undefined, staggerMs };
 }
 
-export function buildCronPayload(form: CronFormState) {
+function buildCronPayload(form: CronFormState) {
   if (form.payloadKind === "systemEvent") {
     const text = form.payloadText.trim();
     if (!text) {
diff --git a/ui/src/ui/controllers/exec-approvals.ts b/ui/src/ui/controllers/exec-approvals.ts
@@ -106,10 +106,7 @@ export async function loadExecApprovals(
   }
 }
 
-export function applyExecApprovalsSnapshot(
-  state: ExecApprovalsState,
-  snapshot: ExecApprovalsSnapshot,
-) {
+function applyExecApprovalsSnapshot(state: ExecApprovalsState, snapshot: ExecApprovalsSnapshot) {
   state.execApprovalsSnapshot = snapshot;
   if (!state.execApprovalsDirty) {
     state.execApprovalsForm = cloneConfigObject(snapshot.file ?? {});
diff --git a/ui/src/ui/gateway.ts b/ui/src/ui/gateway.ts
@@ -147,7 +147,7 @@ type SelectedConnectAuth = {
   canFallbackToShared: boolean;
 };
 
-export const CONTROL_UI_OPERATOR_ROLE = "operator";
+const CONTROL_UI_OPERATOR_ROLE = "operator";
 
 export const CONTROL_UI_OPERATOR_SCOPES = [
   "operator.admin",
diff --git a/ui/src/ui/session-key.ts b/ui/src/ui/session-key.ts
@@ -36,7 +36,7 @@ export function parseAgentSessionKey(
   return { agentId, rest };
 }
 
-export function normalizeMainKey(value: string | undefined | null): string {
+function normalizeMainKey(value: string | undefined | null): string {
   return normalizeOptionalLowercaseString(value) ?? DEFAULT_MAIN_KEY;
 }
 
diff --git a/ui/src/ui/theme.ts b/ui/src/ui/theme.ts
@@ -11,7 +11,7 @@ export type ResolvedTheme =
   | "custom-light";
 
 export const VALID_THEME_NAMES = new Set<ThemeName>(["claw", "knot", "dash", "custom"]);
-export const VALID_THEME_MODES = new Set<ThemeMode>(["system", "light", "dark"]);
+const VALID_THEME_MODES = new Set<ThemeMode>(["system", "light", "dark"]);
 
 type ThemeSelection = { theme: ThemeName; mode: ThemeMode };
 
@@ -29,7 +29,7 @@ const LEGACY_MAP: Record<string, ThemeSelection> = {
   system: { theme: "claw", mode: "system" },
 };
 
-export function prefersLightScheme(): boolean {
+function prefersLightScheme(): boolean {
   if (typeof globalThis.matchMedia !== "function") {
     return false;
   }

Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ export function getChatAttachmentPreviewUrl(attachment: ChatAttachment): string`
`50`	`50`	`);`
`51`	`51`	`}`
`52`	`52`
`53`		`-export function cloneChatAttachmentMetadata(attachment: ChatAttachment): ChatAttachment {`
	`53`	`+function cloneChatAttachmentMetadata(attachment: ChatAttachment): ChatAttachment {`
`54`	`54`	`const { dataUrl: _dataUrl, ...metadata } = attachment;`
`55`	`55`	`return metadata;`
`56`	`56`	`}`
`@@ -76,7 +76,7 @@ export function releaseChatAttachmentPayloads(attachments: readonly ChatAttachme`
`76`	`76`	`}`
`77`	`77`	`}`
`78`	`78`
`79`		`-export function discardChatAttachmentDataUrl(id: string): void {`
	`79`	`+function discardChatAttachmentDataUrl(id: string): void {`
`80`	`80`	`const payload = payloads.get(id);`
`81`	`81`	`if (!payload) {`
`82`	`82`	`return;`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-export const MAX_CACHED_CHAT_SESSIONS = 20;`
	`1`	`+const MAX_CACHED_CHAT_SESSIONS = 20;`
`2`	`2`
`3`	`3`	`export function getOrCreateSessionCacheValue<T>(`
`4`	`4`	`map: Map<string, T>,`
Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ export async function loadConfigSchema(state: ConfigState) {`
`78`	`78`	`}`
`79`	`79`	`}`
`80`	`80`
`81`		`-export function applyConfigSchema(state: ConfigState, res: ConfigSchemaResponse) {`
	`81`	`+function applyConfigSchema(state: ConfigState, res: ConfigSchemaResponse) {`
`82`	`82`	`state.configSchema = res.schema ?? null;`
`83`	`83`	`state.configUiHints = res.uiHints ?? {};`
`84`	`84`	`state.configSchemaVersion = res.version ?? null;`
Original file line number	Diff line number	Diff line change
`@@ -106,10 +106,7 @@ export async function loadExecApprovals(`
`106`	`106`	`}`
`107`	`107`	`}`
`108`	`108`
`109`		`-export function applyExecApprovalsSnapshot(`
`110`		`- state: ExecApprovalsState,`
`111`		`- snapshot: ExecApprovalsSnapshot,`
`112`		`-) {`
	`109`	`+function applyExecApprovalsSnapshot(state: ExecApprovalsState, snapshot: ExecApprovalsSnapshot) {`
`113`	`110`	`state.execApprovalsSnapshot = snapshot;`
`114`	`111`	`if (!state.execApprovalsDirty) {`
`115`	`112`	`state.execApprovalsForm = cloneConfigObject(snapshot.file ?? {});`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ export function parseAgentSessionKey(`
`36`	`36`	`return { agentId, rest };`
`37`	`37`	`}`
`38`	`38`
`39`		`-export function normalizeMainKey(value: string \| undefined \| null): string {`
	`39`	`+function normalizeMainKey(value: string \| undefined \| null): string {`
`40`	`40`	`return normalizeOptionalLowercaseString(value) ?? DEFAULT_MAIN_KEY;`
`41`	`41`	`}`
`42`	`42`