feature request: Support promises

[michael-ts]

Looks like node-sqlite3 has a wrapper module called node-sqlite which provides promises. However, it seems the hard-code the sqlite3 module. Since cross-sqlcipher is ultimately based on node-sqlite3, maybe some of this effort could be re-used?

[fritx]

@michael-ts hi, you mean the promise wrapper has to hard-code some C++ parts?

I used to think that promise wrapper is only some JS wrapper around the library --
for example, we can write a library to wrap all the APIs of it.

[fritx]

Sorry for the misunderstanding if exists, I meant I totally agree the feature request.

And my suggestion is to make another library that based on cross-sqlcipher, and provide promise support, is it right?

[michael-ts]

I mean, in the node-sqlite code it looks like it might do exactly what is wanted, except it is requiring 'sqlite3' instead of 'cross-sqlcipher'. Maybe there is some easy way to "trick" it into requiring 'cross-sqlcipher' instead? If not, I suppose a fork that simply does a string replace would work, but then somebody has to make the effort to detect and sync changes any time node-sqlite is updated.

[fritx]

OK got it! Hmm... since the cross-sqlcipher is made to be API-compatible with node-sqlite3,
I suggest that we just set an alias would be a best way!

In web-frontend, we already had some webpack settings to do it (alias),
in node, I guess we can use module-alias or require-alias.

See also: Better local require() paths for Node.js

I was recently doing the exact same thing in my node server stuff!

[fritx]

@michael-ts oops, seems I was wrong, 'cause cross-sqlcipher itself depends on node-sqlite3 too,
so an alias won't be the solution.

[fritx]

Now what I want to do is to provide a PR to node-sqlite,
which supports a custom sqlite3 argument just like the Promise arg to custom the sqlite3 source.