pi-crew 0.9.16 → 0.9.17

package/src/extension/message-renderers.ts

@@ -7,8 +7,9 @@
  * raw JSON in the conversation. These renderers give them a clean,
  * crew-branded look using the active theme.
  */
-import { Text } from "@earendil-works/pi-tui";
+
 import type { ExtensionAPI, MessageRenderOptions, Theme } from "@earendil-works/pi-coding-agent";
+import { Text } from "@earendil-works/pi-tui";
 
 interface CrewMessageDetails {
 	runId?: string;
@@ -96,9 +97,7 @@ export function renderResumeDirective(message: MessageLike, _options: MessageRen
 }
 
 /** Register all crew message renderers. Safe to call once at extension load. */
-export function registerCrewMessageRenderers(
-	pi: { registerMessageRenderer?: ExtensionAPI["registerMessageRenderer"] },
-): void {
+export function registerCrewMessageRenderers(pi: { registerMessageRenderer?: ExtensionAPI["registerMessageRenderer"] }): void {
 	// Optional chaining guards against older Pi versions (and test stubs)
 	// without registerMessageRenderer.
 	// The renderers return Text (a Component) — cast through never to match

package/src/extension/notification-router.ts

@@ -22,9 +22,17 @@ export interface NotificationRouterOptions {
 }
 
 const DEFAULT_SEVERITY_FILTER: Severity[] = ["warning", "error", "critical"];
-const SEVERITY_RANK: Record<Severity, number> = { info: 0, warning: 1, error: 2, critical: 3 };
+const SEVERITY_RANK: Record<Severity, number> = {
+	info: 0,
+	warning: 1,
+	error: 2,
+	critical: 3,
+};
 
-export function parseHHMMRange(range: string): { startMin: number; endMin: number } {
+export function parseHHMMRange(range: string): {
+	startMin: number;
+	endMin: number;
+} {
 	const match = /^(\d{2}):(\d{2})-(\d{2}):(\d{2})$/.exec(range);
 	if (!match) throw new Error(`Invalid quiet-hours range '${range}'. Expected HH:MM-HH:MM.`);
 	const [, sh, sm, eh, em] = match;
@@ -33,7 +41,10 @@ export function parseHHMMRange(range: string): { startMin: number; endMin: numbe
 	const endHour = Number(eh);
 	const endMinute = Number(em);
 	if (startHour > 23 || endHour > 23 || startMinute > 59 || endMinute > 59) throw new Error(`Invalid quiet-hours range '${range}'.`);
-	return { startMin: startHour * 60 + startMinute, endMin: endHour * 60 + endMinute };
+	return {
+		startMin: startHour * 60 + startMinute,
+		endMin: endHour * 60 + endMinute,
+	};
 }
 
 export function isInQuietHours(range: string, now = new Date()): boolean {
@@ -48,7 +59,10 @@ function notificationKey(notification: NotificationDescriptor): string {
 }
 
 function batchSeverity(items: NotificationDescriptor[]): Severity {
-	return items.reduce((highest, item) => SEVERITY_RANK[item.severity] > SEVERITY_RANK[highest] ? item.severity : highest, "info" as Severity);
+	return items.reduce(
+		(highest, item) => (SEVERITY_RANK[item.severity] > SEVERITY_RANK[highest] ? item.severity : highest),
+		"info" as Severity,
+	);
 }
 
 export class NotificationRouter {
@@ -81,7 +95,10 @@ export class NotificationRouter {
 
 	enqueue(notification: NotificationDescriptor): boolean {
 		const now = this.opts.now?.() ?? Date.now();
-		const withTime = { ...notification, timestamp: notification.timestamp ?? now };
+		const withTime = {
+			...notification,
+			timestamp: notification.timestamp ?? now,
+		};
 		try {
 			this.opts.sink?.(withTime);
 		} catch (sinkError) {

package/src/extension/notification-sink.ts

@@ -1,8 +1,8 @@
 import * as fs from "node:fs";
 import * as path from "node:path";
-import type { NotificationDescriptor } from "./notification-router.ts";
-import { redactSecrets } from "../utils/redaction.ts";
 import { logInternalError } from "../utils/internal-error.ts";
+import { redactSecrets } from "../utils/redaction.ts";
+import type { NotificationDescriptor } from "./notification-router.ts";
 
 export interface NotificationSink {
 	write(notification: NotificationDescriptor): void;
@@ -36,7 +36,10 @@ export function createJsonlSink(crewRoot: string, retentionDays = 7): Notificati
 					lastRotateDate = date;
 				}
 				fs.mkdirSync(dir, { recursive: true });
-				const payload = redactSecrets({ ...notification, timestamp }) as NotificationDescriptor;
+				const payload = redactSecrets({
+					...notification,
+					timestamp,
+				}) as NotificationDescriptor;
 				fs.appendFileSync(path.join(dir, `${date}.jsonl`), `${JSON.stringify(payload)}\n`, "utf-8");
 			} catch (error) {
 				logInternalError("notification-sink.write", error);

package/src/extension/pi-api.ts

@@ -29,19 +29,28 @@
  * and centralizes the type-level surface.
  */
 export type {
+	AgentSessionEvent,
+	BeforeAgentStartEvent,
 	ExtensionAPI,
-	ExtensionContext,
 	ExtensionCommandContext,
+	ExtensionContext,
 	ToolDefinition,
-	AgentSessionEvent,
-	BeforeAgentStartEvent,
 } from "@earendil-works/pi-coding-agent";
 
-export { defineTool, createBashTool } from "@earendil-works/pi-coding-agent";
+export { createBashTool, defineTool } from "@earendil-works/pi-coding-agent";
 
 /**
- * The Pi package version pi-crew was built against. Used for diagnostics
- * and to surface version-drift if Pi upgrades introduce breaking changes.
- * Update this when bumping the pi-coding-agent dependency.
+ * @deprecated Drift detector removed in Phase 5 follow-up to H1.
+ *
+ * History: this constant was meant to surface version-drift between
+ * dev-time type-check and runtime peer-dep version. pi-crew declares
+ * peer deps as `*` so the runtime version is set by whatever host pi
+ * install the user has — pinning this constant to a specific devDep
+ * range was a false invariant that did not reflect how extensions
+ * actually consume peer packages.
+ *
+ * Kept exported (now `"0.0.0-unset"`) only for any downstream consumers
+ * that may import it via re-exports. No diagnostic uses it anymore.
+ * Will be removed in a future major version.
  */
-export const BUILT_AGAINST_PI_VERSION = "0.77.0";
+export const BUILT_AGAINST_PI_VERSION = "0.0.0-unset";

package/src/extension/plan-orchestrate.ts

@@ -102,10 +102,7 @@ function parsePlanDocumentContent(content: string): OrchestratedStep[] {
 		// Find the heading by looking back from the tag position.
 		// Use global regex to find ALL headings in the window, then take the LAST
 		// (nearest to the tag) — simple `.match()` only finds the FIRST heading.
-		const textBeforeTag = content.slice(
-			Math.max(0, current.start - 500),
-			current.start,
-		);
+		const textBeforeTag = content.slice(Math.max(0, current.start - 500), current.start);
 		const headingRegex = /(^|\n)(#{1,6})\s+(.+?)(\n|$)/g;
 		let lastHeadingMatch: RegExpExecArray | null = null;
 		let hm: RegExpExecArray | null;
@@ -120,10 +117,7 @@ function parsePlanDocumentContent(content: string): OrchestratedStep[] {
 
 		// Find the section content: capture until next heading (##) or next tag
 		const sectionEndMatch = afterTagContent.search(/(^|\n)(#{1,6}\s|\n<!--\s*tag:)/m);
-		const sectionContent =
-			sectionEndMatch >= 0
-				? afterTagContent.slice(0, sectionEndMatch)
-				: afterTagContent;
+		const sectionContent = sectionEndMatch >= 0 ? afterTagContent.slice(0, sectionEndMatch) : afterTagContent;
 
 		// Extract prompt text - remove the tag comment lines and empty lines
 		const promptLines: string[] = [];
@@ -191,8 +185,7 @@ function detectImplicitTag(content: string): string | undefined {
 	const lowerContent = content.toLowerCase();
 	// Use word-boundary regex to avoid substring false matches.
 	// E.g., "implementation" must NOT trigger "impl" — only the word "implement" should.
-	const hasWord = (word: string): boolean =>
-		new RegExp(`\\b${word}\\b`).test(lowerContent);
+	const hasWord = (word: string): boolean => new RegExp(`\\b${word}\\b`).test(lowerContent);
 
 	if (hasWord("design") || hasWord("architecture")) return "design";
 	if (hasWord("implement") || hasWord("coding")) return "impl";
@@ -224,9 +217,7 @@ export function buildAgentChain(steps: OrchestratedStep[]): string[] {
  * @param steps - Array of OrchestratedStep to convert.
  * @returns Array of chain objects with step data and commands.
  */
-export function buildChainData(
-	steps: OrchestratedStep[],
-): Array<{
+export function buildChainData(steps: OrchestratedStep[]): Array<{
 	step: OrchestratedStep;
 	commands: string[];
 }> {
@@ -258,10 +249,7 @@ export function formatPlanOverview(planPath: string): string {
 }
 
 function formatStepsOverview(steps: OrchestratedStep[]): string {
-	const lines: string[] = [
-		`Plan Orchestration: ${steps.length} step(s)`,
-		"",
-	];
+	const lines: string[] = [`Plan Orchestration: ${steps.length} step(s)`, ""];
 
 	const tagCounts: Record<string, number> = {};
 	for (const step of steps) {
@@ -276,15 +264,10 @@ function formatStepsOverview(steps: OrchestratedStep[]): string {
 
 	lines.push("", "Steps:");
 	for (const step of steps) {
-		const preview =
-			step.prompt.length > 60
-				? step.prompt.slice(0, 60) + "..."
-				: step.prompt;
+		const preview = step.prompt.length > 60 ? step.prompt.slice(0, 60) + "..." : step.prompt;
 		// Include heading if available (e.g., "Security Review" from the plan's heading)
 		const headingPrefix = step.heading ? `${step.heading}: ` : "";
-		lines.push(
-			`  ${step.stepId} [${step.tag}] ${step.chain.join(",")}: ${headingPrefix}${preview}`,
-		);
+		lines.push(`  ${step.stepId} [${step.tag}] ${step.chain.join(",")}: ${headingPrefix}${preview}`);
 	}
 
 	return lines.join("\n");
@@ -293,9 +276,7 @@ function formatStepsOverview(steps: OrchestratedStep[]): string {
 /**
  * Main orchestration function that parses a plan and returns pre-formatted output.
  */
-export async function orchestratePlan(
-	options: OrchestrateOptions,
-): Promise<{
+export async function orchestratePlan(options: OrchestrateOptions): Promise<{
 	steps: OrchestratedStep[];
 	chain: string[];
 	overview: string;

package/src/extension/project-init.ts

@@ -70,7 +70,13 @@ const DEFAULT_PI_CREW_CONFIG = {
 	},
 };
 
-function copyBuiltinDir(kind: "agents" | "teams" | "workflows", targetDir: string, overwrite: boolean, copiedFiles: string[], skippedFiles: string[]): void {
+function copyBuiltinDir(
+	kind: "agents" | "teams" | "workflows",
+	targetDir: string,
+	overwrite: boolean,
+	copiedFiles: string[],
+	skippedFiles: string[],
+): void {
 	const sourceDir = path.join(packageRoot(), kind);
 	if (!fs.existsSync(sourceDir)) return;
 	for (const entry of fs.readdirSync(sourceDir)) {
@@ -97,7 +103,8 @@ export function initializeProject(cwd: string, options: ProjectInitOptions = {})
 	const teamsDir = path.join(crewRoot, "teams");
 	const workflowsDir = path.join(crewRoot, "workflows");
 	const configScope = options.configScope ?? "global";
-	const configPath = configScope === "project" ? path.join(projectPiRoot(cwd), "pi-crew.json") : configScope === "global" ? globalConfigPath() : "";
+	const configPath =
+		configScope === "project" ? path.join(projectPiRoot(cwd), "pi-crew.json") : configScope === "global" ? globalConfigPath() : "";
 	ensureDir(agentsDir, createdDirs);
 	ensureDir(teamsDir, createdDirs);
 	ensureDir(workflowsDir, createdDirs);
@@ -124,9 +131,7 @@ export function initializeProject(cwd: string, options: ProjectInitOptions = {})
 
 	const ignoreMethod = options.ignoreMethod ?? "gitignore";
 	const desired = [`${ignorePrefix}/state/`, `${ignorePrefix}/artifacts/`, `${ignorePrefix}/worktrees/`, `${ignorePrefix}/imports/`];
-	const gitignorePath = ignoreMethod === "exclude"
-		? path.join(cwd, ".git", "info", "exclude")
-		: path.join(cwd, ".gitignore");
+	const gitignorePath = ignoreMethod === "exclude" ? path.join(cwd, ".git", "info", "exclude") : path.join(cwd, ".gitignore");
 	let gitignoreUpdated = false;
 	if (ignoreMethod === "exclude") {
 		// Ensure .git/info/ directory exists
@@ -151,5 +156,15 @@ export function initializeProject(cwd: string, options: ProjectInitOptions = {})
 	// learns pi-crew's commands from tool registration, not AGENTS.md.
 	// `team action=cleanup` still removes any block injected by older versions.
 
-	return { createdDirs, copiedFiles, skippedFiles, gitignorePath, gitignoreUpdated, configPath, configScope, configCreated, configSkipped };
+	return {
+		createdDirs,
+		copiedFiles,
+		skippedFiles,
+		gitignorePath,
+		gitignoreUpdated,
+		configPath,
+		configScope,
+		configCreated,
+		configSkipped,
+	};
 }