chore(deps): bump the npm_and_yarn group across 4 directories with 6 updates#1967
chore(deps): bump the npm_and_yarn group across 4 directories with 6 updates#1967dependabot[bot] wants to merge 1 commit into
Conversation
|
Prompt To Fix All With AIFix the following 2 code review issues. Work through them one at a time, proposing concise fixes.
---
### Issue 1 of 2
packages/wallets/package.json:42
**Large codegen tool version jump may silently break generated client**
`@hey-api/openapi-ts` is jumping ~28 minor versions (0.69.2 → 0.97.3) with multiple breaking changes across that range (plugin API overhaul, changed default output structure, removal of `plugin.getSymbol()`). While the current `openapi-ts.config.ts` is minimal and looks compatible, the next time the codegen script is executed the output in `src/api/gen` will be regenerated with the new tool's output format. Depending on how the generated client differs, callers of the API client could silently break (renamed exports, changed type shapes). Consider running the codegen and verifying the diff in `src/api/gen` before or immediately after merging.
### Issue 2 of 2
apps/auth/remix-ssr/package.json:38
**Vite major version bump (5 → 6) needs a build/dev smoke-test**
Vite 6 introduced breaking changes from v5 (default build target change, SSR module handling updates, env transport path checks). This app's `vite.config.ts` uses `@remix-run/dev`'s `vitePlugin`, which gained official Vite 6 support around Remix 2.13. At `@remix-run/dev@^2.16.8` compatibility is expected, but a quick `remix vite:build` and `remix vite:dev` run is worth doing to confirm nothing regressed in the SSR-rendered auth flow.
Reviews (1): Last reviewed commit: "chore(deps): bump the npm_and_yarn group..." | Re-trigger Greptile |
| @@ -41,7 +41,7 @@ | |||
| "viem": "2.33.1" | |||
| }, | |||
There was a problem hiding this comment.
Large codegen tool version jump may silently break generated client
@hey-api/openapi-ts is jumping ~28 minor versions (0.69.2 → 0.97.3) with multiple breaking changes across that range (plugin API overhaul, changed default output structure, removal of plugin.getSymbol()). While the current openapi-ts.config.ts is minimal and looks compatible, the next time the codegen script is executed the output in src/api/gen will be regenerated with the new tool's output format. Depending on how the generated client differs, callers of the API client could silently break (renamed exports, changed type shapes). Consider running the codegen and verifying the diff in src/api/gen before or immediately after merging.
Prompt To Fix With AI
This is a comment left during a code review.
Path: packages/wallets/package.json
Line: 42
Comment:
**Large codegen tool version jump may silently break generated client**
`@hey-api/openapi-ts` is jumping ~28 minor versions (0.69.2 → 0.97.3) with multiple breaking changes across that range (plugin API overhaul, changed default output structure, removal of `plugin.getSymbol()`). While the current `openapi-ts.config.ts` is minimal and looks compatible, the next time the codegen script is executed the output in `src/api/gen` will be regenerated with the new tool's output format. Depending on how the generated client differs, callers of the API client could silently break (renamed exports, changed type shapes). Consider running the codegen and verifying the diff in `src/api/gen` before or immediately after merging.
How can I resolve this? If you propose a fix, please make it concise.Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
| "tailwindcss": "^3.4.4", | ||
| "typescript": "^5.1.6", | ||
| "vite": "^5.1.0", | ||
| "vite": "^6.4.3", |
There was a problem hiding this comment.
Vite major version bump (5 → 6) needs a build/dev smoke-test
Vite 6 introduced breaking changes from v5 (default build target change, SSR module handling updates, env transport path checks). This app's vite.config.ts uses @remix-run/dev's vitePlugin, which gained official Vite 6 support around Remix 2.13. At @remix-run/dev@^2.16.8 compatibility is expected, but a quick remix vite:build and remix vite:dev run is worth doing to confirm nothing regressed in the SSR-rendered auth flow.
Prompt To Fix With AI
This is a comment left during a code review.
Path: apps/auth/remix-ssr/package.json
Line: 38
Comment:
**Vite major version bump (5 → 6) needs a build/dev smoke-test**
Vite 6 introduced breaking changes from v5 (default build target change, SSR module handling updates, env transport path checks). This app's `vite.config.ts` uses `@remix-run/dev`'s `vitePlugin`, which gained official Vite 6 support around Remix 2.13. At `@remix-run/dev@^2.16.8` compatibility is expected, but a quick `remix vite:build` and `remix vite:dev` run is worth doing to confirm nothing regressed in the SSR-rendered auth flow.
How can I resolve this? If you propose a fix, please make it concise.…updates Bumps the npm_and_yarn group with 4 updates in the / directory: [tsup](https://github.com/egoist/tsup), [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite), [@babel/core](https://github.com/babel/babel/tree/HEAD/packages/babel-core) and [@hey-api/openapi-ts](https://github.com/hey-api/hey-api/tree/HEAD/packages/openapi-ts). Bumps the npm_and_yarn group with 1 update in the /apps/auth/remix-ssr directory: [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite). Bumps the npm_and_yarn group with 1 update in the /packages/client/wallets/walletconnect directory: [postcss](https://github.com/postcss/postcss). Bumps the npm_and_yarn group with 1 update in the /packages/wallets directory: [@hey-api/openapi-ts](https://github.com/hey-api/hey-api/tree/HEAD/packages/openapi-ts). Updates `tsup` from 8.1.0 to 8.3.5 - [Release notes](https://github.com/egoist/tsup/releases) - [Commits](egoist/tsup@v8.1.0...v8.3.5) Updates `vite` from 5.4.21 to 6.4.3 - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/v6.4.3/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v6.4.3/packages/vite) Updates `@babel/core` from 7.28.5 to 7.29.6 - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.29.6/packages/babel-core) Updates `@hey-api/openapi-ts` from 0.69.2 to 0.97.3 - [Release notes](https://github.com/hey-api/hey-api/releases) - [Changelog](https://github.com/hey-api/hey-api/blob/main/packages/openapi-ts/CHANGELOG.md) - [Commits](https://github.com/hey-api/hey-api/commits/@hey-api/openapi-ts@0.97.3/packages/openapi-ts) Updates `js-yaml` from 3.14.2 to 3.15.0 - [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md) - [Commits](nodeca/js-yaml@3.14.2...3.15.0) Updates `vite` from 5.4.21 to 6.4.3 - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/v6.4.3/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v6.4.3/packages/vite) Updates `@babel/core` from 7.28.5 to 7.29.6 - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.29.6/packages/babel-core) Updates `@hey-api/openapi-ts` from 0.69.2 to 0.97.3 - [Release notes](https://github.com/hey-api/hey-api/releases) - [Changelog](https://github.com/hey-api/hey-api/blob/main/packages/openapi-ts/CHANGELOG.md) - [Commits](https://github.com/hey-api/hey-api/commits/@hey-api/openapi-ts@0.97.3/packages/openapi-ts) Updates `tsup` from 8.1.0 to 8.3.5 - [Release notes](https://github.com/egoist/tsup/releases) - [Commits](egoist/tsup@v8.1.0...v8.3.5) Updates `js-yaml` from 3.14.2 to 3.15.0 - [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md) - [Commits](nodeca/js-yaml@3.14.2...3.15.0) Updates `@babel/core` from 7.28.5 to 7.29.6 - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.29.6/packages/babel-core) Updates `@hey-api/openapi-ts` from 0.69.2 to 0.97.3 - [Release notes](https://github.com/hey-api/hey-api/releases) - [Changelog](https://github.com/hey-api/hey-api/blob/main/packages/openapi-ts/CHANGELOG.md) - [Commits](https://github.com/hey-api/hey-api/commits/@hey-api/openapi-ts@0.97.3/packages/openapi-ts) Updates `tsup` from 8.1.0 to 8.3.5 - [Release notes](https://github.com/egoist/tsup/releases) - [Commits](egoist/tsup@v8.1.0...v8.3.5) Updates `vite` from 5.4.21 to 6.4.3 - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/v6.4.3/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v6.4.3/packages/vite) Updates `js-yaml` from 3.14.2 to 3.15.0 - [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md) - [Commits](nodeca/js-yaml@3.14.2...3.15.0) Updates `@hey-api/openapi-ts` from 0.69.2 to 0.97.3 - [Release notes](https://github.com/hey-api/hey-api/releases) - [Changelog](https://github.com/hey-api/hey-api/blob/main/packages/openapi-ts/CHANGELOG.md) - [Commits](https://github.com/hey-api/hey-api/commits/@hey-api/openapi-ts@0.97.3/packages/openapi-ts) Updates `@babel/core` from 7.28.5 to 7.29.6 - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.29.6/packages/babel-core) Updates `tsup` from 8.1.0 to 8.3.5 - [Release notes](https://github.com/egoist/tsup/releases) - [Commits](egoist/tsup@v8.1.0...v8.3.5) Updates `vite` from 5.4.21 to 6.4.3 - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/v6.4.3/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v6.4.3/packages/vite) Updates `js-yaml` from 3.14.2 to 3.15.0 - [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md) - [Commits](nodeca/js-yaml@3.14.2...3.15.0) Updates `vite` from 5.4.21 to 6.4.3 - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/v6.4.3/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v6.4.3/packages/vite) Updates `postcss` from 8.4.35 to 8.5.10 - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](postcss/postcss@8.4.35...8.5.10) Updates `postcss` from 8.4.35 to 8.5.10 - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](postcss/postcss@8.4.35...8.5.10) Updates `postcss` from 8.4.35 to 8.5.10 - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](postcss/postcss@8.4.35...8.5.10) Updates `postcss` from 8.4.35 to 8.5.10 - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](postcss/postcss@8.4.35...8.5.10) Updates `@hey-api/openapi-ts` from 0.69.2 to 0.97.3 - [Release notes](https://github.com/hey-api/hey-api/releases) - [Changelog](https://github.com/hey-api/hey-api/blob/main/packages/openapi-ts/CHANGELOG.md) - [Commits](https://github.com/hey-api/hey-api/commits/@hey-api/openapi-ts@0.97.3/packages/openapi-ts) Updates `@hey-api/openapi-ts` from 0.69.2 to 0.97.3 - [Release notes](https://github.com/hey-api/hey-api/releases) - [Changelog](https://github.com/hey-api/hey-api/blob/main/packages/openapi-ts/CHANGELOG.md) - [Commits](https://github.com/hey-api/hey-api/commits/@hey-api/openapi-ts@0.97.3/packages/openapi-ts) Updates `@hey-api/openapi-ts` from 0.69.2 to 0.97.3 - [Release notes](https://github.com/hey-api/hey-api/releases) - [Changelog](https://github.com/hey-api/hey-api/blob/main/packages/openapi-ts/CHANGELOG.md) - [Commits](https://github.com/hey-api/hey-api/commits/@hey-api/openapi-ts@0.97.3/packages/openapi-ts) Updates `@hey-api/openapi-ts` from 0.69.2 to 0.97.3 - [Release notes](https://github.com/hey-api/hey-api/releases) - [Changelog](https://github.com/hey-api/hey-api/blob/main/packages/openapi-ts/CHANGELOG.md) - [Commits](https://github.com/hey-api/hey-api/commits/@hey-api/openapi-ts@0.97.3/packages/openapi-ts) --- updated-dependencies: - dependency-name: "@babel/core" dependency-version: 7.29.6 dependency-type: direct:development - dependency-name: "@hey-api/openapi-ts" dependency-version: 0.97.3 dependency-type: direct:development - dependency-name: "@hey-api/openapi-ts" dependency-version: 0.97.3 dependency-type: direct:development - dependency-name: js-yaml dependency-version: 3.15.0 dependency-type: indirect - dependency-name: postcss dependency-version: 8.5.10 dependency-type: direct:development - dependency-name: tsup dependency-version: 8.3.5 dependency-type: direct:development - dependency-name: vite dependency-version: 6.4.3 dependency-type: direct:development - dependency-name: vite dependency-version: 6.4.3 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com>
3a048c3 to
a7e53f9
Compare
|
Reviews (2): Last reviewed commit: "chore(deps): bump the npm_and_yarn group..." | Re-trigger Greptile |
Bumps the npm_and_yarn group with 4 updates in the / directory: tsup, vite, @babel/core and @hey-api/openapi-ts.
Bumps the npm_and_yarn group with 1 update in the /apps/auth/remix-ssr directory: vite.
Bumps the npm_and_yarn group with 1 update in the /packages/client/wallets/walletconnect directory: postcss.
Bumps the npm_and_yarn group with 1 update in the /packages/wallets directory: @hey-api/openapi-ts.
Updates
tsupfrom 8.1.0 to 8.3.5Release notes
Sourced from tsup's releases.
... (truncated)
Commits
cd03e1echore: release v8.3.5fddd451fix: runexperimentalDtsonly once (#1236)21b1193chore: release v8.3.4580e03dci: fix release workflow01b38f2chore: release v8.3.34f5b71eci: fix release workflowe80dad6chore: release v8.3.2f4af79aci: fix release workflow (#1241)4b72d61chore: release v8.3.141c98fffix: supportNode16andNodeNextmodule resolution inexperimentalDts(...Updates
vitefrom 5.4.21 to 6.4.3Release notes
Sourced from vite's releases.
Changelog
Sourced from vite's changelog.
... (truncated)
Commits
6c2c881release: v6.4.396b0c10fix: backport #22572, reject windows alternate paths (#22576)8fed5cffix(deps): backport #22571, reject UNC paths for launch-editor-middleware (#2...6b3fad0release: v6.4.2ca4da5dfix: avoid path traversal with optimize deps sourcemap handler (#22161)fe28e47fix: apply server.fs check to env transport (#22159) (#22163)5487f4frelease: v6.4.11114b5dfix(dev): trim trailing slash beforeserver.fs.denycheck (#20968) (#20969)f12697crelease: v6.4.0ca6455efeat: allow passing down resolved config to vite's createServer (#20932)Updates
@babel/corefrom 7.28.5 to 7.29.6Release notes
Sourced from @babel/core's releases.
... (truncated)
Commits
04ea6b2v7.29.699f498a[7.x packport]Improve input source map handling (#18001)feba0a3Preserve original identifier names from input sourcemaps (#17992) (#17998)aa8394ev7.29.0ad0d03f[7.x backport] feat: Allow specifying startLine in code frame (#17739)d7f4008v7.28.6e130225Polish(standalone): improve message on invalid preset/plugin (#17606)99dcba5chore: enable some ts-eslint rules (#17592)c92c491Improve Unicode handling in code-frame tokenizer (#17589)d725e39AddBABEL_7_TO_8_DANGEROUSLY_DISABLE_VERSION_CHECK(#17569)Updates
@hey-api/openapi-tsfrom 0.69.2 to 0.97.3Release notes
Sourced from @hey-api/openapi-ts's releases.
... (truncated)
Changelog
Sourced from @hey-api/openapi-ts's changelog.
... (truncated)
Commits
d833f73ci: release4987bc3docs: add changesets30498ceMerge branch 'main' of https://github.com/hey-api/openapi-ts into feat/tansta...9be7e7fchore: one more try4c5916dchore: one more try83b4464chore: one more try597e151Merge branch 'main' into pullfrog/3914-zod-discriminated-union-empty-object462c2e9Merge branch 'main' into feat/transformers-temporal3367e64Merge branch 'main' into fix/sse-iterator-treturn-miswiringe6d4416refactor: valibot pluginMaintainer changes
This version was pushed to npm by GitHub Actions, a new releaser for
@hey-api/openapi-tssince your current version.Updates
js-yamlfrom 3.14.2 to 3.15.0Changelog
Sourced from js-yaml's changelog.
... (truncated)
Commits
c34b6c43.15.0 released21e13d3dist rebuild4165c62Add v3-legacy tag for publishd8ff750Add package lock24f13e7AddedmaxTotalMergeKeys(10000) loader option (v5 backport)Updates
vitefrom 5.4.21 to 6.4.3Release notes
Sourced from vite's releases.
Changelog
Sourced from vite's changelog.
... (truncated)
Commits
6c2c881release: v6.4.396b0c10fix: backport #22572, reject windows alternate paths (#22576)8fed5cffix(deps): backport #22571, reject UNC paths for launch-editor-middleware (#2...6b3fad0release: v6.4.2ca4da5dfix: avoid path traversal with optimize deps sourcemap handler (#22161)fe28e47fix: apply server.fs check to env transport (#22159) (#22163)5487f4frelease: v6.4.11114b5dfix(dev): trim trailing slash beforeserver.fs.denycheck (#20968) (#20969)f12697crelease: v6.4.0ca6455efeat: allow passing down resolved config to vite's createServer (#20932)Updates
@babel/corefrom 7.28.5 to 7.29.6Release notes
Sourced from @babel/core's releases.
... (truncated)
Commits
04ea6b2v7.29.699f498a[7.x packport]Improve input source map handling (#18001)feba0a3Preserve original identifier names from input sourcemaps (#17992) (#17998)aa8394ev7.29.0ad0d03f[7.x backport] feat: Allow specifying startLine in code frame (#17739)d7f4008v7.28.6e130225Polish(standalone): improve message on invalid preset/plugin (#17606)99dcba5chore: enable some ts-eslint rules (#17592)c92c491Improve Unicode handling in code-frame tokenizer (#17589)d725e39AddBABEL_7_TO_8_DANGEROUSLY_DISABLE_VERSION_CHECK(#17569)Updates
@hey-api/openapi-tsfrom 0.69.2 to 0.97.3Release notes
Sourced from @hey-api/openapi-ts's releases.
... (truncated)
Changelog
Sourced from @hey-api/openapi-ts's changelog.