Polish Agentic result display

Author: supermang617

app/hermes-agentic-result.js

@@ -0,0 +1,19 @@
+export function formatAgenticTaskResult(response) {
+  const text = String(response?.text || "").trim();
+  const activity = (response?.events || [])
+    .filter((event) => event?.type === "tool_activity")
+    .map((event) => String(event.payload || "").trim())
+    .filter(Boolean)
+    .filter((payload) => !text || !/\bin_progress\b/i.test(payload));
+
+  if (text && activity.length > 0) {
+    return `${text}\n\nTool activity:\n${activity.join("\n\n")}`;
+  }
+  if (text) {
+    return text;
+  }
+  if (activity.length > 0) {
+    return `Tool activity:\n${activity.join("\n\n")}`;
+  }
+  return "";
+}

app/hermes-agentic-result.test.mjs

@@ -0,0 +1,48 @@
+import assert from "node:assert/strict";
+import test from "node:test";
+
+import { formatAgenticTaskResult } from "./hermes-agentic-result.js";
+
+test("completed agentic responses show final text before tool activity", () => {
+  const output = formatAgenticTaskResult({
+    text: "Latest Ollama release: v0.30.10",
+    events: [
+      {
+        type: "tool_activity",
+        payload: "browser_open: completed\nhttps://github.com/ollama/ollama/releases/tag/v0.30.10"
+      }
+    ]
+  });
+
+  assert.match(output, /^Latest Ollama release: v0\.30\.10/);
+  assert.match(output, /Tool activity:/);
+  assert.match(output, /browser_open: completed/);
+});
+
+test("completed agentic responses hide stale in progress tool lines", () => {
+  const output = formatAgenticTaskResult({
+    text: "Latest Ollama release: v0.30.10",
+    events: [
+      {
+        type: "tool_activity",
+        payload: "browser_open: in_progress\nhttps://github.com/ollama/ollama/releases/latest"
+      }
+    ]
+  });
+
+  assert.equal(output, "Latest Ollama release: v0.30.10");
+});
+
+test("agentic progress still shows tool activity before final text exists", () => {
+  const output = formatAgenticTaskResult({
+    text: "",
+    events: [
+      {
+        type: "tool_activity",
+        payload: "browser_open: in_progress"
+      }
+    ]
+  });
+
+  assert.equal(output, "Tool activity:\nbrowser_open: in_progress");
+});

app/main.js

@@ -25,6 +25,7 @@ import {
   shouldSubmitComposer
 } from "./composer-state.js";
 import { formatAgenticHermesPrompt } from "./hermes-agentic-prompt.js";
+import { formatAgenticTaskResult } from "./hermes-agentic-result.js";
 import { formatHermesMode, parseHermesControlCommand } from "./hermes-mode.js";
 import { classifyHermesRoute } from "./hermes-routing.js";
 import { shouldClearInputOnSubmit } from "./input-state.js";
@@ -797,17 +798,6 @@ function formatRiskClass(value) {
   return labels[String(value || "")] || "Confirmation required";
 }
 
-function formatAgenticTaskResult(response) {
-  const activity = (response?.events || [])
-    .filter((event) => event?.type === "tool_activity")
-    .map((event) => String(event.payload || "").trim())
-    .filter(Boolean);
-  if (activity.length === 0) {
-    return String(response?.text || "");
-  }
-  return `Tool activity:\n${activity.join("\n\n")}\n\nResult:\n${response.text}`;
-}
-
 function formatImageGenerationResult(response) {
   const provenance = response?.provenance || {};
   return [

package.json

@@ -4,7 +4,7 @@
   "private": true,
   "type": "module",
   "scripts": {
-    "test:voice": "node --test app/voice-state.test.mjs app/attachment-state.test.mjs app/attachment-url.test.mjs app/browser-preview.test.mjs app/composer-state.test.mjs app/dynamic-context-state.test.mjs app/hermes-agentic-prompt.test.mjs app/hermes-routing.test.mjs app/hermes-mode.test.mjs app/panic-state.test.mjs app/speech-chunks.test.mjs app/speech-output.test.mjs app/staging-state.test.mjs app/input-state.test.mjs",
+    "test:voice": "node --test app/voice-state.test.mjs app/attachment-state.test.mjs app/attachment-url.test.mjs app/browser-preview.test.mjs app/composer-state.test.mjs app/dynamic-context-state.test.mjs app/hermes-agentic-prompt.test.mjs app/hermes-agentic-result.test.mjs app/hermes-routing.test.mjs app/hermes-mode.test.mjs app/panic-state.test.mjs app/speech-chunks.test.mjs app/speech-output.test.mjs app/staging-state.test.mjs app/input-state.test.mjs",
     "test:python": "powershell -NoProfile -ExecutionPolicy Bypass -File scripts/test_python.ps1",
     "test:site": "node scripts/validate_site.mjs",
     "tauri": "tauri",