Conversation
|
Warning Rate limit exceeded@ujinsim has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 11 minutes and 19 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
워크플로우 개요CI/CD 워크플로우를 통합하여 버전 범프, PR 생성/자동 병합, npm 게시를 단일 워크플로우로 처리하도록 변경. 변경 사항
시퀀스 다이어그램sequenceDiagram
participant main as main 브랜치
participant workflow as Bump & Publish 워크플로우
participant gh as GitHub API
participant npm as npm 레지스트리
main->>workflow: 트리거 (main에 푸시)
workflow->>workflow: package.json 버전 변경 감지
workflow->>workflow: UI 폴더 변경 감지
workflow->>workflow: 버전 범프 및 내보내기 동기화
workflow->>gh: 범프 브랜치 생성 및 푸시
workflow->>gh: PR 생성 (gh pr create)
gh->>workflow: PR 생성 완료
workflow->>gh: PR 자동 병합 (gh pr merge)
gh->>main: 변경 사항 병합
workflow->>npm: npm에 게시 (NODE_AUTH_TOKEN)
npm->>workflow: 게시 완료
예상 코드 리뷰 노력🎯 3 (보통) | ⏱️ ~20-30분
관련 가능성이 있는 PR
제안 라벨
제안 리뷰어
Pre-merge checks and finishing touches❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
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[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
.github/workflows/bump-and-publish.yaml (1)
71-89: 버전 파싱 오류 처리 누락package.json의 버전 문자열을 파싱할 때 형식 검증이 없습니다. 잘못된 형식의 버전(예: "1.2", "invalid")이 있으면 split 결과가 3개가 아니므로 undefined 값이 발생할 수 있습니다.
버전 파싱 전 검증을 추가하세요:
node -e " const fs = require('fs'); const pr_title = process.env.PR_TITLE; const is_new_ui = process.env.NEW_UI_FOLDER === 'true'; const pkg = JSON.parse(fs.readFileSync('package.json', 'utf8')); - const [a,b,c] = pkg.version.split('.').map(Number); + const parts = pkg.version.split('.').map(Number); + if (parts.length !== 3 || parts.some(isNaN)) { + console.error('Invalid version format:', pkg.version); + process.exit(1); + } + const [a,b,c] = parts; let new_version; if(is_new_ui) new_version = [a+1,0,0].join('.'); else if(pr_title.toLowerCase().startsWith('feat')) new_version = [a,b+1,0].join('.'); else new_version = [a,b,c+1].join('.'); pkg.version = new_version; fs.writeFileSync('package.json', JSON.stringify(pkg,null,2)+'\n'); "
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
📒 Files selected for processing (2)
.github/workflows/bump-and-publish.yaml(6 hunks).github/workflows/bump-pr.yaml(0 hunks)
💤 Files with no reviewable changes (1)
- .github/workflows/bump-pr.yaml
🔇 Additional comments (1)
.github/workflows/bump-and-publish.yaml (1)
26-30: Node.js 레지스트리 설정 확인npm 배포를 위해
registry-url을 npmjs.org로 설정했습니다.NODE_AUTH_TOKEN환경 변수가 올바르게 설정되었는지 확인하세요.NPM_TOKEN 시크릿이 GitHub Actions 환경에서 사용 가능한지, 그리고 해당 토큰이 npm 배포 권한이 있는지 확인해주세요.
1 participant
🔥 연관 이슈
🚀 작업 내용
Flow1에서 PR merge 후 브랜치 생성 → Flow2 workflow를 기다리는 구조를 제거
🤔 고민했던 내용
💬 리뷰 중점사항
Summary by CodeRabbit
릴리스 노트