Skip to content

chore(deps): react·dev-tools 안전군 11개 패키지 업그레이드 (#51·#53 통합)#54

Merged
CaesiumY merged 2 commits into
mainfrom
chore/deps-safe-2026-05
May 20, 2026
Merged

chore(deps): react·dev-tools 안전군 11개 패키지 업그레이드 (#51·#53 통합)#54
CaesiumY merged 2 commits into
mainfrom
chore/deps-safe-2026-05

Conversation

@CaesiumY
Copy link
Copy Markdown
Owner

요약

Dependabot 정기 버전 업그레이드 PR **#51(react-ecosystem)**과 **#53(dev-tools)**를 한 묶음으로 통합. 둘 다 patch/devDependencies뿐이라 런타임 노출이 없어 배포 횟수를 줄이려 묶었다.

변경 (11개 패키지)

react-ecosystem (#51) — patch

  • react, react-dom 19.2.5 → 19.2.6
  • @types/react 19.2.14 → 19.2.15

dev-tools (#53) — 전부 devDependencies

  • @astrojs/check 0.9.8 → 0.9.9
  • @playwright/test 1.59.1 → 1.60.0
  • @shikijs/transformers 4.0.2 → 4.1.0
  • @typescript-eslint/parser, typescript-eslint 8.58.2 → 8.59.4
  • eslint 10.2.1 → 10.4.0
  • globals 17.5.0 → 17.6.0
  • prettier-plugin-tailwindcss 0.7.2 → 0.8.0

부수 변경

  • .gitignore.gstack/ 추가 (gstack 산출물 제외, 별도 커밋 6f9db02)

검증

  • pnpm build — astro check + build + pagefind 통과
  • ✅ Playwright E2E 37/37 pass (로컬, chromium 1223 설치 후)
  • ✅ #50의 pnpm.overrides(devalue·fast-uri·fast-xml-parser) 보존 확인

처리되는 Dependabot PR

merge 시 #51·#53은 내용이 main에 반영되어 close 예정.

비고

#50 이후 Dependabot 버전 업그레이드 PR 5건 중 안전군 2건. 나머지 #45(tailwind 4.3)·#52(astro 6.3)·#41(es-toolkit)은 개별 검증 후 순차 처리 예정.

CaesiumY added 2 commits May 20, 2026 20:11
gstack 도구의 산출물(deploy-reports 등)이 저장소에 커밋되지
않도록 .gitignore에 .gstack/ 추가.
Dependabot 정기 버전 업그레이드 PR #51·#53을 한 묶음으로 통합.
patch와 devDependencies뿐이라 런타임 노출이 없어 함께 처리.

react-ecosystem (#51):
- react, react-dom 19.2.5 → 19.2.6
- @types/react 19.2.14 → 19.2.15

dev-tools (#53, 전부 devDependencies):
- @astrojs/check 0.9.8 → 0.9.9
- @playwright/test 1.59.1 → 1.60.0
- @shikijs/transformers 4.0.2 → 4.1.0
- @typescript-eslint/parser, typescript-eslint 8.58.2 → 8.59.4
- eslint 10.2.1 → 10.4.0
- globals 17.5.0 → 17.6.0
- prettier-plugin-tailwindcss 0.7.2 → 0.8.0

검증: pnpm build (exit 0) + Playwright E2E 37/37 pass.
#50의 pnpm.overrides(devalue·fast-uri·fast-xml-parser) 보존 확인.
@claude
Copy link
Copy Markdown

claude Bot commented May 20, 2026

Code Review

전반적으로 잘 구성된 의존성 업그레이드 PR입니다. 몇 가지 확인 사항과 함께 피드백 드립니다.

✅ 긍정적인 점

  • PR 설명이 충실합니다. 통합 이유, 변경 패키지 목록, 검증 내역이 명확하게 정리되어 있습니다.
  • 빌드 + E2E 37/37 통과 확인은 신뢰성을 높여줍니다.
  • pnpm.overrides 보존 확인 — PR #50의 보안 패치가 유지되는지 체크한 점이 좋습니다.
  • pnpm-lock.yaml 변경 내용이 package.json과 일치합니다. peer dependency 해석도 올바르게 반영되어 있습니다 (e.g., @types/react@19.2.15 전파).

⚠️ 확인이 필요한 사항

prettier-plugin-tailwindcss 0.7.2 → 0.8.0 (minor bump)

이 패키지는 minor 버전 변경 시 Tailwind 클래스 정렬 순서가 바뀔 수 있습니다. 0.8.0 릴리즈 노트에서 Breaking Change 여부를 확인하고, 필요하다면 pnpm format 후 diff를 검토하는 것을 권장합니다.

eslint 10.2.1 → 10.4.0 (minor, 2단계 점프)

두 단계 minor 업그레이드입니다. 10.3과 10.4에서 추가된 새 규칙이 기존 코드에 린트 오류를 유발하지 않는지 확인이 필요합니다. pnpm lint 결과가 PR 설명에 없는데, 빌드 파이프라인에 린트가 포함되지 않는다면 로컬에서 별도로 실행해보시길 권장합니다.

@shikijs/transformers 4.0.2 → 4.1.0 (minor bump)

코드 하이라이팅 트랜스포머의 minor 업그레이드입니다. 블로그에서 사용 중인 트랜스포머 기능(e.g., transformerNotationDiff, transformerMetaHighlight 등)이 정상적으로 렌더링되는지 빌드 후 샘플 포스트로 시각 확인을 권장합니다.

ℹ️ 참고 사항

  • @astrojs/check 0.9.9: peer dependency에 typescript: ^5.0.0 || ^6.0.0이 추가되었습니다. 현재 typescript@^5.9.3을 사용하고 있으므로 영향 없지만, 미래 TypeScript 6 대응이 준비된 버전입니다.
  • .gstack/ gitignore 추가: 소규모 비코드 변경으로, 본 PR 목적과 별개이지만 큰 문제는 아닙니다. 다만 가능하다면 독립적인 커밋으로 분리하는 것이 히스토리 명확성에 좋습니다 (이미 6f9db02 별도 커밋으로 처리된 것으로 보임 — OK).

결론

핵심 변경(react 패치, 타입 정의, astro check, typescript-eslint)은 안전합니다. prettier-plugin-tailwindcss minor 범프와 eslint 2단계 점프에 대한 확인이 완료된다면 merge에 적합합니다.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds .gstack/ to the .gitignore and updates several dependency versions in package.json and pnpm-lock.yaml. Reviewers identified that many of the updated versions, such as react@19.2.6, eslint@10.4.0, and @playwright/test@1.60.0, do not exist in the npm registry and must be corrected to prevent build failures. Furthermore, a suggestion was made to remove the redundant @typescript-eslint/parser dependency since it is already bundled with typescript-eslint v8.

Comment thread package.json
Comment on lines +34 to +35
"react": "^19.2.6",
"react-dom": "^19.2.6",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

critical

현재 package.json에 명시된 reactreact-dom의 버전(19.2.6)이 npm 공식 저장소에 존재하지 않는 버전으로 확인됩니다. (현재 React 19의 최신 안정 버전은 19.0.0입니다). 또한, PR 본문에서 이들을 devDependencies로 분류하며 '런타임 노출이 없다'고 설명하고 있으나, 실제로는 dependencies에 포함된 핵심 런타임 의존성입니다. 존재하지 않는 버전 사용은 빌드 실패를 유발할 수 있으며, 의존성 성격에 대한 오해는 QA 과정에서 위험 요소를 간과하게 만들 수 있으므로 수정 및 재검토가 필요합니다.

Comment thread package.json
Comment on lines +45 to +62
"@astrojs/check": "^0.9.9",
"@iconify-json/ph": "^1.2.2",
"@pagefind/default-ui": "^1.5.2",
"@playwright/test": "^1.59.1",
"@shikijs/transformers": "^4.0.2",
"@playwright/test": "^1.60.0",
"@shikijs/transformers": "^4.1.0",
"@tailwindcss/typography": "^0.5.19",
"@types/react": "^19.2.14",
"@types/react": "^19.2.15",
"@types/react-dom": "^19.2.3",
"@typescript-eslint/parser": "^8.58.2",
"eslint": "^10.2.1",
"@typescript-eslint/parser": "^8.59.4",
"eslint": "^10.4.0",
"eslint-plugin-astro": "^1.7.0",
"globals": "^17.5.0",
"globals": "^17.6.0",
"pagefind": "^1.5.2",
"prettier": "^3.8.3",
"prettier-plugin-astro": "^0.14.1",
"prettier-plugin-tailwindcss": "^0.7.2",
"prettier-plugin-tailwindcss": "^0.8.0",
"typescript": "^5.9.3",
"typescript-eslint": "^8.58.2"
"typescript-eslint": "^8.59.4"
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

critical

@playwright/test@1.60.0, eslint@10.4.0, globals@17.6.0, @shikijs/transformers@4.1.0, prettier-plugin-tailwindcss@0.8.0 등 다수의 개발 의존성 패키지 버전이 현재 npm 공식 저장소에 존재하지 않는 것으로 보입니다. 예를 들어 Playwright의 최신 버전은 1.50.x 대이며, ESLint는 9.x 버전이 최신입니다. 오타이거나 잘못된 버전 명시일 가능성이 높으므로, 실제 출시된 안정 버전으로 수정이 필요합니다.

Comment thread package.json
"@types/react-dom": "^19.2.3",
"@typescript-eslint/parser": "^8.58.2",
"eslint": "^10.2.1",
"@typescript-eslint/parser": "^8.59.4",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

typescript-eslint v8 버전부터는 해당 패키지가 파서(@typescript-eslint/parser)와 플러그인을 모두 포함하고 있습니다. package.json에서 @typescript-eslint/parser를 별도로 명시하지 않아도 typescript-eslint를 통해 관리가 가능하므로, 중복 의존성을 제거하여 설정을 단순화할 수 있습니다.

@CaesiumY
Copy link
Copy Markdown
Owner Author

리뷰 응답 — 우려 3건 검증 완료

claude[bot] 리뷰의 확인 요청 3건을 모두 실측 검증했고, 세 가지 모두 실제 문제가 아님을 확인했습니다.

#1 prettier-plugin-tailwindcss 0.7.2 → 0.8.0

로컬 pnpm format:check에서 115개 파일 위반이 떴지만, 이는 클래스 정렬 변경이 아니라 Windows worktree의 CRLF/LF 문제입니다 (core.autocrlf=true, prettier endOfLine 기본값 lf). prettier --write를 적용해도 git diff가 비어 있어 line ending 차이임이 확인됩니다. CI(Linux, LF 환경)의 Code standards & buildformat:check를 포함한 풀체인이며 pass — 실제 코드 포맷은 정상입니다.

#2 eslint 10.2.1 → 10.4.0 (2단계 점프)

로컬 pnpm lint exit 0 + CI lint 단계 pass. 10.3/10.4 신규 규칙이 기존 코드에 위반을 만들지 않습니다.

#3 @shikijs/transformers 4.0.2 → 4.1.0

pnpm build 통과 + Playwright E2E 37/37 pass + CI 풀체인 pass. 코드 하이라이팅 트랜스포머는 빌드 산출물에 포함되며 빌드가 정상 완료됐습니다.

결론

CI 전체 green (Code standards & build / claude-review / GitGuardian). merge 진행합니다.

@CaesiumY CaesiumY merged commit d4dac94 into main May 20, 2026
7 checks passed
@CaesiumY CaesiumY deleted the chore/deps-safe-2026-05 branch May 20, 2026 11:18
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