@llblab/pi-actors 0.45.1 → 0.46.0

package/lib/extension-runtime.ts

@@ -20,6 +20,7 @@ export interface ActorExtensionRuntime {
   beforeAgentStart(
     systemPrompt: string,
     skills: RecipesReferences.ActiveSkillRecipeSource[],
+    ctx: Pi.ExtensionContext,
   ): { systemPrompt: string };
   discoverResources(metaUrl: string): { skillPaths: string[] } | undefined;
   getRunOwnerId(ctx: Pi.ExtensionContext): string;
@@ -33,7 +34,14 @@ export function createActorExtensionRuntime(
   pi: Pi.ExtensionAPI,
 ): ActorExtensionRuntime {
   let activeRunContext: Pi.ExtensionContext | undefined;
+  const skillContextsBySession = new Map<
+    string,
+    RecipesReferences.ActiveSkillRecipeContext
+  >();
   const getRunOwnerId = Pi.getSessionId;
+  const getSkillContext = (ctx: Pi.ExtensionContext) =>
+    skillContextsBySession.get(getRunOwnerId(ctx)) ??
+    RecipesReferences.EMPTY_ACTIVE_SKILL_RECIPE_CONTEXT;
   const automaticReview = AutomaticReviewRuntime.createAutomaticReviewRuntime({
     getActiveContext: () => activeRunContext,
     getRunOwnerId,
@@ -59,6 +67,9 @@ export function createActorExtensionRuntime(
         if (ctx && typeof ctx === "object") {
           nextArgs[4] = {
             ...(ctx as Record<string, unknown>),
+            activeSkillRecipeContext: getSkillContext(
+              ctx as Pi.ExtensionContext,
+            ),
             getThinkingLevel: () => pi.getThinkingLevel(),
           };
         }
@@ -84,8 +95,11 @@ export function createActorExtensionRuntime(
   });
   const recipeReload = Runtime.createRecipeToolReloadWatcher(runtime);
   return {
-    beforeAgentStart(systemPrompt, skills) {
-      RecipesReferences.setActiveSkillRecipeSources(skills);
+    beforeAgentStart(systemPrompt, skills, ctx) {
+      skillContextsBySession.set(
+        getRunOwnerId(ctx),
+        RecipesReferences.createActiveSkillRecipeContext(skills),
+      );
       return {
         systemPrompt: `${systemPrompt}\n\n${Prompts.ONBOARDING_SYSTEM_PROMPT}`,
       };
@@ -99,12 +113,17 @@ export function createActorExtensionRuntime(
       if (activeRunContext === ctx) automaticReview.schedule();
     },
     onSessionShutdown(reason, ctx) {
+      skillContextsBySession.delete(getRunOwnerId(ctx));
       activeRunContext = undefined;
       automaticReview.close();
       recipeReload.close();
       runUiRuntime.shutdown(reason, ctx);
     },
     async onSessionStart(ctx) {
+      skillContextsBySession.set(
+        getRunOwnerId(ctx),
+        RecipesReferences.EMPTY_ACTIVE_SKILL_RECIPE_CONTEXT,
+      );
       ctx.ui.setWidget("zz-pi-actors-comms", undefined);
       activeRunContext = ctx;
       runUiRuntime.close();

package/lib/inspector.ts

@@ -102,17 +102,51 @@ export function readActorInspectorRecipe(
     !Array.isArray(primary.recipe)
       ? (primary.recipe as Record<string, unknown>)
       : undefined;
-  const redactedRecord = (value: Record<string, unknown>): Record<string, unknown> =>
-    SessionEvidence.redactSessionEvidenceValue(
-      Object.fromEntries(
-        Object.entries(value).filter(
+  const stripPrivateOrigins = (value: unknown): unknown => {
+    if (Array.isArray(value)) return value.map(stripPrivateOrigins);
+    if (!value || typeof value !== "object") return value;
+    return Object.fromEntries(
+      Object.entries(value as Record<string, unknown>)
+        .filter(
           ([key, entry]) =>
             entry !== undefined &&
             key !== "recipe_dir" &&
-            key !== "skill_dir",
-        ),
-      ),
+            key !== "skill_dir" &&
+            key !== "source_file",
+        )
+        .map(([key, entry]) => [
+          key,
+          key === "actorRecipeContext" && entry && typeof entry === "object"
+            ? stripPrivateOrigins(
+                Object.fromEntries(
+                  Object.entries(entry as Record<string, unknown>).filter(
+                    ([contextKey]) => contextKey !== "file",
+                  ),
+                ),
+              )
+            : stripPrivateOrigins(entry),
+        ]),
+    );
+  };
+  const redactedRecord = (value: Record<string, unknown>): Record<string, unknown> =>
+    SessionEvidence.redactSessionEvidenceValue(
+      stripPrivateOrigins(value),
     ) as Record<string, unknown>;
+  const primarySourceKind =
+    primary?.source_kind === "active_skill_component"
+      ? "active_skill_component"
+      : primary?.source_kind === "user_registry_capability" ||
+          meta.launch_source === "tool"
+        ? "user_registry_capability"
+        : "explicit_file_recipe";
+  const primaryLogicalReference =
+    typeof primary?.logical_reference === "string"
+      ? primary.logical_reference
+      : typeof primary?.name === "string"
+        ? primary.name
+        : typeof meta.recipe === "string"
+          ? meta.recipe
+          : "unknown";
   const launchValues =
     meta.values && typeof meta.values === "object" && !Array.isArray(meta.values)
       ? Object.fromEntries(
@@ -126,10 +160,18 @@ export function readActorInspectorRecipe(
       redactedRecord({
         ...(typeof entry.alias === "string" ? { alias: entry.alias } : {}),
         depth: entry.depth,
-        file: entry.file,
         import_path: entry.import_path,
-        name: entry.name,
-        qualified_name: entry.qualified_name,
+        logical_reference: entry.logical_reference ?? entry.name,
+        recipe_stem: entry.name,
+        role: entry.role,
+        ...(typeof entry.skill === "string" ? { skill: entry.skill } : {}),
+        source_kind:
+          entry.source_kind === "active_skill_component"
+            ? "active_skill_component"
+            : entry.source_kind === "user_registry_capability" ||
+                (entry.depth === 0 && meta.launch_source === "tool")
+              ? "user_registry_capability"
+              : "explicit_file_recipe",
       }),
     ),
     ...(definition ? { definition: redactedRecord(definition) } : {}),
@@ -137,7 +179,10 @@ export function readActorInspectorRecipe(
     identity: redactedRecord({
       run: meta.run,
       recipe: meta.recipe,
-      recipe_file: meta.recipe_file,
+      recipe_stem: primary?.name ?? meta.recipe,
+      logical_reference: primaryLogicalReference,
+      ...(typeof primary?.skill === "string" ? { skill: primary.skill } : {}),
+      source_kind: primarySourceKind,
       launch_source: meta.launch_source,
     }),
     launch: redactedRecord({

package/lib/paths.ts

@@ -109,10 +109,3 @@ export function getRecipeRoot(agentDir = getAgentDir()): string {
 export function getRecipeDraftRoot(agentDir = getAgentDir()): string {
   return join(getRecipeRoot(agentDir), "drafts");
 }
-
-export function getPackagedRecipeRoot(): string {
-  const here = dirname(fileURLToPath(import.meta.url));
-  const compiledRoot = resolve(here, "..", "..", "recipes");
-  if (existsSync(compiledRoot)) return compiledRoot;
-  return resolve(here, "..", "recipes");
-}

package/lib/prompts.ts

@@ -30,8 +30,8 @@ export const ONBOARDING_SYSTEM_PROMPT = `pi-actors quick model:
 - Actor-mode trigger: if work may outlive this turn, need steering/follow-up/artifacts, run as a service, fan out, or be resumed/inspected later, use spawn -> message -> inspect instead of ad hoc shell backgrounding.
 - Use spawn/message/inspect for actor-level start/send/observe; short foreground checks can stay ordinary tools/templates; avoid internal transport vocabulary in public guidance.
 - Run state lives under ~/.pi/agent/tmp/pi-actors/runs. Inspect intentionally and avoid busy-polling. Terminal and coordinator-bound notifications queue as Pi follow-ups so concurrently completed actors can reach the coordinator after current work instead of steering between tool calls. Terminal follow-up content stays minimal: run, status, one base path, and relative artifact names only; semantic output stays in non-LLM details and run state. When a deferred actor result gates the next step, wait for its terminal follow-up; do not schedule continuation loops, repeatedly inspect, or mutate its reviewed scope while it runs. Inspect early only for an operator request, a meaningful actor event, or diagnosis of an overdue/stuck run.
-- Maintain ~/.pi/agent/recipes like MEMORY.md for capabilities: keep useful tools, curate stale ones, and fix/remove/disable invalid recipes flagged by registry warnings; packaged/ad hoc recipes are lower-priority components; offer to save successful recurring patterns only after confirmation.
-- Prefer maintained packaged recipes/pipelines with spawn file=<recipe> before ad hoc scripts/wrappers; review swarms inherit current model/thinking, preflight before fanout, and expose quorum/concurrency/TTL knobs unless explicit args are passed.
+- Maintain ~/.pi/agent/recipes like MEMORY.md for capabilities: keep useful tools, curate stale ones, and fix/remove/disable invalid recipes flagged by registry warnings; active-Skill and explicit file Recipes remain components outside user tool discovery; offer to save successful recurring patterns only after confirmation.
+- Prefer maintained active-Skill Recipes with spawn recipe=<skill>/<recipe> before ad hoc scripts/wrappers; explicit file Recipes use exact .json/.md paths; review swarms inherit current model/thinking, preflight before fanout, and expose quorum/concurrency/TTL knobs unless explicit args are passed.
 - For any non-trivial actor use or pi-actors change, read the bundled actors skill first. Before launching multiple actors/subagents for parallel implementation, independent artifact generation, delegated audit, or review, also read the bundled swarm skill; the coordinator owns decomposition, disjoint scopes, launch correctness, integration, and final validation. For deeper guidance, inspect installed extension sources/docs/recipes because README/docs are not automatically in context.`;
 
 export const REGISTER_TOOL_PARAM_DESCRIPTIONS = {

package/lib/recipes-context.ts

@@ -96,7 +96,7 @@ function matchesActorContext(
   context: CommandTemplateActorRecipeContext | undefined,
 ): boolean {
   if (!context) return record.role === "entry";
-  if (context.file && context.file === record.file) return true;
+  if (context.file && context.file === record.source_file) return true;
   if (context.name && context.name === record.name) return true;
   if (context.alias && context.alias === record.alias) return true;
   return false;
@@ -131,7 +131,7 @@ export function formatRecipeContextJsonl(
   context?: CommandTemplateActorRecipeContext,
 ): string {
   return markRecipeContextRecords(records, context)
-    .map((record) => JSON.stringify(record))
+    .map(({ source_file: _sourceFile, ...record }) => JSON.stringify(record))
     .join("\n");
 }