This Claude skill automates the code refactoring process to significantly enhance maintainability, readability, and performance without altering external behavior. It systematically addresses technical debt by identifying improvement opportunities—such as naming clarity, function decomposition, and DRY (Don't Repeat Yourself) principles—while ensuring functional integrity through rigorous automated testing and static analysis.

When should I use refactor?

refactor is useful in the following scenarios: • Modularizing Complex Logic: Automatically split oversized, multi-responsibility functions into smaller, testable components to adhere to the Single Responsibility Principle. • Eliminating Technical Debt: Identify and remove duplicate code blocks by creating reusable helper functions or generic implementations, making the codebase easier to maintain. • Improving Code Readability: Enhance developer productivity by renaming vague variables and functions to be more descriptive and consistent with project standards. • Safe Performance Optimization: Refactor inefficient logic or complex conditional expressions while verifying results against existing test suites to guarantee zero regressions.

refactor – AI Agent Skills

name	refactor
description	>-
allowed-tools	Read, Write, Edit, Grep, Glob, Bash

手順

対象の特定: リファクタリング対象のファイル/関数を特定
既存テストの確認:
- 対応する *_test.go を確認
- go test ./... を実行してベースライン確立（全テストがパスすることを確認）
リファクタリング計画: 以下の観点で改善機会を特定
- 命名改善（変数名、関数名の明確化）
- 関数分割（長すぎる関数、複数の責任を持つ関数）
- 重複コード削除（DRY原則）
- 複雑な条件式の簡素化
- マジックナンバーの定数化
- エラーハンドリングの改善
リファクタリング実施: 小さなステップで段階的に変更
テスト実行: 各ステップ後に go test ./... で機能不変を確認
コード品質確認:
- go fmt ./... でフォーマット
- go vet ./... で静的解析
コミット: 変更が成功したら、適切なメッセージでコミット（refactor: ...形式）

リファクタリングパターン

1. 命名改善

// Before
func f(n []int) int { ... }

// After
func Sum(numbers []int) int { ... }

2. 関数分割

// Before: 1つの関数が複数の責任
func ProcessData(data []byte) error {
    // バリデーション
    // パース
    // 変換
    // 保存
}

// After: 単一責任に分割
func ValidateData(data []byte) error { ... }
func ParseData(data []byte) (*Data, error) { ... }
func TransformData(data *Data) *Result { ... }
func SaveResult(result *Result) error { ... }

3. 重複削除

// Before: 重複コード
func AddUser(u User) { db.Insert("users", u) }
func AddPost(p Post) { db.Insert("posts", p) }

// After: 汎用化
func Add[T any](table string, entity T) { db.Insert(table, entity) }

4. マジックナンバーの定数化

// Before
if len(items) > 100 { ... }

// After
const MaxItems = 100
if len(items) > MaxItems { ... }

重要な原則

✅ 機能不変: 外部から見た動作は一切変更しない
✅ テストファースト: リファクタ前に必ずテストを実行
✅ 小さなステップ: 一度に多くを変更しない
✅ 継続的な検証: 各ステップ後にテスト実行
❌ 新機能追加: リファクタリングと機能追加は分離

リファクタリング対象の優先順位

High: 複雑度が高く、バグを生みやすい箇所
Medium: 頻繁に変更される箇所
Low: 安定していて変更頻度が低い箇所

出力例

# Refactoring Report

## Target
- File: `pkg/calc/sum.go`

## Changes Applied
1. ✅ Improved function naming clarity
2. ✅ Added nil check for defensive programming
3. ✅ Simplified loop condition

## Test Results
✅ All tests pass (before and after)
- Tests run: 5
- Coverage: 85% (unchanged)

## Performance Impact
No performance regression detected

ベストプラクティス

リファクタリング前後でベンチマークを取る（パフォーマンス改善の場合）
コミットメッセージに「何を」「なぜ」を明記
レビューしやすいように、変更を論理的に分割

refactor

When & Why to Use This Skill

Use Cases

手順