Skip to content

fix: emit user-facing message on compaction failure#7975

Merged
jh-block merged 1 commit intoaaif-goose:mainfrom
rabi:compaction_failure
Mar 19, 2026
Merged

fix: emit user-facing message on compaction failure#7975
jh-block merged 1 commit intoaaif-goose:mainfrom
rabi:compaction_failure

Conversation

@rabi
Copy link
Copy Markdown
Contributor

@rabi rabi commented Mar 18, 2026

Summary

When compaction fails, the user sees "Compacting to continue conversation..." with a thinking indicator, then the stream ends with no follow-up. The user is left with a stalled thinking spinner and no indication that compaction failed.

Emit a message explaining the failure and suggesting they retry or start a new session.

When compaction fails, the user sees "Compacting to continue
conversation..." with a thinking indicator, then the stream ends
with no follow-up. The user is left with a stalled thinking spinner
and no indication that compaction failed.

Emit a message explaining the failure and suggesting they retry
or start a new session.

Change-Id: Id65286bc0adbb143d7ed71856a4ffb9837836cff
Signed-off-by: Rabi Mishra <mishra.rabi@gmail.com>
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3847fb848a

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread crates/goose/src/agents/agent.rs
@jh-block jh-block added this pull request to the merge queue Mar 19, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Mar 19, 2026
@jh-block jh-block added this pull request to the merge queue Mar 19, 2026
Merged via the queue into aaif-goose:main with commit c97fbb9 Mar 19, 2026
20 checks passed
cloorc pushed a commit to LeafAI/Leaf-ARC that referenced this pull request Mar 20, 2026
Signed-off-by: Rabi Mishra <mishra.rabi@gmail.com>
cloorc pushed a commit to LeafAI/Leaf-ARC that referenced this pull request Mar 20, 2026
Signed-off-by: Rabi Mishra <mishra.rabi@gmail.com>
elijahsgh pushed a commit to elijahsgh/goose that referenced this pull request Mar 21, 2026
Signed-off-by: Rabi Mishra <mishra.rabi@gmail.com>
Signed-off-by: esnyder <elijah.snyder1@gmail.com>
elijahsgh pushed a commit to elijahsgh/goose that referenced this pull request Mar 21, 2026
Signed-off-by: Rabi Mishra <mishra.rabi@gmail.com>
Signed-off-by: esnyder <elijah.snyder1@gmail.com>
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.

2 participants