🤖 refactor: auto-cleanup

[github-actions]

Summary

Periodic auto-cleanup: remove dead code, fix stale config, deduplicate helpers, and tighten module exports.

Latest Change

• Remove dead buildPendingFromContent and buildEditingStateFromContent: These helpers in chatEditing.ts were added in PR 🤖 feat: add /<model> one-shot model override syntax #2142 as symmetrical counterparts to the fromDisplayed variants, but the fromContent variant was never imported or called anywhere. Net -14 lines.

Previous changes

• Deduplicate coerceAgentStatus into shared extensionMetadata utility: ExtensionMetadataService had private coerceAgentStatus and coerceStatusUrl methods near-identical to the module-level functions in extensionMetadata.ts (added in PR 🤖 fix: propagate status_set via workspace activity snapshots #2532). Export both from the shared utility and import in the service. Net -21 lines, identical behavior.

• Extract hasRuntimeOverrides helper to deduplicate 3 identical checks: The runtime enablement feature (PR 🤖 feat: add runtime enablement settings #2387) introduced the same 3-line boolean expression in ChatInput/index.tsx, RuntimesSection.tsx, and useDraftWorkspaceSettings.ts. Extracted into a shared hasRuntimeOverrides() function in src/common/types/project.ts. Net -6 lines, identical behavior.

• Remove dead getSessionTimingKey storage constant: The function was introduced in PR 🤖 feat: add timing stats indicator in workspace sidebar #1177 (timing stats feature) but its functionality migrated to the file-based SessionTimingService. No code in the codebase imports or calls it. Net -9 lines.

• Remove dead extractCommonPrefix from numstatParser.ts: The function was exported but never imported or referenced anywhere in production code or tests. Its JSDoc noted it was "used for display purposes only" but no display code used it. Net -34 lines.

• Use coerceNonEmptyString for model resolution in dispatchAgentSwitch: The new switch_agent model-resolution cascade (from PR 🤖 fix: honor target agent settings in switch_agent follow-ups #2524) manually repeated the trim() + empty-check → undefined pattern three times. Replaced with the existing coerceNonEmptyString utility from taskUtils.ts. Net -8 lines, identical behavior.

• Remove dead readExtensionMetadata function: This sync file-reading function was superseded by ExtensionMetadataService's async implementation. Zero imports outside the definition file. Also removes now-unused fs sync imports, isThinkingLevel, and log imports. Net -43 lines.

• Fix missing blank line between functions in workspaceTitleGenerator.ts: The new cleanExtractedValue function added in c240bae was missing the blank line separator that every other function boundary in the file uses. Pure formatting fix.

• Remove dead createManualRetryState function: This function was only used by the frontend RetryBarrier component's manual retry flow. PR 🤖 refactor: move auto-compaction & auto-retry from frontend to backend #2469 moved retry management to the backend RetryManager, which uses createFreshRetryState and createFailedRetryState instead. The function definition and its 7 tests were left behind as dead code. Net -96 lines.

• Use shared UpdateChannel type in AboutDialog: Replace inline "stable" | "nightly" type annotations with the shared UpdateChannel type from src/common/types/project.ts, eliminating type duplication.

• Remove dead timing stats types from WorkspaceStore: Remove StreamTimingStats, ModelTimingStats, and SessionTimingStats interfaces that were defined but never referenced. The authoritative versions live in src/common/orpc/schemas/workspaceStats.ts. Net -74 lines.

• Remove dead code and narrow exports in agentTools: Remove collectToolConfigsFromDefinitionGraph (exported but never imported anywhere) and its supporting AgentToolsLike interface. Also make isToolEnabledByConfigs and isToolEnabledInResolvedChain module-private since they are only called within the same file. Net -51 lines.

• Deduplicate onboarding wizard splash ID constant.

• Deduplicate MessageWindow blocks in UserMessage.

• Inline single-use resolveForkRuntimeConfigs into its only caller.

• Remove dead fuzzy-search helpers from fuzzySearch.ts.

• Deduplicate ConnectionHealth interface from SSH2ConnectionPool.ts.

• Delete src/common/utils/tools/taskToolTypeGuards.ts: Dead module with zero imports.

• Delete src/common/utils/ai/providerFactory.ts: Dead since folder restructure.

• Fix stale ESLint dynamic-import override paths.

• Remove dead isStreamingPart type guard from messageUtils.ts.

• Deduplicate formatDuration into shared utility.

• Remove dead formatNewCommand, ThemeToggleButton, canonicalModelId, hasSubtitle prop.

• Deduplicate compaction type guards into shared module.

• Remove dead re-exports from RightSidebar.tsx.

• Rename partialService.test.ts → historyService.partial.test.ts.

• Fix orphaned JSDoc comment on UpdaterService.getStatus().

• Deduplicate normalizeAgentId, getProviderDisplayName, getCustomModels/getAllCustomModels.

• Remove dead backward-compat aliases in codexOauthAuth.ts and re-exports.

• Unexport file-private helpers in modelEntries.ts.

• Remove dead pendingReplayReset field from WorkspaceStore.ts.

• Remove dead onLogEntry, formatPendingCommand, parseGitShowBranchForStatus, path helpers.

• Remove dead ChatInputProps barrel re-export.

• Unexport file-private isBashOutputTool.

• Remove dead code in chatEditing.ts.

Validation

• make typecheck passes
• make lint passes
• make fmt-check passes (excluding shfmt not available in CI env)
• Related tests pass (16/16 in useDraftWorkspaceSettings.test.tsx and useCreationWorkspace.test.tsx)

Auto-cleanup checkpoint: 60c63c8

---

Generated with mux · Model: anthropic:claude-opus-4-6 · Thinking: xhigh