feat: remove provider ID normalization and update related functionality

packages/ramps-controller/CHANGELOG.md

@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Changed
+
+- **BREAKING:** Provider IDs are no longer normalized by stripping a `/providers/` prefix. `RampsController.getQuotes` now matches provider IDs from the providers list, quotes, custom actions, and sort order as-is, and a precreated stub order's `provider.id` is the canonical provider code passed to the create-order call rather than a `/providers/`-prefixed value. Consumers must supply non-prefixed (canonical) provider IDs ([#9448](https://github.com/MetaMask/core/pull/9448))
+
+### Removed
+
+- **BREAKING:** Remove the `normalizeProviderCode` export ([#9448](https://github.com/MetaMask/core/pull/9448))
+
 ## [15.1.0]
 
 ### Added

packages/ramps-controller/src/RampsController-method-action-types.ts

@@ -289,7 +289,7 @@ export type RampsControllerGetBuyWidgetDataAction = {
  *
  * @param params - Object containing order identifiers and wallet info.
  * @param params.orderId - Full order ID (e.g. "/providers/paypal/orders/abc123") or order code.
- * @param params.providerCode - Provider code (e.g. "paypal", "transak"), with or without /providers/ prefix.
+ * @param params.providerCode - Canonical provider code (e.g. "paypal", "transak").
  * @param params.walletAddress - Wallet address for the order.
  * @param params.chainId - Optional chain ID for the order.
  */

packages/ramps-controller/src/RampsController.test.ts

@@ -16,7 +16,6 @@ import type {
   UserRegion,
 } from './RampsController';
 import {
-  normalizeProviderCode,
   RampsController,
   getDefaultRampsControllerState,
   RAMPS_CONTROLLER_REQUIRED_SERVICE_ACTIONS,
@@ -66,20 +65,6 @@ describe('RampsController', () => {
   const circuitBreakerOpenErrorMessage =
     'Execution prevented because the circuit breaker is open';
 
-  describe('normalizeProviderCode', () => {
-    it('strips /providers/ prefix', () => {
-      expect(normalizeProviderCode('/providers/transak')).toBe('transak');
-      expect(normalizeProviderCode('/providers/transak-staging')).toBe(
-        'transak-staging',
-      );
-    });
-
-    it('returns string unchanged when no prefix', () => {
-      expect(normalizeProviderCode('transak')).toBe('transak');
-      expect(normalizeProviderCode('')).toBe('');
-    });
-  });
-
   describe('RAMPS_CONTROLLER_REQUIRED_SERVICE_ACTIONS', () => {
     it('includes every RampsService action that RampsController calls', async () => {
       expect.hasAssertions();
@@ -8023,7 +8008,7 @@ describe('RampsController', () => {
         expect(controller.state.orders).toHaveLength(1);
         const stub = controller.state.orders[0];
         expect(stub?.providerOrderId).toBe('abc123');
-        expect(stub?.provider?.id).toBe('/providers/paypal');
+        expect(stub?.provider?.id).toBe('paypal');
         expect(stub?.walletAddress).toBe('0xabc');
         expect(stub?.status).toBe(RampsOrderStatus.Precreated);
       });

packages/ramps-controller/src/RampsController.ts

@@ -765,10 +765,6 @@ function findRegionFromCode(
   };
 }
 
-export function normalizeProviderCode(providerCode: string): string {
-  return providerCode.replace(/^\/providers\//u, '');
-}
-
 /**
  * Returns the internal MetaMask order code used for state lookups and polling.
  * Prefers the code embedded in the canonical order `id` path over `providerOrderId`,
@@ -2089,21 +2085,16 @@ export class RampsController extends BaseController<
     },
   ): Quote | undefined {
     const providerByCode = new Map(
-      providers.map((provider) => [
-        normalizeProviderCode(provider.id),
-        provider,
-      ]),
+      providers.map((provider) => [provider.id, provider]),
     );
     const customActionProviderCodes = new Set(
-      response.customActions.map((action: QuoteCustomAction) =>
-        normalizeProviderCode(action.buy.providerId),
+      response.customActions.map(
+        (action: QuoteCustomAction) => action.buy.providerId,
       ),
     );
 
     const fitsProviderLimits = (quote: Quote): boolean => {
-      const provider = providerByCode.get(
-        normalizeProviderCode(quote.provider),
-      );
+      const provider = providerByCode.get(quote.provider);
       const limit = provider?.limits?.fiat?.[fiat]?.[quote.quote.paymentMethod];
       if (!limit) {
         // No published limits for this provider/payment method: treat as
@@ -2117,7 +2108,7 @@ export class RampsController extends BaseController<
       // `all` (Phase 2) skips the in-app-only exclusions; both scopes still
       // enforce provider limits up front.
       if (scope !== 'all') {
-        const providerCode = normalizeProviderCode(quote.provider);
+        const providerCode = quote.provider;
         if (customActionProviderCodes.has(providerCode)) {
           return false;
         }
@@ -2141,7 +2132,7 @@ export class RampsController extends BaseController<
     }
 
     const candidateByCode = new Map(
-      candidates.map((quote) => [normalizeProviderCode(quote.provider), quote]),
+      candidates.map((quote) => [quote.provider, quote]),
     );
 
     const pickBySortOrder = (sortBy: QuoteSortBy): Quote | undefined => {
@@ -2152,7 +2143,7 @@ export class RampsController extends BaseController<
         return undefined;
       }
       for (const providerId of order) {
-        const match = candidateByCode.get(normalizeProviderCode(providerId));
+        const match = candidateByCode.get(providerId);
         if (match) {
           return match;
         }
@@ -2581,7 +2572,7 @@ export class RampsController extends BaseController<
    *
    * @param params - Object containing order identifiers and wallet info.
    * @param params.orderId - Full order ID (e.g. "/providers/paypal/orders/abc123") or order code.
-   * @param params.providerCode - Provider code (e.g. "paypal", "transak"), with or without /providers/ prefix.
+   * @param params.providerCode - Canonical provider code (e.g. "paypal", "transak").
    * @param params.walletAddress - Wallet address for the order.
    * @param params.chainId - Optional chain ID for the order.
    */
@@ -2597,12 +2588,10 @@ export class RampsController extends BaseController<
     if (!orderCode?.trim()) {
       return;
     }
-    const normalizedProviderCode = normalizeProviderCode(providerCode);
-
     const stubOrder: RampsOrder = {
       providerOrderId: orderCode,
       provider: {
-        id: `/providers/${normalizedProviderCode}`,
+        id: providerCode,
         name: '',
         environmentType: '',
         description: '',

packages/ramps-controller/src/index.ts

@@ -66,7 +66,6 @@ export {
   RampsController,
   getDefaultRampsControllerState,
   getInternalOrderCode,
-  normalizeProviderCode,
   RAMPS_CONTROLLER_REQUIRED_SERVICE_ACTIONS,
 } from './RampsController';
 export type {