feat(remote-feature-flag-controller): segment threshold flags by metaMetricsFlags with canonical ID default

[asalsys]

Summary

• Add required getCanonicalProfileId constructor callback and optional metaMetricsFlags map for threshold flag segmentation.
• Threshold flags use canonical profile ID by default; flags whose names are present in metaMetricsFlags use MetaMetrics ID instead.
• Generalize threshold calculation to hash the selected segmentation ID while preserving the existing deterministic bucketing behavior.
• Pass the new segmentation options through @metamask/wallet remote feature flag controller initialization.

Segmentation Behavior

Flag configuration	Identifier used	Constructor callback
Flag name absent from metaMetricsFlags	Canonical profile ID	getCanonicalProfileId()
Flag name present in metaMetricsFlags	MetaMetrics ID	getMetaMetricsId()

If the selected identifier is empty, threshold arrays are preserved as-is and not processed.

Example threshold config using canonical segmentation by default:

{
  "myCanonicalFlag": [
    {
      "name": "groupA",
      "scope": { "type": "threshold", "value": 0.5 },
      "value": "valueA"
    },
    {
      "name": "groupB",
      "scope": { "type": "threshold", "value": 1.0 },
      "value": "valueB"
    }
  ]
}

Example controller setup for a flag that should keep MetaMetrics segmentation:

new RemoteFeatureFlagController({
  // ...
  getMetaMetricsId,
  getCanonicalProfileId,
  metaMetricsFlags: {
    legacyMetaMetricsFlag: true,
  },
});

Migration

Consumers must pass getCanonicalProfileId for threshold flags that should use canonical profile segmentation, which is now the default for flags absent from metaMetricsFlags.

Existing threshold flags that must keep MetaMetrics-based bucketing should be listed in metaMetricsFlags and continue to provide getMetaMetricsId.

Test plan

• yarn workspace @metamask/remote-feature-flag-controller run jest --no-coverage src/utils/user-segmentation-utils.test.ts src/remote-feature-flag-controller.test.ts
• Threshold flags absent from metaMetricsFlags use getCanonicalProfileId for segmentation
• Threshold flags present in metaMetricsFlags use getMetaMetricsId for segmentation
• Threshold arrays are preserved when the selected identifier is empty
• Threshold cache entries use the selected segmentation ID and stale entries are cleaned up when flags are removed from the server response

Related

• Documentation: MetaMask/contributor-docs#176
• Ticket - https://consensyssoftware.atlassian.net/browse/MCWP-662

---

Note

High Risk
Changes default A/B bucketing for threshold flags, so users can land in different groups unless legacy flags are listed in metaMetricsFlags; rollout and cache behavior are user-facing and migration-sensitive.

Overview
Threshold rollout no longer always keys off MetaMetrics ID. RemoteFeatureFlagController now accepts optional getCanonicalProfileId and metaMetricsFlags: flags not listed in the map hash canonicalProfileId + flag name for bucketing and cache keys; flags in the map keep MetaMetrics-based segmentation (for legacy behavior).

When the chosen segmentation ID is empty, threshold arrays are left unprocessed (same as before for missing MetaMetrics). Explicit metaMetricsIds targeting is unchanged but only runs when MetaMetrics ID is available. calculateThresholdForFlag is generalized to any non-empty segmentation ID. Stale thresholdCache cleanup now drops entries for either the current MetaMetrics or canonical ID.

@metamask/wallet forwards the new options into remote feature flag initialization. Tests and changelogs are updated accordingly.

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

[NicolasMassart]

Two comments, one blocking and a nit.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 1e9b936. Configure here.}

[cursor]

Hash fallback test misses MetaMetrics opt-in

Medium Severity

The malformed-metaMetricsIds hash-fallback case still expects the MetaMetrics bucketing result for testFlag, but never lists that flag in metaMetricsFlags. With the new canonical default, this path hashes getCanonicalProfileId instead, so the assertion no longer validates the intended MetaMetrics fallback behavior.

 

^{Reviewed by Cursor Bugbot for commit 1e9b936. Configure here.}