review-solid
Installation
SKILL.md
SOLID·클린코드 렌즈 (review-solid)
책임 경계·이름·의존성·추상화를 본다. evidence-gated — 관찰된 신호 없이는 침묵한다. 원문 자체가 "기계적으로 쫓지 말라"고 반복하므로, 이 렌즈의 핵심은 지적 못지않게 지적을 참는 것이다.
원문: 이 스킬 디렉토리의
references/guide.md(동봉 — 외부 경로 의존 없음) 아래 목록은 그 문서의 압축본이다. 원칙별 절(SRP·OCP·LSP·ISP·DIP)과 클린코드 절이 있고, "조건문이 항상 나쁜 것은 아니다"·"과도하게 분리한 예" 같은 반대 방향 절이 지적 금지의 근거다. 판단이 애매하면 해당 절을 직접 읽고 결정한다.⚠ 아래 신호 목록이 곧 게이트다. 원문에 있어도 여기 없는 항목은 리뷰에서 존재하지 않는 것과 같다 — "애매하면 원문을 읽어라"는 목록에 있는 항목을 판정할 때만 발동하기 때문이다. 그러므로 신호를 늘리는 유일한 방법은 이 목록에 줄을 추가하는 것이고, 추가할 때는 반드시 good·good2 픽스처로 오탐 회귀를 확인한다. 원문에 대응 절이 없는 항목(상태 조합 모순·중첩 임계)은 이 파일이 정본이다.
실행
- 대상 결정: 인자 파일, 없으면
git diff변경 파일. 기본 브랜치를main으로 하드코딩하지 말 것 — 레포마다 다르다(dev·master·trunk등).git diff --name-only --diff-filter=d HEAD # staged + unstaged (--diff-filter=d: 삭제 파일 제외) BASE=$(git symbolic-ref -q --short refs/remotes/origin/HEAD) # 비어 있으면 PR base 나 사용자가 알려 준 브랜치 git diff --name-only --diff-filter=d "$BASE...HEAD" # 브랜치 변경분 - 각 파일에 3단 게이트 적용.
primary(설계) /other(정확성·안정성) 분리 보고. 각 finding에 관찰한signal을 명시.