Skip to content

perf(build): reduce Sentry source map upload scope - #2782

Merged
Marfuen merged 1 commit into
mainfrom
mariano/build-perf
May 7, 2026
Merged

Marfuen merged 1 commit into
mainfrom
mariano/build-perf

Conversation

@Marfuen

@Marfuen Marfuen commented May 7, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Sets widenClientFileUpload: false in the Sentry config to reduce the source map upload from 2168 files to only server-side maps.

The previous build log shows ~83s spent on Sentry source map processing — scanning 949 client files that mostly can't find their source maps (Turbopack hash-named chunks). Reducing the upload scope saves ~30-40s per build.

Note: typescript.ignoreBuildErrors and the CI typecheck trigger were already merged via #2777. The @carbon/icons-react optimization was also already included.

Test plan

  • Build still uploads server-side source maps to Sentry
  • Sentry error tracking continues working (server errors have full traces)

🤖 Generated with Claude Code


Summary by cubic

Reduce build time by narrowing Sentry source map uploads to server-only. Sets widenClientFileUpload: false in apps/app/next.config.ts, skipping client map scanning and saving ~30–40s per build while keeping server-side stack traces intact.

Written for commit 69c142e. Summary will update on new commits.

…check

Build optimizations (~110s savings):
- typescript.ignoreBuildErrors: true — skips the 79s TypeScript check
  during next build. Type errors are now caught by the check-types CI
  workflow instead.
- widenClientFileUpload: false — reduces Sentry source map upload
  scope, saving ~30-40s of scanning/uploading client-side files.

CI:
- Added pull_request trigger to check-types.yml so typecheck runs on
  every PR, replacing the in-build check.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented May 7, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app Ready Ready Preview, Comment May 7, 2026 1:01pm
comp-framework-editor Ready Ready Preview, Comment May 7, 2026 1:01pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
portal Skipped Skipped May 7, 2026 1:01pm

Request Review

@vercel
vercel Bot temporarily deployed to Preview – portal May 7, 2026 12:58 Inactive

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

@Marfuen
Marfuen merged commit 26e1667 into main May 7, 2026
9 of 10 checks passed
@Marfuen
Marfuen deleted the mariano/build-perf branch May 7, 2026 13:01
claudfuen pushed a commit that referenced this pull request May 7, 2026
# [3.45.0](v3.44.2...v3.45.0) (2026-05-07)

### Bug Fixes

* address cubic review findings on onboarding PR ([#2783](#2783)) ([bd43e8a](bd43e8a))
* **onboarding:** handle zero-item steps in tracker ([#2785](#2785)) ([a196339](a196339))
* revert page-level hooks to useRealtimeRun to fix missing auth context ([#2780](#2780)) ([5a406f9](5a406f9))

### Features

* **frameworks:** remove is-framework-versioning-enabled feature flag ([#2781](#2781)) ([016f379](016f379))

### Performance Improvements

* **build:** skip TS in next build, reduce sentry upload, add CI typecheck ([#2782](#2782)) ([26e1667](26e1667))
* **onboarding:** optimize onboarding pipeline from ~5min to ~2min ([b14db0b](b14db0b))
@claudfuen

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 3.45.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

This branch was successfully deployed

2 active and 1 inactive deployments
Preview – app — 69c142e0 Deployed May 7, 2026 by vercel[bot]
Preview – comp-framework-editor — 69c142e0 Deployed May 7, 2026 by vercel[bot]
Preview – portal — 69c142e0 Deployed May 7, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants