finishlisted

# finish 작업 끝의 반복 체인 `정리 → 리뷰 → 문서 점검 → 커밋 → push → PR`을 한 호출로 묶는다. 세션 로그 분석 결과 이 체인(특히 `commit → create-pr`, `simplify → code-review`)이 가장 빈번한 수동 반복이었다 — 단계마다 다음 스킬을 손으로 부르던 것을 한 번에 잇는다. > **핵심 원칙: 커밋·PR 스킬을 하드코딩하지 않는다.** > 저장소 전용 커밋·PR 스킬은 다른 환경에 없을 수 있다. 커밋 단계와 PR 단계는 항상 > **그 프로젝트에 적용되는 스킬을 감지해서** 이어 수행한다(아래 Step 4·Step 6). --- ## 실행 절차 호출과 함께 전달된 자연어 인자에서 스킵 옵션을 먼저 파싱한다(`no-simplify`/`no-review`/`no-pr`). 다른 스킬로 이어갈 때 스킬 호출 도구가 있으면 대상 스킬을 호출한다. 없으면 사용 가능한 스킬 목록에서 대상 지침을 찾아 읽고 현재 에이전트가 그대로 수행한다. 대상 지침도 없으면 조용히 건너뛰지 말고 사용자에게 알린다. ### Step 0: 변경 확인 ```bash git status --short && git diff --stat ``` 변경이 없으면 즉시 중단하고 사용자에게 알린다(커밋할 것 없음). 현재 브랜치가 `main`/`master`/`develop`이고 적용 중인 프로젝트 지침이 직접 작업을 허용하지 않으면 **경고**하고 계속할지 확인한다. 프로젝트 지침이 직접 작업을 명시적으로 허용하면 그 규칙을 우선한다. > 이 빈 변경 체크는 "정리·리뷰 전" 기준이다 — Step 1·2(simplify·code-review `--fix`)가 새 변경을 > 만들 수 있으므로, 그 단계 후 변경이 생기면 정상 진행한다. ### Step 1: 코드 정리 (`no-simplify`면 생략) `simplify` 스킬을 호출해 reuse·중복·불필요 코드를 정리한다. ### Step 2: 코드 리뷰 (`no-review`면 생략) `code-review` 스킬을 **effort `medium`**, **위험 지적만 수정**(`--fix`)으로 호출한다. > **effort 정규화:** code-review에 넘기는 effort는 항상 유효값(`low`/`medium`/`high`/`max`/`ultra`) > 중 하나로 **명시 전달**한다. 오타(`meduim`)나 미지정이면 기본값 `medium`으로 교정해서 넘긴다 — > 인식 못 한 값을 넘기면 의도와 다른 강도로 실행될 수 있으므로 추측에 맡기지 않는다. 리뷰가 🔴 위험(버그·보안·데이터 손상)을 지적하면 그것만 반영하고, 스타일/취향은 건너뛴다. ### Step 3: 문서 점검 — 이 변경이 어긋나게 만든 문서 찾기 코드는 리뷰를 받지만 문서는 아무도 안 보는 채로 틀린 상태가 된다. PR 시점이 가장 싸게 고칠 마지막 자리다. 아래 중 **해당되는 것만** 본다. 1. **이 작업이 참조하거나 이끈 문서** — `R