Skip to content

Conversation

@alexander-akait
Copy link
Member

What kind of change does this PR introduce?

Feature

Did you add tests for your changes?

Yes

Does this PR introduce a breaking change?

No

What needs to be documented once your changes are merged?

Yes, new asset modules type

@alexander-akait
Copy link
Member Author

@styfle Here is support with { type: "bytes }, so webpack supports this construction out of box and with extra options

@codspeed-hq
Copy link

codspeed-hq bot commented Sep 23, 2025

CodSpeed Performance Report

Merging #19928 will degrade performances by 27.5%

Comparing feat-asset-bytes (f3829f6) with main (c647cf1)

Summary

⚡ 1 improvement
❌ 1 regression
✅ 40 untouched

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark BASE HEAD Change
benchmark "devtool-source-map", scenario '{"name":"mode-development-rebuild","mode":"development","watch":true}' 84.7 ms 116.8 ms -27.5%
benchmark "many-modules-commonjs", scenario '{"name":"mode-development-rebuild","mode":"development","watch":true}' 167.5 ms 129.1 ms +29.75%

@alexander-akait alexander-akait merged commit f6c3c24 into main Sep 24, 2025
41 of 44 checks passed
@alexander-akait alexander-akait deleted the feat-asset-bytes branch September 24, 2025 18:01
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.

2 participants