feat(deps): L3-0000 bundled dependabot updates from passing CI runs - #908
Conversation
Consolidates 24 open dependabot PRs whose CI passed. Excludes #907 (vite/@vitejs/plugin-react — failed build), #904 (markdown-it — failed tests), #808 (react-dom majors — failed CI), #807 (@eslint/js majors — failed CI). Direct dependency bumps in package.json: - dompurify: ^3.2.4 → ^3.4.12 (#899, supersedes #819) - @types/dompurify: ^3.0.5 → ^3.2.0 (#819) - uuid: ^11.1.0 → ^14.0.0 (#855) - react-zoom-pan-pinch: ^3.6.1 → ^3.7.0 (#811) - @commitlint/config-conventional: ^19.2.2 → ^20.4.1 (#810) - prettier: 3.3.3 → 3.8.1 (#809) Transitive lockfile bumps (via npm update --package-lock-only): - ws 8.20.0→8.21.1 (#906), postcss 8.5.6→8.5.23 (#905), form-data 4.0.4→4.0.6 (#903), fast-uri 3.1.2→3.1.4 (#902), immutable 5.0.3→5.1.9 (#900), svgo 3.3.2→3.3.4 (#898), linkify-it 5.0.0→5.0.2 (#897), axios 1.8.2→1.18.1 (#896), js-yaml 4.1.1→4.3.0 (#889), undici (#887), qs 6.14.1→6.15.2 (#873), follow-redirects 1.15.6→1.16.0 (#844), lodash & @microsoft/api-extractor (#842), picomatch (#840), lodash-es 4.17.21→4.18.1 (#839), handlebars 4.7.8→4.7.9 (#837), flatted 3.3.1→3.4.2 (#832), tar & npm (#824) Also includes cosmetic reformats from prettier 3.8: - Dropdown.tsx: single-line multi-extends - _vars.scss: multi-line $text-tokens uses leading-newline style - .cursor/**/*.md: markdown formatting normalization Verified: build passes (tsc + vite build), lint passes, 1042/1042 unit tests pass. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
✅ Deploy Preview for phillips-seldon ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Tip All tests passed and all changes approved!🟢 UI Tests: 4 visual and accessibility changes accepted as baselines |
|
🚀 Storybook preview is ready. • Preview: https://68b9f094608b90f3cfec5a06-whvfvsjuev.chromatic.com/ |
There was a problem hiding this comment.
Pull request overview
This PR consolidates multiple Dependabot updates into a single dependency-housekeeping change for the Seldon React + TypeScript component library, with a small amount of Prettier-driven formatting churn.
Changes:
- Bumped several direct dependencies/devDependencies (notably
dompurify,uuid,react-zoom-pan-pinch,@commitlint/config-conventional,prettier) and re-resolved the lockfile for a batch of transitive updates. - Applied Prettier formatting-only updates to a small set of TS/SCSS files.
- Normalized minor markdown formatting in
.cursor/command/agent docs.
Reviewed changes
Copilot reviewed 10 out of 11 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
package.json |
Direct dependency/devDependency version bumps. |
package-lock.json |
Lockfile re-resolution reflecting bundled Dependabot updates (direct + transitive). |
src/components/Dropdown/Dropdown.tsx |
Prettier-only formatting of the DropdownProps interface extends clause. |
src/scss/_vars.scss |
Prettier-only reformat of the long $text-tokens list for readability. |
.cursor/commands/run-tests-and-fix-failure.md |
Markdown normalization (removed empty list spacing lines). |
.cursor/commands/new-feature.md |
Markdown normalization (removed empty list spacing lines). |
.cursor/commands/lint-and-format.md |
Markdown normalization (removed empty list spacing lines). |
.cursor/agents/test-coverage-checker.md |
Markdown normalization (removed empty list spacing lines). |
.cursor/agents/playwright-route-checker.md |
Markdown normalization (removed empty list spacing lines). |
.cursor/agents/component-unit-test-checker.md |
Markdown normalization (removed empty list spacing lines). |
.cursor/agents/component-storybook-checker.md |
Markdown normalization (removed empty list spacing lines). |
| "@radix-ui/react-tabs": "^1.1.1", | ||
| "@radix-ui/react-toast": "^1.2.11", | ||
| "@types/dompurify": "^3.0.5", | ||
| "@types/dompurify": "^3.2.0", |
There was a problem hiding this comment.
Good catch — done in 241d8fb. Removed @types/dompurify entirely (dompurify 3.x provides its own types, and nothing in src/ imports it directly either).
dompurify 3.x ships its own type definitions, and @types/dompurify is now published as a deprecated stub whose only content is a "dompurify": "*" dependency (which can pull in duplicate dompurify resolutions). Nothing in src/ imports dompurify or DOMPurify, so removing the stub is a safe cleanup. Verified: build passes, lint passes, 1042/1042 unit tests pass. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
🎉 This PR is included in version 1.277.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Jira ticket
N/A — dependency housekeeping. Consolidates 24 open Dependabot PRs whose CI was passing.
Screenshots
Figma link
N/A — no design changes.
Summary
Consolidates 24 open Dependabot PRs whose CI passed into a single, reviewable diff, reducing the review-and-merge overhead of dealing with each dependency bump individually.
Explicitly excluded (their CI was failing on the dependabot branches themselves):
vite+@vitejs/plugin-react— build failuresmarkdown-it/markdownlint-cli— mixed failuresreact-dom+@types/react-dom(major bumps) — CI failed@eslint/js(major bump) — CI failedWHY: A bundled bump is easier to validate downstream in a single Chromatic +
phillips-public-remixsmoke test than 24 individual merges. Grouping into one PR also collapses one lockfile re-resolution instead of 24 sequential ones.Change List (describe the changes made to the files)
Direct dependency bumps in
package.json:dompurify^3.2.4 → ^3.4.12 (fix(deps): bump dompurify from 3.2.4 to 3.4.12 #899, supersedes fix(deps): bump dompurify and @types/dompurify #819)@types/dompurify^3.0.5 → ^3.2.0 (fix(deps): bump dompurify and @types/dompurify #819)uuid^11.1.0 → ^14.0.0 (fix(deps): bump uuid from 11.1.0 to 14.0.0 #855)react-zoom-pan-pinch^3.6.1 → ^3.7.0 (fix(deps): bump react-zoom-pan-pinch from 3.6.1 to 3.7.0 #811)@commitlint/config-conventional^19.2.2 → ^20.4.1 (build(deps-dev): bump @commitlint/config-conventional from 19.2.2 to 20.4.1 #810)prettier3.3.3 → 3.8.1 (build(deps-dev): bump prettier from 3.3.3 to 3.8.1 #809)Transitive lockfile-only bumps (18 PRs) applied via
npm update --package-lock-onlytargeting the specific packages Dependabot proposed:ws#906,postcss#905,form-data#903,fast-uri#902,immutable#900,svgo#898,linkify-it#897,axios#896,js-yaml#889,undici#887,qs#873,follow-redirects#844,lodash+@microsoft/api-extractor#842,picomatch#840,lodash-es#839,handlebars#837,flatted#832,tar+npm#824.Prettier 3.8 cosmetic reformats (no behavior change):
src/components/Dropdown/Dropdown.tsx— single-line multi-extends interfacesrc/scss/_vars.scss— leading-newline style for the long$text-tokenslist.cursor/**/*.md— minor markdown normalizationAcceptance Test (how to verify the PR)
@phillips/seldoninphillips-public-remixto a preview version of this branch and smoke-test the following pages/flows:RichText/ Portable Text (exercisesdompurify3.2 → 3.4)uuidv11 → v14 viaToastContextProvider)react-zoom-pan-pinch(image zoom lots)Regression Test
npm test.commitstill works via husky since@commitlint/config-conventionalwent 19 → 20 (this very commit already validates that path).Evidence of testing
Local verification against
main+ this branch:npm run build→ exit 0 (tsc + vite build)npm run lint→ exit 0 (tsc --noEmit, eslint, stylelint, markdownlint)npx vitest run --project=unit→ 1042 / 1042 tests passingNotable risk areas to spot-check during review:
uuidv11 → v14 (3 major versions). Used insrc/components/Toast/ToastContextProvider.tsxviav4 as uuidv4. Thev4()API is stable across all uuid majors, but worth verifying the Toast story in Chromatic.@commitlint/config-conventional19 → 20 — dev-only; affects commit-message linting. Husky commit-msg hook accepted this commit, so v20 rules are compatible with the current commit convention.prettier3.3 → 3.8 — cosmetic reformats visible in diff; anyone with in-flight PRs should rebase.dompurify3.2 → 3.4 — check any HTML sanitization surfaces in Chromatic.Things to look for during review
feat(scope): ...if aminorrelease should be triggered.phillipsclass prefix are using the prefix variabledata-testidattribute.🤖 Generated with Claude Code