llm_meta_widget 0.4.1 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 17cad71af08d5e4c49a3e7418055bd9c034880d9ab6cae2cbb3fb6cf3a0f0804
-  data.tar.gz: 24e7aa0c38400bccee9ddc832a6da10cf2788f42e343b67901d0cfc9a7748e24
+  metadata.gz: 2503371d1d6229fbe6e57a807fb8d672190de589ccdb877d2a90a06feae20be0
+  data.tar.gz: bce91755bb903ab02f886be7e9ab90733566699359e468624f3244f1202ad17a
 SHA512:
-  metadata.gz: 2e55a245ad87d21f2d235fe9c211751520172df72f7197ad70b558a546c0b80e62ce63e9f00718ac0f91145d2a14bf920db808f70a550c7822d603c20a565833
-  data.tar.gz: bdad3eda95fd643db1dc11fc10c4e2c48d0d3442d3e2a1a79012f1e8bb2af769dc98217d897b6fd7b173a17f06e7ba55ab3209d50a3cd7186349dbce5920030b
+  metadata.gz: d833790ba0fcdb48cb291a9cde65ab45e2c07284a18a9699c755289a2ded830695e84b3125e552f770c66db8e9930e9c28410baaef2c5e0459d75ef9b93636a4
+  data.tar.gz: 819cd9aa17be97c44b3828980b9faa6a3f79476a2802337b3e59a4149c17a45058a0355a7e5a29f39594ed8c8dfae373cb6040dde762795b6c031b9467d70128

data/app/assets/javascripts/llm_meta_widget/orchestrator.js

@@ -32,6 +32,7 @@
 //     onTextDelta:     (str)  => {},
 //     onThinkingDelta: (str)  => {},
 //     onToolCall:      (tc)   => {},         // { id, name, arguments }
+//     onToolDispatched: ({toolCall, value, error}) => {},  // after it ran
 //     onPhase:         (name) => {},         // 'thinking' | 'tool_execution' | ...
 //     signal:          abortController.signal
 //   })
@@ -329,6 +330,11 @@ export async function runChatLoop(opts) {
     onTextDelta,
     onThinkingDelta,
     onToolCall,
+    // Fired as each tool finishes, so a caller can show what ran WHILE the
+    // turn is still going. The returned `dispatched` array says the same
+    // thing, but only once the whole loop ends — which on a slow model is
+    // minutes after the work happened, with the user watching unnamed tools.
+    onToolDispatched,
     onPhase,
     ...singleOpts
   } = opts
@@ -454,7 +460,8 @@ export async function runChatLoop(opts) {
     // the write. It also means a failed action is something the model can
     // see and correct, instead of a red mark only the user notices.
     const roundTripResults = []
-    for (const { toolCall, error } of localOut.dispatched) {
+    for (const { toolCall, value, error } of localOut.dispatched) {
+      onToolDispatched?.({ toolCall, value, error })
       roundTripResults.push({
         tc: toolCall,
         result: error ? { error: String(error.message || error) } : { ok: true, applied: toolCall.name }
@@ -468,9 +475,11 @@ export async function runChatLoop(opts) {
       try {
         const value = await callMcpTool({ endpoint: tool.endpoint, name: tc.name, args, signal })
         allDispatched.push({ toolCall: tc, value })
+        onToolDispatched?.({ toolCall: tc, value })
         roundTripResults.push({ tc, result: value })
       } catch (error) {
         allDispatched.push({ toolCall: tc, error })
+        onToolDispatched?.({ toolCall: tc, error })
         roundTripResults.push({ tc, result: { error: String(error.message || error) } })
       }
     }
@@ -488,9 +497,11 @@ export async function runChatLoop(opts) {
           signal
         })
         allDispatched.push({ toolCall: tc, value })
+        onToolDispatched?.({ toolCall: tc, value })
         roundTripResults.push({ tc, result: value })
       } catch (error) {
         allDispatched.push({ toolCall: tc, error })
+        onToolDispatched?.({ toolCall: tc, error })
         // Feed the error text back to the LLM as the tool result — better
         // than dropping it (the LLM can react, apologize, retry differently).
         roundTripResults.push({ tc, result: { error: String(error.message || error) } })

data/app/views/llm_meta_widget/_chat_panel.html.erb

@@ -1070,6 +1070,48 @@
 		label.textContent = roleLabel("assistant");
 	}
 
+	// Tool chips, written as the tools run rather than assembled at the end.
+	// The end-of-turn footer was invisible for as long as the turn lasted —
+	// minutes on a thinking model — so a visitor watched tools execute with
+	// no idea which ones.
+	var liveChips = null;      // the container inside the current bubble
+	var pendingChips = [];     // chips awaiting their outcome, oldest first
+
+	function chipRow() {
+		if (liveChips && liveChips.parentNode) return liveChips;
+		if (!activeAssistantBody || !activeAssistantBody.parentNode) return null;
+		liveChips = document.createElement("div");
+		liveChips.className = "lmw-tool-chips";
+		// Inside the bubble but OUTSIDE .message-content: every text delta
+		// re-renders that element's markdown from scratch, which silently
+		// erased any chip already written into it.
+		activeAssistantBody.parentNode.appendChild(liveChips);
+		return liveChips;
+	}
+
+	function announceToolCall(toolCall) {
+		var row = chipRow();
+		if (!row) return;
+		var chip = document.createElement("span");
+		chip.className = "lmw-tool-chip running";
+		chip.textContent = "⏳ " + toolCall.name;
+		chip.dataset.tool = toolCall.name;
+		row.appendChild(chip);
+		pendingChips.push(chip);
+		historyEl.scrollTop = historyEl.scrollHeight;
+	}
+
+	function resolveToolCall(outcome) {
+		// Match by name, oldest first: ids are not always echoed back, and a
+		// repeated name resolves in call order.
+		var idx = pendingChips.findIndex(function(c) { return c.dataset.tool === outcome.toolCall.name; });
+		var chip = idx === -1 ? null : pendingChips.splice(idx, 1)[0];
+		if (!chip) return;
+		chip.className = "lmw-tool-chip" + (outcome.error ? " error" : "");
+		chip.textContent = (outcome.error ? "❌ " : "🔧 ") + outcome.toolCall.name;
+		if (outcome.error) chip.title = outcome.error.message || String(outcome.error);
+	}
+
 	var currentThinkingBlock = null;
 	var currentThinkingBody  = null;
 	// The assistant bubble of the turn in flight. Reasoning belongs ABOVE that
@@ -1196,6 +1238,8 @@
 
 		var assistantBody = appendTurn("assistant", "");
 		activeAssistantBody = assistantBody;
+		liveChips = null;
+		pendingChips = [];
 		markWorking(assistantBody.roleLabel);
 		var assistantMarkdown = "";  // accumulate raw markdown, re-render on each delta
 
@@ -1222,6 +1266,8 @@
 				aiActions:     window[ACTIONS_GLOBAL] || {},
 				maxRounds:     MAX_ROUNDS,
 				signal:        currentAbort.signal,
+				onToolCall: function(toolCall) { announceToolCall(toolCall); },
+				onToolDispatched: function(outcome) { resolveToolCall(outcome); },
 				onPhase: function(name) {
 					// 'thinking' covers the long silence before the first
 					// token; 'responding' means text is on its way.
@@ -1258,23 +1304,15 @@
 			conversation.push({ role: "user",      content: userText });
 			conversation.push({ role: "assistant", content: result.content });
 
-			// Append a compact "tools used" footer INSIDE the assistant
-			// bubble so users have visual confirmation of what the LLM
-			// actually did — especially important when the LLM's own text
-			// is incomplete (e.g. "Let me first search…" with no follow-up
-			// synthesis, common with weaker tool-use models).
-			if (result.dispatched.length > 0) {
-				var chips = document.createElement("div");
-				chips.className = "lmw-tool-chips";
-				result.dispatched.forEach(function(d) {
-					var chip = document.createElement("span");
-					chip.className = "lmw-tool-chip" + (d.error ? " error" : "");
-					chip.textContent = (d.error ? "❌ " : "🔧 ") + d.toolCall.name;
-					if (d.error) chip.title = d.error.message;
-					chips.appendChild(chip);
-				});
-				assistantBody.appendChild(chips);
-			}
+			// Anything still pending never reported an outcome (a class that
+			// does not round-trip, or a dispatch that vanished). Reconcile
+			// from the loop's own record so no chip is left spinning.
+			result.dispatched.forEach(function(d) { resolveToolCall(d); });
+			pendingChips.forEach(function(chip) {
+				chip.className = "lmw-tool-chip";
+				chip.textContent = "🔧 " + chip.dataset.tool;
+			});
+			pendingChips = [];
 
 			// Skipped = LLM tried a tool that doesn't exist. Real signal.
 			if (result.skipped.length > 0) {

data/lib/llm_meta_widget/version.rb

@@ -1,3 +1,3 @@
 module LlmMetaWidget
-  VERSION = "0.4.1"
+  VERSION = "0.4.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: llm_meta_widget
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.4.2
 platform: ruby
 authors:
 - jdkim