Skip to content

fix(storybook): remove STORYBOOK_PROJECT_ROOT when running automigrate to prevent hanging#33567

Merged
jaysoo merged 1 commit intomasterfrom
NXC-3504
Nov 21, 2025
Merged

fix(storybook): remove STORYBOOK_PROJECT_ROOT when running automigrate to prevent hanging#33567
jaysoo merged 1 commit intomasterfrom
NXC-3504

Conversation

@jaysoo
Copy link
Copy Markdown
Member

@jaysoo jaysoo commented Nov 21, 2025

This PR fixes an issue where migrations can hang due to out invocation of storybook automigrate. We are passing both --config-dir and STORYBOOK_PROJECT_ROOT, the latter causes hanging with Storybook v9.

https://www.loom.com/share/39bbb350595c4a13aef86ec29f4b748f

Current Behavior

Hangs

Expected Behavior

Does not hang

Related Issue(s)

Fixes #32492

@jaysoo jaysoo requested a review from a team as a code owner November 21, 2025 15:04
@jaysoo jaysoo requested a review from Coly010 November 21, 2025 15:04
@vercel
Copy link
Copy Markdown

vercel Bot commented Nov 21, 2025

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

Project Deployment Preview Updated (UTC)
nx-dev Ready Ready Preview Nov 21, 2025 3:08pm

@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 21, 2025

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit 651f988
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/69207f79459b420007be6f2b
😎 Deploy Preview https://deploy-preview-33567--nx-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Nov 21, 2025

View your CI Pipeline Execution ↗ for commit 651f988

Command Status Duration Result
nx affected --targets=lint,test,test-kt,build,e... ✅ Succeeded 7m 53s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 1m 48s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 11s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-11-21 15:16:18 UTC

@jaysoo jaysoo merged commit 57f28ba into master Nov 21, 2025
21 checks passed
@jaysoo jaysoo deleted the NXC-3504 branch November 21, 2025 16:34
jaysoo added a commit that referenced this pull request Nov 21, 2025
…e to prevent hanging (#33567)

This PR fixes an issue where migrations can hang due to out invocation
of `storybook automigrate`. We are passing both `--config-dir` and
`STORYBOOK_PROJECT_ROOT`, the latter causes hanging with Storybook v9.

https://www.loom.com/share/39bbb350595c4a13aef86ec29f4b748f

## Current Behavior
Hangs

## Expected Behavior
Does not hang

## Related Issue(s)
<!-- Please link the issue being fixed so it gets closed when this is
merged. -->

Fixes #32492
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Nov 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

nx migrate latest with storybook will hang

2 participants