Repository navigation
feat: github_review_threads strips CodeRabbit boilerplate before the cap; body_max_chars param (github-plugin#63) - #65
Merged
mabry1985 merged 1 commit intoNov 7, 2026
Conversation
…cap; body_max_chars param (github-plugin#63)
There was a problem hiding this comment.
QA panel review — PASS
code-review-structural · head aa95a02c2f1f · formal
No findings were raised, so there was nothing to verify. The panel found no defects across all angles; the verifier confirmed nothing-to-verify. No coverage gaps, no disagreements, no prior items to disposition.
No findings — the review came back clean.
findings JSON (machine-readable)
[]
mabry1985
deleted the
feat/bd-srb5a-github-review-threads-strips-coderabbit
branch
November 7, 2026 02:55
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
github_review_threadstruncated each thread's first comment at a fixed 1000 chars on the raw markdown, so CodeRabbit findings — which open with<!-- … -->markers and a long collapsible<details>…</details>analysis chain before the actual finding — were always cut off before the useful text.This change adds a
_strip_review_boilerplatehelper that drops HTML comments and<details>…</details>blocks and collapses whitespace before capping, falling back to the first<summary>line only when stripping leaves nothing (so a boilerplate-only thread still shows its headline). The tool gains abody_max_chars: int = 1000parameter (clamped 200..8000); the overall response remains bounded by_bounded(…, _MAX_PR_CHARS), and the one-line docstring is updated. Plain-body behavior is unchanged. The change is additive — no existing behavior was removed.Fixes #63