pi-cohort 7.1.1 → 7.1.2

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [7.1.2] - 2026-09-29
+
+### Fixed
+
+- `typebox` moves from `dependencies` to an optional `peerDependencies: "*"` entry, which silences pi's "Host-provided extension packages must be declared in peerDependencies" startup warning and removes the second typebox copy from consumer installs. The detached async runner keeps working without its own copy: `spawnRunner` now passes pi's own `typebox`, `typebox/compile`, and `typebox/value` files to the bare jiti child through `JITI_ALIAS`, the same alias map pi's extension loader applies in-process, so the `Cannot find module 'typebox/compile'` crash from [#9](https://github.com/jjuraszek/pi-cohort/issues/9) stays fixed.
+
 ## [7.1.1] - 2026-09-24
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-cohort",
-  "version": "7.1.1",
+  "version": "7.1.2",
   "description": "Delegate Pi work to focused child agents: code review, scouting, implementation, parallel audits, saved chains, and background jobs.",
   "author": "Jacek Juraszek",
   "license": "MIT",
@@ -64,7 +64,8 @@
     "@earendil-works/pi-agent-core": "*",
     "@earendil-works/pi-ai": "*",
     "@earendil-works/pi-coding-agent": ">=0.85.0",
-    "@earendil-works/pi-tui": "*"
+    "@earendil-works/pi-tui": "*",
+    "typebox": "*"
   },
   "peerDependenciesMeta": {
     "@earendil-works/pi-agent-core": {
@@ -78,16 +79,19 @@
     },
     "@earendil-works/pi-tui": {
       "optional": true
+    },
+    "typebox": {
+      "optional": true
     }
   },
   "dependencies": {
-    "jiti": "^2.7.0",
-    "typebox": "^1.3.11"
+    "jiti": "^2.7.0"
   },
   "devDependencies": {
     "@earendil-works/pi-agent-core": "^0.85.0",
     "@earendil-works/pi-ai": "^0.85.0",
     "@earendil-works/pi-coding-agent": "^0.85.0",
-    "@earendil-works/pi-tui": "^0.85.0"
+    "@earendil-works/pi-tui": "^0.85.0",
+    "typebox": "^1.3.11"
   }
 }

package/src/runs/background/async-execution.ts

@@ -14,7 +14,7 @@ import { injectSingleOutputInstruction, normalizeSingleOutputOverride, resolvePa
 import { buildChainInstructions, isDynamicParallelStep, isParallelStep, resolveStepBehavior, suppressProgressForReadOnlyTask, writeInitialProgressFile, type ChainStep, type ResolvedStepBehavior, type SequentialStep, type StepOverrides } from "../../shared/settings.ts";
 import type { RunnerStep } from "../shared/parallel-utils.ts";
 import { resolvePiPackageRoot } from "../shared/pi-spawn.ts";
-import { ensureJitiCliPath } from "../shared/jiti-cli.ts";
+import { ensureJitiCliPath, resolveHostModuleAliases } from "../shared/jiti-cli.ts";
 export { createJitiCliResolver } from "../shared/jiti-cli.ts";
 import { buildSkillInjection, normalizeSkillInput, resolveSkillsWithFallback } from "../../agents/skills.ts";
 import { resolveChildCwd } from "../../shared/utils.ts";
@@ -145,7 +145,7 @@ export function isAsyncAvailable(): boolean {
  * Falls back to ignored stdio if the log file cannot be opened; the parent always
  * closes its copy of the fd, on every exit path.
  */
-export function spawnDetachedWithLog(command: string, args: string[], cwd: string, asyncDir: string, spawnImpl: typeof spawn = spawn): { pid?: number; error?: string } {
+export function spawnDetachedWithLog(command: string, args: string[], cwd: string, asyncDir: string, spawnImpl: typeof spawn = spawn, env?: NodeJS.ProcessEnv): { pid?: number; error?: string } {
 	let fd: number | undefined;
 	try {
 		try {
@@ -156,6 +156,7 @@ export function spawnDetachedWithLog(command: string, args: string[], cwd: strin
 		}
 		const proc = spawnImpl(command, args, {
 			cwd,
+			env,
 			detached: true,
 			stdio: fd === undefined ? "ignore" : ["ignore", fd, fd],
 			windowsHide: true,
@@ -187,7 +188,7 @@ export function spawnRunner(
 	suffix: string,
 	cwd: string,
 	asyncDir: string,
-	deps: { ensureJiti?: () => string | undefined; spawnImpl?: typeof spawn } = {},
+	deps: { ensureJiti?: () => string | undefined; spawnImpl?: typeof spawn; hostAliases?: () => Record<string, string> } = {},
 ): { pid?: number; error?: string } {
 	const ensureJiti = deps.ensureJiti ?? ensureJitiCliPath;
 	const jitiCliPath = ensureJiti();
@@ -209,7 +210,12 @@ export function spawnRunner(
 	fs.writeFileSync(cfgPath, JSON.stringify(cfg));
 	const runner = path.join(path.dirname(fileURLToPath(import.meta.url)), "subagent-runner.ts");
 
-	return spawnDetachedWithLog(process.execPath, [jitiCliPath, runner, cfgPath], cwd, asyncDir, deps.spawnImpl ?? spawn);
+	const aliases = (deps.hostAliases ?? resolveHostModuleAliases)();
+	const env = Object.keys(aliases).length > 0
+		? { ...process.env, JITI_ALIAS: JSON.stringify(aliases) }
+		: process.env;
+
+	return spawnDetachedWithLog(process.execPath, [jitiCliPath, runner, cfgPath], cwd, asyncDir, deps.spawnImpl ?? spawn, env);
 }
 
 /**

package/src/runs/shared/jiti-cli.ts

@@ -24,17 +24,23 @@ function resolveJitiCliFromPackageJson(packageJsonPath: string): string | undefi
 	return undefined;
 }
 
-function resolveJitiCliPath(): string | undefined {
-	const candidates: Array<() => string | undefined> = [
-		() => require.resolve("jiti/package.json"),
+function hostResolveCandidates(specifier: string): Array<() => string | undefined> {
+	return [
+		() => require.resolve(specifier),
 		() => piPackageRoot
-			? createRequire(path.join(piPackageRoot, "package.json")).resolve("jiti/package.json")
+			? createRequire(path.join(piPackageRoot, "package.json")).resolve(specifier)
 			: undefined,
 		() => {
 			if (!process.argv[1]) return undefined;
 			const piEntry = fs.realpathSync(process.argv[1]);
-			return createRequire(piEntry).resolve("jiti/package.json");
+			return createRequire(piEntry).resolve(specifier);
 		},
+	];
+}
+
+function resolveJitiCliPath(): string | undefined {
+	const candidates = [
+		...hostResolveCandidates("jiti/package.json"),
 		() => piPackageRoot ? path.join(piPackageRoot, "node_modules", "jiti", "package.json") : undefined,
 	];
 	for (const candidate of candidates) {
@@ -50,6 +56,31 @@ function resolveJitiCliPath(): string | undefined {
 	return undefined;
 }
 
+function resolveFromHost(specifier: string): string | undefined {
+	for (const candidate of hostResolveCandidates(specifier)) {
+		try {
+			const resolved = candidate();
+			if (resolved) return resolved;
+		} catch {
+			// Candidate not available in this install, continue probing.
+		}
+	}
+	return undefined;
+}
+
+// Pi's extension loader aliases these to its own copies; the detached runner boots
+// under the bare jiti CLI, so it gets the same map through JITI_ALIAS.
+const HOST_ALIASED_SPECIFIERS = ["typebox", "typebox/compile", "typebox/value"];
+
+export function resolveHostModuleAliases(): Record<string, string> {
+	const aliases: Record<string, string> = {};
+	for (const specifier of HOST_ALIASED_SPECIFIERS) {
+		const resolved = resolveFromHost(specifier);
+		if (resolved) aliases[specifier] = resolved;
+	}
+	return aliases;
+}
+
 export function createJitiCliResolver(deps: { resolve?: () => string | undefined; exists?: (p: string) => boolean } = {}): () => string | undefined {
 	const resolve = deps.resolve ?? resolveJitiCliPath;
 	const exists = deps.exists ?? ((p: string) => fs.existsSync(p));