@@ -7,6 +7,7 @@
|
|||||||
- Memory: allow custom OpenAI-compatible embedding endpoints for memory search (remote baseUrl/apiKey/headers). (#819 — thanks @mukhtharcm)
|
- Memory: allow custom OpenAI-compatible embedding endpoints for memory search (remote baseUrl/apiKey/headers). (#819 — thanks @mukhtharcm)
|
||||||
|
|
||||||
### Fixes
|
### Fixes
|
||||||
|
- TUI: keep the last streamed response instead of replacing it with “(no output)”. (#747 — thanks @thewilloftheshadow)
|
||||||
- Slack: accept slash commands with or without leading `/` for custom command configs. (#798 — thanks @thewilloftheshadow)
|
- Slack: accept slash commands with or without leading `/` for custom command configs. (#798 — thanks @thewilloftheshadow)
|
||||||
- Onboarding/Configure: refuse to proceed with invalid configs; run `clawdbot doctor` first to avoid wiping custom fields. (#764 — thanks @mukhtharcm)
|
- Onboarding/Configure: refuse to proceed with invalid configs; run `clawdbot doctor` first to avoid wiping custom fields. (#764 — thanks @mukhtharcm)
|
||||||
- Anthropic: merge consecutive user turns (preserve newest metadata) before validation to avoid “Incorrect role information” errors. (#804 — thanks @ThomsenDrake)
|
- Anthropic: merge consecutive user turns (preserve newest metadata) before validation to avoid “Incorrect role information” errors. (#804 — thanks @ThomsenDrake)
|
||||||
|
|||||||
@@ -49,6 +49,14 @@ export class ChatLog extends Container {
|
|||||||
this.streamingAssistant.setText(text);
|
this.streamingAssistant.setText(text);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
getStreamingText(runId?: string) {
|
||||||
|
if (!this.streamingAssistant) return null;
|
||||||
|
if (runId && this.streamingRunId && runId !== this.streamingRunId) {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
return this.streamingText;
|
||||||
|
}
|
||||||
|
|
||||||
finalizeAssistant(text: string, runId?: string) {
|
finalizeAssistant(text: string, runId?: string) {
|
||||||
if (
|
if (
|
||||||
this.streamingAssistant &&
|
this.streamingAssistant &&
|
||||||
|
|||||||
20
src/tui/tui.test.ts
Normal file
20
src/tui/tui.test.ts
Normal file
@@ -0,0 +1,20 @@
|
|||||||
|
import { describe, expect, it } from "vitest";
|
||||||
|
|
||||||
|
import { resolveFinalAssistantText } from "./tui.js";
|
||||||
|
|
||||||
|
describe("resolveFinalAssistantText", () => {
|
||||||
|
it("falls back to streamed text when final text is empty", () => {
|
||||||
|
expect(
|
||||||
|
resolveFinalAssistantText({ finalText: "", streamedText: "Hello" }),
|
||||||
|
).toBe("Hello");
|
||||||
|
});
|
||||||
|
|
||||||
|
it("prefers the final text when present", () => {
|
||||||
|
expect(
|
||||||
|
resolveFinalAssistantText({
|
||||||
|
finalText: "All done",
|
||||||
|
streamedText: "partial",
|
||||||
|
}),
|
||||||
|
).toBe("All done");
|
||||||
|
});
|
||||||
|
});
|
||||||
@@ -39,6 +39,17 @@ export type TuiOptions = {
|
|||||||
message?: string;
|
message?: string;
|
||||||
};
|
};
|
||||||
|
|
||||||
|
export function resolveFinalAssistantText(params: {
|
||||||
|
finalText?: string | null;
|
||||||
|
streamedText?: string | null;
|
||||||
|
}) {
|
||||||
|
const finalText = params.finalText ?? "";
|
||||||
|
if (finalText.trim()) return finalText;
|
||||||
|
const streamedText = params.streamedText ?? "";
|
||||||
|
if (streamedText.trim()) return streamedText;
|
||||||
|
return "(no output)";
|
||||||
|
}
|
||||||
|
|
||||||
type ChatEvent = {
|
type ChatEvent = {
|
||||||
runId: string;
|
runId: string;
|
||||||
sessionKey: string;
|
sessionKey: string;
|
||||||
@@ -642,7 +653,11 @@ export async function runTui(opts: TuiOptions) {
|
|||||||
const text = extractTextFromMessage(evt.message, {
|
const text = extractTextFromMessage(evt.message, {
|
||||||
includeThinking: showThinking,
|
includeThinking: showThinking,
|
||||||
});
|
});
|
||||||
chatLog.finalizeAssistant(text || "(no output)", evt.runId);
|
const finalText = resolveFinalAssistantText({
|
||||||
|
finalText: text,
|
||||||
|
streamedText: chatLog.getStreamingText(evt.runId),
|
||||||
|
});
|
||||||
|
chatLog.finalizeAssistant(finalText, evt.runId);
|
||||||
noteFinalizedRun(evt.runId);
|
noteFinalizedRun(evt.runId);
|
||||||
activeChatRunId = null;
|
activeChatRunId = null;
|
||||||
setActivityStatus("idle");
|
setActivityStatus("idle");
|
||||||
|
|||||||
Reference in New Issue
Block a user