feat: add compilation.hooks.chunkHashDependencies - #22810

⚙️ Run configuration

• Source mode: Auto
• Repository: webpack/webpack
• Branch: feat/chunk-hash-dependencies
• File: .coderabbit.yaml
• Commit: a1866dd
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 2077bd9e-3611-4a6f-abac-c12217d327c6

📥 Commits

Reviewing files that changed from the base of the PR and between 90a7e89 and a1866dd.

⛔ Files ignored due to path filters: 1

• types.d.ts is excluded by !types.d.ts

📒 Files selected for processing: 4

• lib/Compilation.js
• lib/javascript/CommonJsChunkFormatPlugin.js
• lib/template/RuntimeTemplate.js
• test/unitCases/javascript/parser.unittest.js

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 7 remain after this review.

Compilation adds a hook for declaring chunk-hash dependencies. It uses declared dependencies to order chunk hashing and include referenced chunk hashes. Chunk-format integrations and configuration and watch-cache tests cover dependency ordering, warnings, and emitted asset references.

Changes: Chunk Hash Dependency Processing

Layer / File(s)	Summary
Dependency tracking and hash ordering .changeset/020-chunk-hash-dependencies-hook.md, lib/Compilation.js	Compilation adds and caches the dependency hook, computes chunk order across four hash rounds, reports ordering warnings, and propagates full-hash requirements.
Chunk formats and hash consumers lib/esm/ModuleChunkFormatPlugin.js, lib/javascript/*, lib/template/RuntimeTemplate.js	ESM and CommonJS formats register hash dependencies. JavaScript chunk hashing includes dependency hashes, and RuntimeTemplate uses the computed chunk order.
Ordering and warning configuration tests test/configCases/hash/chunk-hash-dependencies-order/*, test/configCases/hash/chunk-hash-dependencies-warnings/*, test/unitCases/javascript/parser.unittest.js	Tests check dependency-aware ordering, compare runtime-template order with hash order, and cover warnings for circular and later-round dependencies. The parser unit test adds a ParserOptions typedef.
Watch-cache declared dependency test test/watchCases/cache/chunk-hash-dependencies-plugin/*	The test checks that a filename written into a chunk from a declared dependency exists among the current compilation’s assets.
Watch-cache hashed asset reference tests test/watchCases/cache/chunk-kinds-imported-hash/*, test/watchCases/cache/entry-full-hash-runtime/*	Tests check that emitted JavaScript, CSS, HTML, module, and runtime references resolve to assets in the current compilation across multiple compiler settings.

Possibly related PRs

• webpack/webpack#22851: Both changes add the ParserOptions type declaration in the same parser unit test.
• webpack/webpack#22825: Both changes add parser option type declarations in the same parser unit test.
• webpack/webpack#22876: Both changes add the ParserOptions type declaration in the same parser unit test.

Suggested reviewers: hai-x

No identified issue currently blocks merging after normal checks.

Pre-merge checks: 3 · 1

Failed checks: 1 inconclusive

Check name	Status	Explanation	Resolution
Title check		The title uses valid Conventional Commit syntax with the allowed type feat and clearly describes the main change. The branch prefix is not provided, so the required type-prefix match cannot be verif…	Provide the branch name or confirm that its prefix is feat.}]} ува? Wait JSON malformed due missing quote? Need exact valid. Also response format says resolution string. Let's formulate. Ensure no stray. Output only JSON. encased proper. …

Passed checks: 3 passed

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Title check

Explanation

The title uses valid Conventional Commit syntax with the allowed type feat and clearly describes the main change. The branch prefix is not provided, so the required type-prefix match cannot be verified.

Resolution

Provide the branch name or confirm that its prefix is feat.}]} ува? Wait JSON malformed due missing quote? Need exact valid. Also response format says resolution string. Let's formulate. Ensure no stray. Output only JSON. encased proper. [

• X
• Mastodon
• Reddit
• LinkedIn

⚠️ 111 benchmarks are dominated by system calls

System calls cannot be simulated consistently and are left out of the measure. The reported value understates the real cost. Switch to the Walltime instrument to measure them.

Learn more about system calls

See affected benchmarks

⚠️ Base and head ran on different runner types

Some benchmarks with significant performance changes were compared across different runner types, so the changes might not come from the code. Run base and head on the same runner type, or use CodSpeed Macro Runners for more stable results.

Learn more about Macro Runners

Open the report in CodSpeed to investigate

ℹ️ 46 benchmarks have no flamegraph

The flamegraph wasn't generated: most of the time was spent in runtime internals, which would leave nothing useful to analyze.

See affected benchmarks

@@             Coverage Diff             @@
##             main   #22810       +/-   ##
===========================================
+ Coverage   77.15%   96.54%   +19.39%     
===========================================
  Files         601      778      +177     
  Lines      133153   141164     +8011     
  Branches    46098    48311     +2213     
===========================================
+ Hits       102733   136289    +33556     
+ Misses      30420     4875    -25545     

Flags with carried forward coverage won't be shown. Click here to find out more.

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.