chore: PR 자동화 워크플로에서 담당자 할당은 초기 1회만 동작하도록 변경 #1210
Conversation
📝 WalkthroughWalkthroughGitHub Actions workflow .github/workflows/pull_request_auto_fill.yml updated: the PR title/body automation step now uses actions/github-script@v8, and the auto-assign job using kentaro-m/[email protected] was removed. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Pre-merge checks (3 passed)✅ Passed checks (3 passed)
Poem
✨ Finishing Touches🧪 Generate unit tests
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 |
github-actions
bot
changed the title
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (5)
.github/workflows/pull_request_auto_fill.yml (5)
47-49: 이슈 제목의 첫 토큰 무조건 제거는 과제거 위험 — 이모지/콜론 코드만 보수적으로 제거하세요현재
/^([^\s]+)\s+/는[feat],WIP등 유효 접두어도 제거합니다. 아래처럼 이모지(또는:bug:스타일)만 제거하도록 완화해 주세요.- // 깃모지가 첫 번째 토큰이고 공백 하나가 뒤따른다고 가정 - const cleanTitle = issueTitle.replace(/^([^\s]+)\s+/, ''); + // 선행 이모지(예: "🐛 ") 또는 콜론 코드(예: ":bug: ")만 제거 + const emojiOrColonCode = /^(?:(:[^:\s]+:)|([\p{Emoji_Presentation}\p{Extended_Pictographic}]+))\s+/u; + const cleanTitle = issueTitle.replace(emojiOrColonCode, '');
3-5: 불필요한 재실행(플래핑) 방지: pull_request 이벤트 types를 한정하세요타이틀/바디 자동 수정은 열림/동기화/수정일 때만 필요할 가능성이 높습니다. 이벤트를 한정해 불필요한 실행을 줄이세요.
pull_request: branches: ["develop"] + types: [opened, reopened, synchronize, edited]
89-91: 에러 시 워크플로 성공으로 처리됨 — step 실패로 표시하도록 변경 권장현재는 콘솔 로그만 출력합니다. 실패를 명확히 하기 위해
core.setFailed로 스텝을 실패 처리하세요.- } catch (error) { - console.error(`Error: ${error.message}`); - } + } catch (error) { + console.error(`Error: ${error.message}`); + core.setFailed(error.message); + }
24-26: 브랜치 네이밍 스킴 확장 제안(feat/hotfix/perf 등)실제 관용 접두어(
feat,hotfix,perf,style,build,ci,revert)까지 허용하면 미스매치가 줄어듭니다. 현재 PR 브랜치(chore/1173-...)와도 잘 맞습니다.- const branchPattern = /^(feature|fix|refactor|chore|test|docs)\/(\d+)-.*$/; + const branchPattern = /^(feat|feature|fix|hotfix|refactor|chore|test|docs|perf|style|build|ci|revert)\/(\d+)-.*$/;
69-77: 플레이스홀더가 없을 때도 자동으로 close 문구를 보강하세요작성자가 플레이스홀더를 지우더라도, 중복 없이 말미에
close #<num>를 추가하면 이슈 자동 종결이 안정적입니다.// 플레이스홀더가 있으면 'close #{issue_number}'로 치환 if (prBody.includes('{issue-close-placeholder-do-not-modify}')) { prBody = prBody.replace(/\{issue-close-placeholder-do-not-modify\}/g, `close #${issueNumber}`); console.log('Updated PR body with issue close statement'); } else { - console.log('No issue close placeholder found in PR body'); + // 기존 본문에 close 구문이 없으면 말미에 추가 + const closeRegex = new RegExp(`\\bclose\\s+#${issueNumber}\\b`, 'i'); + if (!closeRegex.test(prBody)) { + prBody = `${prBody}\n\nclose #${issueNumber}`; + console.log('Appended issue close statement to PR body'); + } else { + console.log('Issue close statement already present'); + } }
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
.github/workflows/pull_request_auto_fill.yml(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: build-test
🔇 Additional comments (2)
.github/workflows/pull_request_auto_fill.yml (2)
7-13: Auto-assign 제거 의도 확인요약에 따르면 중복 할당 방지를 위해 auto-assign 잡을 제거했다고 이해했습니다. 팀 내 다른 자동화(예: CODEOWNERS, 저장소 설정의 자동 리뷰어 요청)에 의해 리뷰어가 계속 지정되는지 확인해 주세요.
15-15: actions/github-script v8 핀 고정 및 런타임 검증
- 최신 v8 태그 커밋 SHA ed59741로 핀(uses: actions/github-script@ed59741) 적용 권장
- v8 기본 런타임이 Node20이 아닌 Node24로 변경되었으므로(‘using: node24’) 스크립트 정규식/유니코드 플래그 등 호환성만점검 부탁드립니다.
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
🌱 관련 이슈
📌 작업 내용 및 특이사항
📝 참고사항
📚 기타
Summary by CodeRabbit