chore: delete multichain accounts feature flag

[gantunesr]

Description

Multichain accounts is permanently enabled, so the remote feature flag helpers in app/multichain-accounts/remote-feature-flag.ts had been reduced to stubs that always return true. Nothing in the codebase imports isMultichainAccountsRemoteFeatureEnabled or isMultichainAccountsState2Enabled anymore, so this PR deletes the module and its test file to remove dead code.

Changelog

CHANGELOG entry: null

Related issues

Fixes: N/A

Manual testing steps

N/A — pure dead-code deletion with no runtime behavior change. Verified via search that no remaining code references the deleted functions, and CI (unit tests, TypeScript, lint) covers the rest.

Screenshots/Recordings

Before

N/A

After

N/A

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
• I've completed the PR template to the best of my ability
• I've included tests if applicable
• I've documented my code using JSDoc format if applicable
• I've applied the right labels on the PR (see labeling guidelines). 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 to import wallets with many accounts and tokens
• I've instrumented key operations with Sentry traces for production performance metrics
  • See trace() for usage and addToken for an example

For performance guidelines and tooling, see the Performance Guide.

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 all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

Made with Cursor

[metamask-ci]

PR template — items to address before "Ready for review"

Warnings — informational, address before merging:

• Pre-merge author checklist has unchecked items (e.g. "I've tested on Android"). Every box must be consciously checked — see docs/readme/ready-for-review.md.

See docs/readme/ready-for-review.md for the full Definition of Ready for Review.

[github-actions]

🔍 Smart E2E Test Selection

• Selected E2E tags: None (no tests recommended)
• Selected Performance tags: None (no tests recommended)
• Risk Level: low
• AI Confidence: 97%

click to see 🤖 AI reasoning details

E2E Test Selection:
Both changed files are being deleted:

1. app/multichain-accounts/remote-feature-flag.ts - A stub file where both exported functions always returned true (permanently enabled, no real logic).
2. app/multichain-accounts/remote-feature-flag.test.ts - The unit test for the above stub.

A codebase search confirmed that no other files import from app/multichain-accounts/remote-feature-flag — there are zero downstream consumers. This is pure dead-code cleanup with no functional impact on any user flows, UI components, controllers, or E2E test paths. No E2E tags or performance tests are warranted.

Performance Test Selection:
The deleted files were stubs that always returned true with no performance-sensitive logic. No performance-impacting code paths are affected.

View GitHub Actions results

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud