@nredd/pi-subagents 0.22.0-nredd.7 → 0.22.0-nredd.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nredd/pi-subagents",
-  "version": "0.22.0-nredd.7",
+  "version": "0.22.0-nredd.9",
   "description": "A pi extension that brings Claude Code-like sub-agents and workflow orchestration to pi — parallel execution, live widget, fleet view, custom agent types, mid-run steering, dynamic workflows, Claude Code compatibility, look and feel.",
   "author": "tintinweb",
   "license": "MIT",
@@ -42,30 +42,33 @@
   "scripts": {
     "build": "node -e \"require('fs').rmSync('dist',{recursive:true,force:true})\" && tsc",
     "prepublishOnly": "npm run lint && npm run typecheck && npm run test && npm run build",
-    "test": "vitest run",
-    "test:watch": "vitest",
-    "test:e2e": "vitest run e2e --reporter=verbose",
+    "test": "node ../../scripts/run-vitest.ts node --coverage",
+    "test:watch": "JITI_FS_CACHE=false vitest",
+    "test:e2e": "JITI_FS_CACHE=false vitest run e2e --reporter=verbose",
     "typecheck": "tsc --noEmit",
     "lint": "biome check src/ test/",
     "lint:fix": "biome check --fix src/ test/",
-    "check": "npm run lint && npm run typecheck && npm run test && npm run test:bun",
-    "test:bun": "bun test test/session-scenarios.test.ts test/fleet-session.test.ts test/viewer-session.test.ts test/worktree-isolation-e2e.test.ts",
-    "test:coverage": "vitest run --coverage",
-    "bench": "vitest bench",
+    "check": "npm run lint && npm run typecheck && npm run test && npm run test:workflow-examples && npm run test:bun",
+    "test:bun": "node ../../scripts/run-vitest.ts bun",
+    "test:workflow-examples": "JITI_FS_CACHE=false vitest run --config vitest.workflow-examples.config.ts --coverage",
+    "test:coverage": "node ../../scripts/run-vitest.ts node --coverage",
+    "bench": "JITI_FS_CACHE=false vitest bench",
     "bench:ab": "node test/perf/ab.mjs"
   },
   "devDependencies": {
     "@biomejs/biome": "^2.4.14",
-    "@earendil-works/pi-ai": "https://github.com/nredd/pi/releases/download/v1.3.0-nredd.6/earendil-works-pi-ai-1.3.0.tgz",
-    "@earendil-works/pi-coding-agent": "https://github.com/nredd/pi/releases/download/v1.3.0-nredd.6/earendil-works-pi-coding-agent-1.3.0.tgz",
-    "@earendil-works/pi-tui": "https://github.com/nredd/pi/releases/download/v1.3.0-nredd.6/earendil-works-pi-tui-1.3.0.tgz",
+    "@earendil-works/pi-ai": "https://github.com/nredd/pi/releases/download/v1.3.0-nredd.10/earendil-works-pi-ai-1.3.0.tgz",
+    "@earendil-works/pi-coding-agent": "https://github.com/nredd/pi/releases/download/v1.3.0-nredd.10/earendil-works-pi-coding-agent-1.3.0.tgz",
+    "@earendil-works/pi-tui": "https://github.com/nredd/pi/releases/download/v1.3.0-nredd.10/earendil-works-pi-tui-1.3.0.tgz",
     "@xterm/headless": "5.5.0",
     "@sinclair/typebox": "^0.34.49",
     "@types/node": "^25.5.0",
-    "@vitest/coverage-istanbul": "^4.1.11",
+    "@vitest/coverage-istanbul": "5.0.3",
+    "@vitest/istanbul-lib-coverage": "1.0.2",
+    "@vitest/istanbul-lib-instrument": "1.0.2",
     "typebox": "^1.3.27",
     "typescript": "^6.0.0",
-    "vitest": "^4.1.11"
+    "vitest": "5.0.3"
   },
   "pi": {
     "extensions": [

package/src/abortable.ts

@@ -17,8 +17,8 @@ export function abortable<T>(promise: Promise<T>, signal?: AbortSignal): Promise
   return new Promise<T>((resolve, reject) => {
     let settled = false;
     const cleanup = () => signal.removeEventListener("abort", onAbort);
+    // The listener is removed on every settle path, so abort can only fire while pending.
     const onAbort = () => {
-      if (settled) return;
       settled = true;
       cleanup();
       reject(signal.reason);

package/src/agent-file-toggle.ts

@@ -27,15 +27,15 @@
 
 import { existsSync } from "node:fs";
 import { join, sep } from "node:path";
-import { getAgentDir } from "@earendil-works/pi-coding-agent";
 import { parseAgentFrontmatter } from "./custom-agents.js";
+import { resolveAgentDir } from "./home-paths.js";
 import type { AgentConfig } from "./types.js";
 
 export type AgentFileLocation = "project" | "workspace" | "personal";
 
 export const projectAgentsDir = (cwd: string = process.cwd()) => join(cwd, ".pi", "agents");
 export const workspaceAgentsDir = (cwd: string = process.cwd()) => join(cwd, ".agents", "agents");
-export const personalAgentsDir = () => join(getAgentDir(), "agents");
+export const personalAgentsDir = (homeDir?: string) => join(resolveAgentDir(homeDir), "agents");
 
 /**
  * Find the file path of a custom agent by name, in discovery-precedence order
@@ -46,12 +46,13 @@ export const personalAgentsDir = () => join(getAgentDir(), "agents");
 export function findAgentFile(
   name: string,
   cwd: string = process.cwd(),
+  homeDir?: string,
 ): { path: string; location: AgentFileLocation } | undefined {
   const projectPath = join(projectAgentsDir(cwd), `${name}.md`);
   if (existsSync(projectPath)) return { path: projectPath, location: "project" };
   const workspacePath = join(workspaceAgentsDir(cwd), `${name}.md`);
   if (existsSync(workspacePath)) return { path: workspacePath, location: "workspace" };
-  const personalPath = join(personalAgentsDir(), `${name}.md`);
+  const personalPath = join(personalAgentsDir(homeDir), `${name}.md`);
   if (existsSync(personalPath)) return { path: personalPath, location: "personal" };
   return undefined;
 }
@@ -83,11 +84,12 @@ export function locateAgentFile(
   sourcePath: string | undefined,
   cwd: string = process.cwd(),
   source?: AgentConfig["source"],
+  homeDir?: string,
 ): { path: string; location: AgentFileLocation } | undefined {
   if (source !== "package" && sourcePath && existsSync(sourcePath)) {
     return { path: sourcePath, location: classifyAgentDir(sourcePath, cwd) };
   }
-  return findAgentFile(name, cwd);
+  return findAgentFile(name, cwd, homeDir);
 }
 
 /**
@@ -126,7 +128,6 @@ function splitFrontmatter(content: string):
   | { lines: string[]; openIdx: number; closeIdx: number; eol: string }
   | undefined {
   const lines = content.split(/(?<=\n)/);
-  if (lines.length === 0) return undefined;
   // The BOM stays where it is — it belongs to the file, not the block — so the
   // fence test looks past it and every index below is unaffected.
   const bom = content.startsWith("\uFEFF");