refactor(types): let plugins declare their stats keys - #22849

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

⚙️ Run configuration

• Source mode: Auto
• Repository: webpack/webpack
• Branch: refactor/stats-plugin-value
• File: .coderabbit.yaml
• Commit: d92633d
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 2a716c85-42b4-47ab-8bc4-ecc683792091

📥 Commits

Reviewing files that changed from the base of the PR and between bf84a6f and d92633d.

⛔ Files ignored due to path filters: 1

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

📒 Files selected for processing: 3

• lib/index.js
• lib/stats/DefaultStatsFactoryPlugin.js
• lib/stats/StatsFactory.js

🚧 Files skipped from review as they are similar to previous changes: 1

• lib/stats/DefaultStatsFactoryPlugin.js

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

The type generator now supports augmentable declarations and keyof types. StatsFactory and StatsPrinter hook maps use augmentable registries for plugin-defined keys. Default stats plugins derive configuration types from these registries.

Changes: Stats plugin type augmentation

Layer / File(s)	Summary
Generate augmentable declarations tooling/generate-types.js, eslint.config.mjs, cspell.json	The type generator parses and emits augmentable interfaces inside the entry namespace and renders keyof types. The JSDoc configuration and spell-checker word list recognize augmentable.
StatsFactory registries and hook maps lib/stats/StatsFactory.js, lib/stats/DefaultStatsFactoryPlugin.js, lib/index.js	StatsFactory defines augmentable key registries and types hook maps against registered keys. Open string-key overloads use StatsPluginValue. Array-path results use a registered merged-result type when specified, or the created-item array otherwise. DefaultStatsFactoryPlugin derives hook configuration and registration types from these mappings and defines group types that lib/index.js re-exports.
StatsPrinter registries and hook maps lib/stats/StatsPrinter.js, lib/stats/DefaultStatsPrinterPlugin.js	StatsPrinter defines augmentable print, item, and item-name registries. Its hook maps use registered keys for specific types and StatsPluginValue for open string-key overloads. DefaultStatsPrinterPlugin derives relevant item types from these registries.

Suggested labels: area: types

Suggested reviewers: avivkeller

Merge Risk: Minimal · previous review d9263

The reviewed type exports and hook typing show no identified blocker; the change is ready for normal merge checks.

Pre-merge checks: 3 · 1

Failed checks: 1 inconclusive

Check name	Status	Explanation	Resolution
Title check		The title uses valid Conventional Commit syntax, uses the allowed type refactor, and accurately describes the plugin stats-key changes. The branch prefix is not provided, so the required type-to-bra…	Provide the branch name or confirm that its prefix is refactor.

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, uses the allowed type refactor, and accurately describes the plugin stats-key changes. The branch prefix is not provided, so the required type-to-branch-prefix match cannot be verified.

Resolution

Provide the branch name or confirm that its prefix is refactor.

🛠️ Fix failing CI checks: 💡

• Commit to this branch
• Create a new PR

• X
• Mastodon
• Reddit
• LinkedIn

⚠️ 117 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

ℹ️ 44 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   #22849   +/-   ##
=======================================
  Coverage   96.95%   96.95%           
=======================================
  Files         795      795           
  Lines      150726   150727    +1     
  Branches    52823    52823           
=======================================
+ Hits       146138   146139    +1     
  Misses       4588     4588           

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.