Skip to content

개발 의존성 보안 취약점 해소 - #599

Merged
goder-0 merged 1 commit into
mainfrom
feature/dependabot-dev-security
Oct 1, 2026
Merged

goder-0 merged 1 commit into
mainfrom
feature/dependabot-dev-security

Conversation

@goder-0

@goder-0 goder-0 commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

관련 이슈

PR 설명

main에 남아 있던 개발 의존성 보안 경고 5건 중 패치가 배포된 4건을 해소합니다. 운영 의존성 선언은 변경하지 않았으며, pnpm audit --prod는 계속 0건입니다.

변경 사항

  • Vitest와 전이 @vitest/mocker를 3.2.7에서 4.1.11로 업데이트해 파일 경로 검증 취약점을 해소했습니다. 3.x에는 해당 패치가 제공되지 않습니다.
  • 공식 Vitest 4 마이그레이션에 맞춰 브라우저 provider를 @vitest/browser-playwright로 교체하고 projects, instances, playwright() 설정을 적용했습니다.
  • 기존 허용 범위 안에서 잠금 파일의 yaml을 2.8.2 → 2.9.1, @humanfs/node를 0.16.7 → 0.16.8로 갱신했습니다. 새 override는 추가하지 않았습니다.
보안 권고 대상 결과
GHSA-82fw-gwwq-j7x9 Vitest, @vitest/mocker moderate 2건 해소
GHSA-48c2-rrv3-qjmp yaml moderate 1건 해소
GHSA-p498-v437-472g @humanfs/node moderate 1건 해소

남은 경고와 Dependabot 상태

  • 전체 audit: moderate 4 / low 1 → low 1. 운영 audit: 0 → 0.
  • 남은 elliptic 경고는 Storybook의 node-polyfill-webpack-plugin → crypto-browserify 경로에만 존재합니다. 2026-09-18 npm 최신 배포는 6.6.1이며 공식 권고에도 패치가 없습니다. 해결 또는 위험 수용으로 처리하지 않았습니다.
  • 저장소 Dependabot API에서 열린 경고와 종료된 경고가 모두 빈 목록으로 반환됐습니다. 따라서 이번 작업에서 수동으로 종료한 Dependabot 경고는 없습니다. 기존 #590은 의존성 보안 취약점 업데이트 및 audit 정리 #598 병합 후 이미 닫혀 있습니다.

검증 및 제한

  • pnpm 10.19.0 frozen lockfile 설치, Node.js 22 단위 테스트 23개, TypeScript 검사, Next.js 프로덕션 빌드 및 Storybook 정적 빌드 통과.
  • ESLint는 NODE_PATH=$PWD/node_modules/.pnpm/node_modules를 지정해 통과했습니다. 기본 실행의 @next/eslint-plugin-next 탐색 오류는 변경 전 main에서도 동일하게 재현됩니다.
  • Storybook 브라우저 테스트는 기존 .storybook/vitest.setup.ts 누락으로 수집 단계에서 실패합니다. 변경 전 main에서도 같은 오류가 발생하며, 이 PR에서는 Storybook 테스트 환경 재구성을 포함하지 않았습니다.

- Vitest와 브라우저 provider를 4.1.11로 갱신하고 설정을 마이그레이션합니다
- yaml과 humanfs 전이 의존성을 패치해 moderate 취약점 4건을 해소합니다
@coderabbitai

coderabbitai Bot commented Sep 18, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: 4b909b3a-c77c-49eb-bbe6-b5940470c26d

📥 Commits

Reviewing files that changed from the base of the PR and between 32f5533 and 13a8720.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (2)
  • package.json
  • vitest.config.ts

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

변경 사항

영역 변경 내용
의존성 @vitest/browser를 @vitest/browser-playwright ^4.1.11로 교체했습니다. vitest를 ^4.1.11로 업데이트했습니다.
설정 workspace를 projects로 변경했습니다. Storybook 브라우저 테스트에 playwright() 프로바이더와 Chromium 인스턴스 배열을 사용합니다.

Priority: ➖ Normal

Merge Risk: ⚪ Minimal · up to 13a87

No PR-introduced production or development workflow regression is established.

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed 개발 의존성 보안 취약점 해소라는 PR의 주된 변경 목적을 정확히 설명합니다. 제목은 간결하고 관련성이 높습니다.
Description check ✅ Passed 필수 섹션인 관련 이슈와 PR 설명을 모두 포함합니다. 보안 업데이트 내용, 남은 경고, 검증 결과와 제한 사항도 구체적으로 설명합니다.
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 unit tests (beta)
  • Commit to this branch
  • Create a new PR

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@goder-0
goder-0 merged commit ef78a78 into main Oct 1, 2026
3 checks passed

This branch was successfully deployed

1 active deployment
Preview — 13a87201 Deployed Sep 18, 2026 by vercel[bot]
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.

2 participants