Skip to content

fix: npm 보안 의존성 버전 갱신 - #191

Open
happycastle114 wants to merge 1 commit into
mainfrom
security/npm-deps-20261023
Open

happycastle114 wants to merge 1 commit into
mainfrom
security/npm-deps-20261023

Conversation

@happycastle114

Copy link
Copy Markdown

문제와 변경

main의 pnpm 잠금 파일에서 런타임 및 빌드 의존성 보안 경고가 확인됐습니다. pnpm으로 호환 가능한 수정 버전을 재해석하고 잠금 파일을 갱신했습니다. 주요 변경에는 @grpc/grpc-js 1.14.4 → 1.14.6, multer 2.2.0 → 2.4.0, proxy-addr 2.0.7 → 2.0.8, fast-uri 3.1.4 → 3.1.8, vite 6.4.3 → 6.4.5가 포함됩니다. 예제의 Vite 선언과 필요한 pnpm override도 맞췄습니다.

검증

  • corepack pnpm install --frozen-lockfile 통과 (pnpm 11.24.0, Node 24.22.1)
  • corepack pnpm build, corepack pnpm typecheck, corepack pnpm test 통과 (41개 테스트 파일, 799개 테스트)
  • make lint, make format-check, make proto-check 통과
  • cd ts && ../scripts/smoke-test-cli-create.sh 통과
  • corepack pnpm audit --prod --json 런타임 경고 0건
  • 전체 corepack pnpm audit --json은 56건에서 개발 도구의 6건으로 감소

make test를 저장소 루트에서 실행하면 호스트의 Corepack이 pnpm 12를 선택하여 CLI smoke pack 단계가 중단됩니다. 같은 smoke를 ts 디렉터리에서 pnpm 11.24.0으로 실행해 통과했습니다. 이 환경 차이는 PR 자체의 빌드 결과와 구분합니다.

남은 개발 도구 경고는 Vitest 4 이행 시 기존 테스트 타입 검사 실패가 발생하는 항목과 아직 수정 릴리스가 없는 braces입니다. 이 PR에서 테스트 코드 변경은 포함하지 않았습니다. 열려 있는 Dependabot PR #162, #166, #167, #168과 잠금 파일이 겹치므로 병합 순서에 따라 다시 기반을 맞추고 검증해야 합니다. Go 의존성 수정은 별도 PR #190에서 검토할 수 있습니다.

스캔 SHA: de4f5c4000ce7dbf87db7a7f9bc86c925c3e3247
기준 브랜치/SHA: main / de4f5c4000ce7dbf87db7a7f9bc86c925c3e3247

실제 배포 및 소비자의 Node 실행 버전은 확인되지 않았습니다. 이 PR은 배포 완료를 뜻하지 않습니다.

참고: gRPC JS 권고, Multer 권고, proxy-addr 권고, Vite 권고, braces 권고

@coderabbitai

coderabbitai Bot commented Oct 23, 2026

Copy link
Copy Markdown

Warning

Review limit reached

  • Run on-demand review

This review includes 5 billable files and costs up to $1.25.

  • Add your request for automatic reviews

Open in CodeRabbit

Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing.

Or wait 18 minutes for your next included review.

Check out review usage here.

View limit details

Limit details: You’ve used all 5 included reviews currently available. Your 20 included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: e26e5948-8ba8-485f-8439-be8dbc45312a
📥 Commits

Reviewing files that changed from the base of the PR and between de4f5c4 and a7a6890.

⛔ Files ignored due to path filters: 1
  • ts/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing: 5
  • ts/examples/basic/apps/wam/package.json
  • ts/examples/calendar/apps/wam/package.json
  • ts/examples/tutorial/apps/wam/package.json
  • ts/packages/server/package.json
  • ts/pnpm-workspace.yaml
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@channeltalk

channeltalk Bot commented Oct 23, 2026

Copy link
Copy Markdown

This branch has not been deployed

No deployments
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.

1 participant