fix: 리뷰 봇 경고 코멘트에 엔진 이름 명시 - #20
Conversation
- 로그인 만료 경고가 "eottabom-review-bot"으로만 찍혀 claude/codex 중 어느 세션이 만료됐는지 구분 불가하던 문제 수정 - eottabom-claude-review-bot / eottabom-codex-review-bot 으로 분리하고 리뷰 본문 헤더도 동일하게 통일
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
📜 Recent review details🔇 Additional comments (4)
📝 Walkthrough리뷰 요약엔진별 봇 식별자를 분리하여 로그인 만료 경고와 리뷰 본문의 출처를 명확하게 개선했습니다. 다만 동일한 문자열이 여러 워크플로에 직접 작성되어 있어 향후 봇 이름 변경 시 불일치가 발생할 수 있습니다. 공통 환경 변수 또는 재사용 가능한 설정으로 관리하면 유지보수성이 향상됩니다. 인라인 리뷰
|
| Layer / File(s) | Summary |
|---|---|
리뷰 본문 식별자 변경 .github/scripts/claude-review-run.sh, .github/scripts/codex-review-run.sh |
리뷰 본문 제목을 각각 eottabom-claude-review-bot과 eottabom-codex-review-bot으로 변경했습니다. |
로그인 만료 알림 식별자 변경 .github/workflows/pr-auto-review.yml, .github/workflows/pr-mention-review.yml |
Claude 및 Codex 로그인 만료 알림에 각각 전용 봇 이름을 적용했습니다. |
Estimated code review effort: 1 (Trivial) | ~3 minutes
Possibly related PRs
- eottabom/sandbox-load-test#19: 이번 변경 대상인 리뷰 스크립트와 워크플로를 추가한 PR입니다.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
| Check name | Status | Explanation |
|---|---|---|
| Title check | ✅ Passed | 제목이 리뷰 봇 경고 코멘트에 엔진 이름을 명시하는 주요 변경 사항을 간결하게 설명합니다. |
| Description check | ✅ Passed | 변경 내용과 테스트 계획을 설명합니다. 다만 저장소 템플릿의 섹션 제목과 관련 이슈 항목은 포함하지 않았습니다. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
| Linked Issues check | ✅ Passed | Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | ✅ Passed | Check skipped because no linked issues were found for this pull request. |
✨ Finishing Touches
📝 Generate docstrings
- Create stacked PR
- Commit on current branch
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Commit unit tests in branch
fix/review-bot-engine-naming
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
Summary
Test plan