Skip to content

feat(storybook): generate ai instructions for converting from CJS to ESM after migration#33395

Merged
Coly010 merged 2 commits intomasterfrom
storybook/output-ai-migration
Nov 6, 2025
Merged

feat(storybook): generate ai instructions for converting from CJS to ESM after migration#33395
Coly010 merged 2 commits intomasterfrom
storybook/output-ai-migration

Conversation

@Coly010
Copy link
Copy Markdown
Contributor

@Coly010 Coly010 commented Nov 6, 2025

Current Behavior

Storybook 10 Changelog states that CJS is no longer supported.
The migration does not do this automatically.
https://github.com/storybookjs/storybook/blob/next/MIGRATION.md#the-storybookmain-file-and-other-presets-must-be-valid-esm

Expected Behavior

Generate instructions for an AI Agent to find and convert CJS Storybook Config files to ESM.

Related Issue(s)

Fixes NXC-3409
Fixes NXC-3336

@Coly010 Coly010 requested a review from a team as a code owner November 6, 2025 10:50
@Coly010 Coly010 requested a review from jaysoo November 6, 2025 10:50
@Coly010 Coly010 self-assigned this Nov 6, 2025
@vercel
Copy link
Copy Markdown

vercel Bot commented Nov 6, 2025

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

Project Deployment Preview Updated (UTC)
nx-dev Ready Ready Preview Nov 6, 2025 2:20pm

@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 6, 2025

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit d81e412
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/690cada34b77fa00083d1cce
😎 Deploy Preview https://deploy-preview-33395--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 6, 2025

View your CI Pipeline Execution ↗ for commit d81e412

Command Status Duration Result
nx affected --targets=lint,test,test-kt,build,e... ✅ Succeeded 7m 37s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 1m 37s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 10s 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-06 14:28:10 UTC

@Coly010 Coly010 enabled auto-merge (squash) November 6, 2025 14:19
@Coly010 Coly010 merged commit f2072f0 into master Nov 6, 2025
20 checks passed
@Coly010 Coly010 deleted the storybook/output-ai-migration branch November 6, 2025 14:28
@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 12, 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.

2 participants