feat(bridge): add solana chain support and improve bridge state management (#14713)

<!--
Please submit this PR as a draft initially.
Do not mark it as "Ready for review" until the template has been
completely filled out, and PR status checks have passed at least once.
-->

## **Description**

The PR focuses on:
1. Improving error handling and state management in the bridge feature
2. Adding support for Solana chain destination addresses
3. Simplifying the code by removing unnecessary state and effects
4. Improving the user experience with better keypad and layout handling

## Changes

### Bridge View Component Changes
- Simplified state management by removing `isWaitingForInitialQuote` and
related effects
- Improved error handling logic
- Added new state `isNoQuotesAvailable` to better handle quote
availability
- Refactored keypad display logic
- Improved layout and styling for dynamic content

###  Bridge Quote Request Hook Changes
- Removed try-catch block in `useBridgeQuoteRequest` for cleaner error
propagation
- Added support for Solana chain destination addresses
- Added `destWalletAddress` parameter that uses:
  - `destAddress` for Solana chains
  - `walletAddress` for other chains

### Bridge Quote Data Hook Changes
- Added `isNoQuotesAvailable` state to better track quote availability
- Updated tests to reflect new state handling
- Improved error state management

## **Related issues**

Fixes:
[MMS-1914](https://consensyssoftware.atlassian.net/browse/MMS-1914)
Fixes:
[MMS-1915](https://consensyssoftware.atlassian.net/browse/MMS-1915)

## **Manual testing steps**

1. Open Bridge and verify the layout spacing and padding look correct
2. Test keypad behavior:
   - Tap input field, verify keypad appears
   - Enter valid amount, verify keypad behavior
   - Clear input, verify keypad state
3. Test Solana quote fetching:
   - Select Solana as source chain, enter amount, check quote
   - Select any chain → Solana, enter amount, check quote
4. Test error handling:
   - Enter invalid amounts

## **Screenshots/Recordings**


https://github.com/user-attachments/assets/349010b4-6734-44f4-9fda-6d8e5ea4a8c4


## **Pre-merge author checklist**

- [x] I’ve followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile
Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [x] I've completed the PR template to the best of my ability
- [x] I’ve included tests if applicable
- [x] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [x] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

## **Pre-merge reviewer checklist**

- [ ] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.


[MMS-1914]:
https://consensyssoftware.atlassian.net/browse/MMS-1914?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
[MMS-1915]:
https://consensyssoftware.atlassian.net/browse/MMS-1915?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ

Author: amitabh94

CHANGELOG.md

@@ -17,6 +17,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Added
 
+- feat(bridge): add solana chain support and improve bridge state management ([#14713](https://github.com/MetaMask/metamask-mobile/pull/14713))
 - feat(bridge): add error handling and input management for bridge quotes ([#14693](https://github.com/MetaMask/metamask-mobile/pull/14693))
 - feat(multi-srp): enable multi-srp in main and beta ([#14558](https://github.com/MetaMask/metamask-mobile/pull/14558))
 - feat(ramp): Update ramp data flow to fetch cryptos before payment methods ([#14437](https://github.com/MetaMask/metamask-mobile/pull/14437))

app/components/UI/Bridge/Views/BridgeView/BridgeView.styles.ts

@@ -18,7 +18,6 @@ export const createStyles = (params: { theme: Theme }) => {
       width: '100%',
       paddingHorizontal: 16,
       paddingBottom: 24,
-
       gap: 12,
       alignItems: 'center',
       justifyContent: 'center',
@@ -59,7 +58,6 @@ export const createStyles = (params: { theme: Theme }) => {
     },
     quoteContainer: {
       paddingHorizontal: 24,
-      paddingVertical: 24,
     },
     keypadContainer: {
       flex: 1,
@@ -77,16 +75,18 @@ export const createStyles = (params: { theme: Theme }) => {
     },
     dynamicContent: {
       flex: 1,
-    },
-    dynamicContentWithKeypad: {
-      justifyContent: 'flex-start',
-    },
-    dynamicContentWithoutKeypad: {
+      paddingBottom: 12,
       justifyContent: 'flex-start',
     },
     keypadContainerWithDestinationPicker: {
       justifyContent: 'center',
       paddingVertical: 12,
     },
+    dynamicContentWithDestinationPickerAndQuoteCard: {
+      justifyContent: 'flex-start',
+    },
+    dynamicContentWithOnlyQuoteCard: {
+      justifyContent: 'space-around',
+    },
   });
 };

app/components/UI/Bridge/Views/BridgeView/BridgeView.test.tsx

@@ -9,6 +9,8 @@ import { Hex } from '@metamask/utils';
 import BridgeView from '.';
 import { createBridgeTestState } from '../../testUtils';
 import { initialState } from '../../_mocks_/initialState';
+import { RequestStatus, type QuoteResponse } from '@metamask/bridge-controller';
+import mockQuotes from '../../_mocks_/mock-quotes-sol-sol.json';
 
 // TODO remove this mock once we have a real implementation
 jest.mock('../../../../../selectors/confirmTransaction');
@@ -72,23 +74,21 @@ jest.mock('../../hooks/useLatestBalance', () => ({
 }));
 
 describe('BridgeView', () => {
-  const mockChainId = '0x1' as Hex;
-  const optimismChainId = '0xa' as Hex;
   const token2Address = '0x0000000000000000000000000000000000000002' as Hex;
 
   beforeEach(() => {
     jest.clearAllMocks();
   });
 
   it('renders', async () => {
-    const { getByText } = renderScreen(
+    const { toJSON } = renderScreen(
       BridgeView,
       {
         name: Routes.BRIDGE.ROOT,
       },
       { state: initialState },
     );
-    expect(getByText('Select amount')).toBeDefined();
+    expect(toJSON()).toMatchSnapshot();
   });
 
   it('should open BridgeTokenSelector when clicking source token', async () => {
@@ -275,6 +275,8 @@ describe('BridgeView', () => {
           quoteRequest: {
             insufficientBal: true,
           },
+          quotesLoadingStatus: RequestStatus.FETCHED,
+          quotes: [mockQuotes[0] as unknown as QuoteResponse],
         },
       });
 
@@ -295,6 +297,8 @@ describe('BridgeView', () => {
           quoteRequest: {
             insufficientBal: false,
           },
+          quotesLoadingStatus: RequestStatus.FETCHED,
+          quotes: [mockQuotes[0] as unknown as QuoteResponse],
         },
         bridgeReducerOverrides: {
           sourceAmount: '1.0', // Less than balance of 2.0 ETH
@@ -314,28 +318,26 @@ describe('BridgeView', () => {
     });
 
     it('should handle Continue button press', async () => {
+      const testState = createBridgeTestState({
+        bridgeControllerOverrides: {
+          quoteRequest: {
+            insufficientBal: false,
+          },
+          quotesLoadingStatus: RequestStatus.FETCHED,
+          quotes: [mockQuotes[0] as unknown as QuoteResponse],
+        },
+        bridgeReducerOverrides: {
+          sourceAmount: '1.0', // Less than balance of 2.0 ETH
+        },
+      });
+
       const { getByText } = renderScreen(
         BridgeView,
         {
           name: Routes.BRIDGE.ROOT,
         },
         {
-          state: {
-            ...initialState,
-            bridge: {
-              ...initialState.bridge,
-              sourceAmount: '1.0',
-              destToken: {
-                address: token2Address,
-                symbol: 'TOKEN2',
-                decimals: 18,
-                image: 'https://token2.com/logo.png',
-                chainId: optimismChainId,
-              },
-              selectedDestChainId: optimismChainId,
-              selectedSourceChainIds: [mockChainId, optimismChainId],
-            },
-          },
+          state: testState,
         },
       );
 
@@ -352,6 +354,8 @@ describe('BridgeView', () => {
           quoteRequest: {
             insufficientBal: false,
           },
+          quotesLoadingStatus: RequestStatus.FETCHED,
+          quotes: [mockQuotes[0] as unknown as QuoteResponse],
         },
         bridgeReducerOverrides: {
           sourceAmount: '1.0', // Less than balance of 2.0 ETH
@@ -372,4 +376,4 @@ describe('BridgeView', () => {
       // TODO: Add expectations once Terms navigation is implemented
     });
   });
-});
+});