Merge branch 'develop' into nh/automatic-function-middleware-instrumentation

Author: nicohrubec

.claude/skills/add-cdn-bundle/SKILL.md

@@ -0,0 +1,56 @@
+---
+name: add-cdn-bundle
+description: Create a new CDN bundle for the browser package with specified features
+argument-hint: <feature-combo> (e.g., replay.logs.metrics, tracing.logs, tracing.replay.feedback.logs.metrics)
+---
+
+# Add CDN Bundle Skill
+
+This skill creates a new CDN bundle for the browser package that includes a specific combination of features.
+
+## Input
+
+The user provides a feature combination using dot notation:
+
+- `logs.metrics` - Bundle with logs and metrics
+- `replay.logs.metrics` - Bundle with replay, logs, and metrics
+- `tracing.replay.logs` - Bundle with tracing, replay, and logs
+- `tracing.replay.feedback.logs.metrics` - Full featured bundle
+
+**Feature order in bundle names:** `tracing` → `replay` → `feedback` → `logs` → `metrics`
+
+## Instructions
+
+Follow the detailed guide at [docs/adding-cdn-bundle.md](../../../docs/adding-cdn-bundle.md) to create the bundle.
+
+### Quick Reference - Naming Conventions
+
+Given a feature combination, derive these variants:
+
+| Placeholder                     | Example (`replay.logs.metrics`) |
+| ------------------------------- | ------------------------------- |
+| `{FEATURE_COMBO}`               | `replay.logs.metrics`           |
+| `{feature_combo}`               | `replay_logs_metrics`           |
+| `{featureCombo}`                | `replayLogsMetrics`             |
+| `{Human Readable Features}`     | `Replay, Logs, Metrics`         |
+| `{Human Readable Feature List}` | `Replay, Logs, and Metrics`     |
+
+### Quick Reference - Files to Create/Modify
+
+1. **Create** `packages/browser/src/index.bundle.{FEATURE_COMBO}.ts`
+2. **Create** `packages/browser/test/index.bundle.{FEATURE_COMBO}.test.ts`
+3. **Modify** `packages/browser/rollup.bundle.config.mjs`
+4. **Modify** `.size-limit.js`
+5. **Modify** `dev-packages/browser-integration-tests/package.json`
+6. **Modify** `dev-packages/browser-integration-tests/utils/generatePlugin.ts`
+7. **Modify** `.github/workflows/build.yml`
+
+### Verification Steps
+
+After making changes:
+
+```bash
+yarn lint
+cd packages/browser && yarn build:dev
+cd packages/browser && yarn test
+```

.cursor/commands/add_cdn_bundle.md

@@ -0,0 +1,35 @@
+# Add CDN Bundle for `{FEATURE_COMBO}`
+
+Create a new CDN bundle for the browser package that includes `{FEATURE_COMBO}` (e.g., `replay.logs.metrics`, `tracing.logs`, etc.).
+
+## Instructions
+
+Follow the detailed guide at [docs/adding-cdn-bundle.md](../../docs/adding-cdn-bundle.md) to create the bundle.
+
+## Quick Reference - Naming Conventions
+
+| Placeholder                     | Example (`replay.logs.metrics`) |
+| ------------------------------- | ------------------------------- |
+| `{FEATURE_COMBO}`               | `replay.logs.metrics`           |
+| `{feature_combo}`               | `replay_logs_metrics`           |
+| `{featureCombo}`                | `replayLogsMetrics`             |
+| `{Human Readable Features}`     | `Replay, Logs, Metrics`         |
+| `{Human Readable Feature List}` | `Replay, Logs, and Metrics`     |
+
+## Quick Reference - Files to Create/Modify
+
+1. **Create** `packages/browser/src/index.bundle.{FEATURE_COMBO}.ts`
+2. **Create** `packages/browser/test/index.bundle.{FEATURE_COMBO}.test.ts`
+3. **Modify** `packages/browser/rollup.bundle.config.mjs`
+4. **Modify** `.size-limit.js`
+5. **Modify** `dev-packages/browser-integration-tests/package.json`
+6. **Modify** `dev-packages/browser-integration-tests/utils/generatePlugin.ts`
+7. **Modify** `.github/workflows/build.yml`
+
+## Verification Steps
+
+After making changes:
+
+1. Run `yarn lint` to check for linting issues
+2. Run `cd packages/browser && yarn build:dev` to verify TypeScript compilation
+3. Run `cd packages/browser && yarn test` to run the unit tests

.github/dependabot.yml

@@ -12,21 +12,21 @@ updates:
     directory: '/'
     schedule:
       interval: 'weekly'
-    allow:
-      - dependency-name: '@sentry/*'
-      - dependency-name: '@playwright/test'
-      - dependency-name: '@opentelemetry/*'
     ignore:
       - dependency-name: '@opentelemetry/instrumentation'
       - dependency-name: '@opentelemetry/instrumentation-*'
+      - dependency-name: 'typescript'
     groups:
       opentelemetry:
         patterns:
           - '@opentelemetry/*'
+      remix:
+        patterns:
+          - '@remix-run/*'
     versioning-strategy: increase
     commit-message:
       prefix: feat
-      prefix-development: feat
+      prefix-development: chore
       include: scope
     exclude-paths:
-      - 'dev-packages/e2e-tests/test-applications/'
+      - 'dev-packages/e2e-tests/test-applications/**'

.github/workflows/build.yml

@@ -175,9 +175,6 @@ jobs:
             'nx-never-restore'}}
 
       - name: Build packages
-        # Set the CODECOV_TOKEN for Bundle Analysis
-        env:
-          CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
         run: yarn build
 
       - name: Upload build artifacts
@@ -431,18 +428,12 @@ jobs:
         run: yarn test:ci:browser
         if: github.event_name != 'pull_request'
 
-      - name: Compute test coverage
-        uses: codecov/codecov-action@v5
-        with:
-          token: ${{ secrets.CODECOV_TOKEN }}
-
-      - name: Upload test results to Codecov
+      - name: Parse and Upload Coverage
         if: cancelled() == false
         continue-on-error: true
-        uses: codecov/test-results-action@v1
+        uses: getsentry/codecov-action@main
         with:
-          files: packages/**/*.junit.xml
-          token: ${{ secrets.CODECOV_TOKEN }}
+          token: ${{ secrets.GITHUB_TOKEN }}
 
   job_bun_unit_tests:
     name: Bun Unit Tests
@@ -538,18 +529,12 @@ jobs:
         env:
           NODE_VERSION: ${{ matrix.node }}
 
-      - name: Compute test coverage
-        uses: codecov/codecov-action@v5
-        with:
-          token: ${{ secrets.CODECOV_TOKEN }}
-
-      - name: Upload test results to Codecov
+      - name: Parse and Upload Coverage
         if: cancelled() == false
         continue-on-error: true
-        uses: codecov/test-results-action@v1
+        uses: getsentry/codecov-action@main
         with:
-          files: packages/**/*.junit.xml
-          token: ${{ secrets.CODECOV_TOKEN }}
+          token: ${{ secrets.GITHUB_TOKEN }}
 
   job_browser_playwright_tests:
     name:
@@ -566,21 +551,24 @@ jobs:
           - esm
           - bundle
           - bundle_min
+          - bundle_logs_metrics
           - bundle_replay
+          - bundle_replay_logs_metrics
           - bundle_tracing
           - bundle_tracing_logs_metrics
           - bundle_tracing_replay
+          - bundle_tracing_replay_logs_metrics
           - bundle_tracing_replay_feedback
-          - bundle_tracing_replay_feedback_min
           - bundle_tracing_replay_feedback_logs_metrics
+          - bundle_tracing_replay_feedback_logs_metrics_min
         project:
           - chromium
         include:
           # Only check all projects for full bundle
           # We also shard the tests as they take the longest
-          - bundle: bundle_tracing_replay_feedback_min
+          - bundle: bundle_tracing_replay_feedback_logs_metrics_min
             project: 'webkit'
-          - bundle: bundle_tracing_replay_feedback_min
+          - bundle: bundle_tracing_replay_feedback_logs_metrics_min
             project: 'firefox'
           - bundle: esm
             project: chromium
@@ -641,13 +629,13 @@ jobs:
           overwrite: true
           retention-days: 7
 
-      - name: Upload test results to Codecov
+      - name: Parse and Upload Coverage
         if: cancelled() == false
         continue-on-error: true
-        uses: codecov/test-results-action@v1
+        uses: getsentry/codecov-action@main
         with:
+          token: ${{ secrets.GITHUB_TOKEN }}
           directory: dev-packages/browser-integration-tests
-          token: ${{ secrets.CODECOV_TOKEN }}
 
   job_browser_loader_tests:
     name: PW ${{ matrix.bundle }} Tests
@@ -702,13 +690,13 @@ jobs:
           overwrite: true
           retention-days: 7
 
-      - name: Upload test results to Codecov
+      - name: Parse and Upload Coverage
+        uses: getsentry/codecov-action@main
         if: cancelled() == false
         continue-on-error: true
-        uses: codecov/test-results-action@v1
         with:
+          token: ${{ secrets.GITHUB_TOKEN }}
           directory: dev-packages/browser-integration-tests
-          token: ${{ secrets.CODECOV_TOKEN }}
 
   job_check_for_faulty_dts:
     name: Check for faulty .d.ts files
@@ -776,13 +764,13 @@ jobs:
         working-directory: dev-packages/node-integration-tests
         run: yarn test
 
-      - name: Upload test results to Codecov
+      - name: Parse and Upload Coverage
         if: cancelled() == false
         continue-on-error: true
-        uses: codecov/test-results-action@v1
+        uses: getsentry/codecov-action@main
         with:
+          token: ${{ secrets.GITHUB_TOKEN }}
           directory: dev-packages/node-integration-tests
-          token: ${{ secrets.CODECOV_TOKEN }}
 
   job_cloudflare_integration_tests:
     name: Cloudflare Integration Tests
@@ -843,13 +831,13 @@ jobs:
           cd packages/remix
           yarn test:integration:ci
 
-      - name: Upload test results to Codecov
+      - name: Parse and Upload Coverage
         if: cancelled() == false
         continue-on-error: true
-        uses: codecov/test-results-action@v1
+        uses: getsentry/codecov-action@main
         with:
           directory: packages/remix
-          token: ${{ secrets.CODECOV_TOKEN }}
+          token: ${{ secrets.GITHUB_TOKEN }}
 
   job_e2e_prepare:
     name: Prepare E2E tests
@@ -1034,13 +1022,13 @@ jobs:
           retention-days: 7
           if-no-files-found: ignore
 
-      - name: Upload test results to Codecov
+      - name: Parse and Upload Coverage
         if: cancelled() == false
         continue-on-error: true
-        uses: codecov/test-results-action@v1
+        uses: getsentry/codecov-action@main
         with:
           directory: dev-packages/e2e-tests
-          token: ${{ secrets.CODECOV_TOKEN }}
+          token: ${{ secrets.GITHUB_TOKEN }}
 
   # - We skip optional tests on release branches
   job_optional_e2e_tests:

.github/workflows/external-contributors.yml

@@ -37,13 +37,13 @@ jobs:
 
       - name: Generate GitHub App token
         id: app-token
-        uses: actions/create-github-app-token@v1
+        uses: actions/create-github-app-token@v2
         with:
           app-id: ${{ vars.GITFLOW_APP_ID }}
           private-key: ${{ secrets.GITFLOW_APP_PRIVATE_KEY }}
 
       - name: Create PR with changes
-        uses: peter-evans/create-pull-request@98357b18bf14b5342f975ff684046ec3b2a07725
+        uses: peter-evans/create-pull-request@c0f553fe549906ede9cf27b5156039d195d2ece0
         with:
           token: ${{ steps.app-token.outputs.token }}
           commit-message: 'chore: Add external contributor to CHANGELOG.md'

.github/workflows/gitflow-sync-develop.yml

@@ -27,7 +27,7 @@ jobs:
 
       - name: Generate GitHub App token
         id: app-token
-        uses: actions/create-github-app-token@v1
+        uses: actions/create-github-app-token@v2
         with:
           app-id: ${{ vars.GITFLOW_APP_ID }}
           private-key: ${{ secrets.GITFLOW_APP_PRIVATE_KEY }}

.gitignore

@@ -64,3 +64,6 @@ packages/gatsby/gatsby-node.d.ts
 
 #junit reports
 packages/**/*.junit.xml
+
+# Local Claude Code settings that should not be committed
+.claude/settings.local.json

.prettierignore

@@ -2,3 +2,6 @@ packages/browser/test/loader.js
 packages/replay-worker/examples/worker.min.js
 dev-packages/browser-integration-tests/fixtures
 **/test.ts-snapshots/**
+
+/.nx/cache
+/.nx/workspace-data