fix(bridge): fix: cp-7.47.3 prevent crash when viewing Solana asset details (#16770)

## **Description**

Fixes a crash that occurs when viewing asset details for Solana tokens
in the Bridge flow. The issue was caused by `getTokenDetails` assuming
all non-EVM asset addresses are already in CAIP format, when the Bridge
API sometimes returns raw Solana addresses.

## **Related issues**

Fixes #16734
Fixes
[SWAPS-2525](https://consensyssoftware.atlassian.net/browse/SWAPS-2525)

## **Manual testing steps**

1. Navigate to Bridge feature
2. Select Solana as source or destination chain
3. Select any Solana token from the token list
4. Tap on the asset detail info button
5. Verify the asset details screen loads without crashing
6. Verify contract address and other details display correctly

## **Screenshots/Recordings**

<!-- Add screenshots or recordings demonstrating the fix -->

**Before:**


https://github.com/user-attachments/assets/cf41e9fa-ae3d-4336-aefe-40017e831364

**After:**


https://github.com/user-attachments/assets/ccacf57e-1971-42c4-a3d7-a3978e7c954b


## **Pre-merge author checklist**

- [x] I've followed [MetaMask Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md)
- [x] I've clearly explained what problem this PR solves
- [x] I've linked the issue that this PR fixes
- [x] I've included manual testing steps
- [x] I've included screenshots/recordings if applicable
- [x] I've added unit tests if applicable

## **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 the issue linked above
- [ ] I've checked that all unit tests pass

## **Solution Details**

### **Root Cause**

The `getTokenDetails` function was calling `parseCaipAssetType` directly
on `asset.address` for non-EVM assets, assuming it was already in CAIP
format. However, the Bridge API sometimes returns raw Solana addresses
instead of CAIP-formatted addresses, causing `parseCaipAssetType` to
throw an error.

### **Fix Applied**

Implemented the same defensive approach used in
`useTokenHistoricalPrices`:

```typescript
// Detect if address is already in CAIP format
const isCaipAssetType = asset.address.startsWith(`${asset.chainId}`);

// Convert to CAIP format if needed
const normalizedCaipAssetTypeAddress = isCaipAssetType
  ? asset.address
  : `${asset.chainId}/token:${asset.address}`;
```

### **Benefits**

- ✅ **Fixes crash**: Solana asset details now load without errors
- ✅ **Backward compatible**: Handles both raw and CAIP format addresses
- ✅ **Consistent pattern**: Uses same approach as
`useTokenHistoricalPrices`
- ✅ **Zero breaking changes**: All existing functionality preserved
- ✅ **Future-proof**: Works with any non-EVM chain automatically

### **Testing**

- Added comprehensive unit tests covering both scenarios
- All 12 tests pass including new test cases
- Verified with existing Bridge test suite
- Manual testing confirms crash is resolved


[SWAPS-2525]:
https://consensyssoftware.atlassian.net/browse/SWAPS-2525?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ

Author: amitabh94

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - feat(bridge): add "hardware wallets not supported" error when attempting Solana swap/bridge ([#15743](https://github.com/MetaMask/metamask-mobile/pull/15743))
 - feat(bridge): fetch all tokens for bridge input([#15993](https://github.com/MetaMask/metamask-mobile/pull/15993))
 - fix(bridge): don't show currency value unless amount is above zero ([#15798](https://github.com/MetaMask/metamask-mobile/pull/15798))
+- fix(bridge): prevent crash when viewing Solana asset details ([#16770](https://github.com/MetaMask/metamask-mobile/pull/16770))
 
 ## [7.47.1]
 

app/components/UI/AssetOverview/utils/getTokenDetails.test.ts

@@ -30,6 +30,11 @@ describe('getTokenDetails', () => {
     aggregators: ['uniswap', '1inch'],
   };
 
+  beforeEach(() => {
+    // Clear mock calls before each test for proper isolation
+    (parseCaipAssetType as jest.Mock).mockClear();
+  });
+
   describe('Network-specific behavior', () => {
     it('should format token details for non-EVM networks for spl token', () => {
       (parseCaipAssetType as jest.Mock).mockReturnValue({
@@ -114,6 +119,111 @@ describe('getTokenDetails', () => {
         tokenList: 'uniswap, 1inch',
       });
     });
+
+    it('converts raw Solana address to CAIP format for non-EVM networks', () => {
+      const solanaAsset: TokenI = {
+        ...mockAsset,
+        address: 'EPjFWdd5AufqSSqeM2qN1xzybapC8G4wEGGkZwyTDt1v', // Raw Solana address
+        chainId: 'solana:5eykt4UsFv8P8NJdTREpY1vzqKqZKvdp', // Solana chainId
+      };
+
+      (parseCaipAssetType as jest.Mock).mockReturnValue({
+        assetNamespace: 'token',
+        assetReference: 'EPjFWdd5AufqSSqeM2qN1xzybapC8G4wEGGkZwyTDt1v',
+        chainId: 'solana:5eykt4UsFv8P8NJdTREpY1vzqKqZKvdp',
+        chain: {
+          namespace: 'solana',
+          reference: '5eykt4UsFv8P8NJdTREpY1vzqKqZKvdp',
+        },
+      });
+
+      const result = getTokenDetails(
+        solanaAsset,
+        true, // isNonEvmAsset
+        undefined,
+        mockEvmMetadata,
+      );
+
+      // Verify parseCaipAssetType was called with the converted CAIP format
+      expect(parseCaipAssetType).toHaveBeenCalledWith(
+        'solana:5eykt4UsFv8P8NJdTREpY1vzqKqZKvdp/token:EPjFWdd5AufqSSqeM2qN1xzybapC8G4wEGGkZwyTDt1v',
+      );
+
+      expect(result).toEqual({
+        contractAddress: 'EPjFWdd5AufqSSqeM2qN1xzybapC8G4wEGGkZwyTDt1v',
+        tokenDecimal: 18,
+        tokenList: 'uniswap, 1inch',
+      });
+    });
+
+    it('handles address already in CAIP format for non-EVM networks', () => {
+      // Test that addresses already in CAIP format are handled correctly
+      const solanaAssetWithCaipAddress: TokenI = {
+        ...mockAsset,
+        address:
+          'solana:5eykt4UsFv8P8NJdTREpY1vzqKqZKvdp/token:EPjFWdd5AufqSSqeM2qN1xzybapC8G4wEGGkZwyTDt1v', // Already CAIP format
+        chainId: 'solana:5eykt4UsFv8P8NJdTREpY1vzqKqZKvdp',
+      };
+
+      (parseCaipAssetType as jest.Mock).mockReturnValue({
+        assetNamespace: 'token',
+        assetReference: 'EPjFWdd5AufqSSqeM2qN1xzybapC8G4wEGGkZwyTDt1v',
+        chainId: 'solana:5eykt4UsFv8P8NJdTREpY1vzqKqZKvdp',
+        chain: {
+          namespace: 'solana',
+          reference: '5eykt4UsFv8P8NJdTREpY1vzqKqZKvdp',
+        },
+      });
+
+      const result = getTokenDetails(
+        solanaAssetWithCaipAddress,
+        true, // isNonEvmAsset
+        undefined,
+        mockEvmMetadata,
+      );
+
+      // Verify parseCaipAssetType was called with the original CAIP address (no conversion needed)
+      expect(parseCaipAssetType).toHaveBeenCalledWith(
+        'solana:5eykt4UsFv8P8NJdTREpY1vzqKqZKvdp/token:EPjFWdd5AufqSSqeM2qN1xzybapC8G4wEGGkZwyTDt1v',
+      );
+
+      expect(result).toEqual({
+        contractAddress: 'EPjFWdd5AufqSSqeM2qN1xzybapC8G4wEGGkZwyTDt1v',
+        tokenDecimal: 18,
+        tokenList: 'uniswap, 1inch',
+      });
+    });
+
+    it('should handle empty address in asset', () => {
+      const assetWithoutAddress: TokenI = {
+        ...mockAsset,
+        address: '',
+      };
+
+      // Mock for empty address case - should return null/empty assetReference
+      (parseCaipAssetType as jest.Mock).mockReturnValue({
+        assetNamespace: 'token',
+        assetReference: '', // Empty asset reference for empty address
+        chainId: 'test:test',
+        chain: {
+          namespace: 'slip44',
+          reference: '0x123',
+        },
+      });
+
+      const result = getTokenDetails(
+        assetWithoutAddress,
+        true, // isNonEvmAsset
+        undefined,
+        mockEvmMetadata,
+      );
+
+      expect(result).toEqual({
+        contractAddress: null, // Empty assetReference should result in null
+        tokenDecimal: 18,
+        tokenList: 'uniswap, 1inch',
+      });
+    });
   });
 
   describe('Metadata handling', () => {
@@ -177,26 +287,6 @@ describe('getTokenDetails', () => {
   });
 
   describe('Asset property handling', () => {
-    it('should handle empty address in asset', () => {
-      const assetWithoutAddress: TokenI = {
-        ...mockAsset,
-        address: '',
-      };
-
-      const result = getTokenDetails(
-        assetWithoutAddress,
-        true, // isNonEvmAsset
-        undefined,
-        mockEvmMetadata,
-      );
-
-      expect(result).toEqual({
-        contractAddress: null,
-        tokenDecimal: 18,
-        tokenList: 'uniswap, 1inch',
-      });
-    });
-
     it('should handle zero decimals in asset', () => {
       (parseCaipAssetType as jest.Mock).mockReturnValue({
         assetNamespace: 'token',

app/components/UI/AssetOverview/utils/getTokenDetails.ts

@@ -10,8 +10,16 @@ export const getTokenDetails = (
   tokenMetadata: Record<string, string | number | string[]>,
 ): TokenDetails => {
   if (isNonEvmAsset) {
+    // Use the same approach as useTokenHistoricalPrices
+    const isCaipAssetType = asset.address.startsWith(`${asset.chainId}`);
+
+    // Ensure we have proper CAIP format address for parsing
+    const normalizedCaipAssetTypeAddress = isCaipAssetType
+      ? asset.address
+      : `${asset.chainId}/token:${asset.address}`;
+
     const { assetNamespace, assetReference } = parseCaipAssetType(
-      asset.address as `${string}:${string}/${string}:${string}`,
+      normalizedCaipAssetTypeAddress as `${string}:${string}/${string}:${string}`,
     );
     const isNative = assetNamespace === 'slip44';
     return {