Skip to content

Conversation

@JakeLane
Copy link
Contributor

Motivation

Provide relevant context and motivation behind the change, if any

Changes

Describe the solution and changes you have made

Checklist

  • Existing or new tests cover this change
  • There is a changeset for this change, or one is not required
  • Added documentation for any new features to the docs/ folder

@changeset-bot
Copy link

changeset-bot bot commented Oct 31, 2025

⚠️ No Changeset found

Latest commit: 3498e78

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

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

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Oct 31, 2025

📊 Type Coverage Report

Coverage Comparison

Metric Baseline Current Change
Coverage Percentage 92.04% 92.04% ➡️ 0.00%
Correctly Typed 200,529 201,405 +876
Total Expressions 217,870 218,818 +948
Untyped Expressions 17,341 17,413 +72

Files with Most Type Issues (Top 15)

File Issues Affected Lines
packages/core/integration-tests/test/javascript.ts 983 648
packages/core/integration-tests/test/cache.ts 885 626
packages/core/integration-tests/test/scope-hoisting.ts 623 490
packages/utils/node-resolver-core/test/resolver.ts 476 177
packages/core/integration-tests/test/html.ts 468 294
packages/core/integration-tests/test/sourcemaps.ts 356 176
packages/core/test-utils/src/utils.ts 330 205
packages/core/integration-tests/test/incremental-bundling.ts 298 206
packages/core/core/src/dumpGraphToGraphViz.ts 251 108
packages/core/integration-tests/test/output-formats.ts 227 161
packages/transformers/webextension/src/WebExtensionTransformer.ts 210 80
packages/core/integration-tests/test/react-refresh.ts 197 71
packages/core/integration-tests/test/css-modules.ts 191 107
packages/core/core/src/requests/TargetRequest.ts 190 133
packages/core/integration-tests/test/babel.ts 187 115

This report was generated by the Type Coverage GitHub Action

@github-actions
Copy link
Contributor

📊 Benchmark Results

🎉 Performance improvements detected!

📊 Benchmark Results

Overall Performance

Test Duration JS Memory Peak Native Memory Peak vs Baseline Status
Three.js Real Repository 18.76s 1.25GB 1.32GB -1.25% duration, -23.24% JS memory 🟢 Improvement

🔍 Detailed Phase Analysis

Three.js Real Repository

Phase Duration (avg) Duration (p95) Memory Peak (avg) Memory Peak (p95)
resolving 4.82s 5.03s 964.07MB 1.04GB
transforming 4.81s 5.03s 966.55MB 1.04GB
bundling 4.69s 4.93s 968.29MB 1.04GB
bundled 4.43s 4.70s 1019.20MB 1.08GB
packaging 3.17s 3.38s 1.02GB 1.11GB
optimizing 3.03s 3.19s 1.25GB 1.32GB

💾 Unified Memory Analysis

Three.js Real Repository Memory Statistics

Memory Type Metric Min Mean Median P95 P99 Max Std Dev
JavaScript RSS 827.49MB 1.08GB 1.16GB 1.32GB 1.32GB 1.32GB 186.88MB
Heap Used 48.44MB 50.77MB 50.80MB 53.55MB 53.55MB 53.55MB 1.50MB
Heap Total 91.93MB 94.76MB 95.68MB 96.68MB 96.68MB 96.68MB 1.67MB
External 61.86MB 73.62MB 79.45MB 97.05MB 97.05MB 97.05MB 13.11MB
Native (Rust) Physical Memory 899.66MB 1.05GB 1.04GB 1.13GB 1.26GB 1.32GB 66.81MB
Virtual Memory 28.94GB 29.60GB 29.59GB 29.69GB 29.85GB 29.90GB 106.59MB

Sample Counts: JS: 6, Native: 348

🖥️ Environment

  • Node.js: v22.17.0
  • Platform: linux (x64)
  • CPU: AMD EPYC 7763 64-Core Processor
  • Total Memory: 15.62GB
  • Git SHA: 3498e78
  • Branch: refactored-compiled-plugin
  • Timestamp: 2025-11-10T23:17:07.167Z

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