Skip to content

[review] PR 자동 리뷰를 최초 생성 시점에만 실행 - #19

Merged
Dino0204 merged 2 commits into
mainfrom
fix/pr-auto-review-trigger
Aug 19, 2026
Merged

[review] PR 자동 리뷰를 최초 생성 시점에만 실행#19
Dino0204 merged 2 commits into
mainfrom
fix/pr-auto-review-trigger

Conversation

@Dino0204

Copy link
Copy Markdown
Collaborator

변경 사항

PR 자동 리뷰가 실행되는 시점을 opened 액션(최초 생성)으로만 제한하였습니다. 기존에는 opened, reopened, synchronize, ready_for_review 모두에서 자동 리뷰가 실행되어, PR에 커밋을 푸시할 때마다(synchronize) 자동으로 재리뷰가 돌았습니다.

  • src/server/handler.ts: PR_ACTIONSAUTO_REVIEW_PR_ACTIONS = ['opened']로 변경
  • src/config.ts: autoReview 필드 주석을 새 동작에 맞게 수정

변경 이유

synchronize(푸시) 시점마다 자동 리뷰가 반복 실행되어 GSML 토큰 사용량이 불필요하게 늘어났습니다. 리뷰 후 추가 커밋에 대한 재검토는 /review 코멘트로 명시적으로 요청하는 흐름으로 충분하다고 판단하여, 자동 리뷰는 PR 최초 생성 시점에만 실행되도록 하였습니다.

확인 사항

  • npm run typecheck, npm run build 통과 확인
  • 자동화된 테스트가 없는 리포지토리이므로, 실제 웹훅 동작 확인은 이 PR을 실제 리포지토리에 반영한 뒤 opened/synchronize 이벤트로 직접 검증이 필요합니다.

@Dino0204 Dino0204 self-assigned this Aug 19, 2026
@Dino0204

Copy link
Copy Markdown
Collaborator Author

/review

@itplay-code-review-bot itplay-code-review-bot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 코드 리뷰

요약을 생성하지 못했다.

지적 사항: 🟡 1


모델 darwin-35b-q4_k_m.gguf · 파일 2개 리뷰 · 토큰 2,706 in / 736 out

Comment thread src/server/handler.ts Outdated

@itplay-code-review-bot itplay-code-review-bot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 코드 리뷰

PR 자동 리뷰 실행 시점을 synchronize(커밋 푸시)를 제외하고 최초 생성 시점으로 제한하는 변경입니다. GSML 토큰 사용량 감소를 위한 의도적인 변경으로, 코드 변경이 명확하고 일관성 있게 이루어졌습니다. reopenedready_for_review 액션이 여전히 포함되어 있어 PR 재개 시 자동 리뷰가 실행되는 점은 의도된 동작입니다.

지적 사항 없음 — 변경분에서 문제를 찾지 못했다.


모델 darwin-35b-q4_k_m.gguf · 파일 2개 리뷰 · 토큰 2,719 in / 478 out

@Dino0204
Dino0204 merged commit 85332ba into main Aug 19, 2026
4 checks passed
@Dino0204
Dino0204 deleted the fix/pr-auto-review-trigger branch August 19, 2026 07:29
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