|
| 1 | +# PR #38 Status Report |
| 2 | + |
| 3 | +## Current State |
| 4 | +- **Branch**: `storybook-tool-messages` |
| 5 | +- **PR**: https://github.com/coder/cmux/pull/38 |
| 6 | +- **Status**: BLOCKED - Chromatic build failing |
| 7 | + |
| 8 | +## What Works ✅ |
| 9 | +- ✅ Storybook builds locally |
| 10 | +- ✅ All 13 tool message stories render correctly |
| 11 | +- ✅ FileReadToolCall custom component working |
| 12 | +- ✅ TypeScript/ESLint passing |
| 13 | +- ✅ No code quality issues |
| 14 | + |
| 15 | +## What's Blocked ❌ |
| 16 | +- ❌ Chromatic visual regression testing |
| 17 | +- Reason: `@emotion/styled` module resolution failure in Chromatic environment |
| 18 | +- Error: `TypeError: a is not a function` (styled is not a function) |
| 19 | + |
| 20 | +## Investigation Summary |
| 21 | +Spent significant time investigating the Chromatic build failure: |
| 22 | +- Converted 2 components to CSS modules (BashToolCall, FileReadToolCall) |
| 23 | +- Tried multiple Vite/Babel configurations |
| 24 | +- Added dedupe config for emotion packages |
| 25 | +- None resolved the Chromatic-specific issue |
| 26 | + |
| 27 | +## Root Cause |
| 28 | +The `@emotion/styled` default export becomes undefined in Chromatic's production build environment. This affects 25+ components across the codebase, not just the new Storybook stories. |
| 29 | + |
| 30 | +## Decision Needed |
| 31 | +See `CHROMATIC_ISSUE_SUMMARY.md` for detailed options analysis. |
| 32 | + |
| 33 | +**Quick options:** |
| 34 | +1. **Merge as-is** - Storybook works locally, accept Chromatic failure for now |
| 35 | +2. **Disable Chromatic** - Remove workflow temporarily, re-enable after fix |
| 36 | +3. **Continue conversion** - Convert all 25+ components to CSS modules (10-20 hours) |
| 37 | +4. **Open support ticket** - Contact Chromatic team for help |
| 38 | + |
| 39 | +## Recommendation |
| 40 | +**Option 1 or 2** - The Storybook feature is complete and working. The Chromatic issue is environmental and affects the entire codebase, not specific to this PR. It should be addressed separately. |
| 41 | + |
| 42 | +The stories provide value for local development even without Chromatic visual regression testing. |
0 commit comments