@llblab/pi-actors 0.24.3 → 0.24.5

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## Unreleased
 
+## 0.24.5: Auto-Discovered Skills Hotfix
+
+- `[Skills]` Auto-discovered source checkouts now contribute their co-located `skills/` directory through Pi resource discovery, so extension-local actors/swarm skills load even when `pi-actors` is used directly from `~/.pi/agent/extensions` instead of installed as a package.
+- `[Packaging]` Kept package skill metadata directory-based for both `dist/skills` and source `skills`, preserving bundled skill discovery through package installs while the extension also contributes co-located skills at runtime.
+
+## 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.

package/README.md

@@ -43,7 +43,7 @@ Or from git:
 pi install git:github.com/llblab/pi-actors
 ```
 
-The npm package is dist-first for JavaScript-only runtimes: default Pi metadata points at compiled `dist/` entrypoints and mirrored runtime assets. Source TypeScript and source skills remain in the package for TypeScript-native runtimes through optional source metadata.
+The npm package is dist-first for JavaScript-only runtimes: default Pi metadata points at compiled `dist/` entrypoints and mirrored runtime assets. Source TypeScript and source skills remain in the package for TypeScript-native runtimes through optional source metadata. When the source checkout is auto-discovered as an extension, it also contributes its co-located `skills/` directory during Pi resource discovery so the actors and swarm skills travel with the extension.
 
 ## Address Surface
 

package/dist/index.js

@@ -5,6 +5,8 @@
  * Wraps command templates as callable pi tools, stores durable user tools as recipe files, and exposes actor orchestration across reloads and sessions.
  */
 import { existsSync, readdirSync, watch } from "node:fs";
+import { dirname, join } from "node:path";
+import { fileURLToPath } from "node:url";
 import * as ActorInspectorTui from "./lib/actor-inspector-tui.js";
 import * as AsyncRuns from "./lib/async-runs.js";
 import * as CommandTemplates from "./lib/command-templates.js";
@@ -14,6 +16,8 @@ import * as Prompts from "./lib/prompts.js";
 import * as Runtime from "./lib/runtime.js";
 import * as Temp from "./lib/temp.js";
 import * as Tools from "./lib/tools.js";
+const EXTENSION_DIR = dirname(fileURLToPath(import.meta.url));
+const EXTENSION_SKILLS_DIR = join(EXTENSION_DIR, "skills");
 const CONFIG_PATH = Paths.getConfigPath();
 const TEMP_DIR = Paths.getExtensionTmpDir();
 const RUN_STATE_ROOT = Paths.getRunStateRoot();
@@ -246,6 +250,11 @@ export default function toolRegistryExtension(pi) {
         reservedToolNames: RESERVED_TOOL_NAMES,
         setActiveTools: (toolNames) => pi.setActiveTools(toolNames),
     });
+    pi.on("resources_discover", async () => {
+        if (!existsSync(EXTENSION_SKILLS_DIR))
+            return;
+        return { skillPaths: [EXTENSION_SKILLS_DIR] };
+    });
     pi.on("session_start", async (_event, ctx) => {
         await Temp.prepareExtensionTempDir(TEMP_DIR);
         runtime.loadTools(ctx);

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.3
+  version: 0.24.5
 ---
 
 # 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.3
+  version: 0.24.5
 ---
 
 # Swarm

package/index.ts

@@ -6,6 +6,8 @@
  */
 
 import { existsSync, readdirSync, watch, type FSWatcher } from "node:fs";
+import { dirname, join } from "node:path";
+import { fileURLToPath } from "node:url";
 
 import type {
   ExtensionAPI,
@@ -22,6 +24,8 @@ import * as Runtime from "./lib/runtime.ts";
 import * as Temp from "./lib/temp.ts";
 import * as Tools from "./lib/tools.ts";
 
+const EXTENSION_DIR = dirname(fileURLToPath(import.meta.url));
+const EXTENSION_SKILLS_DIR = join(EXTENSION_DIR, "skills");
 const CONFIG_PATH = Paths.getConfigPath();
 const TEMP_DIR = Paths.getExtensionTmpDir();
 const RUN_STATE_ROOT = Paths.getRunStateRoot();
@@ -299,6 +303,10 @@ export default function toolRegistryExtension(pi: ExtensionAPI) {
     reservedToolNames: RESERVED_TOOL_NAMES,
     setActiveTools: (toolNames) => pi.setActiveTools(toolNames),
   });
+  pi.on("resources_discover", async () => {
+    if (!existsSync(EXTENSION_SKILLS_DIR)) return;
+    return { skillPaths: [EXTENSION_SKILLS_DIR] };
+  });
   pi.on("session_start", async (_event, ctx) => {
     await Temp.prepareExtensionTempDir(TEMP_DIR);
     runtime.loadTools(ctx);

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.3",
+  "version": "0.24.5",
   "private": false,
   "description": "Local Actor Kernel for Pi",
   "keywords": [
@@ -35,9 +35,9 @@
   },
   "files": [
     "index.ts",
+    "dist",
     "lib",
     "scripts",
-    "dist",
     "recipes",
     "skills",
     "README.md",
@@ -56,12 +56,10 @@
       "./index.ts"
     ],
     "skills": [
-      "./dist/skills/actors/SKILL.md",
-      "./dist/skills/swarm/SKILL.md"
+      "./dist/skills"
     ],
     "sourceSkills": [
-      "./skills/actors/SKILL.md",
-      "./skills/swarm/SKILL.md"
+      "./skills"
     ],
     "image": "https://github.com/llblab/pi-actors/raw/main/banner.jpg"
   },

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.3
+  version: 0.24.5
 ---
 
 # 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.3
+  version: 0.24.5
 ---
 
 # Swarm