From 845fdcc8c858567fbd62dfc693f4a0ca103f05ac Mon Sep 17 00:00:00 2001 From: Evans Mungai Date: Thu, 16 Apr 2026 15:40:45 +0100 Subject: [PATCH] Move REVIEW.md to .cursor/BUGBOT.md (#2021) * Move REVIEW.md to .cursor/BUGBOT.md Signed-off-by: Evans Mungai * Comment update Signed-off-by: Evans Mungai --------- Signed-off-by: Evans Mungai --- REVIEW.md => .cursor/BUGBOT.md | 2 +- CLAUDE.md | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) rename REVIEW.md => .cursor/BUGBOT.md (98%) diff --git a/REVIEW.md b/.cursor/BUGBOT.md similarity index 98% rename from REVIEW.md rename to .cursor/BUGBOT.md index a1645f15..ddc2a114 100644 --- a/REVIEW.md +++ b/.cursor/BUGBOT.md @@ -19,4 +19,4 @@ - **Dedicated documentation needs** — For large or complex changes, consider whether CLI users or SDK consumers need standalone documentation (migration guides, new feature walkthroughs, updated examples). - **Backwards compatibility** — Spec changes must consider existing specs in the wild. New fields should have sensible zero-value defaults. Removed fields should not cause parse failures. - **Downstream impact on sbctl** — Changes to public Go packages (`pkg/collect`, `pkg/analyze`, API types, etc.) may require follow-up changes in [replicatedhq/sbctl](https://github.com/replicatedhq/sbctl), which imports this project as a dependency. Flag any breaking or behavioral changes that could affect sbctl. -- **Repository docs** — If your changes affect build commands, architecture, project conventions, or review guidelines, update `CLAUDE.md`, `README.md`, and `REVIEW.md` accordingly. +- **Repository docs** — If your changes affect build commands, architecture, project conventions, or review guidelines, update `CLAUDE.md`, `README.md`, and this file accordingly. diff --git a/CLAUDE.md b/CLAUDE.md index 56b9f91e..43d4525f 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -27,4 +27,4 @@ Collectors live in `pkg/collect/`, analyzers in `pkg/analyze/`. When adding eith ## Code Review -See [REVIEW.md](REVIEW.md) for the full review checklist covering basic checks (API breaks, pattern violations, security, test coverage) and advanced checks (cross-feature impact, CLI vs SDK consumers, documentation needs). +See [.cursor/BUGBOT.md](.cursor/BUGBOT.md) for the full review checklist covering basic checks (API breaks, pattern violations, security, test coverage) and advanced checks (cross-feature impact, CLI vs SDK consumers, documentation needs).