Skip to content

chore: add guardrails for PR's#1902

Merged
nhironaka merged 2 commits into
mainfrom
nhironaka/pr_guardrails
May 29, 2026
Merged

chore: add guardrails for PR's#1902
nhironaka merged 2 commits into
mainfrom
nhironaka/pr_guardrails

Conversation

@nhironaka

@nhironaka nhironaka commented May 29, 2026

Copy link
Copy Markdown
Contributor

Summary

Updating ./cursor/BUGBOT.md

Screenshots (if appropriate):

Testing approaches


Note

Low Risk
Documentation-only change to .cursor/BUGBOT.md; no runtime, API, or dependency impact.

Overview
Extends .cursor/BUGBOT.md with a PR readiness section so automated review checks the PR before diving into code.

Reviewers (and Bugbot) are told to flag oversized or mixed-concern diffs (e.g. ~400-line ideal, one component per PR, isolated token cascades), thin descriptions (what / why / how verified, including AI boilerplate), and changeset level vs. actual API impact, as top-level comments—not inline nits.

A new automated checks table maps CI signals (PR comment bot, story coverage, export/publint/attw, Storybook axe, Chromatic, package size) and says to flag when failures aren’t explained in the description. A horizontal rule separates this block from the existing “What to focus on” review priorities.

Reviewed by Cursor Bugbot for commit 90831b4. Bugbot is set up for automated code reviews on this repo. Configure here.

@changeset-bot

changeset-bot Bot commented May 29, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 90831b4

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

@nhironaka nhironaka force-pushed the nhironaka/pr_guardrails branch from d459e34 to 90831b4 Compare May 29, 2026 13:55
@nhironaka nhironaka added the skip changeset To indicate a pull request does not need a changeset label May 29, 2026
@nhironaka nhironaka marked this pull request as ready for review May 29, 2026 15:02
@nhironaka nhironaka requested a review from a team as a code owner May 29, 2026 15:02

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@nhironaka nhironaka merged commit 38730e3 into main May 29, 2026
22 checks passed
@nhironaka nhironaka deleted the nhironaka/pr_guardrails branch May 29, 2026 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip changeset To indicate a pull request does not need a changeset

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants