refactor: sdk rhp

[alexfreska]

• Updated sdk to latest core changes, updated structure.
• Follow up will migrate walletd methods into sdk, use sdk in walletd.

[changeset-bot]

🦋 Changeset detected

Latest commit: 8d4eb6a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@siafoundation/sdk	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[alexfreska]

• walletd testing #561
• feat: walletd api changes and app updates #532
• refactor: migrate wallet wasm into sdk #520
• refactor: sdk rhp #519 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alexfreska and the rest of your teammates on Graphite

[n8maninger]

I'm not sure I believe the rhp and wallet functions deserve their own package. It requires creating small utility packages (api, utils, marshal) to share javascript utilities.

If we do split them up into distinct packages, I don't think those packages should be "js-aware" and instead have concrete arguments and return values rather than js.Value and api.Result.

[alexfreska]

I'm not sure I believe the rhp and wallet functions deserve their own package. It requires creating small utility packages (api, utils, marshal) to share javascript utilities.

If we do split them up into distinct packages, I don't think those packages should be "js-aware" and instead have concrete arguments and return values rather than js.Value and api.Result.

@n8maninger I moved everything into main 👍🏼

[ChrisSchinnerl]

I think after addressing Nate's comments this is good to go

[alexfreska]

I think after addressing Nate's comments this is good to go

👍🏼 gtg