@design.estate/dees-catalog 11.1.0 → 11.2.1

package/dist_bundle/bundle.js

@@ -168881,7 +168881,7 @@ var harnessTerminalResultSummary = /* @__PURE__ */ __name((callArg) => {
   const combined = `${stdout}${stdout && stderr ? "\n" : ""}${stderr}`;
   const lineCount = combined ? harnessStripAnsi(combined).split("\n").filter((line) => line.trim()).length : 0;
   if (lineCount > 0) parts.push(`${lineCount} ${lineCount === 1 ? "line" : "lines"}`);
-  else if (exitCode3 !== void 0) parts.push("no output");
+  else parts.push("no output");
   const lastLine = harnessLastMeaningfulLine(stderr) || harnessLastMeaningfulLine(stdout);
   const prefix = parts.join(" \xB7 ");
   if (!lastLine) return prefix;
@@ -168925,12 +168925,16 @@ var defaultDescriptors = [
     defaultExpanded: true,
     layout: "full-row",
     /*
-     * Only a command that can be PROVEN to have succeeded collapses. An absent exit code is not
-     * evidence of success: a producer that reports a failed command as completed and encodes the
-     * failure in its output would otherwise have that failure hidden behind a one-line summary.
-     * Such a card stays open, and the producer reporting an exit code is the real fix.
+     * A finished command collapses unless it reports a failure. A missing exit code is not a
+     * failure — most harnesses never send one — so requiring proof of success kept nearly every
+     * terminal card open and left whole transcripts expanded. A non-zero exit keeps its output on
+     * screen; an error or a stopped command never reaches this rule, because the card keeps those
+     * open on status alone. The user's own expand or collapse still wins over all of it.
      */
-    collapseWhenFinished: /* @__PURE__ */ __name((call) => harnessTerminalOutcome(call).exitCode === 0, "collapseWhenFinished"),
+    collapseWhenFinished: /* @__PURE__ */ __name((call) => {
+      const { exitCode: exitCode3 } = harnessTerminalOutcome(call);
+      return exitCode3 === void 0 || exitCode3 === 0;
+    }, "collapseWhenFinished"),
     summary: harnessTerminalResultSummary
   },
   {
@@ -169797,8 +169801,8 @@ var toolShowcase = /* @__PURE__ */ __name(() => {
       text: "Re-ran the suite to confirm the cache key fix.",
       createdAt: now3 - 14e4
     },
-    // A terminal card alone in its row: expanded it takes the pane, and once it collapses it
-    // returns to the reading measure like the prose around it.
+    // A finished command that never reported an exit code: it collapses like any other clean
+    // finish, alone in its row, so it shows the collapsed reading measure.
     {
       id: "demo-t5",
       role: "tool",
@@ -169809,10 +169813,32 @@ var toolShowcase = /* @__PURE__ */ __name(() => {
         name: "run_command",
         status: "completed",
         input: { command: "pnpm test --filter sitemap", cwd: "/srv/app" },
-        output: { exitCode: 0, stdout: "14 tests passed\n0 failed" },
+        output: "14 tests passed\n0 failed",
         startedAt: now3 - 136e3,
         finishedAt: now3 - 131e3
       }
+    },
+    {
+      id: "demo-t6",
+      role: "assistant",
+      text: "The lint run is still failing though.",
+      createdAt: now3 - 12e4
+    },
+    // The other half of the rule: a non-zero exit keeps its output on screen and the pane with it.
+    {
+      id: "demo-t7",
+      role: "tool",
+      text: "",
+      createdAt: now3 - 11e4,
+      toolCall: {
+        id: "call-7",
+        name: "run_command",
+        status: "completed",
+        input: { command: "pnpm lint", cwd: "/srv/app" },
+        output: { exitCode: 1, stdout: "", stderr: "sitemap.job.ts:12:3  error  Unexpected any" },
+        startedAt: now3 - 116e3,
+        finishedAt: now3 - 111e3
+      }
     }
   ];
 }, "toolShowcase");
@@ -176517,13 +176543,15 @@ __publicField(_DeesHarnessMessageList, "styles", [
 
       /*
        * The pane is what an open command or diff needs, not what its one-line collapsed header
-       * needs, so a closed card returns to the reading measure inside its row. Only a row holding
-       * a single card offers it: in a multi-card grid the cell width is the layout, and shrinking
-       * one card would break the row. The card applies this itself — it alone knows whether it is
-       * open — and the measure stays defined here, with the rest of the column rules.
+       * needs, so a closed card returns to the reading measure inside its row. Every full-row card
+       * qualifies, however many share the group: they span all columns (see the grid-column rule
+       * below) and stack vertically, so shrinking one never disturbs a neighbour. The side-by-side
+       * grids are the compact and subagent ones, and neither is offered this width. The card
+       * applies it itself — it alone knows whether it is open — and the measure stays defined
+       * here, with the rest of the column rules.
        */
-      .stack > [data-tool-layout='full-row'][data-tool-count='1'],
-      .virtualItem[data-tool-layout='full-row'][data-tool-count='1'] {
+      .stack > [data-tool-layout='full-row'],
+      .virtualItem[data-tool-layout='full-row'] {
         --dees-harness-collapsed-tool-width: min(var(--dees-harness-content-width, 820px), 100%);
       }
 
@@ -176584,8 +176612,13 @@ __publicField(_DeesHarnessMessageList, "styles", [
         }
       }
 
+      /*
+       * The activity line is the tail of the transcript, not chrome around it, so it sits on the
+       * same measure as the prose rows and starts at their left edge instead of spanning the pane.
+       */
       .statusRow {
-        padding: 0 var(--dees-spacing-xs);
+        width: min(var(--dees-harness-content-width, 820px), 100%);
+        margin-inline: auto;
       }
 
       .empty {
@@ -223811,7 +223844,7 @@ init_group_runtime();
 // ts_web/00_commitinfo_data.ts
 var commitinfo = {
   name: "@design.estate/dees-catalog",
-  version: "11.1.0",
+  version: "11.2.1",
   description: "A comprehensive library that provides dynamic web components for building sophisticated and modern web applications using JavaScript and TypeScript."
 };