Skip to content

Keep provider requests valid across recovery - #630

Merged
fazxes merged 7 commits into
mainfrom
fazxes/provider-request-pipeline
Sep 4, 2026
Merged

Keep provider requests valid across recovery#630
fazxes merged 7 commits into
mainfrom
fazxes/provider-request-pipeline

Conversation

@fazxes

@fazxes fazxes commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Prevent provider message-ordering errors during recovery, history replay, and permission review.
  • Restart interrupted responses without joining text from separate requests.
  • Preserve interrupted previews for crash recovery while excluding them from replacement requests.
  • Preserve settled tool results across retries without repeating completed actions.
  • Replace failed JSON previews only after accepting a replacement, preserving the latest preview when recovery pauses.
  • Show restarted responses separately in terminal and ACP output.
  • Reduce CPU and memory used to process long headless responses.

Separate request instructions from chronological conversation and validate
the boundary before provider serialization. Restart interrupted responses
from settled work instead of stitching text across requests.

Reuse SSE event storage, derive language evidence in one scan, and skip
unused Markdown rendering for source-only consumers.
@fazxes fazxes added the type: bug Fixes incorrect behavior label Sep 3, 2026
Keep retained interruption evidence in outstanding request reservations,
including the first request after restoring a saved checkpoint.

Cover retry reservations and process termination before replacement output.
Keep the instruction and conversation rules in one core validator while
preserving the Gateway review error mapping.
Delay removing a failed preview until replacement output is accepted.
Keep the latest preview on pause, failure, or cancellation, and preserve
earlier completed commentary without joining separate responses.

Use the existing response-start event for accepted tool-only and empty
responses, including duplicate provider-result continuation.
Keep root-history bounds and authority checks after the instruction prefix.
Remove unused trailing-instruction fixture helpers.
@fazxes
fazxes marked this pull request as ready for review September 4, 2026 00:25
@fazxes
fazxes merged commit 7238966 into main Sep 4, 2026
48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug Fixes incorrect behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant