Skip to content

feat(deps): L3-0000 bundle remaining low-risk dependabot updates - #909

Merged
scottdickerson merged 1 commit into
mainfrom
chore/bundled-dependabot-low-risk-2
Jul 27, 2026
Merged

scottdickerson merged 1 commit into
mainfrom
chore/bundled-dependabot-low-risk-2

Conversation

@scottdickerson

Copy link
Copy Markdown
Contributor

Jira ticket

N/A — dependency housekeeping follow-up to #908.

Screenshots

Before After
N/A — no runtime UI changes. Verify via Chromatic diff on this PR.

Figma link

N/A — no design changes.

Summary

Follow-up to #908 which merged most of the low-risk dependabot updates. This PR picks up the two transitive bumps that #908 did not resolve because npm update --package-lock-only chose not to bump them at that time:

WHY: Same rationale as #908 — bundling low-risk lockfile bumps into one PR reduces per-PR review overhead. Every other PR carrying the dependency-upgrade-risk:low label is already satisfied on main at or above its target version (see Change List for the full accounting).

Change List (describe the changes made to the files)

package-lock.json only — 13-line diff:

  • node_modules/picomatch 2.3.1 → 2.3.2 (patch bump)
  • node_modules/lodash 4.17.23 → 4.18.1 (minor bump)

Other low-risk PRs already covered by main (no action needed here, they will be closed as duplicates):

PR Package Target Main state
#902 fast-uri 3.1.4 3.1.4 ✅
#900 immutable 5.1.9 5.1.9 ✅
#898 svgo 3.3.4 3.3.4 ✅
#897 linkify-it 5.0.2 5.0.2 ✅
#889 js-yaml 4.3.0 4.3.0 ✅
#887 undici 7.28.0 / 6.27.0 7.29.0 / 6.28.0 / 6.27.0 ✅
#873 qs 6.15.2 6.15.3 ✅ (higher)
#844 follow-redirects 1.16.0 1.16.0 ✅
#842 @microsoft/api-extractor 7.58.1 7.58.12 ✅ (higher)
#839 lodash-es 4.18.1 4.18.1 ✅
#837 handlebars 4.7.9 4.7.9 ✅
#832 flatted 3.4.2 3.4.3 ✅ (higher)
#824 tar + npm 11.12.1 11.18.0 ✅ (higher)
#819 dompurify + @types/dompurify 3.3.1 / 3.2.0 3.4.12 in #908; stub removed in #908
#811 react-zoom-pan-pinch 3.7.0 3.7.0 ✅
#809 prettier 3.8.1 3.8.1 ✅

Acceptance Test (how to verify the PR)

  1. CI on this PR passes (build + lint + unit tests + Chromatic).
  2. Review the Chromatic diff — no visual regressions expected (both bumps are transitive).
  3. After merge, dependabot will auto-close build(deps-dev): bump picomatch from 2.3.1 to 2.3.2 #840 and fix(deps): bump lodash and @microsoft/api-extractor #842 once main catches up; close the 15 fully-applied PRs manually or via bot.

Regression Test

  • picomatch is used by glob/vite/vitest/rollup toolchain — CI running here exercises all of them. If build and tests pass here, downstream consumers are unaffected.
  • lodash is used only transitively by @microsoft/api-extractor (see PR fix(deps): bump lodash and @microsoft/api-extractor #842). Not imported by anything in src/.

Evidence of testing

  • npm ci → clean install
  • npm run build → exit 0
  • npm run lint → exit 0 (tsc --noEmit, eslint, stylelint, markdownlint)
  • npx vitest run --project=unit1042 / 1042 tests passing

Things to look for during review

  • PR title should correctly describe the most significant type of commit. I.e. feat(scope): ... if a minor release should be triggered.
  • All commit messages follow convention and are appropriate for the changes
  • All references to phillips class prefix are using the prefix variable
  • All major areas have a data-testid attribute.
  • Document all props with jsdoc comments
  • All strings should be translatable.
  • Unit tests should be written and should have a coverage of 90% or higher in all areas.

🤖 Generated with Claude Code

Follow-up to #908 which merged most of the low-risk dependabot updates.
This PR picks up the two transitive bumps that #908 did not resolve:

- picomatch 2.3.1 → 2.3.2 (#840)
- lodash 4.17.23 → 4.18.1 (#842)

All other PRs carrying the `dependency-upgrade-risk:low` label are
already satisfied on main at or above their target versions:

- fast-uri #902 (main 3.1.4)
- immutable #900 (main 5.1.9)
- svgo #898 (main 3.3.4)
- linkify-it #897 (main 5.0.2)
- js-yaml #889 (main 4.3.0)
- undici #887 (main 6.28.0 / 6.27.0 / 7.29.0 across paths)
- qs #873 (main 6.15.3, higher than target)
- follow-redirects #844 (main 1.16.0)
- @microsoft/api-extractor #842 (main 7.58.12, higher than target)
- lodash-es #839 (main 4.18.1)
- handlebars #837 (main 4.7.9)
- flatted #832 (main 3.4.3, higher than target)
- tar & npm #824 (main npm 11.18.0, tar 7.5.19)
- dompurify & @types/dompurify #819 (dompurify 3.4.12 in #908,
  @types/dompurify stub removed in #908)
- react-zoom-pan-pinch #811 (main 3.7.0)
- prettier #809 (main 3.8.1)

Verified: build passes, lint passes, 1042/1042 unit tests pass.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 27, 2026 18:43
@netlify

netlify Bot commented Jul 27, 2026

Copy link
Copy Markdown

Deploy Preview for phillips-seldon ready!

Name Link
🔨 Latest commit 9db928b
🔍 Latest deploy log https://app.netlify.com/projects/phillips-seldon/deploys/6a67a6def481c5000818f809
😎 Deploy Preview https://deploy-preview-909--phillips-seldon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

@scottdickerson scottdickerson changed the title chore(deps): L3-0000 bundle remaining low-risk dependabot updates feat(deps): L3-0000 bundle remaining low-risk dependabot updates Jul 27, 2026
@chromatic-com

chromatic-com Bot commented Jul 27, 2026

Copy link
Copy Markdown

Tip

All tests passed and all changes approved!

🟢 UI Tests: 476 tests unchanged
🟢 UI Review: 238 stories published -- no changes
Storybook icon Storybook Publish: 238 stories published

@github-actions

Copy link
Copy Markdown

@scottdickerson
scottdickerson enabled auto-merge (squash) July 27, 2026 18:46
@scottdickerson
scottdickerson merged commit 1551ea9 into main Jul 27, 2026
17 checks passed
@scottdickerson
scottdickerson deleted the chore/bundled-dependabot-low-risk-2 branch July 27, 2026 18:46
davidicus pushed a commit that referenced this pull request Jul 27, 2026
# [1.277.0](v1.276.0...v1.277.0) (2026-07-27)

### Features

* **deps:** L3-0000 bundle remaining low-risk dependabot updates ([#909](#909)) ([1551ea9](1551ea9))
@davidicus

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.277.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants