@arhen/pi-core-subagent 1.3.65 → 1.3.67

package/README.md

@@ -1,7 +1,7 @@
 # @arhen/pi-core-subagent
 
 [![npm version](https://img.shields.io/npm/v/%40arhen%2Fpi-core-subagent?color=cb3837&logo=npm)](https://www.npmjs.com/package/@arhen/pi-core-subagent)
-[![CI](https://img.shields.io/github/actions/workflow/status/arhen/pi-extensions/pi-core-subagent.yml?branch=main&logo=github&label=CI)](https://github.com/arhen/pi-extensions/actions/workflows/pi-core-subagent.yml)
+[![CI](https://img.shields.io/github/actions/workflow/status/arhen/pi-extensions/ci.yml?branch=main&logo=github&label=CI)](https://github.com/arhen/pi-extensions/actions/workflows/ci.yml)
 [![license](https://img.shields.io/badge/license-MIT-blue.svg)](./LICENSE)
 [![pi extension](https://img.shields.io/badge/pi-extension-7c3aed)](https://github.com/earendil-works/pi)
 
@@ -327,9 +327,12 @@ still applies Pi\'s global script-only policy; this extension does not override
 - `/subagents mode codemode` — explicitly select script-based routing (direct fallback when unavailable).
 - `/subagents mode direct` — return to legacy direct behavior.
 
-Choice is stored per session branch. New sessions/branches without a preference start direct;
-existing explicit choices remain intact. Published 1.3.63 defaulted to `auto`: this compatibility
-correction requires the corrected source/package, not a retroactive change to that npm release.
+Choice is stored globally in `~/.pi/agent/subagents-config.json` (the same file as `auto-limit`,
+written with a merge, never a clobber) and applies to every session — reload, resume, fork and tree
+navigation included. Sessions that chose a mode before the preference moved into that file keep their
+branch entry only while the config has no `mode`; the built-in default is still `direct`. Published
+1.3.63 defaulted to `auto`: this compatibility correction requires the corrected source/package, not
+a retroactive change to that npm release.
 
 ## Commands
 

package/package.json

@@ -1,16 +1,15 @@
 {
 	"name": "@arhen/pi-core-subagent",
-	"version": "1.3.65",
+	"version": "1.3.67",
 	"type": "module",
 	"description": "pi extension: fast in-process subagents with a dependency-graph scheduler (needs edges gate tasks and carry upstream output into dependent prompts), plus background runs, intercom and agent-to-agent mailbox. Leader defines agents inline.",
 	"license": "MIT",
 	"main": "./src/index.ts",
 	"scripts": {
-		"test": "bun test",
 		"typecheck": "tsc --noEmit",
 		"lint": "biome check .",
-		"format": "biome format --write .",
-		"check": "bun run typecheck && bun run lint && bun test"
+		"test": "bun test",
+		"check": "npm run typecheck && npm run lint && npm test"
 	},
 	"files": [
 		"src",
@@ -40,11 +39,6 @@
 			"./src/index.ts"
 		]
 	},
-	"devDependencies": {
-		"@biomejs/biome": "^2.5.8",
-		"@types/bun": "^1.3.14",
-		"typescript": "^7.0.2"
-	},
 	"repository": {
 		"type": "git",
 		"url": "https://github.com/arhen/pi-extensions",

package/src/config.ts

@@ -0,0 +1,60 @@
+import { readFileSync } from "node:fs";
+import { writeFile } from "node:fs/promises";
+import { join } from "node:path";
+import { getAgentDir } from "@earendil-works/pi-coding-agent";
+
+/**
+ * Shared user config for the extension: exposure mode plus the auto-limit ceiling. One file, so a
+ * write of one key must never drop the other — every writer merges through `writeSubagentConfig`.
+ */
+export const SUBAGENT_CONFIG_FILENAME = "subagents-config.json";
+
+/**
+ * Presentation preference for the subagent toolset, stored globally in `subagents-config.json`.
+ * - auto: native codemode presentation when the codemode tool is active, direct otherwise.
+ * - direct (default): legacy native declarations and active-tool script callability.
+ * - codemode: subagent tools are callable from codemode scripts but not declared or listed.
+ */
+export type SubagentMode = "auto" | "direct" | "codemode";
+
+export const DEFAULT_SUBAGENT_MODE: SubagentMode = "direct";
+
+export function isSubagentMode(value: unknown): value is SubagentMode {
+	return value === "auto" || value === "direct" || value === "codemode";
+}
+
+/** Resolved per call, so a test or `PI_CODING_AGENT_DIR` override applies to the running process. */
+export function subagentConfigPath(): string {
+	return join(getAgentDir(), SUBAGENT_CONFIG_FILENAME);
+}
+
+/**
+ * Tolerant read: a missing, unreadable or non-object file degrades to an empty object, so a config
+ * typo can neither break delegation nor hide the built-in defaults.
+ */
+export function readSubagentConfig(path = subagentConfigPath()): Record<string, unknown> {
+	try {
+		const raw: unknown = JSON.parse(readFileSync(path, "utf8"));
+		if (raw === null || typeof raw !== "object" || Array.isArray(raw)) return {};
+		return raw as Record<string, unknown>;
+	} catch {
+		return {};
+	}
+}
+
+/** The stored exposure mode, or undefined when the file has no usable one. */
+export function readStoredMode(path = subagentConfigPath()): SubagentMode | undefined {
+	const mode = readSubagentConfig(path).mode;
+	return isSubagentMode(mode) ? mode : undefined;
+}
+
+let writeQueue: Promise<unknown> = Promise.resolve();
+
+/** Read-modify-write on a single queue, so concurrent writers of different keys cannot clobber each other. */
+export function writeSubagentConfig(patch: Record<string, unknown>, path = subagentConfigPath()): Promise<void> {
+	const write = writeQueue.then(async () => {
+		await writeFile(path, `${JSON.stringify({ ...readSubagentConfig(path), ...patch }, null, 2)}\n`);
+	});
+	writeQueue = write.catch(() => {});
+	return write;
+}