convention-review

name	convention-review
description	현재 diff 또는 프로젝트 전체 코드가 udumeoli-web 전용 룰(FSD 경계, good-* 기준, simplicity 사다리)에 맞는지 검토한다. TRIGGER when: user invokes /convention-review directly, or asks for a "컨벤션 리뷰"/"룰 위반" check on the current diff or on the whole codebase.

Convention Review

diff 또는 프로젝트 전체가 이 저장소 고유 컨벤션에 맞는지만 검토한다. 범용 버그·가독성·성능 리뷰는 /code-review가 다룬다 — 여기선 이 프로젝트 룰 파일에 적힌 기준 위반만 잡는다.

모드

• diff 모드 (기본): 요청에 "전체", "프로젝트 전체", "코드베이스 전체" 같은 명시가 없으면 이 모드. 지금 작업 중인 변경만 본다.
• 전체 스캔 모드: "프로젝트 전체 훑어줘", "전체 코드베이스 컨벤션 확인해줘"처럼 명시적으로 요청할 때만. src/ 전체를 대상으로 한다 — 범위가 크므로 아래 "전체 스캔 시 추가 규칙"을 반드시 따른다.

절차

Step 1. 대상 확보

• diff 모드 기본: git diff HEAD (스테이징 여부 무관하게 작업 중인 변경)
• diff 모드, 브랜치 전체 리뷰 요청 시: git diff main...HEAD
• 전체 스캔 모드: src/ 아래 파일 전체 (레이어별로 나눠 순회 — app → pages → shared → 그 외)

Step 2. 파일별 적용 룰 선택

대상	적용 룰
항상	simplicity.md — 과설계, 재사용 안 하고 새로 만든 것, 사다리(YAGNI→재사용→stdlib→네이티브→기존 의존성→한 줄) 위반
src/**/*.ts(x) 전부	front.md — FSD 레이어 단방향 의존성, 슬라이스 간 cross-import 금지, index.ts Public API 우회
.tsx	good-code, good-a11y, good-responsive
.ts (비-.tsx)	good-code
vite.config·tsconfig·의존성(package.json) 변경	good-bundling
버그 수정으로 보이는 diff	good-debug — 증상만 땜빵했는지, 공유 호출자를 다 확인했는지

Step 3. 위반만 찾는다

• diff 모드: diff에 없는 기존 코드(범위 밖)의 기존 위반은 지적하되 "diff 밖"이라고 명시하고 강제하지 않는다
• 룰 파일에 명시 안 된 스타일 취향은 지적하지 않는다

전체 스캔 시 추가 규칙

• 린트가 이미 잡는 건 다시 잡지 않는다. eslint-plugin-boundaries가 설정돼 있으면(eslint.config.*에서 확인) front.md의 FSD 경계 위반은 스킵한다 — pnpm lint가 기계적으로 잡는 걸 AI가 또 훑으면 중복이다.
• 알려진 debt는 재지적하지 않는다. eslint-disable 옆에 사유 주석이 달려 있으면 그건 이미 추적 중인 위반이다 — 새로 발견한 것처럼 보고하지 않는다.
• 결과가 많으면 룰별로 묶어서 보고한다. 파일 하나하나 나열하지 않고 "같은 위반 N건" 식으로 압축해, 사람이 우선순위를 잡을 수 있게 한다.
• 전체 스캔은 비용이 크다 — 요청에 명시적으로 "전체"가 없으면 절대 이 모드로 넘어가지 않는다.

출력 형식

path:line: <위반 룰 파일/스킬>: <문제>. <고칠 방법>.

위반이 하나도 없으면:

컨벤션 위반 없음 — 검토한 룰: <목록>

행동 원칙

• 범용 코드 품질·버그·효율성은 다루지 않는다 — /code-review 몫이다
• 근거 없는 지적 금지 — 어떤 룰 파일의 어떤 기준을 위반했는지 항상 명시한다
• 이미 있던 코드(diff 밖)의 위반을 이번 변경 탓으로 돌리지 않는다