perf(ci): ktlint 를 PR 당 두 번 검사 — reviewdog 경로 32초·버전 이원화 정리 #1012

현황

lint.yml 의 ktlint job 은 같은 규칙을 두 번 검사한다.

reviewdog 경로가 32초, job 전체의 약 30% 이고 ktlint 실행 자체가 이중이다. 44개 PR 을 한 번에 갱신하면 약 23분이 여기 쓰인다.

왜 지금 다시 보는가

인라인 코멘트의 원래 수요자는 "Actions 로그 수만 줄을 뒤지기 싫은 사람" 이다. 현재 이 저장소는 작업자 전원이 에이전트(Codex·Claude Code)를 쓰고, 에이전트는 gh run view --log 로 로그를 직접 읽는다 — 인라인 코멘트는 오히려 API 를 한 번 더 호출해야 닿는 곳에 있다.

반면 게이트는 그대로 필요하다. 에이전트도 틀린다 (2026-08-24 세션에서 git add 누락으로 컴파일 불가 상태가 push 됐고, pre-commit 훅은 staged 스냅샷만 보므로 통과했다. 막은 건 CI 였다).

"막는 것"(ktlintCheck)과 "알려주는 것"(reviewdog)을 나눠 보면 앞은 필수, 뒤는 비용 대비 가치가 역전됐다.

부수 문제

• ktlint 버전이 이원화돼 있다. ScaCap/action-ktlint 가 자체 ktlint 를 내려받으므로 ktlint_version: "1.8.0" 을 libs.versions.toml 과 손으로 맞추는 중이다. 표류하면 두 검사 결과가 갈린다.
• @master 미고정 액션이라 Actions 참조 SHA 고정 · secret scanning · push protection 적용 #847 의 SHA 고정 대상이기도 하다.

Android Lint 는 대상이 아니다

같은 job 의 dvdandroid/action-android-lint 는 lintDebug 가 만든 XML 을 재활용한다. 검사를 다시 하지 않으므로 추가 비용이 거의 없고, 모듈별 위반이 수만 줄 로그에 묻히던 문제(#279)를 푼다. 유지한다.

ktlint 만 검사를 두 번 한다.

범위

둘 중 하나를 택해 적용한다.

A. reviewdog 단계 제거 (단순)

• Run Ktlint(ScaCap) 스텝을 없애고 ktlintCheck 하나만 남긴다.
• 실패 위치는 ktlintCheck 콘솔 출력(파일:라인)으로 충분한지 확인하고, 부족하면 job summary 에 요약을 붙인다.

B. 검사 1회 + 인라인 유지 (android-lint 패턴)

• ktlintCheck 가 checkstyle XML 리포트를 내도록 하고, 그 XML 을 reviewdog generic reporter 에 먹인다.
• 인라인 코멘트를 유지하면서 ktlint 실행은 1회가 되고 버전 이원화도 사라진다.

어느 쪽이든 ScaCap/action-ktlint@master 의존이 없어진다.

완료 조건

• ktlint 실행이 PR 당 1회다.
• ktlint 버전 정본이 libs.versions.toml 하나다 — workflow 에 손으로 적은 버전이 없다.
• ci: 삭제로 발생한 ktlint 위반을 전체 검사에서 감지 #875 회귀 없음: 삭제만으로 발생한 파일 단위 위반이 여전히 CI 를 실패시킨다. 그 성질은 원래 ktlintCheck 가 담당하므로 reviewdog 를 빼도 유지되지만, 회귀 테스트로 확인한다.
• lint.yml 의 ktlint job 소요가 실측으로 감소한다. 수치를 이 이슈에 남긴다.
• B 를 택했다면 위반이 있는 PR 에서 인라인 코멘트가 실제로 달린다.

관련

• 삭제로 생긴 위반 감지(ktlintCheck 도입): ci: 삭제로 발생한 ktlint 위반을 전체 검사에서 감지 #875
• Actions SHA 고정·@master 제거: Actions 참조 SHA 고정 · secret scanning · push protection 적용 #847
• Android Lint 멀티모듈 인라인: chore(ci): Reviewdog 가 :feature/* · :core/* 모듈 lint 도 PR 인라인 표시하도록 multi-module 지원 #279

근거

• 스텝 소요는 lint.yml 최근 성공 run 의 job steps API 실측 (2026-08-24)
• reviewdog filter_mode: https://github.com/reviewdog/reviewdog#filter-mode