Skip to content

chore(aws-serverless): Don't build layer in build:dev command#19586

Open
s1gr1d wants to merge 5 commits intodevelopfrom
sig/aws-dev-build
Open

chore(aws-serverless): Don't build layer in build:dev command#19586
s1gr1d wants to merge 5 commits intodevelopfrom
sig/aws-dev-build

Conversation

@s1gr1d
Copy link
Member

@s1gr1d s1gr1d commented Mar 2, 2026

Similar to browser bundles, the layer does not need to be built during dev builds.

Closes #19587 (added automatically)

@github-actions
Copy link
Contributor

github-actions bot commented Mar 2, 2026

size-limit report 📦

⚠️ Warning: Base artifact is not the latest one, because the latest workflow run is not done yet. This may lead to incorrect results. Try to re-run all tests to get up to date results.

Path Size % Change Change
@sentry/browser 25.63 kB - -
@sentry/browser - with treeshaking flags 24.13 kB - -
@sentry/browser (incl. Tracing) 42.43 kB +0.01% +2 B 🔺
@sentry/browser (incl. Tracing, Profiling) 47.09 kB - -
@sentry/browser (incl. Tracing, Replay) 81.25 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 70.87 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 85.95 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 98.21 kB - -
@sentry/browser (incl. Feedback) 42.44 kB - -
@sentry/browser (incl. sendFeedback) 30.3 kB +0.01% +1 B 🔺
@sentry/browser (incl. FeedbackAsync) 35.35 kB - -
@sentry/browser (incl. Metrics) 26.8 kB - -
@sentry/browser (incl. Logs) 26.94 kB +0.01% +1 B 🔺
@sentry/browser (incl. Metrics & Logs) 27.61 kB - -
@sentry/react 27.38 kB - -
@sentry/react (incl. Tracing) 44.77 kB - -
@sentry/vue 30.08 kB - -
@sentry/vue (incl. Tracing) 44.3 kB - -
@sentry/svelte 25.66 kB - -
CDN Bundle 28.17 kB +0.01% +1 B 🔺
CDN Bundle (incl. Tracing) 43.26 kB - -
CDN Bundle (incl. Logs, Metrics) 29.01 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 44.1 kB +0.01% +1 B 🔺
CDN Bundle (incl. Replay, Logs, Metrics) 68.09 kB - -
CDN Bundle (incl. Tracing, Replay) 80.14 kB +0.01% +1 B 🔺
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 81 kB +0.01% +1 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) 85.65 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 86.53 kB - -
CDN Bundle - uncompressed 82.35 kB - -
CDN Bundle (incl. Tracing) - uncompressed 128.07 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 85.19 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 130.9 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 208.85 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 244.95 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 247.77 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 257.86 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 260.67 kB - -
@sentry/nextjs (client) 47.18 kB - -
@sentry/sveltekit (client) 42.89 kB - -
@sentry/node-core 52.25 kB +0.03% +12 B 🔺
@sentry/node 174.71 kB +0.02% +22 B 🔺
@sentry/node - without tracing 97.4 kB +0.02% +18 B 🔺
@sentry/aws-serverless 113.2 kB +0.02% +16 B 🔺

View base workflow run

@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2026

node-overhead report 🧳

Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.

Scenario Requests/s % of Baseline Prev. Requests/s Change %
GET Baseline 8,975 - 9,268 -3%
GET With Sentry 1,644 18% 1,666 -1%
GET With Sentry (error only) 5,987 67% 6,140 -2%
POST Baseline 1,152 - 1,158 -1%
POST With Sentry 566 49% 561 +1%
POST With Sentry (error only) 1,041 90% 1,043 -0%
MYSQL Baseline 3,162 - 3,185 -1%
MYSQL With Sentry 388 12% 429 -10%
MYSQL With Sentry (error only) 2,574 81% 2,608 -1%

View base workflow run

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Missing cache configuration for expensive build:layer target
    • Added "cache": true to the build:layer Nx target in packages/aws-serverless/package.json so unchanged layer builds can be served from cache.

Create PR

Or push these changes by commenting:

@cursor push 4438b9be5f
Preview (4438b9be5f)
diff --git a/packages/aws-serverless/package.json b/packages/aws-serverless/package.json
--- a/packages/aws-serverless/package.json
+++ b/packages/aws-serverless/package.json
@@ -123,6 +123,7 @@
           "production",
           "^production"
         ],
+        "cache": true,
         "dependsOn": [
           "build:transpile"
         ],
This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.

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.

chore(aws-serverless): Don't build layer in build:dev command

2 participants