fix(predict): update Polymarket adapter contract addresses#29573
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
These addresses are used in SmokePredictions is the primary tag - these contract addresses are directly used in Polymarket prediction market transactions. The tests cover the full position lifecycle which relies on these adapter addresses. SmokeConfirmations is required per SmokePredictions tag description: "opening/closing positions are on-chain transactions" - confirmations are part of the flow. SmokeWalletPlatform is required per SmokePredictions tag description: "Predictions is also a section inside the Trending tab (SmokeWalletPlatform)" - changes to Predictions views affect Trending. No other tags are impacted - this is a Polymarket-specific change with no effect on accounts, swaps, networks, browser, snaps, or other features. Performance Test Selection: |
|
❌ The last analysis has failed. |
The stable→release/7.77.0 merge silently took stable's older Polymarket CtfCollateralAdapter / NegRiskCtfCollateralAdapter addresses, regressing fix #29573 that landed on release/7.77.0 (and main). Polymarket stopped accepting the old adapters after May 1, so without this revert the sync PR would break Polymarket relayed transactions on 7.77.0. Restores release/7.77.0's values: - CTF_COLLATERAL_ADAPTER_ADDRESS = 0xAdA100Db...218FcE1f - NEG_RISK_CTF_COLLATERAL_ADAPTER_ADDRESS = 0xadA20056...DB6eAab Co-authored-by: Cursor <cursoragent@cursor.com>
Description
Updates the Polymarket adapter contract addresses used by Predict to the new
CtfCollateralAdapterandNegRiskCtfCollateralAdaptervalues shared by Polymarket.Polymarket added new events on these adapters and will stop accepting relayed transactions through the old adapters on May 1 at 3pm UTC. Without this change, Polymarket transactions routed through the old adapters will start failing.
Changelog
CHANGELOG entry: Updated Polymarket adapter contracts so Polymarket prediction transactions continue working after the relayer migration.
Related issues
Fixes:
Manual testing steps
Not run locally; constants-only configuration update.
Screenshots/Recordings
Not applicable.
Before
Not applicable.
After
Not applicable.
Pre-merge author checklist
Performance checks (if applicable)
trace()for usage andaddTokenfor an exampleFor performance guidelines and tooling, see the Performance Guide.
Pre-merge reviewer checklist
Note
Medium Risk
Low code complexity, but changes production on-chain contract targets for Polymarket claim/relay flows; incorrect addresses would cause transactions to fail.
Overview
Updates Predict’s Polymarket configuration to use new
CtfCollateralAdapterandNegRiskCtfCollateralAdaptercontract addresses inpolymarket/constants.ts.This shifts v2 claim/relay routing (as referenced by protocol
claim.standardTarget/claim.negRiskTarget) to the new adapter contracts to maintain compatibility with Polymarket’s relayer migration.Reviewed by Cursor Bugbot for commit 08b08be. Bugbot is set up for automated code reviews on this repo. Configure here.