From 51f005ad4da2c866412f7dbb565f2bda60933d3b Mon Sep 17 00:00:00 2001 From: Brandonma21 Date: Mon, 14 Sep 2026 11:13:57 +0200 Subject: [PATCH 1/3] Expand CodeRabbit config for full auto-review and finishing touches (LS-4124) Auto-review coverage - Widen base_branches to include feature/* and fix/* so stacked PRs get reviewed, not just PRs into main/develop - Enable auto_incremental_review to re-review on each push Review quality - Add path_filters to exclude node_modules, vendor, and CSS map files - Enable assess_linked_issues, related_issues, related_prs - Enable tools: eslint, markdownlint, gitleaks, trufflehog - Point knowledge_base.code_guidelines at AGENTS.md - Set profile to assertive and enable request_changes_workflow Finishing touches - Enable finishing_touches.autofix, docstrings, unit_tests (invoked manually per PR, no automatic usage cost) Context - Restores/extends prior working auto-review setup; reviews had stopped firing, likely a GitHub App/dashboard access issue outside this config, to be confirmed by an org admin --- .coderabbit.yml | 38 ++++++++++++++++++++++++++++++++++++++ 1 file changed, 38 insertions(+) diff --git a/.coderabbit.yml b/.coderabbit.yml index d8f345f..3c7040d 100644 --- a/.coderabbit.yml +++ b/.coderabbit.yml @@ -2,12 +2,42 @@ language: en-US reviews: enabled: true + profile: assertive auto_review: enabled: true drafts: false + auto_incremental_review: true base_branches: - main - develop + - feature/* + - fix/* + request_changes_workflow: true + high_level_summary: true + collapse_walkthrough: true + assess_linked_issues: true + related_issues: true + related_prs: true + path_filters: + - "!node_modules/**" + - "!vendor/**" + - "!assets/css/*.map" + finishing_touches: + autofix: + enabled: true + docstrings: + enabled: true + unit_tests: + enabled: true + tools: + eslint: + enabled: true + markdownlint: + enabled: true + gitleaks: + enabled: true + trufflehog: + enabled: true path_instructions: - path: "theme.json" instructions: | @@ -111,5 +141,13 @@ reviews: - Check that placeholder tokens have been replaced (if not a starter) - Check for consistent text domain usage +knowledge_base: + code_guidelines: + enabled: true + filePatterns: + - "AGENTS.md" + learnings: + scope: auto + chat: enabled: true From 6cf832c946564295e4742da8c7582885d5fa29ae Mon Sep 17 00:00:00 2001 From: Brandonma21 Date: Mon, 14 Sep 2026 11:17:36 +0200 Subject: [PATCH 2/3] Add changelog entry for CodeRabbit config expansion (LS-4124) --- CHANGELOG.md | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 93a0903..0c569bb 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 --- +## [Unreleased] — Expand CodeRabbit config for full auto-review and finishing touches (LS-4124) + +### Changed + +- `.coderabbit.yml`: widened `auto_review.base_branches` to include `feature/*` and `fix/*` so stacked PRs (based on a prior feature branch rather than `main`/`develop`) trigger an automatic review, not just PRs targeting `main`/`develop`. Enabled `auto_incremental_review`, `assess_linked_issues`, `related_issues`, `related_prs`, `path_filters`, `finishing_touches` (autofix/docstrings/unit_tests), `tools` (eslint/markdownlint/gitleaks/trufflehog), and `knowledge_base.code_guidelines` (pointed at `AGENTS.md`). Set `profile: assertive` and enabled `request_changes_workflow`. ([#57](https://github.com/lightspeedwp/ls-theme/pull/57)) + +--- + ## [Unreleased] — Build Search template (LS-2594) ### Added From 2bffd450af5965697c973a758ad4daf031ec4da3 Mon Sep 17 00:00:00 2001 From: "coderabbitai[bot]" <136622811+coderabbitai[bot]@users.noreply.github.com> Date: Mon, 14 Sep 2026 10:01:53 +0000 Subject: [PATCH 3/3] Fix CodeRabbit base branch regexes for stacked PRs --- .coderabbit.yml | 4 ++-- CHANGELOG.md | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.coderabbit.yml b/.coderabbit.yml index 3c7040d..6bcdebe 100644 --- a/.coderabbit.yml +++ b/.coderabbit.yml @@ -10,8 +10,8 @@ reviews: base_branches: - main - develop - - feature/* - - fix/* + - feature/.* + - fix/.* request_changes_workflow: true high_level_summary: true collapse_walkthrough: true diff --git a/CHANGELOG.md b/CHANGELOG.md index 0c569bb..823c3c6 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -11,7 +11,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 ### Changed -- `.coderabbit.yml`: widened `auto_review.base_branches` to include `feature/*` and `fix/*` so stacked PRs (based on a prior feature branch rather than `main`/`develop`) trigger an automatic review, not just PRs targeting `main`/`develop`. Enabled `auto_incremental_review`, `assess_linked_issues`, `related_issues`, `related_prs`, `path_filters`, `finishing_touches` (autofix/docstrings/unit_tests), `tools` (eslint/markdownlint/gitleaks/trufflehog), and `knowledge_base.code_guidelines` (pointed at `AGENTS.md`). Set `profile: assertive` and enabled `request_changes_workflow`. ([#57](https://github.com/lightspeedwp/ls-theme/pull/57)) +- `.coderabbit.yml`: corrected `auto_review.base_branches` to use the `feature/.*` and `fix/.*` regular expressions, so stacked PRs (based on a prior feature or fix branch rather than `main`/`develop`) trigger an automatic review. Enabled `auto_incremental_review`, `assess_linked_issues`, `related_issues`, `related_prs`, `path_filters`, `finishing_touches` (autofix/docstrings/unit_tests), `tools` (eslint/markdownlint/gitleaks/trufflehog), and `knowledge_base.code_guidelines` (pointed at `AGENTS.md`). Set `profile: assertive` and enabled `request_changes_workflow`. ([#57](https://github.com/lightspeedwp/ls-theme/pull/57)) ---