Blake3Hasher is not a constructor when using in the worker threads

[sureshvenkata12]

When using Blake3Hasher in the worker threads, I get the following error:

TypeError [Error]: blake_hash_1.Blake3Hasher is not a constructor

[Brooooooklyn]

Please provide the reproduce codes

[Brooooooklyn]

I've tested @napi-rs/blake-hash@1.3.0 with node:16.13.1 and macOS:12.0.1 x64, everything work fine:

index.js

const { Worker } = require('worker_threads')

const worker = new Worker('./worker.js')

worker.on('message', (message) => {
  console.log(message)
})

worker.js

const { parentPort } = require('worker_threads')

const { Blake3Hasher } = require('@napi-rs/blake-hash')

const hasher = new Blake3Hasher()
hasher.update('hello')

parentPort.postMessage(hasher.digest('hex'))

[Brooooooklyn]

Tested on debian:9 with node:14.18.2, also works fine @sureshvenkata12

[Brooooooklyn]

@sureshvenkata12 do you have any codes or git branch I can debug?

[Brooooooklyn]

In ironfish/build/src/workerPool/tasks/mineHeader.js, the return type of require('@napi-rs/blake-hash-darwin-x64') became {}, an empty object. I don't know why because the start process of ironfish is too complex.

[sureshvenkata12]

Do you have any idea what might be the issue? I will look into it deeper if you have any pointers. I've tried napi-rs/canvas package in the worker, it worked well.

[Brooooooklyn]

@sureshvenkata12 thanks for your investigation, I'll dig into this issue when I have time.

[Brooooooklyn]

@sureshvenkata12 Please try the https://github.com/Brooooooklyn/blake-hash/releases/tag/v1.3.1. It should fix this bug.

[dguenther]

v1.3.1 works based on my testing, thanks for the update!

[sureshvenkata12]

@dguenther btw, how did you find out me trying "blake-hash" for ironfish? I guess, you will incorporate this into the main, and I will not have any advantage over others now :)

[dguenther]

It was just a coincidence! We wanted to test a native addon replacement for blake3 to address WASM out of memory issues.

[Brooooooklyn]

@dguenther You can consider https://napi.rs/ to replace neon in ironfish too. It provides more functionality and is easier to use.