Skip to content

fix(analytics-browser): bump up GTM version with new session replay#1709

Open
daniel-graham-amplitude wants to merge 1 commit intohotfix/2.39from
SR-3531-upgrade-psr-gtm
Open

fix(analytics-browser): bump up GTM version with new session replay#1709
daniel-graham-amplitude wants to merge 1 commit intohotfix/2.39from
SR-3531-upgrade-psr-gtm

Conversation

@daniel-graham-amplitude
Copy link
Copy Markdown
Collaborator

@daniel-graham-amplitude daniel-graham-amplitude commented Apr 30, 2026

Summary

Upgrade plugin-session-replay-browser to 1.27.10 (hardcode the script). Add dummy change to analytics-browser to force a patch upgrade

Checklist

  • Does your PR title have the correct title format?
  • Does your PR have a breaking change?: No

Note

Low Risk
Changes are limited to release/publish configuration plus a comment-only code change; minimal impact on runtime behavior, with primary risk being altered publish semantics.

Overview
Updates release tooling: lerna.json now sets excludeDependents: true for lerna version, and deploy:publish adds --no-git-checks to align with the dry-run publish script.

Adds a no-op comment in packages/analytics-browser/src/index.ts to force a patch release (e.g., for GTM deployment), without changing runtime behavior.

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

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: excludeDependents is not a valid lerna version option
    • Removed the unsupported excludeDependents key from command.version so the Lerna configuration no longer implies ignored behavior.

Create PR

Or push these changes by commenting:

@cursor push 1ed8ee8fcc
Preview (1ed8ee8fcc)
diff --git a/lerna.json b/lerna.json
--- a/lerna.json
+++ b/lerna.json
@@ -8,8 +8,7 @@
       "conventionalCommits": true,
       "message": "chore(release): publish",
       "preid": "beta",
-      "exact": true,
-      "excludeDependents": true
+      "exact": true
     }
   },
   "$schema": "node_modules/lerna/schemas/lerna-schema.json"

You can send follow-ups to the cloud agent here.

Reviewed by Cursor Bugbot for commit f535b12. Configure here.

Comment thread lerna.json
"preid": "beta",
"exact": true
"exact": true,
"excludeDependents": true
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

excludeDependents is not a valid lerna version option

Medium Severity

excludeDependents is a filter option for commands like lerna run and lerna ls (requiring --since), not a recognized configuration option for the version command. Placed under command.version in lerna.json, it will be silently ignored by lerna, meaning dependent packages will still receive cascading version bumps during future releases. If the team relies on this setting to control versioning scope, unexpected bumps will occur.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit f535b12. Configure here.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant