Skip to content

speed up bid discovery for intents#674

Merged
Wizdave97 merged 27 commits into
mainfrom
dami/speed-up-discovery
Mar 6, 2026
Merged

speed up bid discovery for intents#674
Wizdave97 merged 27 commits into
mainfrom
dami/speed-up-discovery

Conversation

@dharjeezy

@dharjeezy dharjeezy commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

Adds an RPC server that watches the transaction pool for place_bid and retract_bid extrinsics, making bids queryable before block inclusion.

Changes

  • New pallet-intents-rpc crate RPC handler with in-memory bid cache
  • Introduces the bid watcher and cleanup tasks in the node service

RPC Methods

  • intents_getBidsForOrder(commitment) which query current bids for an order
  • intents_subscribeBids(commitment?) which subscribe to bid notifications, optionally filtered

closes #658

@dharjeezy dharjeezy marked this pull request as draft March 3, 2026 07:47
Comment thread modules/pallets/intents-coprocessor/rpc/src/lib.rs Outdated
Comment thread modules/pallets/intents-coprocessor/rpc/src/lib.rs Outdated
Comment thread modules/pallets/intents-coprocessor/rpc/src/lib.rs Outdated
Comment thread modules/pallets/intents-coprocessor/rpc/src/lib.rs Outdated
group bids by commitment,
tests
@dharjeezy dharjeezy marked this pull request as ready for review March 4, 2026 10:03
Comment thread parachain/runtimes/gargantua/src/lib.rs Outdated
Comment thread parachain/runtimes/gargantua/src/lib.rs Outdated
Comment thread modules/pallets/intents-coprocessor/rpc/src/lib.rs Outdated
Comment thread modules/pallets/intents-coprocessor/rpc/src/lib.rs Outdated
Comment thread modules/pallets/intents-coprocessor/rpc/src/lib.rs Outdated
Comment thread modules/pallets/intents-coprocessor/rpc/src/lib.rs Outdated
Comment thread modules/pallets/intents-coprocessor/rpc/src/lib.rs Outdated
Comment thread modules/pallets/intents-coprocessor/rpc/src/lib.rs Outdated
@dharjeezy dharjeezy requested a review from Wizdave97 March 4, 2026 16:26
Comment thread modules/pallets/intents-coprocessor/rpc/src/lib.rs Outdated
Comment thread modules/pallets/intents-coprocessor/rpc/src/lib.rs Outdated
Comment thread modules/pallets/intents-coprocessor/runtime-api/src/lib.rs Outdated
Comment thread parachain/node/src/command.rs Outdated
use BTreeSet,
move extract_bid into the rpc
Comment thread modules/pallets/intents-coprocessor/runtime-api/Cargo.toml Outdated
Comment thread modules/pallets/intents-coprocessor/rpc/src/lib.rs Outdated
Comment thread modules/pallets/intents-coprocessor/src/lib.rs
Comment thread parachain/node/src/command.rs Outdated
Comment thread parachain/node/src/command.rs Outdated
@dharjeezy

Copy link
Copy Markdown
Contributor Author

LGTM i think @Wizdave97 has a few unresolved comments, also bump the hyperbridge node verison

I have already gone ahead to apply his suggestions.

@Wizdave97 Wizdave97 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, @dharjeezy let's compare the governance calls in the pallet intent coprocessor to the corresponding message handlers in the solidity contract and update anything that's outdated.

@dharjeezy dharjeezy requested a review from Wizdave97 March 5, 2026 15:58
Comment thread modules/pallets/intents-coprocessor/src/types.rs Outdated
@dharjeezy dharjeezy requested a review from seunlanlege March 5, 2026 16:22

@seunlanlege seunlanlege left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why is CI broken?

@dharjeezy

Copy link
Copy Markdown
Contributor Author

why is CI broken?

something to do with foundry failing to compile due to dependency confliction.

@dharjeezy dharjeezy force-pushed the dami/speed-up-discovery branch from bb0c835 to e075f48 Compare March 5, 2026 23:15
@Wizdave97 Wizdave97 merged commit 7ff2464 into main Mar 6, 2026
0 of 10 checks passed
@Wizdave97 Wizdave97 deleted the dami/speed-up-discovery branch March 6, 2026 05:35
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.

[Intents] Speed up bid discovery

3 participants