Repository navigation
chore(deps): @astrojs/react를 7로 올린다 (#207 대체) - #219
Merged
Merged
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
리뷰: @astrojs/react 6 → 7변경은 코드 품질 / 호환성
잠재 이슈
성능
보안
테스트 커버리지
전반적으로 승인해도 좋은 변경입니다. 위의 🤖 Generated with Claude Code |
…act-7 # Conflicts: # package.json # pnpm-lock.yaml
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
요약
React 컴포넌트를 Astro 페이지에 붙여 주는
@astrojs/react를 6.0.6에서 7.0.1로 올립니다(Dependabot #207). 7은 JSX 변환을 Babel에서 Oxc(Rust로 만든 빠른 JS 변환기)로 바꾼 메이저 버전입니다. 이 블로그는 사라진 옵션을 쓰지 않아서 코드 수정 없이 버전만 올렸습니다.#207은 보안 패치(#211, #216) 이전 main 기준이라, 최신 main 위에서
@astrojs/react한 줄만 바꿔 다시 설치했습니다.영향 검토
babel옵션:astro.config.ts가 인자 없는react()를 쓰므로 해당 없음oxc-transform-react: 선택 사항(optional)이고, React Compiler를 켤 때만 필요@babel/*,gensync,browserslist,react-refresh등 Babel 관련 전이 의존성이 빠지고@vitejs/plugin-react6.1.2와@astrojs/internal-helpers0.12.0이 들어옴. 피어 의존성 경고 0건SonnerProvider(전 페이지),Comments(Giscus 댓글),ContinueReadingToast(이어 읽기). 빌드 결과물을astro preview로 띄워 포스트 페이지에서 세 아일랜드가 모두 로드되고 Giscus 위젯과 Sonner 토스트 영역이 마운트되는 것을 확인했고, 콘솔 에러는 없었음. 이어 읽기 토스트는e2e/continue-reading.spec.ts가 검증검증 (로컬, Windows)
pnpm audit --audit-level=high: 통과 (moderate 2건은 main과 같음)astro check: 0 errors,astro build: 성공pnpm test:scripts: 통과listener-leak.spec.ts의/projects왕복 테스트(astro:page-load리스너 수 6 → 7)가 로컬 첫 실행과 이 PR의 CI에서 재시도 끝에 통과했습니다. 다만 React 6 그대로인 chore(deps): Dependabot 안전군 10건을 일괄 업그레이드한다 (#213, #212 통합) #217, #218의 CI에서도 같은 테스트가 같은 방식으로 재시도 끝에 통과했고(#218은 다음 실행에서 재시도 없이 통과), 해당 스펙만 로컬에서 3회 반복(21건)했을 때도 모두 통과했습니다. 그래서 React 7이 만든 회귀가 아니라 브랜치와 무관하게 원래 있던 불안정성으로 판단합니다. 원인은 #221에서 확인했습니다. dev 서버에만 있는 Astro 도구 막대의 audit 앱이 리스너를 늦게 달아 생긴 측정 오차였고, E2E에서 도구 막대를 끄는 것으로 고칩니다.머지 순서
#217(안전군 묶음), #218(satori)과 잠금 파일이 겹치므로 먼저 머지되는 쪽 다음에 최신 main 위에서 잠금 파일을 다시 만들어야 합니다. 머지 후 #207은 이 PR로 대체했다는 코멘트와 함께 닫습니다.