chore(cleanup): remove all unused exports flagged by knip

yeahhe365 · yeahhe365 · commit 09c626faa392 · 2026-07-31T19:44:28.000+08:00
knip reported 25 unused exports + 4 unused types (dead code from earlier
stream-journal and third-party work); CI's knip step fails on them.

- server: drop export keyword from internally-used helpers
  (streamJobStore getJob/createJob/appendChunk/flushToResponse/
  attachJobStream, geminiProxy GEMINI_UPSTREAM_BASE_HEADER, mcpClient
  McpClientBridgeOptions, imageProxyFetch constants/type)
- delete fully-unreferenced code: ThemeIcons.tsx (4 icons) + its index
  export, designTokens TYPE_* classes, runtimeConfig
  isThirdPartyProxyRelativePath, geminiApiBaseUrl
  resolveGeminiFrontendBaseUrl, liveArtifactsMode isLiveArtifactsModeActive,
  thirdPartyApiProviders getThirdPartyProviderModels/ModelId/
  updateActiveThirdPartyProviderConfig, ModelIcon
  THIRD_PARTY_PROVIDER_ICON_COLOR
- drop export from internally-used symbols: modelConfiguration
  REMOVED_MODEL_ID_MIGRATIONS, chatStoreSync SESSION_LOADING_STALE_CHECK_MS,
  chat-input/focus FocusChatInputOptions, lastActiveSession
  LastActiveSessionSnapshot

knip now exits 0; all 2057 tests, typecheck, lint, format and both builds pass.
diff --git a/server/src/geminiProxy.ts b/server/src/geminiProxy.ts
@@ -7,7 +7,7 @@ import { maybeStreamWithJob } from './streamJobs.js';
 import { isPrivateNetworkHostname } from '../../shared/privateNetwork.js';
 
 export const GEMINI_PROXY_PREFIX = '/api/gemini';
-export const GEMINI_UPSTREAM_BASE_HEADER = 'x-gemini-upstream-base-url';
+const GEMINI_UPSTREAM_BASE_HEADER = 'x-gemini-upstream-base-url';
 
 const HOP_BY_HOP_HEADERS = new Set([
   'connection',
diff --git a/server/src/mcpClient.ts b/server/src/mcpClient.ts
@@ -30,7 +30,7 @@ const APP_VERSION = (() => {
   }
 })();
 
-export interface McpClientBridgeOptions {
+interface McpClientBridgeOptions {
   allowPrivateHttp?: boolean;
   fetchImpl?: FetchLike;
   /** Override idle eviction for tests. */
diff --git a/server/src/streamJobStore.ts b/server/src/streamJobStore.ts
@@ -65,9 +65,9 @@ sweeper.unref();
 
 // ── CRUD ────────────────────────────────────────────────────────────────────
 
-export const getJob = (id: string): StreamJob | undefined => jobs.get(id);
+const getJob = (id: string): StreamJob | undefined => jobs.get(id);
 
-export const createJob = (id: string): StreamJob => {
+const createJob = (id: string): StreamJob => {
   const job: StreamJob = {
     id,
     firstSeq: 1,
@@ -83,7 +83,7 @@ export const createJob = (id: string): StreamJob => {
   return job;
 };
 
-export const appendChunk = (job: StreamJob, data: string): void => {
+const appendChunk = (job: StreamJob, data: string): void => {
   const seq = job.firstSeq + job.chunks.length;
   job.chunks.push({ seq, data });
   job.updatedAt = Date.now();
@@ -184,7 +184,7 @@ export function pumpUpstreamBodyIntoJob(job: StreamJob, upstreamResponse: Respon
  * onward. Each call drains everything currently buffered. When the job is
  * done, the response is closed. Returns the new cursor.
  */
-export function flushToResponse(job: StreamJob, response: ServerResponse, cursor: number): number {
+function flushToResponse(job: StreamJob, response: ServerResponse, cursor: number): number {
   let nextCursor = cursor;
   for (const chunk of job.chunks) {
     if (chunk.seq > nextCursor && chunk.seq >= job.firstSeq) {
@@ -247,7 +247,7 @@ export async function maybeStreamWithSharedJob(
  * (for a new job) started the upstream fetch; this function only manages the
  * browser-side subscription. Returns true when handled.
  */
-export async function attachJobStream(
+async function attachJobStream(
   request: IncomingMessage,
   response: ServerResponse,
   config: AttachJobStreamConfig,
diff --git a/shared/imageProxyFetch.ts b/shared/imageProxyFetch.ts
@@ -2,9 +2,9 @@ import { assertImageProxyHostResolvesPublic, type ImageProxyDnsLookup } from './
 import { isPrivateNetworkHostname } from './privateNetwork.js';
 
 /** Max redirects followed while re-validating each hop (CDN signed URLs often 302 once). */
-export const IMAGE_PROXY_MAX_REDIRECTS = 3;
+const IMAGE_PROXY_MAX_REDIRECTS = 3;
 
-export const IMAGE_PROXY_REQUEST_HEADERS = {
+const IMAGE_PROXY_REQUEST_HEADERS = {
   accept: 'image/avif,image/webp,image/apng,image/svg+xml,image/*,*/*;q=0.8',
   'user-agent': 'AMC-WebUI image proxy',
 } as const;
@@ -24,7 +24,7 @@ export const isUnsafeImageProxyRedirect = (redirectUrl: URL): boolean => {
   }
 };
 
-export type ImageProxyFetchResult =
+type ImageProxyFetchResult =
   | { ok: true; response: Response }
   | { ok: false; kind: 'unsafe_redirect' }
   | { ok: false; kind: 'blocked'; message: string }
diff --git a/src/components/icons/groups/ThemeIcons.tsx b/src/components/icons/groups/ThemeIcons.tsx
diff --git a/src/components/icons/index.ts b/src/components/icons/index.ts
@@ -1,5 +1,4 @@
 export * from './groups/SettingsIcons';
-export * from './groups/ThemeIcons';
 export * from './groups/AttachmentIcons';
 export * from './groups/GeneralIcons';
 export * from './groups/LanguageIcons';
diff --git a/src/components/shared/ModelIcon.tsx b/src/components/shared/ModelIcon.tsx
@@ -29,17 +29,6 @@ const THIRD_PARTY_PROVIDER_LOGO: Record<ThirdPartyProviderId, string> = {
   custom: customLogoUrl,
 };
 
-const THIRD_PARTY_PROVIDER_ICON_COLOR: Partial<Record<ThirdPartyProviderId, string>> = {
-  openai: 'text-emerald-500 dark:text-emerald-400',
-  deepseek: 'text-blue-500 dark:text-blue-400',
-  anthropic: 'text-orange-500 dark:text-orange-400',
-  openrouter: 'text-fuchsia-500 dark:text-fuchsia-400',
-  qwen: 'text-violet-500 dark:text-violet-400',
-  kimi: 'text-cyan-500 dark:text-cyan-400',
-  glm: 'text-rose-500 dark:text-rose-400',
-  custom: 'text-slate-500 dark:text-slate-400',
-};
-
 type ModelBrandIconKey = 'gemini' | 'gemma' | 'nanobanana';
 
 const BRAND_ICON_SRC: Record<ModelBrandIconKey, string> = {
@@ -141,4 +130,4 @@ export const getModelIcon = (model: ModelOption | undefined) => {
   );
 };
 
-export { THIRD_PARTY_PROVIDER_LOGO, THIRD_PARTY_PROVIDER_ICON_COLOR };
+export { THIRD_PARTY_PROVIDER_LOGO };
diff --git a/src/constants/designTokens.ts b/src/constants/designTokens.ts
@@ -3,18 +3,6 @@
  * Prefer these over ad-hoc radius / chip classes so composer, sidebar, and chips stay aligned.
  */
 
-/** Caption / meta line — minimum practical UI size (12px at default root). */
-export const TYPE_CAPTION_CLASS = 'text-xs leading-tight';
-
-/** Dense mono meta (ids, timestamps) — same floor as caption. */
-export const TYPE_CAPTION_MONO_CLASS = 'text-xs font-mono leading-tight tabular-nums';
-
-/** Section overline labels (uppercase groups, code headers). Prefer secondary for contrast. */
-export const TYPE_OVERLINE_CLASS = 'text-xs font-bold uppercase tracking-wider text-[var(--theme-text-secondary)]';
-
-/** Muted overline when hierarchy needs more de-emphasis than TYPE_OVERLINE_CLASS. */
-export const TYPE_OVERLINE_MUTED_CLASS = 'text-xs font-bold uppercase tracking-wider text-[var(--theme-text-tertiary)]';
-
 const RADIUS_CLASS = {
   /** 6px — dense controls, inline badges */
   sm: 'rounded-md',
diff --git a/src/constants/modelConfiguration.ts b/src/constants/modelConfiguration.ts
@@ -22,7 +22,7 @@ export const MODELS_SUPPORTING_RAW_MODE = [
 ];
 
 /** Built-in model IDs removed from the app; remap saved settings to a supported replacement. */
-export const REMOVED_MODEL_ID_MIGRATIONS: Readonly<Record<string, string>> = {
+const REMOVED_MODEL_ID_MIGRATIONS: Readonly<Record<string, string>> = {
   'gemini-3.1-flash-lite': 'gemini-3.5-flash-lite',
   'models/gemini-3.1-flash-lite': 'gemini-3.5-flash-lite',
   // Exact IDs only — must not match gemini-3.5-flash-lite.
diff --git a/src/runtime/runtimeConfig.ts b/src/runtime/runtimeConfig.ts
@@ -74,22 +74,6 @@ export function getThirdPartyProxyBaseUrl(): string | null {
   return readNullableString(getRuntimeConfig()?.thirdPartyProxyUrl) ?? null;
 }
 
-/**
- * Whether the deployment routes third-party (OpenAI-compatible / Anthropic)
- * requests through our own api container where the stream-journal lives. True
- * only when the Docker web container injected a relative third-party proxy URL
- * (e.g. "/api/openai"). False in static/Pages deploys where the browser calls
- * the upstream directly and journaling is impossible — there is no container
- * to buffer the upstream independent of the browser connection.
- *
- * This gates x-amc-job-id stamping and resume for third-party streams, the same
- * way isGeminiProxyRelativePath gates the Gemini path.
- */
-export function isThirdPartyProxyRelativePath(): boolean {
-  const proxyUrl = getThirdPartyProxyBaseUrl();
-  return Boolean(proxyUrl) && !/^https?:\/\//i.test((proxyUrl ?? '').trim());
-}
-
 /**
  * Whether the deployment provides an api container that can proxy /api/gemini
  * requests. True only when the runtime config (Docker web-server.js) explicitly
diff --git a/src/services/api/geminiApiBaseUrl.ts b/src/services/api/geminiApiBaseUrl.ts
@@ -11,23 +11,6 @@ export const resolveConfiguredGeminiBaseUrl = (appSettings: GeminiApiBaseUrlSett
 
 const isAbsoluteHttpUrl = (url: string): boolean => /^https?:\/\//i.test(url.trim());
 
-/**
- * The URL the browser should send Gemini API requests TO.
- *
- * - Docker runtime: always the api container's relative path (e.g. "/api/gemini")
- *   so all requests route through our backend whether or not the user configured
- *   an absolute upstream proxy URL.
- * - Static/Pages: what the user configured, or the default Gemini API URL.
- */
-export const resolveGeminiFrontendBaseUrl = (appSettings: GeminiApiBaseUrlSettings): string => {
-  const runtimeBaseUrl = getGeminiApiProxyBaseUrl();
-  if (runtimeBaseUrl) {
-    return normalizeGeminiApiBaseUrl(runtimeBaseUrl);
-  }
-  const configuredBaseUrl = resolveConfiguredGeminiBaseUrl(appSettings);
-  return normalizeGeminiApiBaseUrl(configuredBaseUrl ?? DEFAULT_GEMINI_API_BASE_URL);
-};
-
 /**
  * Whether the effective frontend base URL is a relative path, meaning the
  * browser sends Gemini requests through our own api container where the
diff --git a/src/stores/chatStoreSync.ts b/src/stores/chatStoreSync.ts
@@ -11,7 +11,7 @@ import { getChatSyncChannel } from './chatSyncChannel';
 import type { UpdaterOrValue } from './stateUpdaters';
 
 /** How often to sweep remote loading flags that lost their lease. */
-export const SESSION_LOADING_STALE_CHECK_MS = 30_000;
+const SESSION_LOADING_STALE_CHECK_MS = 30_000;
 
 interface ChatSyncStore {
   getState: () => {
diff --git a/src/utils/chat-input/focus.ts b/src/utils/chat-input/focus.ts
@@ -1,6 +1,6 @@
 import { CHAT_INPUT_TEXTAREA_SELECTOR } from '@/constants/layout';
 
-export type FocusChatInputOptions = {
+type FocusChatInputOptions = {
   /** Place the caret at the end of the current input value after focusing. */
   caret?: 'end';
 };
diff --git a/src/utils/chat/lastActiveSession.ts b/src/utils/chat/lastActiveSession.ts
@@ -5,7 +5,7 @@ import type { ChatSettings } from '@/types';
  * 最后活跃会话快照。存于 localStorage（跨标签共享），
  * 让新开的标签页能以"源标签页正在查看的会话"为模板创建新聊天。
  */
-export interface LastActiveSessionSnapshot {
+interface LastActiveSessionSnapshot {
   sessionId: string;
   settings: ChatSettings;
   ts: number;
diff --git a/src/utils/live-artifacts/liveArtifactsMode.ts b/src/utils/live-artifacts/liveArtifactsMode.ts
@@ -1,43 +1,5 @@
 import type { LiveArtifactsPromptMode } from '@/types';
 
-/**
- * Checks whether Live Artifacts mode is currently active for a given set of
- * settings. The key indicator is a system instruction that contains the Live
- * Artifacts protocol marker string.
- */
-export function isLiveArtifactsModeActive(args: {
-  systemInstruction?: string | null;
-  promptMode?: LiveArtifactsPromptMode;
-  liveArtifactsSystemPrompt?: string | null;
-  liveArtifactsSystemPrompts?: Partial<Record<LiveArtifactsPromptMode, string>> | null;
-}): boolean {
-  const { systemInstruction, promptMode, liveArtifactsSystemPrompt, liveArtifactsSystemPrompts } = args;
-
-  // Check for the protocol markers in the system instruction
-  const instruction = systemInstruction?.trim() ?? '';
-  if (
-    instruction.includes('[Live Artifacts Inline Protocol') ||
-    instruction.includes('[Live Artifacts System Prompt')
-  ) {
-    return true;
-  }
-
-  // Check prompt mode override
-  if (promptMode && liveArtifactsSystemPrompts?.[promptMode]?.trim()) {
-    const overridePrompt = liveArtifactsSystemPrompts[promptMode]!.trim();
-    if (instruction === overridePrompt) {
-      return true;
-    }
-  }
-
-  // Fallback: check the single-system-prompt override
-  if (liveArtifactsSystemPrompt?.trim() && instruction === liveArtifactsSystemPrompt.trim()) {
-    return true;
-  }
-
-  return false;
-}
-
 /**
  * Determine if Live Artifacts mode is active from the app settings and
  * current chat settings. This is a convenience wrapper for CommonComponentData.
diff --git a/src/utils/thirdPartyApiProviders.ts b/src/utils/thirdPartyApiProviders.ts
@@ -141,14 +141,6 @@ export const getThirdPartyProviderConfig = (settings: Pick<AppSettings, 'thirdPa
   return thirdPartyApi.providers[thirdPartyApi.activeProvider] ?? thirdPartyApi.providers.openai;
 };
 
-export const getThirdPartyProviderModels = (settings: Pick<AppSettings, 'thirdPartyApi'>): ModelOption[] =>
-  getThirdPartyProviderConfig(settings).models;
-
-export const getThirdPartyProviderModelId = (
-  settings: Pick<AppSettings, 'thirdPartyApi'>,
-  sessionModelId?: string,
-): string => sessionModelId ?? getThirdPartyProviderConfig(settings).modelId;
-
 /**
  * Returns all enabled third-party providers as { id, config } pairs.
  * A provider is considered enabled only when `config.enabled === true`.
@@ -270,8 +262,3 @@ export const updateThirdPartyProviderConfig = (
     }),
   },
 });
-
-export const updateActiveThirdPartyProviderConfig = (
-  thirdPartyApi: ThirdPartyApiSettings,
-  updates: Partial<ThirdPartyProviderConfig>,
-): ThirdPartyApiSettings => updateThirdPartyProviderConfig(thirdPartyApi, thirdPartyApi.activeProvider, updates);

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ const APP_VERSION = (() => {`
`30`	`30`	`}`
`31`	`31`	`})();`
`32`	`32`
`33`		`-export interface McpClientBridgeOptions {`
	`33`	`+interface McpClientBridgeOptions {`
`34`	`34`	`allowPrivateHttp?: boolean;`
`35`	`35`	`fetchImpl?: FetchLike;`
`36`	`36`	`/** Override idle eviction for tests. */`