Skip to content

Rr/additional testnet warp routes#59

Open
Roee-87 wants to merge 4 commits into
testnet-deploymentfrom
rr/additional-testnet-warp-routes
Open

Rr/additional testnet warp routes#59
Roee-87 wants to merge 4 commits into
testnet-deploymentfrom
rr/additional-testnet-warp-routes

Conversation

@Roee-87

@Roee-87 Roee-87 commented May 6, 2026

Copy link
Copy Markdown
Collaborator

This PR adds four warp routes to be deployed to Aleo Testnet:

  • test_hyp_warp_token_usdc.aleo
  • test_hyp_warp_token_usdt.aleo
  • test_hyp_warp_token_sol.aleo
  • test_hyp_warp_token_wbtc.aleo

These four routes include modifications to transfer_remote and transfer_remote_with_hook which replaces self.signer with self.caller along with adding dedicated transfer remote methods that invoke self.signer. The remainder of the existing warp routes will be upgraded with these changes.

Note that test_hyp_warp_token_eth.aleo is already deployed to testnet but was not part of the repository.

@Roee-87 Roee-87 self-assigned this May 6, 2026
@troykessler

Copy link
Copy Markdown
Collaborator

Two differences from the mainnet implementation in #58:

1. Dead signer parameter in finalize_transfer_remote

All 5 programs accept signer: address as a parameter but never reference it in the function body:

async function finalize_transfer_remote(..., signer: address) {
    let actual_remote_router = remote_routers.get(destination);
    assert_eq(actual_remote_router, unverified_remote_router);
    // signer is never used
    dispatch_future.await();
    token_future.await();
}

The mainnet programs in #58 don't have this parameter at all.

2. transfer_remote (caller-based, default hook) is missing from 4 of 5 programs

test_hyp_warp_token_eth has all 4 transitions:

  • transfer_remote — caller-based, default hook
  • transfer_remote_as_signer — signer-based, default hook
  • transfer_remote_with_hook — caller-based, custom hook
  • transfer_remote_with_hook_as — signer-based, custom hook

But test_hyp_warp_token_sol, test_hyp_warp_token_usdc, test_hyp_warp_token_usdt, and test_hyp_warp_token_wbtc are missing transfer_remote, only having 3 transitions. This doesn't match the pattern established in #58 where all programs have the full set of 4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants