Skip to content

docs: mention changesets in rule validation - #571

Merged
rayhanadev merged 2 commits into
mainfrom
ray/9c876497
May 29, 2026
Merged

docs: mention changesets in rule validation#571
rayhanadev merged 2 commits into
mainfrom
ray/9c876497

Conversation

@rayhanadev

@rayhanadev rayhanadev commented May 29, 2026

Copy link
Copy Markdown
Member

Why

Make the rule validation skill remind agents to create a changeset when a rule change affects published packages.

This keeps validation, PR prep, and release notes aligned so user-facing rule work does not skip the versioning step.

Before:

rule-validate verifies noise, correctness, PR copy, and review feedback.

After:

rule-validate verifies noise, correctness, changesets, PR copy, and review feedback.

What changed

  • Added changeset generation to the rule-validate skill description and pipeline summary.
  • Documents when to run nr changeset, which packages to select, and when patch is appropriate.
  • Adds changeset path or skip reason to the validation output template.

Test plan

  • Not run; docs-only skill update.

Eval results

RDE was not run; this only updates agent guidance and does not change a React Doctor rule implementation.

Changeset

Skipped; docs-only skill update with no published package behavior change.

Made with Cursor


Note

Low Risk
Docs-only skill guidance plus a cosmetic progress-message string change; no rule logic, auth, or release behavior changes.

Overview
Extends the rule-validate agent skill so validation explicitly covers changesets before PR copy: when to run nr changeset, package selection, default patch bumps for user-facing rule work, skip rules for docs-only work, and a changeset path or skip reason in the validation output template.

Also updates the lint scan progress line in run-inspect from Scanning (n/m)... to Scanning files (n/m)... for clearer CLI feedback during file scans.

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

rayhanadev and others added 2 commits May 27, 2026 16:29
Copilot AI review requested due to automatic review settings May 29, 2026 03:12

@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

@rayhanadev
rayhanadev merged commit 1c87c65 into main May 29, 2026
15 of 16 checks passed
@rayhanadev
rayhanadev deleted the ray/9c876497 branch May 29, 2026 03:14

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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