test: New integration tests by timfish · Pull Request #896 · getsentry/sentry-javascript-bundler-plugins

timfish · 2026-02-28T12:04:44Z

Ref Wider integration test coverage #879

This PR adds a new integration test package which will slowly replace the existing integration tests. For now I have only added Rolldown, partly because it has not been tested up until now. I will slowly migrate the other bundlers over and eventually remove the old integration tests.

The new integration tests run the bundler CLI's which allows us to test both CJS and ESM bundler configuration files. It also allows us to test the bundlers in complete isolation unlike the existing integration tests which call the JavaScript APIs of the bundlers. This has potential to leak context between tests, etc.

Rather than try and read the output files and assert specific code exists in the bundle, in the most part I just snapshot the entire output directory. As long as the snapshots are checked thoroughly for correctness, this is a much more robust way to see when unexpected changes occur. It also lets us test multiple features in less tests.

Worth noting:

@sentry/cli's helper.js is patched via patch-package so that we can also snapshot the cli args too
Each test has its own file which makes them fast, 17 tests in
- 2s on my machine
- 7s Ubuntu CI
- 9s Windows CI

…ation-next

github-actions · 2026-02-28T12:04:58Z

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

New Features ✨

Use crypto.randomUUID rather than uuid by timfish in #892

Internal Changes 🔧

New integration tests by timfish in #896

Remove lerna by timfish in #895
Migrate to Vitest by timfish in #894

_{🤖 This preview updates automatically when you update the PR.}

…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

cursor

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.}

chargome · 2026-03-04T15:51:18Z

+ if (process.env["SENTRY_TEST_OUT_DIR"]) {
+ // eslint-disable-next-line @typescript-eslint/unbound-method
+ const { join } = await import("node:path");
+ const { appendFileSync } = await import("node:fs");
+ const path = join(process.env["SENTRY_TEST_OUT_DIR"], "sentry-telemetry.json");
+ appendFileSync(path, JSON.stringify(request) + ",\n");
+ return { statusCode: 200 };
+ }


q: Not blocking but would there be a way around this?

I tried a mock server but spent too long trying to get it working and then found we already did something similar above for the existing integration tests.
I will revisit this though!

chargome · 2026-03-04T15:52:24Z

- "outputs": [],
- "cache": true
+ "inputs": ["default"],
+ "cache": false


q: That is intentional I guess?

It was intentional but this probably needs revising too.
I was changing the integration tests and re-running and it was hitting the cache.
Should we ever cache test output?

No let's not

@chargome

@chargome

timfish · 2026-02-28T12:04:44Z

Ref Wider integration test coverage #879

This PR adds a new integration test package which will slowly replace the existing integration tests. For now I have only added Rolldown, partly because it has not been tested up until now. I will slowly migrate the other bundlers over and eventually remove the old integration tests.

The new integration tests run the bundler CLI's which allows us to test both CJS and ESM bundler configuration files. It also allows us to test the bundlers in complete isolation unlike the existing integration tests which call the JavaScript APIs of the bundlers. This has potential to leak context between tests, etc.

Rather than try and read the output files and assert specific code exists in the bundle, in the most part I just snapshot the entire output directory. As long as the snapshots are checked thoroughly for correctness, this is a much more robust way to see when unexpected changes occur. It also lets us test multiple features in less tests.

Worth noting:

@sentry/cli's helper.js is patched via patch-package so that we can also snapshot the cli args too
Each test has its own file which makes them fast, 17 tests in
- 2s on my machine
- 7s Ubuntu CI
- 9s Windows CI

…ation-next

github-actions · 2026-02-28T12:04:58Z

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

New Features ✨

Use crypto.randomUUID rather than uuid by timfish in #892

Internal Changes 🔧

New integration tests by timfish in #896

Remove lerna by timfish in #895
Migrate to Vitest by timfish in #894

_{🤖 This preview updates automatically when you update the PR.}

…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

cursor

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.}

chargome · 2026-03-04T15:51:18Z

+ if (process.env["SENTRY_TEST_OUT_DIR"]) {
+ // eslint-disable-next-line @typescript-eslint/unbound-method
+ const { join } = await import("node:path");
+ const { appendFileSync } = await import("node:fs");
+ const path = join(process.env["SENTRY_TEST_OUT_DIR"], "sentry-telemetry.json");
+ appendFileSync(path, JSON.stringify(request) + ",\n");
+ return { statusCode: 200 };
+ }


q: Not blocking but would there be a way around this?

I tried a mock server but spent too long trying to get it working and then found we already did something similar above for the existing integration tests.
I will revisit this though!

chargome · 2026-03-04T15:52:24Z

- "outputs": [],
- "cache": true
+ "inputs": ["default"],
+ "cache": false


q: That is intentional I guess?

It was intentional but this probably needs revising too.
I was changing the integration tests and re-running and it was hitting the cache.
Should we ever cache test output?

No let's not

@chargome

@chargome

timfish · 2026-02-28T12:04:44Z

Ref Wider integration test coverage #879

This PR adds a new integration test package which will slowly replace the existing integration tests. For now I have only added Rolldown, partly because it has not been tested up until now. I will slowly migrate the other bundlers over and eventually remove the old integration tests.

The new integration tests run the bundler CLI's which allows us to test both CJS and ESM bundler configuration files. It also allows us to test the bundlers in complete isolation unlike the existing integration tests which call the JavaScript APIs of the bundlers. This has potential to leak context between tests, etc.

Rather than try and read the output files and assert specific code exists in the bundle, in the most part I just snapshot the entire output directory. As long as the snapshots are checked thoroughly for correctness, this is a much more robust way to see when unexpected changes occur. It also lets us test multiple features in less tests.

Worth noting:

@sentry/cli's helper.js is patched via patch-package so that we can also snapshot the cli args too
Each test has its own file which makes them fast, 17 tests in
- 2s on my machine
- 7s Ubuntu CI
- 9s Windows CI

…ation-next

github-actions · 2026-02-28T12:04:58Z

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

New Features ✨

Use crypto.randomUUID rather than uuid by timfish in #892

Internal Changes 🔧

New integration tests by timfish in #896

Remove lerna by timfish in #895
Migrate to Vitest by timfish in #894

_{🤖 This preview updates automatically when you update the PR.}

…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

cursor

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.}

chargome · 2026-03-04T15:51:18Z

+ if (process.env["SENTRY_TEST_OUT_DIR"]) {
+ // eslint-disable-next-line @typescript-eslint/unbound-method
+ const { join } = await import("node:path");
+ const { appendFileSync } = await import("node:fs");
+ const path = join(process.env["SENTRY_TEST_OUT_DIR"], "sentry-telemetry.json");
+ appendFileSync(path, JSON.stringify(request) + ",\n");
+ return { statusCode: 200 };
+ }


q: Not blocking but would there be a way around this?

I tried a mock server but spent too long trying to get it working and then found we already did something similar above for the existing integration tests.
I will revisit this though!

chargome · 2026-03-04T15:52:24Z

- "outputs": [],
- "cache": true
+ "inputs": ["default"],
+ "cache": false


q: That is intentional I guess?

It was intentional but this probably needs revising too.
I was changing the integration tests and re-running and it was hitting the cache.
Should we ever cache test output?

No let's not

@chargome

@chargome

timfish · 2026-02-28T12:04:44Z

Ref Wider integration test coverage #879

This PR adds a new integration test package which will slowly replace the existing integration tests. For now I have only added Rolldown, partly because it has not been tested up until now. I will slowly migrate the other bundlers over and eventually remove the old integration tests.

The new integration tests run the bundler CLI's which allows us to test both CJS and ESM bundler configuration files. It also allows us to test the bundlers in complete isolation unlike the existing integration tests which call the JavaScript APIs of the bundlers. This has potential to leak context between tests, etc.

Rather than try and read the output files and assert specific code exists in the bundle, in the most part I just snapshot the entire output directory. As long as the snapshots are checked thoroughly for correctness, this is a much more robust way to see when unexpected changes occur. It also lets us test multiple features in less tests.

Worth noting:

@sentry/cli's helper.js is patched via patch-package so that we can also snapshot the cli args too
Each test has its own file which makes them fast, 17 tests in
- 2s on my machine
- 7s Ubuntu CI
- 9s Windows CI

…ation-next

github-actions · 2026-02-28T12:04:58Z

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

New Features ✨

Use crypto.randomUUID rather than uuid by timfish in #892

Internal Changes 🔧

New integration tests by timfish in #896

Remove lerna by timfish in #895
Migrate to Vitest by timfish in #894

_{🤖 This preview updates automatically when you update the PR.}

…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

cursor

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.}

chargome · 2026-03-04T15:51:18Z

+ if (process.env["SENTRY_TEST_OUT_DIR"]) {
+ // eslint-disable-next-line @typescript-eslint/unbound-method
+ const { join } = await import("node:path");
+ const { appendFileSync } = await import("node:fs");
+ const path = join(process.env["SENTRY_TEST_OUT_DIR"], "sentry-telemetry.json");
+ appendFileSync(path, JSON.stringify(request) + ",\n");
+ return { statusCode: 200 };
+ }


q: Not blocking but would there be a way around this?

I tried a mock server but spent too long trying to get it working and then found we already did something similar above for the existing integration tests.
I will revisit this though!

chargome · 2026-03-04T15:52:24Z

- "outputs": [],
- "cache": true
+ "inputs": ["default"],
+ "cache": false


q: That is intentional I guess?

It was intentional but this probably needs revising too.
I was changing the integration tests and re-running and it was hitting the cache.
Should we ever cache test output?

No let's not

@chargome

@chargome

timfish · 2026-02-28T12:04:44Z

Ref Wider integration test coverage #879

This PR adds a new integration test package which will slowly replace the existing integration tests. For now I have only added Rolldown, partly because it has not been tested up until now. I will slowly migrate the other bundlers over and eventually remove the old integration tests.

The new integration tests run the bundler CLI's which allows us to test both CJS and ESM bundler configuration files. It also allows us to test the bundlers in complete isolation unlike the existing integration tests which call the JavaScript APIs of the bundlers. This has potential to leak context between tests, etc.

Rather than try and read the output files and assert specific code exists in the bundle, in the most part I just snapshot the entire output directory. As long as the snapshots are checked thoroughly for correctness, this is a much more robust way to see when unexpected changes occur. It also lets us test multiple features in less tests.

Worth noting:

@sentry/cli's helper.js is patched via patch-package so that we can also snapshot the cli args too
Each test has its own file which makes them fast, 17 tests in
- 2s on my machine
- 7s Ubuntu CI
- 9s Windows CI

…ation-next

github-actions · 2026-02-28T12:04:58Z

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

New Features ✨

Use crypto.randomUUID rather than uuid by timfish in #892

Internal Changes 🔧

New integration tests by timfish in #896

Remove lerna by timfish in #895
Migrate to Vitest by timfish in #894

_{🤖 This preview updates automatically when you update the PR.}

…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

cursor

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.}

chargome · 2026-03-04T15:51:18Z

+ if (process.env["SENTRY_TEST_OUT_DIR"]) {
+ // eslint-disable-next-line @typescript-eslint/unbound-method
+ const { join } = await import("node:path");
+ const { appendFileSync } = await import("node:fs");
+ const path = join(process.env["SENTRY_TEST_OUT_DIR"], "sentry-telemetry.json");
+ appendFileSync(path, JSON.stringify(request) + ",\n");
+ return { statusCode: 200 };
+ }


q: Not blocking but would there be a way around this?

I tried a mock server but spent too long trying to get it working and then found we already did something similar above for the existing integration tests.
I will revisit this though!

chargome · 2026-03-04T15:52:24Z

- "outputs": [],
- "cache": true
+ "inputs": ["default"],
+ "cache": false


q: That is intentional I guess?

It was intentional but this probably needs revising too.
I was changing the integration tests and re-running and it was hitting the cache.
Should we ever cache test output?

No let's not

@chargome

@chargome

timfish · 2026-02-28T12:04:44Z

Ref Wider integration test coverage #879

This PR adds a new integration test package which will slowly replace the existing integration tests. For now I have only added Rolldown, partly because it has not been tested up until now. I will slowly migrate the other bundlers over and eventually remove the old integration tests.

The new integration tests run the bundler CLI's which allows us to test both CJS and ESM bundler configuration files. It also allows us to test the bundlers in complete isolation unlike the existing integration tests which call the JavaScript APIs of the bundlers. This has potential to leak context between tests, etc.

Rather than try and read the output files and assert specific code exists in the bundle, in the most part I just snapshot the entire output directory. As long as the snapshots are checked thoroughly for correctness, this is a much more robust way to see when unexpected changes occur. It also lets us test multiple features in less tests.

Worth noting:

@sentry/cli's helper.js is patched via patch-package so that we can also snapshot the cli args too
Each test has its own file which makes them fast, 17 tests in
- 2s on my machine
- 7s Ubuntu CI
- 9s Windows CI

…ation-next

github-actions · 2026-02-28T12:04:58Z

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

New Features ✨

Use crypto.randomUUID rather than uuid by timfish in #892

Internal Changes 🔧

New integration tests by timfish in #896

Remove lerna by timfish in #895
Migrate to Vitest by timfish in #894

_{🤖 This preview updates automatically when you update the PR.}

…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

cursor

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.}

chargome · 2026-03-04T15:51:18Z

+ if (process.env["SENTRY_TEST_OUT_DIR"]) {
+ // eslint-disable-next-line @typescript-eslint/unbound-method
+ const { join } = await import("node:path");
+ const { appendFileSync } = await import("node:fs");
+ const path = join(process.env["SENTRY_TEST_OUT_DIR"], "sentry-telemetry.json");
+ appendFileSync(path, JSON.stringify(request) + ",\n");
+ return { statusCode: 200 };
+ }


q: Not blocking but would there be a way around this?

I tried a mock server but spent too long trying to get it working and then found we already did something similar above for the existing integration tests.
I will revisit this though!

chargome · 2026-03-04T15:52:24Z

- "outputs": [],
- "cache": true
+ "inputs": ["default"],
+ "cache": false


q: That is intentional I guess?

It was intentional but this probably needs revising too.
I was changing the integration tests and re-running and it was hitting the cache.
Should we ever cache test output?

No let's not

@chargome

@chargome

timfish · 2026-02-28T12:04:44Z

Ref Wider integration test coverage #879

This PR adds a new integration test package which will slowly replace the existing integration tests. For now I have only added Rolldown, partly because it has not been tested up until now. I will slowly migrate the other bundlers over and eventually remove the old integration tests.

The new integration tests run the bundler CLI's which allows us to test both CJS and ESM bundler configuration files. It also allows us to test the bundlers in complete isolation unlike the existing integration tests which call the JavaScript APIs of the bundlers. This has potential to leak context between tests, etc.

Rather than try and read the output files and assert specific code exists in the bundle, in the most part I just snapshot the entire output directory. As long as the snapshots are checked thoroughly for correctness, this is a much more robust way to see when unexpected changes occur. It also lets us test multiple features in less tests.

Worth noting:

@sentry/cli's helper.js is patched via patch-package so that we can also snapshot the cli args too
Each test has its own file which makes them fast, 17 tests in
- 2s on my machine
- 7s Ubuntu CI
- 9s Windows CI

…ation-next

github-actions · 2026-02-28T12:04:58Z

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

New Features ✨

Use crypto.randomUUID rather than uuid by timfish in #892

Internal Changes 🔧

New integration tests by timfish in #896

Remove lerna by timfish in #895
Migrate to Vitest by timfish in #894

_{🤖 This preview updates automatically when you update the PR.}

…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

cursor

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.}

chargome · 2026-03-04T15:51:18Z

+ if (process.env["SENTRY_TEST_OUT_DIR"]) {
+ // eslint-disable-next-line @typescript-eslint/unbound-method
+ const { join } = await import("node:path");
+ const { appendFileSync } = await import("node:fs");
+ const path = join(process.env["SENTRY_TEST_OUT_DIR"], "sentry-telemetry.json");
+ appendFileSync(path, JSON.stringify(request) + ",\n");
+ return { statusCode: 200 };
+ }


q: Not blocking but would there be a way around this?

I tried a mock server but spent too long trying to get it working and then found we already did something similar above for the existing integration tests.
I will revisit this though!

chargome · 2026-03-04T15:52:24Z

- "outputs": [],
- "cache": true
+ "inputs": ["default"],
+ "cache": false


q: That is intentional I guess?

It was intentional but this probably needs revising too.
I was changing the integration tests and re-running and it was hitting the cache.
Should we ever cache test output?

No let's not

@chargome

@chargome

timfish · 2026-02-28T12:04:44Z

Ref Wider integration test coverage #879

This PR adds a new integration test package which will slowly replace the existing integration tests. For now I have only added Rolldown, partly because it has not been tested up until now. I will slowly migrate the other bundlers over and eventually remove the old integration tests.

The new integration tests run the bundler CLI's which allows us to test both CJS and ESM bundler configuration files. It also allows us to test the bundlers in complete isolation unlike the existing integration tests which call the JavaScript APIs of the bundlers. This has potential to leak context between tests, etc.

Rather than try and read the output files and assert specific code exists in the bundle, in the most part I just snapshot the entire output directory. As long as the snapshots are checked thoroughly for correctness, this is a much more robust way to see when unexpected changes occur. It also lets us test multiple features in less tests.

Worth noting:

@sentry/cli's helper.js is patched via patch-package so that we can also snapshot the cli args too
Each test has its own file which makes them fast, 17 tests in
- 2s on my machine
- 7s Ubuntu CI
- 9s Windows CI

…ation-next

github-actions · 2026-02-28T12:04:58Z

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

New Features ✨

Use crypto.randomUUID rather than uuid by timfish in #892

Internal Changes 🔧

New integration tests by timfish in #896

Remove lerna by timfish in #895
Migrate to Vitest by timfish in #894

_{🤖 This preview updates automatically when you update the PR.}

…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

cursor

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.}

chargome · 2026-03-04T15:51:18Z

+ if (process.env["SENTRY_TEST_OUT_DIR"]) {
+ // eslint-disable-next-line @typescript-eslint/unbound-method
+ const { join } = await import("node:path");
+ const { appendFileSync } = await import("node:fs");
+ const path = join(process.env["SENTRY_TEST_OUT_DIR"], "sentry-telemetry.json");
+ appendFileSync(path, JSON.stringify(request) + ",\n");
+ return { statusCode: 200 };
+ }


q: Not blocking but would there be a way around this?

I tried a mock server but spent too long trying to get it working and then found we already did something similar above for the existing integration tests.
I will revisit this though!

chargome · 2026-03-04T15:52:24Z

- "outputs": [],
- "cache": true
+ "inputs": ["default"],
+ "cache": false


q: That is intentional I guess?

It was intentional but this probably needs revising too.
I was changing the integration tests and re-running and it was hitting the cache.
Should we ever cache test output?

No let's not

@chargome

@chargome

timfish added 2 commits February 28, 2026 11:52

test: New integration tests
60ddcf3

Merge remote-tracking branch 'upstream/main' into timfish/test/integr…
91a1ac7
…ation-next

timfish force-pushed the timfish/test/integration-next branch 3 times, most recently from 531df3b to dfa1552Compare March 1, 2026 20:05

make it faster and better
e93d586

timfish force-pushed the timfish/test/integration-next branch from dfa1552 to e93d586Compare March 1, 2026 20:11

timfish added 5 commits March 1, 2026 21:42

disable query param test on Windows
61b1a36

more tests
c182d26

Merge branch 'timfish/test/integration-next' ofhttps://github.com/ge…
0fc97ee
…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

Exclude integration from unit tests
5eea904

fix CI
22d4fe6

timfish marked this pull request as ready for review March 3, 2026 15:19

timfish requested review from andreiborza and chargome March 3, 2026 15:19

This comment was marked as outdated.
Sign in to view

Fixes from cursor PR review
457e22a

sentryBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/bundler-plugin-core/src/sentry/transports.ts

cursorBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/integration-tests-next/fixtures/rolldown/utils.ts

chargome approved these changes Mar 4, 2026
View reviewed changes

chargome reviewed Mar 4, 2026
View reviewed changes

timfish merged commit ef7d97f into mainMar 5, 2026
33 checks passed

timfish deleted the timfish/test/integration-next branch March 5, 2026 23:42

This was referenced Apr 7, 2026
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7730
Closed
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7738
Closed

timfish added 2 commits February 28, 2026 11:52

test: New integration tests
60ddcf3

Merge remote-tracking branch 'upstream/main' into timfish/test/integr…
91a1ac7
…ation-next

timfish force-pushed the timfish/test/integration-next branch 3 times, most recently from 531df3b to dfa1552Compare March 1, 2026 20:05

make it faster and better
e93d586

timfish force-pushed the timfish/test/integration-next branch from dfa1552 to e93d586Compare March 1, 2026 20:11

timfish added 5 commits March 1, 2026 21:42

disable query param test on Windows
61b1a36

more tests
c182d26

Merge branch 'timfish/test/integration-next' ofhttps://github.com/ge…
0fc97ee
…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

Exclude integration from unit tests
5eea904

fix CI
22d4fe6

timfish marked this pull request as ready for review March 3, 2026 15:19

timfish requested review from andreiborza and chargome March 3, 2026 15:19

This comment was marked as outdated.
Sign in to view

Fixes from cursor PR review
457e22a

sentryBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/bundler-plugin-core/src/sentry/transports.ts

cursorBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/integration-tests-next/fixtures/rolldown/utils.ts

chargome approved these changes Mar 4, 2026
View reviewed changes

chargome reviewed Mar 4, 2026
View reviewed changes

timfish merged commit ef7d97f into mainMar 5, 2026
33 checks passed

timfish deleted the timfish/test/integration-next branch March 5, 2026 23:42

This was referenced Apr 7, 2026
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7730
Closed
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7738
Closed

timfish added 2 commits February 28, 2026 11:52

test: New integration tests
60ddcf3

Merge remote-tracking branch 'upstream/main' into timfish/test/integr…
91a1ac7
…ation-next

timfish force-pushed the timfish/test/integration-next branch 3 times, most recently from 531df3b to dfa1552Compare March 1, 2026 20:05

make it faster and better
e93d586

timfish force-pushed the timfish/test/integration-next branch from dfa1552 to e93d586Compare March 1, 2026 20:11

timfish added 5 commits March 1, 2026 21:42

disable query param test on Windows
61b1a36

more tests
c182d26

Merge branch 'timfish/test/integration-next' ofhttps://github.com/ge…
0fc97ee
…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

Exclude integration from unit tests
5eea904

fix CI
22d4fe6

timfish marked this pull request as ready for review March 3, 2026 15:19

timfish requested review from andreiborza and chargome March 3, 2026 15:19

This comment was marked as outdated.
Sign in to view

Fixes from cursor PR review
457e22a

sentryBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/bundler-plugin-core/src/sentry/transports.ts

cursorBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/integration-tests-next/fixtures/rolldown/utils.ts

chargome approved these changes Mar 4, 2026
View reviewed changes

chargome reviewed Mar 4, 2026
View reviewed changes

timfish merged commit ef7d97f into mainMar 5, 2026
33 checks passed

timfish deleted the timfish/test/integration-next branch March 5, 2026 23:42

This was referenced Apr 7, 2026
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7730
Closed
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7738
Closed

timfish added 2 commits February 28, 2026 11:52

test: New integration tests
60ddcf3

Merge remote-tracking branch 'upstream/main' into timfish/test/integr…
91a1ac7
…ation-next

timfish force-pushed the timfish/test/integration-next branch 3 times, most recently from 531df3b to dfa1552Compare March 1, 2026 20:05

make it faster and better
e93d586

timfish force-pushed the timfish/test/integration-next branch from dfa1552 to e93d586Compare March 1, 2026 20:11

timfish added 5 commits March 1, 2026 21:42

disable query param test on Windows
61b1a36

more tests
c182d26

Merge branch 'timfish/test/integration-next' ofhttps://github.com/ge…
0fc97ee
…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

Exclude integration from unit tests
5eea904

fix CI
22d4fe6

timfish marked this pull request as ready for review March 3, 2026 15:19

timfish requested review from andreiborza and chargome March 3, 2026 15:19

This comment was marked as outdated.
Sign in to view

Fixes from cursor PR review
457e22a

sentryBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/bundler-plugin-core/src/sentry/transports.ts

cursorBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/integration-tests-next/fixtures/rolldown/utils.ts

chargome approved these changes Mar 4, 2026
View reviewed changes

chargome reviewed Mar 4, 2026
View reviewed changes

timfish merged commit ef7d97f into mainMar 5, 2026
33 checks passed

timfish deleted the timfish/test/integration-next branch March 5, 2026 23:42

This was referenced Apr 7, 2026
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7730
Closed
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7738
Closed

timfish added 2 commits February 28, 2026 11:52

test: New integration tests
60ddcf3

Merge remote-tracking branch 'upstream/main' into timfish/test/integr…
91a1ac7
…ation-next

timfish force-pushed the timfish/test/integration-next branch 3 times, most recently from 531df3b to dfa1552Compare March 1, 2026 20:05

make it faster and better
e93d586

timfish force-pushed the timfish/test/integration-next branch from dfa1552 to e93d586Compare March 1, 2026 20:11

timfish added 5 commits March 1, 2026 21:42

disable query param test on Windows
61b1a36

more tests
c182d26

Merge branch 'timfish/test/integration-next' ofhttps://github.com/ge…
0fc97ee
…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

Exclude integration from unit tests
5eea904

fix CI
22d4fe6

timfish marked this pull request as ready for review March 3, 2026 15:19

timfish requested review from andreiborza and chargome March 3, 2026 15:19

This comment was marked as outdated.
Sign in to view

Fixes from cursor PR review
457e22a

sentryBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/bundler-plugin-core/src/sentry/transports.ts

cursorBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/integration-tests-next/fixtures/rolldown/utils.ts

chargome approved these changes Mar 4, 2026
View reviewed changes

chargome reviewed Mar 4, 2026
View reviewed changes

timfish merged commit ef7d97f into mainMar 5, 2026
33 checks passed

timfish deleted the timfish/test/integration-next branch March 5, 2026 23:42

This was referenced Apr 7, 2026
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7730
Closed
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7738
Closed

timfish added 2 commits February 28, 2026 11:52

test: New integration tests
60ddcf3

Merge remote-tracking branch 'upstream/main' into timfish/test/integr…
91a1ac7
…ation-next

timfish force-pushed the timfish/test/integration-next branch 3 times, most recently from 531df3b to dfa1552Compare March 1, 2026 20:05

make it faster and better
e93d586

timfish force-pushed the timfish/test/integration-next branch from dfa1552 to e93d586Compare March 1, 2026 20:11

timfish added 5 commits March 1, 2026 21:42

disable query param test on Windows
61b1a36

more tests
c182d26

Merge branch 'timfish/test/integration-next' ofhttps://github.com/ge…
0fc97ee
…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

Exclude integration from unit tests
5eea904

fix CI
22d4fe6

timfish marked this pull request as ready for review March 3, 2026 15:19

timfish requested review from andreiborza and chargome March 3, 2026 15:19

This comment was marked as outdated.
Sign in to view

Fixes from cursor PR review
457e22a

sentryBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/bundler-plugin-core/src/sentry/transports.ts

cursorBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/integration-tests-next/fixtures/rolldown/utils.ts

chargome approved these changes Mar 4, 2026
View reviewed changes

chargome reviewed Mar 4, 2026
View reviewed changes

timfish merged commit ef7d97f into mainMar 5, 2026
33 checks passed

timfish deleted the timfish/test/integration-next branch March 5, 2026 23:42

This was referenced Apr 7, 2026
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7730
Closed
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7738
Closed

timfish added 2 commits February 28, 2026 11:52

test: New integration tests
60ddcf3

Merge remote-tracking branch 'upstream/main' into timfish/test/integr…
91a1ac7
…ation-next

timfish force-pushed the timfish/test/integration-next branch 3 times, most recently from 531df3b to dfa1552Compare March 1, 2026 20:05

make it faster and better
e93d586

timfish force-pushed the timfish/test/integration-next branch from dfa1552 to e93d586Compare March 1, 2026 20:11

timfish added 5 commits March 1, 2026 21:42

disable query param test on Windows
61b1a36

more tests
c182d26

Merge branch 'timfish/test/integration-next' ofhttps://github.com/ge…
0fc97ee
…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

Exclude integration from unit tests
5eea904

fix CI
22d4fe6

timfish marked this pull request as ready for review March 3, 2026 15:19

timfish requested review from andreiborza and chargome March 3, 2026 15:19

This comment was marked as outdated.
Sign in to view

Fixes from cursor PR review
457e22a

sentryBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/bundler-plugin-core/src/sentry/transports.ts

cursorBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/integration-tests-next/fixtures/rolldown/utils.ts

chargome approved these changes Mar 4, 2026
View reviewed changes

chargome reviewed Mar 4, 2026
View reviewed changes

timfish merged commit ef7d97f into mainMar 5, 2026
33 checks passed

timfish deleted the timfish/test/integration-next branch March 5, 2026 23:42

This was referenced Apr 7, 2026
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7730
Closed
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7738
Closed

timfish added 2 commits February 28, 2026 11:52

test: New integration tests
60ddcf3

Merge remote-tracking branch 'upstream/main' into timfish/test/integr…
91a1ac7
…ation-next

timfish force-pushed the timfish/test/integration-next branch 3 times, most recently from 531df3b to dfa1552Compare March 1, 2026 20:05

make it faster and better
e93d586

timfish force-pushed the timfish/test/integration-next branch from dfa1552 to e93d586Compare March 1, 2026 20:11

timfish added 5 commits March 1, 2026 21:42

disable query param test on Windows
61b1a36

more tests
c182d26

Merge branch 'timfish/test/integration-next' ofhttps://github.com/ge…
0fc97ee
…tsentry/sentry-javascript-bundler-plugins into timfish/test/integration-next

Exclude integration from unit tests
5eea904

fix CI
22d4fe6

timfish marked this pull request as ready for review March 3, 2026 15:19

timfish requested review from andreiborza and chargome March 3, 2026 15:19

This comment was marked as outdated.
Sign in to view

Fixes from cursor PR review
457e22a

sentryBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/bundler-plugin-core/src/sentry/transports.ts

cursorBot reviewed Mar 3, 2026
View reviewed changes

Comment threadpackages/integration-tests-next/fixtures/rolldown/utils.ts

chargome approved these changes Mar 4, 2026
View reviewed changes

chargome reviewed Mar 4, 2026
View reviewed changes

timfish merged commit ef7d97f into mainMar 5, 2026
33 checks passed

timfish deleted the timfish/test/integration-next branch March 5, 2026 23:42

This was referenced Apr 7, 2026
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7730
Closed
publish: getsentry/sentry-javascript-bundler-plugins@5.2.0 getsentry/publish#7738
Closed

Uh oh!

Conversation

timfish commented Feb 28, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Worth noting:

Uh oh!

github-actionsBot commented Feb 28, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Semver Impact of This PR

New Features ✨

Internal Changes 🔧

Uh oh!

This comment was marked as outdated.

Uh oh!

Uh oh!

cursorBot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

timfishMar 4, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

timfish commented Feb 28, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Worth noting:

Uh oh!

github-actionsBot commented Feb 28, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Semver Impact of This PR

New Features ✨

Internal Changes 🔧

Uh oh!

This comment was marked as outdated.

Uh oh!

Uh oh!

cursorBot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

timfishMar 4, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

timfish commented Feb 28, 2026•
edited
Loading

github-actionsBot commented Feb 28, 2026•
edited
Loading

timfishMar 4, 2026•
edited
Loading

timfish commented Feb 28, 2026•
edited
Loading

github-actionsBot commented Feb 28, 2026•
edited
Loading

timfishMar 4, 2026•
edited
Loading

timfish commented Feb 28, 2026•
edited
Loading

github-actionsBot commented Feb 28, 2026•
edited
Loading

timfishMar 4, 2026•
edited
Loading