refactor(agents): localize internal-only symbols (#100909)

vincentkoc · web-flow · commit e35c7a3c95b2 · 2026-07-06T06:40:19.000-07:00
diff --git a/src/agents/acp-spawn.ts b/src/agents/acp-spawn.ts
@@ -118,12 +118,12 @@ const ACP_RUNTIME_TIMEOUT_MAX_SECONDS = 24 * 60 * 60;
 
 export const ACP_SPAWN_MODES = ["run", "session"] as const;
 export type SpawnAcpMode = (typeof ACP_SPAWN_MODES)[number];
-export const ACP_SPAWN_SANDBOX_MODES = ["inherit", "require"] as const;
+const ACP_SPAWN_SANDBOX_MODES = ["inherit", "require"] as const;
 export type SpawnAcpSandboxMode = (typeof ACP_SPAWN_SANDBOX_MODES)[number];
 export const ACP_SPAWN_STREAM_TARGETS = ["parent"] as const;
-export type SpawnAcpStreamTarget = (typeof ACP_SPAWN_STREAM_TARGETS)[number];
+type SpawnAcpStreamTarget = (typeof ACP_SPAWN_STREAM_TARGETS)[number];
 
-export type SpawnAcpParams = {
+type SpawnAcpParams = {
   task: string;
   label?: string;
   agentId?: string;
@@ -182,7 +182,7 @@ export type SpawnAcpContext = {
   inheritedToolDenylist?: string[];
 };
 
-export const ACP_SPAWN_ERROR_CODES = [
+const ACP_SPAWN_ERROR_CODES = [
   "acp_disabled",
   "requester_session_required",
   "runtime_policy",
diff --git a/src/agents/agent-bundle-lsp-runtime.ts b/src/agents/agent-bundle-lsp-runtime.ts
@@ -51,7 +51,7 @@ type LspServerCapabilities = {
 };
 
 /** Materialized LSP tools plus session capabilities and cleanup handle. */
-export type BundleLspToolRuntime = {
+type BundleLspToolRuntime = {
   tools: AnyAgentTool[];
   sessions: Array<{ serverName: string; capabilities: LspServerCapabilities }>;
   dispose: () => Promise<void>;
diff --git a/src/agents/bash-tools.exec-runtime.ts b/src/agents/bash-tools.exec-runtime.ts
@@ -200,7 +200,7 @@ function emitExecProcessCompleted(params: {
 }
 
 /** Renders a host label for user-facing exec policy messages. */
-export function renderExecHostLabel(host: ExecHost) {
+function renderExecHostLabel(host: ExecHost) {
   return host === "sandbox" ? "sandbox" : host === "gateway" ? "gateway" : "node";
 }
 
diff --git a/src/agents/cli-runner/claude-skills-plugin.ts b/src/agents/cli-runner/claude-skills-plugin.ts
@@ -35,7 +35,7 @@ function sanitizeSkillDirName(name: string, used: Set<string>): string {
 }
 
 /** Returns whether a resolved skill file is readable before linking it into the Claude plugin. */
-export function isClaudeCliSkillFileAccessible(skillFilePath: string): boolean {
+function isClaudeCliSkillFileAccessible(skillFilePath: string): boolean {
   try {
     accessSync(skillFilePath);
     return true;
diff --git a/src/agents/cli-runner/reseed-envelope.ts b/src/agents/cli-runner/reseed-envelope.ts
@@ -10,7 +10,7 @@ const RESEED_PREFIX = `${RESEED_HEADER}\n`;
 const RESEED_USER_BOUNDARY = "\n</conversation_history>\n\n<next_user_message>\n";
 const RESEED_USER_CLOSE = "\n</next_user_message>";
 
-export type ParsedCliReseedPrompt =
+type ParsedCliReseedPrompt =
   | { kind: "none" }
   | { kind: "legacy"; userMessage: string }
   | { kind: "invalid" };
diff --git a/src/agents/cli-session.ts b/src/agents/cli-session.ts
@@ -114,12 +114,7 @@ export function clearAllCliSessions(entry: Partial<MutableCliSessionFields>): vo
   entry.claudeCliSessionId = undefined;
 }
 
-export type CliSessionInvalidatedReason =
-  | "auth-profile"
-  | "auth-epoch"
-  | "message-policy"
-  | "cwd"
-  | "mcp";
+type CliSessionInvalidatedReason = "auth-profile" | "auth-epoch" | "message-policy" | "cwd" | "mcp";
 
 export type CliSessionContentDriftReason = "system-prompt" | "prompt-tools";
 
diff --git a/src/agents/code-mode-control-tools.ts b/src/agents/code-mode-control-tools.ts
@@ -14,7 +14,7 @@ export const CODE_MODE_WAIT_TOOL_NAME = "wait";
 export const CODE_MODE_EXEC_TOOL_KIND = "code_mode_exec";
 
 /** Hook metadata kind type for Code Mode exec tools. */
-export type CodeModeExecToolKind = typeof CODE_MODE_EXEC_TOOL_KIND;
+type CodeModeExecToolKind = typeof CODE_MODE_EXEC_TOOL_KIND;
 /** Source language accepted by the Code Mode exec tool. */
 export type CodeModeExecToolInputKind = "javascript" | "typescript";
 /** Metadata attached to before-tool-call events for Code Mode exec. */
diff --git a/src/agents/embedded-agent-helpers/errors.ts b/src/agents/embedded-agent-helpers/errors.ts
@@ -1587,7 +1587,7 @@ export function formatAssistantErrorText(
   return raw.length > 600 ? `${truncateUtf16Safe(raw, 600)}…` : raw;
 }
 
-export function isRawAssistantErrorPassthrough(params: {
+function isRawAssistantErrorPassthrough(params: {
   friendlyError?: string;
   rawError?: string;
 }): boolean {
@@ -1653,7 +1653,7 @@ const IMAGE_DIMENSION_ERROR_RE =
 const IMAGE_DIMENSION_PATH_RE = /messages\.(\d+)\.content\.(\d+)\.image/i;
 const IMAGE_SIZE_ERROR_RE = /image exceeds\s*(\d+(?:\.\d+)?)\s*mb/i;
 
-export function isMissingToolCallInputError(raw: string): boolean {
+function isMissingToolCallInputError(raw: string): boolean {
   if (!raw) {
     return false;
   }
diff --git a/src/agents/embedded-agent-runner/run/types.ts b/src/agents/embedded-agent-runner/run/types.ts
@@ -43,7 +43,7 @@ type EmbeddedRunAttemptBase = Omit<
   | "sessionFile"
 >;
 
-export type EmbeddedRunContextWindowInfo = {
+type EmbeddedRunContextWindowInfo = {
   tokens: number;
   referenceTokens?: number;
   source: "model" | "modelsConfig" | "agentContextTokens" | "default";
diff --git a/src/agents/model-selection-shared.ts b/src/agents/model-selection-shared.ts
@@ -1117,7 +1117,7 @@ export type ModelRefStatus = {
   allowed: boolean;
 };
 
-export type ResolveAllowedModelRefResult =
+type ResolveAllowedModelRefResult =
   | { ref: ModelRef; key: string }
   | {
       error: string;
diff --git a/src/agents/models-config.ts b/src/agents/models-config.ts
@@ -41,7 +41,7 @@ import { stableStringify } from "./stable-stringify.js";
 
 export { resetModelsJsonReadyCacheForTest } from "./models-config-state.js";
 
-export type PreparedOpenClawModelsJsonSource = ModelsJsonReadyResult & {
+type PreparedOpenClawModelsJsonSource = ModelsJsonReadyResult & {
   fingerprint: string;
   workspaceDir?: string;
 };
diff --git a/src/agents/modes/interactive/theme/theme.ts b/src/agents/modes/interactive/theme/theme.ts
@@ -695,7 +695,7 @@ function startThemeWatcher(): void {
     ) ?? undefined;
 }
 
-export function stopThemeWatcher(): void {
+function stopThemeWatcher(): void {
   if (themeReloadTimer) {
     clearTimeout(themeReloadTimer);
     themeReloadTimer = undefined;
diff --git a/src/agents/run-session-target.ts b/src/agents/run-session-target.ts
@@ -17,7 +17,7 @@ export type AgentRunSessionTarget = {
 };
 
 /** File-backed target resolved from the storage-neutral run identity. */
-export type ResolvedAgentRunSessionTarget = SessionTranscriptRuntimeTarget;
+type ResolvedAgentRunSessionTarget = SessionTranscriptRuntimeTarget;
 
 /** Resolves the active file-backed target used by current run/session internals. */
 export async function resolveAgentRunSessionTarget(params: {
diff --git a/src/agents/run-wait.ts b/src/agents/run-wait.ts
@@ -70,7 +70,7 @@ export type AgentWaitResult = {
 };
 
 /** Summary returned after waiting for a dynamic set of pending runs to drain. */
-export type AgentRunsDrainResult = {
+type AgentRunsDrainResult = {
   timedOut: boolean;
   pendingRunIds: string[];
   deadlineAtMs: number;
diff --git a/src/agents/sandbox/docker.ts b/src/agents/sandbox/docker.ts
@@ -206,7 +206,7 @@ const log = createSubsystemLogger("docker");
 
 const HOT_CONTAINER_WINDOW_MS = 5 * 60 * 1000;
 
-export type ExecDockerOptions = ExecDockerRawOptions;
+type ExecDockerOptions = ExecDockerRawOptions;
 
 function envRecordsEqual(left: Record<string, string>, right: Record<string, string>): boolean {
   const leftEntries = Object.entries(left).toSorted(([leftKey], [rightKey]) =>
diff --git a/src/agents/session-suspension.ts b/src/agents/session-suspension.ts
@@ -21,7 +21,7 @@ import type { FailoverReason } from "./embedded-agent-helpers/types.js";
 const log = createSubsystemLogger("session-suspension");
 
 const DEFAULT_CUSTOM_LANE_RESUME_CONCURRENCY = 1;
-export const DEFAULT_QUOTA_SUSPENSION_RESUME_MS = 30 * 60 * 1000; // 30 min
+const DEFAULT_QUOTA_SUSPENSION_RESUME_MS = 30 * 60 * 1000; // 30 min
 
 const laneResumeTimers = new Map<string, ReturnType<typeof setTimeout>>();
 const deferredSessionSuspension = new AsyncLocalStorage<{
@@ -30,7 +30,7 @@ const deferredSessionSuspension = new AsyncLocalStorage<{
 }>();
 
 export type SessionSuspensionReason = "quota_exhausted" | "manual" | "circuit_open";
-export type SessionSuspensionTarget =
+type SessionSuspensionTarget =
   | { mode: "defer"; defer: (params: SessionSuspensionParams) => void }
   | { mode: "suspend" };
 export type SessionSuspensionParams = {
diff --git a/src/agents/sessions/extensions/runner.ts b/src/agents/sessions/extensions/runner.ts
@@ -191,7 +191,7 @@ export type SwitchSessionHandler = (
   options?: { withSession?: (ctx: ReplacedSessionContext) => Promise<void> },
 ) => Promise<{ cancelled: boolean }>;
 
-export type ReloadHandler = () => Promise<void>;
+type ReloadHandler = () => Promise<void>;
 
 export type ShutdownHandler = () => void;
 
diff --git a/src/agents/sessions/keybindings.ts b/src/agents/sessions/keybindings.ts
@@ -295,7 +295,7 @@ function toKeybindingsConfig(value: unknown): KeybindingsConfig {
 }
 
 /** Migrates legacy keybinding names and orders known entries ahead of unknown extras. */
-export function migrateKeybindingsConfig(rawConfig: Record<string, unknown>): {
+function migrateKeybindingsConfig(rawConfig: Record<string, unknown>): {
   config: Record<string, unknown>;
   migrated: boolean;
 } {
diff --git a/src/agents/sessions/prompt-templates.ts b/src/agents/sessions/prompt-templates.ts
@@ -108,7 +108,7 @@ function loadTemplatesFromDir(
   return templates;
 }
 
-export interface LoadPromptTemplatesOptions {
+interface LoadPromptTemplatesOptions {
   /** Working directory for project-local templates. */
   cwd: string;
   /** Agent config directory for global templates. */
diff --git a/src/agents/sessions/tools/edit-diff.ts b/src/agents/sessions/tools/edit-diff.ts
@@ -150,7 +150,7 @@ function applyReplacements(content: string, replacements: TextReplacement[], off
  * Rewrite only lines touched by fuzzy replacements. Untouched lines retain
  * their original bytes even though matching used normalized content.
  */
-export function applyReplacementsPreservingUnchangedLines(
+function applyReplacementsPreservingUnchangedLines(
   originalContent: string,
   baseContent: string,
   replacements: TextReplacement[],
diff --git a/src/agents/subagent-registry.types.ts b/src/agents/subagent-registry.types.ts
@@ -84,7 +84,7 @@ export type SubagentCompletionDeliveryState = {
     | "waiting_for_requester_turn";
 };
 
-export type SubagentKillReconciliationState = {
+type SubagentKillReconciliationState = {
   /** Actual cancellation time; a yielded run may have an older execution end. */
   killedAt: number;
   /** Requester aborts must not re-inject a delayed completion after queues are cleared. */
diff --git a/src/agents/subagent-spawn-ownership.ts b/src/agents/subagent-spawn-ownership.ts
@@ -10,7 +10,7 @@ import {
   resolveMainSessionAlias,
 } from "./tools/sessions-helpers.js";
 
-export type SubagentSpawnOwnership = {
+type SubagentSpawnOwnership = {
   controllerSessionKey: string;
   threadBindingRequesterSessionKey: string;
   completionRequesterSessionKey: string;
diff --git a/src/agents/subagent-spawn.ts b/src/agents/subagent-spawn.ts
@@ -158,7 +158,7 @@ const SUBAGENT_CONTROL_GATEWAY_TIMEOUT_MS = 60_000;
 const DEFAULT_SUBAGENT_AGENT_GATEWAY_TIMEOUT_MS = 60_000;
 const MAX_SUBAGENT_AGENT_GATEWAY_TIMEOUT_MS = 300_000;
 
-export type SpawnSubagentParams = {
+type SpawnSubagentParams = {
   task: string;
   label?: string;
   agentId?: string;
@@ -183,7 +183,7 @@ export type SpawnSubagentParams = {
   attachMountPath?: string;
 };
 
-export type SpawnSubagentContext = {
+type SpawnSubagentContext = {
   agentSessionKey?: string;
   /** Separate key used only for completion routing, not sandbox policy. */
   completionOwnerKey?: string;
diff --git a/src/agents/tools-effective-inventory-build.ts b/src/agents/tools-effective-inventory-build.ts
@@ -129,7 +129,7 @@ export function buildReadableToolsByName(
 }
 
 /** Builds effective inventory entries from already runtime-compatible tools. */
-export function buildEffectiveToolInventoryEntries(
+function buildEffectiveToolInventoryEntries(
   tools: readonly AnyAgentTool[],
   rawToolsByName: ReadonlyMap<string, AnyAgentTool> = new Map(),
 ): EffectiveToolInventoryEntry[] {
diff --git a/src/agents/tools/skill-workshop-tool.ts b/src/agents/tools/skill-workshop-tool.ts
@@ -124,7 +124,7 @@ const SkillWorkshopToolSchema = Type.Object(
   { additionalProperties: false },
 );
 
-export type SkillWorkshopToolOptions = {
+type SkillWorkshopToolOptions = {
   workspaceDir: string;
   config?: OpenClawConfig;
   agentId?: string;
diff --git a/src/agents/utils/image-resize.ts b/src/agents/utils/image-resize.ts
@@ -28,7 +28,7 @@ interface ResizedImage {
   wasResized: boolean;
 }
 
-export type ProcessImageResult =
+type ProcessImageResult =
   | { ok: true; image: ImageContent; hints: string[] }
   | { ok: false; message: string };
 
diff --git a/src/agents/utils/syntax-highlight.ts b/src/agents/utils/syntax-highlight.ts
@@ -13,7 +13,7 @@ export type HighlightFormatter = (text: string) => string;
 export type HighlightTheme = Partial<Record<string, HighlightFormatter>>;
 
 /** Options used when highlighting code and rendering themed text. */
-export interface HighlightOptions {
+interface HighlightOptions {
   language?: string;
   ignoreIllegals?: boolean;
   languageSubset?: string[];
@@ -96,7 +96,7 @@ function isSpanOpenTagStart(html: string, index: number): boolean {
 }
 
 /** Renders highlight.js span HTML into themed plain text. */
-export function renderHighlightedHtml(html: string, theme: HighlightTheme = {}): string {
+function renderHighlightedHtml(html: string, theme: HighlightTheme = {}): string {
   let output = "";
   let textBuffer = "";
   const scopes: Array<string | undefined> = [];
diff --git a/src/agents/worktrees/git.ts b/src/agents/worktrees/git.ts
@@ -53,7 +53,7 @@ export async function requireGitRaw(cwd: string, args: string[]): Promise<string
   return result.stdout;
 }
 
-export function parseWorktreeList(output: string): WorktreeListEntry[] {
+function parseWorktreeList(output: string): WorktreeListEntry[] {
   const entries: WorktreeListEntry[] = [];
   let current: WorktreeListEntry | undefined;
   for (const field of output.split("\0")) {
diff --git a/src/agents/worktrees/service.ts b/src/agents/worktrees/service.ts
@@ -48,7 +48,7 @@ type ServiceOptions = {
   now?: () => number;
 };
 
-export type ManagedWorktreeGcParams = {
+type ManagedWorktreeGcParams = {
   isOwnerActive?: (ownerKind: ManagedWorktreeOwnerKind, ownerId: string) => boolean;
 };
 

Original file line number	Diff line number	Diff line change
`@@ -200,7 +200,7 @@ function emitExecProcessCompleted(params: {`
`200`	`200`	`}`
`201`	`201`
`202`	`202`	`/** Renders a host label for user-facing exec policy messages. */`
`203`		`-export function renderExecHostLabel(host: ExecHost) {`
	`203`	`+function renderExecHostLabel(host: ExecHost) {`
`204`	`204`	`return host === "sandbox" ? "sandbox" : host === "gateway" ? "gateway" : "node";`
`205`	`205`	`}`
`206`	`206`
Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ function sanitizeSkillDirName(name: string, used: Set<string>): string {`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`/** Returns whether a resolved skill file is readable before linking it into the Claude plugin. */`
`38`		`-export function isClaudeCliSkillFileAccessible(skillFilePath: string): boolean {`
	`38`	`+function isClaudeCliSkillFileAccessible(skillFilePath: string): boolean {`
`39`	`39`	`try {`
`40`	`40`	`accessSync(skillFilePath);`
`41`	`41`	`return true;`