Repository navigation
fix: return five-field rows from list_reviews - #191
Conversation
list_reviews pushed the whole public ledger of every review. Against a store of 1133 reviews the unfiltered result was about 19 MB, past the 10 MB stdio message limit: none of 24 recorded author calls got a usable inline result, and four closed the connection. Each row now carries id, status, repository_path, the latest round's head_sha, and updated_at; details come from get_review_summary or get_review. list_pending_reviews returns the same rows, so the destructure that stripped rationale and intent texts has nothing left to strip.
|
@codex review When you finish, append exactly this marker to the review summary: |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. What shall we delve into next? Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
list_reviewsreturnedpublicReviewfor every review: requirement, scope, rounds, findings, resolutions, rereview decisions, errata, carried findings and history. Against a store of 1133 reviews the unfiltered result was 18,997,767 bytes. The MCP SDK stdio transport caps one message at 10 MB, and in agent transcripts from 2026-09-10 to 2026-10-17 none of 24 author calls got a usable inline result; four failed with "Connection closed".Change
listReviewsreturns one row per review with exactlyid,status,repository_path, the latest round'shead_sha, andupdated_at. The status filter, reviewer-provider filter and newest-first sort byupdated_atare unchanged; no inputs are added.listPendingReviewsreturns those rows directly. The destructure that strippedrationale,reconstructed_intentandintent_comparisonis gone, since a row cannot carry them.list_reviewsdescription now states what a row carries and points toget_review_summary/get_reviewfor details.docs/reference.mdis regenerated.Only these two MCP tools call
listReviews/listPendingReviews; the reviewer skills use the pending list only to select areview_idbeforeopen_review.Verification
updated_at, the latest round'shead_sha, and the five sorted keys. Four mutants each turn it red: no sort, sort by id, first round'shead_sha, and the full ledger.npm test,npm audit --omit=dev,npm run build,npm run verify:build.