🤖 refactor: auto-cleanup #2194
Merged
🤖 refactor: auto-cleanup #2194
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
force-pushed
the
auto-cleanup
branch
6 times, most recently
from
0c1cc01 to
42783c5
Compare
…age alias - AIServiceEvent union type in stream.ts was defined but never imported or used - isDefaultContinueMessage was a deprecated alias for isDefaultSourceContent, never referenced
…erConfigs Remove dead code from runtime.ts: - RUNTIME_MODES_REQUIRING_GIT: exported constant never imported anywhere - hasDevcontainerConfigs: exported type guard never called anywhere (getDevcontainerConfigs covers the same logic and is actively used)
Deduplicate the identical model fallback resolution logic that was repeated in both ProjectSidebar (MuxChatStatusIndicator) and WorkspaceListItem. Both components subscribe to the global default model preference and workspace-scoped model preference, migrating gateway models and falling back through the same chain. The new hook centralizes this in one place.
The type was marked @deprecated with zero consumers—it pointed to FrontendWorkspaceMetadata and was never imported anywhere in the codebase.
Replace the manually duplicated enum values in ThinkingLevelSchema with an import of THINKING_LEVELS from thinking.ts. The old code had a comment noting the duplication; this import keeps them in sync automatically.
The 9 short-name aliases (ISSUER, CLIENT_ID, AUTHORIZE_ENDPOINT, etc.) were never imported by any consumer. All call sites use the prefixed CODEX_* originals. Removing dead code.
Extract the triplicate cloneToolPreservingDescriptors function from tools.ts, withHooks.ts, and system1ToolWrapper.ts into a single shared module at common/utils/tools/cloneToolPreservingDescriptors.ts. Also removes now-unused assert imports from the consumer files and narrows appendToolNote visibility to file-private in system1ToolWrapper.
github-actions
bot
force-pushed
the
auto-cleanup
branch
from
42783c5 to
5be2858
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Periodic automated cleanup of the codebase. Each commit is a small, behavior-preserving refactor.
Current cleanups
refactor: remove unused AIServiceEvent type and isDefaultContinueMessage alias— Removed the deadAIServiceEventunion type from stream types and the unusedisDefaultContinueMessagealias from message types (no callers remained).refactor: remove unused RUNTIME_MODES_REQUIRING_GIT and hasDevcontainerConfigs— Removed the deadRUNTIME_MODES_REQUIRING_GITconstant andhasDevcontainerConfigsfunction from runtime types (no callers remained after runtime refactoring).refactor: extract useWorkspaceFallbackModel hook— Deduplicate identical model fallback resolution logic from ProjectSidebar and WorkspaceListItem into a shared hook.refactor: remove deprecated WorkspaceMetadataWithPaths type alias— Removed the deadWorkspaceMetadataWithPathstype alias (marked @deprecated, zero consumers, pointed toFrontendWorkspaceMetadata).refactor: use THINKING_LEVELS constant in ThinkingLevelSchema— Replace manually duplicated enum values inThinkingLevelSchema(instream.ts) with an import of theTHINKING_LEVELSconstant fromthinking.ts, eliminating drift risk.refactor: remove dead constants/agents.ts— Deletedsrc/common/constants/agents.tswhich exportedBUILT_IN_SUBAGENT_TYPES,BuiltInSubagentType, andBUILT_IN_SUBAGENTSwith zero consumers (orphaned since the user-defined agents refactor in 🤖 feat: user-defined agents (unify modes + subagents) #1289).Auto-cleanup checkpoint: 3b5e294
Generated with
mux• Model:anthropic:claude-opus-4-6• Thinking:xhigh• Cost:$0