Release latest version on NPM?

[Taytay]

It looks like SQL.js was improved to use the latest version of SQLite3 and emscripten recently, but the NPM package hasn't been updated since March 24th. Any chance you could publish the latest version so we could give it a go? Thanks!

[lovasoa]

Of course 👍 Let's do that

[lovasoa]

🙄 Tests don't pass! I just git pull and npm test, and there is a worker test that fails !

[dinedal]

@lovasoa what node version? I'm not able to repro?

[lovasoa]

Yes, and I see that travis-CI successfully passes all tests too.

Node v6.2.2
The test that fails is the one with BLOBs inside web workers.

    ✗ Reading BLOB
      Expected: 
    [
      0,
      66
    ]
      Actual: 
    [
    
    ]

(test/test_worker.js, line 23)

[lovasoa]

Yes, the problem is related to the node version: https://travis-ci.org/kripken/sql.js/builds/177810598

[dinedal]

@lovasoa https://github.com/kripken/sql.js/pull/121 is still the issue.

I've looked into this for a few days. I've come to the conclusion we would need to do a non trivial amount of work to worker.js to fix it the way proposed there.

[lovasoa]

Maybe we should use an other implementation of web workers ? https://www.npmjs.com/package/webworker-threads for instance ?
I'll try that.

[lovasoa]

Unfortunately, it doesn't work either.

[dinedal]

Yeah, the implementation of "the structured clone algorithm in Javascript as a separate project" is where we would need to start...

That is, as long as we aren't open to the idea of forking / creating our own modified workerjs that patches this issue in a custom way.

[lovasoa]

Isn't there any fully working javascript serializer out there ?

[dinedal]

The ones that I found had difficulty dealing with Blob / UInt8Array types, but perhaps I didn't try enough of them.

[lovasoa]

I think I found one: https://www.npmjs.com/package/bson
See: https://runkit.com/580f350772ebca0014ce054e/58338521cf2933001455b100

Forking workerjs to use bson shouldn't be too hard...

[dinedal]

I'll take it on next chance I get, unless you want to go for it