@lazyingart/agintiflow 0.20.173 → 0.20.174

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lazyingart/agintiflow",
-  "version": "0.20.173",
+  "version": "0.20.174",
   "type": "module",
   "description": "AgInTiFlow is a project-aware agent workspace for hybrid wet-dry R&D, hardware-aware intelligence, software automation, and industrial workflows.",
   "license": "Apache-2.0",

package/public/app.js

@@ -1796,6 +1796,83 @@ function renderPlanEvent(entry) {
   `;
 }
 
+function embeddedWorkspaceChangesFromText(value = "") {
+  const lines = String(value || "").split(/\r?\n/);
+  const changes = [];
+  let currentTool = "";
+
+  for (let index = 0; index < lines.length; index += 1) {
+    const toolMatch = lines[index].match(/^\s*Tool:\s*(.+?)\s*$/i);
+    if (toolMatch) {
+      currentTool = toolMatch[1].trim();
+      continue;
+    }
+
+    const pathMatch = lines[index].match(/^\s*Path:\s*(.+?)\s*$/i);
+    if (!pathMatch) continue;
+
+    let cursor = index + 1;
+    while (cursor < lines.length && !/^\s*Diff:\s*$/i.test(lines[cursor]) && !/^\s*(?:Tool|Path):\s*/i.test(lines[cursor])) {
+      cursor += 1;
+    }
+    if (!/^\s*Diff:\s*$/i.test(lines[cursor] || "")) continue;
+
+    const diffLines = [];
+    cursor += 1;
+    while (cursor < lines.length) {
+      if (diffLines.length && /^\s*(?:Tool|Path):\s*/i.test(lines[cursor])) break;
+      if (
+        diffLines.length &&
+        /^\s*(?:Output:|No command output|Blocked by guardrail\.|Mock run complete\.)/i.test(lines[cursor]) &&
+        !/^[-+@ ]/.test(lines[cursor])
+      ) {
+        break;
+      }
+      diffLines.push(lines[cursor]);
+      cursor += 1;
+    }
+    while (diffLines.length && !diffLines[diffLines.length - 1].trim()) diffLines.pop();
+    if (diffLines.length) {
+      changes.push({
+        toolName: currentTool || "write_file",
+        path: pathMatch[1].trim(),
+        diff: diffLines.join("\n"),
+      });
+      index = Math.max(index, cursor - 1);
+    }
+  }
+
+  return changes;
+}
+
+function embeddedWorkspaceSummary(value = "") {
+  const summary = [];
+  let skippingDiff = false;
+  for (const line of String(value || "").split(/\r?\n/)) {
+    if (/^\s*Diff:\s*$/i.test(line)) {
+      skippingDiff = true;
+      continue;
+    }
+    if (skippingDiff && /^\s*(?:Tool|Path):\s*/i.test(line)) skippingDiff = false;
+    if (skippingDiff) continue;
+    if (/^\s*Path:\s*/i.test(line)) continue;
+    summary.push(line);
+  }
+  return summary.join("\n").replace(/\n{3,}/g, "\n\n").trim();
+}
+
+function renderEmbeddedWorkspaceResult(value = "", entry = {}) {
+  const changes = embeddedWorkspaceChangesFromText(value);
+  if (changes.length === 0) return "";
+  const summary = embeddedWorkspaceSummary(value);
+  return `
+    ${summary ? `<div class="event-result-summary markdown-body">${renderMarkdown(summary)}</div>` : ""}
+    <div class="embedded-change-list">
+      ${changes.map((change) => renderWorkspaceChangeEvent({ data: change, at: entry.at, content: change.path })).join("")}
+    </div>
+  `;
+}
+
 function renderWorkspaceChangeEvent(entry) {
   const data = entry.data || {};
   const toolName = data.toolName || data.action || "file.changed";
@@ -1833,14 +1910,18 @@ function renderToolEvent(entry) {
   const skipped = message === "tool.skipped";
   const toolName = data.toolName || "tool";
   const args = data.args || {};
+  const resultText = String(data.result || args.result || "");
+  const embeddedResult = resultText ? renderEmbeddedWorkspaceResult(resultText, entry) : "";
   const argPreview =
-    toolName === "run_command" && args.command
+    toolName === "finish"
+      ? ""
+      : toolName === "run_command" && args.command
       ? args.command
       : args.path || args.url || args.query || args.q || (Object.keys(args).length ? JSON.stringify(args) : "");
   const stdout = data.stdout ? outputPreviewText(data.stdout) : null;
   const stderr = data.stderr ? outputPreviewText(data.stderr) : null;
   return `
-    <article class="event-card ${failed ? "event-failed" : skipped ? "event-muted" : "event-tool"}">
+    <article class="event-card ${failed ? "event-failed" : skipped ? "event-muted" : toolName === "finish" ? "event-finish" : "event-tool"}">
       <div class="event-card-meta">
         <span>${escapeHtml(failed ? "tool failed" : skipped ? "tool skipped" : message === "tool.started" ? "tool" : "tool done")}</span>
         <span>${entry.at ? escapeHtml(new Date(entry.at).toLocaleString()) : ""}</span>
@@ -1848,6 +1929,7 @@ function renderToolEvent(entry) {
       <strong class="event-path">${escapeHtml(toolName)}</strong>
       ${argPreview ? `<code class="event-inline-code">${escapeHtml(argPreview)}</code>` : ""}
       ${data.error || data.reason ? `<div class="event-fold-note">${escapeHtml(data.error || data.reason)}</div>` : ""}
+      ${embeddedResult || (toolName === "finish" && resultText ? `<div class="markdown-body">${renderMarkdown(resultText)}</div>` : "")}
       ${stdout ? `<div class="log-stream-title">stdout</div><pre class="event-output">${escapeHtml(stdout.text)}</pre>` : ""}
       ${stdout?.hidden > 0 ? `<div class="event-fold-note">... ${stdout.hidden} more stdout line(s) folded</div>` : ""}
       ${stderr ? `<div class="log-stream-title">stderr</div><pre class="event-output">${escapeHtml(stderr.text)}</pre>` : ""}
@@ -1862,13 +1944,14 @@ function renderStatusEvent(entry) {
   const label = entry.eventLabel || message.replace(/^[^.]+\./, "");
   const content = data.result || data.error || data.reason || entry.content || "";
   const failed = message === "session.failed";
+  const embeddedResult = content ? renderEmbeddedWorkspaceResult(content, entry) : "";
   return `
-    <article class="event-card ${failed ? "event-failed" : "event-muted"}">
+    <article class="event-card ${failed ? "event-failed" : message === "session.finished" && embeddedResult ? "event-finish" : "event-muted"}">
       <div class="event-card-meta">
         <span>${escapeHtml(label)}</span>
         <span>${entry.at ? escapeHtml(new Date(entry.at).toLocaleString()) : ""}</span>
       </div>
-      ${content ? `<div class="chat-content">${escapeHtml(content)}</div>` : ""}
+      ${embeddedResult || (content ? `<div class="chat-content">${escapeHtml(content)}</div>` : "")}
     </article>
   `;
 }
@@ -1909,9 +1992,14 @@ function renderStructuredEvent(entry) {
 
 function renderLogs(run) {
   logsEl.dataset.mode = "active";
+  const structuredRunResult = run.result ? renderEmbeddedWorkspaceResult(run.result, { at: run.endedAt || run.updatedAt || "" }) : "";
   const parts = [
     `<div class="log-line">${escapeHtml(`status=${run.status} session=${run.sessionId} provider=${run.provider} model=${run.model}`)}</div>`,
-    run.result ? `<div class="log-line">${escapeHtml(`result=${run.result}`)}</div>` : "",
+    structuredRunResult
+      ? `<article class="event-card event-finish"><div class="event-card-meta"><span>result</span></div>${structuredRunResult}</article>`
+      : run.result
+        ? `<div class="log-line">${escapeHtml(`result=${run.result}`)}</div>`
+        : "",
     run.error ? `<div class="log-line error">${escapeHtml(`error=${run.error}`)}</div>` : "",
   ];
 
@@ -2174,6 +2262,28 @@ function renderChat(chatEntries) {
           </article>
         `;
       }
+      if (entry.role === "tool") {
+        try {
+          const parsed = JSON.parse(entry.content || "{}");
+          const toolName = parsed.toolName || "tool";
+          return renderToolEvent({
+            role: "event",
+            eventType: parsed.ok === false ? "tool.failed" : "tool.completed",
+            message: parsed.ok === false ? "tool.failed" : "tool.completed",
+            eventLabel: parsed.ok === false ? "tool failed" : "tool done",
+            data: parsed,
+            content: parsed.result || toolName,
+            at: entry.at,
+          });
+        } catch {
+          return `
+            <article class="event-card event-tool">
+              <div class="event-card-meta">tool${entry.at ? ` · ${new Date(entry.at).toLocaleString()}` : ""}</div>
+              <pre class="event-output">${escapeHtml(entry.content || "")}</pre>
+            </article>
+          `;
+        }
+      }
       const role = entry.role === "assistant" ? "assistant" : "user";
       const label = role === "assistant" ? t("assistantLabel") : t("youLabel");
       const content =

package/public/styles.css

@@ -1264,6 +1264,19 @@ button.danger {
   border-color: rgba(59, 130, 246, 0.28);
 }
 
+.event-finish {
+  border-color: rgba(16, 185, 129, 0.36);
+  background:
+    radial-gradient(circle at 0% 0%, rgba(16, 185, 129, 0.14), transparent 30%),
+    rgba(236, 253, 245, 0.46);
+}
+
+#logs .event-finish {
+  background:
+    radial-gradient(circle at 0% 0%, rgba(16, 185, 129, 0.16), transparent 30%),
+    rgba(6, 78, 59, 0.28);
+}
+
 .event-failed {
   border-color: rgba(248, 113, 113, 0.45);
   background: rgba(127, 29, 29, 0.12);
@@ -1337,6 +1350,27 @@ button.danger {
   font-size: 0.8rem;
 }
 
+.event-result-summary {
+  margin: 4px 0 12px;
+  padding: 10px 12px;
+  border-radius: 12px;
+  background: rgba(15, 118, 110, 0.08);
+}
+
+#logs .event-result-summary {
+  background: rgba(255, 255, 255, 0.08);
+}
+
+.embedded-change-list {
+  display: grid;
+  gap: 8px;
+}
+
+.embedded-change-list > .event-card {
+  margin: 0;
+  box-shadow: none;
+}
+
 #logs .event-fold-note {
   color: #94a3b8;
 }

package/scripts/smoke-cli-chat.js

@@ -896,6 +896,9 @@ try {
   if (!latest.stdout.includes(" write ") || !latest.stdout.includes("+Created by AgInTiFlow mock mode.")) {
     throw new Error("bare aginti resume did not replay formatted file-change diff context");
   }
+  if (!/\bfinish\b/.test(latest.stdout) || latest.stdout.includes('{"result":"Mock run complete')) {
+    throw new Error("bare aginti resume did not render finish-result diffs in formatted mode");
+  }
   if (latest.stdout.includes("resume note=showing chat transcript only")) {
     throw new Error("bare aginti resume regressed to chat-only history");
   }

package/scripts/smoke-web-ui.js

@@ -209,6 +209,21 @@ try {
     const chatTail = (await page.locator("#chat-thread").innerText().catch(() => "")).slice(-1200);
     throw new Error(`web UI did not render formatted file-change diff event; state=${state}\nlogs tail:\n${logsTail}\nchat tail:\n${chatTail}`);
   }
+  try {
+    await page.waitForFunction(
+      () =>
+        [...document.querySelectorAll(".event-finish .change-diff")].some((node) =>
+          (node.textContent || "").includes("+Created by AgInTiFlow mock mode.")
+        ),
+      null,
+      { timeout: 30000 }
+    );
+  } catch (error) {
+    const state = await page.locator("#run-state").evaluate((node) => node.dataset.status || "").catch(() => "unknown");
+    const logsTail = (await page.locator("#logs").innerText().catch(() => "")).slice(-1200);
+    const chatTail = (await page.locator("#chat-thread").innerText().catch(() => "")).slice(-1200);
+    throw new Error(`web UI did not render formatted finish-result diff event; state=${state}\nlogs tail:\n${logsTail}\nchat tail:\n${chatTail}`);
+  }
 
   await page.click("#open-settings");
   await page.waitForSelector("#settings-modal[open]");
@@ -246,6 +261,7 @@ try {
           "new-session-resets-scope",
           "formatted-plan-event-card",
           "formatted-file-diff-event-card",
+          "formatted-finish-result-diff-card",
           "settings-provider-dropdowns",
           "settings-wrapper-dropdowns",
         ],

package/src/interactive-cli.js

@@ -778,6 +778,71 @@ export function formatWorkspaceChange(change = {}) {
   };
 }
 
+function embeddedWorkspaceChangesFromText(value = "") {
+  const lines = String(value || "").split(/\r?\n/);
+  const changes = [];
+  let currentTool = "";
+
+  for (let index = 0; index < lines.length; index += 1) {
+    const toolMatch = lines[index].match(/^\s*Tool:\s*(.+?)\s*$/i);
+    if (toolMatch) {
+      currentTool = toolMatch[1].trim();
+      continue;
+    }
+
+    const pathMatch = lines[index].match(/^\s*Path:\s*(.+?)\s*$/i);
+    if (!pathMatch) continue;
+
+    let cursor = index + 1;
+    while (cursor < lines.length && !/^\s*Diff:\s*$/i.test(lines[cursor]) && !/^\s*(?:Tool|Path):\s*/i.test(lines[cursor])) {
+      cursor += 1;
+    }
+    if (!/^\s*Diff:\s*$/i.test(lines[cursor] || "")) continue;
+
+    const diffLines = [];
+    cursor += 1;
+    while (cursor < lines.length) {
+      if (diffLines.length && /^\s*(?:Tool|Path):\s*/i.test(lines[cursor])) break;
+      if (
+        diffLines.length &&
+        /^\s*(?:Output:|No command output|Blocked by guardrail\.|Mock run complete\.)/i.test(lines[cursor]) &&
+        !/^[-+@ ]/.test(lines[cursor])
+      ) {
+        break;
+      }
+      diffLines.push(lines[cursor]);
+      cursor += 1;
+    }
+    while (diffLines.length && !diffLines[diffLines.length - 1].trim()) diffLines.pop();
+    if (diffLines.length) {
+      changes.push({
+        toolName: currentTool || "write_file",
+        path: pathMatch[1].trim(),
+        diff: diffLines.join("\n"),
+      });
+      index = Math.max(index, cursor - 1);
+    }
+  }
+
+  return changes;
+}
+
+function embeddedWorkspaceSummary(value = "") {
+  const summary = [];
+  let skippingDiff = false;
+  for (const line of String(value || "").split(/\r?\n/)) {
+    if (/^\s*Diff:\s*$/i.test(line)) {
+      skippingDiff = true;
+      continue;
+    }
+    if (skippingDiff && /^\s*(?:Tool|Path):\s*/i.test(line)) skippingDiff = false;
+    if (skippingDiff) continue;
+    if (/^\s*Path:\s*/i.test(line)) continue;
+    summary.push(line);
+  }
+  return summary.join("\n").replace(/\n{3,}/g, "\n\n").trim();
+}
+
 function printWorkspaceChange(change = {}) {
   if (!change?.diff) return;
   const formatted = formatWorkspaceChange(change);
@@ -787,6 +852,15 @@ function printWorkspaceChange(change = {}) {
   for (const line of formatted.lines) outputLine(`${gutter}${line}`);
 }
 
+function printEmbeddedWorkspaceResult(result = "", { labelName = "finish", time = "" } = {}) {
+  const changes = embeddedWorkspaceChangesFromText(result);
+  if (changes.length === 0) return false;
+  const summary = embeddedWorkspaceSummary(result);
+  if (summary) printHistoryBlock(labelName, summary, { time, bg: ansi.systemBg });
+  for (const change of changes) printWorkspaceChange(change);
+  return true;
+}
+
 function printPreviewBlock(role, text, { time = "", bg = ansi.systemBg, maxLines = 5 } = {}) {
   const header = [label(role, bg).trimEnd(), time ? color(time, ansi.dim) : ""].filter(Boolean).join(" ");
   outputLine(header);
@@ -2201,6 +2275,24 @@ async function latestSession() {
 }
 
 function printHistoryEntry(entry) {
+  if (entry.role === "tool") {
+    const time = entry.at ? new Date(entry.at).toLocaleTimeString([], { hour: "2-digit", minute: "2-digit" }) : "";
+    try {
+      const parsed = JSON.parse(entry.content || "{}");
+      if (parsed.toolName === "finish" && printEmbeddedWorkspaceResult(parsed.result || parsed.args?.result || "", { labelName: "finish", time })) {
+        return;
+      }
+      if (parsed.diff) {
+        printWorkspaceChange(parsed);
+        return;
+      }
+      outputLine(`${label(parsed.ok === false ? "fail" : "done", parsed.ok === false ? ansi.red : ansi.systemBg)} ${toolStatusDetails(parsed)}`);
+      return;
+    } catch {
+      printHistoryBlock("tool", entry.content, { time, bg: ansi.systemBg });
+      return;
+    }
+  }
   const role = entry.role === "assistant" ? "aginti>" : entry.role === "user" ? "user>" : String(entry.role || "note");
   const bg = role === "aginti>" ? ansi.agentBg : role === "user>" ? ansi.userBg : ansi.systemBg;
   const time = entry.at ? new Date(entry.at).toLocaleTimeString([], { hour: "2-digit", minute: "2-digit" }) : "";
@@ -2279,6 +2371,9 @@ function printResumeEvent(event = {}) {
       printWorkspaceChange(data);
       return true;
     }
+    if (data.toolName === "finish" && printEmbeddedWorkspaceResult(data.result || data.args?.result || "", { labelName: "finish" })) {
+      return true;
+    }
     if (data.toolName === "run_command") {
       printCommandOutputLog({
         command: data.args?.command || data.command || "",
@@ -2318,6 +2413,7 @@ function printResumeEvent(event = {}) {
     return true;
   }
   if (type === "session.finished") {
+    if (printEmbeddedWorkspaceResult(data.result || "", { labelName: "finish" })) return true;
     outputLine(`${label("state", ansi.systemBg)} status=finished${data.result ? ` result=${compactLine(data.result, 86)}` : ""}`);
     return true;
   }
@@ -4178,7 +4274,11 @@ async function runPrompt(prompt, state, packageDir, { approvalDepth = 0 } = {})
         } else if (type === "tool.started") {
           printStatusEvent(state, "tool", toolStatusDetails(data));
         } else if (type === "tool.completed" || type === "tool.failed") {
-          printStatusEvent(state, type === "tool.failed" || data.ok === false ? "tool_failed" : "tool_done", toolStatusDetails(data));
+          if (data.toolName === "finish" && printEmbeddedWorkspaceResult(data.result || data.args?.result || "", { labelName: "finish" })) {
+            // The finish tool can recap a workspace diff; render it like the original write/patch event.
+          } else {
+            printStatusEvent(state, type === "tool.failed" || data.ok === false ? "tool_failed" : "tool_done", toolStatusDetails(data));
+          }
         } else if (type === "file.changed") {
           printWorkspaceChange(data);
         } else if (type === "tool.blocked") {