mini-coder 0.5.1 → 0.5.3

package/src/index.ts

@@ -28,6 +28,7 @@ import {
   resolveHeadlessPrompt,
   shouldUseHeadlessMode,
 } from "./cli.ts";
+import { getErrorMessage } from "./errors.ts";
 import { type GitState, getGitState } from "./git.ts";
 import { canonicalizePath } from "./paths.ts";
 import {
@@ -56,6 +57,7 @@ import {
   truncateSessions,
 } from "./session.ts";
 import {
+  type CustomProvider,
   loadSettings,
   resolveStartupSettings,
   type UserSettings,
@@ -70,6 +72,7 @@ import {
   readImageTool,
   shellTool,
 } from "./tools.ts";
+import { resolveAppVersionLabel } from "./version.ts";
 
 // ---------------------------------------------------------------------------
 // Constants
@@ -98,10 +101,6 @@ export const MAX_SESSIONS_PER_CWD = 20;
 /** Maximum raw prompt-history entries to retain globally. */
 export const MAX_PROMPT_HISTORY = 1_000;
 
-function getErrorMessage(error: unknown): string {
-  return error instanceof Error ? error.message : String(error);
-}
-
 // ---------------------------------------------------------------------------
 // OAuth credential persistence
 // ---------------------------------------------------------------------------
@@ -211,6 +210,101 @@ async function discoverProviders(): Promise<DiscoveryResult> {
   return { providers, oauthCredentials };
 }
 
+// ---------------------------------------------------------------------------
+// Custom provider discovery
+// ---------------------------------------------------------------------------
+
+/** Timeout for custom provider model discovery requests. */
+const CUSTOM_PROVIDER_TIMEOUT_MS = 3_000;
+
+/** Default API key for custom providers that don't require authentication. */
+const CUSTOM_PROVIDER_DEFAULT_KEY = "no-key";
+
+/** Result of custom provider discovery. */
+interface CustomDiscoveryResult {
+  /** Discovered models from all reachable custom providers. */
+  models: Model<"openai-completions">[];
+  /** Provider name → API key for discovered providers. */
+  providers: Map<string, string>;
+  /** Warning messages for unreachable or invalid providers. */
+  warnings: string[];
+}
+
+/**
+ * Discover models from user-configured OpenAI-compatible endpoints.
+ *
+ * Queries each provider's `/models` endpoint and constructs pi-ai Model
+ * objects from the response. Unreachable endpoints produce a warning
+ * instead of failing startup.
+ *
+ * @param customProviders - Configured custom provider entries.
+ * @param builtInProviderNames - Names of built-in providers (to detect collisions).
+ * @returns Discovered models, provider credentials, and warnings.
+ */
+export async function discoverCustomProviders(
+  customProviders: readonly CustomProvider[],
+  builtInProviderNames: ReadonlySet<string>,
+): Promise<CustomDiscoveryResult> {
+  const models: Model<"openai-completions">[] = [];
+  const providers = new Map<string, string>();
+  const warnings: string[] = [];
+
+  for (const entry of customProviders) {
+    if (builtInProviderNames.has(entry.name)) {
+      warnings.push(
+        `Custom provider "${entry.name}" skipped: name conflicts with a built-in provider.`,
+      );
+      continue;
+    }
+
+    const apiKey = entry.apiKey ?? CUSTOM_PROVIDER_DEFAULT_KEY;
+    const modelsUrl = `${entry.baseUrl}/models`;
+
+    try {
+      const response = await fetch(modelsUrl, {
+        signal: AbortSignal.timeout(CUSTOM_PROVIDER_TIMEOUT_MS),
+      });
+
+      if (!response.ok) {
+        warnings.push(
+          `Custom provider "${entry.name}": ${response.status} ${response.statusText} (${modelsUrl})`,
+        );
+        continue;
+      }
+
+      const body = (await response.json()) as {
+        data?: { id: string }[];
+      };
+      const modelList = body.data ?? [];
+
+      for (const item of modelList) {
+        if (typeof item.id !== "string" || !item.id) continue;
+
+        models.push({
+          id: item.id,
+          name: item.id,
+          api: "openai-completions",
+          provider: entry.name,
+          baseUrl: entry.baseUrl,
+          reasoning: false,
+          input: ["text"],
+          cost: { input: 0, output: 0, cacheRead: 0, cacheWrite: 0 },
+          contextWindow: 131072,
+          maxTokens: 8192,
+        });
+      }
+
+      providers.set(entry.name, apiKey);
+    } catch (error) {
+      warnings.push(
+        `Custom provider "${entry.name}": ${getErrorMessage(error)} (${modelsUrl})`,
+      );
+    }
+  }
+
+  return { models, providers, warnings };
+}
+
 // ---------------------------------------------------------------------------
 // Model selection
 // ---------------------------------------------------------------------------
@@ -440,8 +534,12 @@ export interface AppState {
   plugins: LoadedPlugin[];
   /** Active theme (default + plugin overrides). */
   theme: Theme;
+  /** Version label shown in the empty conversation banner. */
+  versionLabel: string;
   /** Current git state (null if not in a repo). */
   git: GitState | null;
+  /** Optional git state loader override used by tests. */
+  loadGitState?: (cwd: string) => Promise<GitState | null>;
   /** Available provider credentials (provider → API key). */
   providers: Map<string, string>;
   /** OAuth credentials on disk. */
@@ -464,6 +562,10 @@ export interface AppState {
   showReasoning: boolean;
   /** Whether to show full (un-truncated) tool output. */
   verbose: boolean;
+  /** Models discovered from custom OpenAI-compatible providers. */
+  customModels: Model<string>[];
+  /** Warnings from startup (e.g. unreachable custom providers). */
+  startupWarnings: string[];
 }
 
 // ---------------------------------------------------------------------------
@@ -482,7 +584,21 @@ export async function init(): Promise<AppState> {
 
   // Load user settings and resolve startup defaults
   const settings = loadSettings(SETTINGS_PATH);
-  const availableModels = listAvailableModels(providers);
+
+  // Discover custom providers from settings
+  const builtInProviderNames = new Set(providers.keys());
+  const customResult = await discoverCustomProviders(
+    settings.customProviders ?? [],
+    builtInProviderNames,
+  );
+
+  // Merge custom provider credentials
+  for (const [name, key] of customResult.providers) {
+    providers.set(name, key);
+  }
+
+  const builtInModels = listAvailableModels(providers);
+  const availableModels = [...builtInModels, ...customResult.models];
   const startup = resolveStartupSettings(
     settings,
     availableModels.map((model) => `${model.provider}/${model.id}`),
@@ -509,6 +625,7 @@ export async function init(): Promise<AppState> {
     skills: promptContext.skills,
     plugins: promptContext.plugins,
     theme: promptContext.theme,
+    versionLabel: resolveAppVersionLabel(),
     git: promptContext.git,
     providers,
     oauthCredentials,
@@ -521,6 +638,8 @@ export async function init(): Promise<AppState> {
     activeTurnPromise: null,
     showReasoning: startup.showReasoning,
     verbose: startup.verbose,
+    customModels: customResult.models,
+    startupWarnings: customResult.warnings,
   };
 }
 
@@ -593,7 +712,7 @@ export function ensureSession(
  * for, suitable for the `/model` selector.
  */
 export function getAvailableModels(state: AppState): Model<string>[] {
-  return listAvailableModels(state.providers);
+  return [...listAvailableModels(state.providers), ...state.customModels];
 }
 
 /** Clean up resources on shutdown. */

package/src/prompt.ts

@@ -192,14 +192,14 @@ const BASE_INSTRUCTIONS = `You are mini-coder, a coding agent running in the use
 
 # Role
 
-You are an autonomous, senior-level coding assistant. When the user gives a direction, proactively gather context, plan with the user, implement, and verify. Bias toward action: use planning first to clear any assumptions with the user, then implement the plan. Deliver working code, unless you are genuinely blocked.
+You are an autonomous, senior-level coding assistant. When the user gives a direction, proactively gather context, plan with the user, implement, and verify. Bias toward action: plan briefly when needed to clear important assumptions, then continue into implementation. First identify the task contract: required files, names, interfaces, output format, and checks for success. Treat those details as part of correctness, not as polish. Deliver working code, unless you are genuinely blocked.
 
 # Tools
 
 You have these core tools:
 
-- \`shell\` — run commands in the user's shell. Use this to explore the codebase (rg, find, ls, cat), run tests, build, git, and any other command. Prefer \`rg\` over \`grep\` for speed.
-- \`edit\` — make exact-text replacements in files. Provide the file path, the exact text to find, and the replacement text. The old text must match exactly one location in the file. To create a new file, use an empty old text and the full file content as new text.
+- \`shell\` — run commands in the user's shell. Use this to explore the codebase, read tests/verifiers/examples, inspect required outputs, and run targeted checks, builds, or git commands. Prefer \`rg\` over \`grep\` for speed.
+- \`edit\` — make exact-text replacements in files. Provide the file path, the exact text to find, and the replacement text. The old text must match exactly one location in the file. To create a new file, use an empty old text and the full file content as new text. Use this to write the exact final file content the task requires.
 
 You may also have additional tools provided by plugins. Use them when they match the task.
 
@@ -208,7 +208,7 @@ Workflow: **inspect with shell → mutate with edit → verify with shell**.
 # Code quality
 
 - Conform to the codebase's existing conventions: patterns, naming, formatting, language idioms.
-- Write correct, clear, minimal code. Don't over-engineer, don't add abstractions for hypothetical futures.
+- Write correct, clear, minimal code. Prefer the simplest solution that satisfies the task's checks exactly. Don't over-engineer, don't add abstractions for hypothetical futures.
 - Reuse before creating. Search for existing helpers before writing new ones.
 - Tight error handling: no broad try/catch, no silent failures, no swallowed errors.
 - Keep type safety. Avoid \`any\` casts. Use proper types and guards.
@@ -224,6 +224,7 @@ Workflow: **inspect with shell → mutate with edit → verify with shell**.
 # Exploring the codebase
 
 - Think first: before any tool call, decide all files and information you need.
+- Early in the task, look for acceptance criteria in tests, verifier scripts, eval scripts, examples, and expected-output files. Do not rely on the task text alone when machine-checkable criteria are available.
 - Batch reads: if you need multiple files, read them together in parallel rather than one at a time.
 - Only make sequential calls when a later call genuinely depends on an earlier result.
 
@@ -238,7 +239,9 @@ Workflow: **inspect with shell → mutate with edit → verify with shell**.
 # Persistence
 
 - Carry work through to completion within the current turn. Don't stop at analysis or partial fixes.
+- Once the contract is clear, create the required artifact early, then iterate and improve it. Do not spend most of the turn exploring.
 - If you encounter an error, diagnose and fix it rather than reporting it and stopping.
+- Before concluding, run the smallest targeted verification that checks the exact contract: required files exist, names and signatures match, outputs are in the required format, and no forbidden extra artifacts were left behind.
 - Avoid excessive looping: if you're re-reading or re-editing the same files without progress, stop and ask the user.`;
 
 // ---------------------------------------------------------------------------

package/src/settings.ts

@@ -10,6 +10,17 @@
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
 import { dirname } from "node:path";
 import type { ThinkingLevel } from "@mariozechner/pi-ai";
+import { getErrorMessage } from "./errors.ts";
+
+/** A user-configured OpenAI-compatible provider endpoint. */
+export interface CustomProvider {
+  /** Provider identifier, e.g. "ollama". Shown as the provider prefix in model names. */
+  name: string;
+  /** OpenAI-compatible API base URL, e.g. "http://localhost:11434/v1". */
+  baseUrl: string;
+  /** Optional API key. Defaults to "no-key" at discovery time. */
+  apiKey?: string;
+}
 
 /** Default reasoning effort when no saved setting exists. */
 const DEFAULT_EFFORT: ThinkingLevel = "medium";
@@ -30,6 +41,8 @@ export interface UserSettings {
   showReasoning?: boolean;
   /** Whether full tool output is shown in the UI. */
   verbose?: boolean;
+  /** Custom OpenAI-compatible provider endpoints. */
+  customProviders?: CustomProvider[];
 }
 
 /** Resolved startup settings after applying defaults and availability checks. */
@@ -51,10 +64,6 @@ const THINKING_LEVELS = new Set<ThinkingLevel>([
   "xhigh",
 ]);
 
-function getErrorMessage(error: unknown): string {
-  return error instanceof Error ? error.message : String(error);
-}
-
 /**
  * Load and validate user settings from disk.
  *
@@ -175,9 +184,62 @@ function sanitizeSettings(value: unknown): UserSettings {
     settings.verbose = candidate.verbose;
   }
 
+  const customProviders = sanitizeCustomProviders(candidate.customProviders);
+  if (customProviders) {
+    settings.customProviders = customProviders;
+  }
+
   return settings;
 }
 
+/** Try to parse a single custom provider entry, returning null on failure. */
+function parseCustomProvider(item: unknown): CustomProvider | null {
+  if (item == null || typeof item !== "object" || Array.isArray(item)) {
+    return null;
+  }
+
+  const candidate = item as Record<string, unknown>;
+  const name = typeof candidate.name === "string" ? candidate.name.trim() : "";
+  const baseUrl =
+    typeof candidate.baseUrl === "string" ? candidate.baseUrl.trim() : "";
+
+  if (!name || !baseUrl) {
+    return null;
+  }
+
+  const entry: CustomProvider = { name, baseUrl };
+  if (typeof candidate.apiKey === "string") {
+    entry.apiKey = candidate.apiKey;
+  }
+  return entry;
+}
+
+/**
+ * Validate and normalize custom provider entries.
+ *
+ * Drops entries with missing/empty name or baseUrl, and deduplicates by name
+ * (first entry wins).
+ */
+function sanitizeCustomProviders(value: unknown): CustomProvider[] | undefined {
+  if (!Array.isArray(value)) {
+    return undefined;
+  }
+
+  const result: CustomProvider[] = [];
+  const seen = new Set<string>();
+
+  for (const item of value) {
+    const entry = parseCustomProvider(item);
+    if (!entry || seen.has(entry.name)) {
+      continue;
+    }
+    seen.add(entry.name);
+    result.push(entry);
+  }
+
+  return result.length > 0 ? result : undefined;
+}
+
 /**
  * Check whether a value is a valid thinking level.
  *

package/src/submit.ts

@@ -8,6 +8,7 @@ import { readFileSync } from "node:fs";
 import type { UserMessage } from "@mariozechner/pi-ai";
 import type { AgentEvent } from "./agent.ts";
 import { runAgentLoop } from "./agent.ts";
+import { getErrorMessage } from "./errors.ts";
 import { getGitState } from "./git.ts";
 import {
   type AppState,
@@ -103,10 +104,6 @@ export function isEmptyUserContent(content: UserMessage["content"]): boolean {
   );
 }
 
-function getErrorMessage(error: unknown): string {
-  return error instanceof Error ? error.message : String(error);
-}
-
 function buildSkillMessageContent(
   skillName: string,
   userText: string,
@@ -276,12 +273,13 @@ export async function submitResolvedInput(
     content,
     timestamp: Date.now(),
   } satisfies UserMessage;
+  const loadGitState = state.loadGitState ?? getGitState;
 
   const turn = appendMessage(state.db, session.id, userMessage);
   state.messages.push(userMessage);
   hooks?.onUserMessage?.(state);
 
-  state.git = await getGitState(state.cwd);
+  state.git = await loadGitState(state.cwd);
 
   const systemPrompt = buildPrompt(state);
   const { tools, toolHandlers } = buildToolList(state);
@@ -313,7 +311,7 @@ export async function submitResolvedInput(
       },
     });
     stopReason = result.stopReason;
-    state.git = await getGitState(state.cwd);
+    state.git = await loadGitState(state.cwd);
     return result.stopReason;
   } finally {
     state.running = false;