Skip to content

fix: AI 리뷰 미완성 응답 진단 및 재시도 - #28

Merged
hyesngy merged 1 commit into
developfrom
fix/ai-review-incomplete-response
Oct 16, 2026
Merged

hyesngy merged 1 commit into
developfrom
fix/ai-review-incomplete-response

Conversation

@hyesngy

@hyesngy hyesngy commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

관련 이슈

변경사항

  • Gemini 응답의 종료 이유를 허용 목록으로 제한하여 오류에 표시합니다. 응답 본문과 diff는 로그에 남기지 않습니다.
  • MAX_TOKENS로 끝나면 출력 한도를 8,192에서 16,384로 높여 한 번 재시도합니다. 다른 종료 이유는 재시도하지 않습니다.
  • 재시도, 재실패, 안전한 오류 출력 테스트와 운영 문서를 추가합니다.

변경·영향 범위

담당 주 영역:

  • 앱 기능 (Capture / AI / Storage / UI)
  • StoreKit·사용량 정책
  • API·프록시
  • 앱 설정·권한·릴리스
  • 테스트·CI·개발 도구
  • 규칙·문서 (AGENTS.md, docs/)
  • 기타:

PR 분류

  • 일반 변경
  • Critical Path 변경

CI의 AI 리뷰 스크립트 변경이므로 머지 전 담당자 확인이 필요합니다.

사전 예측

  • 예상 소요·기대 효과: 종료 원인을 식별하고 출력 한도로 인한 미완성 응답을 자동 복구합니다.
  • 예상 병목·리스크: 기존 실패의 실제 종료 이유가 로그에 없으므로 원인이 MAX_TOKENS인지 아직 확인할 수 없습니다. 재시도 시 Gemini 호출 비용과 시간이 증가할 수 있습니다.
  • 담당 영역 밖 확인 사항: pull_request_target은 기준 브랜치의 스크립트를 실행하므로, 이 PR 머지 후 #27의 AI 리뷰를 다시 실행해야 새 동작을 확인할 수 있습니다.

실제 결과·트러블슈팅

  • 실제 소요·반영 결과: 동일한 실패를 두 번 확인한 뒤 종료 이유 표시와 제한된 재시도를 구현했습니다.
  • 실제 병목·해결 과정: 로컬 샌드박스에서 SwiftUI 매크로 서비스가 동작하지 않아 Xcode에 정상 시스템 접근을 허용하여 빌드·테스트를 완료했습니다.
  • 배운 점·후속 작업: 현재 로그만으로 원인은 확정할 수 없습니다. 머지 후 #27의 리뷰를 재실행하고 새 종료 이유와 결과를 확인합니다.

검증 결과

  • 검증 내용: node --check .github/ai-review/run.mjs 통과, node --test .github/ai-review/*.test.mjs 19개 통과, xcrun swift-format lint --recursive --strict Paxo PaxoTests 통과, macOS xcodebuild build 통과, xcodebuild test 72개 통과, git diff --check 통과. 실제 Gemini 호출은 이 PR 머지 전에는 기준 브랜치의 기존 스크립트가 실행되므로 새 코드로 검증할 수 없습니다.
  • API Contract 앱·프록시 양쪽 반영 여부: 해당 없음. 앱·프록시 API를 변경하지 않았습니다.
  • 시크릿 포함 여부: 없음. 기존 테스트의 가짜 키 문자열만 유지했고 실제 키와 토큰은 추가하지 않았습니다.

@github-actions

Copy link
Copy Markdown

AI 사전 코드 리뷰 (파일럿)

검토 SHA: fd6aaa434b2a · 방식: 전체
상태: 미해결 0 · 해결 0 · 오탐 0 · 위험 수용 0
AI 호출: 1/20

AI 코드 리뷰 파이프라인에서 Gemini 응답의 출력 토큰 한도(`MAX_TOKENS`) 초과 시 최대 토큰 수를 확장(8,192 → 16,384)하여 1회 재시도하는 로직을 추가하고, 실패 시 민감한 본문 유출 없이 안전한 종료 이유만 예외 메시지로 남기도록 개선했습니다. 이에 대한 단위 테스트 및 작업 문서 업데이트도 함께 포함되어 있습니다.

ID 심각도 확신·구분 위치 문제 상태
— — — — 수정할 가치가 높은 문제를 찾지 못했습니다. —

사람의 판정: /ai-review false-positive AX-... 이유(10자 이상), /ai-review accepted-risk AX-... 이유(10자 이상), /ai-review reopen AX-...
AI 결과는 승인이나 머지 조건을 대체하지 않습니다.

@hyesngy hyesngy self-assigned this Oct 16, 2026
@hyesngy
hyesngy merged commit 3b2765a into develop Oct 16, 2026
7 checks passed
@hyesngy
hyesngy deleted the fix/ai-review-incomplete-response branch October 16, 2026 04:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant