@mercury-fw/cli-engine 0.69.0 → 0.71.0

package/CHANGELOG.md

@@ -1,5 +1,32 @@
 # @mercury-fw/cli-engine
 
+## 0.71.0
+
+### Minor Changes
+
+- 2b08f7e: - `formatterPlugin` renders the `display` of every tool a plugin contributes, typed tools included (`{ ok: true, …, display }`)
+  - formatter rule `{ item, empty?, always: true }`: that kind is shown after the answer without `present`, the model gets `shown: true` next to its `displayRef`
+  - `ToolDisplay.always`
+  - the core strips `display` from every plugin tool's result and stashes it, a typed tool's too
+  - stashed list labels: the tool name, its input and the item count
+  - a failed or cancelled turn's displays are no longer appended to the next answer
+  - `present` is offered with any plugin tool
+  - **Breaking:** `createCliTool` no longer takes `stashDisplay` and returns the post-processor's `display` as it is: drop the option
+  - **Breaking:** a hand-written `DisplayHandler` returns `{ text, always? }` instead of a string: wrap the text
+
+### Patch Changes
+
+- Updated dependencies [2b08f7e]
+  - @mercury-fw/plugin-types@0.71.0
+  - @mercury-fw/utils@0.71.0
+
+## 0.70.0
+
+### Patch Changes
+
+- @mercury-fw/plugin-types@0.70.0
+- @mercury-fw/utils@0.70.0
+
 ## 0.69.0
 
 ### Patch Changes

package/cli-tool.ts

@@ -170,11 +170,6 @@ export function createCliTool(
     postProcess?: CliPostProcessor;
     /** The texts of the turn's person (`SessionToolContext.t`), handed to the post-processor. */
     t: Translate;
-    /** Stashes a post-processor's rendered `display` artifact and returns a ref
-     * the model can `present`. Pre-bound to this turn's session by the
-     * composition root. Optional: with none wired the inline `display` is left
-     * untouched and no `displayRef` is minted. */
-    stashDisplay?: (artifact: string, label?: string) => string;
     /** The person the turn is for (see `SessionToolContext.person`): every
      * command runs as them, with `--user` and the id their CLI knows them by.
      * Absent or `null`: the CLI's own service identity. */
@@ -266,21 +261,8 @@ export function createCliTool(
         ? opts.postProcess({ binary: parsed.binary, args: parsed.args, prefix: match.prefix }, result, opts.t)
         : result;
 
-      // A rendered display artifact is stashed, not returned to be
-      // force-appended: the model gets only a `displayRef` and decides whether
-      // to `present` it. Only string items (what the formatter renders) are
-      // showable; a display still carrying structured items (no formatter
-      // applied) has nothing to stash. With no `stashDisplay` wired, the result
-      // is left exactly as-is.
-      if (opts.stashDisplay && processed.ok && processed.display) {
-        const stringItems = processed.display.items.filter((i): i is string => typeof i === "string");
-        if (stringItems.length > 0) {
-          // What the list is, so the model still knows a turn later.
-          const count = `${stringItems.length} ${stringItems.length === 1 ? "item" : "items"}`;
-          const displayRef = opts.stashDisplay(stringItems.join("\n\n"), `${command.trim()} (${count})`);
-          return { ...processed, displayRef };
-        }
-      }
+      // The display stays on the result: the core's display channel stashes
+      // it and hands the model a ref, for every plugin tool alike.
       return processed;
     },
     toModelOutput: ({ output }) => ({ type: "json", value: omitLoginLinkForModel(omitDisplayForModel(output)) as JSONValue }),

package/dist/cli-tool.d.ts

@@ -99,11 +99,6 @@ export declare function createCliTool(runCliFn: typeof runCli, configs: Record<s
     postProcess?: CliPostProcessor;
     /** The texts of the turn's person (`SessionToolContext.t`), handed to the post-processor. */
     t: Translate;
-    /** Stashes a post-processor's rendered `display` artifact and returns a ref
-     * the model can `present`. Pre-bound to this turn's session by the
-     * composition root. Optional: with none wired the inline `display` is left
-     * untouched and no `displayRef` is minted. */
-    stashDisplay?: (artifact: string, label?: string) => string;
     /** The person the turn is for (see `SessionToolContext.person`): every
      * command runs as them, with `--user` and the id their CLI knows them by.
      * Absent or `null`: the CLI's own service identity. */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mercury-fw/cli-engine",
-  "version": "0.69.0",
+  "version": "0.71.0",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -23,8 +23,8 @@
     }
   },
   "dependencies": {
-    "@mercury-fw/plugin-types": "0.69.0",
-    "@mercury-fw/utils": "0.69.0",
+    "@mercury-fw/plugin-types": "0.71.0",
+    "@mercury-fw/utils": "0.71.0",
     "ai": "^7.0.126",
     "shell-quote": "^1.11.0",
     "zod": "^4.6.5"