build(vite): enable default minification

[s0up4200]

Summary by CodeRabbit

• Chores
  • Simplified build configuration to enable default optimization behaviors for application caching and file minification.

[coderabbitai]

Walkthrough

Vite configuration updated with reordered imports, removal of explicit cache size hardening for VitePWA, and elimination of minify false settings. Default Vite and plugin behaviors now apply; other configurations remain unchanged.

Changes

Cohort / File(s)	Summary
Vite Configuration Cleanup web/vite.config.ts	Import statements reordered; maximumFileSizeToCacheInBytes cache size configuration commented out, defaulting to standard caching behavior; explicit minify: false settings removed, enabling default minification behavior.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

• Configuration changes are minimal and non-breaking; verify that default Vite minification and VitePWA cache behaviors align with project requirements.

Poem

🐰 A tweak to the config, imports rearranged,
Cache limits and minify flags swapped and changed,
Back to defaults we hop, let Vite decide,
Simpler, cleaner—the defaults our guide! 🎉

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: enabling default minification in the Vite configuration by removing the explicit minify: false setting.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch build/enable-default-minification

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Re-enable Workbox cache size override

The new code comments out maximumFileSizeToCacheInBytes, so the PWA falls back to Workbox’s 2 MB default instead of the previous 3 MB override. Large vendor chunks (React, TanStack, polyfills) can still exceed 2 MB even after minification, which means they are no longer precached and the app will miss critical assets when offline. If the intention was to increase the limit to 4 MB as the comment states, the property needs to remain active rather than commented out.

Useful? React with 👍 / 👎.

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

web/vite.config.ts (1)

32-32: Critical: minify: false should be removed to achieve PR objective.

The PR title and AI summary indicate that default minification should be enabled by removing explicit minify: false settings. However, this line still explicitly disables minification for the VitePWA plugin, which contradicts the stated objective.

Apply this diff to enable default minification:

     VitePWA({
       registerType: "autoUpdate",
       injectRegister: null,
-      minify: false,
       devOptions: {
         enabled: false,
       },

🧹 Nitpick comments (1)

web/vite.config.ts (1)

7-7: Import reordering is cosmetic.

The import statements have been reordered without any functional impact.

Also applies to: 10-12

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6e8fdbd and 1afb264.

📒 Files selected for processing (1)

• web/vite.config.ts (2 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Run tests

🔇 Additional comments (1)

web/vite.config.ts (1)

38-38: No action required—the original review comment is incorrect.

The change from a 4MB explicit limit to Workbox's 2MB default is intentional and safe. The project is explicitly engineered to avoid this concern: manual chunks (react-vendor, tanstack, ui-vendor) keep pieces small, and the chunkSizeWarningLimit of 750KB provides a substantial buffer below the 2MB default. This makes the precache limit more restrictive, not less—the opposite of the implied risk in the original comment.

Likely an incorrect or invalid review comment.