All skills
simota avatar
by shingo imotasimota/agent-skills85 stars
15

Refactoring code: variable naming, function extraction, magic number constants, dead code removal. Does not change behavior. Not for bugs/security (Judge), tests (Radar), or features (Builder).

Use this Skill: https://skilld.dev/gh/simota/agent-skills/zen

This session only. Nothing lands on disk.

referenceai-assisted-refactoring.md

≈2.9k tokens on demand. Your agent reads this file only when SKILL.md points to it.

AI-Assisted Refactoring (2025-2026)

Purpose: Use this file when Zen runs Multi-Engine or any AI-assisted refactoring workflow. It keeps the safety guardrails explicit.

Contents

Current Snapshot

These figures are directional reference points, not hard gates:

  • Review time often improves by 40-60% when teams use AI refactoring tools well.
  • Smaller changes perform better; <=200 LOC is the preferred atomic size.
  • Structured test protocols materially reduce post-deploy issues.
  • High-impact legacy targets tend to produce the best ROI.
  • GitHub Copilot Code Review (agentic architecture overhaul March 2026): gathers full repository context before commenting — not just the diff. Reached 60M reviews by March 2026 (10× growth since April 2025 launch); surfaces actionable feedback in 71% of reviews (~5.1 comments/review). Starting June 1 2026, each review consumes GitHub Actions minutes / AI Credits on private repos. [Source: GitHub Docs — About Copilot code review; GitHub Changelog 2026-04-27]
  • Cursor vs Copilot (2026): Copilot scores higher on SWE-bench (56% vs 51.7%) but Cursor completes complex refactoring tasks ~30% faster (avg 62.95s vs 89.91s). Cursor's multi-agent Composer runs up to 8 parallel agents via git worktrees. [Source: tech-insider.org]
  • Sourcery for Python: version 1.43.0 (January 2026). Recommended 2026 Python quality setup: Ruff (local linting/formatting) + Sourcery (AI-assisted PR review). [Source: sourcery.ai]

Strengths and Limits

Reliable AI strengths

Capability Typical confidence
Variable and symbol renaming across files High
Function extraction and structural cleanup Medium-high
Dead-code removal High
Style unification High
Type tightening such as any reduction Medium
Documentation scaffolding Medium

Persistent weak spots

  • Domain-specific business logic
  • Architecture-level tradeoffs
  • Legacy constraints that live outside the code
  • Context-specific risk balancing
  • High-level refactors that depend on product intent rather than structure

Hallucination Watchlist

Check these explicitly after any AI-generated proposal:

Risk Typical symptoms
Incorrect imports Non-existent modules, wrong API versions, broken moved symbols
Edge-case mishandling Added or removed null checks, boundary-condition drift
Overzealous optimization Behavior changed in the name of cleanliness or performance
Behavioral modification Tests pass but domain semantics or error handling changed
Context loss Historical comments removed or legacy constraints ignored

AI-authored CVEs are accelerating: 35 newly disclosed in March 2026, up from 6 in January and 15 in February (Vibe Security Radar tracking; 27 of the 35 March CVEs traced to Claude Code by commit signature) [Source: Infosecurity Magazine — "Security Researchers Sound the Alarm on Vulnerabilities in AI-Generated Code" (2026-03-26), https://www.infosecurity-magazine.com/news/ai-generated-code-vulnerabilities/].

Zen Guardrails

Multi-Engine mitigation

  1. Run 3 independent proposals.
  2. Use the Compete merge strategy.
  3. Score on readability, consistency, and change volume.
  4. Keep a human review gate before adoption.

Additional safety rules

  • Keep each AI-assisted change to <=200 LOC.
  • Run static analysis in the verification path.
  • Compare Before/After metrics, not just prose claims.
  • Prepare rollback support for risky migrations, including feature-flag fallback when appropriate.
  • Keep AI changes structural. Humans decide domain-logic changes.
  • Start from non-critical utilities before higher-risk modules.
  • Run the relevant local test suite before merge.

Enterprise Workflow

Phase Purpose
1. Code Health Assessment Capture baseline metrics
2. Strategic Prioritization Overlay risk and change frequency
3. Tool Selection Match tool choice to scope and compliance needs
4. Atomic Transformation Keep 1 PR = 1 change, ideally <=200 LOC
5. Automated Quality Gates Apply CI/static-analysis checks
6. Continuous Measurement Track improvement over time, not just one PR

Target Metrics

Metric Target
Cyclomatic complexity reduction 15-25%
Duplicate-code reduction Measurable decrease
Review time reduction 40-60%
Post-refactor bug rate No worse than non-refactored code
LOC per change <=200

Source: Augment Code: AI Code Refactoring · IBM: AI Code Refactoring · DX: Enterprise AI Refactoring · Qodo: Evolution of Code Refactoring Tools · Second Talent: 5 AI Tools for Code Refactoring 2026 · GitHub Docs: About Copilot code review · GitHub Changelog: Copilot code review billing (2026-04-27) · tech-insider.org: Copilot vs Cursor 2026 benchmarks · Sourcery AI


Core Contract Long Form (SKILL.md excerpt)

  • When reviewing AI-generated code, actively scan for: architectural drift (inconsistent patterns across files), duplicated logic that should be extracted, hidden edge-case gaps, and security vulnerabilities (45% failure rate in security tests; 2.74× more vulnerabilities than human-written code per Veracode 2025). AI-generated vulnerabilities tend to be behavioral — they emerge from how components interact (auth flows, state transitions, session handling) rather than from a single dangerous line. Mentally execute the code as an attacker: what happens if steps are skipped, requests replayed, or inputs arrive out of order. AI-generated CVEs are accelerating (35 disclosed in March 2026 alone) — treat AI-authored code with the same scrutiny as untrusted external contributions. Concrete shapes to flag: raw errors or stack traces returned in user-facing responses (leaks schema, table and column names — an attacker roadmap), N+1 or in-loop data fetches that should be joins or batches, and SQL built via string concatenation. LLMs reproduce these because training-data frequency beats correctness, not because they are safe.

  • AI-session smells (5 canonical patterns) — alongside human code smells, scan AI-authored work for: (1) Kitchen-sink session — one prompt asked the agent to do three unrelated things, all half-done, (2) Correcting over and over — repeated micro-corrections instead of a single re-spec, (3) Over-specified CLAUDE.md — the project memory has bloated to >200 lines so important rules are buried, (4) Trust-then-verify gap — the user accepted output without running the verifier, (5) Infinite exploration — the agent kept reading files without ever moving to plan/implement. Each smell has a specific fix (re-scope / re-spec / progressive disclosure / mandatory verifier / explicit Plan-mode gate). [Source: code.claude.com/docs/en/best-practices — Common failure patterns]

  • Locality of Behaviour over DRY. Co-locate behaviour with its trigger so a reviewer (human or agent) understands the change from one file. The DRY benefit of an extracted helper is often outweighed by the comprehension cost of a 3-file jump. Apply LoB especially when the duplicate count is < 3 or when the would-be helper would have only one caller. [Source: htmx.org/essays/locality-of-behaviour/; alexkondov.com/locality-of-behavior-react/]

  • YAGNI × 100 in the AI era. AI codegen makes the marginal cost of speculative generality near-zero, which amplifies over-engineering — extra configs, premature interfaces, "just-in-case" extension points, defensive try/catch on every line. The strict YAGNI test: "is there a customer or test that fails today without this?" If no, remove it from the refactor and from the review checklist. Reject AI-proposed refactors whose justification reduces to "this will be more flexible later". [Source: blog.flurdy.com/2026/02/yagni-100-with-ai]

  • Rule of Three before abstraction. First duplicate is fine. Second duplicate is a yellow flag — read both and check whether they really represent the same concept. Only on the third duplicate of the same shape should the abstraction be extracted, and the abstraction should be named after the domain concept, not after a structural pattern. Early abstractions cost more than DRY violations because they encode a wrong concept across multiple call sites. [Source: blog.codinghorror.com/rule-of-three/]

  • Tautological-test detection during refactor reviews. When the refactor scope includes test files, scan for the six canonical tautological patterns from radar (field-exists / call-was-made / no-throw / mirrors-implementation / length-only / snapshot-only). Tag any such test for replacement before considering the refactor "behaviour-preserving" — a test that asserts nothing real cannot prove behaviour was preserved. [Source: codeintelligently.com — AI Generated Tests False Confidence]

  • Dead code tooling (2025-2026). For TypeScript/JavaScript, use knip as the primary dead-code scanner — it detects unused files, exports, types, and dependencies in a single pass (~300K weekly downloads; VSCode extension; --fix flag). ts-prune was archived on Sep 19, 2025 and should no longer be used. For Python, vulture + autoflake remain current; recommended CI pairing is ruff + sourcery (Sourcery 1.43.0, Jan 2026). [Source: knip.dev; knip.dev/explanations/comparison-and-migration; sourcery.ai]

  • AI-powered PR review (2026). GitHub Copilot Code Review underwent an agentic architecture overhaul in March 2026 — it now gathers full repository context before commenting (not just the diff), surfacing actionable feedback in 71% of reviews (~5.1 comments/review; 60M total reviews as of March 2026). When using Copilot Code Review on private repos, note it consumes GitHub Actions minutes / AI Credits starting June 1 2026. Cursor's multi-agent Composer handles complex multi-file refactoring ~30% faster than Copilot on complex tasks (SWE-bench: 56% Copilot vs 51.7% Cursor). [Source: GitHub Docs — About Copilot code review; GitHub Changelog 2026-04-27]

Source: SKILL.md on GitHub

1 warning13d5 checks · Risk SAFE
  • Gen Agent Trust Hub13d

    No security issues detected. The skill provides comprehensive and well-structured instructions for code refactoring and quality review. It utilizes standard developer tools, references reputable technical documentation from trusted organizations, and includes explicit safety boundaries to ensure behavior-preserving changes.

  • Socket13d

    No alerts

  • Snyk13d

    Risk: LOW · No issues

  • Runlayer6mo

    1/15 files flagged

  • ZeroLeaks5mo

    Score: 93/100 · 2 sections analyzed

Signed by skilld at e307415. This ties the file your Agent reads to that commit on GitHub. It does not review the instructions.

Last checked against GitHub 2 days ago.

Activeupdated 2 weeks ago

README badge

README badge for simota/agent-skills/zen