Skip to content

chore(deps): land bot dependency bumps in one consolidated lockfile - #1356

Merged
h4yfans merged 1 commit into
mainfrom
deps/security-bumps-and-motion-migration
Aug 12, 2026
Merged

chore(deps): land bot dependency bumps in one consolidated lockfile#1356
h4yfans merged 1 commit into
mainfrom
deps/security-bumps-and-motion-migration

Conversation

@h4yfans

@h4yfans h4yfans commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Consolidates five stale bot PRs into a single change against current main. Each of them was 162–348 commits behind and every one touched pnpm-lock.yaml, so merging them in sequence would have meant eight rounds of conflict resolution and a real risk of landing a lockfile that no longer matched the manifests. This is one coherent regen instead.

What landed

Source Change
#1197 prettier ^3.8.3^3.9.6
#1190 ip-address override >=10.1.1>=10.3.1 (security) — resolves 10.5.0
#1189 hono override >=4.12.27>=4.12.34 (security) — resolves 4.13.1
#1194 nanoid@>=4.0.0 <5.0.9 override 5.0.95.1.16
#1196 landing: framer-motionmotion ^12.24.7

The motion swap needed more than the bot did

#1196 replaced the package in apps/landing/package.json but left all 49 from 'framer-motion' imports in place. Once framer-motion is off the manifest those no longer resolve, so this repoints them at motion/react and fixes the manualChunks entry in apps/landing/vite.config.ts that still named the old package.

Deliberately left out

Both advisories are already closed by the pins currently on main (3.3.8 / 5.0.95.1.16), so these two add breakage risk without security value. Left open for a separate call.

Already obsolete

Verification

  • pnpm install --frozen-lockfileAlready up to date (no ERR_PNPM_OUTDATED_LOCKFILE); lockfile diff is 41/41 lines, no incidental package sweep
  • pnpm --filter @memry/landing build → pass (typecheck + vite build + prerender) — this is the real proof for the motion migration
  • pnpm typecheck → pass · pnpm lint → pass · git diff --check → clean
  • pnpm test8/8 tasks successful: desktop 1242 passed/1 skipped, sync-server 60, contracts 50, i18n 11, sync-harness 5

Pre-existing and untouched by this branch: apps/landing/src/pages/Checkout.tsx fails react-hooks/set-state-in-effect under pnpm --filter @memry/landing lint. The file is byte-identical to origin/main and fails there too.

Bundles five stale bot PRs into a single change against current main so
one coherent lockfile lands instead of nine conflicting stale ones.

- prettier ^3.8.3 -> ^3.9.6 (#1197)
- ip-address override >=10.1.1 -> >=10.3.1, resolves 10.5.0 (#1190, security)
- hono override >=4.12.27 -> >=4.12.34, resolves 4.13.1 (#1189, security)
- nanoid@>=4.0.0 <5.0.9 override 5.0.9 -> 5.1.16 (#1194)
- framer-motion -> motion ^12.24.7 in landing (#1196)

The renovate PR for the motion swap replaced the package but left all 49
`from 'framer-motion'` imports untouched, which does not resolve once
framer-motion is off the manifest. Repointed them at `motion/react` and
updated the vite manualChunks entry that named the old package.

Left out deliberately: #1193 (nanoid 6.0.0 major) and #1195 (forces a 3.x
transitive to v5) — both advisories are already closed by the existing
pins, so they add breakage risk without security value.
@github-actions

Copy link
Copy Markdown

React Doctor found no new issues. 🎉

Reviewed by React Doctor for commit 6a41dd8.

@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant