Skip to content

Bump pnpm deps to latest within existing semver ranges - #1234

Draft
dgee2 wants to merge 1 commit into
mainfrom
deps-bulk-minor-patch
Draft

Bump pnpm deps to latest within existing semver ranges#1234
dgee2 wants to merge 1 commit into
mainfrom
deps-bulk-minor-patch

Conversation

@dgee2

@dgee2 dgee2 commented Aug 10, 2026

Copy link
Copy Markdown
Owner

Bulk minor/patch bump across vue, vue-router, auth0-vue, tanstack-query, vite, vitest, storybook, eslint, playwright, and other tooling in ui/menu-website. All bumps stay within their current major version.

Also:

  • Drops the @storybook/addon-vitest patch: the Windows absolute-path bug it worked around is fixed differently upstream in 10.5.7, and pnpm can no longer apply the patch (ERR_PNPM_UNUSED_PATCH).
  • Adds a pnpm override pinning vue to a single resolved version: @auth0/auth0-vue declares vue as a regular (non-peer) dependency, so bumping our own vue left two incompatible vue installs, breaking app.use(createAuth0()) under vue-tsc.
  • Adds a pnpm override pinning msw to 2.12.14: bumping vitest pulled in msw 2.15 transitively via @vitest/mocker, which is incompatible with msw-storybook-addon@2.0.6's use of worker.context. msw and msw-storybook-addon move together once the addon is upgraded to v3 (see the msw-storybook-addon PR later in this stack, which removes this override).
  • Removes lint suppressions and assertions the eslint/typescript-eslint bumps flag as stale: the @tanstack/query/exhaustive-deps eslint-disable directives in recipe-service.ts, and two as RecipeDetail type assertions in recipe-api.ts.
  • Adds .claude/launch.json so the dev server can be previewed in-browser for e2e smoke checks.

This is the first PR in a stack of dependency upgrades — see the linked stack for the rest.

Verification

  • lint, build, unit tests (57/57), storybook interaction tests (78/78)
  • dev-server smoke check: app boots with no console errors, nav renders

Note: the as RecipeDetail removal only shows up as necessary once src/generated/open-api/menu-api.ts exists — that file is gitignored, so a local lint run in a fresh worktree without it will not reproduce the error CI sees.

🤖 Generated with Claude Code

Bulk minor/patch bump across vue, vue-router, auth0-vue, tanstack-query,
vite, vitest, storybook, eslint, playwright, and other tooling in
ui/menu-website. All bumps stay within their current major version.

Also:
- Drop the @storybook/addon-vitest patch: the Windows absolute-path bug
  it worked around is fixed differently upstream in 10.5.7, and pnpm
  can no longer apply the patch (ERR_PNPM_UNUSED_PATCH).
- Add a pnpm override pinning vue to a single resolved version:
  @auth0/auth0-vue declares vue as a regular (non-peer) dependency, so
  bumping our own vue left two incompatible vue installs, breaking
  app.use(createAuth0()) under vue-tsc.
- Add a pnpm override pinning msw to 2.12.14: bumping vitest pulled in
  msw 2.15 transitively via @vitest/mocker, which is incompatible with
  msw-storybook-addon@2.0.6's use of worker.context. msw and
  msw-storybook-addon move together once the addon is upgraded to v3.
- Remove now-unused @tanstack/query/exhaustive-deps eslint-disable
  directives, and two now-unnecessary `as RecipeDetail` assertions in
  recipe-api.ts, that the eslint/typescript-eslint bumps flag as stale.
- Add .claude/launch.json so the dev server can be previewed in-browser
  for e2e smoke checks.

Co-authored-by: Claude <noreply@anthropic.com>
@dgee2
dgee2 force-pushed the deps-bulk-minor-patch branch from 3cbbb1e to 9ddb9bc Compare August 13, 2026 21:45
@sonarqubecloud

Copy link
Copy Markdown

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