From f86cb612b94c7cbac4b37502522cc4f894638da3 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Fri, 1 May 2026 21:09:05 +0100 Subject: [PATCH] refactor: trim tool display exports --- src/agents/tool-display-common.ts | 26 ++++++++++++-------------- 1 file changed, 12 insertions(+), 14 deletions(-) diff --git a/src/agents/tool-display-common.ts b/src/agents/tool-display-common.ts index 487682de7da..c8123e20a2f 100644 --- a/src/agents/tool-display-common.ts +++ b/src/agents/tool-display-common.ts @@ -44,7 +44,7 @@ export function defaultTitle(name: string): string { .join(" "); } -export function normalizeVerb(value?: string): string | undefined { +function normalizeVerb(value?: string): string | undefined { const trimmed = normalizeOptionalString(value); if (!trimmed) { return undefined; @@ -52,7 +52,7 @@ export function normalizeVerb(value?: string): string | undefined { return trimmed.replace(/_/g, " "); } -export function resolveActionArg(args: unknown): string | undefined { +function resolveActionArg(args: unknown): string | undefined { if (!args || typeof args !== "object") { return undefined; } @@ -89,7 +89,7 @@ export function resolveToolVerbAndDetailForArgs(params: { }); } -export function coerceDisplayValue( +function coerceDisplayValue( value: unknown, opts: CoerceDisplayValueOptions = {}, ): string | undefined { @@ -141,7 +141,7 @@ export function coerceDisplayValue( return undefined; } -export function lookupValueByPath(args: unknown, path: string): unknown { +function lookupValueByPath(args: unknown, path: string): unknown { if (!args || typeof args !== "object") { return undefined; } @@ -171,7 +171,7 @@ export function formatDetailKey(raw: string, overrides: Record = return normalizeLowercaseStringOrEmpty(spaced) || normalizeLowercaseStringOrEmpty(last); } -export function resolvePathArg(args: unknown): string | undefined { +function resolvePathArg(args: unknown): string | undefined { const record = asRecord(args); if (!record) { return undefined; @@ -188,7 +188,7 @@ export function resolvePathArg(args: unknown): string | undefined { return undefined; } -export function resolveReadDetail(args: unknown): string | undefined { +function resolveReadDetail(args: unknown): string | undefined { const record = asRecord(args); if (!record) { return undefined; @@ -225,7 +225,7 @@ export function resolveReadDetail(args: unknown): string | undefined { return `from ${path}`; } -export function resolveWriteDetail(toolKey: string, args: unknown): string | undefined { +function resolveWriteDetail(toolKey: string, args: unknown): string | undefined { const record = asRecord(args); if (!record) { return undefined; @@ -257,7 +257,7 @@ export function resolveWriteDetail(toolKey: string, args: unknown): string | und return `${destinationPrefix} ${path}`; } -export function resolveWebSearchDetail(args: unknown): string | undefined { +function resolveWebSearchDetail(args: unknown): string | undefined { const record = asRecord(args); if (!record) { return undefined; @@ -276,7 +276,7 @@ export function resolveWebSearchDetail(args: unknown): string | undefined { return count !== undefined ? `for "${query}" (top ${count})` : `for "${query}"`; } -export function resolveWebFetchDetail(args: unknown): string | undefined { +function resolveWebFetchDetail(args: unknown): string | undefined { const record = asRecord(args); if (!record) { return undefined; @@ -303,9 +303,7 @@ export function resolveWebFetchDetail(args: unknown): string | undefined { return suffix ? `from ${url} (${suffix})` : `from ${url}`; } -export { resolveExecDetail }; - -export function resolveActionSpec( +function resolveActionSpec( spec: ToolDisplaySpec | undefined, action: string | undefined, ): ToolDisplayActionSpec | undefined { @@ -315,7 +313,7 @@ export function resolveActionSpec( return spec.actions?.[action] ?? undefined; } -export function resolveDetailFromKeys( +function resolveDetailFromKeys( args: unknown, keys: string[], opts: { @@ -372,7 +370,7 @@ export function resolveDetailFromKeys( .join(" ยท "); } -export function resolveToolVerbAndDetail(params: { +function resolveToolVerbAndDetail(params: { toolKey: string; args?: unknown; meta?: string;