Skip to content

feat: Add beforeModuleIds hook#13223

Merged
hardfist merged 4 commits intoweb-infra-dev:v1.xfrom
hamlim:hamlim/backport-before-module-ids-hook
Mar 6, 2026
Merged

feat: Add beforeModuleIds hook#13223
hardfist merged 4 commits intoweb-infra-dev:v1.xfrom
hamlim:hamlim/backport-before-module-ids-hook

Conversation

@hamlim
Copy link
Copy Markdown
Contributor

@hamlim hamlim commented Mar 5, 2026

Summary

This change backports the added beforeModuleIds hook to the v1 release branch, specifically it backports both of these PRs:

Which adds support for the beforeModuleIds hook to rspack, allowing JS access to customize module ids within a build, specifically allowing for both strings and numbers as valid values for the module id.

Related links

See above mentioned PRs.

Checklist

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

@hamlim hamlim requested a review from hardfist as a code owner March 5, 2026 18:45
@hamlim hamlim changed the title chore: Add beforeModuleIds hook chore: Add beforeModuleIds hook Mar 5, 2026
@hamlim hamlim mentioned this pull request Mar 5, 2026
7 tasks
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Mar 5, 2026

Merging this PR will not alter performance

✅ 16 untouched benchmarks
⏩ 3 skipped benchmarks1


Comparing hamlim:hamlim/backport-before-module-ids-hook (892195a) with main (153d0f2)2

Open in CodSpeed

Footnotes

  1. 3 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

  2. No successful run was found on v1.x (ff45adf) during the generation of this report, so main (153d0f2) was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@hardfist hardfist changed the title chore: Add beforeModuleIds hook feat: Add beforeModuleIds hook Mar 6, 2026
@github-actions github-actions bot added the release: feature release: feature related release(mr only) label Mar 6, 2026
@hardfist hardfist merged commit 748247a into web-infra-dev:v1.x Mar 6, 2026
56 checks passed
@hamlim
Copy link
Copy Markdown
Contributor Author

hamlim commented Mar 9, 2026

@hardfist Would it be possible for another 1.7.x release (or 1.8) to be cut with this change included?

@hardfist
Copy link
Copy Markdown
Contributor

@hamlim 1.7.8 is released which includes this feature

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: feature release: feature related release(mr only)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants