add support for exported classes about as-bind

[mathe42]

Or instead of wrapping useing a proxy.

[mathe42]

Got a small working prototype (all manual but can be automated):

Assemblyscript

export class Foo {
  constructor(public str: string) {}

  getString(): string {
    return this.str
  }
}

Host:

import * as AsBind from "/node_modules/as-bind/dist/as-bind.esm.js";

const asyncTask = async () => {
  const wasm = await fetch("/build/optimized.wasm").then(v => v.arrayBuffer());

  const asBindInstance = await AsBind.instantiate(wasm);

  let e = asBindInstance.exports

  // When not supported noop
  const pointerRegistry = FinalizationRegistry ? new FinalizationRegistry(ptr => {
    e.__unpin(ptr)
  }) : {register(){}};

  function createProxyClass(klass, definition) {
    const newConstructor = new Proxy(klass, {
      construct(target, args) {
        // TODO: wrap args - get from definition
        const ptr = new target(...args)
        return newConstructor.wrap(ptr)
      },
      get(_, prop) {
        if (prop === 'wrap') {
          return (ptr) => {
            e.__pin(ptr)

            const instance = klass.wrap(ptr)

            const a = new Proxy({}, {
              ownKeys() {
                return Reflect.ownKeys(instance);
              },
              defineProperty() {
                throw new Error("Not allowed!")
              },
              deleteProperty() {
                throw new Error("Not allowed!")
              },
              get(_, ...args) {
                if (prop === '__collect') {
                  return () => e.__unpin(ptr)
                }
                const value = Reflect.get(instance, ...args)
                // TODO: wrap function - or wrap value
                return value
              },
              set(_, ...args) {
                // TODO: check if is settable; wrap value if needed
                return Reflect.set(instance, ...args)
              },
              setPrototypeOf() {
                throw new Error("Not allowed!")
              },
              preventExtensions() {
                throw new Error("Not allowed!")
              },
              isExtensible() {
                return false;
              },
              has(_, p) {
                return Reflect.has(instance, p)
              },
              getPrototypeOf() {
                throw new Error("Not allowed!")
              }
            })

            // handle GC
            pointerRegistry.register(a, ptr)

            return a
          }
        }
        return undefined
      },
      set() {
        throw new Error("Not allowed!")
      }
    })

    return newConstructor
  }


  window.e = e
  window.Foo = createProxyClass(e.Foo, {
    constructor: {
      memberType: "constructor",
      parameterTypes: ["~lib/string/String"],
      returnType: null
    },
    str: {
      memberType: "field",
      set: true,
      get: true,
      type: "~lib/string/String"
    },
    getString: {
      memberType: "function",
      parameterTypes: [],
      returnType: "~lib/string/String"
    }
  })

  // example usage
  const sptr = e.__newString("test")
  const f = new Foo(sptr)
  console.log(e.__getString(f.getString())) // "test"
  const sptr2 = e.__newString("test 42")
  f.str = sptr2
  console.log(e.__getString(f.getString())) // "test 42"
};
asyncTask();

[mathe42]

@torch2424 If this looks good to you I will integrate it. :D

[torch2424]

@mathe42 Thank you very much for drafting up this implementation! 😄 Yeah! I like this approach and it makes sense to me, so let's go for it! 😄