feat: remove provider ID normalization and update related functionality

[meltingice1337]

Explanation

@metamask/ramps-controller normalized provider IDs by stripping a /providers/ prefix through a normalizeProviderCode helper. This normalization was applied in two places: across the getQuotes scope-aware selection logic (matching the providers list, each quote's provider, custom-action provider IDs, and the sort-order IDs) and when building a precreated stub order, where the provider code was re-wrapped as /providers/<code>. The Ramps backend now returns canonical, non-prefixed provider IDs, so this normalization is unnecessary and inconsistent with the new contract.

This PR removes provider-ID normalization entirely so IDs are compared and stored as-is (canonical). Concretely: the normalizeProviderCode function and its public export are removed; the six normalizeProviderCode(...) call sites in the getQuotes selection logic are dropped so the map keys and lookups use the raw provider IDs; and a precreated stub order's provider.id is now the canonical provider code passed to the create-order call rather than a /providers/-prefixed value. The dedicated normalizeProviderCode unit test is removed and the stub-order test now expects the canonical paypal ID.

One detail worth calling out for reviewers: the six removed call sites were added recently in #9353 (scope-aware in-app provider widening). Because normalization is removed uniformly from both sides of every map key and lookup, matching behavior is preserved for consistently-formatted IDs, and the existing getQuotes tests continue to pass unchanged (308 tests green).

This is a breaking change: normalizeProviderCode is no longer exported, and the precreated stub order's provider.id changes shape from /providers/<code> to <code>. Consumers must supply non-prefixed (canonical) provider IDs. The mobile consumer PR removes all normalizeProviderCode usages and the app-side normalization to adopt this.

References

• Fixes: https://consensyssoftware.atlassian.net/browse/TRAM-3560
• Consumer PR (mobile): feat: implement non prefix ids backwards compatiblity on ramps metamask-mobile#32819
• Related to feat(ramps-controller): scope-aware in-app provider widening in getQuotes #9353 (introduced the getQuotes normalization removed here)

Checklist

• I've updated the test suite for new or updated code as appropriate
• I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
• I've communicated my changes to consumers by updating changelogs for packages I've changed
• I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

---

Note

Medium Risk
Breaking public API and order stub shape; quote auto-selection can mis-match if callers still send prefixed IDs, but behavior is aligned with the backend contract.

Overview
Breaking: @metamask/ramps-controller stops stripping /providers/ from provider IDs. The normalizeProviderCode helper and its package export are removed.

getQuotes scope-aware selection (providers list, quote provider, custom actions, sort order) now compares IDs as returned by the API. addPrecreatedOrder stores the stub’s provider.id as the canonical code you pass in (e.g. paypal), not /providers/paypal.

Consumers must pass and expect non-prefixed provider IDs everywhere this package matches or persists them. Changelog and JSDoc are updated; tests drop the normalizer suite and assert the new stub shape.

^{Reviewed by Cursor Bugbot for commit d67a44d. Bugbot is set up for automated code reviews on this repo. Configure here.}