pi-goala 0.3.0 → 0.3.1

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 0.3.1 (2026-07-26)
+
+- Make each phase default to keeping its current model configuration.
+- Separate reasoning-effort changes from provider/model changes.
+- Select provider, then a filtered model, while skipping provider selection
+  when only one authenticated provider is available.
+- Review, edit, save, or cancel the complete configuration atomically.
+
 ## 0.3.0 (2026-07-26)
 
 - Add interactive per-phase provider, model, and reasoning selection from Pi's

package/README.md

@@ -39,7 +39,7 @@ pi install npm:pi-goala
 Pin a release for reproducible team installations:
 
 ```text
-pi install npm:pi-goala@0.3.0
+pi install npm:pi-goala@0.3.1
 ```
 
 Pi's package command both downloads Goala and registers its extension. A plain
@@ -49,7 +49,7 @@ supported installation path.
 From GitHub or a local checkout:
 
 ```text
-pi install git:github.com/barryking/pi-goala@v0.3.0
+pi install git:github.com/barryking/pi-goala@v0.3.1
 pi install /absolute/path/to/pi-goala
 ```
 
@@ -115,7 +115,7 @@ Useful commands:
 /memory retire <id>     Exclude an obsolete episode from recall
 /memory restore <id>    Restore a retired episode
 /goala-setup            Choose a model preset
-/goala-setup custom     Choose an available provider/model for every role
+/goala-setup custom     Review or change each role's model and reasoning
 /goala-setup status     Show effective configuration
 ```
 
@@ -383,8 +383,11 @@ when Pi started. Run `/goala-setup current` to persist that portable
 single-model configuration. Run `/goala-setup custom` to choose an
 authenticated provider/model and reasoning level separately for planning,
 execution, step verification, final verification, and repeated repair. The
-interactive picker reads Pi's available model registry and saves only after
-every role has been selected.
+interactive wizard defaults to keeping each current role. It can change
+reasoning alone or select a provider followed by one of that provider's
+available models. Provider selection is skipped when only one is authenticated.
+A final summary can be edited, saved, or cancelled; nothing is written before
+save.
 
 Advanced users can edit the same per-role profiles in
 `~/.pi/agent/pi-goala/config.json`.

package/docs/architecture.md

@@ -124,9 +124,10 @@ step verification, and a balanced fallback after repeated repair. Independent
 step context and the final strong verifier preserve the trust boundary without
 paying the strongest-model cost at every human checkpoint. A portable
 current-model preset is available for other providers. The advanced setup
-enumerates Pi's authenticated available models and stores provider, model, and
-reasoning independently for every role; mixed-provider workflows therefore use
-the same phase-routing path as the bundled preset.
+defaults to retaining each role, supports reasoning-only changes, and filters
+models after provider selection. It stores provider, model, and reasoning
+independently for every role only after final review; mixed-provider workflows
+therefore use the same phase-routing path as the bundled preset.
 
 If a configured model cannot be resolved and current-model fallback is
 enabled, Goala uses the model that was active when the extension session

package/docs/configuration.md

@@ -51,9 +51,12 @@ and restore the backup to `~/.pi/agent/`.
 
 `openai` selects the tested Sol/Luna/Terra model split when those models are
 available. `current` assigns the model active at Pi startup to every role.
-`custom` interactively selects a provider/model and reasoning level for each
-role from Pi's authenticated, available model registry. Cancelling any prompt
-leaves the existing configuration unchanged.
+`custom` reviews each role and defaults to keeping its current configuration.
+For a change, it can adjust reasoning alone or select a provider followed by
+one of that provider's authenticated, available models. Provider selection is
+skipped when only one provider is available. The final summary can be edited,
+saved, or cancelled. Cancelling any prompt leaves the existing configuration
+unchanged.
 
 ## Full schema
 

package/extensions/goala/index.ts

@@ -605,7 +605,7 @@ Do not edit files or rely on executor claims. Finish with submit_step_verificati
 		);
 		if (missing.length > 0) {
 			ctx.ui.notify(
-				`Preset not saved; unavailable model(s): ${missing.map((model) => `${model.provider}/${model.id}`).join(", ")}.`,
+				`Configuration not saved; unavailable model(s): ${missing.map((model) => `${model.provider}/${model.id}`).join(", ")}.`,
 				"warning",
 			);
 			return;

package/extensions/goala/setup.ts

@@ -12,6 +12,10 @@ type ModelRole =
 	| "verifier"
 	| "fallbackExecutor";
 
+type AvailableModel = ReturnType<
+	ExtensionCommandContext["modelRegistry"]["getAvailable"]
+>[number];
+
 const MODEL_ROLES: ReadonlyArray<{ key: ModelRole; label: string }> = [
 	{ key: "planner", label: "Planner" },
 	{ key: "executor", label: "Executor" },
@@ -30,8 +34,226 @@ const THINKING_LEVELS: readonly ThinkingLevel[] = [
 	"max",
 ];
 
-function profileLabel(profile: ModelProfile): string {
-	return `${profile.provider}/${profile.model}:${profile.thinkingLevel}`;
+const KEEP_CURRENT = "Keep current";
+const CHANGE_REASONING = "Change reasoning effort";
+const CHANGE_MODEL = "Change provider or model";
+const SAVE_CONFIGURATION = "Save configuration";
+const EDIT_PHASE = "Edit a phase";
+const CANCEL = "Cancel";
+
+function titleCase(value: string): string {
+	return `${value.charAt(0).toUpperCase()}${value.slice(1)}`;
+}
+
+function thinkingLabel(level: ThinkingLevel): string {
+	if (level === "off") return "Off";
+	if (level === "xhigh") return "Extra high";
+	if (level === "max") return "Maximum";
+	return titleCase(level);
+}
+
+function providerLabel(
+	ctx: ExtensionCommandContext,
+	provider: string,
+): string {
+	const displayName = ctx.modelRegistry.getProviderDisplayName(provider);
+	return displayName === provider ? provider : `${displayName} (${provider})`;
+}
+
+function modelLabel(model: AvailableModel): string {
+	return model.name && model.name !== model.id
+		? `${model.name} (${model.id})`
+		: model.id;
+}
+
+function profileLabel(
+	ctx: ExtensionCommandContext,
+	available: AvailableModel[],
+	profile: ModelProfile,
+): string {
+	const model = available.find(
+		(candidate) =>
+			candidate.provider === profile.provider && candidate.id === profile.model,
+	);
+	const reasoning =
+		profile.thinkingLevel === "off"
+			? "Reasoning off"
+			: `${thinkingLabel(profile.thinkingLevel)} reasoning`;
+	return [
+		providerLabel(ctx, profile.provider),
+		model ? modelLabel(model) : `${profile.model} (unavailable)`,
+		reasoning,
+	].join(" / ");
+}
+
+function setProfile(
+	config: GoalaConfig,
+	role: ModelRole,
+	profile: ModelProfile,
+): void {
+	if (role === "fallbackExecutor") {
+		config.fallbackExecutor = {
+			...profile,
+			afterRepairCycle: config.fallbackExecutor.afterRepairCycle,
+		};
+		return;
+	}
+	config[role] = profile;
+}
+
+async function selectThinkingLevel(
+	ctx: ExtensionCommandContext,
+	roleLabel: string,
+	profile: ModelProfile,
+): Promise<ThinkingLevel | undefined> {
+	const ordered = [
+		profile.thinkingLevel,
+		...THINKING_LEVELS.filter((level) => level !== profile.thinkingLevel),
+	];
+	const choices = ordered.map((level, index) =>
+		index === 0 ? `${thinkingLabel(level)} (current)` : thinkingLabel(level),
+	);
+	const selected = await ctx.ui.select(
+		`${roleLabel} reasoning effort\nCurrent: ${thinkingLabel(profile.thinkingLevel)}`,
+		choices,
+	);
+	if (!selected) return undefined;
+	return ordered[choices.indexOf(selected)];
+}
+
+async function selectProvider(
+	ctx: ExtensionCommandContext,
+	available: AvailableModel[],
+	roleLabel: string,
+	currentProvider: string,
+): Promise<string | undefined> {
+	const providers = [...new Set(available.map((model) => model.provider))].sort(
+		(left, right) => {
+			if (left === currentProvider) return -1;
+			if (right === currentProvider) return 1;
+			return providerLabel(ctx, left).localeCompare(providerLabel(ctx, right));
+		},
+	);
+	if (providers.length === 1) return providers[0];
+
+	const choices = providers.map((provider, index) =>
+		index === 0 && provider === currentProvider
+			? `${providerLabel(ctx, provider)} (current)`
+			: providerLabel(ctx, provider),
+	);
+	const selected = await ctx.ui.select(
+		`${roleLabel} provider\nCurrent: ${providerLabel(ctx, currentProvider)}`,
+		choices,
+	);
+	if (!selected) return undefined;
+	return providers[choices.indexOf(selected)];
+}
+
+async function selectModel(
+	ctx: ExtensionCommandContext,
+	available: AvailableModel[],
+	roleLabel: string,
+	provider: string,
+	current: ModelProfile,
+): Promise<AvailableModel | undefined> {
+	const models = available
+		.filter((model) => model.provider === provider)
+		.sort((left, right) => {
+			const leftCurrent =
+				left.provider === current.provider && left.id === current.model;
+			const rightCurrent =
+				right.provider === current.provider && right.id === current.model;
+			if (leftCurrent !== rightCurrent) return leftCurrent ? -1 : 1;
+			return modelLabel(left).localeCompare(modelLabel(right));
+		});
+	const choices = models.map((model, index) =>
+		index === 0 &&
+		model.provider === current.provider &&
+		model.id === current.model
+			? `${modelLabel(model)} (current)`
+			: modelLabel(model),
+	);
+	const selected = await ctx.ui.select(
+		`${roleLabel} model\nProvider: ${providerLabel(ctx, provider)}`,
+		choices,
+	);
+	if (!selected) return undefined;
+	return models[choices.indexOf(selected)];
+}
+
+async function changeModel(
+	ctx: ExtensionCommandContext,
+	available: AvailableModel[],
+	roleLabel: string,
+	current: ModelProfile,
+): Promise<ModelProfile | undefined> {
+	const provider = await selectProvider(
+		ctx,
+		available,
+		roleLabel,
+		current.provider,
+	);
+	if (!provider) return undefined;
+	const model = await selectModel(ctx, available, roleLabel, provider, current);
+	if (!model) return undefined;
+
+	const profile: ModelProfile = {
+		provider: model.provider,
+		model: model.id,
+		thinkingLevel: model.reasoning ? current.thinkingLevel : "off",
+	};
+	if (!model.reasoning) return profile;
+
+	const thinkingLevel = await selectThinkingLevel(ctx, roleLabel, profile);
+	if (!thinkingLevel) return undefined;
+	return { ...profile, thinkingLevel };
+}
+
+async function configureRole(
+	ctx: ExtensionCommandContext,
+	available: AvailableModel[],
+	config: GoalaConfig,
+	role: { key: ModelRole; label: string },
+): Promise<boolean> {
+	const current = config[role.key];
+	const currentModel = available.find(
+		(model) =>
+			model.provider === current.provider && model.id === current.model,
+	);
+	const actions = [
+		KEEP_CURRENT,
+		...(currentModel?.reasoning ? [CHANGE_REASONING] : []),
+		CHANGE_MODEL,
+	];
+	const action = await ctx.ui.select(
+		`${role.label}\nCurrent: ${profileLabel(ctx, available, current)}`,
+		actions,
+	);
+	if (!action) return false;
+	if (action === KEEP_CURRENT) return true;
+
+	if (action === CHANGE_REASONING) {
+		const thinkingLevel = await selectThinkingLevel(ctx, role.label, current);
+		if (!thinkingLevel) return false;
+		setProfile(config, role.key, { ...current, thinkingLevel });
+		return true;
+	}
+
+	const profile = await changeModel(ctx, available, role.label, current);
+	if (!profile) return false;
+	setProfile(config, role.key, profile);
+	return true;
+}
+
+function reviewSummary(
+	ctx: ExtensionCommandContext,
+	available: AvailableModel[],
+	config: GoalaConfig,
+): string {
+	return MODEL_ROLES.map(
+		(role) =>
+			`${role.label}: ${profileLabel(ctx, available, config[role.key])}`,
+	).join("\n");
 }
 
 export async function selectModelRoles(
@@ -59,51 +281,25 @@ export async function selectModelRoles(
 
 	const proposed = structuredClone(current);
 	for (const role of MODEL_ROLES) {
-		const existing = proposed[role.key];
-		const ordered = [...available].sort((left, right) => {
-			const leftCurrent =
-				left.provider === existing.provider && left.id === existing.model;
-			const rightCurrent =
-				right.provider === existing.provider && right.id === existing.model;
-			if (leftCurrent !== rightCurrent) return leftCurrent ? -1 : 1;
-			return `${left.provider}/${left.id}`.localeCompare(`${right.provider}/${right.id}`);
-		});
-		const choices = ordered.map((model) => {
-			const provider = ctx.modelRegistry.getProviderDisplayName(model.provider);
-			const name = model.name && model.name !== model.id ? `${model.name} · ` : "";
-			return `${provider} · ${name}${model.provider}/${model.id}`;
-		});
-		const selected = await ctx.ui.select(
-			`${role.label} model\nCurrent: ${profileLabel(existing)}`,
-			choices,
+		if (!(await configureRole(ctx, available, proposed, role))) return undefined;
+	}
+
+	while (true) {
+		const action = await ctx.ui.select(
+			`Review Goala configuration\n${reviewSummary(ctx, available, proposed)}`,
+			[SAVE_CONFIGURATION, EDIT_PHASE, CANCEL],
 		);
-		if (!selected) return undefined;
-		const model = ordered[choices.indexOf(selected)];
-		if (!model) return undefined;
-
-		let thinkingLevel: ThinkingLevel = "off";
-		if (model.reasoning) {
-			const selectedThinking = await ctx.ui.select(
-				`${role.label} reasoning\n${model.provider}/${model.id}`,
-				[...THINKING_LEVELS],
-			);
-			if (!selectedThinking) return undefined;
-			thinkingLevel = selectedThinking as ThinkingLevel;
-		}
+		if (!action || action === CANCEL) return undefined;
+		if (action === SAVE_CONFIGURATION) return proposed;
 
-		const profile: ModelProfile = {
-			provider: model.provider,
-			model: model.id,
-			thinkingLevel,
-		};
-		if (role.key === "fallbackExecutor") {
-			proposed.fallbackExecutor = {
-				...profile,
-				afterRepairCycle: current.fallbackExecutor.afterRepairCycle,
-			};
-		} else {
-			proposed[role.key] = profile;
+		const selectedRole = await ctx.ui.select(
+			"Choose a phase to edit",
+			MODEL_ROLES.map((role) => role.label),
+		);
+		if (!selectedRole) return undefined;
+		const role = MODEL_ROLES.find((candidate) => candidate.label === selectedRole);
+		if (!role || !(await configureRole(ctx, available, proposed, role))) {
+			return undefined;
 		}
 	}
-	return proposed;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-goala",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Goal-oriented agent lifecycle, evidence, and verified memory for Pi.",
   "type": "module",
   "author": "Barry King",