Skip to content

chore(deps): bump five npm packages in one branch - #53

Merged
htcom-code merged 1 commit into
mainfrom
chore/npm-deps-bump
Sep 7, 2026
Merged

htcom-code merged 1 commit into
mainfrom
chore/npm-deps-bump

Conversation

@htcom-code

Copy link
Copy Markdown
Owner

Consolidates this week's two dependabot PRs into one branch, so the tree is
reviewed and merged once instead of twice.

Closes #51
Closes #52

What moves

package from to reaches the bundle?
lucide-react 1.34.0 1.39.0 yes — the only runtime bump
@types/node 26.3.0 26.4.1 no (types)
@vitejs/plugin-react 6.1.0 6.1.1 no (build)
oxlint 1.80.0 1.81.0 no (lint)
shadcn 4.19.0 4.20.1 no (CLI)

Why one branch

Both PRs rewrite package.json and package-lock.json. Merging them one at a
time makes the second wait for a dependabot rebase and a second CI run to reach
the same tree, and leaves two commits in the history for one week of updates.

The lockfile is dependabot's, not a re-resolution

The two dependabot branches merge onto main with no conflict, and the result
was installed with npm ci — no npm install, so nothing was re-resolved
inside the semver ranges. Every one of the five packages is pinned to exactly
the version dependabot picked. Comparing the whole lockfile against main, the
only other changes are oxlint's 19 platform binding packages following it to
1.81.0; nothing else changed, and nothing was added or removed.

Verification

On the merged tree, the full local gate:

  • npm run lint — exit 0, 7 warnings, all the pre-existing
    only-export-components ones. oxlint 1.81.0 reports no new diagnostic; its
    breaking change (react-compiler rules split per category) landed in 1.79.0,
    which this repo is already past.
  • npm run build — tsc -b clean, 481.11 kB / 152.58 kB gzip. This is what
    clears lucide-react: all 22 icons imported across 13 files still resolve,
    so the minor brought no removal or rename.
  • npm test — 15 files, 152 tests, all passing.

@vitejs/plugin-react 6.1.1 makes recoverable React Compiler diagnostics
opt-in behind compiler.logDiagnostics; vite.config.ts does not set that
option, so the change is inert here.

Known audit finding, unchanged

npm audit reports one moderate advisory on qs 6.15.3, pulled in through
shadcn → @modelcontextprotocol/sdk → express. It is already on main at
the same version and this branch does not move it. npm audit --omit=dev
reports 0 — the chain is a dev-only CLI and never enters the bundle.

- dependabot opened #51 and #52 for the same week's updates, and both
  rewrite package.json and the lockfile; merging them one at a time
  costs a rebase round and a second CI run for one green tree
- every version is pinned to what dependabot resolved, so this branch
  is the two PRs applied together and not a re-resolution: the only
  other lockfile change is oxlint's platform bindings following 1.81.0
- lucide-react is the sole runtime bump; the rest are types and
  toolchain, and shadcn is a CLI that never enters the bundle

Co-Authored-By: htjulia <htjulia1@gmail.com>
@htcom-code
htcom-code merged commit 65ea0ed into main Sep 7, 2026
4 checks passed
@htcom-code
htcom-code deleted the chore/npm-deps-bump branch September 7, 2026 04:26
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