Skip to content

test: Add esbuild integration tests#911

Open
timfish wants to merge 5 commits intomainfrom
timfish/test/esbuild
Open

test: Add esbuild integration tests#911
timfish wants to merge 5 commits intomainfrom
timfish/test/esbuild

Conversation

@timfish
Copy link
Copy Markdown
Collaborator

@timfish timfish commented Apr 5, 2026

Add esbuild as the final bundler in the integration tests.

It appears that esbuild ends up with unique debug IDs per platform so these are replaced before snapshot capture.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

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 ✨

  • (core) Pass mapDir to rewriteSourcesHook by chargome in #908
  • Use crypto.randomUUID rather than uuid by timfish in #892

Bug Fixes 🐛

  • (core) Conditionally add tracing headers by chargome in #907
  • (e2e-tests) Pin axios to 1.13.5 to avoid compromised 1.14.1 by andreiborza in #906
  • Add missing webpack5 entrypoint in webpack-plugin by brunodccarvalho in #905

Internal Changes 🔧

  • (deps) Bump vulnerable webpack version by chargome in #909
  • Add esbuild integration tests by timfish in #911
  • Vite integration tests by timfish in #899
  • Webpack integration tests by timfish in #904
  • 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.

@timfish timfish marked this pull request as ready for review April 5, 2026 21:36
cursor[bot]

This comment was marked as outdated.

@timfish timfish force-pushed the timfish/test/esbuild branch from 5a73ad4 to dc58c4a Compare April 5, 2026 21:52
cursor[bot]

This comment was marked as outdated.

@timfish timfish requested a review from chargome April 5, 2026 22:06
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.

1 participant