Skip to content

retry on compaction/branch summarization retryable failures - #6775

Closed
davidbrai wants to merge 3 commits into
mainfrom
fix/issue-6647-retry-summary-requests
Closed

retry on compaction/branch summarization retryable failures#6775
davidbrai wants to merge 3 commits into
mainfrom
fix/issue-6647-retry-summary-requests

Conversation

@davidbrai

@davidbrai davidbrai commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

fixes #6647

@badlogic questions:

  1. do you think we should add a UI indication when a retry happens or no need?
  2. I supposed we need to do a similar fix for agent-harness too?

@davidbrai
davidbrai requested a review from badlogic July 17, 2026 12:54
@petrroll

Copy link
Copy Markdown
Contributor

Absolutely not my place, but wouldn't defaulting to whatever is set in user settings/retry be more expected than static one immediate retry?

I get that setting is conceptually for something a bit different but as I used I'd expect it to govern most/all agentic layer operations. Compaction being one of them.

@davidbrai

Copy link
Copy Markdown
Contributor Author

@petrroll yes, I agree. will update soon

@xl0

xl0 commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

@davidbrai Please see if you can make the retry (and transport) settinfs available to extensions. If I need to call models from the extension, I would prefer to have an option to use the current settings.

@badlogic

Copy link
Copy Markdown
Collaborator

@davidbrai

  1. no, would complicate matters too much.
  2. yes please.

@xl0 separate concern :)

@davidbrai

Copy link
Copy Markdown
Contributor Author

closing. will open a separate pr

@davidbrai davidbrai closed this Jul 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.

Compaction fails on a single transient stream drop (no retry)

4 participants