bug: must use the class type not the instance type

[yanick]

Potential solution for #194

Basically, what render gets is the class, not instances of that class.

Also, for the options, we were not dealing well with the shortcut of passing the props directly.

[ITenthusiasm]

Yeah so for some reason, typeof doesn't really play nicely when it comes to generics. I'm not sure if TypeScript gives an escape hatch for that. But what's currently here may not work.

[molily]

This doesn't work for me either, TS cannot parse this. If I understand correctly, the class definition should be something like

type SvelteComponentTypedClass<P, E, S> = new (options: SvelteComponentOptions<P>) => SvelteComponentTyped<P, E, S>;

See also svelte2tsx:
https://github.com/sveltejs/language-tools/blob/1415196df7118f4361385a6450ff2756bca3ff00/packages/svelte2tsx/svelte-shims.d.ts#L56-L76

(If I use this type definition I still get clashes with the svelte2tsx's generated component type definition, since it's target: Element | ShadowRoot from svelte2tsx and target?: HTMLElement in types/index.d.ts.)

[mcous]

@yanick since this PR was written, SvelteComponentTyped has been deprecated by Svelte. It looks like maybe some other things have changed, too.

I think the current typings that use ComponentConstructorOptions to accept constructor that returns a SvelteComponent instance are acceptable. This PR can probably be closed

[yanick]

Agreed!