Skip to content

Conversation

@PinkChampagne17
Copy link
Contributor

  • This is a bugfix
  • This is a feature
  • This is a code refactor
  • This is a test update
  • This is a docs update
  • This is a metadata update

For Bugs and Features; did you add new tests?

Yes.

Motivation / Use-Case

Some code will throw non-error value (e.g., plain objects, strings, or primitives), which will be converted to an instance of Error with the message '[object Object]' and passed to runtimeErrors. This makes filtering errors difficult.

This PR enhances error handling by storing the original thrown value in the Error.cause property to access the raw thrown value for better error identification.

Breaking Changes

No.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jun 10, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: PinkChampagne17 / name: Pink Champagne (dfb0050)
  • ✅ login: alexander-akait / name: Alexander Akait (cff2809)

@codecov
Copy link

codecov bot commented Jun 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.53%. Comparing base (3f3d124) to head (cff2809).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5518      +/-   ##
==========================================
+ Coverage   83.42%   83.53%   +0.11%     
==========================================
  Files          13       13              
  Lines        2033     2035       +2     
  Branches      749      750       +1     
==========================================
+ Hits         1696     1700       +4     
+ Misses        303      301       -2     
  Partials       34       34              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ 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.

@PinkChampagne17
Copy link
Contributor Author

Corrected the test case to maintain coverage.

@alexander-akait alexander-akait merged commit 37b033d into webpack:master Jul 16, 2025
33 of 53 checks passed
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