Conversation
## major 승격 - eslint 9→10 (+@eslint/js), typescript 5.9→6, eslint-plugin-react-hooks 5→7 - globals 16→17, i18nexus 3→4 (+i18nexus-tools 2→3), lucide-react 0.577→1 - react-ga4 2→3, react-pdf 9→10, puppeteer 24→25, katex 0.16→0.17 - eslint-plugin-react-refresh 0.4→0.5, react/react-dom 19.1→19.2 - CI: actions/checkout v4→v7, actions/setup-node v4→v6 - Tier A minor/보안: vite 6.4(SEC), axios 1.16(SEC), mermaid 11.15(SEC), radix 1.6, react-router 7.18, tailwindcss 4.3, typescript-eslint 8.62 등 ## breaking change 해소 1. TS6: deprecated baseUrl 제거 (paths는 tsconfig 상대 경로라 무영향) 2. eslint10 recommended no-useless-assignment: text-animate.tsx dead-store 제거 3. react-hooks7 신규 컴파일러 rule(refs/set-state-in-effect/purity 등) warn 강등 → 개별 코드 수정은 후속 티켓으로 분리 4. eslint flat config: 강등 블록에 react-hooks 플러그인 등록 + files 스코프 5. react-pdf 10 ↔ pdfjs-dist 5.4.296 정렬 (worker/API 단일 버전 → 불일치 방지) 6. prettier 3.9 union 타입 포맷 적용 7. eslint ignores에 .claude 추가 → npm run lint의 벤더 스킬 코드 오염(5373 errors, pre-existing) 제외해 lint 게이트 복구 (npm run lint GREEN) ## 검증 - npm run build GREEN (vite + prerender 7라우트 + sitemap) - npm run lint GREEN (0 errors, 82 warnings) / prettier --check . GREEN - react-pdf 런타임: pdfjs 단일 5.4.296 트리, API 실동작(3p 파싱) 확인 ## DEFER (스코프 밖) - #286 @vitejs/plugin-react v6 (peer로 vite 8 필요, 미제공) - #292 pdfjs-dist v6 (react-pdf 10이 pdfjs 5 번들 → v6 지원 시 재평가) ## 후속 티켓 후보 - react-hooks7 신규 rule 82 warnings 정리 - tsc 크래시(#pages/* 매핑 유발, pre-existing) 조사 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
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.
major 승격
breaking change 해소
검증
DEFER (스코프 밖)
후속 티켓 후보
ICC-391 — Renovate 대규모 의존성 승격 (2026-07)
처리 대상
오픈 Renovate PR 37건 → 결정:
세이프티 넷 정의 (프론트엔드 각색)
백엔드 스킬의
./gradlew test대신 클라이언트 기준으로 정의:npm run build(vite + prerender 7라우트 + sitemap)npm run lint(eslint .).claudeignore 추가 후 복구eslint srcprettier --check .tsc --noEmit버전 승격 상세
major
no-useless-assignment등). flat config 검증 엄격화baseUrldeprecated. 크래시는 미해결(아래)minor / 보안 (Tier A — semver 범위 내라 lockfile만 반영)
vite 6.4 🔒 · axios 1.16 🔒 · mermaid 11.15 🔒 · radix-ui 1.6 · react-router-dom 7.18 · tailwindcss 4.3 · typescript-eslint 8.62 · framer-motion/motion 12.42 · prettier 3.9 · zustand 5.0.14 · react-toastify 11.1 · tailwind-merge 3.6 · shadcn 4.13 · @uiw/react-md-editor 4.1.1
계단식 breaking change 로그 (재사용 자산)
@vitejs/plugin-react@6peer =vite ^8(vite 8 미제공)react-pdf@10이pdfjs-dist@5.4.296exact 번들 → top-level v6이면 worker/API 불일치baseUrldeprecated (TS5101)baseUrl제거 (paths는 tsconfig 상대라 무영향)refs/set-state-in-effect/purity/static-components)warn강등 + 후속 티켓no-useless-assignment신규 error (실 dead-store)react-hooks플러그인 미등록 → flat config 검증 실패 (.cjs등에서)files스코프 + 플러그인 등록--writenpm run lint이.claude/skills/seo벤더 코드 5373 errors 린트 (pre-existing).claude추가계단식 8회지만 모두 수렴(BUILD SUCCESSFUL 도달) → 스코프 초과 아님.
react-pdf ↔ pdfjs 커플링 (핵심 리스크, 해소됨)
pdfjs-dist/build/pdf.worker.min.mjs에서, API(pdfjs)는 react-pdf 번들에서 로드. 두 버전이 다르면 런타임"API version does not match Worker version"에러.5.4.296로 pin. 트리에 pdfjs 단일 버전(중복 0) → 불일치 구조적 배제. #292는 DEFER.find node_modules -name pdfjs-dist→ 단일 5.4.296. ② node로 3페이지 PDF 파싱·텍스트 추출 성공. ③ 브라우저 로드·업로드 트리거 시 불일치 에러 없음(콘솔 에러는 백엔드 미기동 Network Error뿐).pre-existing 이슈 (승격 무관, 미수정)
resolveExternalModuleDebug Failure. TS 5.9.3부터 발생, TS6에서도 지속.#pages/*→./src/pages/*/index.tsx(와일드카드→특정 파일) 매핑이 유발하는 TS 컴파일러 버그로 추정. typecheck 스크립트 부재로 실제 개발엔 영향 없었음. → 후속 티켓.→ 이번 커밋 #8에서 해소..claudelint 오염검증 요약
prerender가 headless Chrome에서 전체 앱을 렌더 → lucide 1 · radix 1.6 · react-router 7.18 · i18nexus 4 · react 19.2 · puppeteer 25 런타임 검증됨.
남은 후속 티켓 후보
refs/set-state-in-effect/purity등. 일부 실버그 가능, 개별 검토 필요.#pages/*매핑 재설계 or TS 이슈 트래킹.Phase 5 — Renovate 운영 주의
소요
Phase 0(조사) → Phase 1(넷) → Phase 2(스파이크·계단식 fix) → Phase 3/4(통합·단일 커밋) → react-pdf 런타임 검증.
📢 설명
변경 1: [제목]
[변경에 대한 설명]
의사 선택과정 (trade-off)
얻은 것
잃은 것
코드 설명: 인라인 코멘트 확인
✅ 체크 리스트
재현 확인
기본