You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
CI·배포에 Node 22.12.0이 개별 선언되어 있고 Node 타입은 25 계열이어서 실행 환경과 타입의 기준이 달랐음. 공용 버전 파일로 CI·배포의 Node를 24.21.0에 맞추고 로컬 실행 범위와 Node 타입을 24 계열로 정리함.
.node-version에 24.21.0을 기록함. CI·개발·미리보기·생산·Storybook workflow 5개가 이 파일을 읽도록 변경함.
버전 파일을 읽기 전에 checkout을 수행하도록 순서를 맞춤. 기존 checkout 옵션, npm ci, 캐시와 배포 조건은 유지함.
engines.node는 >=24.12.0 <25로 선언함. Linux 선택 의존성 @napi-rs/lzma-linux-x64-gnu의 ^24.12 요구 조건을 반영한 하한임. CI·배포는 검증한 패치를 사용하고 로컬은 요구 범위를 만족하는 24 계열을 허용함.
@types/node는 ^24.19.2로 선언하고 실제 설치 버전 24.19.2를 lockfile에 기록함. 그 외 직접 의존성과 기존 버전 제한·override는 유지함. npm 버전은 명시하거나 강제하지 않음.
문서 지도는 구체적 버전을 중복 작성하지 않고 설정 파일을 참조하도록 정리함.
환경·타입·workflow 설정과 문서 변경을 두 커밋으로 나눴음. 제품 동작·API 변경과 React·Vite·Storybook 갱신은 포함하지 않음.
검증
Windows의 Node 24.21.0·npm 11.19.1에서 확인했음.
PR 준비 시 전체 테스트 59개 파일·546개 항목, lint·타입 검사, 앱 생산 빌드가 통과했음.
앞선 구현 검증에서 최종 lockfile의 npm ci --engine-strict --strict-peer-deps, npm ls --depth=0, npm run build-storybook가 통과했음. 이후 의존성·workflow 변경은 없었음.
앱·Storybook 개발 서버의 HTML·진입 모듈·CSS·MSW worker 응답을 확인했음. Storybook 색인 170개 항목과 Storybook 빌드 이후 앱 서버 재시작·응답도 확인했음. HTTP 확인은 화면 조작 검증을 대체하지 않음.
Node 실행 범위와 lockfile, workflow YAML의 순서·기존 옵션 보존, 문서 상대 링크와 Git diff를 확인했음.
선행 코드 리뷰에서 상·중·하·컨벤션 위반으로 분류할 새 발견 사항은 없었음.
설치 시 변경하지 않은 glob·uuid·ESLint의 비권장 버전 안내, npm 감사 결과 10건(낮음 1·보통 3·높음 6), Sentry CLI·esbuild·MSW 설치 스크립트 허용 안내를 관측했음. 자동 수정이나 허용 설정 추가는 수행하지 않았음.
앱·Storybook 묶음 크기, MDX 파일 부재, Storybook 내부 eval 경고도 남아 있음. DEP0190은 기존 Storybook 8.6.18의 패키지 관리자 탐지 코드에서 shell: true와 인자를 함께 전달하는 호출로 추적했으며 후속 Storybook 업데이트에서 다시 확인함. 이전 Node 런타임과 경고를 직접 비교하지는 않았음.
npm ci --engine-strict --strict-peer-deps
npm ls --depth=0
npm run build
npm run build-storybook
npm run dev
npm run storybook
Linux 실제 설치와 원격 CI 결과를 확인함. Storybook 빌드 후 앱 서버를 다시 실행해 산출물이 의존성 탐색에 섞이지 않는지 확인함.
브라우저에서 타이머·청중 공유·투표·모달·입력·포커스·반응형 화면을 직접 확인하는 검증, 실제 API·소켓 서버 연결, Chromatic·인증을 사용하는 Sentry 업로드는 미실행이므로 추가 확인함.
Summary by CodeRabbit
업데이트
프로젝트의 Node.js 실행 환경이 24.x로 변경되었습니다.
로컬 실행에 필요한 Node.js 버전 범위와 CI·배포 환경의 버전 설정 기준이 문서에 추가되었습니다.
No actionable comments were generated in the recent review. 🎉
ℹ️ Recent review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Advanced
Run ID: 3a359a90-4f6c-4fef-9950-45c912a06ef1
📥 Commits
Reviewing files that changed from the base of the PR and between dd7f668 and 1caabd5.
⛔ Files ignored due to path filters (1)
package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (8)
.github/workflows/CI.yml
.github/workflows/deploy-development.yml
.github/workflows/deploy-preview.yml
.github/workflows/deploy-production.yml
.github/workflows/deploy-storybook.yml
.node-version
docs/README.md
package.json
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.
Walkthrough
Node 버전 기준을 .node-version의 24.21.0으로 설정하고, package.json에 Node 엔진 범위와 @types/node 버전을 지정했습니다. CI 및 배포 워크플로가 공용 버전 파일을 사용하도록 변경하고 README를 갱신했습니다.
Changes
Node 실행 환경
Layer / File(s)
Summary
Node 버전 및 패키지 범위 .node-version, package.json
.node-version에 24.21.0을 지정했습니다. package.json에 Node 엔진 범위 >=24.12.0 <25를 추가하고 @types/node를 ^24.19.2로 변경했습니다.
워크플로 설정 및 문서 .github/workflows/*, docs/README.md
CI 및 네 배포 워크플로가 .node-version에서 Node 버전을 읽도록 변경했습니다. 배포 워크플로에서는 체크아웃 단계를 Node 설정 단계보다 먼저 실행합니다. README에 실행 환경 설정 및 워크플로 시점을 설명하는 내용을 추가했습니다.
Node 24 configuration and install metadata are consistent, and the checked workflow reads the shared version file after checkout. No concrete merge blocker is identified; remote Linux CI remains routine validation.
Pre-merge checks · 4 · 1
Failed checks (1 inconclusive)
Check name
Status
Explanation
Resolution
Linked Issues check
직접 연결된 #523의 요구사항 대부분은 요약으로 확인됩니다. .node-version의 24.21.0, 다섯 workflow의 checkout 후 공용 파일 사용, engines.node의 >=24.12.0 <25, @types/node의 ^24.19.2, 기존 workflow 옵션 유지, 문서 참조 변경이 포함되어 있습니다. 전체 …
package-lock.json의 @types/node 24.19.2 기록과 관련 의존성 상태를 검토 가능한 증거로 제공하고, npm ci --engine-strict --strict-peer-deps 결과가 해당 lockfile과 일치하는지 확인해야 합니다.
Passed checks (4 passed)
Check name
Status
Explanation
Description Check
Check skipped - CodeRabbit’s high-level summary is enabled.
Title check
제목은 CI·배포·로컬 실행 환경의 Node.js 버전을 Node 24 계열로 통일하는 주요 변경 사항을 정확히 요약합니다.
Out of Scope Changes check
변경은 #523의 범위에 해당합니다. Node 버전 파일, 다섯 workflow, package.json, 문서와 관련 검증만 변경했습니다. 제품 동작·API 변경, React·Vite·Storybook 갱신, 기존 의존성 제한·override 해제와 같은 별도 작업은 포함되지 않았습니다.
Docstring Coverage
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Full details: Linked Issues check
Explanation
직접 연결된 #523의 요구사항 대부분은 요약으로 확인됩니다. .node-version의 24.21.0, 다섯 workflow의 checkout 후 공용 파일 사용, engines.node의 >=24.12.0 <25, @types/node의 ^24.19.2, 기존 workflow 옵션 유지, 문서 참조 변경이 포함되어 있습니다. 전체 테스트, lint·타입 검사, 앱·Storybook 빌드와 개발 서버 확인도 기록되어 있습니다. 그러나 package-lock.json은 검토에서 제외되었고, #523은@types/node 24.19.2의 lockfile 기록과 엄격한 설치를 요구합니다. 해당 lockfile 상태를 독립적인 검토 가능 증거로 확인할 수 없습니다.
✨ Finishing Touches
🧪 Generate unit tests (beta)
Commit to this branch
Create a new PR
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
chore파일 내부가 아닌 파일 자체에 대한 변경 (디렉토리 이동, 파일 이름 변경, 파일 삭제 등)
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🚩 연관 이슈
Closes #523
📝 작업 내용
CI·배포에 Node 22.12.0이 개별 선언되어 있고 Node 타입은 25 계열이어서 실행 환경과 타입의 기준이 달랐음. 공용 버전 파일로 CI·배포의 Node를 24.21.0에 맞추고 로컬 실행 범위와 Node 타입을 24 계열로 정리함.
.node-version에24.21.0을 기록함. CI·개발·미리보기·생산·Storybook workflow 5개가 이 파일을 읽도록 변경함.npm ci, 캐시와 배포 조건은 유지함.engines.node는>=24.12.0 <25로 선언함. Linux 선택 의존성@napi-rs/lzma-linux-x64-gnu의^24.12요구 조건을 반영한 하한임. CI·배포는 검증한 패치를 사용하고 로컬은 요구 범위를 만족하는 24 계열을 허용함.@types/node는^24.19.2로 선언하고 실제 설치 버전 24.19.2를 lockfile에 기록함. 그 외 직접 의존성과 기존 버전 제한·override는 유지함. npm 버전은 명시하거나 강제하지 않음.환경·타입·workflow 설정과 문서 변경을 두 커밋으로 나눴음. 제품 동작·API 변경과 React·Vite·Storybook 갱신은 포함하지 않음.
검증
Windows의 Node 24.21.0·npm 11.19.1에서 확인했음.
npm ci --engine-strict --strict-peer-deps,npm ls --depth=0,npm run build-storybook가 통과했음. 이후 의존성·workflow 변경은 없었음.설치 시 변경하지 않은 glob·uuid·ESLint의 비권장 버전 안내, npm 감사 결과 10건(낮음 1·보통 3·높음 6), Sentry CLI·esbuild·MSW 설치 스크립트 허용 안내를 관측했음. 자동 수정이나 허용 설정 추가는 수행하지 않았음.
앱·Storybook 묶음 크기, MDX 파일 부재, Storybook 내부 eval 경고도 남아 있음. DEP0190은 기존 Storybook 8.6.18의 패키지 관리자 탐지 코드에서
shell: true와 인자를 함께 전달하는 호출로 추적했으며 후속 Storybook 업데이트에서 다시 확인함. 이전 Node 런타임과 경고를 직접 비교하지는 않았음.🗣️ 리뷰 요구사항 (선택)
Linux 실제 설치와 원격 CI 결과를 확인함. Storybook 빌드 후 앱 서버를 다시 실행해 산출물이 의존성 탐색에 섞이지 않는지 확인함.
브라우저에서 타이머·청중 공유·투표·모달·입력·포커스·반응형 화면을 직접 확인하는 검증, 실제 API·소켓 서버 연결, Chromatic·인증을 사용하는 Sentry 업로드는 미실행이므로 추가 확인함.
Summary by CodeRabbit