chore(deps): bump @storybook-vue/nuxt to latest#2664
chore(deps): bump @storybook-vue/nuxt to latest#2664ghostdevv merged 3 commits intonpmx-dev:mainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughThe Changes
Suggested reviewers
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
All alerts resolved. Learn more about Socket for GitHub. This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
e18e dependency analysisNo dependency warnings found. |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
pnpm-workspace.yaml (1)
20-27: Consider moving to a published package version when available.This catalogue entry currently uses a preview release from
pkg.pr.new, which is designed for development and testing rather than production use. Whilstpkg.pr.newpackages do not expire and are persistent, they lack the stability guarantees of officially published releases. Since@storybook-vue/nuxtis a development dependency, the current approach is functional; however, plan to transition to a published version once it becomes available to align with standard dependency practices.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@pnpm-workspace.yaml` around lines 20 - 27, The storybook catalogue entry uses a preview registry URL for `@storybook-vue/nuxt`; replace the pkg.pr.new reference with the official published version specifier (e.g., a semver like '^x.y.z' or the matching Storybook release) in the storybook dependency block where '@storybook-vue/nuxt' is declared, update the lockfile by running the package manager install, and ensure the other storybook addons remain version-compatible.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@pnpm-workspace.yaml`:
- Around line 20-27: The storybook catalogue entry uses a preview registry URL
for `@storybook-vue/nuxt`; replace the pkg.pr.new reference with the official
published version specifier (e.g., a semver like '^x.y.z' or the matching
Storybook release) in the storybook dependency block where '@storybook-vue/nuxt'
is declared, update the lockfile by running the package manager install, and
ensure the other storybook addons remain version-compatible.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 3b58cd7a-fd30-4aa2-9daa-9d1830853123
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (1)
pnpm-workspace.yaml
|
|
||
| storybook: | ||
| '@storybook-vue/nuxt': '9.0.1' | ||
| '@storybook-vue/nuxt': 'https://pkg.pr.new/@storybook-vue/nuxt@1021' |
There was a problem hiding this comment.
my pkg.pr.new game is weak - how do I see where this is going 😅
There was a problem hiding this comment.
The number at the end is the PR number on the repo. So in this case nuxt-modules/storybook#1021 (comment) .
I picked the latest PR merged to main in that package’s history at the time. https://github.com/nuxt-modules/storybook/commits/main/packages/storybook-addon
Looks like we could probably go with 1051 now if preferred, but either have storybook 10 versions which is the main reason for bumping at this point.
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
🔗 Linked issue
🧭 Context
The latest version of
@storybook-vue/nuxtpublished to npm is not fully compatible with Storybook 10, Nuxt 4, or Vite 8.This PR updates the version to the latest merged PR for that package using pkg.pr.new.
📚 Description
Grabbed pkg.pr.new version from latest merged PR for
@storybook-vue/nuxtand removed the override set to force Storybook to version 10.