@ocis/myagent-cli 0.2.3 → 0.2.4

package/dist/ui/app.js

@@ -18,7 +18,7 @@ import { collectFileDiff, collectGitStatus, EMPTY_GIT_STATUS } from "../git/stat
 import { sanitizeForTerminal } from "../sanitize.js";
 import { bgRgb, isColorEnabled, modeBorder, overlayBackgroundRgb, panelBackgroundRgb, style, userMessageBackgroundRgb } from "./colors.js";
 import { helpLines } from "./help.js";
-import { buildModelItems, currentModelId } from "./model-list.js";
+import { buildModelItems, currentModelId, modelDisplayName } from "./model-list.js";
 import { createQuitConfirm } from "./quit-confirm.js";
 import { selectCardWidth, selectDescriptionMinWidth, selectLabelColumn, selectVisibleRows } from "./select-popup.js";
 import { editorTheme, selectListTheme } from "./theme.js";
@@ -349,24 +349,30 @@ export async function runTui(opts) {
     if (opts.initialModel)
         editor.model = opts.initialModel;
     /**
-     * Cached GET /v1/models list — the picker refreshes on open; this cache is
-     * for the startup default display (and stays empty when the server is
-     * unreachable or has no models configured).
+     * Cached GET /v1/models list — the picker refreshes on open; this cache backs
+     * the model display (label, not id) and the startup default. Stays empty when
+     * the server is unreachable or has no models configured.
      */
+    let modelList = [];
     let defaultModelId = null;
+    /** Display name for a model id — the server's label when known, else the id. */
+    const modelName = (id) => modelDisplayName(modelList, id);
     void opts.client.listModels().then((models) => {
         if (models.length === 0)
             return;
+        modelList = models;
         defaultModelId = models.find((m) => m.default)?.id ?? models[0].id;
         if (opts.initialModel && !models.some((m) => m.id === opts.initialModel)) {
             addTranscript(new NoticeComponent(`Model "${opts.initialModel}" is not available on this integration — the server default applies instead.`, "warn"));
         }
-        // Only fill the display before any thread/model knowledge exists.
+        // Only fill the model before any thread/model knowledge exists.
         if (!runner.threadId && !runner.selectedModel) {
             runner.model = defaultModelId;
-            editor.model = defaultModelId;
-            tui.requestRender();
         }
+        // The display may have been seeded from an id before the list arrived —
+        // refresh it now that labels are known.
+        editor.model = modelName(runner.model);
+        tui.requestRender();
     }).catch(() => { });
     function log(line) {
         logLines.push(line);
@@ -377,7 +383,7 @@ export async function runTui(opts) {
     function applySessionDetail(detail, options = {}) {
         if (detail.model)
             runner.model = detail.model;
-        editor.model = detail.model ?? runner.model;
+        editor.model = modelName(detail.model ?? runner.model);
         if (detail.title)
             sidebar.state.title = detail.title;
         const usage = usageFromSession(detail.usage);
@@ -675,7 +681,7 @@ export async function runTui(opts) {
         runner.setThinkingLevel(opts.initialThinking);
         // Back to the pending client-choice selection (or the server default).
         runner.model = runner.selectedModel ?? defaultModelId;
-        editor.model = runner.model;
+        editor.model = modelName(runner.model);
         firstUserMessage = null;
         clearView();
         addTranscript(new NoticeComponent("New session.", "info"));
@@ -812,16 +818,18 @@ export async function runTui(opts) {
             tui.requestRender();
             return;
         }
+        // Refresh the display cache — the policy may have changed since startup.
+        modelList = models;
         const current = currentModelId(models, runner.model, runner.selectedModel);
         showSelectOverlay("Model (applies to a new session)", buildModelItems(models, current), (value) => {
             runner.setModel(value);
             if (runner.threadId) {
-                addTranscript(new NoticeComponent(`Model for new sessions: ${value}. This thread keeps ${runner.model ?? "its pinned model"} — run /new to switch.`, "info"));
+                addTranscript(new NoticeComponent(`Model for new sessions: ${modelName(value)}. This thread keeps ${modelName(runner.model) ?? "its pinned model"} — run /new to switch.`, "info"));
             }
             else {
                 runner.model = value;
-                editor.model = value;
-                addTranscript(new NoticeComponent(`Model: ${value}`, "info"));
+                editor.model = modelName(value);
+                addTranscript(new NoticeComponent(`Model: ${modelName(value)}`, "info"));
             }
             tui.requestRender();
         }, {

package/dist/ui/components.d.ts

@@ -135,6 +135,7 @@ export declare class SidebarComponent implements Component {
 export type EditorChip = "mode" | "model" | "thinking";
 export declare class PromptEditor extends Editor {
     mode: AgentMode;
+    /** Display name of the model in use (the server's label when known, else its id). */
     model: string | null;
     thinking: string | undefined;
     /** Fired when a bottom-edge chip (mode / model / thinking) is clicked. */

package/dist/ui/components.js

@@ -406,6 +406,7 @@ export class SidebarComponent {
 }
 export class PromptEditor extends Editor {
     mode;
+    /** Display name of the model in use (the server's label when known, else its id). */
     model = null;
     thinking;
     /** Fired when a bottom-edge chip (mode / model / thinking) is clicked. */

package/dist/ui/model-list.d.ts

@@ -8,3 +8,10 @@ import type { ModelInfo } from "../protocol/types.js";
 export declare function currentModelId(models: ModelInfo[], activeModel: string | null, selectedModel: string | null): string | undefined;
 /** Build the `/model` picker rows for a model list. */
 export declare function buildModelItems(models: ModelInfo[], currentId: string | undefined): SelectItem[];
+/**
+ * Display name for a model id: the server's label when the list knows the id,
+ * else the id itself (a model the list no longer carries, or a list that has
+ * not loaded yet). The label is remote text — callers render it through the
+ * usual sanitizing path (the editor border and notices both do).
+ */
+export declare function modelDisplayName(models: ModelInfo[], id: string | null): string | null;

package/dist/ui/model-list.js

@@ -34,3 +34,14 @@ export function buildModelItems(models, currentId) {
         };
     });
 }
+/**
+ * Display name for a model id: the server's label when the list knows the id,
+ * else the id itself (a model the list no longer carries, or a list that has
+ * not loaded yet). The label is remote text — callers render it through the
+ * usual sanitizing path (the editor border and notices both do).
+ */
+export function modelDisplayName(models, id) {
+    if (!id)
+        return null;
+    return models.find((m) => m.id === id)?.label ?? id;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ocis/myagent-cli",
-  "version": "0.2.3",
+  "version": "0.2.4",
   "description": "Terminal coding agent for your local repository, powered by a remote MyAgent server",
   "type": "module",
   "bin": {