aljaz/goose
1
0
Fork 0
mirror of https://github.com/aljazceru/goose.git synced 2026-01-21 23:34:22 +01:00
Code Issues Packages Projects Releases Wiki Activity

Added option to summarize the chat when an error is triggered (#3598)

Browse Source
This commit is contained in:
Zane
2025-07-22 18:21:17 -07:00
committed by GitHub
parent eab80366e5
commit b061fdb198
3 changed files with 55 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

63
ui/desktop/src/components/BaseChat.tsx
View File

@@ -157,6 +157,7 @@ function BaseChatContent({
updateMessageStreamBody,
sessionMetadata,
isUserMessage,
clearError,
} = useChatEngine({
chat,
setChat,
@@ -434,21 +435,51 @@ function BaseChatContent({
{error.message || 'Honk! Goose experienced an error while responding'}
</div>
{/* Regular retry button for non-token-limit errors */}
<div
className="px-3 py-2 mt-2 text-center whitespace-nowrap cursor-pointer text-textStandard border border-borderSubtle hover:bg-bgSubtle rounded-full inline-block transition-all duration-150"
onClick={async () => {
// Find the last user message
const lastUserMessage = messages.reduceRight(
(found, m) => found || (m.role === 'user' ? m : null),
null as Message | null
);
if (lastUserMessage) {
append(lastUserMessage);
}
}}
>
Retry Last Message
{/* Action buttons for non-token-limit errors */}
<div className="flex gap-2 mt-2">
<div
className="px-3 py-2 text-center whitespace-nowrap cursor-pointer text-textStandard border border-borderSubtle hover:bg-bgSubtle rounded-full inline-block transition-all duration-150"
onClick={async () => {
// Create a contextLengthExceeded message similar to token limit errors
const contextMessage: Message = {
id: `context-${Date.now()}`,
role: 'assistant',
created: Math.floor(Date.now() / 1000),
content: [
{
type: 'contextLengthExceeded',
msg: 'Summarization requested due to error. Creating summary to help resolve the issue.',
},
],
display: true,
sendToLLM: false,
};
// Add the context message to trigger ContextHandler
const updatedMessages = [...messages, contextMessage];
setMessages(updatedMessages);
// Clear the error state since we're handling it with summarization
clearError();
}}
>
Summarize Conversation
</div>
<div
className="px-3 py-2 text-center whitespace-nowrap cursor-pointer text-textStandard border border-borderSubtle hover:bg-bgSubtle rounded-full inline-block transition-all duration-150"
onClick={async () => {
// Find the last user message
const lastUserMessage = messages.reduceRight(
(found, m) => found || (m.role === 'user' ? m : null),
null as Message | null
);
if (lastUserMessage) {
append(lastUserMessage);
}
}}
>
Retry Last Message
</div>
</div>
</div>
</>
@@ -472,7 +503,7 @@ function BaseChatContent({
{/* Fixed loading indicator at bottom left of chat container */}
{isLoading && (
<div className="absolute bottom-1 left-4 z-20 pointer-events-none">
<LoadingGoose
<LoadingGoose
message={isLoadingSummary ? 'summarizing conversation…' : undefined}
isWaiting={isWaiting}
isStreaming={isStreaming}

4
ui/desktop/src/hooks/useChatEngine.ts
View File

@@ -79,6 +79,7 @@ export const useChatEngine = ({
updateMessageStreamBody,
notifications,
sessionMetadata,
setError,
} = useMessageStream({
api: getApiUrl('/reply'),
id: chat.id,
@@ -402,5 +403,8 @@ export const useChatEngine = ({
// Utilities
isUserMessage,
// Error management
clearError: () => setError(undefined),
};
};

4
ui/desktop/src/hooks/useMessageStream.ts
View File

@@ -173,6 +173,9 @@ export interface UseMessageStreamHelpers {
/** Session metadata including token counts */
sessionMetadata: SessionMetadata | null;
/** Clear error state */
setError: (error: Error | undefined) => void;
}
/**
@@ -709,5 +712,6 @@ export function useMessageStream({
notifications,
currentModelInfo,
sessionMetadata,
setError,
};
}