@gtrabanco/pi-agentic-workflow 0.8.0 → 0.9.0

package/README.es.md

@@ -77,7 +77,8 @@ Dos archivos JSON, ambos opcionales:
     "plan-feature": { "model": ["anthropic/claude-sonnet-4-5", "openai/gpt-5.2"], "thinking": "medium" },
     "review-change": { "thinking": "max" }
   },
-  "onUnavailableRoute": "stop"
+  "onUnavailableRoute": "stop",
+  "onSettle": "keep"
 }
 ```
 
@@ -97,6 +98,8 @@ que ya tuviera la sesión, porque la ruta por defecto del paquete es
   prueba `anthropic/claude-sonnet-4-5` primero y luego `openai/gpt-5.2`.
 - `thinking` es uno de `off`, `minimal`, `low`, `medium`, `high`, `xhigh`, `max`,
   o `"inherit"`.
+- `` `onSettle` `` es uno de `"keep"` (el default) o `"restore"` — ver
+  [Tu sesión después de un comando](#tu-sesión-después-de-un-comando).
 - Claves desconocidas, `null` y referencias mal formadas se **rechazan**, no se
   ignoran: un error tipográfico que no hace nada en silencio es el bug que nunca
   encuentras.
@@ -116,14 +119,28 @@ aun así:
 { "onUnavailableRoute": "inherit" }
 ```
 
-## Tu sesión vuelve
+## Tu sesión después de un comando
 
-El enrutamiento dura un comando. Cuando el turno termina, la sesión vuelve a
-como estaba — el modelo *y* el nivel de thinking, porque seleccionar un modelo
-puede mover el nivel. Si cambias el modelo tú mismo a mitad de turno, con
-`/model` por ejemplo, no se restaura nada: tu elección gana, y el comando lo
-dice. Si mueves solo el nivel de thinking, lo conservas mientras el modelo
-vuelve.
+Por defecto (**`onSettle: "keep"`**), el modelo enrutado y el nivel de thinking
+**se quedan** en la ventana de chat abierta cuando el comando termina. Si
+`plan-feature` corre en `glm-5` y quieres retocar el plan que produjo, tu siguiente
+prompt sigue corriendo en `glm-5` — igual con una pregunta posterior. Cuando no
+necesitas el modelo pesado, lo cambias tú con `/model` (o Ctrl+P) a algo más
+barato; nada restaura sobre tu elección.
+
+Para volver a traer el modelo y el nivel de thinking previos al comando cuando este
+termina, pon:
+
+```json
+{ "onSettle": "restore" }
+```
+
+(El modo `restore` es el histórico contrato AC8: tras un comando enrutado la
+sesión vuelve a como estaba — el modelo *y* el nivel de thinking, porque
+seleccionar un modelo puede mover el nivel. Si cambias el modelo tú mismo a
+mitad de turno, con `/model` por ejemplo, no se restaura nada: tu elección gana,
+y el comando lo dice. Si mueves solo el nivel de thinking, lo conservas mientras
+el modelo vuelve.)
 
 ## Consola de configuración
 
@@ -159,7 +176,7 @@ consola nunca se queda sin salida.
 | `could not be selected` | Pi rechazó el cambio. El comando se detiene con el motivo — salvo que `onUnavailableRoute` sea `inherit`, en cuyo caso avisa y se ejecuta con tu modelo actual. |
 | `refused: the agent is busy` | Hay un turno en ejecución. Espera a que termine. |
 | `is still routed` | El comando enrutado anterior no ha terminado. |
-| `leaving the model you chose in place` | Cambiaste el modelo durante el turno enrutado, así que no se restauró nada: tu elección ganó. |
+| `leaving the model you chose in place` | (en modo `restore`) Cambiaste el modelo durante un turno enrutado, así que no se restauró nada: tu elección ganó. |
 | `these configured routes match no command` | Una clave de `commands` no nombra nada. Corrige la escritura o elimina la entrada. |
 
 ## Notas

package/README.md

@@ -75,7 +75,8 @@ Two JSON files, both optional:
     "plan-feature": { "model": ["anthropic/claude-sonnet-4-5", "openai/gpt-5.2"], "thinking": "medium" },
     "review-change": { "thinking": "max" }
   },
-  "onUnavailableRoute": "stop"
+  "onUnavailableRoute": "stop",
+  "onSettle": "keep"
 }
 ```
 
@@ -95,6 +96,8 @@ whatever the session already had, because the shipped default route is
   then `openai/gpt-5.2`.
 - `thinking` is one of `off`, `minimal`, `low`, `medium`, `high`, `xhigh`, `max`,
   or `"inherit"`.
+- `` `onSettle` `` is one of `"keep"` (the default) or `"restore"` — see
+  [Your session after a command](#your-session-after-a-command).
 - Unknown keys, `null`s and malformed references are **rejected**, not ignored: a
   typo that silently did nothing is the bug you would never find.
 
@@ -112,13 +115,28 @@ nothing runs on a model you did not pick. To run anyway on the current model, se
 { "onUnavailableRoute": "inherit" }
 ```
 
-## Your session comes back
+## Your session after a command
 
-Routing lasts one command. When the turn settles, the session is put back the way
-you had it — the model *and* the thinking level, because selecting a model can
-move the level. If you change the model yourself mid-turn, with `/model`, say,
-nothing is restored: your choice wins, and the command says so. Change only the
-thinking level and you keep it while the model still comes back.
+By default (**`onSettle: "keep"`**), the routed model and thinking level **stay**
+in the open chat window once the command settles. If `plan-feature` runs on
+`glm-5` and you want to tweak the plan it produced, your next prompt keeps
+running on `glm-5` — same for a follow-up question. When you don't need the
+heavy model, switch it yourself with `/model` (or Ctrl+P) to something cheaper;
+nothing restores over your choice.
+
+To bring back the pre-command model and thinking level after a command settles,
+set:
+
+```json
+{ "onSettle": "restore" }
+```
+
+(The `restore` mode is the historical AC8 contract: after a routed command the
+session is put back the way you had it — the model *and* the thinking level,
+because selecting a model can move the level. If you change the model yourself
+mid-turn, with `/model` say, nothing is restored: your choice wins, and the
+command says so. Change only the thinking level and you keep it while the model
+still comes back.)
 
 ## Settings console
 
@@ -152,7 +170,7 @@ fall back to a plain prompt, so the console never dead-ends.
 | `could not be selected` | Pi refused the switch. The command stops with the reason — unless `onUnavailableRoute` is `inherit`, in which case it warns and runs on your current model. |
 | `refused: the agent is busy` | A turn is running. Wait for it to settle. |
 | `is still routed` | The previous routed command has not settled yet. |
-| `leaving the model you chose in place` | You changed the model during a routed turn, so nothing was restored — your choice won. |
+| `leaving the model you chose in place` | (in `restore` mode) You changed the model during a routed turn, so nothing was restored — your choice won. |
 | `these configured routes match no command` | A `commands` key names nothing. Fix the spelling or delete the entry. |
 
 ## Notes

package/dist/config/defaults.js

@@ -8,4 +8,5 @@ export const DEFAULT_CONFIG = {
     default: { ...DEFAULT_ROUTE },
     commands: {},
     onUnavailableRoute: "stop",
+    onSettle: "keep",
 };

package/dist/config/merge.js

@@ -35,6 +35,7 @@ export function mergeConfigs(globalFile = {}, projectFile = {}) {
         default: effectiveDefault,
         commands,
         onUnavailableRoute: projectFile.onUnavailableRoute ?? globalFile.onUnavailableRoute ?? DEFAULT_CONFIG.onUnavailableRoute,
+        onSettle: projectFile.onSettle ?? globalFile.onSettle ?? DEFAULT_CONFIG.onSettle,
     };
 }
 /** The route a command runs under: its own resolved override, else the default route. */

package/dist/config/schema.js

@@ -1,4 +1,4 @@
-import { MAX_MODEL_CHAIN, THINKING_LEVELS, UNAVAILABLE_ROUTE_POLICIES } from "./types.js";
+import { MAX_MODEL_CHAIN, SETTLE_POLICIES, THINKING_LEVELS, UNAVAILABLE_ROUTE_POLICIES } from "./types.js";
 /**
  * Strict validator for one config file (SPEC S5-S8, D-E5).
  *
@@ -8,7 +8,7 @@ import { MAX_MODEL_CHAIN, THINKING_LEVELS, UNAVAILABLE_ROUTE_POLICIES } from "./
  * strong model ran when it did not. Only a *missing* file resolves to the
  * default, and that decision belongs to the loader.
  */
-const ROOT_KEYS = new Set(["default", "commands", "onUnavailableRoute"]);
+const ROOT_KEYS = new Set(["default", "commands", "onUnavailableRoute", "onSettle"]);
 const ROUTE_KEYS = new Set(["model", "thinking"]);
 const COMMAND_NAME = /^[a-z0-9][a-z0-9._-]*$/u;
 function isRecord(value) {
@@ -37,6 +37,9 @@ function isThinkingSetting(value) {
 function isUnavailableRoutePolicy(value) {
     return UNAVAILABLE_ROUTE_POLICIES.includes(value);
 }
+function isSettlePolicy(value) {
+    return SETTLE_POLICIES.includes(value);
+}
 function describe(value) {
     return typeof value === "string" ? `"${value}"` : JSON.stringify(value) ?? String(value);
 }
@@ -115,7 +118,7 @@ function validateConfig(value) {
         if (!ROOT_KEYS.has(key)) {
             issues.push({
                 path: `$.${displayKey(key)}`,
-                message: `unknown config key "${key}" (allowed: default, commands, onUnavailableRoute)`,
+                message: `unknown config key "${key}" (allowed: default, commands, onUnavailableRoute, onSettle)`,
             });
         }
     }
@@ -153,6 +156,17 @@ function validateConfig(value) {
             config.onUnavailableRoute = value.onUnavailableRoute;
         }
     }
+    if (value.onSettle !== undefined) {
+        if (!isSettlePolicy(value.onSettle)) {
+            issues.push({
+                path: "$.onSettle",
+                message: `must be "keep" or "restore", got ${describe(value.onSettle)}`,
+            });
+        }
+        else {
+            config.onSettle = value.onSettle;
+        }
+    }
     return issues.length > 0 ? { ok: false, issues } : { ok: true, config };
 }
 /**

package/dist/config/types.d.ts

@@ -37,18 +37,29 @@ export interface ConfigFile {
     default?: RouteFile;
     commands?: Record<string, RouteFile>;
     onUnavailableRoute?: UnavailableRoutePolicy;
+    /** What happens to the routed model and thinking level after the command settles. */
+    onSettle?: SettlePolicy;
 }
 export type UnavailableRoutePolicy = "stop" | "inherit";
+/**
+ * What the session holds after a routed command settles.
+ *  - `"keep"` — the routed model and thinking level stay in the open chat window,
+ *    so a follow-up edit or question keeps running on the model that planned it.
+ *  - `"restore"` — the pre-dispatch model and thinking level come back (AC8).
+ */
+export type SettlePolicy = "keep" | "restore";
 /** One rejected field inside one config file, addressed by a JSON-path-ish string. */
 export interface ConfigIssue {
     path: string;
     message: string;
 }
 export declare const UNAVAILABLE_ROUTE_POLICIES: readonly UnavailableRoutePolicy[];
+export declare const SETTLE_POLICIES: readonly SettlePolicy[];
 export interface EffectiveConfig {
     default: Route;
     commands: Record<string, Route>;
     onUnavailableRoute: UnavailableRoutePolicy;
+    onSettle: SettlePolicy;
 }
 export interface ConfigProblem {
     scope: "global" | "project";

package/dist/config/types.js

@@ -12,3 +12,4 @@ export const THINKING_LEVELS = ["off", "minimal", "low", "medium", "high", "xhig
 /** Cap on chain length (issue deliberation: 8 was "almost certainly a mistake"; 4 keeps the view readable). */
 export const MAX_MODEL_CHAIN = 4;
 export const UNAVAILABLE_ROUTE_POLICIES = ["stop", "inherit"];
+export const SETTLE_POLICIES = ["keep", "restore"];

package/dist/routing/dispatch.js

@@ -79,6 +79,12 @@ export function createRouter({ surface, loadConfig, hint, settingsCommand, known
             // Cleared first: the restore fires its own select events, and a turn that is
             // already over must not read them as operator changes.
             pending = undefined;
+            // `keep`: leave the routed model and thinking level in the open chat window,
+            // so a follow-up edit or question keeps running on the model that planned it.
+            // The operator's own mid-turn choice is still untouched (nothing is restored,
+            // so it cannot be overwritten). Only "restore" puts the snapshot back (AC8).
+            if (turn.settlePolicy === "keep")
+                return;
             await restore(turn, surface(ctx), ctx, "finished");
         },
         async dispatch(command, args, ctx) {
@@ -198,7 +204,14 @@ export function createRouter({ surface, loadConfig, hint, settingsCommand, known
                 ctx.notify(`/${command.name} is running. Per-command models are optional: configure them with /${settingsCommand} or the pi-agentic-workflow.json files. This hint appears once.`, "info");
             }
             if (applied.model || applied.thinking) {
-                pending = { command: command.name, snapshot, applied, userChangedModel: false, userChangedThinking: false };
+                pending = {
+                    command: command.name,
+                    snapshot,
+                    applied,
+                    settlePolicy: loaded.config.onSettle,
+                    userChangedModel: false,
+                    userChangedThinking: false,
+                };
             }
             // Pi expands `/skill:<x>` by the skill's frontmatter `name:`, and passes an
             // unknown key through as literal text — so the name is the only correct
@@ -217,7 +230,16 @@ export function createRouter({ surface, loadConfig, hint, settingsCommand, known
                 pending = undefined;
                 const rolledBack = Boolean(applied.model || applied.thinking);
                 if (rolledBack) {
-                    await restore({ command: command.name, snapshot, applied, userChangedModel: false, userChangedThinking: false }, session, ctx, `dispatch failed (${error.message}); the session was put back`);
+                    await restore({
+                        command: command.name,
+                        snapshot,
+                        applied,
+                        // The turn never started, so the rollback always restores — it is not a
+                        // settle, and `onSettle` does not apply. Explicit, not read from config.
+                        settlePolicy: "restore",
+                        userChangedModel: false,
+                        userChangedThinking: false,
+                    }, session, ctx, `dispatch failed (${error.message}); the session was put back`);
                 }
                 return refuse(ctx, "dispatch-failed", `/${command.name} was not dispatched: ${error.message}. Nothing was sent${rolledBack ? ", and the session model was put back" : ""}.`);
             }

package/dist/settings/console.d.ts

@@ -40,6 +40,7 @@ export declare const prompts: {
     readonly undone: "The routing was undone and the session put back.";
     readonly command: "Which command?";
     readonly policyChoice: "What should happen when a configured model is unavailable?";
+    readonly settleChoice: "What should happen after a routed command settles?";
     readonly saveTo: (path: string) => string;
     readonly discard: "Discard the draft?";
     readonly setDefaultRoute: "Set the default route";
@@ -49,6 +50,7 @@ export declare const prompts: {
     readonly bulkClear: "Clear several overrides";
     readonly addAnother: "Add another?";
     readonly policy: "Set the unavailable-route policy";
+    readonly settle: "Set the post-command model keep/restore policy";
     readonly save: "Save";
     readonly cancel: "Cancel";
     readonly model: (target: string) => string;

package/dist/settings/console.js

@@ -10,7 +10,7 @@
 import { loadConfig, configFilePaths } from "../config/load.js";
 import { effectiveRoute } from "../config/merge.js";
 import { parseConfigFile, parseModelReference } from "../config/schema.js";
-import { MAX_MODEL_CHAIN, THINKING_LEVELS, UNAVAILABLE_ROUTE_POLICIES } from "../config/types.js";
+import { MAX_MODEL_CHAIN, SETTLE_POLICIES, THINKING_LEVELS, UNAVAILABLE_ROUTE_POLICIES } from "../config/types.js";
 import { renderMergedConfig, routePath, DEFAULT_ROUTE } from "./view.js";
 /** The console's questions. `test/settings-console.test.mjs` drives the flow
  * through these strings, so renaming one fails the tests that use it rather than
@@ -23,6 +23,7 @@ export const prompts = {
     undone: "The routing was undone and the session put back.",
     command: "Which command?",
     policyChoice: "What should happen when a configured model is unavailable?",
+    settleChoice: "What should happen after a routed command settles?",
     saveTo: (path) => `Save the draft to ${path}?`,
     discard: "Discard the draft?",
     setDefaultRoute: "Set the default route",
@@ -32,6 +33,7 @@ export const prompts = {
     bulkClear: "Clear several overrides",
     addAnother: "Add another?",
     policy: "Set the unavailable-route policy",
+    settle: "Set the post-command model keep/restore policy",
     save: "Save",
     cancel: "Cancel",
     model: (target) => `Model for ${target}?`,
@@ -71,6 +73,7 @@ export async function runSettingsConsole(deps) {
             prompts.bulkApply,
             prompts.bulkClear,
             prompts.policy,
+            prompts.settle,
             ...(deps.routing?.inFlight() ? [prompts.undoInFlight] : []),
             prompts.save,
             prompts.cancel,
@@ -142,6 +145,12 @@ export async function runSettingsConsole(deps) {
                 draft = { ...draft, onUnavailableRoute: picked };
             continue;
         }
+        if (choice === prompts.settle) {
+            const picked = await deps.ui.select(prompts.settleChoice, [...SETTLE_POLICIES]);
+            if (isSettle(picked))
+                draft = { ...draft, onSettle: picked };
+            continue;
+        }
         if (choice === prompts.save) {
             const saved = await saveScope(deps, scope, path, draft);
             if (saved)
@@ -405,6 +414,7 @@ function describeRouting(file) {
         file.default ? `default: ${file.default.model ?? "inherit"} / ${file.default.thinking ?? "inherit"}` : "default: inherit / inherit",
         ...Object.entries(file.commands ?? {}).map(([name, route]) => `${name}: ${route.model ?? "inherit"} / ${route.thinking ?? "inherit"}`),
         `unavailable: ${file.onUnavailableRoute ?? "stop"}`,
+        `settle: ${file.onSettle ?? "keep"}`,
     ];
     return routes.join(" · ");
 }
@@ -428,6 +438,8 @@ function clean(draft) {
         file.commands = commands;
     if (draft.onUnavailableRoute)
         file.onUnavailableRoute = draft.onUnavailableRoute;
+    if (draft.onSettle)
+        file.onSettle = draft.onSettle;
     return file;
 }
 function parseFile(text, scope) {
@@ -448,3 +460,6 @@ function isThinkingLevel(value) {
 function isPolicy(value) {
     return value === "stop" || value === "inherit";
 }
+function isSettle(value) {
+    return value === "keep" || value === "restore";
+}

package/dist/settings/view.js

@@ -31,6 +31,7 @@ function configLines(config, commands) {
                 `  ${overrides.length} of ${total} commands override the default route`,
             ]),
         `  when a configured model is unavailable: ${config.onUnavailableRoute}`,
+        `  after a routed command settles: ${config.onSettle}`,
     ];
 }
 function route(value) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gtrabanco/pi-agentic-workflow",
-  "version": "0.8.0",
+  "version": "0.9.0",
   "description": "Pi package: canonical agentic-workflow skills, friendly slash commands, and per-command model routing.",
   "keywords": [
     "pi-package",