Incorrect emitted TS declaration

Hi! Thanks for the great library!

Since CodeGen was merged, I switched to the emitted declaration files instead of using manually created ones based on the source code, but the issue is, the nesting is wrong!

Right now the declarations are emitted like this:

export declare const dotnet: {
    BannerlordModuleManager: { }; // The C# bindings
    BootStatus: typeof BootStatus;
    ...
};

But dotnet.js exposes the variable via exports.BannerlordModuleManager instead of exports.dotnet.BannerlordModuleManager So the correct declaration would be this:

export declare const BannerlordModuleManager: { }; // The C# bindings
export declare const dotnet: {
    BootStatus: typeof BootStatus;
    ...
};

PS: Writing it here because Discussions aren’t setup. Since we both are from Russia, I can’t setup PayPal donations via Ko-Fi because Russia<–>Russia transactions are banned. Do you know any way to bypass the restriction?

[Aragas]

Sure, such test should be enough to cover TS generation! And yea, the package emits everything correctly! Closing the issue, thanks!

[Elringus]

Makes sense. Not sure if this one test justifies converting all the suite to TS, though. Maybe comparing the generated file with a reference is enough: https://github.com/Elringus/DotNetJS/blob/04dd07c4964541de7761075d769e983be1de21d9/JavaScript/dotnet-runtime/test/packed.js#L40

Anyway, latest nuget package should produce correct types now. Please let me know if it solves the issue.