@llblab/pi-actors 0.24.2 → 0.24.4

package/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 ## Unreleased
 
+## 0.24.4: Music Player Message Control Hotfix
+
+- `[Music Player]` Preserve typed actor-message envelopes for run-mailbox delivery so `player.<action>` messages dispatch by message type instead of relying on body payload text.
+- `[Music Player]` Emit `player.stopped` when playback stops so actor inspector and run messages show an explicit domain stop event before terminal completion.
+
+## 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`.

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

@@ -318,11 +318,6 @@ function propertyValue(value) {
         return String(value);
     return JSON.stringify(value);
 }
-function itemViewKeyLabel(key) {
-    if (key === "body_preview")
-        return "body";
-    return key;
-}
 function displayWidth(value) {
     return visibleWidth(value);
 }
@@ -507,24 +502,24 @@ export function renderInspectorItemView(previews, width = 80, styles = {}, optio
     const prefix = " ";
     const suffix = " ";
     const contentWidth = Math.max(8, safeWidth - prefix.length - suffix.length);
-    const orderedKeys = [
-        "channel",
-        "run",
-        "event_id",
-        "from",
-        "from_display",
-        "to",
-        "type",
-        "summary",
-        "body_preview",
-        "message_id",
-        "needs_response",
-        "timestamp",
-        "stripe",
+    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)));
@@ -536,13 +531,12 @@ export function renderInspectorItemView(previews, width = 80, styles = {}, optio
     const headerPadding = Math.max(0, contentWidth - visibleWidth(headerPlain));
     const lines = [`${prefix}${header}${" ".repeat(headerPadding)}${suffix}`, ""];
     for (const [key, value] of entries) {
-        const label = itemViewKeyLabel(key);
-        const keyPadding = " ".repeat(Math.max(0, keyWidth - displayWidth(label)));
+        const keyPadding = " ".repeat(Math.max(0, keyWidth - displayWidth(key)));
         const separator = "  ";
         const valueWidth = Math.max(0, contentWidth - keyWidth - separator.length);
         const visibleValue = boundedLine(value, valueWidth);
-        const plain = `${label}${keyPadding}${separator}${visibleValue}`;
-        const rendered = `${style(styles.muted, label)}${keyPadding}${separator}${style(styles.preview, visibleValue)}`;
+        const plain = `${key}${keyPadding}${separator}${visibleValue}`;
+        const rendered = `${style(styles.muted, key)}${keyPadding}${separator}${style(styles.preview, visibleValue)}`;
         const padding = Math.max(0, contentWidth - visibleWidth(plain));
         lines.push(`${prefix}${rendered}${" ".repeat(padding)}${suffix}`);
     }

package/dist/lib/tools.js

@@ -481,6 +481,8 @@ function getRunOwnerId(ctx) {
     return ctx.sessionManager?.getSessionId?.();
 }
 function messageBodyToRunLine(message) {
+    if (message.type !== "run.message")
+        return JSON.stringify(message);
     if (typeof message.body === "string")
         return message.body;
     if (message.body === undefined)

package/dist/recipes/music-player.json

@@ -31,7 +31,8 @@
       "player.status"
     ],
     "emits": [
-      "player.track"
+      "player.track",
+      "player.stopped"
     ]
   },
   "template": "{repo}/scripts/music-player.mjs {command} {source} {loop} {volume} {player} {state_dir}"

package/dist/scripts/music-player.mjs

@@ -404,26 +404,39 @@ function readText(path) {
   }
 }
 
-function emitTrackEvent(ctx, index, count, track, player) {
-  const title = track.split(/[\\/]/).filter(Boolean).pop() || track;
+function emitPlayerEvent(ctx, type, summary, body = {}) {
   writeText(
     ctx.eventFile,
     `${JSON.stringify({
-      body: { count, index, player, track },
-      data: { count, index, player, track },
+      body,
+      data: body,
       delivery: "log",
-      event: "player.track",
+      event: type,
       from: `run:${basename(ctx.stateDir)}`,
       level: "info",
-      summary: `Now playing: ${title}`,
+      summary,
       to: "coordinator",
       ts: new Date().toISOString(),
-      type: "player.track",
+      type,
     })}\n`,
     "a",
   );
 }
 
+function emitTrackEvent(ctx, index, count, track, player) {
+  const title = track.split(/[\\/]/).filter(Boolean).pop() || track;
+  emitPlayerEvent(ctx, "player.track", `Now playing: ${title}`, {
+    count,
+    index,
+    player,
+    track,
+  });
+}
+
+function emitStoppedEvent(ctx, reason = "stop") {
+  emitPlayerEvent(ctx, "player.stopped", "Music player stopped", { reason });
+}
+
 function writeStatus(ctx, state, index, count, track, player, pid = "") {
   const updatedAt = new Date().toISOString();
   writeText(
@@ -611,23 +624,27 @@ function writeInboxMessages(ctx, messages) {
   );
 }
 
+function commandFromControlToken(token) {
+  if (CONTROL_COMMANDS.has(token)) return token;
+  if (token === "control.stop" || token === "control.cancel") return "stop";
+  const match = /^player\.(.+)$/.exec(token);
+  if (match && CONTROL_COMMANDS.has(match[1])) return match[1];
+  return undefined;
+}
+
 function commandFromInboxMessage(message) {
-  if (
-    typeof message.body === "string" &&
-    CONTROL_COMMANDS.has(message.body.trim())
-  ) {
-    return message.body.trim();
+  if (typeof message.type === "string") {
+    const command = commandFromControlToken(message.type.trim());
+    if (command) return command;
+  }
+  if (typeof message.body === "string") {
+    const command = message.body.trim();
+    if (CONTROL_COMMANDS.has(command)) return command;
   }
   if (message.body && typeof message.body === "object") {
     const command = String(message.body.command ?? "").trim();
     if (CONTROL_COMMANDS.has(command)) return command;
   }
-  if (typeof message.type === "string") {
-    if (message.type === "control.stop" || message.type === "control.cancel")
-      return "stop";
-    const match = /^player\.(.+)$/.exec(message.type);
-    if (match && CONTROL_COMMANDS.has(match[1])) return match[1];
-  }
   return undefined;
 }
 
@@ -895,6 +912,7 @@ async function playMain(args) {
       const command = readAndClearCommand(ctx);
       if (command === "stop") {
         writeStatus(ctx, "stopped", index, count, track, player, "");
+        emitStoppedEvent(ctx, "message");
         return;
       }
       if (command === "previous" || command === "prev") {
@@ -913,6 +931,7 @@ async function playMain(args) {
       }
     }
     writeStatus(ctx, "stopped", index, tracks.length, "", player, "");
+    emitStoppedEvent(ctx, "complete");
   } finally {
     cleanup();
     await Promise.race([controlLoop?.promise ?? Promise.resolve(), sleep(100)]);

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.2
+  version: 0.24.4
 ---
 
 # 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.2
+  version: 0.24.4
 ---
 
 # Swarm

package/lib/actor-inspector-tui.ts

@@ -478,11 +478,6 @@ function propertyValue(value: unknown): string {
   return JSON.stringify(value);
 }
 
-function itemViewKeyLabel(key: string): string {
-  if (key === "body_preview") return "body";
-  return key;
-}
-
 function displayWidth(value: string): number {
   return visibleWidth(value);
 }
@@ -735,24 +730,24 @@ export function renderInspectorItemView(
   const prefix = " ";
   const suffix = " ";
   const contentWidth = Math.max(8, safeWidth - prefix.length - suffix.length);
-  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 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(
@@ -769,13 +764,12 @@ export function renderInspectorItemView(
   const headerPadding = Math.max(0, contentWidth - visibleWidth(headerPlain));
   const lines = [`${prefix}${header}${" ".repeat(headerPadding)}${suffix}`, ""];
   for (const [key, value] of entries) {
-    const label = itemViewKeyLabel(key);
-    const keyPadding = " ".repeat(Math.max(0, keyWidth - displayWidth(label)));
+    const keyPadding = " ".repeat(Math.max(0, keyWidth - displayWidth(key)));
     const separator = "  ";
     const valueWidth = Math.max(0, contentWidth - keyWidth - separator.length);
     const visibleValue = boundedLine(value, valueWidth);
-    const plain = `${label}${keyPadding}${separator}${visibleValue}`;
-    const rendered = `${style(styles.muted, label)}${keyPadding}${separator}${style(styles.preview, visibleValue)}`;
+    const plain = `${key}${keyPadding}${separator}${visibleValue}`;
+    const rendered = `${style(styles.muted, key)}${keyPadding}${separator}${style(styles.preview, visibleValue)}`;
     const padding = Math.max(0, contentWidth - visibleWidth(plain));
     lines.push(`${prefix}${rendered}${" ".repeat(padding)}${suffix}`);
   }

package/lib/tools.ts

@@ -593,6 +593,7 @@ function getRunOwnerId(ctx: AsyncRunToolContext): string | undefined {
 }
 
 function messageBodyToRunLine(message: ActorMessages.ActorMessage): string {
+  if (message.type !== "run.message") return JSON.stringify(message);
   if (typeof message.body === "string") return message.body;
   if (message.body === undefined) return message.type;
   return JSON.stringify(message.body);

package/package.json

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

package/recipes/music-player.json

@@ -31,7 +31,8 @@
       "player.status"
     ],
     "emits": [
-      "player.track"
+      "player.track",
+      "player.stopped"
     ]
   },
   "template": "{repo}/scripts/music-player.mjs {command} {source} {loop} {volume} {player} {state_dir}"

package/scripts/music-player.mjs

@@ -404,26 +404,39 @@ function readText(path) {
   }
 }
 
-function emitTrackEvent(ctx, index, count, track, player) {
-  const title = track.split(/[\\/]/).filter(Boolean).pop() || track;
+function emitPlayerEvent(ctx, type, summary, body = {}) {
   writeText(
     ctx.eventFile,
     `${JSON.stringify({
-      body: { count, index, player, track },
-      data: { count, index, player, track },
+      body,
+      data: body,
       delivery: "log",
-      event: "player.track",
+      event: type,
       from: `run:${basename(ctx.stateDir)}`,
       level: "info",
-      summary: `Now playing: ${title}`,
+      summary,
       to: "coordinator",
       ts: new Date().toISOString(),
-      type: "player.track",
+      type,
     })}\n`,
     "a",
   );
 }
 
+function emitTrackEvent(ctx, index, count, track, player) {
+  const title = track.split(/[\\/]/).filter(Boolean).pop() || track;
+  emitPlayerEvent(ctx, "player.track", `Now playing: ${title}`, {
+    count,
+    index,
+    player,
+    track,
+  });
+}
+
+function emitStoppedEvent(ctx, reason = "stop") {
+  emitPlayerEvent(ctx, "player.stopped", "Music player stopped", { reason });
+}
+
 function writeStatus(ctx, state, index, count, track, player, pid = "") {
   const updatedAt = new Date().toISOString();
   writeText(
@@ -611,23 +624,27 @@ function writeInboxMessages(ctx, messages) {
   );
 }
 
+function commandFromControlToken(token) {
+  if (CONTROL_COMMANDS.has(token)) return token;
+  if (token === "control.stop" || token === "control.cancel") return "stop";
+  const match = /^player\.(.+)$/.exec(token);
+  if (match && CONTROL_COMMANDS.has(match[1])) return match[1];
+  return undefined;
+}
+
 function commandFromInboxMessage(message) {
-  if (
-    typeof message.body === "string" &&
-    CONTROL_COMMANDS.has(message.body.trim())
-  ) {
-    return message.body.trim();
+  if (typeof message.type === "string") {
+    const command = commandFromControlToken(message.type.trim());
+    if (command) return command;
+  }
+  if (typeof message.body === "string") {
+    const command = message.body.trim();
+    if (CONTROL_COMMANDS.has(command)) return command;
   }
   if (message.body && typeof message.body === "object") {
     const command = String(message.body.command ?? "").trim();
     if (CONTROL_COMMANDS.has(command)) return command;
   }
-  if (typeof message.type === "string") {
-    if (message.type === "control.stop" || message.type === "control.cancel")
-      return "stop";
-    const match = /^player\.(.+)$/.exec(message.type);
-    if (match && CONTROL_COMMANDS.has(match[1])) return match[1];
-  }
   return undefined;
 }
 
@@ -895,6 +912,7 @@ async function playMain(args) {
       const command = readAndClearCommand(ctx);
       if (command === "stop") {
         writeStatus(ctx, "stopped", index, count, track, player, "");
+        emitStoppedEvent(ctx, "message");
         return;
       }
       if (command === "previous" || command === "prev") {
@@ -913,6 +931,7 @@ async function playMain(args) {
       }
     }
     writeStatus(ctx, "stopped", index, tracks.length, "", player, "");
+    emitStoppedEvent(ctx, "complete");
   } finally {
     cleanup();
     await Promise.race([controlLoop?.promise ?? Promise.resolve(), sleep(100)]);

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.2
+  version: 0.24.4
 ---
 
 # 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.2
+  version: 0.24.4
 ---
 
 # Swarm