Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ite plugin - Upgrade vite 7.3.1 → 8.0.1, @sveltejs/vite-plugin-svelte 6 → 7, vitest/@vitest/* 4.0.18 → 4.1.0 in one atomic commit - Migrate @tailwindcss/postcss → @tailwindcss/vite to fix Rolldown CSS resolution error (ENOENT tailwindcss) - Note: client bundle gzip +75 kB (+22%) due to Rolldown chunk splitting; expected to improve as Rolldown matures Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
📝 WalkthroughWalkthroughVite 7.3.1 から 8.0.1 へメジャーアップグレードし、Svelte plugin、Vitest 関連パッケージを更新。Tailwind CSS を PostCSS プラグインから Vite プラグインに移行。設計計画を文書化。 Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/dev-notes/2026-03-23/vite-upgrade/plan.md`:
- Around line 1-170: The doc has minor Markdown lint issues: add a language
specifier (e.g., "text" or "plaintext") to the fenced code blocks used in the
"dev/prod の分裂問題" example and the error block in "Phase 7 補足", and insert a blank
line immediately before the table headed "変更内容:" so the table is recognized;
locate and update the fenced blocks and the table in the Vite 7 → 8 upgrade plan
document (sections "dev/prod の分裂問題" and "Phase 7 補足" and the "変更内容:" table) to
apply these trivial formatting fixes.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 63f07cb4-1c68-442d-934c-d85d2e533d54
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml,!pnpm-lock.yaml
📒 Files selected for processing (4)
docs/dev-notes/2026-03-23/vite-upgrade/plan.mdpackage.jsonpostcss.config.mjsvite.config.ts
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.
close #3305
アップグレード時の調査内容を備忘録として残しておく
理由: plan.md は参照されない可能性が高いので、PR とセットとすることで後から検索しやすくするため
背景:なぜ Rolldown/Oxc に移行するのか
dev/prod の分裂問題
Vite 7 以前は dev と prod で異なるバンドラーを使っていた:
2 つの異なるバンドラーが同居していたため「dev では動くが prod でクラッシュ」という再現困難なバグが構造的に発生していた。Vite 8 はこれを Rolldown で一本化することで解消する。
VoidZero エコシステム
Rolldown と Oxc はどちらも VoidZero(Evan You が設立)が開発している Rust 製ツールチェーン:
oxlint(ESLint 互換リンター)や oxfmt(フォーマッター)も同じ Oxc モノレポに属し、同一のパーサーと AST を共有している。Vite 8 に乗ることは「Oxc エコシステムの土台に立つ」ことを意味し、各ツールは独立して・好きなタイミングで追加移行できる。
エコシステム一元化のメリット・デメリット
メリット
デメリット・リスク
注意事項:バンドルサイズの増加
Vite 7 → 8 のアップグレードにより、クライアントバンドルのサイズが増加している。
原因: Rolldown の chunk splitting アルゴリズムが Rollup と異なるため。Vite 8.x の成熟とともに改善が期待される。
判断: 許容する。Rolldown が安定するにつれ自然に解消される可能性が高く、dev/prod 動作一致などのメリットが上回る。