Skip to content

Increase bundle size to 6.9MB#16568

Merged
charlesBochet merged 1 commit intomainfrom
increase-bundle-size
Dec 15, 2025
Merged

Increase bundle size to 6.9MB#16568
charlesBochet merged 1 commit intomainfrom
increase-bundle-size

Conversation

@charlesBochet
Copy link
Copy Markdown
Member

Likely a missing lazy load in: #16437

Let's unblock and fix separately

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 1 file

Prompt for AI agents (all 1 issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="packages/twenty-front/vite.config.ts">

<violation number="1" location="packages/twenty-front/vite.config.ts:48">
P2: Comment is out of sync with the actual value. The limit was changed to 6.9MB but the comment still says &quot;6.5MB for main index chunk&quot;.</violation>
</file>

Reply to cubic to teach it or ask questions. Re-run a review with @cubic-dev-ai review this PR

// If it gets too big then find modules in the code base
// that can be loaded lazily, there are more!
const MAIN_CHUNK_SIZE_LIMIT = 6.5 * 1024 * 1024; // 6.5MB for main index chunk
const MAIN_CHUNK_SIZE_LIMIT = 6.9 * 1024 * 1024; // 6.5MB for main index chunk
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot Dec 15, 2025

Choose a reason for hiding this comment

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

P2: Comment is out of sync with the actual value. The limit was changed to 6.9MB but the comment still says "6.5MB for main index chunk".

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/twenty-front/vite.config.ts, line 48:

<comment>Comment is out of sync with the actual value. The limit was changed to 6.9MB but the comment still says &quot;6.5MB for main index chunk&quot;.</comment>

<file context>
@@ -45,7 +45,7 @@ export default defineConfig(({ command, mode }) =&gt; {
   // If it gets too big then find modules in the code base
   // that can be loaded lazily, there are more!
-  const MAIN_CHUNK_SIZE_LIMIT = 6.5 * 1024 * 1024; // 6.5MB for main index chunk
+  const MAIN_CHUNK_SIZE_LIMIT = 6.9 * 1024 * 1024; // 6.5MB for main index chunk
   const OTHER_CHUNK_SIZE_LIMIT = 5 * 1024 * 1024; // 5MB for other chunks
 
</file context>
Suggested change
const MAIN_CHUNK_SIZE_LIMIT = 6.9 * 1024 * 1024; // 6.5MB for main index chunk
const MAIN_CHUNK_SIZE_LIMIT = 6.9 * 1024 * 1024; // 6.9MB for main index chunk
Fix with Cubic

@charlesBochet charlesBochet merged commit 4281a71 into main Dec 15, 2025
61 of 62 checks passed
@charlesBochet charlesBochet deleted the increase-bundle-size branch December 15, 2025 14:25
@twenty-eng-sync
Copy link
Copy Markdown

Hey @charlesBochet! After you've done the QA of your Pull Request, you can mark it as done here. Thank you!

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.

2 participants