2장: AI 코드 리뷰 자동화 - 원리와 아키텍처

LLM의 코드 리뷰 추론 과정:
 
  1단계 - 구문 분석:
    - 언어 식별 (TypeScript, Python 등)
    - 코드 구조 파악 (함수, 클래스, 모듈)
    - 타입 정보 추론
 
  2단계 - 의미 분석:
    - 변수명과 함수명에서 의도 추론
    - 제어 흐름 분석 (분기, 반복, 예외)
    - 데이터 흐름 추적
 
  3단계 - 맥락 매칭:
    - 학습된 코드 패턴과 비교
    - 알려진 안티패턴 식별
    - 보안 취약점 패턴 매칭
 
  4단계 - 피드백 생성:
    - 문제점의 심각도 분류
    - 개선 방안 제안
    - 근거와 설명 생성

맥락 범위와 리뷰 품질의 관계:
 
  변경 수준만 제공:
    - 구문 오류, 명백한 버그 식별 가능
    - 비즈니스 로직 오류 식별 불가
    - 프로젝트 컨벤션 준수 여부 판단 불가
 
  파일 수준 추가:
    - 함수 간 관계, 타입 일관성 검증 가능
    - 같은 파일 내 중복 코드 식별 가능
    - 클래스/모듈의 응집도 평가 가능
 
  프로젝트 수준 추가:
    - 아키텍처 패턴 준수 여부 판단 가능
    - 프로젝트 전체의 일관성 검증 가능
    - 기존 유틸리티 활용 제안 가능

// 맥락 최적화 전략
interface ContextStrategy {
  // 1단계: 변경 파일 분류
  classifyFiles(diff: FileDiff[]): ClassifiedFiles
 
  // 2단계: 우선순위별 맥락 수집
  gatherContext(classified: ClassifiedFiles): ReviewContext
 
  // 3단계: 토큰 예산 내 맥락 축소
  trimToTokenBudget(context: ReviewContext, budget: number): ReviewContext
}
 
interface ClassifiedFiles {
  // 비즈니스 로직 변경 - 최우선 리뷰 대상
  businessLogic: FileDiff[]
  // 인프라/설정 변경 - 중요도 높음
  infrastructure: FileDiff[]
  // 테스트 변경 - 비즈니스 로직과 함께 리뷰
  tests: FileDiff[]
  // 스타일/포맷 변경 - 낮은 우선순위
  formatting: FileDiff[]
}

// 이벤트 수신기 핵심 로직
interface PullRequestEvent {
  action: "opened" | "synchronize" | "reopened"
  pullRequest: {
    number: number
    title: string
    body: string
    baseBranch: string
    headBranch: string
  }
  repository: {
    owner: string
    name: string
  }
}
 
function shouldReview(event: PullRequestEvent): boolean {
  // 리뷰 대상 필터링 조건
  const skipPatterns = [
    /^(chore|style|docs):/,  // 리뷰 불필요한 커밋 타입
    /\[skip-review\]/,        // 명시적 스킵 표시
  ]
 
  const title = event.pullRequest.title
  return !skipPatterns.some(pattern => pattern.test(title))
}

interface FileChange {
  filename: string
  status: "added" | "modified" | "deleted" | "renamed"
  additions: number
  deletions: number
  patch: string        // unified diff 형식
  language: string     // 파일 확장자 기반 언어 식별
  category: ChangeCategory
}
 
type ChangeCategory =
  | "business-logic"   // 핵심 비즈니스 로직
  | "api-endpoint"     // API 엔드포인트
  | "database"         // 스키마, 마이그레이션, 쿼리
  | "test"             // 테스트 코드
  | "config"           // 설정 파일
  | "dependency"       // 패키지 의존성
  | "documentation"    // 문서
  | "style"            // 코드 포맷팅
 
function categorizeChange(file: FileChange): ChangeCategory {
  const path = file.filename.toLowerCase()
 
  if (path.includes("test") || path.includes("spec")) return "test"
  if (path.includes("migration")) return "database"
  if (path.match(/\.(json|ya?ml|toml|env)$/)) return "config"
  if (path.match(/package.*\.json|requirements|go\.mod/)) return "dependency"
  if (path.match(/\.md$/)) return "documentation"
  if (path.match(/route|controller|handler|endpoint/)) return "api-endpoint"
 
  return "business-logic"
}

interface ReviewContext {
  // PR 메타데이터
  prTitle: string
  prDescription: string
 
  // 변경 파일과 diff
  changes: FileChange[]
 
  // 관련 파일의 전체 내용 (변경된 파일 + 참조 파일)
  fullFiles: Map<string, string>
 
  // 프로젝트 맥락
  projectContext: {
    codeStyle: string       // 코딩 컨벤션 문서
    architecture: string    // 아키텍처 가이드
    recentReviews: string   // 최근 리뷰 히스토리
  }
}
 
async function gatherContext(
  event: PullRequestEvent,
  changes: FileChange[]
): Promise<ReviewContext> {
  const fullFiles = new Map<string, string>()
 
  // 변경된 파일의 전체 내용 수집
  for (const change of changes) {
    if (change.status !== "deleted") {
      const content = await fetchFileContent(
        event.repository,
        event.pullRequest.headBranch,
        change.filename
      )
      fullFiles.set(change.filename, content)
    }
  }
 
  // 관련 파일 수집 (임포트 분석)
  for (const change of changes) {
    const content = fullFiles.get(change.filename)
    if (content) {
      const imports = extractImports(content)
      for (const importPath of imports) {
        if (!fullFiles.has(importPath)) {
          const importContent = await fetchFileContent(
            event.repository,
            event.pullRequest.headBranch,
            importPath
          )
          if (importContent) {
            fullFiles.set(importPath, importContent)
          }
        }
      }
    }
  }
 
  return {
    prTitle: event.pullRequest.title,
    prDescription: event.pullRequest.body,
    changes,
    fullFiles,
    projectContext: await fetchProjectContext(event.repository),
  }
}

function buildReviewPrompt(context: ReviewContext): string {
  const systemPrompt = `당신은 시니어 소프트웨어 엔지니어입니다.
PR의 코드 변경을 리뷰하고, 문제점과 개선 사항을 식별해야 합니다.
 
리뷰 기준:
1. 정확성: 로직 오류, 엣지 케이스 누락, 타입 안전성
2. 보안: 인젝션, 인증 우회, 민감 정보 노출
3. 성능: 불필요한 연산, 메모리 누수, N+1 쿼리
4. 가독성: 네이밍, 구조, 복잡도
5. 유지보수성: 결합도, 응집도, 확장 가능성
 
응답 형식:
각 피드백은 다음 JSON 형식으로 제공하세요.
- file: 파일 경로
- line: 해당 줄 번호
- severity: critical | warning | suggestion | praise
- category: bug | security | performance | readability | maintainability
- comment: 구체적인 피드백 내용
- suggestion: 개선된 코드 (선택)`
 
  const userPrompt = buildUserPrompt(context)
  return systemPrompt + "\n\n" + userPrompt
}

interface ReviewFeedback {
  file: string
  line: number
  severity: "critical" | "warning" | "suggestion" | "praise"
  category: string
  comment: string
  suggestion?: string
}
 
function filterFeedback(
  feedbacks: ReviewFeedback[],
  options: FilterOptions
): ReviewFeedback[] {
  let filtered = feedbacks
 
  // 1. 심각도 기반 필터링
  if (options.minSeverity) {
    const severityOrder = ["praise", "suggestion", "warning", "critical"]
    const minIndex = severityOrder.indexOf(options.minSeverity)
    filtered = filtered.filter(f => {
      return severityOrder.indexOf(f.severity) >= minIndex
    })
  }
 
  // 2. 중복 제거 (같은 파일, 같은 줄, 유사한 코멘트)
  filtered = deduplicateFeedbacks(filtered)
 
  // 3. 이전 리뷰와 중복 확인
  if (options.previousReviews) {
    filtered = removeAlreadyReviewed(filtered, options.previousReviews)
  }
 
  // 4. 최대 피드백 수 제한
  if (options.maxFeedbacks) {
    filtered = prioritizeAndLimit(filtered, options.maxFeedbacks)
  }
 
  return filtered
}

프롬프트 구조 권장 순서:
 
  1. 역할 정의: "당신은 10년 경력의 백엔드 엔지니어입니다"
  2. 작업 설명: "PR의 코드 변경을 리뷰하세요"
  3. 리뷰 기준: 정확성, 보안, 성능, 가독성 체크리스트
  4. 맥락 제공: 프로젝트 구조, 코딩 컨벤션
  5. 코드 변경: diff + 관련 파일
  6. 출력 형식: JSON 스키마 명시
  7. 제약 조건: "추측하지 말 것", "근거를 제시할 것"

비용 추정 (2026년 기준, 대략적 수치):
 
  소규모 PR (변경 100줄 이하):
    - 입력 토큰: 약 3,000-5,000
    - 출력 토큰: 약 1,000-2,000
    - 예상 비용: $0.01-0.03 (Claude Sonnet 기준)
 
  중규모 PR (변경 100-500줄):
    - 입력 토큰: 약 10,000-30,000
    - 출력 토큰: 약 2,000-5,000
    - 예상 비용: $0.05-0.15
 
  대규모 PR (변경 500줄 이상):
    - 입력 토큰: 약 30,000-100,000
    - 출력 토큰: 약 5,000-10,000
    - 예상 비용: $0.15-0.50
 
  월간 비용 추정 (팀 5명, 일일 PR 3개):
    - 월간 PR 수: 약 300개
    - 월간 예상 비용: $15-45

지연 시간 최적화 전략:
 
  병렬 처리:
    - 파일별로 독립적인 리뷰 요청을 병렬 실행
    - 맥락 수집과 이전 리뷰 조회를 병렬 수행
 
  캐싱:
    - 프로젝트 맥락 (코딩 컨벤션, 아키텍처 문서) 캐싱
    - 변경되지 않은 파일의 리뷰 결과 캐싱
 
  점진적 리뷰:
    - 첫 번째 패스: critical/warning 피드백만 빠르게 제공
    - 두 번째 패스: suggestion/praise 피드백 추가 제공