test: added new sentry attributes (MetaMask#28138)

<!--
Please submit this PR as a draft initially.
Do not mark it as "Ready for review" until the template has been
completely filled out, and PR status checks have passed at least once.
-->

## **Description**
Added new scenario attributes to Sentry:
- Browserstack recording link
- Github action link
- Scenario team owner
<!--
Write a short description of the changes included in this pull request,
also include relevant motivation and context. Have in mind the following
questions:
1. What is the reason for the change?
2. What is the improvement/solution?
-->

## **Changelog**

<!--
If this PR is not End-User-Facing and should not show up in the
CHANGELOG, you can choose to either:
1. Write `CHANGELOG entry: null`
2. Label with `no-changelog`

If this PR is End-User-Facing, please write a short User-Facing
description in the past tense like:
`CHANGELOG entry: Added a new tab for users to see their NFTs`
`CHANGELOG entry: Fixed a bug that was causing some NFTs to flicker`

(This helps the Release Engineer do their job more quickly and
accurately)
-->

CHANGELOG entry:

## **Related issues**

Fixes:

## **Manual testing steps**

```gherkin
Feature: my feature name

  Scenario: user [verb for user action]
    Given [describe expected initial app state]

    When user [verb for user action]
    Then [describe expected outcome]
```

## **Screenshots/Recordings**

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

### **Before**

<!-- [screenshots/recordings] -->

### **After**

<!-- [screenshots/recordings] -->

## **Pre-merge author checklist**

- [ ] I've followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile
Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [ ] 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](https://jsdoc.app/) format
if applicable
- [ ] I've applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/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]
> **Medium Risk**
> Adds new external BrowserStack API calls and extra metadata to Sentry
transaction payloads during test teardown, which could impact reporting
or introduce intermittent failures if env/config changes. Core app logic
is unaffected.
> 
> **Overview**
> E2E performance runs now publish richer Sentry transactions by
**mirroring key scenario attributes**
(project/provider/team/status/retry/build variant/device/file path) into
both Sentry `tags` and each step `span.data` for easier filtering and
correlation.
> 
> The performance fixture now optionally fetches and attaches a
**BrowserStack session recording URL** (derived from the test
`sessionId` annotation and BrowserStack session details) and the Sentry
publisher also includes **GitHub Actions run/job metadata** (from
`GITHUB_*` env vars) in `extra` and span data.
> 
> Tests were updated to cover the new payload fields and to manage the
additional GitHub env vars during setup/teardown.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
8b2546d. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Author: javiergarciavera

tests/framework/fixtures/performance/performance-fixture.ts

@@ -4,6 +4,7 @@ import {
   type MetricsOutput,
 } from '../../../reporters/PerformanceTracker';
 import { publishPerformanceScenarioToSentry } from '../../../reporters/providers/sentry/PerformanceSentryPublisher';
+import { BrowserStackAPI } from '../../services/providers/browserstack/BrowserStackAPI';
 import {
   QualityGatesValidator,
   markQualityGateFailure,
@@ -17,6 +18,36 @@ interface PerformanceFixtures {
   performanceTracker: PerformanceTracker;
 }
 
+async function getBrowserStackRecordingUrl(
+  sessionId: string | null,
+  projectName: string,
+): Promise<string | null> {
+  if (!sessionId || !projectName.toLowerCase().includes('browserstack')) {
+    return null;
+  }
+
+  try {
+    const api = new BrowserStackAPI();
+    const sessionDetails = await api.getSessionDetails(sessionId);
+    if (!sessionDetails?.buildId) {
+      return null;
+    }
+
+    return api.buildSessionURL(sessionDetails.buildId, sessionId);
+  } catch {
+    return null;
+  }
+}
+
+function getSessionIdFromAnnotations(
+  annotations?: { type: string; description?: string }[],
+): string | null {
+  return (
+    annotations?.find((annotation) => annotation.type === 'sessionId')
+      ?.description ?? null
+  );
+}
+
 // Create a custom test fixture that handles performance tracking and cleanup
 export const test = base.extend<PerformanceFixtures>({
   // eslint-disable-next-line no-empty-pattern
@@ -76,13 +107,21 @@ export const test = base.extend<PerformanceFixtures>({
       );
     }
 
+    const sessionId = getSessionIdFromAnnotations(testInfo.annotations);
+
     if (metrics) {
+      const browserstackRecordingUrl = await getBrowserStackRecordingUrl(
+        sessionId,
+        testInfo.project?.name ?? 'unknown',
+      );
+
       try {
         const sentToSentry = await publishPerformanceScenarioToSentry({
           metrics,
           testTitle: testInfo.title,
           projectName: testInfo.project?.name ?? 'unknown',
           testFilePath: testInfo.file,
+          browserstackRecordingUrl,
           tags: testTags,
           status: testInfo.status,
           retry: testInfo.retry,
@@ -128,15 +167,6 @@ export const test = base.extend<PerformanceFixtures>({
 
     console.log('🔍 Looking for session ID...');
 
-    let sessionId: string | null = null;
-
-    if (testInfo?.annotations) {
-      sessionId =
-        testInfo.annotations.find(
-          (annotation) => annotation.type === 'sessionId',
-        )?.description ?? null;
-    }
-
     if (sessionId) {
       // Store session data as a test attachment for the reporter to find
       // Include team info and tags in session data

tests/reporters/providers/sentry/PerformanceSentryPublisher.test.ts

@@ -66,13 +66,21 @@ describe('PerformanceSentryPublisher', () => {
     process.env.E2E_PERFORMANCE_SENTRY_SAMPLE_RATE;
   const originalSentryEnabled = process.env.E2E_PERFORMANCE_SENTRY_ENABLED;
   const originalBuildVariant = process.env.E2E_PERFORMANCE_BUILD_VARIANT;
+  const originalGithubServerUrl = process.env.GITHUB_SERVER_URL;
+  const originalGithubRepository = process.env.GITHUB_REPOSITORY;
+  const originalGithubRunId = process.env.GITHUB_RUN_ID;
+  const originalGithubJob = process.env.GITHUB_JOB;
 
   beforeEach(() => {
     jest.clearAllMocks();
     delete process.env.E2E_PERFORMANCE_SENTRY_DSN;
     delete process.env.E2E_PERFORMANCE_SENTRY_SAMPLE_RATE;
     delete process.env.E2E_PERFORMANCE_SENTRY_ENABLED;
     delete process.env.E2E_PERFORMANCE_BUILD_VARIANT;
+    delete process.env.GITHUB_SERVER_URL;
+    delete process.env.GITHUB_REPOSITORY;
+    delete process.env.GITHUB_RUN_ID;
+    delete process.env.GITHUB_JOB;
     fetchMock = jest.spyOn(global, 'fetch');
   });
 
@@ -101,6 +109,30 @@ describe('PerformanceSentryPublisher', () => {
       process.env.E2E_PERFORMANCE_BUILD_VARIANT = originalBuildVariant;
     }
 
+    if (originalGithubServerUrl === undefined) {
+      delete process.env.GITHUB_SERVER_URL;
+    } else {
+      process.env.GITHUB_SERVER_URL = originalGithubServerUrl;
+    }
+
+    if (originalGithubRepository === undefined) {
+      delete process.env.GITHUB_REPOSITORY;
+    } else {
+      process.env.GITHUB_REPOSITORY = originalGithubRepository;
+    }
+
+    if (originalGithubRunId === undefined) {
+      delete process.env.GITHUB_RUN_ID;
+    } else {
+      process.env.GITHUB_RUN_ID = originalGithubRunId;
+    }
+
+    if (originalGithubJob === undefined) {
+      delete process.env.GITHUB_JOB;
+    } else {
+      process.env.GITHUB_JOB = originalGithubJob;
+    }
+
     fetchMock.mockRestore();
   });
 
@@ -124,6 +156,10 @@ describe('PerformanceSentryPublisher', () => {
     process.env.E2E_PERFORMANCE_SENTRY_DSN =
       'https://publicKey@o123.ingest.sentry.io/4567';
     process.env.E2E_PERFORMANCE_BUILD_VARIANT = 'exp';
+    process.env.GITHUB_SERVER_URL = 'https://github.com';
+    process.env.GITHUB_REPOSITORY = 'MetaMask/metamask-mobile';
+    process.env.GITHUB_RUN_ID = '12345';
+    process.env.GITHUB_JOB = 'e2e-performance-android';
     fetchMock.mockResolvedValue({
       ok: true,
       status: 200,
@@ -134,6 +170,8 @@ describe('PerformanceSentryPublisher', () => {
       testTitle: 'Import wallet flow',
       projectName: 'browserstack-android',
       testFilePath: 'tests/performance/onboarding/import-wallet.spec.js',
+      browserstackRecordingUrl:
+        'https://app-automate.browserstack.com/builds/build-123/sessions/sess-123',
       tags: ['@PerformanceOnboarding', '@PerformanceLaunch'],
       status: 'passed',
       retry: 0,
@@ -168,7 +206,40 @@ describe('PerformanceSentryPublisher', () => {
     expect(payload.measurements.scenario_total_time_ms.value).toBe(1300);
     expect(payload.tags.project_name).toBe('browserstack-android');
     expect(payload.tags.build_variant).toBe('exp');
+    expect(payload.tags.test_team).toBe('qa-automation');
     expect(payload.extra.timer_steps).toHaveLength(2);
+    expect(payload.extra.recording_url).toBe(
+      'https://app-automate.browserstack.com/builds/build-123/sessions/sess-123',
+    );
+    expect(payload.extra.github_job_url).toBe(
+      'https://github.com/MetaMask/metamask-mobile/actions/runs/12345',
+    );
+    expect(payload.extra.github_job_name).toBe('e2e-performance-android');
+    expect(payload.spans).toHaveLength(2);
+    expect(payload.spans[0].op).toBe('e2e.performance.step');
+    expect(payload.spans[0].data.project_name).toBe('browserstack-android');
+    expect(payload.spans[0].data.test_team).toBe('qa-automation');
+    expect(payload.spans[0].data.provider).toBe('browserstack');
+    expect(payload.spans[0].data.team_id).toBe('qa-automation');
+    expect(payload.spans[0].data.team_name).toBe('QA Automation');
+    expect(payload.spans[0].data.test_status).toBe('passed');
+    expect(payload.spans[0].data.retry).toBe(0);
+    expect(payload.spans[0].data.worker_index).toBe(3);
+    expect(payload.spans[0].data.build_variant).toBe('exp');
+    expect(payload.spans[0].data.device_name).toBe('Samsung Galaxy S23 Ultra');
+    expect(payload.spans[0].data.device_os_version).toBe('13.0');
+    expect(payload.spans[0].data.test_file_path).toBe(
+      'tests/performance/onboarding/import-wallet.spec.js',
+    );
+    expect(payload.spans[0].data.recording_url).toBe(
+      'https://app-automate.browserstack.com/builds/build-123/sessions/sess-123',
+    );
+    expect(payload.spans[0].data.github_job_url).toBe(
+      'https://github.com/MetaMask/metamask-mobile/actions/runs/12345',
+    );
+    expect(payload.spans[0].data.github_job_name).toBe(
+      'e2e-performance-android',
+    );
   });
 
   it('protects reserved aggregate keys from timer-key collisions', async () => {

tests/reporters/providers/sentry/PerformanceSentryPublisher.ts

@@ -13,6 +13,10 @@ const ENV_SENTRY_SAMPLE_RATE = 'E2E_PERFORMANCE_SENTRY_SAMPLE_RATE';
 const ENV_SENTRY_ENVIRONMENT = 'E2E_PERFORMANCE_SENTRY_ENVIRONMENT';
 const ENV_SENTRY_RELEASE = 'E2E_PERFORMANCE_SENTRY_RELEASE';
 const ENV_SENTRY_BUILD_VARIANT = 'E2E_PERFORMANCE_BUILD_VARIANT';
+const ENV_GITHUB_SERVER_URL = 'GITHUB_SERVER_URL';
+const ENV_GITHUB_REPOSITORY = 'GITHUB_REPOSITORY';
+const ENV_GITHUB_RUN_ID = 'GITHUB_RUN_ID';
+const ENV_GITHUB_JOB = 'GITHUB_JOB';
 const MAX_MEASUREMENT_KEY_LENGTH = 64;
 const RESERVED_MEASUREMENT_KEYS = [
   'scenario_total_time_ms',
@@ -24,6 +28,7 @@ interface PublishPerformanceScenarioOptions {
   testTitle: string;
   projectName: string;
   testFilePath?: string;
+  browserstackRecordingUrl?: string | null;
   tags: string[];
   status?: string;
   retry?: number;
@@ -53,6 +58,24 @@ interface SentryMeasurement {
   unit: 'millisecond';
 }
 
+interface MirroredScenarioAttributes {
+  project_name: string;
+  test_team: string;
+  provider: string;
+  team_id: string;
+  team_name: string;
+  test_status: string;
+  retry: number;
+  worker_index: number;
+  build_variant: 'rc' | 'exp' | 'unknown';
+  device_name: string;
+  device_os_version: string;
+  test_file_path: string;
+  recording_url: string | null;
+  github_job_url: string | null;
+  github_job_name: string | null;
+}
+
 function getEnvValue(key: string): string | undefined {
   return Reflect.get(process.env, key) as string | undefined;
 }
@@ -178,6 +201,17 @@ function parseSampleRate(rawSampleRate: string | undefined): number | null {
   return sampleRate;
 }
 
+function getGithubJobUrl(): string | null {
+  const serverUrl = getEnvValue(ENV_GITHUB_SERVER_URL);
+  const repository = getEnvValue(ENV_GITHUB_REPOSITORY);
+  const runId = getEnvValue(ENV_GITHUB_RUN_ID);
+  if (!serverUrl || !repository || !runId) {
+    return null;
+  }
+
+  return `${serverUrl}/${repository}/actions/runs/${runId}`;
+}
+
 export async function publishPerformanceScenarioToSentry(
   options: PublishPerformanceScenarioOptions,
 ): Promise<boolean> {
@@ -255,6 +289,35 @@ export async function publishPerformanceScenarioToSentry(
     };
   }
 
+  const provider = options.metrics.device.provider || 'unknown';
+  const teamId = options.metrics.team?.teamId || 'unknown';
+  const teamName = options.metrics.team?.teamName || 'unknown';
+  const testStatus = options.status || 'unknown';
+  const retry = options.retry ?? 0;
+  const workerIndex = options.workerIndex ?? 0;
+  const buildVariant = normalizeBuildVariant(
+    getEnvValue(ENV_SENTRY_BUILD_VARIANT),
+  );
+  const testFilePath = options.testFilePath || '';
+
+  const mirroredScenarioAttributes: MirroredScenarioAttributes = {
+    project_name: options.projectName,
+    test_team: teamId,
+    provider,
+    team_id: teamId,
+    team_name: teamName,
+    test_status: testStatus,
+    retry,
+    worker_index: workerIndex,
+    build_variant: buildVariant,
+    device_name: options.metrics.device.name,
+    device_os_version: options.metrics.device.osVersion,
+    test_file_path: testFilePath,
+    recording_url: options.browserstackRecordingUrl ?? null,
+    github_job_url: getGithubJobUrl(),
+    github_job_name: getEnvValue(ENV_GITHUB_JOB) ?? null,
+  };
+
   let cursor = startTimestamp;
   const spans = timerMeasurements.map((timerMeasurement) => {
     const spanStart = cursor;
@@ -276,6 +339,7 @@ export async function publishPerformanceScenarioToSentry(
         base_threshold_ms: timerMeasurement.baseThreshold,
         exceeded_ms: timerMeasurement.exceeded,
         percent_over: timerMeasurement.percentOver,
+        ...mirroredScenarioAttributes,
       },
     };
   });
@@ -310,21 +374,23 @@ export async function publishPerformanceScenarioToSentry(
     },
     tags: {
       source: 'appwright-e2e-performance',
-      project_name: options.projectName,
-      provider: options.metrics.device.provider || 'unknown',
-      team_id: options.metrics.team?.teamId || 'unknown',
-      team_name: options.metrics.team?.teamName || 'unknown',
-      test_status: options.status || 'unknown',
-      retry: String(options.retry ?? 0),
-      worker_index: String(options.workerIndex ?? 0),
-      build_variant: normalizeBuildVariant(
-        getEnvValue(ENV_SENTRY_BUILD_VARIANT),
-      ),
+      project_name: mirroredScenarioAttributes.project_name,
+      provider: mirroredScenarioAttributes.provider,
+      team_id: mirroredScenarioAttributes.team_id,
+      team_name: mirroredScenarioAttributes.team_name,
+      test_team: mirroredScenarioAttributes.test_team,
+      test_status: mirroredScenarioAttributes.test_status,
+      retry: String(mirroredScenarioAttributes.retry),
+      worker_index: String(mirroredScenarioAttributes.worker_index),
+      build_variant: mirroredScenarioAttributes.build_variant,
     },
     measurements,
     spans,
     extra: {
-      test_file_path: options.testFilePath || '',
+      test_file_path: mirroredScenarioAttributes.test_file_path,
+      recording_url: mirroredScenarioAttributes.recording_url,
+      github_job_url: mirroredScenarioAttributes.github_job_url,
+      github_job_name: mirroredScenarioAttributes.github_job_name,
       test_tags: options.tags,
       threshold_margin_percent: options.metrics.thresholdMarginPercent,
       has_thresholds: options.metrics.hasThresholds,