pi-open-agents 0.1.25 → 0.1.26

package/README.md

@@ -191,6 +191,12 @@ This means:
 - Each subagent runs with exactly the model and tools it declares
 - Skills load per-agent, with wildcard support (`security-*`, `git-*`)
 
+> **`skills:` is subagent-only (#19).** pi discovers skills once at session
+> startup and exposes no per-agent filter, so a **primary** cannot role-scope
+> them. Declaring `skills:` on a primary (or `all`) agent warns on apply and
+> the field is ignored; the subagent child receives exactly its listed skills,
+> injected into its system prompt.
+
 **Per-call model override:** the caller can override the agent's default for a
 single call — `subagent({ agent: "deep-research", task: "...", model: "zai/glm-5.3-flash" })`.
 Handy for "run this one task on the cheap model" without forking an agent

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-open-agents",
-  "version": "0.1.25",
+  "version": "0.1.26",
   "description": "Unified agent and subagent management for pi coding agent, with OpenCode-compatible agent definitions.",
   "keywords": [
     "pi-package",

package/src/primary/manager.ts

@@ -5,11 +5,28 @@
  * and coordinates with the executor for model/thinking/tools/prompt.
  */
 
-import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
+import type { ExtensionAPI, ExtensionContext, ExtensionUIContext } from "@earendil-works/pi-coding-agent";
 import type { Model } from "@earendil-works/pi-ai";
 import type { AgentDefinition, ThinkingLevel } from "../types.ts";
 import { applyAgentConfig } from "./executor.ts";
 
+/**
+ * Warn when an agent that will act as PRIMARY declares `skills:`.
+ *
+ * `skills:` is honored only for subagent children (the executor injects them
+ * into the child's system prompt). pi loads session skills at startup and
+ * exposes no per-agent filter, so a primary cannot role-scope them — warn
+ * instead of silently ignoring (#19).
+ */
+export function warnAgentSkillsPrimary(agent: AgentDefinition | undefined, ui: ExtensionUIContext): void {
+  if (agent?.skills && agent.skills.length > 0 && agent.mode !== "subagent") {
+    ui.notify(
+      `Agent "${agent.name}": skills: (${agent.skills.join(", ")}) applies to subagent delegation only — primaries use pi's session-wide skill set (#19)`,
+      "warning",
+    );
+  }
+}
+
 interface OriginalState {
   model: Model<any> | undefined;
   thinkingLevel: ThinkingLevel;
@@ -56,6 +73,8 @@ export class AgentManager {
     const agent = this.agents.get(name);
     if (!agent) return false;
 
+    warnAgentSkillsPrimary(agent, ctx.ui);
+
     // Snapshot state on first apply (for restore on clear)
     if (this.activeName === undefined) {
       this.originalState = {