refactor(agents): trim unused model helper exports

vincentkoc · vincentkoc · commit 2ee4b523b41e · 2026-06-17T08:23:32.000+08:00
diff --git a/src/agents/mcp-stdio-transport.ts b/src/agents/mcp-stdio-transport.ts
@@ -11,7 +11,7 @@ import type { JSONRPCMessage } from "@modelcontextprotocol/sdk/types.js";
 import { killProcessTree, signalProcessTree } from "../process/kill-tree.js";
 import { prepareOomScoreAdjustedSpawn } from "../process/linux-oom-score.js";
 
-export type OpenClawStdioServerParameters = {
+type OpenClawStdioServerParameters = {
   command: string;
   args?: string[];
   env?: Record<string, string>;
diff --git a/src/agents/model-auth-runtime-shared.ts b/src/agents/model-auth-runtime-shared.ts
@@ -19,7 +19,7 @@ export type ResolvedProviderAuth = {
 };
 
 /** Stable provider auth error code used by fallback/retry paths. */
-export type ProviderAuthErrorCode = "missing-api-key" | "missing-provider-auth";
+type ProviderAuthErrorCode = "missing-api-key" | "missing-provider-auth";
 
 /** Base provider auth error with a stable code for retry/fallback logic. */
 export class ProviderAuthError extends Error {
diff --git a/src/agents/model-catalog-visibility.ts b/src/agents/model-catalog-visibility.ts
@@ -14,10 +14,7 @@ import {
 } from "./model-visibility-policy.js";
 
 type ModelCatalogVisibilityView = "default" | "configured" | "all";
-export type ProviderAuthChecker = (
-  provider: string,
-  modelApi?: string,
-) => boolean | Promise<boolean>;
+type ProviderAuthChecker = (provider: string, modelApi?: string) => boolean | Promise<boolean>;
 const OPENAI_PROVIDER_ID = "openai";
 const OPENAI_CODEX_RESPONSES_API = "openai-chatgpt-responses";
 const OPENAI_CODEX_ROUTABLE_MODEL_IDS = new Set([
@@ -56,7 +53,7 @@ async function resolveProviderAuthCheck(
   return isPromiseLike(result) ? await result : result;
 }
 
-export async function modelCatalogEntryHasProviderAuth(
+async function modelCatalogEntryHasProviderAuth(
   providerAuthChecker: ProviderAuthChecker,
   entry: ModelCatalogEntry,
 ): Promise<boolean> {
diff --git a/src/agents/model-max-tokens-params.ts b/src/agents/model-max-tokens-params.ts
@@ -6,7 +6,7 @@
 const MAX_TOKENS_PARAM_KEYS = ["maxTokens", "max_completion_tokens", "max_tokens"] as const;
 
 /** Return a finite non-negative max-token value, or undefined for invalid input. */
-export function resolveNonNegativeMaxTokensParam(value: unknown): number | undefined {
+function resolveNonNegativeMaxTokensParam(value: unknown): number | undefined {
   return typeof value === "number" && Number.isFinite(value) && value >= 0 ? value : undefined;
 }
 
diff --git a/src/agents/model-picker-visibility.ts b/src/agents/model-picker-visibility.ts
@@ -31,21 +31,3 @@ export function createModelPickerVisibleProviderPredicate(
     return !isRetiredModelPickerProvider(normalized) && !cliRuntimeProviders.has(normalized);
   };
 }
-
-/** Returns whether a provider id should appear in the model picker. */
-export function isModelPickerVisibleProvider(provider: string): boolean {
-  const normalized = normalizeProviderId(provider);
-  return (
-    !isRetiredModelPickerProvider(normalized) &&
-    !isCliRuntimeProvider(normalized, { includeSetupRegistry: true })
-  );
-}
-
-/** Returns whether a provider/model ref should appear in the model picker. */
-export function isModelPickerVisibleModelRef(ref: string): boolean {
-  const separatorIndex = ref.indexOf("/");
-  if (separatorIndex <= 0) {
-    return true;
-  }
-  return isModelPickerVisibleProvider(ref.slice(0, separatorIndex));
-}

Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@`
`6`	`6`	`const MAX_TOKENS_PARAM_KEYS = ["maxTokens", "max_completion_tokens", "max_tokens"] as const;`
`7`	`7`
`8`	`8`	`/** Return a finite non-negative max-token value, or undefined for invalid input. */`
`9`		`-export function resolveNonNegativeMaxTokensParam(value: unknown): number \| undefined {`
	`9`	`+function resolveNonNegativeMaxTokensParam(value: unknown): number \| undefined {`
`10`	`10`	`return typeof value === "number" && Number.isFinite(value) && value >= 0 ? value : undefined;`
`11`	`11`	`}`
`12`	`12`