feat: MUSD-691 clean up deprecated quick convert code (#29351)

<!--
Please submit this PR as a draft initially.

Do not mark it as "Ready for review" until this PR meets the canonical
Definition of Ready For Review in `docs/readme/ready-for-review.md`.

In short: the template must be materially complete (not just section
titles
present), all status checks must be currently passing, and the only
expected
follow-up commits must be reviewer-driven.
-->

## **Description**
This PR removes the now deprecated Quick Convert code paths. We're
keeping the Max convert bottom sheet and the `MusdConversionAssetRow`
components to integrate into the Money Home screen.
<!--
Write a short description of the changes included in this pull request,
also include relevant motivation and context. Have in mind the following
questions:
1. What is the reason for the change?
2. What is the improvement/solution?
-->

## **Changelog**

<!--
If this PR is not End-User-Facing and should not show up in the
CHANGELOG, you can choose to either:
1. Write `CHANGELOG entry: null`
2. Label with `no-changelog`

If this PR is End-User-Facing, please write a short User-Facing
description in the past tense like:
`CHANGELOG entry: Added a new tab for users to see their NFTs`
`CHANGELOG entry: Fixed a bug that was causing some NFTs to flicker`

(This helps the Release Engineer do their job more quickly and
accurately)
-->

CHANGELOG entry: remove deprecated quick convert feature code paths

## **Related issues**

Fixes:
- [MUSD-691: Clean up deprecated Quick Convert Home and related code
paths](https://consensyssoftware.atlassian.net/browse/MUSD-691)

## **Manual testing steps**

N/A

## **Screenshots/Recordings**

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

### **Before**

<!-- [screenshots/recordings] -->
N/A
### **After**

<!-- [screenshots/recordings] -->
N/A
## **Pre-merge author checklist**

<!--
Every checklist item must be consciously assessed before marking this PR
as
"Ready for review". A checked box means you deliberately considered that
responsibility, not that you literally performed every action listed.

Unchecked boxes are ambiguous: they are not an implicit "N/A" and they
are not
a silent "skip". See `docs/readme/ready-for-review.md` for the full
checklist
semantics.
-->

- [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.

#### Performance checks (if applicable)

- [ ] I've tested on Android
  - Ideally on a mid-range device; emulator is acceptable
- [ ] I've tested with a power user scenario
- Use these [power-user
SRPs](https://consensyssoftware.atlassian.net/wiki/spaces/TL1/pages/edit-v2/401401446401?draftShareId=9d77e1e1-4bdc-4be1-9ebb-ccd916988d93)
to import wallets with many accounts and tokens
- [ ] I've instrumented key operations with Sentry traces for production
performance metrics
- See [`trace()`](/app/util/trace.ts) for usage and
[`addToken`](/app/components/Views/AddAsset/components/AddCustomToken/AddCustomToken.tsx#L274)
for an example

For performance guidelines and tooling, see the [Performance
Guide](https://consensyssoftware.atlassian.net/wiki/spaces/TL1/pages/400085549067/Performance+Guide+for+Engineers).

## **Pre-merge reviewer checklist**

<!--
Reviewer checklist items follow the same semantics as the author
checklist: an
unchecked box is ambiguous, a checked box means the reviewer consciously
assessed that responsibility. See `docs/readme/ready-for-review.md`.
-->

- [ ] 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.

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Medium Risk**
> Medium risk because it removes a navigation route/feature flag and
rewires mUSD conversion CTAs and analytics to always use the
custom-amount flow, which could affect user routing and event reporting
if any remaining callers expected Quick Convert.
> 
> **Overview**
> Removes the deprecated mUSD **Quick Convert** flow end-to-end: deletes
`MusdQuickConvertView` (and `MusdBalanceCard`), drops the
`MM_MUSD_QUICK_CONVERT_ENABLED` env var, removes the quick-convert
feature-flag selector/remote registry entries, and removes the
`Routes.EARN.MUSD.QUICK_CONVERT` route.
> 
> Updates mUSD conversion entry points (`useMusdConversion`, education
screen, cash/token list/asset CTAs) to stop passing/handling
`navigationOverride` and to always route/track redirects to
`CUSTOM_AMOUNT_SCREEN` (while keeping the max-convert bottom sheet
location constant).
> 
> Renames `ConvertTokenRow` to `MusdConversionAssetRow` and updates
downstream usage/tests (e.g., Money convert stablecoins, cash tokens
skeleton) to use the new component/test IDs.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
74947e7. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Author: Matt561

.js.env.example

@@ -114,7 +114,6 @@ export MM_POOLED_STAKING_ENABLED="true"
 export MM_POOLED_STAKING_SERVICE_INTERRUPTION_BANNER_ENABLED="true"
 # mUSD
 export MM_MUSD_CONVERSION_FLOW_ENABLED="false"
-export MM_MUSD_QUICK_CONVERT_ENABLED="false"
 # See app/components/UI/Earn/docs/wildcard-token-list.md for more information.
 export MM_MUSD_CONVERTIBLE_TOKENS_BLOCKLIST=''
 export MM_MUSD_CONVERTIBLE_TOKENS_ALLOWLIST=''

app/components/UI/Earn/Views/EarnMusdConversionEducationView/index.test.tsx

@@ -21,8 +21,6 @@ import { useMusdConversionFlowData } from '../../hooks/useMusdConversionFlowData
 import { useRampNavigation } from '../../../Ramp/hooks/useRampNavigation';
 import Routes from '../../../../../constants/navigation/Routes';
 import AppConstants from '../../../../../core/AppConstants';
-import { MUSD_CONVERSION_NAVIGATION_OVERRIDE } from '../../types/musd.types';
-import { selectMusdQuickConvertEnabledFlag } from '../../selectors/featureFlags';
 import { selectMoneyHubEnabledFlag } from '../../../Money/selectors/featureFlags';
 import { MUSD_EVENTS_CONSTANTS } from '../../constants/events';
 import { MONEY_EVENTS_CONSTANTS } from '../../../Money/constants/moneyEvents';
@@ -74,10 +72,6 @@ jest.mock('../../../Ramp/hooks/useRampNavigation', () => ({
   useRampNavigation: jest.fn(),
 }));
 
-jest.mock('../../selectors/featureFlags', () => ({
-  selectMusdQuickConvertEnabledFlag: jest.fn(() => false),
-}));
-
 jest.mock('../../../Money/selectors/featureFlags', () => ({
   selectMoneyHubEnabledFlag: jest.fn(() => false),
 }));
@@ -123,10 +117,6 @@ const mockUseMusdConversionFlowData =
 const mockUseRampNavigation = useRampNavigation as jest.MockedFunction<
   typeof useRampNavigation
 >;
-const mockSelectMusdQuickConvertEnabledFlag =
-  selectMusdQuickConvertEnabledFlag as jest.MockedFunction<
-    typeof selectMusdQuickConvertEnabledFlag
-  >;
 const mockSelectMoneyHubEnabledFlag =
   selectMoneyHubEnabledFlag as jest.MockedFunction<
     typeof selectMoneyHubEnabledFlag
@@ -236,7 +226,6 @@ describe('EarnMusdConversionEducationView', () => {
       goToSell: jest.fn(),
       goToDeposit: jest.fn(),
     });
-    mockSelectMusdQuickConvertEnabledFlag.mockReturnValue(false);
     mockSelectMoneyHubEnabledFlag.mockReturnValue(false);
 
     mockBuild.mockReturnValue({ name: 'mock-built-event' });
@@ -321,7 +310,6 @@ describe('EarnMusdConversionEducationView', () => {
             chainId: '0x1',
           },
           skipEducationCheck: true,
-          navigationOverride: MUSD_CONVERSION_NAVIGATION_OVERRIDE.QUICK_CONVERT,
         });
         expect(mockNavigation.navigate).not.toHaveBeenCalledWith(
           Routes.WALLET.HOME,
@@ -512,7 +500,6 @@ describe('EarnMusdConversionEducationView', () => {
             chainId: mockConversionTokenHighBalance.chainId,
           },
           skipEducationCheck: true,
-          navigationOverride: MUSD_CONVERSION_NAVIGATION_OVERRIDE.QUICK_CONVERT,
         });
       });
     });
@@ -1057,7 +1044,6 @@ describe('EarnMusdConversionEducationView', () => {
         expect(mockInitiateConversion).toHaveBeenCalledWith({
           preferredPaymentToken: mockRouteParams.preferredPaymentToken,
           skipEducationCheck: true,
-          navigationOverride: MUSD_CONVERSION_NAVIGATION_OVERRIDE.QUICK_CONVERT,
         });
       });
     });
@@ -1115,38 +1101,6 @@ describe('EarnMusdConversionEducationView', () => {
       });
       expect(mockInitiateConversion).not.toHaveBeenCalled();
     });
-
-    it("forwards caller's navigationOverride (CUSTOM) to initiateCustomConversion instead of hardcoding QUICK_CONVERT", async () => {
-      mockUseParams.mockReturnValue({
-        preferredPaymentToken: {
-          address: '0xabc' as Hex,
-          chainId: '0x1' as Hex,
-        },
-        navigationOverride: MUSD_CONVERSION_NAVIGATION_OVERRIDE.CUSTOM,
-      });
-
-      const { getByTestId } = renderWithProvider(
-        <EarnMusdConversionEducationView />,
-        { state: {} },
-      );
-
-      await act(async () => {
-        fireEvent.press(
-          getByTestId(
-            EARN_TEST_IDS.MUSD.CONVERSION_EDUCATION_VIEW.PRIMARY_BUTTON,
-          ),
-        );
-      });
-
-      await waitFor(() => {
-        expect(mockInitiateConversion).toHaveBeenCalledWith(
-          expect.objectContaining({
-            navigationOverride: MUSD_CONVERSION_NAVIGATION_OVERRIDE.CUSTOM,
-            skipEducationCheck: true,
-          }),
-        );
-      });
-    });
   });
 
   describe('MetaMetrics', () => {
@@ -1232,42 +1186,6 @@ describe('EarnMusdConversionEducationView', () => {
       });
     });
 
-    it('tracks quick convert redirect when quick convert is enabled', async () => {
-      mockSelectMusdQuickConvertEnabledFlag.mockReturnValue(true);
-
-      const { getByTestId } = renderWithProvider(
-        <EarnMusdConversionEducationView />,
-        { state: {} },
-      );
-
-      mockTrackEvent.mockClear();
-      mockCreateEventBuilder.mockClear();
-      mockAddProperties.mockClear();
-      mockBuild.mockClear();
-
-      await act(async () => {
-        fireEvent.press(
-          getByTestId(
-            EARN_TEST_IDS.MUSD.CONVERSION_EDUCATION_VIEW.PRIMARY_BUTTON,
-          ),
-        );
-      });
-
-      await waitFor(() => {
-        expect(mockCreateEventBuilder).toHaveBeenCalledWith(
-          MetaMetricsEvents.MUSD_FULLSCREEN_ANNOUNCEMENT_BUTTON_CLICKED,
-        );
-
-        expect(mockAddProperties).toHaveBeenCalledWith({
-          location: 'conversion_education_screen',
-          button_type: 'primary',
-          button_text: strings('earn.musd_conversion.education.primary_button'),
-          redirects_to:
-            MUSD_EVENTS_CONSTANTS.EVENT_LOCATIONS.QUICK_CONVERT_HOME_SCREEN,
-        });
-      });
-    });
-
     it('tracks fullscreen announcement button clicked event when go back button is pressed', () => {
       const { getByTestId } = renderWithProvider(
         <EarnMusdConversionEducationView />,

app/components/UI/Earn/Views/EarnMusdConversionEducationView/index.tsx

@@ -40,11 +40,7 @@ import { useRampNavigation } from '../../../Ramp/hooks/useRampNavigation';
 import { RampIntent } from '../../../Ramp/types';
 import { EARN_TEST_IDS } from '../../constants/testIds';
 import AppConstants from '../../../../../core/AppConstants';
-import {
-  MUSD_CONVERSION_NAVIGATION_OVERRIDE,
-  MusdNavigationTarget,
-} from '../../types/musd.types';
-import { selectMusdQuickConvertEnabledFlag } from '../../selectors/featureFlags';
+import { MusdNavigationTarget } from '../../types/musd.types';
 import { toChecksumAddress } from '../../../../../util/address';
 import { safeFormatChainIdToHex } from '../../../Card/util/safeFormatChainIdToHex';
 import { MONEY_EVENTS_CONSTANTS } from '../../../Money/constants/moneyEvents';
@@ -65,12 +61,6 @@ interface EarnMusdConversionEducationViewRouteParams {
     address: Hex;
     chainId: Hex;
   };
-  /**
-   * Caller's intended navigation override. When present, this is forwarded to
-   * `initiateCustomConversion` on continue so the education screen doesn't hijack
-   * the destination (e.g., pencil-in-Hub preserves CUSTOM; other callers keep their intent).
-   */
-  navigationOverride?: MUSD_CONVERSION_NAVIGATION_OVERRIDE;
   /**
    * Pure-navigation exit target. When present, the primary button routes here and
    * skips conversion entirely. Use for entry points that only needed the education
@@ -86,18 +76,13 @@ interface EarnMusdConversionEducationViewRouteParams {
 const EarnMusdConversionEducationView = () => {
   const dispatch = useDispatch();
 
-  const isQuickConvertEnabled = useSelector(selectMusdQuickConvertEnabledFlag);
   const isMoneyHubEnabled = useSelector(selectMoneyHubEnabledFlag);
 
   const { initiateCustomConversion } = useMusdConversion();
   const { goToBuy } = useRampNavigation();
 
-  const {
-    preferredPaymentToken,
-    isDeeplink,
-    navigationOverride: callerNavigationOverride,
-    returnTo,
-  } = useParams<EarnMusdConversionEducationViewRouteParams>();
+  const { preferredPaymentToken, isDeeplink, returnTo } =
+    useParams<EarnMusdConversionEducationViewRouteParams>();
 
   // Hooks for deeplink case (when no params provided)
   const {
@@ -219,9 +204,7 @@ const EarnMusdConversionEducationView = () => {
   }, [submitScreenViewedEvent]);
 
   const submitContinuePressedEvent = useCallback(() => {
-    let redirectsTo = isQuickConvertEnabled
-      ? MUSD_EVENT_LOCATIONS.QUICK_CONVERT_HOME_SCREEN
-      : MUSD_EVENT_LOCATIONS.CUSTOM_AMOUNT_SCREEN;
+    let redirectsTo = MUSD_EVENT_LOCATIONS.CUSTOM_AMOUNT_SCREEN;
     if (returnTo) {
       redirectsTo = MONEY_EVENT_LOCATIONS.MONEY_HUB;
     } else if (deeplinkState?.action === 'navigate_money_hub') {
@@ -245,9 +228,7 @@ const EarnMusdConversionEducationView = () => {
         .build(),
     );
   }, [
-    isQuickConvertEnabled,
     returnTo,
-    MUSD_EVENT_LOCATIONS.QUICK_CONVERT_HOME_SCREEN,
     MUSD_EVENT_LOCATIONS.CUSTOM_AMOUNT_SCREEN,
     MUSD_EVENT_LOCATIONS.CONVERSION_EDUCATION_SCREEN,
     MUSD_EVENT_LOCATIONS.HOME_SCREEN,
@@ -325,22 +306,15 @@ const EarnMusdConversionEducationView = () => {
           await initiateCustomConversion({
             preferredPaymentToken: deeplinkState.paymentToken,
             skipEducationCheck: true,
-            navigationOverride:
-              MUSD_CONVERSION_NAVIGATION_OVERRIDE.QUICK_CONVERT,
           });
           return;
         }
       }
 
-      // Proceed to conversion flow if we have the required params (normal flow).
-      // Honor caller's navigationOverride; fall back to QUICK_CONVERT.
       if (!isDeeplink && preferredPaymentToken) {
         await initiateCustomConversion({
           preferredPaymentToken,
           skipEducationCheck: true,
-          navigationOverride:
-            callerNavigationOverride ??
-            MUSD_CONVERSION_NAVIGATION_OVERRIDE.QUICK_CONVERT,
         });
         return;
       }
@@ -365,7 +339,6 @@ const EarnMusdConversionEducationView = () => {
     goToBuy,
     isDeeplink,
     returnTo,
-    callerNavigationOverride,
   ]);
 
   const handleNotNow = () => {