test: Isolate integration test package installs by timfish · Pull Request #902 · getsentry/sentry-javascript-bundler-plugins

timfish · 2026-03-25T16:48:56Z

This PR:

Removes the test fixtures from the yarn workspace
Adds a setup script that does an pnpm install for each fixture
Uses pnpm overrides to pickup the local packed packages
Updates the version bump script so the tgz paths in the overrides get correctly updated
Moves the @sentry/cli patching from patch-package to pnpm built in patchedDependencies

github-actions · 2026-03-25T16:49:13Z

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 🔧

Isolate integration test package installs by timfish in #902

Pin GitHub Actions to full-length commit SHAs by joshuarli in #900
Rollup integration tests by timfish in #897
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.}

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

cursor · 2026-03-25T17:57:53Z

+ const pkg = readJson(pkgPath);
+ if (!pkg.dependencies) {
+ continue;
+ }


Early return skips pnpm overrides path update
Low Severity
The !pkg.dependencies guard on line 86 causes bumpTestFixtureVersions to continue to the next fixture, which also skips updating the pnpm.overrides tgz paths below. If a fixture ever has pnpm.overrides pointing to local tarballs but no direct @sentry/ dependencies (e.g., only transitive ones), the override paths would silently go stale after a version bump, causing pnpm install to fail on missing .tgz files.
Additional Locations (1)
scripts/bump-version.js#L95-L103

if it doesn't have any dependencies, it wont have any overrides 🙈

chargome · 2026-03-26T09:16:51Z

+for (const dir of directories) {
+ try {
+ const packageJson = await fs.readFile(join(dir, "package.json"), { encoding: "utf-8" });
+ if (packageJson.length < 50) {


Can you add a comment on this magic number? :D

I'll do this a more sensible way without magic!

@chargome

@chargome

timfish · 2026-03-25T16:48:56Z

This PR:

Removes the test fixtures from the yarn workspace
Adds a setup script that does an pnpm install for each fixture
Uses pnpm overrides to pickup the local packed packages
Updates the version bump script so the tgz paths in the overrides get correctly updated
Moves the @sentry/cli patching from patch-package to pnpm built in patchedDependencies

github-actions · 2026-03-25T16:49:13Z

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 🔧

Isolate integration test package installs by timfish in #902

Pin GitHub Actions to full-length commit SHAs by joshuarli in #900
Rollup integration tests by timfish in #897
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.}

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

cursor · 2026-03-25T17:57:53Z

+ const pkg = readJson(pkgPath);
+ if (!pkg.dependencies) {
+ continue;
+ }


Early return skips pnpm overrides path update
Low Severity
The !pkg.dependencies guard on line 86 causes bumpTestFixtureVersions to continue to the next fixture, which also skips updating the pnpm.overrides tgz paths below. If a fixture ever has pnpm.overrides pointing to local tarballs but no direct @sentry/ dependencies (e.g., only transitive ones), the override paths would silently go stale after a version bump, causing pnpm install to fail on missing .tgz files.
Additional Locations (1)
scripts/bump-version.js#L95-L103

if it doesn't have any dependencies, it wont have any overrides 🙈

chargome · 2026-03-26T09:16:51Z

+for (const dir of directories) {
+ try {
+ const packageJson = await fs.readFile(join(dir, "package.json"), { encoding: "utf-8" });
+ if (packageJson.length < 50) {


Can you add a comment on this magic number? :D

I'll do this a more sensible way without magic!

@chargome

@chargome

timfish · 2026-03-25T16:48:56Z

This PR:

Removes the test fixtures from the yarn workspace
Adds a setup script that does an pnpm install for each fixture
Uses pnpm overrides to pickup the local packed packages
Updates the version bump script so the tgz paths in the overrides get correctly updated
Moves the @sentry/cli patching from patch-package to pnpm built in patchedDependencies

github-actions · 2026-03-25T16:49:13Z

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 🔧

Isolate integration test package installs by timfish in #902

Pin GitHub Actions to full-length commit SHAs by joshuarli in #900
Rollup integration tests by timfish in #897
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.}

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

cursor · 2026-03-25T17:57:53Z

+ const pkg = readJson(pkgPath);
+ if (!pkg.dependencies) {
+ continue;
+ }


Early return skips pnpm overrides path update
Low Severity
The !pkg.dependencies guard on line 86 causes bumpTestFixtureVersions to continue to the next fixture, which also skips updating the pnpm.overrides tgz paths below. If a fixture ever has pnpm.overrides pointing to local tarballs but no direct @sentry/ dependencies (e.g., only transitive ones), the override paths would silently go stale after a version bump, causing pnpm install to fail on missing .tgz files.
Additional Locations (1)
scripts/bump-version.js#L95-L103

if it doesn't have any dependencies, it wont have any overrides 🙈

chargome · 2026-03-26T09:16:51Z

+for (const dir of directories) {
+ try {
+ const packageJson = await fs.readFile(join(dir, "package.json"), { encoding: "utf-8" });
+ if (packageJson.length < 50) {


Can you add a comment on this magic number? :D

I'll do this a more sensible way without magic!

@chargome

@chargome

timfish · 2026-03-25T16:48:56Z

This PR:

Removes the test fixtures from the yarn workspace
Adds a setup script that does an pnpm install for each fixture
Uses pnpm overrides to pickup the local packed packages
Updates the version bump script so the tgz paths in the overrides get correctly updated
Moves the @sentry/cli patching from patch-package to pnpm built in patchedDependencies

github-actions · 2026-03-25T16:49:13Z

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 🔧

Isolate integration test package installs by timfish in #902

Pin GitHub Actions to full-length commit SHAs by joshuarli in #900
Rollup integration tests by timfish in #897
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.}

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

cursor · 2026-03-25T17:57:53Z

+ const pkg = readJson(pkgPath);
+ if (!pkg.dependencies) {
+ continue;
+ }


Early return skips pnpm overrides path update
Low Severity
The !pkg.dependencies guard on line 86 causes bumpTestFixtureVersions to continue to the next fixture, which also skips updating the pnpm.overrides tgz paths below. If a fixture ever has pnpm.overrides pointing to local tarballs but no direct @sentry/ dependencies (e.g., only transitive ones), the override paths would silently go stale after a version bump, causing pnpm install to fail on missing .tgz files.
Additional Locations (1)
scripts/bump-version.js#L95-L103

if it doesn't have any dependencies, it wont have any overrides 🙈

chargome · 2026-03-26T09:16:51Z

+for (const dir of directories) {
+ try {
+ const packageJson = await fs.readFile(join(dir, "package.json"), { encoding: "utf-8" });
+ if (packageJson.length < 50) {


Can you add a comment on this magic number? :D

I'll do this a more sensible way without magic!

@chargome

@chargome

timfish · 2026-03-25T16:48:56Z

This PR:

Removes the test fixtures from the yarn workspace
Adds a setup script that does an pnpm install for each fixture
Uses pnpm overrides to pickup the local packed packages
Updates the version bump script so the tgz paths in the overrides get correctly updated
Moves the @sentry/cli patching from patch-package to pnpm built in patchedDependencies

github-actions · 2026-03-25T16:49:13Z

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 🔧

Isolate integration test package installs by timfish in #902

Pin GitHub Actions to full-length commit SHAs by joshuarli in #900
Rollup integration tests by timfish in #897
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.}

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

cursor · 2026-03-25T17:57:53Z

+ const pkg = readJson(pkgPath);
+ if (!pkg.dependencies) {
+ continue;
+ }


Early return skips pnpm overrides path update
Low Severity
The !pkg.dependencies guard on line 86 causes bumpTestFixtureVersions to continue to the next fixture, which also skips updating the pnpm.overrides tgz paths below. If a fixture ever has pnpm.overrides pointing to local tarballs but no direct @sentry/ dependencies (e.g., only transitive ones), the override paths would silently go stale after a version bump, causing pnpm install to fail on missing .tgz files.
Additional Locations (1)
scripts/bump-version.js#L95-L103

if it doesn't have any dependencies, it wont have any overrides 🙈

chargome · 2026-03-26T09:16:51Z

+for (const dir of directories) {
+ try {
+ const packageJson = await fs.readFile(join(dir, "package.json"), { encoding: "utf-8" });
+ if (packageJson.length < 50) {


Can you add a comment on this magic number? :D

I'll do this a more sensible way without magic!

@chargome

@chargome

timfish · 2026-03-25T16:48:56Z

This PR:

Removes the test fixtures from the yarn workspace
Adds a setup script that does an pnpm install for each fixture
Uses pnpm overrides to pickup the local packed packages
Updates the version bump script so the tgz paths in the overrides get correctly updated
Moves the @sentry/cli patching from patch-package to pnpm built in patchedDependencies

github-actions · 2026-03-25T16:49:13Z

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 🔧

Isolate integration test package installs by timfish in #902

Pin GitHub Actions to full-length commit SHAs by joshuarli in #900
Rollup integration tests by timfish in #897
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.}

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

cursor · 2026-03-25T17:57:53Z

+ const pkg = readJson(pkgPath);
+ if (!pkg.dependencies) {
+ continue;
+ }


Early return skips pnpm overrides path update
Low Severity
The !pkg.dependencies guard on line 86 causes bumpTestFixtureVersions to continue to the next fixture, which also skips updating the pnpm.overrides tgz paths below. If a fixture ever has pnpm.overrides pointing to local tarballs but no direct @sentry/ dependencies (e.g., only transitive ones), the override paths would silently go stale after a version bump, causing pnpm install to fail on missing .tgz files.
Additional Locations (1)
scripts/bump-version.js#L95-L103

if it doesn't have any dependencies, it wont have any overrides 🙈

chargome · 2026-03-26T09:16:51Z

+for (const dir of directories) {
+ try {
+ const packageJson = await fs.readFile(join(dir, "package.json"), { encoding: "utf-8" });
+ if (packageJson.length < 50) {


Can you add a comment on this magic number? :D

I'll do this a more sensible way without magic!

@chargome

@chargome

timfish · 2026-03-25T16:48:56Z

This PR:

Removes the test fixtures from the yarn workspace
Adds a setup script that does an pnpm install for each fixture
Uses pnpm overrides to pickup the local packed packages
Updates the version bump script so the tgz paths in the overrides get correctly updated
Moves the @sentry/cli patching from patch-package to pnpm built in patchedDependencies

github-actions · 2026-03-25T16:49:13Z

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 🔧

Isolate integration test package installs by timfish in #902

Pin GitHub Actions to full-length commit SHAs by joshuarli in #900
Rollup integration tests by timfish in #897
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.}

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

cursor · 2026-03-25T17:57:53Z

+ const pkg = readJson(pkgPath);
+ if (!pkg.dependencies) {
+ continue;
+ }


Early return skips pnpm overrides path update
Low Severity
The !pkg.dependencies guard on line 86 causes bumpTestFixtureVersions to continue to the next fixture, which also skips updating the pnpm.overrides tgz paths below. If a fixture ever has pnpm.overrides pointing to local tarballs but no direct @sentry/ dependencies (e.g., only transitive ones), the override paths would silently go stale after a version bump, causing pnpm install to fail on missing .tgz files.
Additional Locations (1)
scripts/bump-version.js#L95-L103

if it doesn't have any dependencies, it wont have any overrides 🙈

chargome · 2026-03-26T09:16:51Z

+for (const dir of directories) {
+ try {
+ const packageJson = await fs.readFile(join(dir, "package.json"), { encoding: "utf-8" });
+ if (packageJson.length < 50) {


Can you add a comment on this magic number? :D

I'll do this a more sensible way without magic!

@chargome

@chargome

timfish · 2026-03-25T16:48:56Z

This PR:

Removes the test fixtures from the yarn workspace
Adds a setup script that does an pnpm install for each fixture
Uses pnpm overrides to pickup the local packed packages
Updates the version bump script so the tgz paths in the overrides get correctly updated
Moves the @sentry/cli patching from patch-package to pnpm built in patchedDependencies

github-actions · 2026-03-25T16:49:13Z

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 🔧

Isolate integration test package installs by timfish in #902

Pin GitHub Actions to full-length commit SHAs by joshuarli in #900
Rollup integration tests by timfish in #897
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.}

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

cursor · 2026-03-25T17:57:53Z

+ const pkg = readJson(pkgPath);
+ if (!pkg.dependencies) {
+ continue;
+ }


Early return skips pnpm overrides path update
Low Severity
The !pkg.dependencies guard on line 86 causes bumpTestFixtureVersions to continue to the next fixture, which also skips updating the pnpm.overrides tgz paths below. If a fixture ever has pnpm.overrides pointing to local tarballs but no direct @sentry/ dependencies (e.g., only transitive ones), the override paths would silently go stale after a version bump, causing pnpm install to fail on missing .tgz files.
Additional Locations (1)
scripts/bump-version.js#L95-L103

if it doesn't have any dependencies, it wont have any overrides 🙈

chargome · 2026-03-26T09:16:51Z

+for (const dir of directories) {
+ try {
+ const packageJson = await fs.readFile(join(dir, "package.json"), { encoding: "utf-8" });
+ if (packageJson.length < 50) {


Can you add a comment on this magic number? :D

I'll do this a more sensible way without magic!

@chargome

@chargome

timfish force-pushed the timfish/test/isolate-package-installs branch 3 times, most recently from 79bf93a to 0ba91acCompare March 25, 2026 17:19

timfish marked this pull request as ready for review March 25, 2026 17:33

timfish requested review from Lms24 and chargome March 25, 2026 17:34

timfish mentioned this pull request Mar 25, 2026
test: Vite integration tests #899
Merged

cursorBot reviewed Mar 25, 2026
View reviewed changes

Comment threadpackages/webpack-plugin/package.json Outdated

test: Isolate integration test package installs
9a0a117

timfish force-pushed the timfish/test/isolate-package-installs branch from 0ba91ac to 9a0a117Compare March 25, 2026 17:51

cursorBot reviewed Mar 25, 2026
View reviewed changes

chargome approved these changes Mar 26, 2026
View reviewed changes

Remove magic pkgJson length
474b750

timfish enabled auto-merge (squash) March 26, 2026 10:54

timfish merged commit f57805a into mainMar 26, 2026
33 checks passed

timfish deleted the timfish/test/isolate-package-installs branch March 26, 2026 10:55

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 force-pushed the timfish/test/isolate-package-installs branch 3 times, most recently from 79bf93a to 0ba91acCompare March 25, 2026 17:19

timfish marked this pull request as ready for review March 25, 2026 17:33

timfish requested review from Lms24 and chargome March 25, 2026 17:34

timfish mentioned this pull request Mar 25, 2026
test: Vite integration tests #899
Merged

cursorBot reviewed Mar 25, 2026
View reviewed changes

Comment threadpackages/webpack-plugin/package.json Outdated

test: Isolate integration test package installs
9a0a117

timfish force-pushed the timfish/test/isolate-package-installs branch from 0ba91ac to 9a0a117Compare March 25, 2026 17:51

cursorBot reviewed Mar 25, 2026
View reviewed changes

chargome approved these changes Mar 26, 2026
View reviewed changes

Remove magic pkgJson length
474b750

timfish enabled auto-merge (squash) March 26, 2026 10:54

timfish merged commit f57805a into mainMar 26, 2026
33 checks passed

timfish deleted the timfish/test/isolate-package-installs branch March 26, 2026 10:55

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 force-pushed the timfish/test/isolate-package-installs branch 3 times, most recently from 79bf93a to 0ba91acCompare March 25, 2026 17:19

timfish marked this pull request as ready for review March 25, 2026 17:33

timfish requested review from Lms24 and chargome March 25, 2026 17:34

timfish mentioned this pull request Mar 25, 2026
test: Vite integration tests #899
Merged

cursorBot reviewed Mar 25, 2026
View reviewed changes

Comment threadpackages/webpack-plugin/package.json Outdated

test: Isolate integration test package installs
9a0a117

timfish force-pushed the timfish/test/isolate-package-installs branch from 0ba91ac to 9a0a117Compare March 25, 2026 17:51

cursorBot reviewed Mar 25, 2026
View reviewed changes

chargome approved these changes Mar 26, 2026
View reviewed changes

Remove magic pkgJson length
474b750

timfish enabled auto-merge (squash) March 26, 2026 10:54

timfish merged commit f57805a into mainMar 26, 2026
33 checks passed

timfish deleted the timfish/test/isolate-package-installs branch March 26, 2026 10:55

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 force-pushed the timfish/test/isolate-package-installs branch 3 times, most recently from 79bf93a to 0ba91acCompare March 25, 2026 17:19

timfish marked this pull request as ready for review March 25, 2026 17:33

timfish requested review from Lms24 and chargome March 25, 2026 17:34

timfish mentioned this pull request Mar 25, 2026
test: Vite integration tests #899
Merged

cursorBot reviewed Mar 25, 2026
View reviewed changes

Comment threadpackages/webpack-plugin/package.json Outdated

test: Isolate integration test package installs
9a0a117

timfish force-pushed the timfish/test/isolate-package-installs branch from 0ba91ac to 9a0a117Compare March 25, 2026 17:51

cursorBot reviewed Mar 25, 2026
View reviewed changes

chargome approved these changes Mar 26, 2026
View reviewed changes

Remove magic pkgJson length
474b750

timfish enabled auto-merge (squash) March 26, 2026 10:54

timfish merged commit f57805a into mainMar 26, 2026
33 checks passed

timfish deleted the timfish/test/isolate-package-installs branch March 26, 2026 10:55

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 force-pushed the timfish/test/isolate-package-installs branch 3 times, most recently from 79bf93a to 0ba91acCompare March 25, 2026 17:19

timfish marked this pull request as ready for review March 25, 2026 17:33

timfish requested review from Lms24 and chargome March 25, 2026 17:34

timfish mentioned this pull request Mar 25, 2026
test: Vite integration tests #899
Merged

cursorBot reviewed Mar 25, 2026
View reviewed changes

Comment threadpackages/webpack-plugin/package.json Outdated

test: Isolate integration test package installs
9a0a117

timfish force-pushed the timfish/test/isolate-package-installs branch from 0ba91ac to 9a0a117Compare March 25, 2026 17:51

cursorBot reviewed Mar 25, 2026
View reviewed changes

chargome approved these changes Mar 26, 2026
View reviewed changes

Remove magic pkgJson length
474b750

timfish enabled auto-merge (squash) March 26, 2026 10:54

timfish merged commit f57805a into mainMar 26, 2026
33 checks passed

timfish deleted the timfish/test/isolate-package-installs branch March 26, 2026 10:55

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 force-pushed the timfish/test/isolate-package-installs branch 3 times, most recently from 79bf93a to 0ba91acCompare March 25, 2026 17:19

timfish marked this pull request as ready for review March 25, 2026 17:33

timfish requested review from Lms24 and chargome March 25, 2026 17:34

timfish mentioned this pull request Mar 25, 2026
test: Vite integration tests #899
Merged

cursorBot reviewed Mar 25, 2026
View reviewed changes

Comment threadpackages/webpack-plugin/package.json Outdated

test: Isolate integration test package installs
9a0a117

timfish force-pushed the timfish/test/isolate-package-installs branch from 0ba91ac to 9a0a117Compare March 25, 2026 17:51

cursorBot reviewed Mar 25, 2026
View reviewed changes

chargome approved these changes Mar 26, 2026
View reviewed changes

Remove magic pkgJson length
474b750

timfish enabled auto-merge (squash) March 26, 2026 10:54

timfish merged commit f57805a into mainMar 26, 2026
33 checks passed

timfish deleted the timfish/test/isolate-package-installs branch March 26, 2026 10:55

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 force-pushed the timfish/test/isolate-package-installs branch 3 times, most recently from 79bf93a to 0ba91acCompare March 25, 2026 17:19

timfish marked this pull request as ready for review March 25, 2026 17:33

timfish requested review from Lms24 and chargome March 25, 2026 17:34

timfish mentioned this pull request Mar 25, 2026
test: Vite integration tests #899
Merged

cursorBot reviewed Mar 25, 2026
View reviewed changes

Comment threadpackages/webpack-plugin/package.json Outdated

test: Isolate integration test package installs
9a0a117

timfish force-pushed the timfish/test/isolate-package-installs branch from 0ba91ac to 9a0a117Compare March 25, 2026 17:51

cursorBot reviewed Mar 25, 2026
View reviewed changes

chargome approved these changes Mar 26, 2026
View reviewed changes

Remove magic pkgJson length
474b750

timfish enabled auto-merge (squash) March 26, 2026 10:54

timfish merged commit f57805a into mainMar 26, 2026
33 checks passed

timfish deleted the timfish/test/isolate-package-installs branch March 26, 2026 10:55

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 force-pushed the timfish/test/isolate-package-installs branch 3 times, most recently from 79bf93a to 0ba91acCompare March 25, 2026 17:19

timfish marked this pull request as ready for review March 25, 2026 17:33

timfish requested review from Lms24 and chargome March 25, 2026 17:34

timfish mentioned this pull request Mar 25, 2026
test: Vite integration tests #899
Merged

cursorBot reviewed Mar 25, 2026
View reviewed changes

Comment threadpackages/webpack-plugin/package.json Outdated

test: Isolate integration test package installs
9a0a117

timfish force-pushed the timfish/test/isolate-package-installs branch from 0ba91ac to 9a0a117Compare March 25, 2026 17:51

cursorBot reviewed Mar 25, 2026
View reviewed changes

chargome approved these changes Mar 26, 2026
View reviewed changes

Remove magic pkgJson length
474b750

timfish enabled auto-merge (squash) March 26, 2026 10:54

timfish merged commit f57805a into mainMar 26, 2026
33 checks passed

timfish deleted the timfish/test/isolate-package-installs branch March 26, 2026 10:55

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 Mar 25, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actionsBot commented Mar 25, 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!

Uh oh!

cursorBot left a comment

Choose a reason for hiding this comment

Uh oh!

cursorBotMar 25, 2026

Choose a reason for hiding this comment

Early return skips pnpm overrides path update

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!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

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

Uh oh!

Uh oh!

github-actionsBot commented Mar 25, 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!

Uh oh!

cursorBot left a comment

Choose a reason for hiding this comment

Uh oh!

cursorBotMar 25, 2026

Choose a reason for hiding this comment

Early return skips pnpm overrides path update

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!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

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

Uh oh!

Uh oh!

github-actionsBot commented Mar 25, 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!

Uh oh!

timfish commented Mar 25, 2026•
edited
Loading

github-actionsBot commented Mar 25, 2026•
edited
Loading

timfish commented Mar 25, 2026•
edited
Loading

github-actionsBot commented Mar 25, 2026•
edited
Loading

timfish commented Mar 25, 2026•
edited
Loading

github-actionsBot commented Mar 25, 2026•
edited
Loading

timfish commented Mar 25, 2026•
edited
Loading

github-actionsBot commented Mar 25, 2026•
edited
Loading