Rust Refactor Helper
Perform safe refactoring with comprehensive impact analysis.
Usage
/rust-refactor-helper <action> <target> [--dry-run]
Actions:
-
rename <old> <new>
-
Rename symbol
-
extract-fn <selection>
-
Extract to function
-
inline <fn>
-
Inline function
-
move <symbol> <dest>
-
Move to module
Examples:
-
/rust-refactor-helper rename parse_config load_config
-
/rust-refactor-helper extract-fn src/main.rs:20-35
-
/rust-refactor-helper move UserService src/services/
LSP Operations Used
Pre-Refactor Analysis
Find all references before renaming
LSP( operation: "findReferences", filePath: "src/lib.rs", line: 25, character: 8 )
Get symbol info
LSP( operation: "hover", filePath: "src/lib.rs", line: 25, character: 8 )
Check call hierarchy for move operations
LSP( operation: "incomingCalls", filePath: "src/lib.rs", line: 25, character: 8 )
Refactoring Workflows
- Rename Symbol
User: "Rename parse_config to load_config" │ ▼ [1] Find symbol definition LSP(goToDefinition) │ ▼ [2] Find ALL references LSP(findReferences) │ ▼ [3] Categorize by file │ ▼ [4] Check for conflicts - Is 'load_config' already used? - Are there macro-generated uses? │ ▼ [5] Show impact analysis (--dry-run) │ ▼ [6] Apply changes with Edit tool
Output:
Rename: parse_config → load_config
Impact Analysis
Definition: src/config.rs:25 References found: 8
| File | Line | Context | Change |
|---|---|---|---|
| src/config.rs | 25 | pub fn parse_config( | Definition |
| src/config.rs | 45 | parse_config(path)? | Call |
| src/main.rs | 12 | config::parse_config | Import |
| src/main.rs | 30 | let cfg = parse_config( | Call |
| src/lib.rs | 8 | pub use config::parse_config | Re-export |
| tests/config_test.rs | 15 | parse_config("test.toml") | Test |
| tests/config_test.rs | 25 | parse_config("") | Test |
| docs/api.md | 42 | parse_config | Documentation |
Potential Issues
⚠️ Documentation reference: docs/api.md:42 may need manual update ⚠️ Re-export: src/lib.rs:8 - public API change
Proceed?
- --dry-run (preview only)
- Apply changes
- Extract Function
User: "Extract lines 20-35 in main.rs to a function" │ ▼ [1] Read the selected code block │ ▼ [2] Analyze variables - Which are inputs? (used but not defined in block) - Which are outputs? (defined and used after block) - Which are local? (defined and used only in block) │ ▼ [3] Determine function signature │ ▼ [4] Check for early returns, loops, etc. │ ▼ [5] Generate extracted function │ ▼ [6] Replace original code with call
Output:
Extract Function: src/main.rs:20-35
Selected Code
rust let file = File::open(&path)?; let mut contents = String::new(); file.read_to_string(&mut contents)?; let config: Config = toml::from_str(&contents)?; validate_config(&config)?;
Analysis
Inputs: path: &Path Outputs: config: Config Side Effects: File I/O, may return error
Extracted Function
rust fn load_and_validate_config(path: &Path) -> Result<Config> { let file = File::open(path)?; let mut contents = String::new(); file.read_to_string(&mut contents)?; let config: Config = toml::from_str(&contents)?; validate_config(&config)?; Ok(config) }
Replacement
rust let config = load_and_validate_config(&path)?;
- Move Symbol
User: "Move UserService to src/services/" │ ▼ [1] Find symbol and all its dependencies │ ▼ [2] Find all references (callers) LSP(findReferences) │ ▼ [3] Analyze import changes needed │ ▼ [4] Check for circular dependencies │ ▼ [5] Generate move plan
Output:
Move: UserService → src/services/user.rs
Current Location
src/handlers/auth.rs:50-120
Dependencies (will be moved together)
- struct UserService (50-80)
- impl UserService (82-120)
- const DEFAULT_TIMEOUT (48)
Import Changes Required
| File | Current | New |
|---|---|---|
| src/main.rs | use handlers::auth::UserService | use services::user::UserService |
| src/handlers/api.rs | use super::auth::UserService | use crate::services::user::UserService |
| tests/auth_test.rs | use crate::handlers::auth::UserService | use crate::services::user::UserService |
New File Structure
src/ ├── services/ │ ├── mod.rs (NEW - add `pub mod user;`) │ └── user.rs (NEW - UserService moved here) ├── handlers/ │ └── auth.rs (UserService removed)
Circular Dependency Check
✅ No circular dependencies detected
Safety Checks
Check Purpose
Reference completeness Ensure all uses are found
Name conflicts Detect existing symbols with same name
Visibility changes Warn if pub/private scope changes
Macro-generated code Warn about code in macros
Documentation Flag doc comments mentioning symbol
Test coverage Show affected tests
Dry Run Mode
Always use --dry-run first to preview changes:
/rust-refactor-helper rename old_name new_name --dry-run
This shows all changes without applying them.
Related Skills
When See
Navigate to symbol rust-code-navigator
Understand call flow rust-call-graph
Project structure rust-symbol-analyzer
Trait implementations rust-trait-explorer