AssemblyScript Loader

[marcus-sa]

I've been trying this project out for a few days, and I must say, usage wise I'm pretty confused.
First of all there's no NPM package for https://github.com/AssemblyScript/assemblyscript/tree/master/lib/loader available as shown in the docs that there should be.

assemblyscript/lib/loader/index.d.ts

Line 48 in 671121b

export declare function instantiateStreaming<T extends {}>(response: Response, imports?: ImportsObject): Promise<ASUtil & T>;

Should be Promise<Response> and not Response.

assemblyscript/lib/loader/index.d.ts

Line 6 in 671121b

env: {

Should be optional.

We need some examples, because I'm really confused on how some things should be done.
E.g using module.getString and module.newString, because when I try to log something from my AssemblyScript file all it logs is a number which makes sense.
But what are the purpose of these two functions?
The docs definitely needs to be updated.

[dcodeIO]

Thanks, I updated the declaration to Promise<Response>. Regarding documentation, there is an example here. Doesn't that explain the usage sufficiently?

Essentially, newString allocates a new string in WASM memory returning a pointer to it, while getString reads such a segment to a JS string when provided with its pointer.

[marcus-sa]

Surely it does, but you'd have to wait for the initialized stream to complete before the module is available, otherwise it'll throw undefined. So I'd have to end up wrapping the stream in a callback like this and wait for it to finish before defining the imports.
https://github.com/marcus-sa/assemblyscript-starter/blob/master/public/main.ts

[dcodeIO]

Well, you could also use await to simplify this

// (async function() {
  var myModule = await ASLoader.instantiateStreaming(...);
  ...
// })();
...

or make use of the Promise as usual:

ASLoader.instantiateStreaming(...).then(myModule => {
  ...
});

[marcus-sa]

Already tried that, sadly it doesn't work.

[MaxGraey]

In node.js instantiateStreaming not working yet. But it should on latest Firefox, Chrome, Safari and Edge

[dcodeIO]

Closing this issue for now as it hasn't received any replies recently. Feel free to reopen it if necessary!