pi-crew 0.3.1 → 0.3.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-crew",
-  "version": "0.3.1",
+  "version": "0.3.3",
   "description": "Pi extension for coordinated AI teams, workflows, worktrees, and async task orchestration",
   "author": "baphuongna",
   "license": "MIT",
@@ -88,6 +88,7 @@
     "@mariozechner/pi-agent-core": "^0.65.0",
     "@mariozechner/pi-ai": "^0.65.0",
     "@mariozechner/pi-coding-agent": "^0.65.0",
+    "@mariozechner/pi-tui": "^0.65.0",
     "esbuild": "^0.28.0",
     "typescript": "^5.9.3"
   },

package/src/extension/register.ts

@@ -459,6 +459,7 @@ export function registerPiTeams(pi: ExtensionAPI): void {
 			if (manifest) void import("../state/event-log.ts").then(({ appendEventFireAndForget }) => appendEventFireAndForget(manifest.eventsPath, event as Parameters<typeof appendEventFireAndForget>[1]));
 		};
 		registry.waitForAll = async (runId: string) => {
+		// LAZY: state-store only needed for post-completion polling (waitForAll) and sync hasRunning check; avoid at startup.
 			const { loadRunManifestById } = await import("../state/state-store.ts");
 			const check = (): boolean => {
 				const loaded = loadRunManifestById(currentCtx?.cwd ?? process.cwd(), runId);
@@ -470,6 +471,7 @@ export function registerPiTeams(pi: ExtensionAPI): void {
 		registry.hasRunning = (runId: string) => {
 			const manifest = manifestCacheForRegistry.get(runId);
 			if (!manifest) return false;
+		// LAZY: state-store only needed in hasRunning; avoid at startup.
 			const { loadRunManifestById } = require("../state/state-store.ts");
 			const loaded = loadRunManifestById(currentCtx?.cwd ?? process.cwd(), runId);
 			if (!loaded) return false;

package/src/runtime/iteration-hooks.ts

@@ -69,11 +69,18 @@ const HOOK_TIMEOUT_MS = 30_000;
 export function isAllowedHookPath(hookPath: string): boolean {
 	if (!hookPath || hookPath.trim().length === 0) return false;
 	if (!path.isAbsolute(hookPath)) {
-		const normalized = path.normalize(hookPath);
+		// Use path.posix.normalize to ensure forward-slash normalization on all platforms.
+		// On Windows, path.normalize converts .hooks/hook.sh to .hooks\hook.sh (backslash),
+		// breaking the startsWith(".hooks/") check. path.posix.normalize always uses /.
+		const normalized = path.posix.normalize(hookPath);
 		return normalized === ".hooks" || normalized.startsWith(".hooks/");
 	}
-	const homeHooks = path.join(process.env.HOME ?? "", "", ".pi", "hooks");
-	return hookPath === homeHooks || hookPath.startsWith(homeHooks + path.sep);
+	// Normalize to forward slashes for consistent cross-platform comparison.
+	// e.g., "C:\\Users\\runner\\.pi\\hooks\\hook.sh" matches
+	// "C:\\Users\\runner\\.pi\\hooks/hook.sh" from path.join.
+	const normalizedHookPath = hookPath.replace(/\\/g, "/");
+	const homeHooksNormalized = (process.env.HOME ?? "").replace(/\\/g, "/") + "/.pi/hooks";
+	return normalizedHookPath === homeHooksNormalized || normalizedHookPath.startsWith(homeHooksNormalized + "/");
 }
 
 /**