Missing typing for sqlite3Worker1Promiser

[realGriss]

Type

Bug Report

Description

Hi,

I still get typing problems in the current version "3.51.2-build7". Trying to import with

import { sqlite3Worker1Promiser } from '@sqlite.org/sqlite-wasm'

gives me an error. This should be fixed with #54 Or am I missing something?

[jurerotar]

Hey, thank you for raising this! 😄

I'll take a look and keep you updated!

[jurerotar]

Sorry about this! I've prepared a PR (#156) that should resolve this issue. I've also touched a bit on the types, specifically Worker1Promiser type is now exported, so you'll be able to use it for explicitly typing your variable, should you need to, and specific calls like config-get should now no longer require redundant second arguments 😄

[mmomtchev]

The Node.js version (dist/node.mjs) does not export it. I think this is the problem.

[tomayac]

Fixed by #156.

[jurerotar]

@realGriss, new release was just published, please let me know if the issue is solved! 😄

[realGriss]

Types seems to work fine now. Thanks for the quick update.

The only problem I'm running into is that I get a error with the current example:

import {
  sqlite3Worker1Promiser,
  type Worker1Promiser,
} from '@sqlite.org/sqlite-wasm';

const log = console.log;
const error = console.error;

const initializeSQLite = async () => {
  try {
    log('Loading and initializing SQLite3 module...');

    const promiser = await new Promise<Worker1Promiser>((resolve) => {
      sqlite3Worker1Promiser({
        onready: resolve,
      });
    });

    log('Done initializing. Running demo...');

    const configResponse = await promiser('config-get');
    log('Running SQLite3 version', configResponse.result.version.libVersion);

    const openResponse = await promiser('open', {
      filename: 'file:mydb.sqlite3?vfs=opfs',
    });
    const { dbId } = openResponse;
    log(
      'OPFS is available, created persisted database at',
      openResponse.result.filename.replace(/^file:(.*?)\?vfs=opfs$/, '$1'),
    );
    // Your SQLite code here.
  } catch (err) {
    if (!(err instanceof Error)) {
      err = new Error(err.result.message);
    }
    error(err.name, err.message);
  }
};

await initializeSQLite();

TypeError: Cannot create property 'dbId' on string 'config-get'

Not sure if I'm doing something wrong here or this is a bug.

Edit:
It works if I use it like this:

promiser('config-get', {});

[jurerotar]

Not sure if I'm doing something wrong here or this is a bug

This is on me, being overzealous with the type improvements. Promiser API makes an explicit check for amount of parameters and assigns properties based on it. I missed that, sorry!

I don't want to bother tomayac with preparing yet another build, so I'll just prepare a PR that updates the example and makes sure the types require all required params from now on 😄