Skip to content

fix: point DIGEST_API_URL to production endpoint cp-7.71.1 cp-7.72.0#28088

Merged
xavier-brochard merged 2 commits into
mainfrom
fix/digest-api-url-production
Mar 30, 2026
Merged

fix: point DIGEST_API_URL to production endpoint cp-7.71.1 cp-7.72.0#28088
xavier-brochard merged 2 commits into
mainfrom
fix/digest-api-url-production

Conversation

@xavier-brochard

@xavier-brochard xavier-brochard commented Mar 30, 2026 •
edited by cursor Bot
Loading

Copy link
Copy Markdown
Contributor

Description

Switches the hardcoded fallback value for DIGEST_API_URL in AppConstants.ts from the dev endpoint (digest.dev-api.cx.metamask.io) to the production endpoint (digest.api.cx.metamask.io).

Also moves the @metamask/sdk-communication-layer import to the top of the file to follow alphabetical import ordering.

Changelog

CHANGELOG entry: null

Related issues

Fixes:

Manual testing steps

N/A

Screenshots/Recordings

Before

N/A

After

N/A

Pre-merge author checklist

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

Note

Low Risk
Low risk: only changes a constant fallback URL and import ordering. Main impact is that builds without DIGEST_API_URL will now call the production digest service instead of the dev endpoint.

Overview
Updates AppConstants.DIGEST_API_URL to default to the production https://digest.api.cx.metamask.io/api/v1 endpoint when the DIGEST_API_URL env var is not set.

Also reorders the DEFAULT_SERVER_URL import to match expected import ordering.

Written by Cursor Bugbot for commit 9d584a4. This will update automatically on new commits. Configure here.

@xavier-brochard xavier-brochard self-assigned this Mar 30, 2026
@github-actions

github-actions Bot commented Mar 30, 2026

Copy link
Copy Markdown
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot metamaskbot added the team-social-ai Social & AI team label Mar 30, 2026
@xavier-brochard xavier-brochard marked this pull request as ready for review March 30, 2026 11:16
aganglada
aganglada previously approved these changes Mar 30, 2026
View reviewed changes
zone-live
zone-live previously approved these changes Mar 30, 2026
View reviewed changes
@xavier-brochard xavier-brochard dismissed stale reviews from zone-live and aganglada via 9d584a4 March 30, 2026 11:40
@github-actions github-actions Bot added the risk-low Low testing needed · Low bug introduction risk label Mar 30, 2026
@github-actions

github-actions Bot commented Mar 30, 2026

Copy link
Copy Markdown
Contributor

🔍 Smart E2E Test Selection

  • Selected E2E tags: None (no tests recommended)
  • Selected Performance tags: None (no tests recommended)
  • Risk Level: low
  • AI Confidence: 92%
click to see 🤖 AI reasoning details

E2E Test Selection:
The changes in app/core/AppConstants.ts are minimal and low-risk:

  1. Import reordering (cosmetic): DEFAULT_SERVER_URL import moved from bottom to top of imports — zero functional impact.

  2. DIGEST_API_URL promotion: The fallback URL changed from https://digest.dev-api.cx.metamask.io/api/v1 (dev/staging) to https://digest.api.cx.metamask.io/api/v1 (production). This is a standard environment promotion of an API endpoint.

Impact analysis:

  • DIGEST_API_URL is used exclusively by ai-digest-controller-init.ts to configure the AiDigestController's AiDigestService base URL.
  • The AiDigestController is an AI feature controller with no direct mapping to any of the available E2E test tags (SmokeAccounts, SmokeConfirmations, SmokeTrade, etc.).
  • No user-facing flows, UI components, navigation, or critical paths are affected.
  • The change is purely a configuration/URL promotion — the code logic remains identical.
  • No E2E tests need to run to validate a URL string change in a constant file, especially one that only affects an AI digest service endpoint.

No performance tests are needed as this is a URL configuration change with no rendering, state management, or computational impact.

Performance Test Selection:
The change is a URL string promotion from dev to production for the DIGEST_API_URL constant. This has no impact on UI rendering, data loading, state management, or any performance-sensitive code paths. No performance tests are warranted.

View GitHub Actions results

@sonarqubecloud

sonarqubecloud Bot commented Mar 30, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

@xavier-brochard xavier-brochard added this pull request to the merge queue Mar 30, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 30, 2026
@xavier-brochard xavier-brochard added this pull request to the merge queue Mar 30, 2026
Merged via the queue into main with commit 8603e1c Mar 30, 2026
63 checks passed
@xavier-brochard xavier-brochard deleted the fix/digest-api-url-production branch March 30, 2026 13:28
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 30, 2026
@metamaskbot metamaskbot added the release-7.73.0 Issue or pull request that will be included in release 7.73.0 label Mar 30, 2026
@chloeYue chloeYue changed the title fix: point DIGEST_API_URL to production endpoint fix: point DIGEST_API_URL to production endpoint cp-7.71.1 cp-7.72.0 Mar 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@aganglada aganglada aganglada approved these changes

@joaosantos15 joaosantos15 joaosantos15 approved these changes

@zone-live zone-live zone-live left review comments

Assignees

@xavier-brochard xavier-brochard

Labels

release-7.73.0 Issue or pull request that will be included in release 7.73.0 risk-low Low testing needed · Low bug introduction risk size-XS team-social-ai Social & AI team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@xavier-brochard @aganglada @zone-live @joaosantos15 @metamaskbot