테스트 없는 기존 코드에 행위 보존 안전망(Characterization → Approval → Mutation)을 구축한 뒤 개선 스킬로 핸드오프. "레거시에 테스트 추가", "안전망 구축", "characterization test" 요청 시 사용. /tdd-legacy로 호출.
The English summary is being prepared. Showing the original description.
See how to installReview the instructions the agent will receive before you install.
테스트 없는 기존 코드의 현재 행위를 고정하는 안전망을 만들고, 개선은 기존 스킬로 넘깁니다.
/tdd-tidy 또는 /system-wide-refactoring 핸드오프 안내로 종료/tdd-legacy <대상 클래스 FQCN 또는 파일 경로>
test: <대상> characterization 테스트 추가. 커밋 메시지는
docs/reviewable-commits.md(없으면 ${CLAUDE_PLUGIN_ROOT}/references/reviewable-commits.md)
표준을 따른다 — body에 어떤 행위를 고정했고 sabotage 검증 결과를 기록.
길이는 ../../references/commit-style.md의 간결성 규칙(제목 + 핵심 bullet 2~4줄)을 따른다.
커밋 직전 git diff --stat으로 테스트 외 파일 변경이 없는지 확인한다
(sabotage 원복 누락 차단).CombinationApprovals.verifyAllCombinations()로
조합 전체를 승인 파일에 고정 (approvaltests 의존성 필요 — 없으면 추가를
사용자에게 확인).test: <대상> combination approval 추가mutate4java가 있으면 그것을 사용 (1순위) — 에이전트 mutate4java-runner 또는 jar 직접
실행. 설치·판정은 ../../references/hardening-tools.mdplugins { id "info.solidsoft.pitest" version "1.15.0" } +
pitest { targetClasses = ["<대상 FQCN>"] } → ./gradlew pitestorg.pitest:pitest-maven 플러그인 <targetClasses> 설정 →
mvn test-compile org.pitest:pitest-maven:mutationCoveragetest: <대상> 뮤턴트 킬 테스트 보강안전망 완료를 보고하고 종료한다:
안전망 완료 (characterization N개 + approval M개, mutation score X%). 이제 개선을 진행하세요: 코드 정리는
/tdd-tidy, 구조 개선은/system-wide-refactoring. 새 기능 추가는/tdd-feature.
개선 실행은 이 스킬 범위 밖이다.
No reviews yet. Tell others what you thought.
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.
동일한 결과를 내는 여러 조건문(OR 나열·중첩 AND)을 하나로 통합하고 의미 있는 boolean 메서드로 추출. "조건문 합쳐", "같은 결과 반환하는 if 정리", "중첩 if 평탄화", "/consolidate-conditional" 요청 시 사용. 단, 여러 메서드에 흩어진 동일 조건을 호출자 쪽으로 올리는 것은 /lift-up-conditional, 복잡한 조건식·분기를 메서드로 쪼개는 것은 /decompose-conditional이 적합. /consolidate-conditional [commit-ref]로 호출.
The English summary is being prepared. Showing the original description.
기능의 external behavior를 Cucumber 인수 테스트(주 검증층)로 구축 — .feature 실행으로 문서↔코드 드리프트를 구조적으로 차단, Four Layer(Steps→Protocol Driver→SUT), 태그 기반 가역 제외, 기존 JUnit 인수 테스트 이관. "인수 테스트 도입", "Gherkin을 실행 가능하게", "cucumber 셋업" 요청 시 사용. /cucumber-acceptance로 호출.
The English summary is being prepared. Showing the original description.
복잡한 if/then/else의 조건식과 각 분기를 의미 있는 메서드로 추출하여 가독성 향상. "조건문 분해", "if 가독성", "복잡한 조건식에 이름 붙여", "/decompose-conditional" 요청 시 사용. 단, 같은 결과를 내는 조건문들을 하나로 합치는 것은 /consolidate-conditional, 타입별 분기를 클래스로 바꾸는 것은 /replace-conditional-with-poly가 적합. /decompose-conditional [commit-ref]로 호출.
The English summary is being prepared. Showing the original description.
Primitive Obsession 제거 — 검증·연산이 따라다니는 primitive 필드(금액+통화, 이메일 문자열 등)를 도메인 개념을 담은 Value Object로 치환. "값 객체 도입", "primitive obsession", "Money 클래스로", "/discover-value-object" 요청 시 사용. 단, 함께 전달되는 파라미터 묶음을 객체로 바꾸는 것은 /introduce-parameter-object, 컬렉션을 감싸는 것은 /first-class-collection이 적합. /discover-value-object [commit-ref]로 호출.
The English summary is being prepared. Showing the original description.
컬렉션 getter가 내부 List/Set을 직접 노출하는 것을 방지 — unmodifiable 반환 + add/remove 메서드 제공. "컬렉션 캡슐화", "getter가 List 그대로 노출", "unmodifiable로", "/encapsulate-collection" 요청 시 사용. 단, 컬렉션과 관련 로직을 전용 클래스로 뽑는 것은 /first-class-collection이 적합. /encapsulate-collection [commit-ref]로 호출.
The English summary is being prepared. Showing the original description.
암묵적 의존성(전역 변수·클래스 필드·싱글턴 접근)을 명시적 파라미터로 전환하여 메서드 투명성 향상. "숨은 의존성 드러내", "필드 대신 파라미터로", "전역 참조 제거", "/explicit-parameters" 요청 시 사용. 단, 파라미터가 많아져 묶어야 하면 /introduce-parameter-object, I/O와 계산 분리는 /segregate-functional-core가 적합. /explicit-parameters [commit-ref]로 호출.
The English summary is being prepared. Showing the original description.