gsd-pi 2.58.0-dev.778d6ac → 2.58.0-dev.d63175c

package/src/resources/extensions/gsd/tests/worktree-db-respawn-truncation.test.ts

@@ -1,140 +0,0 @@
-/**
- * worktree-db-respawn-truncation.test.ts — Regression test for #2815.
- *
- * Verifies that syncProjectRootToWorktree does NOT delete a non-empty
- * worktree gsd.db. On worker respawn, gsd-migrate populates the DB
- * (~1.7MB) before the auto-loop calls syncProjectRootToWorktree. The
- * sync step must preserve the freshly-migrated DB to avoid truncating
- * it to 0 bytes and causing "no such table: slices" failures.
- *
- * Covers:
- *   - Non-empty worktree gsd.db preserved after sync (#2815)
- *   - Empty (0-byte) worktree gsd.db still deleted (#853 preserved)
- *   - WAL/SHM sidecar files cleaned up when empty DB is deleted
- */
-
-import { mkdtempSync, mkdirSync, writeFileSync, rmSync, existsSync, readFileSync, statSync } from 'node:fs';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-
-import { syncProjectRootToWorktree } from '../auto-worktree.ts';
-import { describe, test } from 'node:test';
-import assert from 'node:assert/strict';
-
-
-function createBase(name: string): string {
-  const base = mkdtempSync(join(tmpdir(), `gsd-wt-respawn-${name}-`));
-  mkdirSync(join(base, '.gsd', 'milestones'), { recursive: true });
-  return base;
-}
-
-function cleanup(base: string): void {
-  rmSync(base, { recursive: true, force: true });
-}
-
-describe('worktree-db-respawn-truncation (#2815)', async () => {
-
-  // ─── 1. Non-empty worktree gsd.db preserved after sync ───────────────
-  console.log('\n=== 1. non-empty worktree gsd.db preserved after sync (#2815) ===');
-  {
-    const mainBase = createBase('main');
-    const wtBase = createBase('wt');
-
-    try {
-      // Set up milestone artifacts in main project root
-      const m001Dir = join(mainBase, '.gsd', 'milestones', 'M001');
-      mkdirSync(m001Dir, { recursive: true });
-      writeFileSync(join(m001Dir, 'M001-ROADMAP.md'), '# Roadmap');
-
-      // Simulate a freshly-migrated worktree DB (non-empty, like after gsd-migrate)
-      // Real DBs are ~1.7MB; we use a smaller payload to prove the size check works
-      const fakeDbContent = Buffer.alloc(4096, 0x42); // 4KB non-empty DB
-      writeFileSync(join(wtBase, '.gsd', 'gsd.db'), fakeDbContent);
-
-      const sizeBefore = statSync(join(wtBase, '.gsd', 'gsd.db')).size;
-      assert.ok(sizeBefore > 0, 'gsd.db is non-empty before sync');
-
-      syncProjectRootToWorktree(mainBase, wtBase, 'M001');
-
-      // The non-empty DB must survive the sync
-      assert.ok(
-        existsSync(join(wtBase, '.gsd', 'gsd.db')),
-        '#2815: non-empty gsd.db must not be deleted by sync',
-      );
-      const sizeAfter = statSync(join(wtBase, '.gsd', 'gsd.db')).size;
-      assert.equal(
-        sizeAfter,
-        sizeBefore,
-        '#2815: gsd.db size must be unchanged after sync',
-      );
-    } finally {
-      cleanup(mainBase);
-      cleanup(wtBase);
-    }
-  }
-
-  // ─── 2. Empty (0-byte) worktree gsd.db still deleted ─────────────────
-  console.log('\n=== 2. empty (0-byte) worktree gsd.db still deleted (#853) ===');
-  {
-    const mainBase = createBase('main');
-    const wtBase = createBase('wt');
-
-    try {
-      const m001Dir = join(mainBase, '.gsd', 'milestones', 'M001');
-      mkdirSync(m001Dir, { recursive: true });
-      writeFileSync(join(m001Dir, 'M001-ROADMAP.md'), '# Roadmap');
-
-      // Create an empty (0-byte) gsd.db — this is stale/corrupt and should be deleted
-      writeFileSync(join(wtBase, '.gsd', 'gsd.db'), '');
-      assert.ok(existsSync(join(wtBase, '.gsd', 'gsd.db')), 'empty gsd.db exists before sync');
-
-      syncProjectRootToWorktree(mainBase, wtBase, 'M001');
-
-      assert.ok(
-        !existsSync(join(wtBase, '.gsd', 'gsd.db')),
-        '#853: empty gsd.db must still be deleted after sync',
-      );
-    } finally {
-      cleanup(mainBase);
-      cleanup(wtBase);
-    }
-  }
-
-  // ─── 3. Milestone artifacts still synced when DB is preserved ────────
-  console.log('\n=== 3. milestone artifacts still synced even when DB preserved ===');
-  {
-    const mainBase = createBase('main');
-    const wtBase = createBase('wt');
-
-    try {
-      const m001Dir = join(mainBase, '.gsd', 'milestones', 'M001');
-      mkdirSync(m001Dir, { recursive: true });
-      writeFileSync(join(m001Dir, 'M001-ROADMAP.md'), '# Roadmap');
-      mkdirSync(join(m001Dir, 'slices', 'S01'), { recursive: true });
-      writeFileSync(join(m001Dir, 'slices', 'S01', 'S01-PLAN.md'), '# Plan');
-
-      // Non-empty DB in worktree
-      writeFileSync(join(wtBase, '.gsd', 'gsd.db'), 'populated-db-data');
-
-      syncProjectRootToWorktree(mainBase, wtBase, 'M001');
-
-      // Artifacts must still be synced
-      assert.ok(
-        existsSync(join(wtBase, '.gsd', 'milestones', 'M001', 'M001-ROADMAP.md')),
-        'milestone artifacts synced even with preserved DB',
-      );
-      assert.ok(
-        existsSync(join(wtBase, '.gsd', 'milestones', 'M001', 'slices', 'S01', 'S01-PLAN.md')),
-        'slice artifacts synced even with preserved DB',
-      );
-      // DB must still exist
-      assert.ok(
-        existsSync(join(wtBase, '.gsd', 'gsd.db')),
-        '#2815: DB preserved alongside artifact sync',
-      );
-    } finally {
-      cleanup(mainBase);
-      cleanup(wtBase);
-    }
-  }
-});

package/src/resources/extensions/gsd/tests/worktree-nested-git-safety.test.ts

@@ -1,101 +0,0 @@
-/**
- * worktree-nested-git-safety.test.ts — #2616
- *
- * When scaffolding tools (create-next-app, cargo init, etc.) run inside a
- * worktree, they create nested .git directories. Git treats these as gitlinks
- * (mode 160000) without a .gitmodules entry, so the worktree cleanup destroys
- * the only copy of those object databases — causing permanent data loss.
- *
- * This test verifies that removeWorktree detects nested .git directories
- * (orphaned gitlinks) and absorbs or removes them before cleanup so files
- * are tracked as regular content instead of unreachable gitlink pointers.
- */
-
-import { readFileSync } from "node:fs";
-import { join } from "node:path";
-import { createTestContext } from "./test-helpers.ts";
-
-const { assertTrue, report } = createTestContext();
-
-const srcPath = join(import.meta.dirname, "..", "worktree-manager.ts");
-const src = readFileSync(srcPath, "utf-8");
-
-console.log("\n=== #2616: Worktree cleanup detects nested .git directories ===");
-
-// ── Test 1: removeWorktree scans for nested .git directories ─────────
-
-const removeWorktreeIdx = src.indexOf("export function removeWorktree");
-assertTrue(removeWorktreeIdx > 0, "worktree-manager.ts exports removeWorktree");
-
-const fnBody = src.slice(removeWorktreeIdx, removeWorktreeIdx + 5000);
-
-const detectsNestedGit =
-  fnBody.includes("nested") && fnBody.includes(".git") ||
-  fnBody.includes("gitlink") ||
-  fnBody.includes("160000") ||
-  fnBody.includes("findNestedGitDirs") ||
-  fnBody.includes("nestedGitDirs");
-
-assertTrue(
-  detectsNestedGit,
-  "removeWorktree detects nested .git directories or gitlinks (#2616)",
-);
-
-// ── Test 2: A helper function exists to find nested .git directories ──
-
-const hasNestedGitHelper =
-  src.includes("findNestedGitDirs") ||
-  src.includes("detectNestedGitDirs") ||
-  src.includes("scanNestedGit") ||
-  src.includes("absorbNestedGit") ||
-  src.includes("nestedGitDirs");
-
-assertTrue(
-  hasNestedGitHelper,
-  "worktree-manager has a helper to find nested .git directories (#2616)",
-);
-
-// ── Test 3: Nested .git dirs are absorbed or removed before cleanup ───
-
-const absorbsOrRemoves =
-  fnBody.includes("absorb") ||
-  fnBody.includes("rmSync") && fnBody.includes("nested") ||
-  (fnBody.includes("nestedGitDirs") || fnBody.includes("findNestedGitDirs")) &&
-    (fnBody.includes("rm") || fnBody.includes("absorb") || fnBody.includes("remove"));
-
-assertTrue(
-  absorbsOrRemoves,
-  "removeWorktree absorbs or removes nested .git dirs before cleanup (#2616)",
-);
-
-// ── Test 4: A warning is logged when nested .git dirs are found ───────
-
-const warnsAboutNestedGit =
-  fnBody.includes("nested") && fnBody.includes("logWarning") ||
-  fnBody.includes("gitlink") && fnBody.includes("logWarning") ||
-  fnBody.includes("scaffold") && fnBody.includes("logWarning");
-
-assertTrue(
-  warnsAboutNestedGit,
-  "removeWorktree warns when nested .git directories are detected (#2616)",
-);
-
-// ── Test 5: The findNestedGitDirs helper correctly identifies nested repos ──
-// Verify the helper scans subdirectories but skips .gsd/, node_modules/, .git/
-
-const helperBody = src.includes("findNestedGitDirs")
-  ? src.slice(src.indexOf("findNestedGitDirs"))
-  : "";
-
-const skipsExcludedDirs =
-  helperBody.includes("node_modules") ||
-  helperBody.includes(".gsd") ||
-  helperBody.includes("skip") ||
-  helperBody.includes("exclude");
-
-assertTrue(
-  skipsExcludedDirs,
-  "findNestedGitDirs skips node_modules and other excluded directories (#2616)",
-);
-
-report();

package/src/resources/extensions/gsd/tests/zombie-gsd-state.test.ts

@@ -1,95 +0,0 @@
-import { readFileSync } from "node:fs";
-import { join } from "node:path";
-
-import { createTestContext } from "./test-helpers.ts";
-
-const { assertTrue, assertMatch, assertNoMatch, report } = createTestContext();
-
-// ─── #2942: Zombie .gsd state skips init wizard ─────────────────────────────
-//
-// A partially initialized .gsd/ (symlink exists but no PREFERENCES.md or
-// milestones/) causes the init wizard gate in showSmartEntry to be skipped,
-// resulting in an uninitialized project session.
-
-console.log("\n=== #2942: zombie .gsd state must not skip init wizard ===");
-
-// ── guided-flow.ts — init wizard gate must check bootstrap completeness ──
-
-const guidedFlowSrc = readFileSync(
-  join(import.meta.dirname, "..", "guided-flow.ts"),
-  "utf-8",
-);
-
-// Find the showSmartEntry function
-const smartEntryIdx = guidedFlowSrc.indexOf("export async function showSmartEntry(");
-assertTrue(smartEntryIdx >= 0, "guided-flow.ts defines showSmartEntry");
-
-// Extract the region between showSmartEntry and the first showProjectInit call
-// This is where the init wizard gate lives.
-const afterSmartEntry = smartEntryIdx >= 0 ? guidedFlowSrc.slice(smartEntryIdx, smartEntryIdx + 3000) : "";
-
-// The gate must NOT be a bare `!existsSync(gsdRoot(basePath))` check.
-// It must also verify that bootstrap artifacts (PREFERENCES.md or milestones/) exist.
-assertTrue(
-  afterSmartEntry.includes("PREFERENCES.md") || afterSmartEntry.includes("PREFERENCES"),
-  "init wizard gate checks for PREFERENCES.md, not just .gsd/ existence (#2942)",
-);
-
-assertTrue(
-  afterSmartEntry.includes("milestones"),
-  "init wizard gate checks for milestones/ directory, not just .gsd/ existence (#2942)",
-);
-
-// The init wizard should be shown when .gsd/ exists but has no bootstrap artifacts.
-// The old code was: if (!existsSync(gsdRoot(basePath))) { ... showProjectInit ... }
-// The fix should use a compound check so zombie states trigger the wizard.
-// Verify we no longer have the bare existence check as the sole gate.
-
-// Find the specific init wizard gate pattern — the detection preamble block.
-const detectionPreambleIdx = afterSmartEntry.indexOf("Detection preamble");
-const detectionRegion = detectionPreambleIdx >= 0
-  ? afterSmartEntry.slice(detectionPreambleIdx, detectionPreambleIdx + 600)
-  : afterSmartEntry.slice(0, 1500);
-
-// The gate condition must reference PREFERENCES.md or milestones (bootstrap artifacts)
-assertMatch(
-  detectionRegion,
-  /PREFERENCES\.md|milestones/,
-  "detection preamble gate references bootstrap artifacts, not just directory existence (#2942)",
-);
-
-// ── auto-start.ts — milestones/ dir creation must not be dead code ──────────
-
-console.log("\n=== #2942: auto-start milestones/ bootstrap not dead code ===");
-
-const autoStartSrc = readFileSync(
-  join(import.meta.dirname, "..", "auto-start.ts"),
-  "utf-8",
-);
-
-// After ensureGsdSymlink, the code that creates milestones/ must check for
-// the milestones directory specifically (not .gsd/ which ensureGsdSymlink already created).
-const symlinkIdx = autoStartSrc.indexOf("ensureGsdSymlink(base)");
-assertTrue(symlinkIdx >= 0, "auto-start.ts calls ensureGsdSymlink(base)");
-
-const afterSymlink = symlinkIdx >= 0 ? autoStartSrc.slice(symlinkIdx, symlinkIdx + 800) : "";
-
-// The milestones bootstrap must check milestones path, not gsdDir
-// Old (dead) code: if (!existsSync(gsdDir)) { mkdirSync(join(gsdDir, "milestones"), ...) }
-// Fixed code should check: if (!existsSync(milestonesPath)) or similar
-assertTrue(
-  afterSymlink.includes("milestones") && afterSymlink.includes("mkdirSync"),
-  "auto-start.ts creates milestones/ directory after ensureGsdSymlink (#2942)",
-);
-
-// The guard for milestones/ creation should NOT be `!existsSync(gsdDir)` —
-// that's dead code since ensureGsdSymlink already created gsdDir.
-// It should check for the milestones/ dir directly.
-const mkdirRegion = afterSymlink.slice(0, afterSymlink.indexOf("mkdirSync") + 200);
-assertMatch(
-  mkdirRegion,
-  /existsSync\([^)]*milestones/,
-  "milestones bootstrap checks milestones path existence, not .gsd/ (#2942)",
-);
-
-report();

package/src/resources/extensions/mcp-client/tests/server-name-spaces.test.ts

@@ -1,55 +0,0 @@
-/**
- * Regression test for #3029 — mcp_discover fails for server names with spaces.
- *
- * The getServerConfig lookup must handle:
- *   1. Exact match (already works)
- *   2. Names with leading/trailing whitespace (trimming)
- *   3. Case-insensitive matching (e.g. "Langgraph code" vs "langgraph Code")
- *
- * We test at the source level since getServerConfig is not exported.
- */
-
-import test from "node:test";
-import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
-import { dirname, join } from "node:path";
-import { fileURLToPath } from "node:url";
-
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = dirname(__filename);
-
-const source = readFileSync(join(__dirname, "..", "index.ts"), "utf-8");
-
-test("#3029: getServerConfig trims whitespace from input name", () => {
-	assert.ok(
-		source.includes(".trim()"),
-		"getServerConfig should trim the input name before comparison",
-	);
-});
-
-test("#3029: getServerConfig performs case-insensitive matching", () => {
-	assert.ok(
-		source.includes(".toLowerCase()"),
-		"getServerConfig should compare names case-insensitively",
-	);
-});
-
-test("#3029: getOrConnect normalizes name for connection cache lookup", () => {
-	// The connections Map key must use the canonical (config) name, not the
-	// raw user input, so that subsequent lookups hit the cache even when the
-	// user's casing differs.
-	const getOrConnectMatch = source.match(
-		/async function getOrConnect\(name: string[\s\S]*?const existing = connections\.get\(/,
-	);
-	assert.ok(
-		getOrConnectMatch,
-		"getOrConnect function should exist",
-	);
-	// After the fix, getOrConnect should normalize the name via getServerConfig
-	// or use config.name as the canonical cache key.
-	assert.ok(
-		source.includes("connections.get(config.name") ||
-		source.includes("connections.set(config.name"),
-		"getOrConnect should use config.name (canonical) as the connections cache key",
-	);
-});

package/src/resources/extensions/ollama/index.ts

@@ -1,130 +0,0 @@
-// GSD2 — Ollama Extension: First-class local LLM support
-/**
- * Ollama Extension
- *
- * Auto-detects a running Ollama instance, discovers locally pulled models,
- * and registers them as a first-class provider. No configuration required —
- * if Ollama is running, models appear automatically.
- *
- * Features:
- * - Auto-discovery of local models via /api/tags
- * - Capability detection (vision, reasoning, context window)
- * - /ollama slash commands for model management
- * - ollama_manage tool for LLM-driven model operations
- * - Zero-cost model registration (local inference)
- *
- * Respects OLLAMA_HOST env var for non-default endpoints.
- */
-
-import { importExtensionModule, type ExtensionAPI } from "@gsd/pi-coding-agent";
-import type { OpenAICompletionsCompat } from "@gsd/pi-ai";
-import * as client from "./ollama-client.js";
-import { discoverModels, getOllamaOpenAIBaseUrl } from "./ollama-discovery.js";
-import { registerOllamaCommands } from "./ollama-commands.js";
-
-/** Default compat settings for Ollama models via OpenAI-compat endpoint */
-const OLLAMA_COMPAT: OpenAICompletionsCompat = {
-	supportsDeveloperRole: false,
-	supportsReasoningEffort: false,
-	supportsUsageInStreaming: false,
-	maxTokensField: "max_tokens",
-	supportsStore: false,
-};
-
-let toolsPromise: Promise<void> | null = null;
-
-async function registerOllamaTools(pi: ExtensionAPI): Promise<void> {
-	if (!toolsPromise) {
-		toolsPromise = (async () => {
-			const { registerOllamaTool } = await importExtensionModule<
-				typeof import("./ollama-tool.js")
-			>(import.meta.url, "./ollama-tool.js");
-			registerOllamaTool(pi);
-		})().catch((error) => {
-			toolsPromise = null;
-			throw error;
-		});
-	}
-	return toolsPromise;
-}
-
-/** Track whether we've registered models so we can clean up on shutdown */
-let providerRegistered = false;
-
-/**
- * Probe Ollama and register discovered models.
- * Safe to call multiple times — re-discovers and re-registers.
- */
-async function probeAndRegister(pi: ExtensionAPI): Promise<boolean> {
-	const running = await client.isRunning();
-	if (!running) {
-		if (providerRegistered) {
-			pi.unregisterProvider("ollama");
-			providerRegistered = false;
-		}
-		return false;
-	}
-
-	const models = await discoverModels();
-	if (models.length === 0) return true; // Running but no models pulled
-
-	const baseUrl = getOllamaOpenAIBaseUrl();
-
-	pi.registerProvider("ollama", {
-		authMode: "none",
-		baseUrl,
-		api: "openai-completions",
-		isReady: () => true,
-		models: models.map((m) => ({
-			id: m.id,
-			name: m.name,
-			reasoning: m.reasoning,
-			input: m.input,
-			cost: m.cost,
-			contextWindow: m.contextWindow,
-			maxTokens: m.maxTokens,
-			compat: OLLAMA_COMPAT,
-		})),
-	});
-
-	providerRegistered = true;
-	return true;
-}
-
-export default function ollama(pi: ExtensionAPI) {
-	// Register slash commands immediately (they check Ollama availability themselves)
-	registerOllamaCommands(pi);
-
-	pi.on("session_start", async (_event, ctx) => {
-		// Register tool (deferred to avoid blocking startup)
-		if (ctx.hasUI) {
-			void registerOllamaTools(pi).catch((error) => {
-				ctx.ui.notify(
-					`Ollama tool failed to load: ${error instanceof Error ? error.message : String(error)}`,
-					"warning",
-				);
-			});
-		} else {
-			await registerOllamaTools(pi);
-		}
-
-		// Async probe — don't block startup
-		probeAndRegister(pi)
-			.then((found) => {
-				if (found && ctx.hasUI) {
-					ctx.ui.setStatus("ollama", "Ollama");
-				}
-			})
-			.catch(() => {
-				// Silently ignore probe failures
-			});
-	});
-
-	pi.on("session_shutdown", async () => {
-		if (providerRegistered) {
-			pi.unregisterProvider("ollama");
-			providerRegistered = false;
-		}
-		toolsPromise = null;
-	});
-}

package/src/resources/extensions/ollama/model-capabilities.ts

@@ -1,145 +0,0 @@
-// GSD2 — Known model capability table for Ollama models
-
-/**
- * Maps well-known Ollama model families to their capabilities.
- * Used to enrich auto-discovered models with accurate context windows,
- * vision support, and reasoning detection.
- *
- * Fallback: estimate from parameter count if model isn't in the table.
- */
-
-export interface ModelCapability {
-	contextWindow?: number;
-	maxTokens?: number;
-	input?: ("text" | "image")[];
-	reasoning?: boolean;
-}
-
-/**
- * Known model family capabilities.
- * Keys are matched as prefixes against the model name (before the colon/tag).
- * More specific entries should appear first.
- */
-const KNOWN_MODELS: Array<[pattern: string, caps: ModelCapability]> = [
-	// ─── Reasoning models ───────────────────────────────────────────────
-	["deepseek-r1", { contextWindow: 131072, reasoning: true }],
-	["qwq", { contextWindow: 131072, reasoning: true }],
-
-	// ─── Vision models ──────────────────────────────────────────────────
-	["llava", { contextWindow: 4096, input: ["text", "image"] }],
-	["bakllava", { contextWindow: 4096, input: ["text", "image"] }],
-	["moondream", { contextWindow: 8192, input: ["text", "image"] }],
-	["llama3.2-vision", { contextWindow: 131072, input: ["text", "image"] }],
-	["minicpm-v", { contextWindow: 4096, input: ["text", "image"] }],
-
-	// ─── Code models ────────────────────────────────────────────────────
-	["codestral", { contextWindow: 262144, maxTokens: 32768 }],
-	["qwen2.5-coder", { contextWindow: 131072, maxTokens: 32768 }],
-	["deepseek-coder-v2", { contextWindow: 131072, maxTokens: 16384 }],
-	["starcoder2", { contextWindow: 16384, maxTokens: 8192 }],
-	["codegemma", { contextWindow: 8192, maxTokens: 8192 }],
-	["codellama", { contextWindow: 16384, maxTokens: 8192 }],
-	["devstral", { contextWindow: 131072, maxTokens: 32768 }],
-
-	// ─── Llama family ───────────────────────────────────────────────────
-	["llama3.3", { contextWindow: 131072, maxTokens: 16384 }],
-	["llama3.2", { contextWindow: 131072, maxTokens: 16384 }],
-	["llama3.1", { contextWindow: 131072, maxTokens: 16384 }],
-	["llama3", { contextWindow: 8192, maxTokens: 8192 }],
-	["llama2", { contextWindow: 4096, maxTokens: 4096 }],
-
-	// ─── Qwen family ────────────────────────────────────────────────────
-	["qwen3", { contextWindow: 131072, maxTokens: 32768 }],
-	["qwen2.5", { contextWindow: 131072, maxTokens: 32768 }],
-	["qwen2", { contextWindow: 131072, maxTokens: 32768 }],
-
-	// ─── Gemma family ───────────────────────────────────────────────────
-	["gemma3", { contextWindow: 131072, maxTokens: 16384 }],
-	["gemma2", { contextWindow: 8192, maxTokens: 8192 }],
-
-	// ─── Mistral family ─────────────────────────────────────────────────
-	["mistral-large", { contextWindow: 131072, maxTokens: 16384 }],
-	["mistral-small", { contextWindow: 131072, maxTokens: 16384 }],
-	["mistral-nemo", { contextWindow: 131072, maxTokens: 16384 }],
-	["mistral", { contextWindow: 32768, maxTokens: 8192 }],
-	["mixtral", { contextWindow: 32768, maxTokens: 8192 }],
-
-	// ─── Phi family ─────────────────────────────────────────────────────
-	["phi4", { contextWindow: 16384, maxTokens: 16384 }],
-	["phi3.5", { contextWindow: 131072, maxTokens: 16384 }],
-	["phi3", { contextWindow: 131072, maxTokens: 4096 }],
-
-	// ─── Command R ──────────────────────────────────────────────────────
-	["command-r-plus", { contextWindow: 131072, maxTokens: 16384 }],
-	["command-r", { contextWindow: 131072, maxTokens: 16384 }],
-];
-
-/**
- * Look up capabilities for a model by name.
- * Matches the longest prefix from the known models table.
- */
-export function getModelCapabilities(modelName: string): ModelCapability {
-	// Strip tag (everything after the colon) for matching
-	const baseName = modelName.split(":")[0].toLowerCase();
-
-	for (const [pattern, caps] of KNOWN_MODELS) {
-		if (baseName === pattern || baseName.startsWith(pattern)) {
-			return caps;
-		}
-	}
-
-	return {};
-}
-
-/**
- * Estimate context window from parameter size string (e.g. "7B", "70B", "1.5B").
- * Used as fallback when model isn't in the known table.
- */
-export function estimateContextFromParams(parameterSize: string): number {
-	const match = parameterSize.match(/([\d.]+)\s*([BbMm])/);
-	if (!match) return 8192;
-
-	const size = parseFloat(match[1]);
-	const unit = match[2].toUpperCase();
-
-	// Convert to billions
-	const billions = unit === "M" ? size / 1000 : size;
-
-	// Rough heuristics: larger models tend to support larger contexts
-	if (billions >= 70) return 131072;
-	if (billions >= 30) return 65536;
-	if (billions >= 13) return 32768;
-	if (billions >= 7) return 16384;
-	return 8192;
-}
-
-/**
- * Humanize a model name for display (e.g. "llama3.1:8b" → "Llama 3.1 8B").
- */
-export function humanizeModelName(modelName: string): string {
-	const [base, tag] = modelName.split(":");
-
-	// Capitalize first letter, add spaces around version numbers
-	let name = base
-		.replace(/([a-z])(\d)/g, "$1 $2")
-		.replace(/(\d)([a-z])/g, "$1 $2")
-		.replace(/^./, (c) => c.toUpperCase());
-
-	// Clean up common patterns
-	name = name.replace(/\s*-\s*/g, " ");
-
-	if (tag && tag !== "latest") {
-		name += ` ${tag.toUpperCase()}`;
-	}
-
-	return name;
-}
-
-/**
- * Format byte size for display (e.g. 4700000000 → "4.7 GB").
- */
-export function formatModelSize(bytes: number): string {
-	if (bytes >= 1e9) return `${(bytes / 1e9).toFixed(1)} GB`;
-	if (bytes >= 1e6) return `${(bytes / 1e6).toFixed(1)} MB`;
-	return `${(bytes / 1e3).toFixed(0)} KB`;
-}