Skip to content

Conversation

@mike-diamond
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Jan 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
vault-interface Ready Ready Preview, Comment Jan 13, 2026 1:10pm

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR refactors the swap functionality by consolidating and modernizing the swap-related hooks and components. The changes centralize swap logic into a new hooks/swap module, remove outdated swap implementations from the stake actions, and update dependencies to support the new swap architecture.

Changes:

  • Consolidated swap hooks into a centralized hooks/swap module with improved SDK integration
  • Removed redundant swap-related code from stake actions and replaced with centralized swap hooks
  • Updated transaction flow to eliminate the SetNextTransactionsFailed callback in favor of a simpler setTransaction API
  • Added new store slices for vault validators, user roles, and allocator actions

Reviewed changes

Copilot reviewed 102 out of 111 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/hooks/swap/* New centralized swap hooks module with token management, quotes, fees, and actions
src/views/SwapView/util/vault/actions/useStake/swap/* Removed old swap implementation files
src/views/SwapView/util/vault/actions/useStake/* Updated to use centralized swap hooks instead of local implementations
src/views/SwapView/util/vault/actions/useBoost/* Simplified transaction error handling
src/views/SwapView/util/vault/actions/useUnboost/* Simplified transaction error handling
src/components/Transactions/util/useLogic.ts Simplified transaction state management API
src/store/store/* Added new store slices for validators, roles, and actions
package.json Added @cowprotocol/sdk-ethers-v6-adapter dependency
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
dfkadyr
dfkadyr previously approved these changes Jan 13, 2026
Signed-off-by: MikeDiam <info@mdiamond.ru>
Signed-off-by: MikeDiam <info@mdiamond.ru>
@mike-diamond mike-diamond merged commit 873f242 into main Jan 13, 2026
3 checks passed
@mike-diamond mike-diamond deleted the swap-updates branch January 13, 2026 13:47
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.

4 participants