code-smell

Reviews code changes for useless, over-designed, unreachable, and deprecated code patterns.

5|Updated Jun 22, 2026
One-click install
npx skills add https://github.com/A158Coke/WotbTools --skill code-smell-a158coke
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-smell
Source: https://github.com/A158Coke/WotbTools/tree/main/.agents/skills/code-smell
Command: npx skills add https://github.com/A158Coke/WotbTools --skill code-smell-a158coke

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? After standard code review confirms code is correct, there is no systematic check for whether the code is over-engineered, contains dead code, or carries deprecated leftovers. This Skill fills that gap with a taste-focused audit that catches unnecessary abstractions, unreachable branches, and residual clutter before tests run. ## Core Features & Use Cases - Eight-Category Checklist: Audits useless code, over-decoration, over-design, meaningless code, unreachable code, deprecated residue, architecture over-engineering, and hardcoding versus over-engineering balance. - Chained Review Workflow: Runs after a standard review-fix pass and before tests, producing a structured verifier report with verdict, evidence, over-engineering warnings, and priority ordering. - Use Case: After refactoring a Spring service, run this audit to catch a single-implementation interface, an unused import, a redundant try/catch that only rethrows, and a commented-out dead code block, then receive a file-and-line evidence list with suggested fixes. ## Quick Start Ask the AI to audit the recently changed files against the code-smell eight-item checklist and report any over-engineering or dead code with file and line evidence.

Frequently Asked Questions about code-smell

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I check code for over-engineering after a review?

Run this audit after your standard review-fix pass and before tests. It applies an eight-item checklist covering useless code, over-design, unreachable branches, and deprecated residue, then reports findings with file and line evidence.

What is the difference between code review and code smell detection?

Standard review checks whether code is correct, while this audit checks whether code is excessive. It targets single-implementation interfaces, premature abstractions, dead code, and decorative patterns that pass correctness checks but add no value.

What code smells does this checklist detect in Java and Spring?

It detects unused private methods and imports, single-implementation service interfaces, unnecessary Lombok on simple DTOs, redundant try/catch rethrows, unreachable statements after return, and @Deprecated elements with no callers.

When should I extract a constant versus keep a value inline?

Extract a constant when a string or number repeats three or more times. Keep single-use values inline unless they document meaning, and always move URLs, file paths, and API keys into configuration or environment variables.

When should dead code be deleted instead of deprecated?

Delete @Deprecated methods or classes immediately when they have no callers, and remove commented-out code blocks since Git history preserves them. Only keep deprecation markers when active consumers still need migration time.