refactor-legacy-code
Modernize and improve legacy codebases while maintaining functionality. Use when you need to refactor old code, reduce technical debt, modernize deprecated patterns, or improve code maintainability without breaking existing behavior.
What this skill does
# Refactor Legacy Code ## Table of Contents - [Overview](#overview) - [When to Use](#when-to-use) - [Quick Start](#quick-start) - [Reference Guides](#reference-guides) - [Best Practices](#best-practices) ## Overview This skill helps you systematically refactor legacy code to improve maintainability, readability, and performance while preserving existing functionality. It follows industry best practices for safe refactoring with comprehensive testing. ## When to Use - Modernizing outdated code patterns or deprecated APIs - Reducing technical debt in existing codebases - Improving code readability and maintainability - Extracting reusable components from monolithic code - Upgrading to newer language features or frameworks - Preparing code for new feature development ## Quick Start First, analyze the legacy code to understand: ```bash # Review the codebase structure tree -L 3 -I 'node_modules|dist|build' # Check for outdated dependencies npm outdated # or pip list --outdated, composer outdated, etc. # Identify code complexity hotspots # Use tools like: # - SonarQube for code smells # - eslint for JavaScript # - pylint for Python # - RuboCop for Ruby ``` ## Reference Guides Detailed implementations in the `references/` directory: | Guide | Contents | |---|---| | [Code Assessment](references/code-assessment.md) | Code Assessment | | [Establish Safety Net](references/establish-safety-net.md) | Establish Safety Net | | [Incremental Refactoring](references/incremental-refactoring.md) | Incremental Refactoring | | [Modernize Patterns](references/modernize-patterns.md) | Modernize Patterns | | [Reduce Dependencies](references/reduce-dependencies.md) | Reduce Dependencies, Documentation | | [Complete Refactoring Example](references/complete-refactoring-example.md) | Complete Refactoring Example | | [Benefits Achieved](references/benefits-achieved.md) | Benefits Achieved | ## Best Practices ### ✅ DO - **Refactor incrementally**: Small, testable changes - **Run tests frequently**: After each refactoring step - **Commit often**: Create logical, atomic commits - **Keep existing tests passing**: Don't break functionality - **Use IDE refactoring tools**: Safer than manual edits - **Review code coverage**: Ensure tests cover refactored code - **Document decisions**: Why, not just what - **Seek peer review**: Fresh eyes catch issues ### ❌ DON'T - **Mix refactoring with new features**: Separate concerns - **Refactor without tests**: Recipe for breaking changes - **Change behavior**: Refactoring should preserve functionality - **Refactor large chunks**: Increases risk and review difficulty - **Ignore code smells**: Address them systematically - **Skip documentation**: Future maintainers need context
Related in Code Review
gstack
IncludedFast headless browser for QA testing and site dogfooding. Navigate pages, interact with elements, verify state, diff before/after, take annotated screenshots, test responsive layouts, forms, uploads, dialogs, and capture bug evidence. Use when asked to open or test a site, verify a deployment, dogfood a user flow, or file a bug with screenshots. (gstack)
startup-due-diligence
IncludedLegal due diligence review for seed-stage and Series A startups (US, Delaware C-Corp focus). Supports both investor and founder perspectives. Capabilities include: (1) Interactive document review and issue spotting; (2) Document request list generation; (3) Cap table and SAFE/convertible note analysis; (4) Red flag identification with severity ratings; (5) Diligence report generation. TRIGGERS: due diligence, DD, startup investment, cap table review, Series A, seed round, investor diligence, legal review startup, SAFE analysis, convertible note, 409A, founder vesting.
interview-master
IncludedThis skill should be used when the user asks to "generate interview questions", "prepare for interview", "optimize resume", "conduct mock interview", "analyze git commits for resume", "generate resume from code", "review my resume", or mentions interview preparation, career assistance, or extracting project experience from git history. Provides comprehensive interview and career development guidance for both job seekers and interviewers.
fix-issue
IncludedFixes GitHub issues using parallel analysis agents for root cause investigation, code exploration, and regression detection. Reads issue context from gh CLI, searches codebase and memory for related patterns, generates a fix with tests, and links the resolution back to the issue via PR. Includes prevention analysis to avoid recurrence. Use when debugging errors, resolving regressions, fixing bugs, or triaging issues.
sf-apex
IncludedGenerates and reviews Salesforce Apex code with 150-point scoring. TRIGGER when: user writes, reviews, or fixes Apex classes, triggers, test classes, batch/queueable/schedulable jobs, or touches .cls/.trigger files. DO NOT TRIGGER when: LWC JavaScript (use sf-lwc), Flow XML (use sf-flow), SOQL-only queries (use sf-soql), or non-Salesforce code.
swift-development
IncludedComprehensive Swift development for building, testing, and deploying iOS/macOS applications. Use when Claude needs to: (1) Build Swift packages or Xcode projects from command line, (2) Run tests with XCTest or Swift Testing framework, (3) Manage iOS simulators with simctl, (4) Handle code signing, provisioning profiles, and app distribution, (5) Format or lint Swift code with SwiftFormat/SwiftLint, (6) Work with Swift Package Manager (SPM), (7) Implement Swift 6 concurrency patterns (async/await, actors, Sendable), (8) Create SwiftUI views with MVVM architecture, (9) Set up Core Data or SwiftData persistence, or any other Swift/iOS/macOS development tasks.