pi-advisor-flow 0.3.0 → 0.3.1

package/CHANGELOG.md

@@ -4,6 +4,16 @@ All notable changes to this project are documented here.
 
 The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## 0.3.1
+
+### Changed
+
+- Updated the supported runtime baseline to Pi 0.84.1 and verified Herdr reporting against Herdr 0.8.0 while retaining the same request compatibility with Herdr 0.7.5.
+
+### Fixed
+
+- Show configured models and thinking levels first and ticked in `/advisor-models`, allowing Enter to keep each current selection.
+
 ## 0.3.0
 
 ### Added

package/README.md

@@ -23,7 +23,7 @@ The idea is simple: keep implementation on a fast model and borrow frontier reas
 
 ## Install
 
-Current release: **0.3.0**. Requires Pi 0.80.7 or later. The extension installs no dependencies of its own; Pi supplies its runtime modules.
+Current release: **0.3.1**. Requires Pi 0.84.1 or later and is compatible with Herdr 0.8.0. The extension installs no dependencies of its own; Pi supplies its runtime modules.
 
 ```bash
 # npm
@@ -41,7 +41,7 @@ Restart or reload Pi after installation.
 ## Quick start
 
 1. Run `/advisor` to enable the flow and register `ask_advisor`.
-2. Run `/advisor-models` to choose the Executor and Advisor models.
+2. Run `/advisor-models` to choose the Executor and Advisor models. Current model and thinking-level selections appear first and ticked, so pressing Enter keeps them.
 3. Run `/advisor-settings` to configure review gates, context, privacy, and limits.
 
 Unknown fields in `advisor.json` are preserved for forward compatibility and reported as non-blocking warnings. Invalid recognized values remain errors, and Advisor commands show the configuration problem without crashing their handlers.
@@ -69,7 +69,7 @@ Successful calls return an opaque `adviceId`. If global outcome logging is enabl
 | --- | --- |
 | `/advisor` | Enable the flow and optionally override the Executor, Advisor, or context limit. |
 | `/advisor-manual [focus]` | Start a parallel consultation without interrupting the current Executor turn. |
-| `/advisor-models` | Choose both models and their reasoning effort. |
+| `/advisor-models` | Choose both models and their reasoning effort; current models are preselected. |
 | `/advisor-settings` | Configure behavior, context, gates, privacy, and output limits. |
 | `/advisor-off` | Disable the flow and turn off persistent activation. |
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-advisor-flow",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "author": "Philip Brembeck",
   "repository": {
     "type": "git",
@@ -9,9 +9,9 @@
   "main": "extensions/index.ts",
   "devDependencies": {
     "@biomejs/biome": "2.5.3",
-    "@earendil-works/pi-ai": "^0.80.7",
-    "@earendil-works/pi-coding-agent": "^0.80.7",
-    "@earendil-works/pi-tui": "^0.80.7",
+    "@earendil-works/pi-ai": "^0.84.1",
+    "@earendil-works/pi-coding-agent": "^0.84.1",
+    "@earendil-works/pi-tui": "^0.84.1",
     "@types/node": "^20.11.0",
     "bun-types": "^1.0.0",
     "husky": "^9.1.7",
@@ -21,9 +21,9 @@
     "ultracite": "7.9.4"
   },
   "peerDependencies": {
-    "@earendil-works/pi-ai": "^0.80.7",
-    "@earendil-works/pi-coding-agent": "^0.80.7",
-    "@earendil-works/pi-tui": "^0.80.7",
+    "@earendil-works/pi-ai": "^0.84.1",
+    "@earendil-works/pi-coding-agent": "^0.84.1",
+    "@earendil-works/pi-tui": "^0.84.1",
     "typebox": "^1.1.38"
   },
   "peerDependenciesMeta": {

package/src/commands.ts

@@ -66,8 +66,10 @@ import {
   SearchableModelSelector,
 } from "./ui.js";
 
+const DEFAULT_EFFORT_LEVEL = "Default (Model Default)";
+const SELECTED_PREFIX = "✓ ";
 const EFFORT_LEVELS = [
-  "Default (Model Default)",
+  DEFAULT_EFFORT_LEVEL,
   "off",
   "minimal",
   "low",
@@ -77,6 +79,21 @@ const EFFORT_LEVELS = [
   "max",
 ];
 
+const effortChoices = (configured: string | undefined): string[] => {
+  const current = configured ?? DEFAULT_EFFORT_LEVEL;
+  return [
+    `${SELECTED_PREFIX}${current}`,
+    ...EFFORT_LEVELS.filter((level) => level !== current),
+  ];
+};
+
+const selectedEffort = (choice: string): string | undefined => {
+  const effort = choice.startsWith(SELECTED_PREFIX)
+    ? choice.slice(SELECTED_PREFIX.length)
+    : choice;
+  return effort === DEFAULT_EFFORT_LEVEL ? undefined : effort;
+};
+
 const CONTEXT_PRESETS: ContextPreset[] = [
   {
     description:
@@ -446,6 +463,7 @@ export const registerCommands = (
         (tui, theme, keybindings, done) =>
           new SearchableModelSelector({
             allOptions: refs,
+            currentOption: executorRef,
             keybindings,
             onCancel: () => done(undefined),
             onSelect: done,
@@ -460,7 +478,7 @@ export const registerCommands = (
 
       const executorEffort = await ctx.ui.select(
         "Select Executor Reasoning/Thinking Level",
-        EFFORT_LEVELS
+        effortChoices(executorEffortRef)
       );
       if (!executorEffort) {
         return;
@@ -470,6 +488,7 @@ export const registerCommands = (
         (tui, theme, keybindings, done) =>
           new SearchableModelSelector({
             allOptions: refs,
+            currentOption: advisorRef,
             keybindings,
             onCancel: () => done(undefined),
             onSelect: done,
@@ -484,7 +503,7 @@ export const registerCommands = (
 
       const advisorEffort = await ctx.ui.select(
         "Select Advisor Reasoning/Thinking Level",
-        EFFORT_LEVELS
+        effortChoices(advisorEffortRef)
       );
       if (!advisorEffort) {
         return;
@@ -492,14 +511,8 @@ export const registerCommands = (
 
       setExecutorRef(executor);
       setAdvisorRef(advisor);
-      setExecutorEffortRef(
-        executorEffort === "Default (Model Default)"
-          ? undefined
-          : executorEffort
-      );
-      setAdvisorEffortRef(
-        advisorEffort === "Default (Model Default)" ? undefined : advisorEffort
-      );
+      setExecutorEffortRef(selectedEffort(executorEffort));
+      setAdvisorEffortRef(selectedEffort(advisorEffort));
 
       const path = saveConfig(ctx);
       ctx.ui.notify(

package/src/ui.ts

@@ -17,6 +17,7 @@ interface RenderRequester {
 }
 interface SearchableModelSelectorOptions {
   allOptions: string[];
+  currentOption?: string;
   keybindings: KeybindingsManager;
   onCancel: () => void;
   onSelect: (value: string) => void;
@@ -50,6 +51,7 @@ export class SearchableModelSelector implements Component, Focusable {
   private readonly tui: RenderRequester;
   private readonly searchInput: Input;
   private readonly allOptions: string[];
+  private readonly currentOption: string | undefined;
   private filteredOptions: string[];
   private selectedIndex = 0;
   private readonly title: string;
@@ -70,7 +72,13 @@ export class SearchableModelSelector implements Component, Focusable {
   constructor(options: SearchableModelSelectorOptions) {
     this.tui = options.tui;
     this.title = options.title;
-    this.allOptions = options.allOptions;
+    this.currentOption = options.currentOption || undefined;
+    this.allOptions = this.currentOption
+      ? [
+          this.currentOption,
+          ...options.allOptions.filter((item) => item !== this.currentOption),
+        ]
+      : options.allOptions;
     this.theme = options.theme;
     this.keybindings = options.keybindings;
     this.onSelect = options.onSelect;
@@ -116,12 +124,13 @@ export class SearchableModelSelector implements Component, Focusable {
       const endIndex = Math.min(startIndex + maxVisible, total);
       for (let i = startIndex; i < endIndex; i += 1) {
         const item = this.filteredOptions[i];
+        const tick = item === this.currentOption ? "✓ " : "  ";
         if (i === this.selectedIndex) {
           lines.push(
-            `  ${this.theme.fg("accent", "→ ")}${this.theme.fg("accent", item)}`
+            `  ${this.theme.fg("accent", "→ ")}${this.theme.fg("accent", `${tick}${item}`)}`
           );
         } else {
-          lines.push(`    ${this.theme.fg("text", item)}`);
+          lines.push(`    ${this.theme.fg("text", `${tick}${item}`)}`);
         }
       }
       if (total > maxVisible) {