Skip to content

release(runway): cherry-pick chore: reduce Sentry trace sampling cp-13.40.0 - #44462

Merged
HowardBraham merged 1 commit into
release/13.40.0from
runway-cherry-pick-13.40.0-1784079358
Jul 15, 2026
Merged

HowardBraham merged 1 commit into
release/13.40.0from
runway-cherry-pick-13.40.0-1784079358

Conversation

@runway-github

@runway-github runway-github Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Description

Reduce Sentry trace sampling rate to reduce the changes we'll spend all
of our performance quota prematurely in the future. Lately we've
accidentally over-spent a few times, this would give us more of a
buffer.

Changelog

CHANGELOG entry: null

Related issues

N/A

Manual testing steps

N/A

Screenshots/Recordings

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.

Note

Low Risk
Single numeric tuning for observability volume; no auth, data
handling, or functional behavior changes.

Overview
Lowers the default Sentry tracesSampleRate for production
extension builds from 0.75% to 0.5% in getTracesSampleRate
(setupSentry.js).

CI branch rates, METAMASK_DEBUG (1.0), fake DSN, and manifest
flags.sentry.tracesSampleRate overrides are unchanged—only the
fallback when none of those apply.

Reviewed by Cursor Bugbot for commit
fadb937. Bugbot is set up for automated
code reviews on this repo. Configure
here.

Co-authored-by: Vaibhav Goyal 57912679+v-goyal@users.noreply.github.com 8b02f4a

…3.40.0 (#44451)

## **Description**

Reduce Sentry trace sampling rate to reduce the changes we'll spend all
of our performance quota prematurely in the future. Lately we've
accidentally over-spent a few times, this would give us more of a
buffer.

## **Changelog**

CHANGELOG entry: null

## **Related issues**

N/A

## **Manual testing steps**

N/A

## **Screenshots/Recordings**

N/A

## **Pre-merge author checklist**

- [x] I've followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask
Extension Coding
Standards](https://github.com/MetaMask/metamask-extension/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [x] I've completed the PR template to the best of my ability
- [x] I’ve included tests if applicable
- [x] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [x] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-extension/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

## **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.

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Single numeric tuning for observability volume; no auth, data
handling, or functional behavior changes.
> 
> **Overview**
> Lowers the **default Sentry `tracesSampleRate`** for production
extension builds from **0.75% to 0.5%** in `getTracesSampleRate`
(`setupSentry.js`).
> 
> CI branch rates, `METAMASK_DEBUG` (1.0), fake DSN, and manifest
`flags.sentry.tracesSampleRate` overrides are unchanged—only the
fallback when none of those apply.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
fadb937. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Co-authored-by: Vaibhav Goyal <57912679+v-goyal@users.noreply.github.com>
@metamask-ci metamask-ci Bot added the team-bots Bot team (for MetaMask Bot, Runway Bot, etc.) label Jul 15, 2026
@sonarqubecloud

Copy link
Copy Markdown

@HowardBraham
HowardBraham merged commit 143f29b into release/13.40.0 Jul 15, 2026
104 of 198 checks passed
@HowardBraham
HowardBraham deleted the runway-cherry-pick-13.40.0-1784079358 branch July 15, 2026 01:58
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 15, 2026
@metamaskbot metamaskbot added the release-13.40.0 Issue or pull request that will be included in release 13.40.0 label Jul 15, 2026
@gauthierpetetin

Copy link
Copy Markdown
Contributor

No release label on PR. Adding release label release-13.40.0 on PR, as PR was cherry-picked in branch 13.40.0.

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

Labels

release-13.40.0 Issue or pull request that will be included in release 13.40.0 team-bots Bot team (for MetaMask Bot, Runway Bot, etc.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants