Skip to content

Release - #730

Open
rhinestone-automations[bot] wants to merge 3 commits into
releasefrom
main
Open

Release#730
rhinestone-automations[bot] wants to merge 3 commits into
releasefrom
main

Conversation

@rhinestone-automations

Copy link
Copy Markdown
Contributor

Promotes releasable changes from main to release after a successful @dev publish.

Destiner and others added 3 commits August 4, 2026 17:10
ci: streamline release verification
…10) (#729)

`tokenRequests[].balance` ('spot' | 'perp') selects which HyperCore balance a
delivery credits — the recipient's spot wallet or the default perp dex's margin
account. The SDK never declared it and rebuilt token requests from named fields
in two separate places, so the field was dropped before the network call and no
consumer could deliver to spot even deliberately; every caller silently took the
orchestrator's 'perp' default. Verified on dev 2026-08-05, where intents asking
for spot completed, filled, and credited perp margin with nothing reporting it.

Declared on both the EVM and non-EVM token-request arms and forwarded through
`adaptTransaction` and `buildIntentRequest`. The non-EVM arm is the one that
matters: `hyperCoreMainnet` is a `NonEvmChain`, because the SDK splits EVM from
non-EVM by CAIP-2 namespace rather than by address shape — HyperCore addresses
are EVM-shaped but `hypercore:` is not `eip155:`. `HyperCoreBalance` is exported
from the package root, and a type-level assertion in `test/types` pins that a
`targetChain: hyperCoreMainnet` request with a venue compiles, since the failure
mode was a compile error no runtime test could catch.

Pairs with rhinestonewtf/orchestrator#1825, which makes the venue required.
@rhinestone-automations rhinestone-automations Bot added the ship Agent approval is enough; merge when CI green label Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ship Agent approval is enough; merge when CI green

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants