Skip to content

fix(storybook): fix missing dependency#304

Merged
gregberge merged 1 commit into
mainfrom
fix-missing-dependency-storybook
May 19, 2026
Merged

fix(storybook): fix missing dependency#304
gregberge merged 1 commit into
mainfrom
fix-missing-dependency-storybook

Conversation

@gregberge

Copy link
Copy Markdown
Member

Fix #303

@gregberge gregberge requested review from a team, Copilot and jsfez and removed request for a team May 19, 2026 06:19
@vercel

vercel Bot commented May 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
argos-js-sdk-reference Ready Ready Preview, Comment May 19, 2026 6:20am

Request Review

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes a runtime ERR_MODULE_NOT_FOUND for @argos-ci/core when consuming @argos-ci/storybook's vitest-plugin under strict-isolating package managers, by promoting @argos-ci/core from devDependencies to dependencies. Also bumps the pnpm packageManager version.

Changes:

  • Move @argos-ci/core to dependencies in packages/storybook/package.json.
  • Update pnpm-lock.yaml to reflect the dependency move.
  • Bump pnpm to 11.1.3 in root package.json.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
packages/storybook/package.json Promote @argos-ci/core from devDependencies to dependencies
pnpm-lock.yaml Lockfile updated to reflect the dependency move
package.json Bump pnpm packageManager version
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@gregberge gregberge merged commit a78556e into main May 19, 2026
94 of 95 checks passed
@gregberge gregberge deleted the fix-missing-dependency-storybook branch May 19, 2026 06:50
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.

Bug: @argos-ci/storybook imports @argos-ci/core at runtime but only declares it in devDependencies

2 participants