Skip to content

chore(deps): Dependabot 안전군 10건을 일괄 업그레이드한다 (#213, #212 통합) - #217

Merged
CaesiumY merged 1 commit into
mainfrom
chore/deps-dependabot-safe-batch
Oct 11, 2026
Merged

CaesiumY merged 1 commit into
mainfrom
chore/deps-dependabot-safe-batch

Conversation

@CaesiumY

Copy link
Copy Markdown
Owner

요약

Dependabot이 따로 연 소규모 업데이트 PR 두 개(#213, #212)를 하나로 합쳤습니다. 두 PR은 서로 다른 시점의 main을 기준으로 만들어져 있었고, #213은 보안 패치(#211, #216) 이전 main 기준이라 그대로 머지하면 잠금 파일이 충돌하거나 패치가 되돌아갈 수 있습니다. 그래서 최신 main 위에서 버전 범위만 올리고 node_modules와 pnpm-lock.yaml을 지운 뒤 잠금 파일을 새로 만들었습니다.

바뀐 버전

패키지 이전 범위 실제 설치
astro 7.3.3 ^7.3.7 7.3.8
@astrojs/markdown-remark 7.3.1 ^7.3.2 7.3.2
@playwright/test 1.63.0 ^1.64.0 1.64.0
@shikijs/transformers 4.4.3 ^4.5.0 4.5.0
@typescript-eslint/parser 8.70.0 ^8.71.1 8.71.1
typescript-eslint 8.70.0 ^8.71.1 8.71.1
eslint 10.10.0 ^10.12.0 10.12.0
globals 17.12.0 ^17.13.0 17.13.0
prettier 3.9.8 ^3.9.9 3.9.10
prettier-plugin-astro 1.0.1 ^1.1.0 1.1.1

astro, prettier, prettier-plugin-astro는 Dependabot 목표보다 한 단계 높은 패치로 설치됐습니다. 범위 안의 최신 패치를 고른 것이라 정상입니다. 잠금 파일 diff가 큰 이유는 전이 의존성(직접 설치하지 않았지만 따라 들어오는 패키지)까지 다시 해소했기 때문입니다.

검증 (로컬, Windows)

  • pnpm install: 피어 의존성 경고 0건
  • pnpm audit --audit-level=high: 통과 (moderate 2건(fflate, postcss-selector-parser)은 main과 같음)
  • astro check: 0 errors, 0 warnings
  • astro build + pagefind: 195페이지 빌드 성공
  • ESLint(추적 파일 전체): 통과
  • Prettier check(추적 파일 전체, 줄바꿈 차이 무시): 새 prettier 3.9.10과 plugin-astro 1.1.1 기준으로도 재포맷할 파일 없음
  • pnpm test:scripts: 통과
  • E2E 전체(76개): 모두 통과

머지 후 할 일

머지되면 #213, #212를 이 PR로 통합했다는 코멘트와 함께 닫습니다. satori(#208)와 @astrojs/react(#207) 메이저 업데이트는 별도 PR로 올립니다.

최신 main 위에서 버전 범위만 올리고 node_modules와 잠금 파일을 지운 뒤 다시 해소했다. #213은 보안 패치(#211, #216) 이전 main 기준이라 그대로 머지하면 패치가 되돌아갈 수 있었다.
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@claude

claude Bot commented Oct 11, 2026

Copy link
Copy Markdown

리뷰 요약

의존성 일괄 업그레이드 PR로, 변경은 package.json(범위 10건)과 pnpm-lock.yaml뿐입니다. 소스 코드 변경은 없습니다. 병합해도 괜찮아 보입니다.

잘 된 점

확인한 사항 (diff 기준)

  • 보안: 새로 추가되는 직접 의존성은 없습니다. 전이 의존성은 @astrojs/compiler-binding* 0.5.1, @astrojs/language-server 2.17.2 등이 갱신됩니다. 본문에 적힌 pnpm audit --audit-level=high 통과와 일치합니다.
  • 성능: esbuild가 @astrojs/react, vite의 peer 목록에서 빠지고 vite가 8.3.0 → 8.3.4로 올라갑니다. 빌드 성능에 문제가 될 만한 변화는 보이지 않습니다.
  • 스타일 규칙(AGENTS.md): ESLint 규칙 비활성화나 파일 제외가 없습니다. Conventional Commits 형식의 제목도 지켜졌습니다.

제안 (블로킹 아님)

  1. 최소 범위 정리: astro ^7.3.7처럼 Dependabot 목표를 그대로 쓰면 잠금 파일은 7.3.8로 해소됩니다. 지금처럼 범위 하한을 목표 버전에 맞추는 방식으로 충분합니다.
  2. Windows 로컬 검증: 검증이 Windows에서 수행되었습니다. 이 레포의 병합 게이트(Code standards & build)는 Linux CI에서 돌아가므로, 특히 @astrojs/compiler-binding-linux-* 0.5.1 네이티브 바인딩과 astro check가 CI에서 통과하는지 꼭 확인해 주세요.
  3. 테스트 커버리지: 이 PR은 로직 변경이 없어 새 테스트가 필요하지 않습니다. 기존 E2E 76개와 build 검증(195페이지)이 회귀 방어선입니다. CI 범위 규칙상 package.json 변경은 전체 E2E를 실행하므로 적절합니다.
  4. 머지 후 후속 작업: 본문에 적힌 대로 chore(deps-dev): bump the dev-tools group across 1 directory with 8 updates #213, #212를 닫는 것을 잊지 않도록 해 주세요. 닫지 않으면 Dependabot이 리베이스를 계속 시도할 수 있습니다.

CI가 모두 초록이면 승인해도 좋겠습니다.

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