@kimbho/kimbho-cli 0.1.3 → 0.1.4

package/README.md

@@ -36,7 +36,7 @@ kimbho plan "build a coding agent"
 kimbho /models openrouter --search claude --limit 10
 ```
 
-Bare `kimbho` opens the interactive shell. Within that shell, plain-English input is treated as `run <goal>`, and slash commands like `/plan`, `/run`, `/models`, and `/brains` are supported.
+Bare `kimbho` opens the interactive shell. Within that shell, plain-English input is treated as `run <goal>`, and slash commands like `/plan`, `/run`, `/models`, and `/brains` are supported. Provider and model selections apply to all agent roles by default.
 
 Useful shell commands:
 
@@ -55,10 +55,19 @@ Useful shell commands:
 /quit
 ```
 
+LM Studio example:
+
+```bash
+/providers add lmstudio lmstudio-lan --base-url http://192.168.0.70:1234
+/models
+/select 1
+```
+
 `run` now auto-executes the supported ready-task frontier and persists session events. `resume --execute` continues that frontier from the latest session snapshot.
 
 ## Notes
 
 - The published CLI bundles the internal Kimbho packages so installation does not depend on the monorepo layout.
 - Brain and provider configuration is stored in `.kimbho/config.json`.
-- Use `kimbho models use <model> --provider <id> --role coder --set-default` to select a provider model and assign it to a brain role.
+- Use `kimbho models use <model> --provider <id> --set-default` to select a provider model for the whole system.
+- Use `kimbho brains assign --role <role> ...` only when you intentionally want an advanced per-role override.

package/dist/index.cjs

@@ -3346,7 +3346,7 @@ var {
 // package.json
 var package_default = {
   name: "@kimbho/kimbho-cli",
-  version: "0.1.3",
+  version: "0.1.4",
   description: "Kimbho CLI is a terminal-native coding agent for planning, execution, and verification.",
   type: "module",
   engines: {
@@ -8072,6 +8072,34 @@ function assignBrain(config, role, settings) {
     }
   });
 }
+function buildRetainedBrainSettings(current, providerId, model) {
+  return {
+    providerId,
+    ...model ? {
+      model
+    } : {},
+    ...typeof current.temperature === "number" ? {
+      temperature: current.temperature
+    } : {},
+    ...typeof current.maxTokens === "number" ? {
+      maxTokens: current.maxTokens
+    } : {},
+    ...current.promptPreamble ? {
+      promptPreamble: current.promptPreamble
+    } : {}
+  };
+}
+function assignProviderToAllBrains(config, providerId, model) {
+  return KimbhoConfigSchema.parse({
+    ...config,
+    brains: {
+      planner: buildRetainedBrainSettings(config.brains.planner, providerId, model),
+      coder: buildRetainedBrainSettings(config.brains.coder, providerId, model),
+      reviewer: buildRetainedBrainSettings(config.brains.reviewer, providerId, model),
+      fast: buildRetainedBrainSettings(config.brains.fast, providerId, model)
+    }
+  });
+}
 function resolveBrainSettings(config, role) {
   return config.brains[role];
 }
@@ -8826,6 +8854,28 @@ function listProviderTemplates() {
 function findProviderTemplate(templateId) {
   return BUILTIN_PROVIDER_TEMPLATES.find((template) => template.id === templateId);
 }
+function trimTrailingSlash(value) {
+  return value.replace(/\/+$/, "");
+}
+function normalizeTemplateBaseUrl(templateId, baseUrl) {
+  if (!baseUrl) {
+    return void 0;
+  }
+  const normalized = trimTrailingSlash(baseUrl);
+  if (templateId !== "lmstudio") {
+    return normalized;
+  }
+  try {
+    const parsed = new URL(normalized);
+    if (!parsed.pathname || parsed.pathname === "/") {
+      parsed.pathname = "/v1";
+      return trimTrailingSlash(parsed.toString());
+    }
+  } catch {
+    return normalized;
+  }
+  return normalized;
+}
 function buildProviderFromTemplate(templateId, options = {}) {
   const template = findProviderTemplate(templateId);
   if (!template) {
@@ -8835,8 +8885,8 @@ function buildProviderFromTemplate(templateId, options = {}) {
     id: options.providerId ?? template.id,
     label: options.label ?? template.label,
     driver: template.driver,
-    ...options.baseUrl ?? template.defaultBaseUrl ? {
-      baseUrl: options.baseUrl ?? template.defaultBaseUrl
+    ...normalizeTemplateBaseUrl(templateId, options.baseUrl) ?? template.defaultBaseUrl ? {
+      baseUrl: normalizeTemplateBaseUrl(templateId, options.baseUrl) ?? template.defaultBaseUrl
     } : {},
     ...options.apiKeyEnv ?? template.defaultApiKeyEnv ? {
       apiKeyEnv: options.apiKeyEnv ?? template.defaultApiKeyEnv
@@ -8875,11 +8925,11 @@ async function requestJson(url, init, timeoutMs = 3e4) {
   }
   return response.json();
 }
-function trimTrailingSlash(value) {
+function trimTrailingSlash2(value) {
   return value.replace(/\/+$/, "");
 }
 function joinUrl(baseUrl, suffix) {
-  const normalizedBase = trimTrailingSlash(baseUrl);
+  const normalizedBase = trimTrailingSlash2(baseUrl);
   const normalizedSuffix = suffix.startsWith("/") ? suffix : `/${suffix}`;
   return `${normalizedBase}${normalizedSuffix}`;
 }
@@ -9187,9 +9237,26 @@ var OpenAICompatibleProvider = class {
       };
     }
     const apiKey = resolveApiKey(this.definition);
+    if (this.definition.apiKeyEnv && !apiKey) {
+      return {
+        ok: false,
+        message: `${this.definition.baseUrl} (${this.definition.defaultModel ?? "model not set"}), missing ${this.definition.apiKeyEnv}`
+      };
+    }
+    try {
+      await requestJson(joinUrl(this.definition.baseUrl, "/models"), {
+        method: "GET",
+        headers: buildProviderHeaders(this.definition, apiKey, false)
+      }, 2e3);
+    } catch (error) {
+      return {
+        ok: false,
+        message: error instanceof Error ? error.message : String(error)
+      };
+    }
     return {
-      ok: !this.definition.apiKeyEnv || Boolean(apiKey),
-      message: `${this.definition.baseUrl} (${this.definition.defaultModel ?? "model not set"})${this.definition.apiKeyEnv && !apiKey ? `, missing ${this.definition.apiKeyEnv}` : ""}`
+      ok: true,
+      message: `${this.definition.baseUrl} (${this.definition.defaultModel ?? "model not set"})`
     };
   }
   async listModels(input) {
@@ -9860,7 +9927,7 @@ function createModelsCommand() {
     console.log(`Updated ${outputPath}`);
     console.log(`Synced ${models.length} models for ${provider.id}`);
   });
-  command.command("use").description("Select a model for a provider and optionally assign it to a brain role.").argument("<model>", "Model id to select").requiredOption("--provider <provider>", "Provider id to use").option("--role <role>", "Brain role to assign: planner, coder, reviewer, or fast").option("--set-default", "Also set this model as the provider default", false).option("--force", "Skip remote model validation", false).option("--temperature <value>", "Temperature override for the role", parseNumber2).option("--max-tokens <value>", "Max token override for the role", parseInteger).action(async (model, options) => {
+  command.command("use").description("Select a model for a provider and assign it globally unless a specific role is requested.").argument("<model>", "Model id to select").requiredOption("--provider <provider>", "Provider id to use").option("--role <role>", "Brain role to assign: planner, coder, reviewer, or fast").option("--set-default", "Also set this model as the provider default", false).option("--force", "Skip remote model validation", false).option("--temperature <value>", "Temperature override for the role", parseNumber2).option("--max-tokens <value>", "Max token override for the role", parseInteger).action(async (model, options) => {
     let config = await loadConfig(import_node_process5.default.cwd());
     if (!config) {
       requireConfigMessage2();
@@ -9910,12 +9977,16 @@ function createModelsCommand() {
           maxTokens: options.maxTokens
         } : {}
       });
+    } else {
+      config = assignProviderToAllBrains(config, provider.id, model);
     }
     const outputPath = await saveConfig(config, import_node_process5.default.cwd());
     console.log(`Updated ${outputPath}`);
     console.log(`Selected ${provider.id}/${model}`);
     if (options.role) {
       console.log(`Assigned role ${options.role}`);
+    } else {
+      console.log("Assigned all roles");
     }
     if (shouldSetDefault) {
       console.log(`Set provider default model to ${model}`);
@@ -10673,12 +10744,13 @@ function renderHelp() {
     "/model                   Show current brain assignments.",
     "/providers               List configured providers.",
     "/providers templates     Show built-in provider templates.",
-    "/providers add <tpl> [id] Add a provider from a built-in template.",
-    "/providers use <id> [role] Use a provider for the active role.",
+    "/providers add <tpl> [id] [--base-url <url>] [--model <id>] [--api-key-env <env>]",
+    "                         Add a provider from a built-in template.",
+    "/providers use <id>      Use a provider for all agent roles.",
     "/providers check         Run provider health checks.",
     "/models [search]         Discover models for the active role's provider.",
     "/select <n>              Select a model from the last numbered /models list.",
-    "/use-model <id>          Assign a model to the active role and provider.",
+    "/use-model <id>          Assign a model to all agent roles.",
     "/plan <goal>             Create a structured implementation plan.",
     "/run <goal>              Start a Kimbho execution session for a goal.",
     "/resume                  Show the latest saved session.",
@@ -10811,6 +10883,52 @@ function defaultProviderIdForTemplate(templateId) {
       return `${templateId}-main`;
   }
 }
+function parseProviderAddOptions(tokens) {
+  const templateId = tokens[2];
+  if (!templateId) {
+    throw new Error("Usage: /providers add <template> [provider-id] [--base-url <url>] [--model <id>] [--api-key-env <env>]");
+  }
+  let index = 3;
+  let providerId;
+  if (tokens[index] && !tokens[index]?.startsWith("-")) {
+    providerId = tokens[index];
+    index += 1;
+  }
+  const options = {
+    templateId,
+    ...providerId ? {
+      providerId
+    } : {}
+  };
+  while (index < tokens.length) {
+    const flag = tokens[index];
+    if (!flag?.startsWith("--")) {
+      throw new Error(`Unexpected token "${flag}".`);
+    }
+    const value = tokens[index + 1];
+    if (!value || value.startsWith("--")) {
+      throw new Error(`Missing value for ${flag}.`);
+    }
+    switch (flag) {
+      case "--label":
+        options.label = value;
+        break;
+      case "--base-url":
+        options.baseUrl = normalizeTemplateBaseUrl(templateId, value) ?? value;
+        break;
+      case "--api-key-env":
+        options.apiKeyEnv = value;
+        break;
+      case "--model":
+        options.model = value;
+        break;
+      default:
+        throw new Error(`Unknown option "${flag}". Supported: --label, --base-url, --api-key-env, --model.`);
+    }
+    index += 2;
+  }
+  return options;
+}
 function buildCachedModels(provider, search, limit = 25) {
   const normalized = search?.trim().toLowerCase();
   const filtered = normalized ? provider.models.filter((modelId) => modelId.toLowerCase().includes(normalized)) : provider.models;
@@ -10857,7 +10975,7 @@ async function fetchModelsForProvider(cwd, config, provider, search, limit = 25)
     throw error;
   }
 }
-async function assignModelToRole(cwd, role, providerId, model) {
+async function assignModelGlobally(cwd, providerId, model) {
   const config = await loadConfig(cwd);
   if (!config) {
     throw new Error("No config found. Run /init or /providers add first.");
@@ -10866,8 +10984,7 @@ async function assignModelToRole(cwd, role, providerId, model) {
   if (!provider) {
     throw new Error(`Unknown provider "${providerId}".`);
   }
-  const currentSettings = resolveBrainSettings(config, role);
-  const nextConfig = assignBrain(
+  const nextConfig = assignProviderToAllBrains(
     upsertProvider(config, {
       ...provider,
       defaultModel: model,
@@ -10876,25 +10993,13 @@ async function assignModelToRole(cwd, role, providerId, model) {
         model
       ]))
     }),
-    role,
-    {
-      providerId,
-      model,
-      ...typeof currentSettings.temperature === "number" ? {
-        temperature: currentSettings.temperature
-      } : {},
-      ...typeof currentSettings.maxTokens === "number" ? {
-        maxTokens: currentSettings.maxTokens
-      } : {},
-      ...currentSettings.promptPreamble ? {
-        promptPreamble: currentSettings.promptPreamble
-      } : {}
-    }
+    providerId,
+    model
   );
   const outputPath = await saveConfig(nextConfig, cwd);
   return outputPath;
 }
-async function useProviderForRole(cwd, role, providerId) {
+async function useProviderGlobally(cwd, providerId) {
   const config = await loadConfig(cwd);
   if (!config) {
     throw new Error("No config found. Run /init or /providers add first.");
@@ -10903,23 +11008,8 @@ async function useProviderForRole(cwd, role, providerId) {
   if (!provider) {
     throw new Error(`Unknown provider "${providerId}".`);
   }
-  const currentSettings = resolveBrainSettings(config, role);
   const resolvedModel = provider.defaultModel ?? provider.models[0] ?? null;
-  const nextConfig = assignBrain(config, role, {
-    providerId: provider.id,
-    ...resolvedModel ? {
-      model: resolvedModel
-    } : {},
-    ...typeof currentSettings.temperature === "number" ? {
-      temperature: currentSettings.temperature
-    } : {},
-    ...typeof currentSettings.maxTokens === "number" ? {
-      maxTokens: currentSettings.maxTokens
-    } : {},
-    ...currentSettings.promptPreamble ? {
-      promptPreamble: currentSettings.promptPreamble
-    } : {}
-  });
+  const nextConfig = assignProviderToAllBrains(config, provider.id, resolvedModel);
   const outputPath = await saveConfig(nextConfig, cwd);
   return {
     outputPath,
@@ -10950,6 +11040,7 @@ async function printProviderList(cwd, focusRole) {
     console.log(`  label: ${provider.label ?? "-"}`);
     console.log(`  driver: ${provider.driver}`);
     console.log(`  model: ${provider.defaultModel ?? "-"}`);
+    console.log(`  baseUrl: ${provider.baseUrl ?? "-"}`);
     console.log(`  auth: ${envState}`);
     console.log(`  cachedModels: ${provider.models.length}`);
   }
@@ -10964,9 +11055,21 @@ function printProviderTemplates() {
     console.log(`  notes: ${template.notes}`);
   }
 }
-async function addProviderFromTemplate(cwd, templateId, providerId) {
-  const provider = buildProviderFromTemplate(templateId, {
-    providerId: providerId ?? defaultProviderIdForTemplate(templateId)
+async function addProviderFromTemplate(cwd, options) {
+  const provider = buildProviderFromTemplate(options.templateId, {
+    providerId: options.providerId ?? defaultProviderIdForTemplate(options.templateId),
+    ...options.label ? {
+      label: options.label
+    } : {},
+    ...options.baseUrl ? {
+      baseUrl: options.baseUrl
+    } : {},
+    ...options.apiKeyEnv ? {
+      apiKeyEnv: options.apiKeyEnv
+    } : {},
+    ...options.model ? {
+      model: options.model
+    } : {}
   });
   const config = await loadConfig(cwd);
   if (!config) {
@@ -11025,30 +11128,26 @@ async function handleProvidersCommand(cwd, tokens, runtime) {
     return;
   }
   if (subcommand === "add") {
-    const templateId = tokens[2];
-    const providerId = tokens[3];
-    if (!templateId) {
-      throw new Error("Usage: /providers add <template> [provider-id]");
-    }
-    const outputPath = await addProviderFromTemplate(cwd, templateId, providerId);
-    const resolvedProviderId = providerId ?? defaultProviderIdForTemplate(templateId);
+    const options = parseProviderAddOptions(tokens);
+    const outputPath = await addProviderFromTemplate(cwd, options);
+    const resolvedProviderId = options.providerId ?? defaultProviderIdForTemplate(options.templateId);
     console.log(`Updated ${outputPath}`);
-    console.log(`Added provider ${resolvedProviderId} from template ${templateId}`);
+    console.log(`Added provider ${resolvedProviderId} from template ${options.templateId}`);
+    if (options.baseUrl) {
+      console.log(`Base URL ${options.baseUrl}`);
+    }
     console.log(`Next: /providers use ${resolvedProviderId}`);
     return;
   }
   if (subcommand === "use") {
     const providerId = tokens[2];
-    const role = tokens[3];
     if (!providerId) {
-      throw new Error("Usage: /providers use <provider-id> [role]");
+      throw new Error("Usage: /providers use <provider-id>");
     }
-    const targetRole = role && isBrainRole(role) ? role : runtime.focusRole;
-    const result = await useProviderForRole(cwd, targetRole, providerId);
-    runtime.focusRole = targetRole;
+    const result = await useProviderGlobally(cwd, providerId);
     runtime.lastModels = null;
     console.log(`Updated ${result.outputPath}`);
-    console.log(`Focus role ${targetRole} now uses provider ${providerId}${result.model ? ` (${result.model})` : ""}`);
+    console.log(`All roles now use provider ${providerId}${result.model ? ` (${result.model})` : ""}`);
     return;
   }
   throw new Error("Usage: /providers [list|templates|add|use|check]");
@@ -11089,10 +11188,10 @@ async function handleModelsCommand(cwd, tokens, runtime) {
     if (!modelId) {
       throw new Error("Usage: /models use <model-id>");
     }
-    const outputPath = await assignModelToRole(cwd, runtime.focusRole, provider.id, modelId);
+    const outputPath = await assignModelGlobally(cwd, provider.id, modelId);
     console.log(`Updated ${outputPath}`);
     console.log(`Selected ${provider.id}/${modelId}`);
-    console.log(`Assigned role ${runtime.focusRole}`);
+    console.log("Assigned all roles");
     return;
   }
   const search = !subcommand || MODEL_SUBCOMMANDS.has(subcommand) ? void 0 : tokens.slice(1).join(" ");
@@ -11118,7 +11217,7 @@ async function handleModelsCommand(cwd, tokens, runtime) {
     console.log(`  ${index + 1}. ${renderModelLine2(model)}`);
   }
   console.log(``);
-  console.log(`Use /select <number> or /use-model <model-id> to assign one to ${runtime.focusRole}.`);
+  console.log("Use /select <number> or /use-model <model-id> to assign one to all roles.");
 }
 async function handleModelSelection(cwd, modelId, runtime) {
   const config = await loadConfig(cwd);
@@ -11130,10 +11229,10 @@ async function handleModelSelection(cwd, modelId, runtime) {
   if (!provider) {
     throw new Error(`Active role "${runtime.focusRole}" points to unknown provider "${providerId}".`);
   }
-  const outputPath = await assignModelToRole(cwd, runtime.focusRole, provider.id, modelId);
+  const outputPath = await assignModelGlobally(cwd, provider.id, modelId);
   console.log(`Updated ${outputPath}`);
   console.log(`Selected ${provider.id}/${modelId}`);
-  console.log(`Assigned role ${runtime.focusRole}`);
+  console.log("Assigned all roles");
 }
 async function handleSelectCommand(cwd, tokens, runtime) {
   const rawIndex = tokens[1];