crypto: fix unhandled error in Hash._transform

[haramj]

Description

This PR fixes a bug where errors occurring during Hash.prototype.write() were silently swallowed instead of being surfaced.

Problem

In lib/internal/crypto/hash.js, the _transform implementation did not check the return value of this[kHandle].update(). While the synchronous update() method correctly throws ERR_CRYPTO_HASH_UPDATE_FAILED on failure, the streaming interface continued execution even if the underlying ncrypto binding returned false. This resulted in incorrect hash outputs without any error notification to the user.

Solution

Updated Hash.prototype._transform to check the return value of the internal update() call. If it returns false, the error is passed to the stream callback, triggering an 'error' event as expected for a stream.Transform instance.

Validation

• Added a regression test test/parallel/test-crypto-hash-stream-error.js.
• The test mocks the internal handle using kHandle to simulate an update failure and verifies that an 'error' event with ERR_CRYPTO_HASH_UPDATE_FAILED is emitted.
• Verified that the test fails without this change and passes with it.
• Verified that existing crypto tests (test/parallel/test-crypto-hash.js) still pass.

Checklist

• make lint-js passes
• make test-parallel (relevant to crypto) passes
• Commit message follows the Node.js guidelines

Fixes: #63258

[nodejs-github-bot]

Review requested:

• @nodejs/crypto

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.04%. Comparing base (78bbee3) to head (931b4bc).
⚠️ Report is 345 commits behind head on main.

Additional details and impacted files

@@ Coverage Diff @@## main #63261 +/- ##
==========================================
- Coverage 90.04% 90.04% -0.01% 
==========================================
Files 713 713 Lines 225003 225004 +1 Branches 42536 42539 +3 ==========================================
- Hits 202606 202603 -3 - Misses 14177 14183 +6 + Partials 8220 8218 -2 

Files with missing lines	Coverage Δ
lib/internal/crypto/hash.js	99.01% <100.00%> (+<0.01%)	⬆️

... and 30 files with indirect coverage changes

🚀 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.

[haramj]

Hi @nodejs/crypto team, just a gentle ping on this PR. All CI checks have passed, and it addresses a silent error swallowing bug in the Hash stream interface. I'd appreciate it if someone could take a look when you have a moment. Thanks

[haramj]

Hi, @nodejs/crypto !
could another member please take a look at this PR when you have some time?

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/74120/

[nodejs-github-bot]

Landed in 120c2a4