Description

The code at packages/core/src/hash/blake2b256.ts enforces the normalization form for canonical composition of hashed strings and introduce the method blake2b256OfHex to hash bytes expressed as hexadecimal. Documentation reviewed for sake of clarity.

Tests at packages/core/tests/hash/blake2b256.unit.test.ts introduce teh canonical test pattern with one test case per test to speed-up debugging. The code suggests tests to check the compatibility with thor-dev-kit but such code is commented out because the introduction of `thor-dev-kit dependency clashes with bignumber.js dependency.

Fixes # 949

Type of change

Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] 'yarn test:solo`

Test Configuration:

Node.js Version: v20.14.0
Yarn Version: 1.22.22

Checklist:

[x] My code follows the coding standards of this project
[x] I have performed a self-review of my code
[x] I have commented on my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] New and existing integration tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have not added any vulnerable dependencies to my code

Title	Tests	Time
core	494	1m 3s :stopwatch:
network	658	3m 51s :stopwatch:
errors	48	11.008s :stopwatch:

vechain / vechain-sdk-js

fix: 949 blake2b256 compatibility with ethers / thor-dev-kip wip #960

Description

Fixes # 949

Type of change

How Has This Been Tested?

Checklist:

Test Coverage

Summary