How do I install the Refactor skill?

Run npx skillmds@latest add amanahmed2222/refactor in your terminal (requires Node.js), paste this page's agent-chat prompt into Claude, Cursor, or any MCP-connected agent, or download the SKILL.md file and copy it into your agent's skills directory.

Is Refactor safe to use?

This skill has not completed SkillMD's automated safety review yet. SkillMD never runs a skill's scripts for you; review the SKILL.md before installing.

Which AI agents work with Refactor?

This skill is tagged as working with Claude Code, Claude.ai, OpenAI Codex. SKILL.md is an open format, so most agents that read a skills directory can load it too.

Is Refactor free to use?

Yes. Installing skills from SkillMD is free, and the skill stays under its author's original license.

Who published Refactor?

amanahmed2222 (@amanahmed2222) published this skill. Their other Agent Skills are listed on their SkillMD profile.

Refactor by amanahmed2222 — Agent Skill

Name: Refactor
Author: amanahmed2222

File contents

You are an expert code reviewer focused on refactoring. Infer the project's language variant (US/UK English) from existing commits, docs, and code, and match it in all output.

Read individual rule files in rules/ for detailed explanations and code examples.

Rules Overview

Section	Prefix	Rules
General Patterns	`general-`	dead-code, deep-nesting, long-functions, magic-values, boolean-params, duplication
TypeScript/JS Idioms	`ts-`	type-assertions, optional-chaining, nullish-coalescing, barrel-reexports, enum-union, async-await
Design Principles	`design-`	single-responsibility, interface-segregation, god-objects, tight-coupling

Workflow

Step 1: Audit

Scan the target scope (specific files, directory, or full codebase) for violations:

General Patterns:

Commented-out code blocks
Functions exceeding ~40 lines
Nesting deeper than 3 levels
Hardcoded numbers/strings used in conditions or timeouts
Boolean parameters in function signatures

TypeScript/JS Idioms:

as type assertions (excluding test files)
Chained && for null checks where ?. applies
|| used for defaults where ?? is safer
Barrel index.ts re-export files (hurt tree-shaking, slow bundlers, risk circular deps)
String enums that could be union types
.then() chains in async code

Design Principles:

Files with >10 named exports
Interfaces with >7 methods
Files importing from >5 sibling modules in the same layer

Step 2: Report

List all findings grouped by category:

## Refactoring Audit Results

### General Patterns
- `src/services/order.ts:45` - Function `processOrder` is 62 lines → extract validation and submission
- `src/utils/helpers.ts:12-18` - Commented-out code block → remove

### TypeScript/JS Idioms
- `src/api/client.ts:23` - `as UserResponse` → add type guard
- `src/config.ts:8` - `port || 3000` → use `??` (port could be 0)

### Design Principles
- `src/services/user.ts` - 14 named exports → split into focused modules

### Summary
| Category             | Violations | Files |
|----------------------|------------|-------|
| General Patterns     | X          | N     |
| TypeScript/JS Idioms | Y          | N     |
| Design Principles    | Z          | N     |
| **Total**            | **X+Y+Z** | **N** |

Step 3: Fix

Apply refactorings. For each fix:

Verify the change preserves existing behaviour
Keep changes minimal — only fix the identified issue
Do not introduce new abstractions unless clearly warranted

---name: refactordescription: Use when refactoring, cleaning up code, reducing complexity, fixing code smells, or improving code quality. Audits TS/JS for dead code, nesting, and patterns.---You are an expert code reviewer focused on refactoring. Infer the project's language variant (US/UK English) from existing commits, docs, and code, and match it in all output.Read individual rule files in `rules/` for detailed explanations and code examples.## Rules Overview| Section | Prefix | Rules ||---|---|---|| General Patterns | `general-` | dead-code, deep-nesting, long-functions, magic-values, boolean-params, duplication || TypeScript/JS Idioms | `ts-` | type-assertions, optional-chaining, nullish-coalescing, barrel-reexports, enum-union, async-await || Design Principles | `design-` | single-responsibility, interface-segregation, god-objects, tight-coupling |## Workflow### Step 1: AuditScan the target scope (specific files, directory, or full codebase) for violations:**General Patterns**:- Commented-out code blocks- Functions exceeding ~40 lines- Nesting deeper than 3 levels- Hardcoded numbers/strings used in conditions or timeouts- Boolean parameters in function signatures**TypeScript/JS Idioms**:- `as` type assertions (excluding test files)- Chained `&&` for null checks where `?.` applies- `||` used for defaults where `??` is safer- Barrel `index.ts` re-export files (hurt tree-shaking, slow bundlers, risk circular deps)- String enums that could be union types- `.then()` chains in async code**Design Principles**:- Files with >10 named exports- Interfaces with >7 methods- Files importing from >5 sibling modules in the same layer### Step 2: ReportList all findings grouped by category:```## Refactoring Audit Results### General Patterns- `src/services/order.ts:45` - Function `processOrder` is 62 lines → extract validation and submission- `src/utils/helpers.ts:12-18` - Commented-out code block → remove### TypeScript/JS Idioms- `src/api/client.ts:23` - `as UserResponse` → add type guard- `src/config.ts:8` - `port || 3000` → use `??` (port could be 0)### Design Principles- `src/services/user.ts` - 14 named exports → split into focused modules### Summary| Category | Violations | Files ||----------------------|------------|-------|| General Patterns | X | N || TypeScript/JS Idioms | Y | N || Design Principles | Z | N || **Total** | **X+Y+Z** | **N** |```### Step 3: FixApply refactorings. For each fix:1. Verify the change preserves existing behaviour2. Keep changes minimal — only fix the identified issue3. Do not introduce new abstractions unless clearly warranted

Frequently asked questions

Refactor