polish

Review changed TypeScript files for DRY violations and testability issues.

Updated Apr 5, 2026
One-click install
npx skills add https://github.com/nickstrad/veil-ai --skill polish-nickstrad
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: polish
Source: https://github.com/nickstrad/veil-ai/tree/main/.claude/skills/polish-branch
Command: npx skills add https://github.com/nickstrad/veil-ai --skill polish-nickstrad

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill eliminates the risk of merging feature branches with hidden code quality issues like duplicated logic, untestable business logic, low-value tests, and inconsistent patterns that accumulate technical debt and increase long-term maintenance costs.

Core Features & Use Cases

  • DRY Violation Detection: Scans all changed files for duplicated logic, constants, and patterns, checking existing utilities in src/common/ and src/db/ before recommending new shared code.
  • Testability Improvements: Identifies business logic embedded in UI components or entry points, recommending extraction into testable service or helper functions with optional database parameters for in-memory test injection.
  • Test Quality Audits: Flags tests that only assert trivial outcomes, miss meaningful edge cases, or duplicate existing coverage, ensuring every test provides clear value.
  • Complexity & Pattern Consistency Checks: Highlights overly nested conditionals, n+1 query patterns, and deviations from established repository conventions including the repository pattern, Zod validation, and co-located test directories.
  • Use Case: Ideal for developers preparing to merge a feature branch, who want to catch quality issues early without manually reviewing every changed file.

Quick Start

Invoke the polish skill to review all changed files in your current branch and receive prioritized, actionable feedback on code quality issues before merging.

Frequently Asked Questions about polish

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

FAQPage Schema
How do I check my feature branch for DRY violations and pattern inconsistencies before merging?

To check feature branches for DRY violations and pattern inconsistencies, run an interactive code review across all changed worktree files. This process identifies duplicated logic and deviations from established repository conventions, delivering prioritized, file-specific findings with proposed fixes for explicit user confirmation.

What is the best way to find untestable business logic in my TypeScript codebase?

Finding untestable business logic requires scanning for code embedded in UI components or entry points. A targeted code review identifies these sections and recommends extracting them into testable service or helper functions with optional database parameters for in-memory test injection.

How do I audit test quality and flag low-value tests in my repository?

To audit test quality and flag low-value tests, review changed files for tests that only assert trivial outcomes, miss meaningful edge cases, or duplicate existing coverage. This ensures every test provides clear value and eliminates unnecessary complexity before merging.

Can I use this code review approach with TypeScript, Prisma, and Zod?

Yes, this code review approach applies to TypeScript codebases using Prisma and Zod. It checks changed files for n+1 query patterns, validates Zod schema usage, and enforces co-located test directories alongside established repository patterns to align with team standards.

How does branch code review handle fixes for technical debt?

Branch code review handles technical debt by delivering prioritized, file-specific findings with proposed fixes. It requires explicit user confirmation before applying any changes, ensuring all modifications align with team standards and existing utilities in src/common/ and src/db/.