Skip to content

Conversation

@snomiao
Copy link
Member

@snomiao snomiao commented Aug 31, 2025

Summary

  • Fix Chromatic deployment failure in GitHub Actions by updating build-storybook script
  • Replace nx build-storybook with direct storybook build command to properly forward --output-dir argument

Problem

The Chromatic deployment was failing because the Nx wrapper for build-storybook wasn't forwarding the --output-dir argument that Chromatic passes. This caused static files (HTML, CSS, fonts) to be empty (0 bytes), making Chromatic reject the build as invalid.

Solution

Changed package.json script from nx build-storybook to storybook build to allow proper argument forwarding.

Test plan

  • Verified local build works with custom output directory
  • Tested that static files are properly generated
  • Verify Chromatic deployment passes in CI

🤖 Generated with Claude Code

┆Issue is synchronized with this Notion page by Unito

christian-byrne and others added 2 commits August 30, 2025 15:33
Replace nx build-storybook with direct storybook build command to properly
forward --output-dir argument from Chromatic deployment.

Fixes invalid Storybook build issue where static files were empty due to
Nx not passing through command-line arguments to the underlying Storybook
build command.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions
Copy link

github-actions bot commented Aug 31, 2025

🎭 Playwright Test Results

All tests passed across all browsers!

⏰ Completed at: 08/31/2025, 04:31:36 AM UTC

📊 Test Reports by Browser


🎉 Your tests are passing across all browsers!

@github-actions
Copy link

github-actions bot commented Aug 31, 2025

🎨 Storybook Build Status

Build completed successfully!

⏰ Completed at: 08/31/2025, 04:09:09 AM UTC

🔗 Links


🎉 Your Storybook is ready for review!

@snomiao snomiao force-pushed the version-bump-1.26.8--sno-debug branch 2 times, most recently from 92a810b to a988461 Compare August 31, 2025 03:47
@snomiao snomiao marked this pull request as ready for review August 31, 2025 03:50
@snomiao snomiao requested a review from a team as a code owner August 31, 2025 03:50
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 31, 2025
@snomiao snomiao force-pushed the version-bump-1.26.8--sno-debug branch from 5bce30e to 06c1014 Compare August 31, 2025 03:54
@christian-byrne christian-byrne merged commit 2358a97 into main Aug 31, 2025
10 of 11 checks passed
@christian-byrne christian-byrne deleted the version-bump-1.26.8--sno-debug branch August 31, 2025 05:01
snomiao added a commit that referenced this pull request Aug 31, 2025
* [release] Increment version to 1.26.8

* [fix] Fix Chromatic build failure by updating build-storybook script

Replace nx build-storybook with direct storybook build command to properly
forward --output-dir argument from Chromatic deployment.

Fixes invalid Storybook build issue where static files were empty due to
Nx not passing through command-line arguments to the underlying Storybook
build command.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Change version to 0.7 for testing purposes

* chore(package.json): update version from 0.7 to 1.26.7 to reflect new release

---------

Co-authored-by: christian-byrne <72887196+christian-byrne@users.noreply.github.com>
Co-authored-by: Claude <noreply@anthropic.com>
@benceruleanlu benceruleanlu mentioned this pull request Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants