Skip to content

fix: retry only failed chunks in large PR reviews - #3

Closed
utsab345 wants to merge 105 commits into
mainfrom
fix/3397-retry-failed-chunks
Closed

fix: retry only failed chunks in large PR reviews#3
utsab345 wants to merge 105 commits into
mainfrom
fix/3397-retry-failed-chunks

Conversation

@utsab345

@utsab345 utsab345 commented Sep 15, 2026

Copy link
Copy Markdown
Owner

References to other Issues or PRs

Fixes The-PR-Agent#3397

Brief description of what is fixed or changed

Large PR reviews can be split into multiple model calls. If one chunk returns malformed output or raises a model error, the previous implementation discarded the successful chunks and reran the entire review with the fallback model.

This change caches the prepared chunks and successfully parsed results. Fallback attempts now retry only failed chunks, then merge all chunk results in their original order once every chunk succeeds.

Regression tests cover malformed output and model-call failures, including verification that successful chunks are not requested again.

Other comments

A review still fails when a chunk cannot be recovered by any configured model, preserving the existing fallback failure behavior.

AI Generation Disclosure

No AI was used.

Release Notes

  • pr_reviewer
    • Retry only failed large-review chunks on fallback models.

shashankvarma499 and others added 30 commits September 8, 2026 13:39
@shashankvarma499
docs: fix automations note to reflect self-hosted deployments (The-PR…
4d9829d 
…-Agent#3201)

Signed-off-by: Shashank Varma <324153016+shashankvarma499@users.noreply.github.com>
@asarakhatun17-lgtm
chore: remove dead bandit configuration (The-PR-Agent#3203)
349d106
@utsab345 Utsab
TYP: ship a py.typed marker for PEP 561 compliance (The-PR-Agent#3205)
b11a1db 
Co-authored-by: Utsab <utsab@pr-agent.dev>
@PeterDaveHello
chore(deps): update yanked Python dependencies (The-PR-Agent#3211)
11c1442
@utsab345 Utsab
ci: run the pre-commit workflow on pull requests (The-PR-Agent#3212)
5e41d97 
Co-authored-by: Utsab <utsab@pr-agent.dev>
@Shaisolaris
docs: drop stale README release entries (The-PR-Agent#3208)
f7656a9
@PeterDaveHello
chore: pin uv 0.12.10 in pyproject.toml (The-PR-Agent#3153)
1dcd346
@auroraxo Aurora
test(providers): enforce capability predicate truth (The-PR-Agent#3156)
29709d4 
Co-authored-by: Aurora <aurora9c69543e@atomicmail.ai>
@PeterDaveHello
fix(polling): bound active workers across polling iterations (The-PR-…
6f2e0ff 
…Agent#3063)
@oleksii-tumanov
fix(describe): retain successful chunks after partial failure (The-PR…
d9395ee 
…-Agent#3163)

Co-authored-by: Alex Tumanov <6143578+oleksii-tumanov@users.noreply.github.com>
@7487 @claude
fix(deps): declare certifi as a range per the security-pin policy (Th…
bf05b46 
…e-PR-Agent#3204)

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
@7487 @claude
refactor(help): select help rendering by provider capability (The-PR-…
172d261 
…Agent#3213)

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
@auroraxo Aurora
fix(providers): report code-suggestion publication failure (The-PR-Ag…
00b8c9d 
…ent#3219)

Co-authored-by: Aurora <aurora9c69543e@atomicmail.ai>
@junnhwan
fix(bitbucket): restore persistent comment links (The-PR-Agent#3226)
d5c15c4 
Signed-off-by: hwan <3373484735@qq.com>
@junnhwan
fix(review): preserve findings after failed chunks (The-PR-Agent#3229)
a648424 
Signed-off-by: hwan <3373484735@qq.com>
@yashaskn8
fix(github): report total inline suggestion fallback failure (The-PR-…
8bd1532 
…Agent#3232)
@PeterDaveHello
Harden Dockerfiles for stable and predictable builds (The-PR-Agent#3218)
70d3f27
@PeterDaveHello
fix(release): keep uv.lock in sync with version bumps (The-PR-Agent#3214
aaa2992 
)
@auroraxo Aurora
refactor: select PR chat link by provider capability (The-PR-Agent#3239)
17651f7 
Co-authored-by: Aurora <aurora9c69543e@atomicmail.ai>
@PeterDaveHello
ci: verify uv.lock freshness (The-PR-Agent#3245)
5307248
@PeterDaveHello
perf(gitlab): avoid redundant project fetches for file reads (The-PR-…
1f1abd1 
…Agent#3247)
@utsab345
refactor: remove 14 dead functions in the tool layer (The-PR-Agent#3182…
539d0b8 
…) (The-PR-Agent#3236)
@utsab345
docs: add a documentation page for the MOSAICO A2A server (The-PR-Age…
2296abb 
…nt#3234)
@even-even
chore: optimize list(x.values) -> x.to_list() pandas (The-PR-Agent#3233)
3abfc84 
Co-authored-by: even-even <>
@utsab345
chore(github): upgrade PyGithub to 2.x and migrate off deprecated aut…
847dcbb 
…h classes (The-PR-Agent#3240)
@utsab345 Utsab
docs: add a complete configuration reference covering every option (T…
5bf8b40 
…he-PR-Agent#3217)

Co-authored-by: Utsab <utsab@pr-agent.dev>
@Mihailchik
test(docs): guard mkdocs and GitBook navs against drift (The-PR-Agent…
db06a90 
…#3231)

Co-authored-by: mihailchkik <Mihailchik@users.noreply.github.com>
@PeterDaveHello @IsmaelMartinez @claude
docs: correct stale and inaccurate documentation and comments (The-PR…
cb4177c 
…-Agent#3250)

Co-authored-by: Ismael Martinez Ramos <ismaelmartinez@gmail.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
@macjayz @IsmaelMartinez @claude
docs: remove stale best practices guidance (The-PR-Agent#3202)
b2d3dde 
Co-authored-by: Ismael Martinez Ramos <ismaelmartinez@gmail.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
@utsab345 Utsab
@IsmaelMartinez @claude
fix(similar_issue): port pinecone backend to the current SDK (The-PR-…
6e13fa6 
…Agent#3227)

Co-authored-by: Utsab <utsab@pr-agent.dev>
Co-authored-by: Ismael Martinez Ramos <ismaelmartinez@gmail.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
utsab345 and others added 27 commits September 13, 2026 11:32
@utsab345 Utsab
fix: remove dead configuration keys with no reader (The-PR-Agent#3279)
74210c4 
Co-authored-by: Utsab <utsab@pr-agent.dev>
@utsab345
fix(algo): skip non-unified hunk headers when anchoring lines (The-PR…
03d532b 
…-Agent#3345)
@oleksii-tumanov
fix(review): align ticket compliance prompt examples (The-PR-Agent#3326)
5a3dbc7 
Co-authored-by: Alex Tumanov <6143578+oleksii-tumanov@users.noreply.github.com>
@junnhwan
perf(gitlab): skip unused project metadata during ticket extraction (T…
a02d418 
…he-PR-Agent#3322)

Signed-off-by: Junhuan Zheng <3373484735@qq.com>
@utsab345
fix(ask): keep image blocks for user-message-only models (The-PR-Agen…
d2d3eea 
…t#3339)
@utsab345
fix(bitbucket): return empty languages when repo has none (The-PR-Age…
2eb64fe 
…nt#3341)
@utsab345
docs: drop dead large-PRs link from usage index (The-PR-Agent#3342)
417b300
@high-tail
refactor(litellm_ai_handler): deduplicate config coercion, API key ca…
bd70782 
…pture, and AWS region resolution (The-PR-Agent#3320)
@Fayupable @L4XB @Ricky-7-Yan
feat(jira): add project-key allowlist to skip lookups for key-shaped …
d24b6f3 
…noise (The-PR-Agent#3304)

Co-authored-by: L4XB <lukas.buck@e-mail.de>
Co-authored-by: Ricky-7-Yan <224488344+Ricky-7-Yan@users.noreply.github.com>
@junnhwan
feat(improve): recover failed chunks with fallback models (The-PR-Age…
2c5967e 
…nt#3324)

Signed-off-by: hwan <3373484735@qq.com>
@oleksii-tumanov @IsmaelMartinez
fix: load repo settings before push automation (The-PR-Agent#3347)
e980788 
Co-authored-by: Alex Tumanov <6143578+oleksii-tumanov@users.noreply.github.com>
Co-authored-by: IsmaelMartinez <ISMAELMARTINEZ@GMAIL.COM>
@oleksii-tumanov @IsmaelMartinez
fix(bitbucket): load repo PR commands before dispatch (The-PR-Agent#3350
c41775b 
)

Co-authored-by: Alex Tumanov <6143578+oleksii-tumanov@users.noreply.github.com>
Co-authored-by: IsmaelMartinez <ISMAELMARTINEZ@GMAIL.COM>
@oleksii-tumanov
fix(help_docs): isolate malformed model rows (The-PR-Agent#3365)
dabb8ce 
Co-authored-by: Alex Tumanov <6143578+oleksii-tumanov@users.noreply.github.com>
@oleksii-tumanov
fix: correct /help project identity (The-PR-Agent#3367)
c1a4398 
Co-authored-by: Alex Tumanov <6143578+oleksii-tumanov@users.noreply.github.com>
@keii-2596
docs: remove unsupported ignore_ticket_labels setting (The-PR-Agent#3369
729877a 
)
@utsab345
fix(improve): don't publish empty suggestions when anchors are unreso…
4fa8427 
…lved (The-PR-Agent#3353)
@utsab345
ci: smoke-run /describe on a real GitHub PR (The-PR-Agent#3358)
8386ba5
@utsab345 Utsab
feat: per-directory .pr_agent.toml for monorepo settings (The-PR-Agen…
48e7c36 
…t#3282)

Co-authored-by: Utsab <utsab@pr-agent.dev>
@oleksii-tumanov @IsmaelMartinez
fix(cli): report propagated request failures (The-PR-Agent#3368)
aa7f5d3 
Co-authored-by: Alex Tumanov <6143578+oleksii-tumanov@users.noreply.github.com>
Co-authored-by: IsmaelMartinez <ISMAELMARTINEZ@GMAIL.COM>
@utsab345
fix: delete Bitbucket Server comments (The-PR-Agent#3376)
b06126f
@L4XB
fix(reviewer): make the prompt examples agree with the types they dec…
d048e72 
…lare (The-PR-Agent#3366)
@taljeon
fix(review): retry malformed responses with fallback models (The-PR-A…
4e01503 
…gent#3359)

Co-authored-by: taljeon <taljeon@users.noreply.github.com>
@AmirF194
test(tools): pin the git-provider capability contract for callers (Th…
d2f8650 
…e-PR-Agent#3361)
@utsab345 Utsab
feat: add typed prompt output models (The-PR-Agent#3371)
a4dfdc5 
Co-authored-by: Utsab <utsab@pr-agent.dev>
@junnhwan
fix(github-app): add opt-in webhook delivery deduplication (The-PR-Ag…
3fc1a6d 
…ent#3374)
@utsab345
feat(context): load repo-context files from sibling repos (same names…
8ac8f7a 
…pace/owner) (The-PR-Agent#3362)
@utsab345
fix: retry only failed large-review chunks
20bfcad
@github-actions github-actions Bot added the bug Something isn't working label Sep 15, 2026
@utsab345

utsab345 commented Sep 15, 2026

Copy link
Copy Markdown
Owner Author

Opened against upstream as The-PR-Agent#3402.

@utsab345 utsab345 closed this Sep 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

review: malformed chunk discards successful chunks in large-PR chunking