Skip to content

Conversation

@chargome
Copy link
Member

@chargome chargome commented Jan 29, 2026

This test did not cover any test cases that we have not already in place in other test apps, apart from testing a navigation transaction on pages router, which makes this test basically a waste of resources.

  • Moved the navigation test to nextjs-pages-dir
  • Deleted the create-next-app test

Closes #19075 (added automatically)

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 is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@github-actions
Copy link
Contributor

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.
⚠️ 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.

Scenario Requests/s % of Baseline Prev. Requests/s Change %
GET Baseline 8,841 - 8,979 -2%
GET With Sentry 1,709 19% 1,637 +4%
GET With Sentry (error only) 5,900 67% 5,905 -0%
POST Baseline 1,180 - 1,170 +1%
POST With Sentry 588 50% 587 +0%
POST With Sentry (error only) 1,017 86% 1,031 -1%
MYSQL Baseline 3,249 - 3,195 +2%
MYSQL With Sentry 532 16% 427 +25%
MYSQL With Sentry (error only) 2,634 81% 2,564 +3%

View base workflow run

@chargome chargome requested a review from s1gr1d January 29, 2026 09:45
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.

test(e2e): Delete create-next-app test

3 participants