Repository navigation
fix(deps): patch legacy browser tooling security (SDK-644) - #632
Conversation
|
bugbot run |
|
Skipping Bugbot: Bugbot is disabled for this repository. Visit the Bugbot dashboard to update your settings. |
|
Hosted follow-up on Bugbot is the remaining readiness blocker. The exact SDK-627, SDK-587, SDK-572, SDK-584, and SDK-562 retain the unresolved parent-toolchain portions recorded in the description. All linked Linear work remains In Progress; no tickets were closed, and no merge, release, or auto-merge was performed. |
|
Test it locally using Node 24.22.2:
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Summary
Patch compatible legacy Browser SDK and documentation-tooling vulnerabilities in one native Yarn refresh, and replace the reported Node 14 development pin with 24.22.2 (includes the October Brotli security fix). The ambiguous SDK-560 package is
trim, patched at 0.0.3. Upgrade the root YAML 2 consumer (lint-staged) within its existing major and retain webpack 5.104.1 for Docusaurus 2 compatibility: newer webpack versions break private plugin APIs/progress options used by this old toolchain.The native lockfile refresh also updates production dependencies within their declared ranges:
@amplitude/analytics-connector1.4.6 → 1.7.0,@babel/runtime7.28.6 → 7.29.10, andtslib2.5.0 → 2.9.1. Documentation CI now reads.node-version, because refreshed Cheerio/Undici dependencies require Node ≥20.18.1; no deployment was run.Validation (original implementation run; completed cloud revalidation is recorded below):
makeis blocked by the unaccepted Xcode license; the exact Rollup/uglify/Karma steps were run directly instead.Still unresolved — keep these tickets In Progress:
These parent-toolchain migrations need separate PRs/validation. Existing single-package Dependabot PRs are untouched. Other reported versions are patched in this graph; Semgrep confirmation requires merge/rescan. Newer unrelated audit findings are not claimed resolved.
Tickets
SDK-644, SDK-645, SDK-646, SDK-647, SDK-648, SDK-628, SDK-622, SDK-626, SDK-627, SDK-624, SDK-604, SDK-605, SDK-608, SDK-606, SDK-607, SDK-584, SDK-588, SDK-587, SDK-585, SDK-586, SDK-572, SDK-570, SDK-571, SDK-569, SDK-560, SDK-562, SDK-555, SDK-554.
Checklist
Cloud follow-up validation
Validated latest head
fedcea6a51457d3b086bfabfc5ca53c42da15128in hosted Linux using official SHA-256-verified Node 24.22.2:make build, lint, JSDoc generation, and documentation production build succeed.main, so these Karma results are hosted local validation, not a GitHub Test check. Thev8.xbranch response lists no required status-check contexts; full classic protection details are inaccessible to the connector.Readiness blocker
The exact
bugbot runtrigger was posted as humanMercy811for the head above. Cursor explicitly skipped it because Bugbot is disabled for this repository. No Bugbot review ran. Keep draft/needs-human-helpuntil a repository administrator enables Bugbot and starts a real review for this existing head in the dashboard. No duplicate trigger will be posted for the unchanged head.