chore(deps-dev): bump vue from 3.5.35 to 3.5.38#125
Conversation
Deploying fs-packages with
|
| Latest commit: |
8469854
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://4258a0a2.fs-packages.pages.dev |
| Branch Preview URL: | https://dependabot-npm-and-yarn-vue-ic6c.fs-packages.pages.dev |
038c810 to
de3ed62
Compare
Bumps [vue](https://github.com/vuejs/core) from 3.5.35 to 3.5.38. - [Release notes](https://github.com/vuejs/core/releases) - [Changelog](https://github.com/vuejs/core/blob/main/CHANGELOG.md) - [Commits](vuejs/core@v3.5.35...v3.5.38) --- updated-dependencies: - dependency-name: vue dependency-version: 3.5.38 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
de3ed62 to
d0ff195
Compare
Dependabot's vue 3.5.35 -> 3.5.38 bump was partial: workspace packages moved to 3.5.38 while the root copy stayed at 3.5.35, pinned by @vue/server-renderer@3.5.35's exact `peer vue@"3.5.35"` (pulled in via @vue/test-utils). vue-router resolves the root copy (3.5.35) and packages/router resolves its own (3.5.38); vue's [RefSymbol] is a unique symbol per copy, so the ShallowRef/Ref brands no longer match -> TS2741/TS2345 in router.ts + the components.ts overload error. Force a single vue across the workspace via a root `overrides` entry (range, so future bumps still float). This re-hoists @vue/server-renderer@3.5.38 to the root, keeping @vue/test-utils able to resolve it. typecheck, build, lint, format, validate:dist and the router suite (93/93) all pass with one vue copy in the tree. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Pushed Cause: this bump was partial. Workspace packages moved to vue Fix: a single root Verified locally at full |
Goosterhof
left a comment
There was a problem hiding this comment.
✅ Approve-worthy
0 blockers · 1 concern · 0 nits · 0 praise · 1 inline
Dependabot bumps vue 3.5.35→3.5.38 (dev-dependency, all bug-fix patches). Both required checks green (check 4m15s, Cloudflare Pages pass). One scope note: the PR body says "bump vue from 3.5.35 to 3.5.38," but the diff does more than that — it raises the published peerDependencies.vue floor from ^3.5.33 to ^3.5.38 across all 8 consumer packages, and adds a new overrides.vue pin to the root package.json. Neither is a correctness problem, but the body understates the consumer-facing surface. The peerDependency floor bump is anchored inline.
Findings (detail inline)
package.json:46— newoverrides.vuepin + the unmentioned peerDependency floor bumps (concern).
Automated war-room agent review — posted because this PR carries the Agent Review Requested label.
Review Loop · 9/10 · PASSfs-packages #125 · AC anchor: none (no issue_key / plan dir / PR AC section — Dependabot dep bump) · head Tip No findings — clean against the review checklist. Actionmerge-ready |
jasperboerhof
left a comment
There was a problem hiding this comment.
Auto-approved — review-loop verdict PASS, CI green, no human blocker, no open MAJOR+ threads. See the verdict comment + inline notes.
Bumps vue from 3.5.35 to 3.5.38.
Release notes
Sourced from vue's releases.
Changelog
Sourced from vue's changelog.
Commits
478e3e8release: v3.5.3830ba647chore(release): make release publishing resumable (#14953)c00b021release: v3.5.3711ac8b4release: v3.5.3643eba78chore(deps): update lint (#14935)3bc7290chore(deps): update build (#14901)8203fc5chore(deps): update all non-major dependencies (#14938)8f4bc85chore(deps): update compiler (#14900)8e63cdfchore(deps): update dependency npm-run-all2 to v9 (#14939)131291achore(deps): update test (#14936)