Skip to content

Conversation

@DDDDDanica
Copy link
Contributor

@DDDDDanica DDDDDanica commented Dec 8, 2025

Description

The test-metamask Sentry project receives significant traffic (284,823 events in 14 days), but there's no clear documentation on sentry setup and usage. This PR add some comprehensive documentation that developers can reference.

Open in GitHub Codespaces

Changelog

CHANGELOG entry: null

Related issues

Fixes: https://github.com/MetaMask/MetaMask-planning/issues/5956

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Before

After

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.

Note

Adds comprehensive Sentry documentation and clarifies Sentry DSN usage in the example config.

  • Documentation (Sentry):
    • Adds detailed sections in development/README.md covering overview, environments, environment determination, DSN selection logic, common Sentry queries, local development setup, and testing methods.
    • Enhances guidance for release publishing with Sentry (UI/CLI setup and source maps verification).
  • Configuration:
    • Updates .metamaskrc.dist with a clear Sentry configuration block, distinguishing SENTRY_DSN (production) vs SENTRY_DSN_DEV (non-production), and retains example DSN for development.

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

@DDDDDanica DDDDDanica self-assigned this Dec 8, 2025
@DDDDDanica DDDDDanica added the team-extension-platform Extension Platform team label Dec 8, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2025

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.

@github-actions github-actions bot added the size-S label Dec 8, 2025
@DDDDDanica DDDDDanica force-pushed the chore/5956-sentry-doc branch from 80c4ceb to 132655d Compare December 8, 2025 18:51
@DDDDDanica DDDDDanica force-pushed the chore/5956-sentry-doc branch from 132655d to 4563265 Compare December 8, 2025 18:56
@metamaskbot
Copy link
Collaborator

Builds ready [4563265]
UI Startup Metrics (1252 ± 82 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup1252105114488213151385
load103386612417110801140
domContentLoaded102686312226910751130
domInteractive21163862237
firstPaint52683125041110351131
backgroundConnect21719625512222246
firstReactRender4630193195175
getState38181262241101
initialActions104112
loadScripts816663102066860908
setupStore1274761324
numNetworkReqs86307630
BrowserifyPower User HomeuiStartup18801616241113719512085
load1084874134110611691256
domContentLoaded1071866129010411631234
domInteractive26175472743
firstPaint54892134642010661202
backgroundConnect24520357876237528
firstReactRender44359674654
getState19714733235214271
initialActions102112
loadScripts85866710871029541025
setupStore17105691540
numNetworkReqs69572032266103
WebpackStandard HomeuiStartup9137341229889631062
load69758994379745853
domContentLoaded69158493579740848
domInteractive24165882544
firstPaint24492914172236714
backgroundConnect13652101437
firstReactRender68373365272175
getState261355103147
initialActions103112
loadScripts68858293379738846
setupStore1273251323
numNetworkReqs86305623
WebpackPower User HomeuiStartup14951210212715815921770
load73660993379781884
domContentLoaded72860392577772873
domInteractive25175482647
firstPaint31493936237259859
backgroundConnect59757310855292
firstReactRender48408765059
getState20314532743237287
initialActions102112
loadScripts72560192277766869
setupStore14104061434
numNetworkReqs6722202176698
FirefoxBrowserifyStandard HomeuiStartup12641091159111213451499
load103493912536910831174
domContentLoaded103493212526910831174
domInteractive53311372674110
firstPaint------
backgroundConnect3720141233697
firstReactRender36306483657
getState115206201021
initialActions102022
loadScripts101392312286610571146
setupStore1062941022
numNetworkReqs86265624
BrowserifyPower User HomeuiStartup25711534344460030913330
load1507947244754121042351
domContentLoaded1507947244754121042351
domInteractive7826106414169132
firstPaint------
backgroundConnect140211188167188319
firstReactRender5332181185679
getState150721044102174236
initialActions204123
loadScripts1435932242649419952230
setupStore765113017460379
numNetworkReqs6944121137197
WebpackStandard HomeuiStartup15681319209214116591876
load1271112215668313191421
domContentLoaded1271112215668313181421
domInteractive63282304082140
firstPaint------
backgroundConnect46221862649106
firstReactRender42345764354
getState218203301794
initialActions102122
loadScripts1246109715428212941388
setupStore136134151229
numNetworkReqs86266825
WebpackPower User HomeuiStartup27891813418057432153518
load18061117274552723202477
domContentLoaded18061116274552723202477
domInteractive148291092281781039
firstPaint------
backgroundConnect152241288231177991
firstReactRender5434103125984
getState149841155111176237
initialActions213123
loadScripts17071101259148722252425
setupStore614103915441288
numNetworkReqs70531241471100
📊 Page Load Benchmark Results

Current Commit: 4563265 | Date: 12/8/2025

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.04s (±96ms) 🟡 | historical mean value: 1.05s ⬇️ (historical data)
  • domContentLoaded-> current mean value: 729ms (±118ms) 🟢 | historical mean value: 730ms ⬇️ (historical data)
  • firstContentfulPaint-> current mean value: 93ms (±186ms) 🟢 | historical mean value: 78ms ⬆️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.04s 96ms 1.00s 1.96s 1.06s 1.96s
domContentLoaded 729ms 118ms 697ms 1.88s 743ms 1.88s
firstPaint 93ms 186ms 56ms 1.94s 84ms 1.94s
firstContentfulPaint 93ms 186ms 56ms 1.94s 84ms 1.94s
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: -911 Bytes (-0.02%)
  • ui: 126.73 KiB (1.64%)
  • common: 1.7 KiB (0.02%)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size-S team-extension-platform Extension Platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants