test(nextjs): Add standalone test - #23056
Conversation
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
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.
Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 48f8d6b. Configure here.
| const standaloneDir = path.join(process.cwd(), '.next', 'standalone'); | ||
| expect(findFileInDir(standaloneDir, path.join('meriyah', 'dist', 'meriyah.mjs'))).toBeDefined(); | ||
| expect(findFileInDir(standaloneDir, path.join('meriyah', 'dist', 'meriyah.cjs'))).toBeDefined(); | ||
| }); |
There was a problem hiding this comment.
Meriyah path assertion is outdated
High Severity
The meriyah standalone check looks for meriyah/dist/meriyah.mjs and .cjs as separate packages, and the comment assumes the SDK force-includes them. On this branch, the @apm-js-collab/* → meriyah chain is bundled into @sentry/server-utils, so those paths will not appear in .next/standalone and this test will fail every run.
Triggered by project rule: PR Review Guidelines for Cursor Bot
Reviewed by Cursor Bugbot for commit 48f8d6b. Configure here.
size-limit report 📦
|
1 participant
#23034 suggests we don't have sufficient tests for Nextjs
standalonemode.This is the same test from #23055 which is only on the v10 branch.