📝 Add Claude Code skill for v6 to v7 SDK migration#4486
Conversation
Bundles Sizes Evolution
🚀 CPU PerformancePending... 🧠 Memory Performance
|
🎉 All green!❄️ No new flaky tests detected 🎯 Code Coverage (details) 🔗 Commit SHA: fab7c14 | Docs | Datadog PR Page | Give us feedback! |
- Add session store key rename (_dd_s → _dd_s_v2) migration note - Replace addTiming section with session_renewal view loading type - Add document resource timing based on PerformanceNavigationTiming - Fix markdown table alignment
Systematic 6-step upgrade guide covering CDN/npm version updates, crossorigin attribute, removed options, changed APIs, behavioral defaults, and infrastructure changes.
thomas-lebeau
force-pushed
the
benoit/release-notes-draft
branch
from
68e1f2f to
8702724
Compare
Content has been moved to the documentation repo: DataDog/documentation#36214
thomas-lebeau
changed the title
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cf3b8903e0
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| | `datadoghq-browser-agent.com/us1/v6/datadog-logs.js` | `datadoghq-browser-agent.com/us1/v7/datadog-logs.js` | | ||
| | `datadoghq-browser-agent.com/us1/v6/datadog-rum-slim.js` | `datadoghq-browser-agent.com/us1/v7/datadog-rum-slim.js` | | ||
|
|
||
| Replace `us1` with your site: `eu1`, `us3`, `us5`, `ap1`. For US1-FED, the pattern is flat: `datadog-rum-v7.js` (no site prefix). |
There was a problem hiding this comment.
For projects deployed on the AP2 Datadog site, this list leaves out the valid ap2 CDN path, even though the repo's datacenter metadata treats ap2 as a production datacenter and the public CDN setup includes /ap2/v6/... URLs. Users following this migration guide may not update https://www.datadoghq-browser-agent.com/ap2/v6/... to /ap2/v7/..., or may incorrectly choose AP1, which prevents the SDK from loading from the right site.
Useful? React with 👍 / 👎.
| | Cancelled request errors removed (Logs) | Aborted fetch/XHR no longer generate network error logs. | No action needed — reduces noise. | | ||
| | Logs always requires session storage | Without cookies or localStorage, Logs SDK won't start. | Use `sessionPersistence: 'memory'` for worker environments. | | ||
| | Session Replay: Change Records | New serialization format. More accurate, less bandwidth. | Update if you depend on raw segment format. | | ||
| | Async chunk names prefixed with `datadog` | e.g., `datadog-rum-recorder.js`. | Update CSP `script-src` rules or caching configs. | |
There was a problem hiding this comment.
For CDN RUM users with CSP or cache rules, datadog-rum-recorder.js is not the file v7 publishes: the webpack chunk names and deploy specs use chunks/datadogRecorder-<hash>-datadog-rum.js and chunks/datadogProfiler-<hash>-datadog-rum.js. Following this example would update allowlists/caches for a non-existent file and still leave Session Replay or profiling dynamic imports blocked.
Useful? React with 👍 / 👎.
- Add ap2 site to CDN replacement list - Fix grep patterns to use single quotes (avoid shell escaping issues) - Update async chunk name examples with hashed format (datadogRecorder/datadogProfiler) - Restructure CSP infrastructure section as a bulleted list for readability
3 participants
Motivation
Help users upgrade from Datadog Browser SDK v6 to v7 with a systematic, step-by-step guide they can run through Claude Code.
Changes
Adds
.claude/skills/upgrade-browser-sdk-v7/SKILL.md, a 6-step upgrade guide covering:v6→v7URL patterns and US1-FED flat layout)crossoriginattributebetaEncodeCookieOptions,allowFallbackToLocalStorage,trackBfcacheViews,usePciIntake, …)The release notes draft previously in this PR has been moved to the documentation repo: DataDog/documentation#36214
Test instructions
upgrade-browser-sdk-v7skill.Checklist