@phi-code-admin/phi-code 0.86.0 → 0.88.0

package/extensions/phi/commit.ts

@@ -162,21 +162,22 @@ function deriveSubject(stagedFiles: string[], assistantHint: string): string {
 
 export default function commitExtension(pi: ExtensionAPI) {
 	pi.registerCommand("commit", {
-		description: "Deterministically commit staged changes ([phi] prefix, no LLM). Use --all to stage everything first.",
+		description:
+			"Deterministically commit staged changes ([phi] prefix, no LLM). Use --all to stage everything first.",
 		handler: async (args: string, ctx: ExtensionCommandContext) => {
 			try {
 				// 1. Parse args: extract the --all flag, keep the rest as the message.
 				const tokens = args.trim().split(/\s+/).filter(Boolean);
 				const stageAll = tokens.includes("--all");
-				const userMessage = tokens.filter((t) => t !== "--all").join(" ").trim();
+				const userMessage = tokens
+					.filter((t) => t !== "--all")
+					.join(" ")
+					.trim();
 
 				// 2. Confirm we are inside a git repo with changes.
 				const status = await pi.exec("git", ["status", "--porcelain"], { cwd: ctx.cwd });
 				if (status.code !== 0) {
-					ctx.ui.notify(
-						"Not a git repository (or git unavailable). Nothing to commit.",
-						"warning",
-					);
+					ctx.ui.notify("Not a git repository (or git unavailable). Nothing to commit.", "warning");
 					return;
 				}
 				if (status.stdout.trim().length === 0) {
@@ -197,10 +198,7 @@ export default function commitExtension(pi: ExtensionAPI) {
 				const afterStatus = await pi.exec("git", ["status", "--porcelain"], { cwd: ctx.cwd });
 				const stagedFiles = parseStagedFiles(afterStatus.stdout);
 				if (stagedFiles.length === 0) {
-					ctx.ui.notify(
-						"No staged changes to commit. Stage files first, or run `/commit --all`.",
-						"warning",
-					);
+					ctx.ui.notify("No staged changes to commit. Stage files first, or run `/commit --all`.", "warning");
 					return;
 				}
 
@@ -243,10 +241,7 @@ export default function commitExtension(pi: ExtensionAPI) {
 					"info",
 				);
 			} catch (err) {
-				ctx.ui.notify(
-					`/commit error: ${err instanceof Error ? err.message : String(err)}`,
-					"error",
-				);
+				ctx.ui.notify(`/commit error: ${err instanceof Error ? err.message : String(err)}`, "error");
 			}
 		},
 	});

package/extensions/phi/goal/index.ts

@@ -1,7 +1,6 @@
+import { randomUUID } from "node:crypto";
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
 import { dirname, join } from "node:path";
-import process from "node:process";
-import { randomUUID } from "node:crypto";
 import { defineTool, type ExtensionAPI, getAgentDir } from "phi-code";
 import { Type } from "typebox";
 
@@ -218,12 +217,7 @@ export default function goal(pi: ExtensionAPI) {
 	});
 }
 
-async function startGoal(
-	objective: string,
-	tokenBudget: number | undefined,
-	pi: ExtensionAPI,
-	ctx: StatusContext,
-) {
+async function startGoal(objective: string, tokenBudget: number | undefined, pi: ExtensionAPI, ctx: StatusContext) {
 	const validationError = validateObjective(objective);
 	if (validationError) {
 		ctx.ui.notify(validationError, "warning");
@@ -300,12 +294,7 @@ function clearGoal(ctx: StatusContext) {
 	ctx.ui.notify(`Goal cleared: ${stoppedGoal}`, "warning");
 }
 
-async function editGoal(
-	objective: string,
-	tokenBudget: number | undefined,
-	pi: ExtensionAPI,
-	ctx: StatusContext,
-) {
+async function editGoal(objective: string, tokenBudget: number | undefined, pi: ExtensionAPI, ctx: StatusContext) {
 	const validationError = validateObjective(objective);
 	if (validationError) {
 		ctx.ui.notify(validationError, "warning");
@@ -368,11 +357,7 @@ function editedGoalStatus(status: GoalStatus): GoalStatus {
 }
 
 function normalizeGoalForBudget(goal: ActiveGoal): ActiveGoal {
-	if (
-		goal.status === "active" &&
-		goal.tokenBudget !== undefined &&
-		goal.tokensUsed >= goal.tokenBudget
-	) {
+	if (goal.status === "active" && goal.tokenBudget !== undefined && goal.tokensUsed >= goal.tokenBudget) {
 		return { ...goal, status: "budget_limited" };
 	}
 	return goal;
@@ -382,11 +367,7 @@ function incrementGoal(goal: ActiveGoal): ActiveGoal {
 	return { ...goal, iteration: goal.iteration + 1, updatedAt: Date.now() };
 }
 
-function pauseGoalAfterAgentEnd(
-	ctx: StatusContext,
-	goal: ActiveGoal,
-	assistant: AssistantMessageLike,
-) {
+function pauseGoalAfterAgentEnd(ctx: StatusContext, goal: ActiveGoal, assistant: AssistantMessageLike) {
 	cancelContinuationPending();
 	activeGoal = transitionGoal(goal, "paused");
 	persistGoal(activeGoal);
@@ -582,7 +563,8 @@ function buildResumePrompt(goal: ActiveGoal) {
 }
 
 export function buildGoalSystemPrompt(goal: ActiveGoal) {
-	const budgetLine = goal.tokenBudget === undefined ? "" : `\n- Respect the goal token budget (${formatBudget(goal)} used).`;
+	const budgetLine =
+		goal.tokenBudget === undefined ? "" : `\n- Respect the goal token budget (${formatBudget(goal)} used).`;
 	return `Active /goal:\n${goalObjectiveBlock(goal)}\n\nGoal-mode rules:\n- Keep going until the active goal is completely resolved end-to-end.\n- Treat the current worktree, command output, tests, and external state as authoritative.\n- Do not redefine the goal into a smaller task; audit every requirement before completion.\n- Do not stop at analysis, a plan, TODO list, partial fixes, or suggested next steps.\n- Autonomously perform implementation and verification with the available tools when they are needed to complete the goal.\n- Persevere through recoverable tool failures by trying reasonable alternatives instead of yielding early.\n- If the goal is not complete at the end of a turn, expect an automatic continuation and keep working from where you left off.\n- Only call the goal_complete tool after the goal is fully complete and verified.${budgetLine}`;
 }
 
@@ -709,12 +691,10 @@ function loadGoalFromSession(ctx: StatusContext): ActiveGoal | undefined {
 		| {
 				getBranch?: () => Array<{ type?: string; customType?: string; data?: unknown }>;
 				getEntries?: () => Array<{ type?: string; customType?: string; data?: unknown }>;
-			}
+		  }
 		| undefined;
 	const entries = sessionManager?.getBranch?.() ?? sessionManager?.getEntries?.() ?? [];
-	const entry = entries
-		.filter((entry) => entry.type === "custom" && entry.customType === GOAL_STATE_ENTRY_TYPE)
-		.pop();
+	const entry = entries.filter((entry) => entry.type === "custom" && entry.customType === GOAL_STATE_ENTRY_TYPE).pop();
 	const data = entry?.data as GoalStateEntryData | undefined;
 	return isGoal(data?.goal) && data.goal.status !== "complete" ? data.goal : undefined;
 }
@@ -750,9 +730,7 @@ function readState(): Record<string, unknown> {
 	if (!existsSync(STATE_FILE)) return {};
 	try {
 		const parsed = JSON.parse(readFileSync(STATE_FILE, "utf8")) as unknown;
-		return parsed && typeof parsed === "object" && !Array.isArray(parsed)
-			? (parsed as Record<string, unknown>)
-			: {};
+		return parsed && typeof parsed === "object" && !Array.isArray(parsed) ? (parsed as Record<string, unknown>) : {};
 	} catch {
 		return {};
 	}
@@ -766,7 +744,6 @@ function clearLegacyPersistedGoal(cwd: string) {
 	writeFileSync(STATE_FILE, `${JSON.stringify(goals, null, 2)}\n`);
 }
 
-
 function isGoal(value: unknown): value is ActiveGoal {
 	if (!value || typeof value !== "object") return false;
 	const goal = value as Partial<ActiveGoal>;

package/extensions/phi/init.ts

@@ -17,11 +17,11 @@
  *    so a network failure on /v1/models cannot lose the user's input.
  */
 
-import type { ExtensionAPI } from "phi-code";
-import { writeFile, mkdir, copyFile, readdir, readFile, chmod } from "node:fs/promises";
-import { join, resolve } from "node:path";
-import { homedir } from "node:os";
 import { existsSync } from "node:fs";
+import { chmod, copyFile, mkdir, readdir, readFile, writeFile } from "node:fs/promises";
+import { homedir } from "node:os";
+import { join, resolve } from "node:path";
+import type { ExtensionAPI } from "phi-code";
 import { fetchLiveModels, pingProvider, toPersistedModel } from "./providers/live-models.js";
 import { isOpenCodeGoAnthropicModel, OPENCODE_GO_ANTHROPIC_BASE_URL } from "./providers/opencode-go.js";
 
@@ -218,12 +218,30 @@ function getAllAvailableModels(providers: DetectedProvider[]): Array<{ ref: stri
 // ─── Routing Presets ─────────────────────────────────────────────────────
 
 const TASK_ROLES = [
-	{ key: "code", label: "Code Generation", desc: "Writing and modifying code", agent: "code", defaultModel: "default" },
+	{
+		key: "code",
+		label: "Code Generation",
+		desc: "Writing and modifying code",
+		agent: "code",
+		defaultModel: "default",
+	},
 	{ key: "debug", label: "Debugging", desc: "Finding and fixing bugs", agent: "code", defaultModel: "default" },
 	{ key: "plan", label: "Planning", desc: "Architecture and design", agent: "plan", defaultModel: "default" },
-	{ key: "explore", label: "Exploration", desc: "Code reading and analysis", agent: "explore", defaultModel: "default" },
+	{
+		key: "explore",
+		label: "Exploration",
+		desc: "Code reading and analysis",
+		agent: "explore",
+		defaultModel: "default",
+	},
 	{ key: "test", label: "Testing", desc: "Running and writing tests", agent: "test", defaultModel: "default" },
-	{ key: "review", label: "Code Review", desc: "Quality and security review", agent: "review", defaultModel: "default" },
+	{
+		key: "review",
+		label: "Code Review",
+		desc: "Quality and security review",
+		agent: "review",
+		defaultModel: "default",
+	},
 ] as const;
 
 const KEYWORDS: Record<string, string[]> = {
@@ -235,9 +253,7 @@ const KEYWORDS: Record<string, string[]> = {
 	review: ["review", "audit", "quality", "security", "improve", "optimize"],
 };
 
-function createRouting(
-	assignments: Record<string, { preferred: string; fallback: string }>,
-): RoutingConfig {
+function createRouting(assignments: Record<string, { preferred: string; fallback: string }>): RoutingConfig {
 	const routes: RoutingConfig["routes"] = {};
 	for (const role of TASK_ROLES) {
 		const assignment = assignments[role.key];
@@ -251,7 +267,7 @@ function createRouting(
 	}
 	return {
 		routes,
-		default: { model: assignments["default"]?.preferred || "default", agent: null },
+		default: { model: assignments.default?.preferred || "default", agent: null },
 	};
 }
 
@@ -358,10 +374,7 @@ _Edit this file to customize Phi Code's behavior for your project._
 		}
 	}
 
-	async function persistProviderKey(
-		provider: DetectedProvider,
-		apiKey: string,
-	): Promise<void> {
+	async function persistProviderKey(provider: DetectedProvider, apiKey: string): Promise<void> {
 		const config = await readModelsConfig();
 		const existing = config.providers[provider.id] ?? {};
 		config.providers[provider.id] = {
@@ -448,7 +461,7 @@ _Edit this file to customize Phi Code's behavior for your project._
 
 		// Orchestrator fallback (used only when a specific route has no model).
 		// This is NOT the chat default — `/model` controls that.
-		assignments["default"] = {
+		assignments.default = {
 			preferred: "default",
 			fallback: availableModels[0]?.ref || "default",
 		};
@@ -464,10 +477,7 @@ _Edit this file to customize Phi Code's behavior for your project._
 			if (result.source === "live" && result.models.length > 0) {
 				provider.models = result.models.map((m) => m.id);
 				provider.available = true;
-				ctx.ui.notify(
-					`${provider.name} is running with ${provider.models.length} model(s).\n`,
-					"info",
-				);
+				ctx.ui.notify(`${provider.name} is running with ${provider.models.length} model(s).\n`, "info");
 			} else {
 				ctx.ui.notify(
 					`${provider.name} not reachable on port ${port}. Start it and re-run \`/phi-init\`.\n`,
@@ -483,10 +493,7 @@ _Edit this file to customize Phi Code's behavior for your project._
 			"info",
 		);
 
-		const apiKey = await ctx.ui.input(
-			`Enter your ${provider.name} API key`,
-			"Paste your key here",
-		);
+		const apiKey = await ctx.ui.input(`Enter your ${provider.name} API key`, "Paste your key here");
 
 		if (apiKey === undefined) {
 			ctx.ui.notify("Cancelled. No key saved.", "warning");
@@ -632,10 +639,7 @@ _Edit this file to customize Phi Code's behavior for your project._
 							return `${status} ${p.name}${tag}${modelCount}`;
 						}),
 					];
-					const addProvider = await ctx.ui.select(
-						"Configure a provider (add multiple!):",
-						providerOptions,
-					);
+					const addProvider = await ctx.ui.select("Configure a provider (add multiple!):", providerOptions);
 
 					const choiceIdx = providerOptions.indexOf(addProvider ?? "");
 					if (choiceIdx <= 0) {
@@ -657,18 +661,12 @@ _Edit this file to customize Phi Code's behavior for your project._
 
 				const available = providers.filter((p) => p.available);
 				if (available.length === 0) {
-					ctx.ui.notify(
-						"No providers available. Run `/phi-init` again after setting up a provider.",
-						"error",
-					);
+					ctx.ui.notify("No providers available. Run `/phi-init` again after setting up a provider.", "error");
 					return;
 				}
 
 				const allModels = getAllAvailableModels(providers);
-				ctx.ui.notify(
-					`\n${allModels.length} models available from ${available.length} provider(s).\n`,
-					"info",
-				);
+				ctx.ui.notify(`\n${allModels.length} models available from ${available.length} provider(s).\n`, "info");
 
 				// 2. Assign models to agents
 				ctx.ui.notify("Assign a model to each agent role:\n", "info");
@@ -680,11 +678,7 @@ _Edit this file to customize Phi Code's behavior for your project._
 
 				ctx.ui.notify("Writing routing configuration...", "info");
 				const routing = createRouting(assignments);
-				await writeFile(
-					join(agentDir, "routing.json"),
-					JSON.stringify(routing, null, 2),
-					"utf-8",
-				);
+				await writeFile(join(agentDir, "routing.json"), JSON.stringify(routing, null, 2), "utf-8");
 
 				ctx.ui.notify("Setting up sub-agents...", "info");
 				await copyBundledAgents();
@@ -702,10 +696,7 @@ _Edit this file to customize Phi Code's behavior for your project._
 					const a = assignments[role.key];
 					ctx.ui.notify(`  ${role.label}: \`${a.preferred}\` (fallback: \`${a.fallback}\`)`, "info");
 				}
-				ctx.ui.notify(
-					"\nChat default model: use `/model` (this wizard does NOT change the chat default).",
-					"info",
-				);
+				ctx.ui.notify("\nChat default model: use `/model` (this wizard does NOT change the chat default).", "info");
 				ctx.ui.notify("\nNext steps:", "info");
 				ctx.ui.notify("  - `/model` to pick the chat default model (sticky across prompts)", "info");
 				ctx.ui.notify("  - `/plan <description>` to run the orchestrator with the roles above", "info");
@@ -729,8 +720,7 @@ _Edit this file to customize Phi Code's behavior for your project._
 		description: "Reconfigure the per-role models used by /plan (provider-qualified, cross-provider)",
 		handler: async (_args, ctx) => {
 			try {
-				const registryModels: Array<{ provider?: string; id?: string }> =
-					ctx.modelRegistry?.getAvailable?.() || [];
+				const registryModels: Array<{ provider?: string; id?: string }> = ctx.modelRegistry?.getAvailable?.() || [];
 				const available: Array<{ ref: string; display: string }> = [];
 				const seen = new Set<string>();
 				for (const m of registryModels) {

package/extensions/phi/keys.ts

@@ -22,9 +22,7 @@
 
 import { ApiKeyStore, type ExtensionAPI, getApiKeyStore, getConfigWatcher } from "phi-code";
 
-interface ProviderPingFn {
-	(key: string, timeoutMs?: number): Promise<{ ok: boolean; error?: string }>;
-}
+type ProviderPingFn = (key: string, timeoutMs?: number) => Promise<{ ok: boolean; error?: string }>;
 
 const PROVIDER_PING_REGISTRY: Record<string, ProviderPingFn> = {};
 
@@ -152,10 +150,7 @@ export default function keysExtension(pi: ExtensionAPI) {
 					return;
 				}
 
-				ctx.ui.notify(
-					"Unknown subcommand. Use: `/keys [list|set|remove|test|reload]`",
-					"warning",
-				);
+				ctx.ui.notify("Unknown subcommand. Use: `/keys [list|set|remove|test|reload]`", "warning");
 			} catch (err) {
 				ctx.ui.notify(`/keys error: ${err instanceof Error ? err.message : String(err)}`, "error");
 			}