From da8f029916817701b59a415b5515f8bbb8cf6f0b Mon Sep 17 00:00:00 2001 From: rkishner Date: Thu, 16 Jul 2026 09:16:27 +0300 Subject: [PATCH] utils: move code rabbit PR Template Validation to custom_checks Co-authored-by: Cursor Signed-off-by: rkishner --- .coderabbit.yaml | 30 +++++++++++++++++------------- 1 file changed, 17 insertions(+), 13 deletions(-) diff --git a/.coderabbit.yaml b/.coderabbit.yaml index 96f6cfaab4..52cedc9d9f 100644 --- a/.coderabbit.yaml +++ b/.coderabbit.yaml @@ -71,18 +71,6 @@ reviews: - path: "**" instructions: | - ## PR Template Validation - Check the PR description for required sections from `.github/pull_request_template.md`. - Required sections (must be present, even if empty): - - `##### What this PR does / why we need it:` — MUST be present AND have meaningful content. - Flag as HIGH if the section is missing, empty, whitespace-only, contains only HTML comments, - or contains only placeholder tokens such as `TBD`, `TBA`, `N/A`, `-`, `—`, `none`, or `.`. - - `##### Which issue(s) this PR fixes:` — must be present (may be empty) - - `##### Special notes for reviewer:` — must be present (may be empty) - - `##### jira-ticket:` — must be present (may be empty) - If any required section is absent, or `What this PR does / why we need it:` has no content, - flag it as HIGH severity and ask the author to restore the missing template section(s). - ## Approval Policy You may approve the PR when ALL of the following are true: - All your review comments have been addressed with either: @@ -136,10 +124,26 @@ reviews: mode: error requirements: "Must be under 120 chars and clearly describe the change." description: - mode: error + mode: "off" docstrings: mode: "off" custom_checks: + - name: "PR Template Sections" + mode: error + instructions: | + Check the current PR description (the actual, live PR body — not a cached or + per-file view of it) for required sections from `.github/pull_request_template.md`. + Required sections (must be present, even if empty): + - `##### What this PR does / why we need it:` — MUST be present AND have meaningful + content. Fail as HIGH if the section is missing, empty, whitespace-only, contains + only HTML comments, or contains only placeholder tokens such as `TBD`, `TBA`, + `N/A`, `-`, `—`, `none`, or `.`. + - `##### Which issue(s) this PR fixes:` — must be present (may be empty) + - `##### Special notes for reviewer:` — must be present (may be empty) + - `##### jira-ticket:` — must be present (may be empty) + If any required section is absent, or `What this PR does / why we need it:` has no content, + flag it as HIGH severity and ask the author to restore the missing template section(s). + - name: "STP link required" mode: error instructions: |