@llblab/pi-actors 0.24.1 → 0.24.3

package/BACKLOG.md

@@ -208,6 +208,25 @@ The backlog is intentionally pruned to the 20% of work most likely to deliver 80
   - Installed-package tests cover every script shim that imports compiled domain logic.
   - Pack dry assertions cover `dist/scripts`, `dist/recipes`, `dist/fixtures`, and `dist/skills`.
 
+### M-11 Actor Termination Semantics
+
+- Priority: Medium.
+- Status: Open.
+- Goal: Make `control.kill` the canonical parent-to-actor termination action while keeping `control.stop` and `control.cancel` as actor-domain messages whose meaning depends on the actor protocol.
+- Why now: Mailbox workers need a clearer lifecycle boundary before v2 patterns harden. Treating `stop`, `cancel`, and `kill` as equivalent stop messages blurs actor termination with domain-specific task or playback control.
+- Direction:
+  - Document `control.kill` as the universal lifecycle action for a parent/supervisor terminating an actor or run.
+  - Reframe `control.stop` as actor-defined domain control, such as stopping music playback or ending an actor-specific loop when that actor declares it.
+  - Reframe `control.cancel` as actor-defined domain control, such as cancelling the current subagent task while keeping the worker actor alive for later assignments.
+  - Split mailbox-loop helper semantics so lifecycle termination detection is distinct from general control-message detection.
+  - While the package is pre-1.0, allow a small intentional minor-version contract break: remove legacy treatment that aliases `control.stop` or `control.cancel` to actor termination.
+  - Do not preserve compatibility shims for the old stop/cancel-as-termination behavior before the first 1.0 major release unless a concrete safety issue appears during implementation.
+- Acceptance:
+  - Docs and actors skill advertise `control.kill` as canonical parent-to-actor termination.
+  - Mailbox-loop helpers/tests distinguish actor termination from actor-domain `stop`/`cancel` handling.
+  - Packaged recipes declare `stop`/`cancel` only when the actor-specific behavior is meaningful.
+  - Tests assert that generic mailbox-loop termination is not triggered by `control.stop` or `control.cancel`.
+
 ## Explicitly Deferred
 
 These are valid ideas but not current focus. Reintroduce only with concrete evidence from real actor workflows.
@@ -226,6 +245,6 @@ These are valid ideas but not current focus. Reintroduce only with concrete evid
 0.25 — Operator remediation and worker maturity:
   M-08, M-09
 
-0.26 — Package contract hardening:
-  M-10
+0.26 — Package contract hardening and lifecycle semantics:
+  M-10, M-11
 ```

package/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 ## Unreleased
 
+## 0.24.3: Actor Inspector Body Field Hotfix
+
+- `[Inspector]` Removed the selected-item key-label shim by projecting actor-message detail rows with a real `body` field instead of rendering `body_preview` under an alias.
+
+## 0.24.2: Actor Inspector Item View Hotfix
+
+- `[Inspector]` Removed the roster panel from selected actor-message item inspection, kept the compact route header styling, added the same left/right gutter used by inspector rows, and renamed the selected-item `body_preview` label to `body`.
+
 ## 0.24.1: Build Script Package Hygiene Hotfix
 
 - `[Backlog]` Added the next focused backlog set for recipe doctor remediation UX, actor worker v2, and dist package contract hardening.

package/dist/index.js

@@ -94,7 +94,7 @@ export default function toolRegistryExtension(pi) {
                             ? ActorInspectorTui.renderInspectorItemView(previews, width, style, { sequence: selectedInspectorSequence })
                             : ActorInspectorTui.renderInspectorWidget(previews, width, style)) ?? [];
                         const run = previews[0]?.run;
-                        const roster = run
+                        const roster = selectedInspectorSequence === undefined && run
                             ? ActorInspectorTui.renderInspectorRosterPanel(ActorInspectorTui.readActorInspectorRoster(RUN_STATE_ROOT, run), width, style)
                             : undefined;
                         return roster ? [...roster, ...rows] : rows;

package/dist/lib/actor-inspector-tui.js

@@ -499,43 +499,46 @@ export function renderInspectorItemView(previews, width = 80, styles = {}, optio
     if (!preview)
         return undefined;
     const safeWidth = Math.max(1, width);
-    const orderedKeys = [
-        "channel",
-        "run",
-        "event_id",
-        "from",
-        "from_display",
-        "to",
-        "type",
-        "summary",
-        "body_preview",
-        "message_id",
-        "needs_response",
-        "timestamp",
-        "stripe",
+    const prefix = " ";
+    const suffix = " ";
+    const contentWidth = Math.max(8, safeWidth - prefix.length - suffix.length);
+    const rawEntries = [
+        ["channel", preview.channel],
+        ["run", preview.run],
+        ["event_id", preview.event_id],
+        ["from", preview.from],
+        ["from_display", preview.from_display],
+        ["to", preview.to],
+        ["type", preview.type],
+        ["summary", preview.summary],
+        ["body", preview.body_preview],
+        ["message_id", preview.message_id],
+        ["needs_response", preview.needs_response],
+        ["timestamp", preview.timestamp],
+        ["stripe", preview.stripe],
     ];
-    const entries = orderedKeys
-        .filter((key) => preview[key] !== undefined)
-        .map((key) => [key, propertyValue(preview[key])]);
+    const entries = rawEntries
+        .filter((entry) => entry[1] !== undefined)
+        .map(([key, value]) => [key, propertyValue(value)]);
     const keyWidth = Math.max(1, ...entries.map(([key]) => displayWidth(key)));
     const sequenceText = String(preview.sequence ?? options.sequence);
     const sequencePadding = " ".repeat(Math.max(0, keyWidth - displayWidth(sequenceText)));
     const headerSeparator = "  ";
     const route = routeText(preview);
-    const visibleRoute = boundedLine(route, Math.max(0, safeWidth - keyWidth - headerSeparator.length));
+    const visibleRoute = boundedLine(route, Math.max(0, contentWidth - keyWidth - headerSeparator.length));
     const headerPlain = `${sequenceText}${sequencePadding}${headerSeparator}${visibleRoute}`;
     const header = `${style(styles.muted, sequenceText)}${sequencePadding}${headerSeparator}${style(styles.target, visibleRoute)}`;
-    const headerPadding = Math.max(0, safeWidth - visibleWidth(headerPlain));
-    const lines = [`${header}${" ".repeat(headerPadding)}`, ""];
+    const headerPadding = Math.max(0, contentWidth - visibleWidth(headerPlain));
+    const lines = [`${prefix}${header}${" ".repeat(headerPadding)}${suffix}`, ""];
     for (const [key, value] of entries) {
         const keyPadding = " ".repeat(Math.max(0, keyWidth - displayWidth(key)));
         const separator = "  ";
-        const valueWidth = Math.max(0, safeWidth - keyWidth - separator.length);
+        const valueWidth = Math.max(0, contentWidth - keyWidth - separator.length);
         const visibleValue = boundedLine(value, valueWidth);
         const plain = `${key}${keyPadding}${separator}${visibleValue}`;
         const rendered = `${style(styles.muted, key)}${keyPadding}${separator}${style(styles.preview, visibleValue)}`;
-        const padding = Math.max(0, safeWidth - visibleWidth(plain));
-        lines.push(`${rendered}${" ".repeat(padding)}`);
+        const padding = Math.max(0, contentWidth - visibleWidth(plain));
+        lines.push(`${prefix}${rendered}${" ".repeat(padding)}${suffix}`);
     }
     return lines;
 }

package/dist/skills/actors/SKILL.md

@@ -2,7 +2,7 @@
 name: actors
 description: Highest-density practical guide for pi-actors. Read this skill whenever prompt and tools are not enough for spawn, message, inspect, actor runs, tools, recipes, command templates, async lifecycle, mailboxes, artifacts, and local orchestration mechanics.
 metadata:
-  version: 0.24.1
+  version: 0.24.3
 ---
 
 # Actors (pi-actors)

package/dist/skills/swarm/SKILL.md

@@ -2,7 +2,7 @@
 name: swarm
 description: Subagent orchestration with scoped locks and quorum consensus. Use for multi-model review, parallel scoped work, delegated audit, and coordinated subagent execution.
 metadata:
-  version: 0.24.1
+  version: 0.24.3
 ---
 
 # Swarm

package/index.ts

@@ -129,16 +129,17 @@ export default function toolRegistryExtension(pi: ExtensionAPI) {
                         style,
                       )) ?? [];
                 const run = previews[0]?.run;
-                const roster = run
-                  ? ActorInspectorTui.renderInspectorRosterPanel(
-                      ActorInspectorTui.readActorInspectorRoster(
-                        RUN_STATE_ROOT,
-                        run,
-                      ),
-                      width,
-                      style,
-                    )
-                  : undefined;
+                const roster =
+                  selectedInspectorSequence === undefined && run
+                    ? ActorInspectorTui.renderInspectorRosterPanel(
+                        ActorInspectorTui.readActorInspectorRoster(
+                          RUN_STATE_ROOT,
+                          run,
+                        ),
+                        width,
+                        style,
+                      )
+                    : undefined;
                 return roster ? [...roster, ...rows] : rows;
               },
             };

package/lib/actor-inspector-tui.ts

@@ -727,24 +727,27 @@ export function renderInspectorItemView(
   const preview = previews.find((item) => item.sequence === options.sequence);
   if (!preview) return undefined;
   const safeWidth = Math.max(1, width);
-  const orderedKeys = [
-    "channel",
-    "run",
-    "event_id",
-    "from",
-    "from_display",
-    "to",
-    "type",
-    "summary",
-    "body_preview",
-    "message_id",
-    "needs_response",
-    "timestamp",
-    "stripe",
-  ] as const;
-  const entries = orderedKeys
-    .filter((key) => preview[key] !== undefined)
-    .map((key) => [key, propertyValue(preview[key])] as const);
+  const prefix = " ";
+  const suffix = " ";
+  const contentWidth = Math.max(8, safeWidth - prefix.length - suffix.length);
+  const rawEntries: Array<readonly [string, unknown]> = [
+    ["channel", preview.channel],
+    ["run", preview.run],
+    ["event_id", preview.event_id],
+    ["from", preview.from],
+    ["from_display", preview.from_display],
+    ["to", preview.to],
+    ["type", preview.type],
+    ["summary", preview.summary],
+    ["body", preview.body_preview],
+    ["message_id", preview.message_id],
+    ["needs_response", preview.needs_response],
+    ["timestamp", preview.timestamp],
+    ["stripe", preview.stripe],
+  ];
+  const entries = rawEntries
+    .filter((entry) => entry[1] !== undefined)
+    .map(([key, value]) => [key, propertyValue(value)] as const);
   const keyWidth = Math.max(1, ...entries.map(([key]) => displayWidth(key)));
   const sequenceText = String(preview.sequence ?? options.sequence);
   const sequencePadding = " ".repeat(
@@ -754,21 +757,21 @@ export function renderInspectorItemView(
   const route = routeText(preview);
   const visibleRoute = boundedLine(
     route,
-    Math.max(0, safeWidth - keyWidth - headerSeparator.length),
+    Math.max(0, contentWidth - keyWidth - headerSeparator.length),
   );
   const headerPlain = `${sequenceText}${sequencePadding}${headerSeparator}${visibleRoute}`;
   const header = `${style(styles.muted, sequenceText)}${sequencePadding}${headerSeparator}${style(styles.target, visibleRoute)}`;
-  const headerPadding = Math.max(0, safeWidth - visibleWidth(headerPlain));
-  const lines = [`${header}${" ".repeat(headerPadding)}`, ""];
+  const headerPadding = Math.max(0, contentWidth - visibleWidth(headerPlain));
+  const lines = [`${prefix}${header}${" ".repeat(headerPadding)}${suffix}`, ""];
   for (const [key, value] of entries) {
     const keyPadding = " ".repeat(Math.max(0, keyWidth - displayWidth(key)));
     const separator = "  ";
-    const valueWidth = Math.max(0, safeWidth - keyWidth - separator.length);
+    const valueWidth = Math.max(0, contentWidth - keyWidth - separator.length);
     const visibleValue = boundedLine(value, valueWidth);
     const plain = `${key}${keyPadding}${separator}${visibleValue}`;
     const rendered = `${style(styles.muted, key)}${keyPadding}${separator}${style(styles.preview, visibleValue)}`;
-    const padding = Math.max(0, safeWidth - visibleWidth(plain));
-    lines.push(`${rendered}${" ".repeat(padding)}`);
+    const padding = Math.max(0, contentWidth - visibleWidth(plain));
+    lines.push(`${prefix}${rendered}${" ".repeat(padding)}${suffix}`);
   }
   return lines;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@llblab/pi-actors",
-  "version": "0.24.1",
+  "version": "0.24.3",
   "private": false,
   "description": "Local Actor Kernel for Pi",
   "keywords": [

package/skills/actors/SKILL.md

@@ -2,7 +2,7 @@
 name: actors
 description: Highest-density practical guide for pi-actors. Read this skill whenever prompt and tools are not enough for spawn, message, inspect, actor runs, tools, recipes, command templates, async lifecycle, mailboxes, artifacts, and local orchestration mechanics.
 metadata:
-  version: 0.24.1
+  version: 0.24.3
 ---
 
 # Actors (pi-actors)

package/skills/swarm/SKILL.md

@@ -2,7 +2,7 @@
 name: swarm
 description: Subagent orchestration with scoped locks and quorum consensus. Use for multi-model review, parallel scoped work, delegated audit, and coordinated subagent execution.
 metadata:
-  version: 0.24.1
+  version: 0.24.3
 ---
 
 # Swarm