@rizal72/oc-models-pro 0.3.1 → 0.3.2

package/aa.ts

@@ -1,13 +1,4 @@
-// oc-models-pro/aa.ts — tokens per second for the models this machine has never run, from
-// Artificial Analysis' free API. Attribution: https://artificialanalysis.ai/
-//
-// Their number is the same measurement as ours — "the average number of tokens received per second,
-// after the first token is received" — so filling an empty cell with it does not mix two families,
-// the way `int` and speed already do. The panel marks those values with a leading `~`, because they
-// were measured on their infrastructure and not on this person's provider.
-//
-// No key, no calls: without ARTIFICIAL_ANALYSIS_API_KEY this module returns an empty map and nothing
-// leaves the machine. With a key the response is cached six hours, which their terms ask for.
+// oc-models-pro/aa.ts — borrowed speed, from Artificial Analysis' free API.
 //
 // Runnable standalone for its self-check: node plugins/oc-models-pro/aa.ts --self-check
 
@@ -21,7 +12,7 @@ export const AA = "https://artificialanalysis.ai/api/v2/data/llms/models"
 const CACHE_DIR = process.env.XDG_CACHE_HOME || join(homedir(), ".cache")
 const CACHE = join(CACHE_DIR, "oc-models-pro", "aa.json")
 const GRACE_MS = 6 * 60 * 60 * 1000
-/** Optional decoration must never hold the panel: a hanging endpoint gives up after this. */
+/** Optional decoration must never hold the panel. */
 const FETCH_MS = 5_000
 
 interface Cached {
@@ -34,7 +25,7 @@ export function apiKey(): string | undefined {
 	return key && key.trim() ? key.trim() : undefined
 }
 
-/** Name first, slug second — the same meeting point `bench.ts` uses, so one model has one key. */
+/** Name first, slug second. */
 export function buildIndex(payload: any): Map<string, number> {
 	const index = new Map<string, number>()
 	for (const entry of payload?.data ?? []) {
@@ -68,7 +59,6 @@ async function cached(): Promise<Cached | undefined> {
 	}
 }
 
-/** Empty without a key: this is the whole cost of the column being optional. */
 export async function aaIndex(): Promise<Map<string, number>> {
 	const key = apiKey()
 	if (!key) return new Map()

package/bench.ts

@@ -1,16 +1,5 @@
 // oc-models-pro: intelligence per model, from the stencil.so catalog.
 //
-// The catalog is keyed by gateway and its ids are qualified (`alibaba/deepseek-v4.1-flash`), so the
-// comparison uses the last segment — and the key is built from *our* model's display name, never its
-// id. That is what makes the aliases meet: the same model is `deepseek/deepseek-flash` here and
-// "DeepSeek V4.1 Flash 1017" to the eye, while stencil calls it `deepseek-v4.1-flash`. The name is
-// stripped of the labels a catalogue attaches (`latest`, `contributor`, `free`, `preview`) and of
-// snapshot stamps (`1017`, `0731`); the model id is the fallback when the name finds nothing.
-//
-// `int` is on most entries; `tps` was on none of them when this was last measured (2026-10-19), so the
-// speed column is fed by `tps.ts` and `aa.ts` instead — this file's `tps` stays only as the last resort.
-// A figure that is not there is never guessed.
-//
 // Runnable standalone for its self-check: node plugins/oc-models-pro/bench.ts --self-check
 
 import { mkdir, readFile, rename, writeFile } from "node:fs/promises"
@@ -86,7 +75,7 @@ async function cached(): Promise<Cached | undefined> {
 	}
 }
 
-/** Six hours of grace: a benchmark does not move between two coffee breaks. */
+/** Grace, not freshness: a benchmark does not move between two coffee breaks. */
 export async function benchIndex(): Promise<Map<string, Bench>> {
 	const previous = await cached()
 	if (previous && Date.now() - previous.fetched < GRACE_MS) return indexOf(previous)

package/dist/gui-experimental.cjs

@@ -37,9 +37,7 @@ Object.defineProperties(exports,{__esModule:{value:!0},[Symbol.toStringTag]:{val
 .omp-warn { padding: 4px 8px; opacity: .7; }
 .omp-fine { padding: 4px 8px; opacity: .55; font-size: 11px;
   border-top: 1px solid color-mix(in srgb, currentColor 8%, transparent); }
-/* The terminal's cascade, in pixels: the name shrinks first (the track is elastic), then whole columns
-   go in the order fit.ts uses — tok/s, then context, then intelligence. The three price figures never
-   go: they are what the list is for. Below the last rung the name truncates instead of scrolling. */
+/* The terminal's cascade, in pixels: the name shrinks first, then whole columns go in fit.ts's order. */
 @container (max-width: 560px) {
   .omp-tps, .omp-note { display: none; }
   .omp-head, .omp-row { grid-template-columns: minmax(9rem, 1fr) 2.8rem 3.4rem 3.4rem 2.4rem 2.6rem; }

package/fit.ts

@@ -1,19 +1,18 @@
 // oc-models-pro — how a row and the footer fit a panel of a given width, in columns.
 //
-// The host hands `session.panel` a `width` in columns (pixels on the very first frame, which the
-// clamp absorbs). Everything that depends on it lives here, so the thresholds sit in one place:
+// Everything that depends on it lives here, so the thresholds sit in one place:
 //   node plugins/oc-models-pro/fit.ts --self-check
 
-/** The name column at its widest: 32 + the fixed 53 is the 85-character table. */
+/** The name column at its widest. */
 export const NAME = 32
-/** ...and at its narrowest. Below this a model name is unreadable, and a column is worth more. */
+/** ...and at its narrowest. */
 export const NAME_MIN = 24
-/** The cells that never go: marker 2, tag 4, in 6, out 6, cred 4 and the nine separators. */
+/** The cells that never go. */
 const FIXED = 53
 
 export type Fit = { name: number; ctx: boolean; int: boolean; tkps: boolean; note: boolean }
 
-/** Columns dropped as the panel narrows: one with no data, then the note, then context, then int. */
+/** Columns dropped as the panel narrows, in the order the width runs out. */
 const DROPS = [
 	{ key: "tkps", width: 7 },
 	{ key: "note", width: 7 },

package/index.ts

@@ -1,12 +1,4 @@
 // oc-models-pro — server half, for the desktop.
-//
-// The terminal reads the catalogue and the four sources in-process. A plugin window cannot: `bun:sqlite`,
-// `node:fs` and the three fetches are all outside its reach — and the AA key lives in this process's
-// environment, which is where the server has it and the window does not. So the whole table is assembled
-// here and the window asks for it over `models.snapshot`.
-//
-// The rows are the terminal's, field for field, so the two views can be compared: same catalogue, same
-// credits, same speeds. Only the rendering differs.
 import { Plugin } from "@opencode/plugin"
 
 import { aaIndex } from "./aa.ts"
@@ -19,8 +11,8 @@ import { Models, type ModelsSnapshot } from "./rpc.ts"
 import type { Row } from "./rows.ts"
 
 /**
- * The fetch-backed sources refresh on their own TTLs (6 h, 1 day, 6 h); the measured speeds cost about a
- * second here and are worth keeping for a minute, which is also what keeps a poll cheap.
+ * The fetch-backed sources keep their own TTLs; the measured speeds are worth a minute, which is also
+ * what keeps a poll cheap.
  */
 const CACHE_MS = 60_000
 
@@ -39,7 +31,7 @@ export default Plugin.define({
 	async setup(ctx) {
 		let cache: { at: number; value: ModelsSnapshot } | undefined
 
-		/** The four sources, each failure named instead of swallowed: a thin column must look thin. */
+		/** The four sources, each failure named instead of swallowed. */
 		async function sources(problems: string[]): Promise<Sources> {
 			const [plan, bench, estimates, speeds] = await Promise.allSettled([
 				goPlan(),
@@ -86,16 +78,14 @@ export default Plugin.define({
 				const cost = (model.cost ?? []).find((entry: any) => !entry.tier) ?? model.cost?.[0]
 				const limit = model.limit ?? {}
 				const score = benchFor(src.bench ?? new Map(), model.name, model.modelID)
-				// Three sources, one order: what this machine measured, then Artificial Analysis, then the
-				// catalogue. A measured figure is never displaced by an estimate.
+				// Three sources, one order: measured, then Artificial Analysis, then the catalogue.
 				const mine = src.measured?.get(model.providerID + "/" + model.modelID)
 				const theirs =
 					mine === undefined
 						? (src.estimated?.get(benchKey(model.name)) ?? src.estimated?.get(benchKey(model.modelID)) ?? score?.tps)
 						: undefined
 				if (mine !== undefined || theirs !== undefined) withSpeed += 1
-				// The measured ratio from the Go plan page first, the native rule second: one is data, the
-				// other is the fact no table reports (the catalogue lists a single DeepSeek price).
+				// The plan page's measured ratio first, the native rule second.
 				const planPeak = isGo(model.providerID) ? src.credits?.get(creditKey(model.modelID))?.peak : undefined
 				const peak = planPeak ?? nativePeak(model.providerID)
 				rows.push({

package/multipliers.ts

@@ -1,20 +1,5 @@
 // oc-models-pro: what the Go plan gives per model, read from the live Go docs page.
 //
-// The table is server-rendered — no JS, no auth — so a plain fetch and a row regex are enough. The
-// mirror that would have been cleaner, the `.mdx` on GitHub, is stale by design: it carries the old
-// limits (a model at $60 where the deployed page says $80), so the page is the source and the cache
-// carries the date it was read.
-//
-// Two different figures live on that page, and confusing them was the first bug here:
-//   - the **credit multiplier**: monthly limit ÷ plan price (DeepSeek V4.1 Flash: $80 on a $10 plan =
-//     8x). Every model has one, and it is what orders the list by effective price;
-//   - the **promotion note** (`4x · Limited time`): limit ÷ the struck-through base, i.e. this model's
-//     allowance is 4x its normal one. Only a few rows carry it.
-//
-// The page has no model ids, rows match by display name; a model with two tariffs appears twice
-// (`… (Off-Peak)` / `… (Peak)`), and the table repeats per plan (`go`, `go-plus`) — the first
-// occurrence wins, so the values are the standard plan's.
-//
 // Runnable standalone for its self-check: node plugins/oc-models-pro/multipliers.ts --self-check
 
 import { mkdir, readFile, rename, writeFile } from "node:fs/promises"
@@ -54,11 +39,8 @@ export interface Credits {
 	peak?: number
 }
 
-/** The native DeepSeek tariff doubles at peak, and the catalogue lists the **off-peak** figure: measured
- *  2026-10-28 against DeepSeek's own pricing page — `deepseek-flash` $0.15 off-peak / $0.30 peak, and the
- *  catalogue carries 0.15. Duplicated from `oc-budget/peak.ts` on purpose: the two packages are published
- *  apart and neither may import the other, so each carries the one fact it needs. That copy also owns the
- *  clock and the wallet; this one only knows which *list* price doubles, so the note stays static. */
+/** The native DeepSeek tariff doubles at peak, and the catalogue lists the **off-peak** figure, so the
+ *  note is static: this function only knows which *list* price doubles. */
 export function nativePeak(providerID: string): number | undefined {
 	return providerID === "deepseek" ? 2 : undefined
 }
@@ -123,7 +105,7 @@ export function parseGoTable(html: string): GoLimit[] {
 	return limits
 }
 
-/** The plan price, `$10/month` in the `Go` heading; the first such figure on the page is Go's. */
+/** The plan price: the first `$/month` figure on the page is Go's. */
 export function parsePlanPrice(html: string): number | undefined {
 	const found = text(html).match(/\$\s?([\d.]+)\s*\/\s*month/i)
 	return found ? Number(found[1]) : undefined

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@rizal72/oc-models-pro",
-	"version": "0.3.1",
+	"version": "0.3.2",
 	"type": "module",
 	"description": "OpenCode v2 model picker with prices, context, intelligence, tok/s, Go credit and the cost of a request",
 	"repository": {
@@ -10,6 +10,7 @@
 	"license": "MIT",
 	"files": [
 		"*.ts",
+		"!*.test.ts",
 		"tui.js",
 		"README.md",
 		"dist"

package/prefs.ts

@@ -1,11 +1,7 @@
 // oc-models-pro/prefs.ts — the picker's own state file, `~/.local/state/opencode/model.json`.
 //
-// Both halves need it: the terminal draws the list and toggles a favourite from the panel, the desktop
-// writes the same file through its server half. It lives here so the recipe exists once — a second copy
-// is a second way to lose `recent[]` or `variant{}`.
-//
-// The host watches this path and `model.json.*`. The write is `.model.json.tmp` then rename: the leading
-// dot keeps the temp name outside that glob, so the host never sees half a file.
+// Both halves write it, so the recipe exists once: a second copy is a second way to lose `recent[]`
+// or `variant{}`.
 import { readFileSync, renameSync, writeFileSync } from "node:fs"
 import { homedir } from "node:os"
 import { join } from "node:path"

package/rows.ts

@@ -1,10 +1,5 @@
 // oc-models-pro/rows.ts — the facts both surfaces read the same way.
 //
-// The terminal draws a monospace table and the desktop draws a grid, but what a row *means* is not the
-// renderer's business: the effective price, what counts as free, and the ten orders are one definition
-// here. Two copies would drift the way they already did once — the terminal kept free models out of the
-// price order and the desktop did not.
-//
 // Runnable standalone for its self-check: node plugins/oc-models-pro/rows.ts --self-check
 
 export interface Row {
@@ -12,14 +7,14 @@ export interface Row {
 	providerID: string
 	modelID: string
 	provider: string
-	/** Two letters from `tags.ts`: the provider's, unique in the catalogue. */
+	/** Two letters from `tags.ts`. */
 	tag: string
-	/** Undefined when the catalogue has no price for the model: an unknown figure, not a free one. */
+	/** Undefined when the catalogue has no price for the model. */
 	input?: number
 	output?: number
 	/** Undefined when the model carries no limit. */
 	context?: number
-	/** Monthly limit ÷ the $10 plan: the ordering multiplier, shown so the order explains itself. */
+	/** Monthly limit ÷ the plan price: the ordering multiplier. */
 	credit: number
 	/** Allowance ÷ its normal one, when the page marks a promotion. */
 	promo?: number
@@ -28,7 +23,7 @@ export interface Row {
 	/** Stencil's intelligence score and its speed; absent when the catalogue has no row for the model. */
 	int?: number
 	tps?: number
-	/** Tokens per second measured here, on this provider. Outranks `tps` and carries no tilde. */
+	/** Tokens per second measured here, on this provider. */
 	measured?: number
 	enabled: boolean
 }
@@ -40,12 +35,12 @@ export function effective(row: Row): number | undefined {
 	return row.input / (row.credit || 1)
 }
 
-/** A missing price is not a free one: both figures must be there, and be zero. */
+/** Both figures must be there, and be zero. */
 export function isFree(row: Row): boolean {
 	return row.input === 0 && row.output === 0
 }
 
-/** The speed to sort and print: what this machine measured first, the borrowed figure after. */
+/** The speed to sort and print. */
 export function speedOf(row: Row): number | undefined {
 	return row.measured ?? row.tps
 }
@@ -78,8 +73,7 @@ export interface Order {
 	value(row: Row, context: SortContext): number | undefined
 }
 
-/** Whatever an order cannot measure sorts last, never first, and in either direction: an unknown price is
- *  not the cheapest thing on the list, and a missing score is not the highest. */
+/** Whatever an order cannot measure sorts last, never first, and in either direction. */
 
 export const ORDERS: Record<SortKey, Order> = {
 	effective: { label: "effective price", short: "effective↑", value: effective },
@@ -92,7 +86,6 @@ export const ORDERS: Record<SortKey, Order> = {
 			row.input === undefined || row.output === undefined ? undefined : (row.input + row.output) / (row.credit || 1),
 	},
 	context: { label: "context window", short: "ctx↓", desc: true, value: (row) => row.context },
-	// A missing score sorts last, never first: Pi's rule, and the honest one.
 	int: { label: "intelligence", short: "int↓", desc: true, value: (row) => row.int },
 	speed: { label: "speed", short: "speed↓", desc: true, value: (row) => speedOf(row) },
 	recent: {
@@ -158,10 +151,7 @@ export interface Section {
 }
 
 /**
- * The order the surfaces draw, which is not the order they sort: free models never enter a price order,
- * so they follow their provider's paying rows; and the model in use is never buried — it leads its
- * provider, and its provider leads the list. Both surfaces held a copy of this (`gui.tsx` and
- * `tui.tsx`), and the copies drifted the moment a filter was added.
+ * The order the surfaces draw, which is not the order they sort.
  */
 export function arrange(list: Row[], options: { inUse?: string; grouped: boolean }): Section[] {
 	const used = options.inUse === undefined ? undefined : list.find((row) => id(row) === options.inUse)

package/rpc.ts

@@ -1,5 +1,4 @@
-// What the desktop's panel reads: a plugin window has no Node built-ins, so every file and every fetch
-// stays in the server half and the window asks for the finished table.
+// The two calls across the split, and the one row shape both surfaces sort.
 import { Rpc } from "@opencode/plugin/rpc"
 
 import type { Row } from "./rows.ts"
@@ -12,22 +11,22 @@ const Row = {
 		providerID: { type: "string" },
 		modelID: { type: "string" },
 		name: { type: "string" },
-		/** The provider's display name, account first as the host writes it. */
+		/** The provider's display name, as the host writes it. */
 		provider: { type: "string" },
 		/** Two letters from `tags.ts`. */
 		tag: { type: "string" },
-		/** Prices per million tokens; absent when the catalogue has none — an unknown figure, not a free one. */
+		/** Prices per million tokens; absent when the catalogue has none. */
 		input: Count,
 		output: Count,
 		context: Count,
-		/** Monthly limit ÷ the $10 plan. 1 when the plan is unknown. */
+		/** Monthly limit ÷ the plan price. */
 		credit: Count,
 		promo: Count,
 		peak: Count,
 		int: Count,
-		/** Borrowed from Artificial Analysis, or the catalogue's own when it still carries one. */
+		/** Borrowed: Artificial Analysis, or the catalogue's own. */
 		tps: Count,
-		/** Measured on this machine, on this provider. Outranks `tps` and carries no tilde. */
+		/** Measured on this machine, on this provider. */
 		measured: Count,
 		enabled: { type: "boolean" },
 	},
@@ -41,11 +40,10 @@ const Snapshot = {
 		rows: { type: "array", items: Row },
 		/** `providerID/modelID` of the models the host keeps as favourites, from `model.json`. */
 		favorites: { type: "array", items: { type: "string" } },
-		/** How many offers the catalogue holds, and how many came out with a speed — the pair that tells a
-		 *  thin source from a broken key. */
+		/** How many offers the catalogue holds, and how many came out with a speed. */
 		offers: Count,
 		withSpeed: Count,
-		/** False when the Go plan could not be read and no cache was there: every credit is 1. */
+		/** False when the Go plan could not be read and no cache was there. */
 		credits: { type: "boolean" },
 		/** What failed, in the words the panel shows. Empty when everything came back. */
 		problems: { type: "array", items: { type: "string" } },

package/tags.ts

@@ -1,10 +1,5 @@
 // Two letters per provider, so a row says where its model comes from without spending a column.
 //
-// The host's own two providers keep a tag chosen by hand, and those two are reserved: they exist in
-// every installation, so nobody else may take their letters. Everything else is derived from the
-// provider's display name — the only place where a merged name still keeps its seams ("DeepSeek" is
-// Deep·Seek; the id `deepseek` no longer says so).
-//
 // Runnable standalone for its self-check: node plugins/oc-models-pro/tags.ts --self-check
 
 /** Tags that belong to the host's own providers, present or not. */
@@ -14,8 +9,7 @@ const RESERVED: Record<string, string> = {
 }
 
 /**
- * The words of a display name: `OpenCode Go` is two, `OpenRouter` is two as well — but a camelCase
- * split only when the name has no separator, so a brand like `GitHub Copilot` keeps `GitHub` whole.
+ * The words of a display name.
  */
 export function words(name: string): string[] {
 	const split = name.split(/[^A-Za-z0-9]+/).filter(Boolean)
@@ -32,8 +26,7 @@ function preferred(name: string): string {
 }
 
 /**
- * A tag per provider id, unique and order-independent. Derived in alphabetical order of id, so the
- * same catalogue always yields the same tags whatever order the providers arrive in.
+ * A tag per provider id, unique and order-independent.
  */
 export function tagsFor(providers: Array<{ id: string; name: string }>): Map<string, string> {
 	const taken = new Set(Object.values(RESERVED))
@@ -51,10 +44,8 @@ export function tagsFor(providers: Array<{ id: string; name: string }>): Map<str
 		const first = letters[0] ?? "?"
 		let tag = preferred(provider.name)
 		if (taken.has(tag)) {
-			// Walk the name for a free pair, so `Google` steps aside from the reserved GO.
 			tag = ""
 			for (const letter of letters.slice(1)) {
-				// A doubled letter reads as a typo ("GG"): prefer anything else.
 				if (letter === first) continue
 				const candidate = first + letter
 				if (!taken.has(candidate)) {

package/tps.ts

@@ -1,15 +1,5 @@
 // oc-models-pro/tps.ts — tokens per second, measured on this machine.
 //
-// Not an estimate from a catalogue: it is what this provider actually did, for the models this
-// person actually ran. The number is the one the industry publishes, so it can be read next to
-// anything online: (output + reasoning tokens) over the window from the first streamed token to the
-// last, time-to-first-token excluded — Artificial Analysis' "output speed", and `completion_tokens`
-// as OpenAI, Anthropic and OpenRouter bill it, where reasoning is part of the output.
-//
-// The window is the last seven days when it holds at least three turns, and all of history when it
-// does not: a number from a provider's good week is worth less than the provider's behaviour now,
-// but an empty cell for a model used once last month is worth less still.
-//
 // Runnable standalone for its self-check: node plugins/oc-models-pro/tps.ts --self-check
 
 import { execFile } from "node:child_process"
@@ -45,7 +35,7 @@ export function databasePath(): string {
 	return process.env.OPENCODE_DB || join(data, "opencode", "opencode.db")
 }
 
-/** The host's own SQLite. Absent outside Bun, where the CLI carries the self-check. */
+/** The host's own SQLite; absent outside Bun. */
 async function sqlite(): Promise<typeof import("bun:sqlite") | undefined> {
 	try {
 		return await import("bun:sqlite")
@@ -141,8 +131,7 @@ export async function speedIndex(database = databasePath()): Promise<{
 		const recent = all.filter((t) => t.at >= since)
 		const use = recent.length >= MIN_SAMPLES ? recent : all
 		if (use.length === 0) continue
-		// One voice per session, then the median of those voices: without it a single long-running
-		// session elects the number on its own, and this person's sessions run for days.
+		// One voice per session, then the median of those voices.
 		const perSession = new Map<string, number[]>()
 		for (const t of use) {
 			const list = perSession.get(t.sid) ?? []

package/tui.js

@@ -8,14 +8,6 @@ import { insertNode as _$insertNode } from "@opentui/solid";
 import { setProp as _$setProp } from "@opentui/solid";
 import { createElement as _$createElement } from "@opentui/solid";
 // oc-models-pro — the model list with prices, ported from Pi's `pi-model-pro`.
-//
-// We render it ourselves, in a `session.panel`: the native dialog caps an option line at 60 columns,
-// measured, which cannot hold prices, context, intelligence and the Go credit side by side. The host
-// owns the panel's placement and focus; the list owns its columns.
-//
-// Prices are native, never adjusted: the multiplier gets its own column and the effective price only
-// decides the order, so a promoted model sorts first while the figures stay honest. The model in use
-// is the first row whatever it costs: the free/paying split never buries it.
 import { Plugin } from "@opencode/plugin/tui";
 import { For, Show, createEffect, createMemo, createSignal, onCleanup } from "solid-js";
 import { appendFileSync, readFileSync, watch, writeFileSync } from "node:fs";
@@ -47,8 +39,7 @@ function tokens(value) {
 }
 
 /**
- * Every cell is exactly `width` characters: text that does not fit is cut, never shifted right. The
- * columns are hand-padded monospace, so one `padStart` short of its width slides the whole row.
+ * Every cell is exactly `width` characters: one `padStart` short of its width slides the whole row.
  */
 function cell(text, width, align = "end") {
   const cut = text.length > width ? text.slice(0, width) : text;
@@ -59,17 +50,11 @@ function cell(text, width, align = "end") {
 function padTag(letters) {
   return (letters.padEnd(2) + "  ").slice(0, 4);
 }
-
-/**
- * Column widths live in `fit.ts`: they are no longer fixed. The table is 85 characters at its widest
- * and the name gives way first (down to 24), then whole columns go. `panel.width` reads as pixels
- * on the first render, so `fit` clamps anything implausible back to the wide layout.
- */
 function label(name, width) {
   return name.length > width ? name.slice(0, width - 1) + "…" : name.padEnd(width);
 }
 
-/** A price cell; blank when the catalogue has no figure, "free" when the figure is zero. */
+/** A price cell: blank for no figure, "free" for zero. */
 function price(value, width) {
   if (value === undefined) return cell("", width);
   const shown = value === 0 ? "free" : String(Math.round(value * 1000) / 1000);
@@ -93,9 +78,7 @@ function speedCell(row, f) {
 function rowParts(row, mine, favourite, f) {
   const notes = [row.peak ? "peak*" + row.peak : "", row.promo ? "+" + row.promo + "x" : ""].filter(Boolean);
   const head = [(mine ? "●" : " ") + (favourite ? "*" : " "), padTag(row.tag), label(row.name, f.name), price(row.input, 6), price(row.output, 6), ...(f.ctx ? [cell(tokens(row.context), 5)] : []), ...(f.int ? [cell(row.int !== undefined ? row.int.toFixed(1) : "", 5)] : [])];
-  const tail = [
-  // Any credit other than 1 changes the effective price, so any of them is shown.
-  cell(row.credit !== 1 ? "×" + row.credit : "", 4), ...(f.note ? [cell(notes.join(" "), 6)] : [])];
+  const tail = [cell(row.credit !== 1 ? "×" + row.credit : "", 4), ...(f.note ? [cell(notes.join(" "), 6)] : [])];
   const speed = speedCell(row, f);
   if (!speed) return {
     before: head.concat(tail).join(" "),
@@ -145,8 +128,7 @@ export default Plugin.define({
     const BASE = themeColour(["text", "base"]);
     const DIM = themeColour(["text", "muted"], ["text", "faint"]) ?? FALLBACK_DIM;
     const ACCENT = themeColour(["syntax", "function"], ["syntax", "variable"]) ?? FALLBACK_ACCENT;
-    // Theme colours reach us as RGBA objects; `fg` takes them, a box background does not — it wants
-    // a string, so the two surfaces are converted to hex.
+    // Theme colours reach us as RGBA objects, which a box background cannot take: convert to hex.
     const hex = node => {
       if (typeof node === "string") return node;
       const buffer = node?.buffer;
@@ -155,7 +137,7 @@ export default Plugin.define({
       return "#" + channels.join("");
     };
     const SURFACE = hex(themeColour(["surface", "base"], ["background", "raised", "base"]));
-    // The cyan the native controls use when focused; the row text flips to the page colour on top.
+    // The cyan the native controls use when focused.
     const SELECTED = hex(themeColour(["background", "action", "primary", "focused"], ["surface", "high"]));
     const ON_SELECTED = hex(themeColour(["background", "base"], ["text", "base"]));
     const BORDER = hex(themeColour(["border", "base"], ["text", "faint"]));
@@ -163,14 +145,12 @@ export default Plugin.define({
     prefs = readPrefs();
     let creditIndex = new Map();
     let bench = new Map();
-    /** Measured on this machine, keyed `providerID/modelID`: it wins over any catalogue figure. */
+    /** Measured on this machine, keyed `providerID/modelID`. */
     let measured = new Map();
-    /** From Artificial Analysis, for the models this machine never ran. */
+    /** From Artificial Analysis. */
     let estimated = new Map();
-    /** False when the Go page could not be read and no cache was there: the credits are all 1. */
+    /** False when the Go page could not be read and no cache was there. */
     let creditsKnown = false;
-    // Signals, not plain variables: the header shows them, and a plain variable left the title
-    // saying "effective price" whatever the list was actually sorted by.
     const [sort, setSort] = createSignal("effective");
     const [grouped, setGrouped] = createSignal(true);
     const [scope, setScope] = createSignal("all");
@@ -209,8 +189,6 @@ export default Plugin.define({
         const cost = (model.cost ?? []).find(entry => !entry.tier) ?? model.cost?.[0];
         const limit = model.limit ?? {};
         const score = benchFor(bench, model.name, model.modelID);
-        // Three sources, one order: what this machine measured, then Artificial Analysis, then
-        // the catalogue. A measured figure is never displaced by an estimate.
         const mine = measured.get(model.providerID + "/" + model.modelID);
         const theirs = mine === undefined ? estimated.get(benchKey(model.name)) ?? estimated.get(benchKey(model.modelID)) ?? score?.tps : undefined;
         result.push({
@@ -221,18 +199,11 @@ export default Plugin.define({
           // header reads as if the account were the thing: provider first, account in brackets.
           provider: (providers.get(model.providerID) ?? model.providerID).replace(/^(.+?) \/ (.+)$/, "$2 ($1)"),
           tag: tags.get(model.providerID) ?? model.providerID.slice(0, 2).toUpperCase(),
-          // A missing figure stays missing: a 0 here would put an unknown model at the top of the
-          // cheapest list and claim a context window it never declared.
           input: typeof cost?.input === "number" ? cost.input : undefined,
           output: typeof cost?.output === "number" ? cost.output : undefined,
           context: typeof limit.context === "number" ? limit.context : undefined,
-          // The page describes the Go plan only, and its names are not the catalogue's: match the
-          // model id, and only on the provider the plan applies to. By name alone, OpenRouter's
-          // "DeepSeek V4.1 Flash" inherited the Go credit.
           credit: isGo(model.providerID) ? creditIndex.get(key(model.modelID))?.credit ?? 1 : 1,
           promo: isGo(model.providerID) ? creditIndex.get(key(model.modelID))?.promo : undefined,
-          // Measured on the Go plan first, then the native rule: the catalogue lists one
-          // DeepSeek price, so only the second can say it doubles.
           peak: (isGo(model.providerID) ? creditIndex.get(key(model.modelID))?.peak : undefined) ?? nativePeak(model.providerID),
           int: score?.int,
           tps: theirs,
@@ -240,8 +211,7 @@ export default Plugin.define({
           enabled: model.enabled !== false
         });
       }
-      // Every provider keeps its row: the same model on `deepseek` and on `opencode-go` is two
-      // different offers, and hiding one hid `deepseek-v4-pro` from the provider that hosts it.
+      // Every provider keeps its row.
       return result;
     }
     async function warm() {
@@ -260,15 +230,12 @@ export default Plugin.define({
       // One fetch a day at most; a failure only costs the credit column and the effective ordering.
       const plan = await goPlan();
       creditIndex = plan ? credits(plan) : new Map();
-      // The three reads run together: the panel waits for the slowest, not for their sum. Measured
-      // comes first in the ordering below — the only figure that is this provider's own behaviour.
+      // The three reads run together: the panel waits for the slowest, not for their sum.
       const started = Date.now();
       const [benchmarks, speeds, estimates] = await Promise.all([benchIndex(), speedIndex(), aaIndex()]);
       bench = benchmarks;
       measured = new Map([...speeds.speeds].map(([key, speed]) => [key, speed.tps]));
       estimated = estimates;
-      // How many of the models on offer actually come out with a figure: the one number that says
-      // whether a thin column is a thin source or a broken key.
       const offer = listModels();
       const withSpeed = offer.filter(model => {
         if (measured.has(model.providerID + "/" + model.modelID)) return true;
@@ -287,8 +254,7 @@ export default Plugin.define({
       trace("open: panel requested");
     }
     const Panel = props => {
-      // Who had the focus before the panel took it: the prompt itself. The renderer can hand it back,
-      // and nothing else in the API can — no host command focuses the prompt from a plugin.
+      // Who had the focus before the panel took it: the prompt itself.
       const previousFocus = context.renderer?.currentFocusedRenderable;
       trace("panel: previous focus=" + (previousFocus ? previousFocus.constructor?.name ?? "?" : "none"));
       const [selected, setSelected] = createSignal(undefined);
@@ -296,7 +262,6 @@ export default Plugin.define({
       const width = createMemo(() => Math.floor(Number(props.panel.width) || 0), undefined, {
         equals: Object.is
       });
-      // The scrollbox draws its scrollbar in the last column, so the table gets one less.
       // Both layouts carry an equality check: the width settles from pixels to columns on open,
       // and a memo that fires on that tick re-renders the panel for no visual change — which is
       // what took the focus off the search field.
@@ -331,9 +296,8 @@ export default Plugin.define({
        */
       const holdsField = () => Boolean(field) && context.renderer?.currentFocusedRenderable === field;
       /**
-       * Hands the focus back to whoever held it before the panel opened — the prompt. The host does not
-       * do it for a panel closed by a click, and there is no command that focuses the prompt: this is
-       * the renderer's own method, and it is what makes a click on a row behave like `esc`.
+       * Hands the focus back to whoever held it before the panel opened, which is what makes a click on
+       * a row behave like `esc`: the renderer's own method, since no command focuses the prompt.
        */
       const focusPrompt = () => {
         try {
@@ -352,18 +316,15 @@ export default Plugin.define({
         if (field && !field.isDestroyed) field.value = "";
       };
 
-      // The host activates a panel's keyboard layers only while it owns input, and it disposes them
-      // when the panel unmounts: measured three openings in a row, every one reading 10 reachable
-      // `models-pro.*` commands and never 20 — a leaked layer would show up here first.
+      // The trace line is the alarm for a leaked layer: it prints on every mount.
       try {
         trace("panel: " + context.keymap.commands().filter(entry => String(entry.id ?? "").startsWith("models-pro.")).length + " models-pro commands reachable");
       } catch (error) {
         fail("panel commands", error);
       }
 
-      // The state file is shared: the native picker and `model.cycle_favorite` write it while this
-      // panel is open. Watching the directory (not the file) survives the rename, and every event
-      // re-reads the file so the star and the `favorites` scope stay true.
+      // The state file is shared and written under us: watch the directory, which survives the
+      // rename, and re-read on every event.
       try {
         const watcher = watch(join(homedir(), ".local/state/opencode"), (_event, name) => {
           if (String(name) !== "model.json") return;
@@ -397,10 +358,6 @@ export default Plugin.define({
           });
         });
 
-        // Free models never enter the price order: they are not cheap, they are unmetered, and at
-        // the top they hide everything that costs something. Grouped, each one still sits under
-        // its own provider, after that provider's paying rows — the group answers "who serves
-        // this", and a global "Free" bucket would answer it wrong. Flat, they keep one bucket.
         // What is drawn, and in which order, is `arrange`'s business — the same function the
         // desktop panel calls. This half only turns sections into the headers and rows a
         // terminal list wants.
@@ -439,8 +396,6 @@ export default Plugin.define({
         setSelected(key);
         reveal(key);
       }
-
-      /** The list is taller than the panel: the selection has to drag the viewport with it. */
       let scroller;
       function reveal(key) {
         const box = scroller;
@@ -478,15 +433,7 @@ export default Plugin.define({
         })();
       }
 
-      /**
-       * The host owns the model selection: the native picker writes it with the host's own
-       * `model.set`, per session and agent, and the prompt sends it to the server. The plugin API
-       * has no setter for it — `ui.model` reads it and changes its variant, nothing more — so the
-       * host makes the change: `model.cycle_favorite` ends on the first favourite, the same
-       * internal call the picker makes. The list is prepared for that call and put back right
-       * after. One entry is what makes the call safe to repeat: whatever is selected, it lands on
-       * that entry.
-       */
+      /** Switches the model through the host's own command, with the favourite list as transport. */
       async function selectModel(row) {
         // The upstream fix, when it lands: the host's setter, and no list at all.
         const native = ui.model.set;
@@ -526,11 +473,7 @@ export default Plugin.define({
         return landed;
       }
 
-      /**
-       * A model whose variant was never chosen has none applied, and the native picker asks once for
-       * it. Same rule here: a saved entry — `default` included — means silence; no entry opens the
-       * question, and the host's own `variant.set` is what persists the answer.
-       */
+      /** The native picker asks once for a variant a model has none of: same rule here. */
       async function settleVariant(row) {
         // The file is the source, not the copy read at open: a variant saved meanwhile (native
         // picker, prompt) has to count as an answer.
@@ -559,8 +502,6 @@ export default Plugin.define({
         });
         // A dialog takes the focus and does not hand it back either: the same command, once it closed.
         setTimeout(focusPrompt, 40);
-        // Cancelling leaves the model without a variant, and the question will come back, as in
-        // the native picker.
         if (chosen === undefined) return;
         const applied = ui.model.variant.set(chosen === "default" ? undefined : chosen);
         trace("variant set: " + chosen + " -> " + applied);
@@ -651,10 +592,7 @@ export default Plugin.define({
           title: "Clear the search, or close the list",
           bind: "esc",
           run: () => query() ? clear() : props.panel.close()
-        },
-        // `ctrl+y`: `f11` never reached the panel in the terminal. `ctrl+f` marks a favourite,
-        // as in the native dialog. `f` alone stays unusable: it would be typed into the search.
-        {
+        }, {
           id: "models-pro.fullscreen",
           title: "Full screen",
           bind: "ctrl+y",
@@ -796,7 +734,6 @@ export default Plugin.define({
             const key = prefKey(row.providerID, row.modelID);
             const mine = () => key === mineKey();
             const active = () => key === selectedKey();
-            // A favourite is marked everywhere but in the favourites scope, where every row is one.
             const favourite = () => {
               revision();
               return scope() !== "favorites" && prefs.favorite.has(key);
@@ -915,8 +852,7 @@ export default Plugin.define({
         }
       }
     };
-    // The types publish the keymap twice — `context.keymap` and `ui.keymap` — and this build answers
-    // for one of them only. Try both, record which one took it, never fail silently.
+    // The command goes on whichever keymap the build publishes; the trace records which took it.
     const keymaps = [["context.keymap", context.keymap], ["ui.keymap", context.ui?.keymap]];
     let registered = false;
     for (const [label, keymap] of keymaps) {