Repository navigation
fix: surface invalid session errors from agent runs - #564
Open
KevinAndrewDong wants to merge 1 commit into
Open
KevinAndrewDong wants to merge 1 commit into
KevinAndrewDong wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes #99.
An agent run against a session removed by a backend reload now shows a message explaining that the session is unavailable and that the user can start a new one using the existing New Session button.
The current session and messages stay available until the user chooses New Session. Requests are not automatically retried.
Evidence
After: All 169 focused checks passed. The full CI test command passed 865 checks, with 37 skipped by the existing suite configuration.
npm run test -- --no-watch --no-progress --browsers=ChromeHeadlessCIwith Chrome configured throughCHROME_BIN.Merge Danger
Door: Two-way.
Blast Radius: Streaming.
Non-success HTTP responses from
AgentService.runSsenow reach its callers' existing error handlers. Successful SSE parsing and cancellation behavior are unchanged.AI assistance: Codex was used for implementation and regression tests; DSH provided an independent read-only review.