Skip to content

test: keep rspack eco ci passing after mangle output change#1574

Merged
Timeless0911 merged 1 commit intomainfrom
fix/core-rspack-eco-ci-mangle-output
Apr 3, 2026
Merged

test: keep rspack eco ci passing after mangle output change#1574
Timeless0911 merged 1 commit intomainfrom
fix/core-rspack-eco-ci-mangle-output

Conversation

@Timeless0911
Copy link
Copy Markdown
Contributor

Summary

Relax the node polyfill integration assertion so it accepts the mangled property name emitted by Rspack.

This keeps rspack eco CI passing after web-infra-dev/rspack#13588 changed the generated output during mangle.

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@Timeless0911 Timeless0911 changed the title test(core): keep rspack eco ci passing after mangle output change test: keep rspack eco ci passing after mangle output change Apr 3, 2026
@Timeless0911 Timeless0911 marked this pull request as ready for review April 3, 2026 09:01
Copilot AI review requested due to automatic review settings April 3, 2026 09:01
@Timeless0911 Timeless0911 enabled auto-merge (squash) April 3, 2026 09:01
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Relaxes the node-polyfill integration test assertion so it remains compatible with Rspack’s mangled output, keeping the rspack-eco CI green after upstream output changes.

Changes:

  • Update the Buffer import detection regex to accept a mangled property name when requiring buffer/index.js.

@Timeless0911 Timeless0911 merged commit 8742596 into main Apr 3, 2026
13 checks passed
@Timeless0911 Timeless0911 deleted the fix/core-rspack-eco-ci-mangle-output branch April 3, 2026 09:13
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.

3 participants