@tangle-network/ui 11.16.10 → 11.16.11

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @tangle-network/ui
 
+## 11.16.11
+
+### Patch Changes
+
+- cbfa303: A terminal output whose height cap hides lines ends in a "Show all stdout (N lines)" control that shows the rest in place, so a region that stops flush with its frame no longer hides that more output exists.
+
 ## 11.16.10
 
 ### Patch Changes

package/dist/tool-previews/command-preview.js

@@ -1,6 +1,6 @@
 import { focusRing } from "../lib/focus.js";
 import { cn } from "../lib/utils.js";
-import { memo, useId, useState } from "react";
+import { memo, useId, useLayoutEffect, useRef, useState } from "react";
 import { Fragment as Fragment$1, jsx, jsxs } from "react/jsx-runtime";
 import { ChevronDown, ChevronRight, Loader2 } from "lucide-react";
 //#region src/tool-previews/command-preview.tsx
@@ -146,6 +146,49 @@ function CommandText({ command }) {
 	}, i) : token) });
 }
 /**
+* One output stream, capped at a readable height. When the cap hides lines,
+* a control says so and shows the rest in place: a region that ends flush
+* with its frame gave no sign that more output was there.
+*/
+function OutputRegion({ label, text, className }) {
+	const ref = useRef(null);
+	const [clipped, setClipped] = useState(false);
+	const [full, setFull] = useState(false);
+	useLayoutEffect(() => {
+		const el = ref.current;
+		if (!el || full) return;
+		const check = () => setClipped(el.scrollHeight > el.clientHeight + 1);
+		check();
+		if (typeof ResizeObserver === "undefined") return;
+		const observer = new ResizeObserver(check);
+		observer.observe(el);
+		return () => observer.disconnect();
+	}, [full]);
+	const lines = text.replace(/\n$/, "").split("\n").length;
+	return /* @__PURE__ */ jsxs("div", {
+		className,
+		children: [/* @__PURE__ */ jsx("pre", {
+			ref,
+			tabIndex: 0,
+			role: "region",
+			"aria-label": label,
+			className: cn("overflow-auto whitespace-pre-wrap [overflow-wrap:anywhere] px-3 py-2 leading-relaxed", full ? "" : "max-h-80", focusRing),
+			children: text
+		}), clipped && !full ? /* @__PURE__ */ jsxs("button", {
+			type: "button",
+			onClick: () => setFull(true),
+			className: cn("w-full border-t border-white/10 px-3 py-1.5 text-left font-sans text-[11px] font-medium text-muted-foreground transition-colors hover:text-foreground", focusRing),
+			children: [
+				"Show all ",
+				label,
+				" (",
+				lines,
+				" lines)"
+			]
+		}) : null]
+	});
+}
+/**
 * One shell command as a terminal block: a `$ command` prompt line with its
 * exit status, and stdout and stderr in separate regions below it.
 *
@@ -208,26 +251,20 @@ const CommandPreview = memo(({ part, defaultExpanded = false, className }) => {
 			id: bodyId,
 			className: "border-t border-white/10",
 			children: [
-				output.stdout ? /* @__PURE__ */ jsx("pre", {
-					tabIndex: 0,
-					role: "region",
-					"aria-label": "stdout",
-					className: cn("max-h-80 overflow-auto whitespace-pre-wrap [overflow-wrap:anywhere] px-3 py-2 leading-relaxed", focusRing),
-					children: output.stdout
+				output.stdout ? /* @__PURE__ */ jsx(OutputRegion, {
+					label: "stdout",
+					text: output.stdout,
+					className: ""
 				}) : null,
-				output.stderr ? /* @__PURE__ */ jsx("pre", {
-					tabIndex: 0,
-					role: "region",
-					"aria-label": "stderr",
-					className: cn("max-h-80 overflow-auto whitespace-pre-wrap [overflow-wrap:anywhere] border-t border-white/10 px-3 py-2 leading-relaxed text-[var(--surface-danger-text)]", focusRing),
-					children: output.stderr
+				output.stderr ? /* @__PURE__ */ jsx(OutputRegion, {
+					label: "stderr",
+					text: output.stderr,
+					className: "border-t border-white/10 text-[var(--surface-danger-text)]"
 				}) : null,
-				errorText ? /* @__PURE__ */ jsx("pre", {
-					tabIndex: 0,
-					role: "region",
-					"aria-label": "error",
-					className: cn("max-h-80 overflow-auto whitespace-pre-wrap [overflow-wrap:anywhere] border-t border-white/10 px-3 py-2 leading-relaxed text-[var(--surface-danger-text)]", focusRing),
-					children: errorText
+				errorText ? /* @__PURE__ */ jsx(OutputRegion, {
+					label: "error",
+					text: errorText,
+					className: "border-t border-white/10 text-[var(--surface-danger-text)]"
 				}) : null
 			]
 		}) : null]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tangle-network/ui",
-  "version": "11.16.10",
+  "version": "11.16.11",
   "description": "Generic React UI components for Tangle products — primitives, chat, run, files, editor, markdown.",
   "type": "module",
   "license": "MIT",

package/src/tool-previews/command-preview.test.tsx

@@ -80,3 +80,21 @@ describe("CommandPreview", () => {
     expect(screen.getByRole("region", { name: "stdout" })).toHaveTextContent("M a");
   });
 });
+
+describe("CommandPreview output", () => {
+  it("offers the rest of an output its height cap hides", () => {
+    const scroll = Object.getOwnPropertyDescriptor(HTMLElement.prototype, "scrollHeight");
+    const client = Object.getOwnPropertyDescriptor(HTMLElement.prototype, "clientHeight");
+    Object.defineProperty(HTMLElement.prototype, "scrollHeight", { configurable: true, get: () => 900 });
+    Object.defineProperty(HTMLElement.prototype, "clientHeight", { configurable: true, get: () => 320 });
+    try {
+      render(<CommandPreview part={part({ output: "a\nb\nc" })} defaultExpanded />);
+      fireEvent.click(screen.getByRole("button", { name: "Show all stdout (3 lines)" }));
+      expect(screen.getByRole("region", { name: "stdout" }).className).not.toContain("max-h-80");
+      expect(screen.queryByRole("button", { name: /Show all stdout/ })).toBeNull();
+    } finally {
+      if (scroll) Object.defineProperty(HTMLElement.prototype, "scrollHeight", scroll);
+      if (client) Object.defineProperty(HTMLElement.prototype, "clientHeight", client);
+    }
+  });
+});

package/src/tool-previews/command-preview.tsx

@@ -1,4 +1,4 @@
-import { memo, useId, useState } from "react";
+import { memo, useId, useLayoutEffect, useRef, useState } from "react";
 import { ChevronDown, ChevronRight, Loader2 } from "lucide-react";
 import { focusRing } from "../lib/focus";
 import { cn } from "../lib/utils";
@@ -205,6 +205,65 @@ function CommandText({ command }: { command: string }) {
   );
 }
 
+/**
+ * One output stream, capped at a readable height. When the cap hides lines,
+ * a control says so and shows the rest in place: a region that ends flush
+ * with its frame gave no sign that more output was there.
+ */
+function OutputRegion({
+  label,
+  text,
+  className,
+}: {
+  label: string;
+  text: string;
+  className?: string;
+}) {
+  const ref = useRef<HTMLPreElement>(null);
+  const [clipped, setClipped] = useState(false);
+  const [full, setFull] = useState(false);
+  useLayoutEffect(() => {
+    const el = ref.current;
+    if (!el || full) return;
+    const check = () => setClipped(el.scrollHeight > el.clientHeight + 1);
+    check();
+    if (typeof ResizeObserver === "undefined") return;
+    const observer = new ResizeObserver(check);
+    observer.observe(el);
+    return () => observer.disconnect();
+  }, [full]);
+  const lines = text.replace(/\n$/, "").split("\n").length;
+  return (
+    <div className={className}>
+      <pre
+        ref={ref}
+        tabIndex={0}
+        role="region"
+        aria-label={label}
+        className={cn(
+          "overflow-auto whitespace-pre-wrap [overflow-wrap:anywhere] px-3 py-2 leading-relaxed",
+          full ? "" : "max-h-80",
+          focusRing,
+        )}
+      >
+        {text}
+      </pre>
+      {clipped && !full ? (
+        <button
+          type="button"
+          onClick={() => setFull(true)}
+          className={cn(
+            "w-full border-t border-white/10 px-3 py-1.5 text-left font-sans text-[11px] font-medium text-muted-foreground transition-colors hover:text-foreground",
+            focusRing,
+          )}
+        >
+          Show all {label} ({lines} lines)
+        </button>
+      ) : null}
+    </div>
+  );
+}
+
 /**
  * One shell command as a terminal block: a `$ command` prompt line with its
  * exit status, and stdout and stderr in separate regions below it.
@@ -305,43 +364,25 @@ export const CommandPreview = memo(
         {expanded && hasBody ? (
           <div id={bodyId} className="border-t border-white/10">
             {output.stdout ? (
-              <pre
-                tabIndex={0}
-                role="region"
-                aria-label="stdout"
-                className={cn(
-                  "max-h-80 overflow-auto whitespace-pre-wrap [overflow-wrap:anywhere] px-3 py-2 leading-relaxed",
-                  focusRing,
-                )}
-              >
-                {output.stdout}
-              </pre>
+              <OutputRegion
+                label="stdout"
+                text={output.stdout}
+                className=""
+              />
             ) : null}
             {output.stderr ? (
-              <pre
-                tabIndex={0}
-                role="region"
-                aria-label="stderr"
-                className={cn(
-                  "max-h-80 overflow-auto whitespace-pre-wrap [overflow-wrap:anywhere] border-t border-white/10 px-3 py-2 leading-relaxed text-[var(--surface-danger-text)]",
-                  focusRing,
-                )}
-              >
-                {output.stderr}
-              </pre>
+              <OutputRegion
+                label="stderr"
+                text={output.stderr}
+                className="border-t border-white/10 text-[var(--surface-danger-text)]"
+              />
             ) : null}
             {errorText ? (
-              <pre
-                tabIndex={0}
-                role="region"
-                aria-label="error"
-                className={cn(
-                  "max-h-80 overflow-auto whitespace-pre-wrap [overflow-wrap:anywhere] border-t border-white/10 px-3 py-2 leading-relaxed text-[var(--surface-danger-text)]",
-                  focusRing,
-                )}
-              >
-                {errorText}
-              </pre>
+              <OutputRegion
+                label="error"
+                text={errorText}
+                className="border-t border-white/10 text-[var(--surface-danger-text)]"
+              />
             ) : null}
           </div>
         ) : null}