Skip to content

fix: use new exchange addresses - #2

Merged
ibruno merged 1 commit into
mainfrom
fix/use-new-exchange-addresses
Jun 19, 2026
Merged

fix: use new exchange addresses#2
ibruno merged 1 commit into
mainfrom
fix/use-new-exchange-addresses

Conversation

@ibruno

@ibruno ibruno commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Summary by cubic

Updated Amoy exchange contract addresses so the order builder signs against the latest deployments. Ensures signatures verify on-chain and prevents failed orders on Amoy.

  • Bug Fixes

    • Adjusted test expectations (struct hashes and signatures) to reflect the new EIP-712 domain from the updated contracts.
    • Affected APIs: order builder signature generation (build_order_signature, build_signed_order). No method or runtime changes; outputs differ on Amoy due to the new domain.
    • Performance: no impact.
  • Migration

    • Update any Amoy config to use:
      • exchange: 0xaCd95F4F42322c7bE215C170362EEc57Ef4E78c2
      • negRiskExchange: 0x961d3230B3BBdb2592D20fa34dBD12Fa19240603

Written for commit a3e9e46. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Re-trigger cubic

@ibruno
ibruno merged commit 92f49e8 into main Jun 19, 2026
2 checks passed
@ibruno
ibruno deleted the fix/use-new-exchange-addresses branch June 19, 2026 12:36
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.

1 participant