Rust Refactor Helper

by actionbook5c40d3ad7851No license1.5K starsListed Oct 8, 2026Updated Oct 8, 2026Repository updated 6 weeks ago

Safe Rust refactoring with LSP analysis. Triggers on: /refactor, rename symbol, move function, extract, 重构, 重命名, 提取函数, 安全重构

Instructions onlySoftware Development
AI-generated overview

Safely refactors Rust code using LSP analysis, with impact previews before applying changes.

What it does
This skill guides an agent through safe Rust refactoring operations: renaming symbols, extracting functions, inlining functions, and moving symbols between modules. It uses LSP operations such as findReferences, hover, goToDefinition, and incomingCalls to build an impact analysis before edits. It produces reports listing affected files, import changes, dependency moves, and potential issues, and can preview changes in dry-run mode before applying them with the Edit tool.
When to use it
Use it when renaming a Rust symbol, extracting a code block into a function, inlining a function, or moving a symbol to another module. It is suited to changes where reference completeness, name conflicts, visibility, and import updates matter.
Requirements
Needs an LSP tool plus Read, Glob, Grep, and Edit tools; no scripts are shipped.

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 renamingLSP(  operation: "findReferences",  filePath: "src/lib.rs",  line: 25,  character: 8)
# Get symbol infoLSP(  operation: "hover",  filePath: "src/lib.rs",  line: 25,  character: 8)
# Check call hierarchy for move operationsLSP(  operation: "incomingCalls",  filePath: "src/lib.rs",  line: 25,  character: 8)

Refactoring Workflows

1. 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?- [x] --dry-run (preview only)- [ ] Apply changes

2. 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​```rustlet 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
​```rustfn 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
​```rustlet config = load_and_validate_config(&path)?;​```

3. 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 Locationsrc/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

CheckPurpose
Reference completenessEnsure all uses are found
Name conflictsDetect existing symbols with same name
Visibility changesWarn if pub/private scope changes
Macro-generated codeWarn about code in macros
DocumentationFlag doc comments mentioning symbol
Test coverageShow 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

WhenSee
Navigate to symbolrust-code-navigator
Understand call flowrust-call-graph
Project structurerust-symbol-analyzer
Trait implementationsrust-trait-explorer

Source and attribution

Source:actionbook/rust-skillsinskills/rust-refactor-helperat commit5c40d3a

License: No license

Content belongs to its original authors. SourceWeft indexes it from a public repository.

Report or request removal