show more

Author: gglucass

.gitignore

@@ -8,3 +8,4 @@ target
 .env
 src-tauri/target
 src-tauri/gen
+announcement-recordings

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "headroom-desktop",
-  "version": "0.5.0-rc.4",
+  "version": "0.5.0-rc.5",
   "private": true,
   "type": "module",
   "scripts": {

package.json

@@ -1,6 +1,6 @@
 [package]
 name = "headroom-desktop"
-version = "0.5.0-rc.4"
+version = "0.5.0-rc.5"
 description = "Headroom v1 local-first LLM optimization tray app"
 authors = ["Codex"]
 license = "MIT"

src-tauri/Cargo.lock

@@ -413,7 +413,13 @@ pub fn list_client_connectors(
                 .find(|client| client.id == spec.id)
                 .map(|client| client.installed)
                 .unwrap_or(false);
-            let enabled = is_configured(&setup_state, spec.id);
+            // Fall back to the remembered snapshot while restore_client_setups
+            // is still re-applying on launch, so the connector doesn't flash
+            // "disabled" during the async restore window after a restart.
+            let enabled = is_configured(&setup_state, spec.id)
+                || setup_state
+                    .remembered_clients
+                    .contains_key(normalized_setup_id(spec.id));
             let verified = if enabled {
                 verify_client_setup(spec.id)
                     .map(|result| result.verified)

src-tauri/Cargo.toml

@@ -1,7 +1,7 @@
 {
   "$schema": "https://schema.tauri.app/config/2",
   "productName": "Headroom",
-  "version": "0.5.0-rc.4",
+  "version": "0.5.0-rc.5",
   "identifier": "com.extraheadroom.headroom",
   "build": {
     "beforeDevCommand": "npm run dev",

src-tauri/src/client_adapters.rs

@@ -826,6 +826,40 @@ describe("formatRequestMessages", () => {
     ).toBe("assistant:\ndone, running it:\n[tool_use]");
   });
 
+  it("surfaces tool_result payload from block.content so compression is diffable", () => {
+    // kompress shrinks the *content* of tool_result blocks. That payload lives
+    // in `block.content` (string or nested text blocks), not `block.text`, so
+    // it must be flattened — otherwise both diff sides render a bare
+    // [tool_result] and the diff shows no change.
+    expect(
+      formatRequestMessages([
+        {
+          role: "user",
+          content: [
+            { type: "tool_result", tool_use_id: "x", content: "huge tool output here" }
+          ]
+        }
+      ])
+    ).toBe("user:\nhuge tool output here");
+    expect(
+      formatRequestMessages([
+        {
+          role: "user",
+          content: [
+            {
+              type: "tool_result",
+              tool_use_id: "x",
+              content: [
+                { type: "text", text: "line one" },
+                { type: "text", text: "line two" }
+              ]
+            }
+          ]
+        }
+      ])
+    ).toBe("user:\nline one\nline two");
+  });
+
   it("labels a missing role as (unknown) instead of rendering a bare newline", () => {
     expect(formatRequestMessages([{ content: "orphan content" }])).toBe(
       "(unknown):\norphan content"

src-tauri/tauri.conf.json

@@ -269,20 +269,30 @@ function workspaceBasename(path: string | null | undefined): string | null {
 // shown as a short `[type]` marker rather than dropped, so the reader sees
 // that something was there.
 function messageText(msg: TransformationRequestMessage): string {
-  const c = msg.content;
+  return flattenContent(msg.content);
+}
+
+// Flatten a message/block `content` value to text. `tool_result` blocks carry
+// their (compressible) payload in `block.content` — a string or a nested block
+// list — not in `block.text`, so without recursing here both the original and
+// compressed sides render as a bare `[tool_result]` and the diff sees no change.
+function flattenContent(c: unknown): string {
   if (typeof c === "string") return c;
-  if (Array.isArray(c)) {
-    return c
-      .map((block) => {
-        if (!block || typeof block !== "object") return "";
-        if (typeof block.text === "string") return block.text;
-        if (typeof block.type === "string") return `[${block.type}]`;
-        return "";
-      })
-      .filter((s) => s.length > 0)
-      .join("\n");
-  }
-  return "";
+  if (!Array.isArray(c)) return "";
+  return c
+    .map((block) => {
+      if (!block || typeof block !== "object") return "";
+      const b = block as { type?: unknown; text?: unknown; content?: unknown };
+      if (typeof b.text === "string") return b.text;
+      if (b.content !== undefined) {
+        const inner = flattenContent(b.content);
+        if (inner.length > 0) return inner;
+      }
+      if (typeof b.type === "string") return `[${b.type}]`;
+      return "";
+    })
+    .filter((s) => s.length > 0)
+    .join("\n");
 }
 
 export function formatRequestMessages(messages: TransformationRequestMessage[]): string {