Skip to content

Remove orca provider#158

Merged
0xh3rman merged 5 commits intomainfrom
h/remove-orca-provider
Apr 29, 2026
Merged

Remove orca provider#158
0xh3rman merged 5 commits intomainfrom
h/remove-orca-provider

Conversation

@0xh3rman
Copy link
Copy Markdown
Collaborator

@0xh3rman 0xh3rman commented Apr 29, 2026

Summary

Removes the Orca provider from core-ts ahead of moving that implementation to Rust, applies the OKX DEX gas-limit guidance for EVM swaps that require approval, and folds in the currently open dependency update PRs.

Changes

  • Removed Orca from the API provider registry and swapper package exports.
  • Deleted the Orca provider implementation, route data model, fee helper, integration tests, and unit tests.
  • Removed Orca-specific quote fixtures while leaving generated SwapProvider.ts unchanged.
  • Dropped the Orca SDK dependencies and now-unused Solana kit/token helper dependencies from @gemwallet/swapper.
  • Trimmed Solana helper code that was only used by the removed Orca provider.
  • Updated the provider benchmark default away from Orca.
  • Buffered OKX-provided EVM tx.gas by 50% when returning a provider gas limit for approval-backed swaps.
  • Folded in dependency update PRs Bump oxlint from 1.61.0 to 1.62.0 #151, Bump @types/node from 25.5.0 to 25.6.0 #152, Bump dotenv from 17.3.1 to 17.4.2 #153, Bump oxfmt from 0.46.0 to 0.47.0 #155, and Bump knip from 6.3.0 to 6.6.2 #156:
    • oxlint 1.58.0 -> 1.61.0
    • @types/node 25.5.0 -> 25.6.0
    • dotenv 17.3.1 -> 17.4.2
    • oxfmt 0.43.0 -> 0.46.0
    • knip 6.3.0 -> 6.6.2

Validation

  • pnpm build
  • pnpm test
  • pnpm lint
  • pnpm dedupe --check
  • pnpm install --frozen-lockfile --ignore-scripts
  • git diff --check
  • pnpm --filter @gemwallet/swapper test -- okx/provider.test.ts
  • pnpm --filter @gemwallet/swapper build
  • pnpm --filter @gemwallet/types test
  • pnpm --filter @gemwallet/types build
  • Targeted oxfmt --check for touched source files

Notes

The remaining OKX references to Orca are OKX aggregator route metadata, not the removed core-ts Orca provider.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes the Orca Whirlpool provider from the swapper package, including its implementation, models, fee logic, and associated tests. Several Solana-related dependencies and utility functions were also removed, and the benchmarking scripts were updated to use stonfi_v2 as the default. Feedback was provided regarding an unused constant in the Solana transaction builder that should be removed as it is now dead code.

Comment thread packages/swapper/src/chain/solana/tx_builder.ts
@0xh3rman 0xh3rman changed the title [codex] remove orca provider Remove orca provider Apr 29, 2026
@0xh3rman 0xh3rman marked this pull request as ready for review April 29, 2026 07:35
@0xh3rman 0xh3rman merged commit e843ae3 into main Apr 29, 2026
4 checks passed
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