fix: parse struct references for type generation

[dominictwlee]

this should fix the issues in v2.4.0 where reference type structs were breaking return types in Typescript. I think the issue stems from valueType capturing what should've been part of the valuePackage group because of ".+".

This should remedy the concerns raised in #2490 around stripping the asterix. In case we need it in the future, we could capture the asterix in a new group. I've added it to the non-capturing group for now.

@OlegGulevskyy @leaanthony let me know what you think.

I've also added a few additional cases to cover promise bindings.

closes #2489

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
wails-v3-docs	❌ Failed (Inspect)			Apr 2, 2023 8:57am

[dominictwlee]

@leaanthony bump, just following up on this to get a review, thanks!

[zllovesuki]

👍 this looks way better

[leaanthony]

👍 this looks way better

Please LMK if you test this and it fixes your issue 👍

[dominictwlee]

https://github.com/dominictwlee/test-ts-gen

@zllovesuki minimal example. clone my fork, uncomment go.mod replace, point to local wails dir and you're good to go.

TLDR;
https://github.com/dominictwlee/test-ts-gen/blob/main/app.go#L33 <= go method
https://github.com/dominictwlee/test-ts-gen/blob/main/frontend/wailsjs/go/main/App.d.ts#L5 <= generated d.ts

[dominictwlee]

@leaanthony thanks for the review. Anything else you need from me to get this merged in?

[AlbinoDrought]

I was updating from 2.3.1 to 2.4.1 to fix the CreateWebResourceResponse issue and experienced this Promise<any> issue, I can confirm this PR fixes it for me 🎊

[leaanthony]

Thanks for this @dominictwlee. It would be good if you could update changelog.mdx under the website directory with your changes 👍

[leaanthony]

Thanks so much for this 🙏