Skip to content

chore: add bug fix workflow rule#3788

Open
twjeffery wants to merge 1 commit intodevfrom
tom/bug-fix-workflow-rule
Open

chore: add bug fix workflow rule#3788
twjeffery wants to merge 1 commit intodevfrom
tom/bug-fix-workflow-rule

Conversation

@twjeffery
Copy link
Copy Markdown
Collaborator

@twjeffery twjeffery commented Apr 9, 2026

Summary

Adds .claude/rules/bug-fix-workflow.md describing the workflow for picking up small bug fix issues from the team backlog, and adds a "Token-First Changes" section to .claude/rules/styling.md.

Bug fix workflow rule (new file)

  • Reading the issue body as the source of truth for scope
  • Targeting V2 code paths
  • Fixing at the source rather than working around bugs in components that use the broken thing
  • Asking before guessing on ambiguity
  • Branch naming, commit format, and PR template requirements
  • Adding playground bug pages
  • Opening PRs as drafts for review

Token-first styling rule (addition to styling.md)

  • When changing component styling, update token values rather than removing CSS that references tokens. Removing the CSS breaks the customization contract for consumers who override that token.
  • Component CSS must reference component tokens, not global tokens directly. If no component token exists, create one that references the global.

@twjeffery twjeffery marked this pull request as ready for review April 9, 2026 06:01
@twjeffery twjeffery requested a review from Spark450 April 9, 2026 06:01
@twjeffery twjeffery force-pushed the tom/bug-fix-workflow-rule branch 2 times, most recently from 183d31d to fb4328f Compare April 9, 2026 17:23
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 9, 2026

Deploy Preview for benji-docs-previews ready!

Name Link
🔨 Latest commit 27b5e5e
🔍 Latest deploy log https://app.netlify.com/projects/benji-docs-previews/deploys/69d7f0fa49927f0008490228
😎 Deploy Preview https://deploy-preview-3788--benji-docs-previews.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.

@twjeffery twjeffery requested a review from bdfranck April 9, 2026 17:28
@twjeffery twjeffery force-pushed the tom/bug-fix-workflow-rule branch from fb4328f to 27b5e5e Compare April 9, 2026 18:33
Copy link
Copy Markdown
Collaborator

@Spark450 Spark450 left a comment

Choose a reason for hiding this comment

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

This all makes sense, and may help with some of the issues we saw from those 1st generated issues in backlog grooming today.

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.

2 participants