let5see/clawdbot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

style: italicize reasoning output

Browse Source
This commit is contained in:
Peter Steinberger
2026-01-12 21:24:19 +00:00
parent cd12ad8aab
commit 490cb834e5
4 changed files with 11 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/agents/pi-embedded-subscribe.test.ts
View File

@@ -164,7 +164,7 @@ describe("subscribeEmbeddedPiSession", () => {
expect(onBlockReply).toHaveBeenCalledTimes(2); expect(onBlockReply).toHaveBeenCalledTimes(2);
expect(onBlockReply.mock.calls[0][0].text).toBe( expect(onBlockReply.mock.calls[0][0].text).toBe(
"Reasoning:\nBecause it helps", "Reasoning:\n_Because it helps_",
); );
expect(onBlockReply.mock.calls[1][0].text).toBe("Final answer"); expect(onBlockReply.mock.calls[1][0].text).toBe("Final answer");
}); });
@@ -209,7 +209,7 @@ describe("subscribeEmbeddedPiSession", () => {
expect(onBlockReply).toHaveBeenCalledTimes(2); expect(onBlockReply).toHaveBeenCalledTimes(2);
expect(onBlockReply.mock.calls[0][0].text).toBe( expect(onBlockReply.mock.calls[0][0].text).toBe(
"Reasoning:\nBecause it helps", "Reasoning:\n_Because it helps_",
); );
expect(onBlockReply.mock.calls[1][0].text).toBe("Final answer"); expect(onBlockReply.mock.calls[1][0].text).toBe("Final answer");
@@ -283,7 +283,7 @@ describe("subscribeEmbeddedPiSession", () => {
const streamTexts = onReasoningStream.mock.calls const streamTexts = onReasoningStream.mock.calls
.map((call) => call[0]?.text) .map((call) => call[0]?.text)
.filter((value): value is string => typeof value === "string"); .filter((value): value is string => typeof value === "string");
expect(streamTexts.at(-1)).toBe("Reasoning:\nBecause it helps"); expect(streamTexts.at(-1)).toBe("Reasoning:\n_Because it helps_");
expect(assistantMessage.content).toEqual([ expect(assistantMessage.content).toEqual([
{ type: "thinking", thinking: "Because it helps" }, { type: "thinking", thinking: "Because it helps" },

4
src/agents/pi-embedded-utils.ts
View File

@@ -37,7 +37,9 @@ export function extractAssistantThinking(msg: AssistantMessage): string {
export function formatReasoningMessage(text: string): string { export function formatReasoningMessage(text: string): string {
const trimmed = text.trim(); const trimmed = text.trim();
if (!trimmed) return ""; if (!trimmed) return "";
return `Reasoning:\n${trimmed}`; // Show reasoning in italics (cursive) for markdown-friendly surfaces (Discord, etc.).
// Keep the plain "Reasoning:" prefix so existing parsing/detection keeps working.
return `Reasoning:\n_${trimmed}_`;
} }
type ThinkTaggedSplitBlock = type ThinkTaggedSplitBlock =

2
src/auto-reply/reply/agent-runner.heartbeat-typing.test.ts
View File

@@ -206,7 +206,7 @@ describe("runReplyAgent typing (heartbeat)", () => {
text?: string; text?: string;
}) => Promise<void> | void; }) => Promise<void> | void;
}) => { }) => {
await params.onReasoningStream?.({ text: "Reasoning:\nstep" }); await params.onReasoningStream?.({ text: "Reasoning:\n_step_" });
await params.onPartialReply?.({ text: "hi" }); await params.onPartialReply?.({ text: "hi" });
return { payloads: [{ text: "final" }], meta: {} }; return { payloads: [{ text: "final" }], meta: {} };
}, },

8
src/infra/heartbeat-runner.test.ts
View File

@@ -286,7 +286,7 @@ describe("runHeartbeatOnce", () => {
}; };
replySpy.mockResolvedValue([ replySpy.mockResolvedValue([
{ text: "Reasoning:\nBecause it helps" }, { text: "Reasoning:\n_Because it helps_" },
{ text: "Final alert" }, { text: "Final alert" },
]); ]);
const sendWhatsApp = vi.fn().mockResolvedValue({ const sendWhatsApp = vi.fn().mockResolvedValue({
@@ -309,7 +309,7 @@ describe("runHeartbeatOnce", () => {
expect(sendWhatsApp).toHaveBeenNthCalledWith( expect(sendWhatsApp).toHaveBeenNthCalledWith(
1, 1,
"+1555", "+1555",
"Reasoning:\nBecause it helps", "Reasoning:\n_Because it helps_",
expect.any(Object), expect.any(Object),
); );
expect(sendWhatsApp).toHaveBeenNthCalledWith( expect(sendWhatsApp).toHaveBeenNthCalledWith(
@@ -361,7 +361,7 @@ describe("runHeartbeatOnce", () => {
}; };
replySpy.mockResolvedValue([ replySpy.mockResolvedValue([
{ text: "Reasoning:\nBecause it helps" }, { text: "Reasoning:\n_Because it helps_" },
{ text: "HEARTBEAT_OK" }, { text: "HEARTBEAT_OK" },
]); ]);
const sendWhatsApp = vi.fn().mockResolvedValue({ const sendWhatsApp = vi.fn().mockResolvedValue({
@@ -384,7 +384,7 @@ describe("runHeartbeatOnce", () => {
expect(sendWhatsApp).toHaveBeenNthCalledWith( expect(sendWhatsApp).toHaveBeenNthCalledWith(
1, 1,
"+1555", "+1555",
"Reasoning:\nBecause it helps", "Reasoning:\n_Because it helps_",
expect.any(Object), expect.any(Object),
); );
} finally { } finally {