本文へ移動
cccskills
無料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.

mangowhoiscloud/geode
インストール方法を見る

含まれるファイル（1）

  • SKILL.md2.9 KB

SKILL.md（原文）

インストールする前に、エージェントに与えられる指示の中身を確認できます。

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.

レビュー

まだレビューはありません。使ってみた感想をお寄せください。

ログインしてレビューを書く

同じリポジトリのスキル

概要と使いどころ

agent-anti-pattern

無料

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.

日本語の概要は準備中です。原文の説明を表示しています。

mangowhoiscloud/geode152026年10月12日 更新

agent-ops-debugging

無料

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

日本語の概要は準備中です。原文の説明を表示しています。

mangowhoiscloud/geode152026年10月12日 更新

agent-world-benchmark

無料

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.

日本語の概要は準備中です。原文の説明を表示しています。

mangowhoiscloud/geode152026年10月12日 更新

anti-deception-checklist

無料

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.

日本語の概要は準備中です。原文の説明を表示しています。

mangowhoiscloud/geode152026年10月12日 更新

architecture-patterns

無料

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.

日本語の概要は準備中です。原文の説明を表示しています。

mangowhoiscloud/geode152026年10月12日 更新

arxiv-digest

無料

Auto-search and summarize latest AI/agent papers.

日本語の概要は準備中です。原文の説明を表示しています。

mangowhoiscloud/geode152026年10月12日 更新

mangowhoiscloud のスキルをすべて見る

このスキルの問題を報告する