@krtclcdy/tersio 2.22.1 → 2.24.0

package/extensions/shared/plugin-settings.js

@@ -1,22 +1,31 @@
-// Plugin settings reader for the Tersio extensions.
-// Reads omp's persisted plugin state (~/.omp/plugins/omp-plugins.lock.json),
-// written by `omp plugin config set @krtclcdy/tersio <key> <value>` and by
-// the installer profile step. Tolerant: any parse failure yields {} so every
-// caller falls back to its own default.
-import { existsSync, readFileSync } from 'node:fs';
+// Settings reader for the Tersio extensions.
+//
+// One file, under Tersio's own home (~/.tersio/settings.json), serves every
+// host: OMP has no reason to keep its own copy and pi never had a store. The
+// profile CLI writes the same file, so the extensions, `tersio settings`, and
+// both hosts always agree.
+//
+// Tolerant: any parse failure yields {} so every caller falls back to its own
+// default.
+import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
+import { tersioHome } from '../lib/utils.js';
 export const PLUGIN_NAME = '@krtclcdy/tersio';
-function lockPaths() {
+// Session-start defaults, shared by every host: one file under Tersio's own
+// home, so OMP and pi read the same values. This module owns the path.
+export function tersioSettingsFile() {
+    return path.join(tersioHome(), 'settings.json');
+}
+// Older installs stored the same values as OMP plugin settings, and an install
+// that predates ~/.tersio still has them there. Read once and keep them, so
+// switching the store does not reset a user's defaults.
+function legacyOmpSettings() {
     const configHome = process.env.XDG_CONFIG_HOME || path.join(os.homedir(), '.config');
-    return [
+    for (const p of [
         path.join(os.homedir(), '.omp', 'plugins', 'omp-plugins.lock.json'),
         path.join(configHome, 'omp', 'plugins', 'omp-plugins.lock.json'),
-    ];
-}
-// `settings[PLUGIN_NAME]` in the lock file; empty object when missing.
-export function readPluginSettings() {
-    for (const p of lockPaths()) {
+    ]) {
         if (!existsSync(p))
             continue;
         try {
@@ -27,11 +36,27 @@ export function readPluginSettings() {
         }
         catch { /* tolerate corrupt file */ }
     }
-    return {};
+    return null;
+}
+function readSettingsFile() {
+    const p = tersioSettingsFile();
+    if (!existsSync(p))
+        return null;
+    try {
+        const parsed = JSON.parse(readFileSync(p, 'utf8'));
+        return parsed && typeof parsed === 'object' && !Array.isArray(parsed) ? parsed : null;
+    }
+    catch {
+        return null;
+    }
+}
+// The stored settings; empty when the file is missing, corrupt, or empty.
+export function readPluginSettings() {
+    return readSettingsFile() ?? legacyOmpSettings() ?? {};
 }
 const COMBO_LEVELS = new Set(['off', 'medium', 'balanced', 'max']);
-const CAVEMAN_MODES = new Set(['off', 'lite', 'full', 'ultra', 'wenyan']);
-const PONYTAIL_MODES = new Set(['off', 'lite', 'full', 'ultra', 'review']);
+const CAVEMAN_MODES = new Set(['off', 'lite', 'full', 'ultra', 'wenyan', 'wenyan-lite', 'wenyan-full', 'wenyan-ultra']);
+const PONYTAIL_MODES = new Set(['off', 'lite', 'full', 'ultra']);
 function readStringDefault(key, valid) {
     const raw = readPluginSettings()[key];
     return typeof raw === 'string' && valid.has(raw) ? raw : 'off';
@@ -49,3 +74,25 @@ export function readRtkDefault() {
 export function readPonytailDefault() {
     return readStringDefault('ponytailDefault', PONYTAIL_MODES);
 }
+export function isComboSetupComplete() {
+    return readPluginSettings().comboSetupComplete === true;
+}
+const COMBO_MODE_DEFAULTS = {
+    off: { cavemanDefault: 'off', rtkDefault: false, ponytailDefault: 'off' },
+    medium: { cavemanDefault: 'lite', rtkDefault: true, ponytailDefault: 'lite' },
+    balanced: { cavemanDefault: 'full', rtkDefault: true, ponytailDefault: 'full' },
+    max: { cavemanDefault: 'ultra', rtkDefault: true, ponytailDefault: 'ultra' },
+};
+export function saveComboSetup(level) {
+    if (!(level in COMBO_MODE_DEFAULTS))
+        return false;
+    const file = tersioSettingsFile();
+    mkdirSync(path.dirname(file), { recursive: true });
+    writeFileSync(file, `${JSON.stringify({
+        ...readPluginSettings(),
+        comboDefault: level,
+        ...COMBO_MODE_DEFAULTS[level],
+        comboSetupComplete: true,
+    }, null, 2)}\n`, 'utf8');
+    return true;
+}

package/extensions/shared/plugin-settings.ts

@@ -1,25 +1,35 @@
-// Plugin settings reader for the Tersio extensions.
-// Reads omp's persisted plugin state (~/.omp/plugins/omp-plugins.lock.json),
-// written by `omp plugin config set @krtclcdy/tersio <key> <value>` and by
-// the installer profile step. Tolerant: any parse failure yields {} so every
-// caller falls back to its own default.
+// Settings reader for the Tersio extensions.
+//
+// One file, under Tersio's own home (~/.tersio/settings.json), serves every
+// host: OMP has no reason to keep its own copy and pi never had a store. The
+// profile CLI writes the same file, so the extensions, `tersio settings`, and
+// both hosts always agree.
+//
+// Tolerant: any parse failure yields {} so every caller falls back to its own
+// default.
 
-import { existsSync, readFileSync } from 'node:fs';
+import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
+import { tersioHome } from '../lib/utils.ts';
+
 export const PLUGIN_NAME = '@krtclcdy/tersio';
 
-function lockPaths(): string[] {
+// Session-start defaults, shared by every host: one file under Tersio's own
+// home, so OMP and pi read the same values. This module owns the path.
+export function tersioSettingsFile(): string {
+  return path.join(tersioHome(), 'settings.json');
+}
+
+// Older installs stored the same values as OMP plugin settings, and an install
+// that predates ~/.tersio still has them there. Read once and keep them, so
+// switching the store does not reset a user's defaults.
+function legacyOmpSettings(): Record<string, unknown> | null {
   const configHome = process.env.XDG_CONFIG_HOME || path.join(os.homedir(), '.config');
-  return [
+  for (const p of [
     path.join(os.homedir(), '.omp', 'plugins', 'omp-plugins.lock.json'),
     path.join(configHome, 'omp', 'plugins', 'omp-plugins.lock.json'),
-  ];
-}
-
-// `settings[PLUGIN_NAME]` in the lock file; empty object when missing.
-export function readPluginSettings(): Record<string, unknown> {
-  for (const p of lockPaths()) {
+  ]) {
     if (!existsSync(p)) continue;
     try {
       const config = JSON.parse(readFileSync(p, 'utf8')) as { settings?: Record<string, Record<string, unknown>> };
@@ -27,12 +37,28 @@ export function readPluginSettings(): Record<string, unknown> {
       if (values && typeof values === 'object' && !Array.isArray(values)) return values;
     } catch { /* tolerate corrupt file */ }
   }
-  return {};
+  return null;
+}
+
+function readSettingsFile(): Record<string, unknown> | null {
+  const p = tersioSettingsFile();
+  if (!existsSync(p)) return null;
+  try {
+    const parsed = JSON.parse(readFileSync(p, 'utf8')) as Record<string, unknown>;
+    return parsed && typeof parsed === 'object' && !Array.isArray(parsed) ? parsed : null;
+  } catch {
+    return null;
+  }
+}
+
+// The stored settings; empty when the file is missing, corrupt, or empty.
+export function readPluginSettings(): Record<string, unknown> {
+  return readSettingsFile() ?? legacyOmpSettings() ?? {};
 }
 
 const COMBO_LEVELS = new Set(['off', 'medium', 'balanced', 'max']);
-const CAVEMAN_MODES = new Set(['off', 'lite', 'full', 'ultra', 'wenyan']);
-const PONYTAIL_MODES = new Set(['off', 'lite', 'full', 'ultra', 'review']);
+const CAVEMAN_MODES = new Set(['off', 'lite', 'full', 'ultra', 'wenyan', 'wenyan-lite', 'wenyan-full', 'wenyan-ultra']);
+const PONYTAIL_MODES = new Set(['off', 'lite', 'full', 'ultra']);
 
 function readStringDefault(key: string, valid: Set<string>): string {
   const raw = readPluginSettings()[key];
@@ -55,3 +81,27 @@ export function readRtkDefault(): boolean {
 export function readPonytailDefault(): string {
   return readStringDefault('ponytailDefault', PONYTAIL_MODES);
 }
+
+export function isComboSetupComplete(): boolean {
+  return readPluginSettings().comboSetupComplete === true;
+}
+
+const COMBO_MODE_DEFAULTS = {
+  off: { cavemanDefault: 'off', rtkDefault: false, ponytailDefault: 'off' },
+  medium: { cavemanDefault: 'lite', rtkDefault: true, ponytailDefault: 'lite' },
+  balanced: { cavemanDefault: 'full', rtkDefault: true, ponytailDefault: 'full' },
+  max: { cavemanDefault: 'ultra', rtkDefault: true, ponytailDefault: 'ultra' },
+} as const;
+
+export function saveComboSetup(level: string): boolean {
+  if (!(level in COMBO_MODE_DEFAULTS)) return false;
+  const file = tersioSettingsFile();
+  mkdirSync(path.dirname(file), { recursive: true });
+  writeFileSync(file, `${JSON.stringify({
+    ...readPluginSettings(),
+    comboDefault: level,
+    ...COMBO_MODE_DEFAULTS[level as keyof typeof COMBO_MODE_DEFAULTS],
+    comboSetupComplete: true,
+  }, null, 2)}\n`, 'utf8');
+  return true;
+}

package/extensions/shared/pricing.js

@@ -1,17 +1,14 @@
 // extensions/shared/pricing.ts — dynamic model pricing from LiteLLM.
-// Lookup order: exact id in the live cache → provider-prefix strip → default.
-// The cache is the full LiteLLM feed (3k+ ids), refreshed lazily in background
-// and on `tersio update`; every reader is sync and offline-safe. No static
-// per-model table — list prices rot, the feed does not. Unknown models report
-// the default with known:false so the dashboard labels them honestly.
+// Exact id → provider-prefix strip → default, which reports with known:false so
+// the Dashboard labels an unpriced model honestly. No static table: prices rot.
 import fs from 'node:fs';
 import path from 'node:path';
 import { tersioDataPath } from '../lib/utils.js';
 // Sonnet-class default for unpriced models; always paired with known:false.
 export const DEFAULT_PRICE = { input: 2, output: 10, cacheRead: 0.2, cacheWrite: 2.5 };
 const LITELLM_URL = 'https://raw.githubusercontent.com/BerriAI/litellm/main/model_prices_and_context_window.json';
-// Live cache refreshes lazily: readers use the file even when stale, and only
-// one refresh runs per process. `tersio update` still forces a fresh fetch.
+// Readers use the file even when stale, and only one refresh runs per process;
+// `tersio update` still forces a fresh fetch.
 const CACHE_TTL_MS = 7 * 24 * 3600 * 1000;
 let refreshInflight = null;
 export function pricesCachePath() {
@@ -26,8 +23,8 @@ export function pricesUrl() {
 function toPrice(t) {
     return { input: t[0], output: t[1], cacheRead: t[2], cacheWrite: t[3] };
 }
-// Tolerate both cache shapes: compact tuples (new) and ModelPrice objects
-// (written by older refreshes) — a version bump never orphans the cache.
+// Tolerate both cache shapes (compact tuples and ModelPrice objects) so a
+// version bump never orphans the cache.
 function asPrice(v) {
     if (Array.isArray(v) && v.length === 4 && v.every((n) => typeof n === 'number' && Number.isFinite(n) && n >= 0)) {
         return toPrice(v);
@@ -35,6 +32,8 @@ function asPrice(v) {
     if (v && typeof v === 'object') {
         const o = v;
         if (['input', 'output', 'cacheRead', 'cacheWrite'].every((k) => typeof o[k] === 'number' && Number.isFinite(o[k]) && o[k] >= 0)) {
+            // SAFETY: the line above verified all four keys are finite numbers >= 0,
+            // which is the whole of ModelPrice.
             return o;
         }
     }
@@ -71,15 +70,28 @@ export function loadLivePrices() {
 function num(v) {
     return typeof v === 'number' && Number.isFinite(v) && v >= 0 ? v : null;
 }
+// One model, two spellings: transcripts say `space-bunny`, the feed keys it
+// `openrouter/stealth/space-bunny-alpha` at zero. Aliases fold to the first, so
+// a free stealth model never reports the default price.
+export const MODEL_ALIASES = [
+    { id: 'space-bunny', feed: 'stealth/space-bunny-alpha' },
+];
+export function canonicalPriceId(model) {
+    const key = model.toLowerCase();
+    return MODEL_ALIASES.find((a) => a.id === key || a.feed === key)?.id ?? key;
+}
 export function priceFor(model, live) {
     const table = live === undefined ? loadLivePrices() : live;
+    // An alias goes through the same exact-then-suffix chain as any other id, so
+    // it still resolves when the feed keys it under a provider prefix.
+    const id = MODEL_ALIASES.find((a) => a.id === model.toLowerCase())?.feed ?? model;
     if (table) {
-        const hit = asPrice(table.exact[model] ?? table.exact[model.toLowerCase()]);
+        const hit = asPrice(table.exact[id] ?? table.exact[id.toLowerCase()]);
         if (hit)
             return { price: hit, known: true, live: true };
         // Provider-prefixed ids ("azure/gpt-4o", "dashscope/qwen-max"): match the
         // first cached id that ends with the bare model name.
-        const name = model.toLowerCase();
+        const name = id.toLowerCase();
         const key = Object.keys(table.exact).find((k) => k.toLowerCase() === name || k.toLowerCase().endsWith(`/${name}`));
         const prefixed = key ? asPrice(table.exact[key]) : null;
         if (prefixed)
@@ -87,8 +99,7 @@ export function priceFor(model, live) {
     }
     return { price: DEFAULT_PRICE, known: false, live: false };
 }
-// Fetch LiteLLM pricing, keep every id with valid input/output rates, write
-// the cache. False on any failure (offline, timeout, shape change) — callers
+// Fetch LiteLLM pricing and write the cache. False on any failure — callers
 // fall back silently.
 export async function refreshPrices() {
     if (refreshInflight)
@@ -153,8 +164,8 @@ export async function refreshPrices() {
         refreshInflight = null;
     }
 }
-// Fire-and-forget refresh when the cache is stale or missing. Readers keep
-// using the stale file (or the built-in table) — pricing never blocks.
+// Fire-and-forget refresh when the cache is stale; readers keep the stale
+// file, so pricing never blocks.
 export function refreshPricesIfStale() {
     const live = loadLivePrices();
     if (live && Date.now() - live.fetchedAt <= CACHE_TTL_MS)

package/extensions/shared/pricing.ts

@@ -1,11 +1,7 @@
 // extensions/shared/pricing.ts — dynamic model pricing from LiteLLM.
-// Lookup order: exact id in the live cache → provider-prefix strip → default.
-// The cache is the full LiteLLM feed (3k+ ids), refreshed lazily in background
-// and on `tersio update`; every reader is sync and offline-safe. No static
-// per-model table — list prices rot, the feed does not. Unknown models report
-// the default with known:false so the dashboard labels them honestly.
+// Exact id → provider-prefix strip → default, which reports with known:false so
+// the Dashboard labels an unpriced model honestly. No static table: prices rot.
 import fs from 'node:fs';
-import os from 'node:os';
 import path from 'node:path';
 import { tersioDataPath } from '../lib/utils.ts';
 
@@ -20,8 +16,8 @@ export interface ModelPrice {
 export const DEFAULT_PRICE: ModelPrice = { input: 2, output: 10, cacheRead: 0.2, cacheWrite: 2.5 };
 
 const LITELLM_URL = 'https://raw.githubusercontent.com/BerriAI/litellm/main/model_prices_and_context_window.json';
-// Live cache refreshes lazily: readers use the file even when stale, and only
-// one refresh runs per process. `tersio update` still forces a fresh fetch.
+// Readers use the file even when stale, and only one refresh runs per process;
+// `tersio update` still forces a fresh fetch.
 const CACHE_TTL_MS = 7 * 24 * 3600 * 1000;
 let refreshInflight: Promise<boolean> | null = null;
 
@@ -47,8 +43,8 @@ function toPrice(t: [number, number, number, number]): ModelPrice {
   return { input: t[0], output: t[1], cacheRead: t[2], cacheWrite: t[3] };
 }
 
-// Tolerate both cache shapes: compact tuples (new) and ModelPrice objects
-// (written by older refreshes) — a version bump never orphans the cache.
+// Tolerate both cache shapes (compact tuples and ModelPrice objects) so a
+// version bump never orphans the cache.
 function asPrice(v: unknown): ModelPrice | null {
   if (Array.isArray(v) && v.length === 4 && v.every((n) => typeof n === 'number' && Number.isFinite(n) && n >= 0)) {
     return toPrice(v as [number, number, number, number]);
@@ -56,6 +52,8 @@ function asPrice(v: unknown): ModelPrice | null {
   if (v && typeof v === 'object') {
     const o = v as Record<string, unknown>;
     if (['input', 'output', 'cacheRead', 'cacheWrite'].every((k) => typeof o[k] === 'number' && Number.isFinite(o[k]) && (o[k] as number) >= 0)) {
+      // SAFETY: the line above verified all four keys are finite numbers >= 0,
+      // which is the whole of ModelPrice.
       return o as unknown as ModelPrice;
     }
   }
@@ -90,22 +88,36 @@ function num(v: unknown): number | null {
   return typeof v === 'number' && Number.isFinite(v) && v >= 0 ? v : null;
 }
 
+// One model, two spellings: transcripts say `space-bunny`, the feed keys it
+// `openrouter/stealth/space-bunny-alpha` at zero. Aliases fold to the first, so
+// a free stealth model never reports the default price.
+export const MODEL_ALIASES: ReadonlyArray<{ id: string; feed: string }> = [
+  { id: 'space-bunny', feed: 'stealth/space-bunny-alpha' },
+];
+
+export function canonicalPriceId(model: string): string {
+  const key = model.toLowerCase();
+  return MODEL_ALIASES.find((a) => a.id === key || a.feed === key)?.id ?? key;
+}
+
 export function priceFor(model: string, live?: LivePrices | null): { price: ModelPrice; known: boolean; live: boolean } {
   const table = live === undefined ? loadLivePrices() : live;
+  // An alias goes through the same exact-then-suffix chain as any other id, so
+  // it still resolves when the feed keys it under a provider prefix.
+  const id = MODEL_ALIASES.find((a) => a.id === model.toLowerCase())?.feed ?? model;
   if (table) {
-    const hit = asPrice(table.exact[model] ?? table.exact[model.toLowerCase()]);
+    const hit = asPrice(table.exact[id] ?? table.exact[id.toLowerCase()]);
     if (hit) return { price: hit, known: true, live: true };
     // Provider-prefixed ids ("azure/gpt-4o", "dashscope/qwen-max"): match the
     // first cached id that ends with the bare model name.
-    const name = model.toLowerCase();
+    const name = id.toLowerCase();
     const key = Object.keys(table.exact).find((k) => k.toLowerCase() === name || k.toLowerCase().endsWith(`/${name}`));
     const prefixed = key ? asPrice(table.exact[key]) : null;
     if (prefixed) return { price: prefixed, known: true, live: true };
   }
   return { price: DEFAULT_PRICE, known: false, live: false };
 }
-// Fetch LiteLLM pricing, keep every id with valid input/output rates, write
-// the cache. False on any failure (offline, timeout, shape change) — callers
+// Fetch LiteLLM pricing and write the cache. False on any failure — callers
 // fall back silently.
 export async function refreshPrices(): Promise<boolean> {
   if (refreshInflight) return refreshInflight;
@@ -160,8 +172,8 @@ export async function refreshPrices(): Promise<boolean> {
   }
 }
 
-// Fire-and-forget refresh when the cache is stale or missing. Readers keep
-// using the stale file (or the built-in table) — pricing never blocks.
+// Fire-and-forget refresh when the cache is stale; readers keep the stale
+// file, so pricing never blocks.
 export function refreshPricesIfStale(): void {
   const live = loadLivePrices();
   if (live && Date.now() - live.fetchedAt <= CACHE_TTL_MS) return;

package/extensions/shared/rtk-gain.js

@@ -1,28 +1,18 @@
-// extensions/shared/rtk-gain.ts — measured RTK savings from its own store.
-// RTK records every invocation (input/output/saved tokens, savings %, exec
-// time) in history.db; `rtk gain` renders the same rows. We read them with
-// the sqlite3 CLI (present on macOS; best-effort elsewhere) so the dashboard
-// shows measured figures, never estimates. Missing DB/CLI → empty, no throw.
-//
-// Caveman and Ponytail have no per-command counters (instruction-following
-// isn't metered); their gains are bench-measured in BENCHMARK.md and cited
-// as static figures wherever RTK rows appear.
-//
-// Rows group by command alone, machine-wide: one row per command no matter
-// which repo ran it. Project filtering stays out — wasted column, same logic.
+// extensions/shared/rtk-gain.ts — measured RTK savings read from rtk's own
+// history.db with the sqlite3 CLI, so the dashboard shows measured figures and
+// never estimates. Missing DB or CLI returns empty. Caveman and Ponytail are
+// bench-measured in BENCHMARK.md instead: instruction-following is not metered.
 import { execFileSync } from 'node:child_process';
 import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
 const EMPTY = { commands: 0, saved: 0, input: 0, avgPct: 0, totalMs: 0, byCommand: [] };
-// RTK stores the whole command line, and `rtk ls -d` with a long path list
-// runs to tens of KB. The table shows one clipped cell, so bound the string
-// before it crosses the sqlite3 CLI and lands in every /data.json poll.
+// The table shows one clipped cell, so bound the command before it crosses the
+// sqlite3 CLI and lands in every /data.json poll.
 const MAX_CMD_CHARS = 200;
-// The query() reader is line- and tab-delimited, and RTK stores heredocs and
-// `node -e` scripts verbatim (~67 rows here), so one logical row can arrive as
-// several lines and derail every row after it. Fold whitespace into spaces
-// first: the stored command keeps its newlines, only the read view flattens.
+// RTK stores heredocs and `node -e` scripts verbatim, so one row can arrive as
+// several lines and derail every row after it. The stored command keeps its
+// newlines; only the read view flattens.
 const CMD_ONE_LINE = `replace(replace(replace(rtk_cmd, char(10), ' '), char(13), ' '), char(9), ' ')`;
 export function rtkDbPath() {
     const override = process.env.TERSIO_RTK_DB;
@@ -49,14 +39,13 @@ export function readRtkGain(limit = 10, cutoffMs) {
     try {
         if (!fs.existsSync(rtkDbPath()))
             return EMPTY;
-        // View-level cutoff only: rows stay in rtk's own database untouched.
-        // strftime('%s') normalizes ISO-8601 timestamps to epoch seconds; rows
-        // with unparseable timestamps (0) drop out of a filtered view.
+        // View-level cutoff only: rtk's own rows stay untouched. Timestamps that do
+        // not parse land on 0 and drop out of a filtered view.
         const where = cutoffMs && cutoffMs > 0
             ? `WHERE CAST(strftime('%s', timestamp) AS INTEGER) >= ${Math.floor(cutoffMs / 1000)}`
             : '';
-        const [[commands = '0', saved = '0', input = '0', avgPct = '0', totalMs = '0'] = []] = query(rtkDbPath(), `SELECT COUNT(*), COALESCE(SUM(saved_tokens),0), COALESCE(SUM(input_tokens),0), COALESCE(AVG(savings_pct),0), COALESCE(SUM(exec_time_ms),0) FROM commands ${where};`);
-        const byCommand = query(rtkDbPath(), `SELECT substr(${CMD_ONE_LINE}, 1, ${MAX_CMD_CHARS}), COUNT(*), COALESCE(SUM(saved_tokens),0), COALESCE(AVG(savings_pct),0), COALESCE(AVG(exec_time_ms),0) FROM commands ${where} GROUP BY rtk_cmd ORDER BY SUM(saved_tokens) DESC LIMIT ${Math.max(1, Math.floor(limit))};`).map(([command, count, savedRow, pct, ms]) => ({
+        const [[commands = '0', saved = '0', input = '0', totalMs = '0'] = []] = query(rtkDbPath(), `SELECT COUNT(*), COALESCE(SUM(saved_tokens),0), COALESCE(SUM(input_tokens),0), COALESCE(SUM(exec_time_ms),0) FROM commands ${where};`);
+        const byCommand = query(rtkDbPath(), `SELECT substr(${CMD_ONE_LINE}, 1, ${MAX_CMD_CHARS}), COUNT(*), COALESCE(SUM(saved_tokens),0), COALESCE(100.0 * SUM(saved_tokens) / NULLIF(SUM(input_tokens), 0), 0), COALESCE(AVG(exec_time_ms),0) FROM commands ${where} GROUP BY rtk_cmd ORDER BY SUM(saved_tokens) DESC LIMIT ${Math.max(1, Math.floor(limit))};`).map(([command, count, savedRow, pct, ms]) => ({
             command,
             count: Number(count) || 0,
             saved: Number(savedRow) || 0,

package/extensions/shared/rtk-gain.ts

@@ -1,15 +1,7 @@
-// extensions/shared/rtk-gain.ts — measured RTK savings from its own store.
-// RTK records every invocation (input/output/saved tokens, savings %, exec
-// time) in history.db; `rtk gain` renders the same rows. We read them with
-// the sqlite3 CLI (present on macOS; best-effort elsewhere) so the dashboard
-// shows measured figures, never estimates. Missing DB/CLI → empty, no throw.
-//
-// Caveman and Ponytail have no per-command counters (instruction-following
-// isn't metered); their gains are bench-measured in BENCHMARK.md and cited
-// as static figures wherever RTK rows appear.
-//
-// Rows group by command alone, machine-wide: one row per command no matter
-// which repo ran it. Project filtering stays out — wasted column, same logic.
+// extensions/shared/rtk-gain.ts — measured RTK savings read from rtk's own
+// history.db with the sqlite3 CLI, so the dashboard shows measured figures and
+// never estimates. Missing DB or CLI returns empty. Caveman and Ponytail are
+// bench-measured in BENCHMARK.md instead: instruction-following is not metered.
 import { execFileSync } from 'node:child_process';
 import fs from 'node:fs';
 import os from 'node:os';
@@ -34,15 +26,13 @@ export interface RtkGain {
 
 const EMPTY: RtkGain = { commands: 0, saved: 0, input: 0, avgPct: 0, totalMs: 0, byCommand: [] };
 
-// RTK stores the whole command line, and `rtk ls -d` with a long path list
-// runs to tens of KB. The table shows one clipped cell, so bound the string
-// before it crosses the sqlite3 CLI and lands in every /data.json poll.
+// The table shows one clipped cell, so bound the command before it crosses the
+// sqlite3 CLI and lands in every /data.json poll.
 const MAX_CMD_CHARS = 200;
 
-// The query() reader is line- and tab-delimited, and RTK stores heredocs and
-// `node -e` scripts verbatim (~67 rows here), so one logical row can arrive as
-// several lines and derail every row after it. Fold whitespace into spaces
-// first: the stored command keeps its newlines, only the read view flattens.
+// RTK stores heredocs and `node -e` scripts verbatim, so one row can arrive as
+// several lines and derail every row after it. The stored command keeps its
+// newlines; only the read view flattens.
 const CMD_ONE_LINE = `replace(replace(replace(rtk_cmd, char(10), ' '), char(13), ' '), char(9), ' ')`;
 
 export function rtkDbPath(): string {
@@ -70,19 +60,18 @@ function query(db: string, sql: string): string[][] {
 export function readRtkGain(limit = 10, cutoffMs?: number): RtkGain {
   try {
     if (!fs.existsSync(rtkDbPath())) return EMPTY;
-    // View-level cutoff only: rows stay in rtk's own database untouched.
-    // strftime('%s') normalizes ISO-8601 timestamps to epoch seconds; rows
-    // with unparseable timestamps (0) drop out of a filtered view.
+    // View-level cutoff only: rtk's own rows stay untouched. Timestamps that do
+    // not parse land on 0 and drop out of a filtered view.
     const where = cutoffMs && cutoffMs > 0
       ? `WHERE CAST(strftime('%s', timestamp) AS INTEGER) >= ${Math.floor(cutoffMs / 1000)}`
       : '';
-    const [[commands = '0', saved = '0', input = '0', avgPct = '0', totalMs = '0'] = []] = query(
+    const [[commands = '0', saved = '0', input = '0', totalMs = '0'] = []] = query(
       rtkDbPath(),
-      `SELECT COUNT(*), COALESCE(SUM(saved_tokens),0), COALESCE(SUM(input_tokens),0), COALESCE(AVG(savings_pct),0), COALESCE(SUM(exec_time_ms),0) FROM commands ${where};`,
+      `SELECT COUNT(*), COALESCE(SUM(saved_tokens),0), COALESCE(SUM(input_tokens),0), COALESCE(SUM(exec_time_ms),0) FROM commands ${where};`,
     );
     const byCommand = query(
       rtkDbPath(),
-      `SELECT substr(${CMD_ONE_LINE}, 1, ${MAX_CMD_CHARS}), COUNT(*), COALESCE(SUM(saved_tokens),0), COALESCE(AVG(savings_pct),0), COALESCE(AVG(exec_time_ms),0) FROM commands ${where} GROUP BY rtk_cmd ORDER BY SUM(saved_tokens) DESC LIMIT ${Math.max(1, Math.floor(limit))};`,
+      `SELECT substr(${CMD_ONE_LINE}, 1, ${MAX_CMD_CHARS}), COUNT(*), COALESCE(SUM(saved_tokens),0), COALESCE(100.0 * SUM(saved_tokens) / NULLIF(SUM(input_tokens), 0), 0), COALESCE(AVG(exec_time_ms),0) FROM commands ${where} GROUP BY rtk_cmd ORDER BY SUM(saved_tokens) DESC LIMIT ${Math.max(1, Math.floor(limit))};`,
     ).map(([command, count, savedRow, pct, ms]) => ({
       command,
       count: Number(count) || 0,

package/extensions/shared/session-state.js

@@ -7,9 +7,9 @@ export const COMBO_LEVELS = Object.freeze({
     max: Object.freeze({ level: 'max', caveman: 'ultra', rtk: 'on', ponytail: 'ultra' }),
 });
 const MODE_VALUES = {
-    caveman: new Set(['off', 'lite', 'full', 'ultra', 'wenyan']),
+    caveman: new Set(['off', 'lite', 'full', 'ultra', 'wenyan', 'wenyan-lite', 'wenyan-full', 'wenyan-ultra']),
     rtk: new Set(['off', 'on']),
-    ponytail: new Set(['off', 'lite', 'full', 'ultra', 'review']),
+    ponytail: new Set(['off', 'lite', 'full', 'ultra']),
 };
 const MODE_ENTRY_TYPES = {
     'caveman-mode': 'caveman',
@@ -20,7 +20,9 @@ export function normalizeMode(name, value) {
     if (name === 'rtk' && typeof value === 'boolean')
         return value ? 'on' : 'off';
     const mode = String(value ?? '').trim().toLowerCase();
-    return MODE_VALUES[name]?.has(mode) ? mode : null;
+    if (!MODE_VALUES[name]?.has(mode))
+        return null;
+    return name === 'caveman' && mode === 'wenyan' ? 'wenyan-full' : mode;
 }
 export function deriveLevel(modes) {
     for (const level of Object.keys(COMBO_LEVELS)) {
@@ -74,23 +76,22 @@ export function normalizeComboLevel(value) {
     const level = String(value || '').trim().toLowerCase();
     return isPresetLevel(level) ? level : null;
 }
-export function paintStatusBar(ui, key, emoji, label, isActive) {
+// OMP themes status text through ctx.ui.theme.fg; pi has no ctx.ui.theme, so
+// the same string renders plain there.
+export function themeStatus(ui, indicator, label, highlight = false) {
     const theme = ui?.theme;
-    const indicator = isActive && theme?.fg ? theme.fg('accent', emoji) : emoji;
-    ui?.setStatus?.(key, theme?.fg ? `${indicator} ${theme.fg('muted', label)}` : `${indicator} ${label}`);
-}
-// Which context a status bar should remember to paint through.
-//
-// A UI-less event — a headless child session_start, or a top-level
-// session_start that fires before the TUI attaches — must never become the
-// remembered context. Two things went wrong when it did: the old
-// `ctx || lastCtx` fallback only triggered for a *falsy* ctx, so a truthy
-// ctx without `ui` returned early anyway, and remembering it destroyed the one
-// paintable context we had. Every later bare syncStatus() then went mute —
-// and the bare call is the bridge-listener path, which is how a mode change
-// made by a sibling extension reaches this bar. The symptom is a bar frozen on
-// a preset the session no longer has: it shows "combo BALANCED" while caveman
-// and rtk are already off.
+    if (!theme?.fg)
+        return `${indicator} ${label}`;
+    return `${highlight ? theme.fg('accent', indicator) : indicator} ${theme.fg('muted', label)}`;
+}
+export function paintStatusBar(ui, key, emoji, label, isActive) {
+    ui?.setStatus?.(key, themeStatus(ui, emoji, label, isActive));
+}
+// Which context a status bar paints through. A UI-less event (a headless child
+// session_start, or a top-level one that fires before the TUI attaches) must
+// never be remembered: doing so mutes every later bare syncStatus(), which is
+// the bridge-listener path a sibling's mode change arrives on. The symptom is a
+// bar frozen on a preset the session no longer has.
 export function paintableCtx(remembered, next) {
     return next?.ui?.setStatus ? next : remembered;
 }
@@ -129,8 +130,7 @@ export function setSharedComboMode(name, value) {
         return getSharedComboState();
     const modes = { ...getSharedComboState(), [name]: mode };
     // Derive the level from the final triplet so a redundant same-value write
-    // (entry replay on resume, upstream ponytail re-affirming its mode) cannot
-    // strand a matching preset at 'custom' and drop the combo bar.
+    // cannot strand a matching preset at 'custom' and drop the combo bar.
     return publish(normalizedState(modes));
 }
 export function reconcileSharedComboEntries(entries) {
@@ -154,19 +154,17 @@ export function reconcileSharedComboEntries(entries) {
                 modes[name] = mode;
         }
     }
-    // The level always reflects the final mode triplet, never the write order:
-    // a preset entry followed by the same values reconciles back to the preset.
+    // The level reflects the final triplet, never the write order, so a preset
+    // entry followed by the same values reconciles back to the preset.
     return publish(normalizedState(modes));
 }
-// One-line session-wide mode summary for change confirmations, e.g.
-// `caveman=ULTRA, rtk=ON, ponytail=ULTRA`.
+// One-line session-wide mode summary for change confirmations.
 export function activeModesSummary(state) {
     return `caveman=${state.caveman.toUpperCase()}, rtk=${state.rtk.toUpperCase()}, ponytail=${state.ponytail.toUpperCase()}`;
 }
 export function setSharedComboListener(listener) {
-    // Additive: every sibling extension syncs its local mirror on publish, so
-    // a /tersio or /combo switch takes effect next turn with no session reload.
-    // A null listener clears all (no current callers; reserved for teardown).
+    // Additive: every sibling syncs its mirror on publish, so a /tersio or
+    // /combo switch lands next turn with no reload. A null listener clears all.
     if (typeof listener === 'function')
         bridge().listeners.add(listener);
     else