Skip to content

feat: github_get_pr / github_get_issue return the full body (bounded) and say [truncated: N of M chars] when cut - #42

Merged
mabry1985 merged 1 commit into
mainfrom
feat/bd-6zrm-github-get-pr-github-get-issue-return-th
Oct 21, 2026
Merged

mabry1985 merged 1 commit into
mainfrom
feat/bd-6zrm-github-get-pr-github-get-issue-return-th

Conversation

@mabry1985

Copy link
Copy Markdown
Member

Summary

Both readers now use the remaining default response budget for bodies and report exact truncation counts. Added body_max_chars, capped at 60,000, preserved body whitespace, and documented the warning against editing truncated bodies.

Tests cover both readers, limits, oversized metadata, and malformed bodies. Ruff lint and formatting checks pass; all 362 tests pass, including the no-raise sweep.

Fixes #41

@protoreview protoreview Bot 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.

QA panel review — PASS

code-review · head eddbbe24ec63 · formal

All four review angles (correctness, cross-file, conventions, tests) came back clean on this PR. No findings to report. The verifier confirmed nothing-to-verify, consistent with an empty panel. No coverage gaps flagged — the diff was small enough that silence across all lanes is credible.

No findings — the review came back clean.

findings JSON (machine-readable)
[]

@protoreview protoreview Bot 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.

Promoting the PASS verdict for head eddbbe24ec63: all checks terminal-green, zero unresolved review threads. (approve-on-green)

@mabry1985
mabry1985 merged commit 524b38c into main Oct 21, 2026
4 checks passed
@mabry1985
mabry1985 deleted the feat/bd-6zrm-github-get-pr-github-get-issue-return-th branch October 21, 2026 23:10
@mabry1985 mabry1985 mentioned this pull request Oct 21, 2026
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.

github_get_pr / github_get_issue cut the body at 2000 chars, so an append via github_edit_pr can drop content

1 participant