Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 30, 2026

Reduces snapshot consumption by disabling Chromatic for 78+ stories that don't provide visual regression value: interactive flows requiring user action, icon galleries, demo composites, redundant variations, and state duplicates.

Changes

Disabled snapshots for stories requiring interaction or showing redundant variants:

  • Interactive stories (23) - Modals, drawers, popovers, tooltips requiring clicks/hovers
  • Icon galleries (2) - Collections rendering 100+ icons
  • Form interactions (2) - State-based stories with interactive toggle buttons
  • Guided tours (3) - Multi-step flows with loading states
  • Demo composites (5) - Multiple variants already tested individually
  • Virtualized lists (10) - Collapsible panels and table variations showing minor differences
  • Redundant variation stories (27) - Button, Tag, ButtonIcon showing prop combinations already covered by template stories
  • State duplicates (13) - Status icon-only variants and RichRadioButton checked state duplicates
  • Range filters - Already disabled (verified no regression)

Implementation adds story-level parameters:

export const Interactive = () => { /* ... */ };
Interactive.parameters = {
  chromatic: { disableSnapshot: true },
};

Stories remain available for development and documentation. Visual regression coverage preserved for static, meaningful variations.

Impact

  • 78+ stories disabled (43% snapshot reduction)
  • Focused on eliminating redundant prop combinations and state variations
  • Preserved unique visual variations important for design system regression testing

Files Modified

19 story files in packages/design-system, packages/components, and packages/icons.

Original prompt

This section details on the original issue you should resolve

<issue_title>Chromatic: disable snapshots in stories that doesn't make sens</issue_title>
<issue_description>Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

We have lots of stories so we consume lots of snapshots

Describe the solution you'd like
A clear and concise description of what you want to happen.

Let's decrease as much stories as possible. Only pick story that has real differences inside each stories file.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.
</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@changeset-bot
Copy link

changeset-bot bot commented Jan 30, 2026

⚠️ No Changeset found

Latest commit: a94fc99

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copilot AI and others added 4 commits January 30, 2026 16:20
Co-authored-by: jmfrancois <19857479+jmfrancois@users.noreply.github.com>
Co-authored-by: jmfrancois <19857479+jmfrancois@users.noreply.github.com>
Co-authored-by: jmfrancois <19857479+jmfrancois@users.noreply.github.com>
Co-authored-by: jmfrancois <19857479+jmfrancois@users.noreply.github.com>
Copilot AI changed the title [WIP] Disable unnecessary snapshots in stories for better performance Disable Chromatic snapshots for non-visual-regression stories Jan 30, 2026
Copilot AI requested a review from jmfrancois January 30, 2026 16:29
Copilot AI temporarily deployed to pull_request_unsafe January 30, 2026 16:34 Inactive
Copilot AI temporarily deployed to pull_request_unsafe January 30, 2026 16:34 Inactive
Copilot AI temporarily deployed to pull_request_unsafe January 30, 2026 16:34 Inactive
@jmfrancois
Copy link
Contributor

Great works. Some of the stories has lots of stories can you elimitates the one which are really similar ?

@jmfrancois jmfrancois changed the title Disable Chromatic snapshots for non-visual-regression stories test: disable some Chromatic snapshots Jan 30, 2026
Copy link
Contributor

@jmfrancois jmfrancois left a comment

Choose a reason for hiding this comment

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

I think you can exclude way much more.
please try to find similar stories ti disable them
I think we can remove 60% of them

@github-actions
Copy link
Contributor

Storybook for this PR deployed on this github page

…tonIcon, and VirtualizedList

Co-authored-by: jmfrancois <19857479+jmfrancois@users.noreply.github.com>
@github-actions
Copy link
Contributor

github-actions bot commented Jan 30, 2026

Title Lines Statements Branches Functions
assets-api Coverage: 28%
28.4% (25/88) 30.76% (16/52) 21.42% (3/14)
cmf Coverage: 89%
89.36% (1243/1391) 80.93% (607/750) 89.51% (350/391)
cmf-cqrs Coverage: 87%
87.43% (160/183) 70.23% (59/84) 84.21% (48/57)
cmf-router Coverage: 70%
69.23% (135/195) 55.71% (78/140) 56.81% (25/44)
components Coverage: 90%
90.7% (5565/6135) 81.59% (3196/3917) 88.14% (1390/1577)
containers Coverage: 84%
83.6% (1392/1665) 74.3% (694/934) 75% (327/436)
dataviz Coverage: 85%
85.44% (323/378) 66.66% (160/240) 75.79% (119/157)
design-system Coverage: 67%
66.49% (1016/1528) 50.84% (543/1068) 53.94% (219/406)
faceted-search Coverage: 85%
85.08% (639/751) 78.63% (287/365) 81.88% (226/276)
flow-designer Coverage: 70%
70.07% (651/929) 66.72% (355/532) 70.92% (200/282)
forms Coverage: 86%
86.27% (1640/1901) 75.89% (929/1224) 85.02% (460/541)
http Coverage: 100%
100% (85/85) 98.07% (51/52) 100% (34/34)
sagas Coverage: 92%
92.3% (24/26) 66.66% (4/6) 50% (2/4)
stepper Coverage: 80%
81.52% (150/184) 59.34% (54/91) 80.85% (38/47)
utils Coverage: 100%
100% (73/73) 90.9% (10/11) 100% (24/24)

…d states

Co-authored-by: jmfrancois <19857479+jmfrancois@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 30, 2026

Great works. Some of the stories has lots of stories can you elimitates the one which are really similar ?

I've identified and disabled similar stories within story files. Found 40+ redundant variation stories across Button, Tag, ButtonIcon, VirtualizedList, Status, and RichRadioButton that show prop combinations or states already covered by other stories. Commit a94fc99.

I think you can exclude way much more.
please try to find similar stories ti disable them
I think we can remove 60% of them

Increased disabled stories from 38 to 78+ (doubled). This includes:

  • 27 redundant variation stories (Button/ButtonIcon/Tag showing size+icon+state combinations)
  • 9 VirtualizedList table variations
  • 6 Status icon-only variants
  • 7 RichRadioButton checked state duplicates

This achieves approximately 43% snapshot reduction while preserving unique visual variations. Commits 552c4cd and a94fc99.

Copilot AI changed the title test: disable some Chromatic snapshots test: disable Chromatic snapshots for redundant and non-visual stories Jan 30, 2026
Copilot AI requested a review from jmfrancois January 30, 2026 16:51
@jmfrancois jmfrancois marked this pull request as ready for review January 30, 2026 17:08
Copilot AI deployed to pull_request_unsafe January 30, 2026 17:08 Active
Copilot AI temporarily deployed to pull_request_unsafe January 30, 2026 17:08 Inactive
Copilot AI temporarily deployed to pull_request_unsafe January 30, 2026 17:08 Inactive
@jmfrancois jmfrancois enabled auto-merge (squash) January 30, 2026 17:09
@jmfrancois jmfrancois disabled auto-merge January 30, 2026 17:09
@jmfrancois jmfrancois merged commit 0fc311e into master Jan 30, 2026
8 checks passed
@jmfrancois jmfrancois deleted the copilot/disable-snapshots-in-stories branch January 30, 2026 17:09
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.

Chromatic: disable snapshots in stories that doesn't make sens

2 participants