Skip to content
cccskills
FreePublished on GitHub

kent-beck-review

Code review from the Kent Beck Simple Design 4 Rules perspective. God Object splitting, SRP, duplication removal, naming, cyclomatic complexity. Triggered by "kent beck", "simple design", "simplify", "refactoring" ("리팩토링"), "refactor", "god object", "SRP" keywords.

The English summary is being prepared. Showing the original description.

mangowhoiscloud/geode
See how to install

Files included (1)

  • SKILL.md2.9 KB

SKILL.md (original)

Review the instructions the agent will receive before you install.

Kent Beck Code Review Lens

"Make the change easy, then make the easy change."

Four Rules of Simple Design (in priority order)

  1. Passes tests — Tests pass
  2. Reveals intention — Names and structure reveal intent
  3. No duplication — Eliminate idea-level duplication
  4. Fewest elements — Minimal components

Review Checklist

File Size

# Detect 500+ line files (God Object candidates)
find core/ -name "*.py" -exec wc -l {} + | sort -rn | head -20

File size identifies inspection candidates, not a split requirement. Trace responsibilities, callers, and reasons to change before proposing extraction. Do not carry a dated file-size snapshot forward as current implementation status.

Method Size & Complexity

Inspect the actual function and its callers. Function length, nesting, and branch counts do not by themselves justify a Strategy, dispatch table, or new module. Prefer an early return or an existing helper when it resolves the identified failure without obscuring domain behavior.

Reveals Intention

Anti-patternImprovement
def process(data)def score_analyst_response(response)
result = fn(x, y, z)Use meaningful variable names
# This function does XReveal through function name, remove comment
magic number 0.7CONFIDENCE_THRESHOLD = 0.7

No Duplication

Detect idea-level duplication (not just copy-paste but missing abstractions):

Trace repeated decisions through their shared owner. Unify behavior that must change together; keep similar-looking paths separate when their contracts differ. Neither repetition count nor structural similarity alone requires a factory, Protocol, or generic abstraction.

Fewest Elements

Unnecessary ElementCriteria
Unused parametersWrappers that only pass **kwargs
ABC with single implementationCheck the substitution contract; a concrete class may be sufficient
Empty __init__.pyCheck package, distribution, discovery, and import contracts before removal
Unused importsAuto-detected by ruff F401

Scope and Verification

A review does not authorize refactoring. Follow the workflow for requested changes and the deletion gate before removing code, tests, or package files. Use verification gates for targeted checks and evidence reuse. Report concrete findings with current file/line evidence; no finding is preferable to a speculative simplification.

Reviews

No reviews yet. Tell others what you thought.

Comments

No login needed. Share how it worked for you, ask a question, or note where it did not run.

No comments yet. Be the first to write one.

Comments are published right away. Do not include personal information such as email addresses or phone numbers. Up to 2 URLs.

0 / 1,000 characters

This form is protected by reCAPTCHA, and the Google Privacy Policy and Terms of Service apply.

More skills from this repository

agent-anti-pattern

Free

Audit agent runtimes and their tests for lifecycle inference, capability overload, context pollution, correlated verification, prompt-only control, false success, dead compatibility paths, test-only seams, and complexity without contribution. Use for whole-repository agent audits, slop or dead-code triage, test contribution reviews, large refactors, tool/context simplification, and cleanup requests that need proof before deleting code or tests.

The English summary is being prepared. Showing the original description.

mangowhoiscloud/geode15Updated Oct 22, 2026

agent-ops-debugging

Free

Diagnose agent operational failures through default-policy checks, multi-gap analysis, explicit service ownership, request-context lifetime, and degradation-versus-correctness checks.

The English summary is being prepared. Showing the original description.

mangowhoiscloud/geode15Updated Oct 22, 2026

agent-world-benchmark

Free

Align, run, audit, and publish GEODE benchmark comparisons against Agent-World arXiv 2604.18292v1 across MCP-Mark, BFCL V4, and tau2-bench. Use for Agent-World or AgentWorld comparison requests, cross-suite agent benchmark design, Tau2 comparator/profile diagnosis, paired thin-runtime-versus-GEODE experiments, mean-accuracy@8 replication, or benchmark comparability and artifact reviews.

The English summary is being prepared. Showing the original description.

mangowhoiscloud/geode15Updated Oct 22, 2026

anti-deception-checklist

Free

Verification checklist to prevent fake success. Detects test deletion/disabling, coverage regression, lint bypass, secret exposure. Triggered by "deception", "fake" ("가짜"), "fake success", "verification" ("검증"), "checklist" ("체크리스트"), "deletion detection" ("삭제 탐지"), "regression" keywords.

The English summary is being prepared. Showing the original description.

mangowhoiscloud/geode15Updated Oct 22, 2026

architecture-patterns

Free

Implement proven backend architecture patterns including Clean Architecture, Hexagonal Architecture, and Domain-Driven Design. Use when architecting complex backend systems or refactoring existing applications for better maintainability.

The English summary is being prepared. Showing the original description.

mangowhoiscloud/geode15Updated Oct 22, 2026

arxiv-digest

Free

Auto-search and summarize latest AI/agent papers.

The English summary is being prepared. Showing the original description.

mangowhoiscloud/geode15Updated Oct 22, 2026

See all skills by mangowhoiscloud

Report a problem with this skill