feat: yield.xyz POC #11578

gomesalexandre · 2026-01-06T01:12:50Z

jk, disregard the below this is a PoC
~~Just digging around and starting exploration with minimax/opus here. Implementation plan really isn't one, more like references of sorts, augmented by actual curls with test API key:~~

Master plan:

Macro plan:

coderabbitai · 2026-01-06T01:12:56Z

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch feat_yield

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

- Add Phase 1.0 TypeScript types with full API enums (TransactionType, ActionType, YieldNetwork, etc.) - Add Phase 1.0.1 network/chainId mapping utilities (CAIP-2 <-> Yield.xyz network) - Document all API response structures from actual testing - Include parsing utilities for JSON string fields (unsignedTransaction, gasEstimate) - Define request/response types for all endpoints

- Add VITE_YIELD_XYZ_API_KEY, VITE_YIELD_XYZ_BASE_URL, VITE_FEATURE_YIELD_XYZ env vars - Add YieldXyz feature flag to preferencesSlice and test mocks - Add CSP for api.yield.xyz and assets.stakek.it - Create types.ts with API response types (no derived types) - Create constants.ts with CHAIN_ID_TO_YIELD_NETWORK mapping - Create utils.ts with parsing and conversion utilities - Create api.ts with all Yield.xyz API endpoints - Create React Query hooks (useYields, useYield, useYieldBalances, useEnterYield, useExitYield, useSubmitYieldTransaction)

- Filter out SKIPPED and CREATED transactions in YieldActionModal before building steps array and execution loop, preventing phantom approval steps - Update YieldStats to use AugmentedYieldDto instead of YieldDto for type safety - Fix YieldEnterExit dependency arrays (use exitBalance instead of withdrawableBalance) - Switch YieldYourInfo formatters to use shared formatLargeNumber helper - Remove unused inputTokenBalance variable from YieldYourInfo - Fix invalid bgOpacity prop in YieldPositionCard (use bg='color.900' pattern) - Remove unused Button import from YieldCard - Add formatLargeNumber utility for consistent number formatting

The yield-xyz provider logoURI from the API returns 403, so we override it with a locally hosted PNG using react-query's select option.

- Add YieldFilters component with network/provider filtering and sort options - Add transaction confirmation polling before marking tx as success - Remove redundant tags display from YieldCard and table - Add pagination params to getProviders API - Show ResultsEmptyNoWallet for positions tab when disconnected

The wip commits broke AssetIcon's discriminated union types and added a proxy asset creation that caused double icon rendering. Reverted to original and updated Yields components to use only src prop.

gomesalexandre · 2026-01-07T01:10:34Z

Ok maybe that is a bit more than an exploration PR.

gomesalexandre · 2026-01-07T01:27:36Z

docs/fixes/yields-table-sorting-fix.md

can revert already, fixed

gomesalexandre · 2026-01-07T01:27:48Z

docs/yield_xyz_asset_section.md

revert, now useless

gomesalexandre · 2026-01-07T01:28:00Z

docs/yield_xyz_fees_plan.md

revert, captured as an issue, all done thru dashboard

gomesalexandre · 2026-01-07T01:28:18Z

src/assets/translations/en/main.json

sanity-check no useless ones here

gomesalexandre · 2026-01-07T01:28:40Z

src/components/Layout/Header/Header.tsx

triple-check this is feature-flag gated

gomesalexandre · 2026-01-07T01:28:53Z

src/hooks/useActionCenterSubscribers/useGenericTransactionSubscriber.tsx

Triple-check this one, seems flaky

gomesalexandre · 2026-01-07T01:29:12Z

src/lib/utils/formatters.ts

hmmm yeah no

gomesalexandre · 2026-01-07T01:29:48Z

src/lib/yieldxyz/api.ts

+  'Content-Type': 'application/json',
+}
+
+const handleResponse = async <T>(response: Response): Promise<T> => {


Seems sloppy - should be handled by axios

gomesalexandre · 2026-01-07T01:29:59Z

src/lib/yieldxyz/api.ts

+    if (params?.limit) searchParams.set('limit', String(params.limit))
+    if (params?.offset) searchParams.set('offset', String(params.offset))
+
+    const response = await fetch(`${BASE_URL}/yields?${searchParams}`, { headers })


axios vs. fetch

gomesalexandre · 2026-01-07T01:30:52Z

src/lib/yieldxyz/augment.ts

Augments pure response with bits we need - clear separation between server response and our augmented DTOs

Also:

few squirly braces we can remove here

tokenToAssetId seems flaky, clean it up

bnOrZero vs. Number things

no eip155:${evmChainId} as ChainId - use toChainId()

naming

gomesalexandre · 2026-01-07T01:33:25Z

src/lib/yieldxyz/executeTransaction.ts

+import { assertGetSuiChainAdapter } from '@/lib/utils/sui'
+import { isStakingChainAdapter } from '@/plugins/cosmos/components/modals/Staking/StakingCommon'
+
+type ParsedEvmTransaction = {


types should live in a types.ts

gomesalexandre · 2026-01-07T01:34:05Z

src/lib/yieldxyz/transaction.ts

+ * Parse the JSON string unsignedTransaction from Yield.xyz API
+ */
+export const parseUnsignedTransaction = (tx: TransactionDto): ParsedUnsignedTransaction => {
+  if (typeof tx.unsignedTransaction === 'string') {


Seems... flaky.

gomesalexandre · 2026-01-07T01:34:56Z

src/lib/yieldxyz/types.ts

+/**
+ * Yield.xyz API Types
+ * These types are derived from actual API responses.
+ * DO NOT add derived/composite types - only what the API returns.


Maybe worth a diff naming for this one then, so other types can live in types.ts

gomesalexandre · 2026-01-07T01:35:34Z

src/lib/yieldxyz/utils.ts

+} from './constants'
+import type { TransactionDto, YieldDto, YieldNetwork } from './types'
+
+export const chainIdToYieldNetwork = (chainId: ChainId): YieldNetwork | undefined =>


Not a constant but should probably be colocated in constants.ts, ditto below

gomesalexandre · 2026-01-07T01:35:55Z

src/lib/yieldxyz/utils.ts

+export const filterSupportedYields = (yields: YieldDto[]): YieldDto[] =>
+  yields.filter(y => isSupportedYieldNetwork(y.network))
+
+export type ParsedUnsignedTransaction = {


ditto types.ts

gomesalexandre · 2026-01-07T01:36:02Z

src/lib/yieldxyz/utils.ts

+}
+
+export const parseUnsignedTransaction = (tx: TransactionDto): ParsedUnsignedTransaction => {
+  if (typeof tx.unsignedTransaction === 'string') {


ditto flaky

src/config.ts

- Add CHAIN_NAMESPACE.Solana case to executeTransaction switch - Implement executeSolanaTransaction that decompiles yield.xyz transactions, filters compute budget instructions, and rebuilds with fresh blockhash - Fix amount format for Solana - yield.xyz expects human-readable SOL, not lamports - Add minimum compute unit buffer (50k) for staking operations

…e types - Show YieldActivePositions on account page (was only on asset page) - Sum all balance types (active, entering, exiting, withdrawable) for total value display instead of just active balance

Query Key Fixes: - useEnterYield: invalidate ['yieldxyz', 'balances', yieldId, address] + allBalances - useExitYield: invalidate ['yieldxyz', 'balances', yieldId, address] + allBalances - useSubmitYieldTransactionHash: accept optional yieldId/address params, invalidate properly - YieldActionModal: remove inline invalidations, pass yieldId/address to mutation The original code had incorrect query key patterns that caused cache invalidation to fail. Mutations were invalidating partial keys that didn't match the actual query keys, leading to stale data after transactions. Bug Fixes: - useYieldOpportunities: add guardrail throw when multiAccountEnabled without accountId - Header: gate Yields nav item behind YieldXyz feature flag Chores: - augment.ts: rename evmChainIdFromString → parseEvmNetworkId, use toChainId() validation - types.ts: rename evmChainId → evmNetworkId for clarity - Remove unused useQueryClient from YieldActionModal

[skip ci] feat: yield.xyz exploration

6f4de0d

gomesalexandre added 18 commits January 6, 2026 02:55

feat: wip

cce008e

wip: wip

988df70

fix: shit

374ce85

feat: wip

412a26f

wip: wip

bdee0d8

fix: patch yield-xyz provider logo with local image

07f2d57

The yield-xyz provider logoURI from the API returns 403, so we override it with a locally hosted PNG using react-query's select option.

fix: revert AssetIcon changes that caused nested icon bug

659dd71

The wip commits broke AssetIcon's discriminated union types and added a proxy asset creation that caused double icon rendering. Reverted to original and updated Yields components to use only src prop.

feat: carrot

5e74f1f

feat: wip

544b651

feat: visual bits

1cf9fd1

[skip ci] wip: disgusting ai spew wip

072cc5e

[skip ci] fix: sui staking tx execution and yield detail icon

8862b52

[skip ci] fix: revert to base unit amount for yield api

8b93520

[skip ci] wip: sui minimum deposit

2f96d10

gomesalexandre changed the title ~~feat: yield.xyz exploration~~ feat: yield.xyz ~~exploration~~ POC Jan 7, 2026

gomesalexandre changed the title ~~feat: yield.xyz ~~exploration~~ POC~~ feat: yield.xyz POC Jan 7, 2026