pi-agents-switch 0.3.1 → 0.3.3

package/README.md

@@ -12,6 +12,10 @@ pi install ~/project/agents-switch
 
 Then `/reload` in pi.
 
+> **Requires Pi ≥ v0.x** (tested with the version shipping `before_agent_start` + `before_provider_request` events).
+> If Pi's system prompt format changes, agent identity will be prepended instead of replacing the default role line —
+> check the Pi console for a `[agents-switch]` warning if this happens.
+
 ## Usage
 
 | Action | How |
@@ -53,8 +57,7 @@ The Builder will design the agent, write the AGENTS.md, and set everything up.
   ```
   ~/.pi/agents/<name>/
   ├── AGENTS.md       ← YAML frontmatter + system prompt (all-in-one)
-  ├── skills/         ← agent-specific skills (auto-loaded)
-  └── prompts/        ← agent-specific prompt templates (optional)
+  └── skills/         ← agent-specific skills (auto-loaded, optional)
   ```
 
 When you switch to an agent, its `AGENTS.md` body is injected into the system prompt. You can also configure model, thinking level, and tools per agent via the frontmatter.

package/frontmatter.ts

@@ -145,7 +145,6 @@ function parseScalar(value: string): string | boolean | number {
 
 /**
  * Serialize frontmatter back to a YAML block (for creating agents).
- * Uses the preferred key names: `excluded_tools`, `excluded_extensions`, `excluded_skills`.
  */
 export function serializeFrontmatter(fm: AgentFrontmatter): string {
 	const lines: string[] = ["---"];

package/index.ts

@@ -140,9 +140,7 @@ If you want to exclude just a few items, add \`*\` to the include list and use t
 
 \`\`\`
 ~/.pi/agents/<name>/
-├── AGENTS.md          ← YAML frontmatter + system prompt (THE ONLY REQUIRED FILE)
-├── skills/            ← Agent-specific skills (mkdir)
-└── prompts/           ← Agent-specific prompt templates (mkdir)
+└── skills/            ← Agent-specific skills (mkdir, optional)
 \`\`\`
 
 ## Workflow: How to create an agent
@@ -752,6 +750,12 @@ export default function agentsSwitch(pi: ExtensionAPI) {
 					section +
 					newPrompt.substring(roleEnd);
 			} else {
+				// Pi version mismatch — role line changed. Fall back to prepend.
+				console.warn(
+					"[agents-switch] Default role line not found in system prompt. " +
+						"Pi may have changed its prompt format. " +
+						"Agent identity will be prepended instead of replacing the role line.",
+				);
 				newPrompt = section + "\n\n" + newPrompt;
 			}
 		}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-agents-switch",
-  "version": "0.3.1",
+  "version": "0.3.3",
   "description": "Tab to switch primary agents in Pi — like OpenCode's agent switching. Each agent gets an isolated profile with its own AGENTS.md, extensions, skills, and settings.",
   "type": "module",
   "pi": {

package/profile-manager.ts

@@ -8,8 +8,7 @@
  * Profile dir layout:
  *   ~/.pi/agents/<name>/          ← user-level agent
  *   ├── AGENTS.md                 ← YAML frontmatter + system prompt
- *   ├── skills/                   ← agent-specific skills
- *   └── prompts/                  ← agent-specific prompt templates
+ *   └── skills/                   ← agent-specific skills
  *
  *   <cwd>/.pi/agents/<name>/      ← project-level agent (overrides user)
  *   └── AGENTS.md
@@ -478,12 +477,10 @@ export class ProfileManager {
 	 * Look up full skill objects by name from Pi's standard skill directories.
 	 * Returns name, description, and filePath for each skill found.
 	 */
-	/** Get PI skill names (global + extension + project, excluding agent-scoped). */
+	/** Get all indexed skill names (global + extension + project). */
 	getPISkillNames(): string[] {
 		if (!this.skillCache) this.rebuildSkillIndex();
-		return [...this.skillCache!.entries()]
-			.filter(([, v]) => v.source !== "agent")
-			.map(([k]) => k);
+		return [...this.skillCache!.keys()];
 	}
 
 	/**
@@ -562,17 +559,76 @@ export class ProfileManager {
 		if (!this.skillCache) {
 			this.rebuildSkillIndex();
 		}
-		return this.skillCache!.get(name);
+		const cached = this.skillCache!.get(name);
+		if (cached) return cached;
+
+		// Fallback: agent-specific skills are not indexed globally.
+		// Check each agent's skills/ dir on demand.
+		return this.findAgentSkillOnDisk(name);
+	}
+
+	/**
+	 * Look up a single agent-specific skill by checking each agent's
+	 * skills/<name>/SKILL.md path directly.
+	 */
+	private findAgentSkillOnDisk(
+		name: string,
+	): { name: string; description: string; filePath: string } | undefined {
+		const read = (
+			filePath: string,
+		): { name: string; description: string; filePath: string } | undefined => {
+			if (!existsSync(filePath)) return undefined;
+			try {
+				const content = readFileSync(filePath, "utf8");
+				const { frontmatter } = parseFrontmatter(content);
+				return {
+					name: frontmatter.name ?? name,
+					description: frontmatter.description ?? "",
+					filePath,
+				};
+			} catch {
+				return undefined;
+			}
+		};
+
+		// User-level agents
+		const userDir = join(homedir(), ".pi", "agents");
+		if (existsSync(userDir)) {
+			try {
+				for (const agent of readdirSync(userDir)) {
+					const found = read(join(userDir, agent, "skills", name, "SKILL.md"));
+					if (found) return found;
+				}
+			} catch {}
+		}
+
+		// Project-level agents
+		if (this.cwd) {
+			const projDir = join(this.cwd, ".pi", "agents");
+			if (existsSync(projDir)) {
+				try {
+					for (const agent of readdirSync(projDir)) {
+						const found = read(
+							join(projDir, agent, "skills", name, "SKILL.md"),
+						);
+						if (found) return found;
+					}
+				} catch {}
+			}
+		}
+
+		return undefined;
 	}
 
 	/**
-	 * Rebuild the skill index by scanning all known skill directories.
+	 * Rebuild the skill index by scanning known skill directories.
+	 * Agent-specific skills are NOT indexed here — they are looked up
+	 * on demand via findSkillOnDisk → scanForAgentSkill().
 	 *
 	 * Locations scanned:
 	 *   1. ~/.pi/agent/skills/<name>/SKILL.md            (flat, user-installed)
 	 *   2. ~/.pi/agent/npm/node_modules/<ext>/skills/    (extension skills)
-	 *   3. ~/.pi/agents/<agent>/skills/                  (agent-specific skills)
-	 *   4. <cwd>/.pi/skills/                             (project skills, flat)
+	 *   3. <cwd>/.pi/skills/                             (project skills, flat)
 	 */
 	rebuildSkillIndex(): void {
 		this.skillCache = new Map();
@@ -580,7 +636,6 @@ export class ProfileManager {
 		const scanDir = (dir: string, maxDepth: number, source: string): void => {
 			if (maxDepth <= 0) return;
 
-			// Check for SKILL.md in this directory
 			const skillMd = join(dir, "SKILL.md");
 			if (existsSync(skillMd)) {
 				try {
@@ -610,7 +665,7 @@ export class ProfileManager {
 		// 1. Global Pi skills
 		scanDir(join(this.piAgentDir, "skills"), 2, "global");
 
-		// 2. Extension skills from npm packages (nested: node_modules/<ext>/skills/<name>/SKILL.md)
+		// 2. Extension skills from npm packages
 		const npmDir = join(this.piAgentDir, "npm", "node_modules");
 		if (existsSync(npmDir)) {
 			for (const ext of readdirSync(npmDir)) {
@@ -620,35 +675,12 @@ export class ProfileManager {
 						if (statSync(extSkills).isDirectory()) {
 							scanDir(extSkills, 3, "extension");
 						}
-					} catch {
-						// Skip
-					}
-				}
-			}
-		}
-
-		// 3. Agent-specific skills (nested: agents/<agent>/skills/<name>/SKILL.md)
-		const agentsDir = join(homedir(), ".pi", "agents");
-		if (existsSync(agentsDir)) {
-			try {
-				for (const agent of readdirSync(agentsDir)) {
-					const agentSkills = join(agentsDir, agent, "skills");
-					if (existsSync(agentSkills)) {
-						try {
-							if (statSync(agentSkills).isDirectory()) {
-								scanDir(agentSkills, 3, "agent");
-							}
-						} catch {
-							// Skip
-						}
-					}
+					} catch {}
 				}
-			} catch {
-				// Skip unreadable agents dir
 			}
 		}
 
-		// 4. Project-level skills (flat)
+		// 3. Project-level skills
 		if (this.cwd) {
 			const projSkillsDir = join(this.cwd, ".pi", "skills");
 			if (existsSync(projSkillsDir)) {
@@ -685,7 +717,6 @@ export class ProfileManager {
 		try {
 			mkdirSync(path, { recursive: true });
 			mkdirSync(join(path, "skills"));
-			mkdirSync(join(path, "prompts"));
 		} catch (err) {
 			try {
 				rmSync(path, { recursive: true, force: true });

package/types.ts

@@ -62,8 +62,8 @@ export interface FrontmatterResult {
 	body: string;
 	/**
 	 * Ordered list of top-level YAML keys as they appear in the file.
-	 * Used for last-write-wins conflict resolution between `tools`/`excluded_tools`,
-	 * `extensions`/`excluded_extensions`/`noextensions`, etc.
+	 * Used for last-write-wins conflict resolution between `tools`/`excluded_tools`
+	 * and `skills`/`excluded_skills`.
 	 */
 	keyOrder: string[];
 }