@krtclcdy/tersio 2.22.1 → 2.24.0

package/cli/common.js

@@ -16,13 +16,12 @@ const OMP_BIN = IS_WINDOWS ? 'omp.cmd' : 'omp';
 const PACKAGE_NAME = '@krtclcdy/tersio';
 const PACKAGE_BIN = 'tersio';
 const { version: PACKAGE_VERSION } = createRequire(import.meta.url)('../package.json');
-// Session-start mode defaults for fresh installs.
-const CAVEMAN_DEFAULTS = new Set(['off', 'lite', 'full', 'ultra', 'wenyan']);
-const PONYTAIL_DEFAULTS = new Set(['off', 'lite', 'full', 'ultra', 'review']);
+const CAVEMAN_DEFAULTS = new Set(['off', 'lite', 'full', 'ultra', 'wenyan-lite', 'wenyan-full', 'wenyan-ultra']);
+const PONYTAIL_DEFAULTS = new Set(['off', 'lite', 'full', 'ultra']);
 const RTK_DEFAULTS = new Set(['on', 'off']);
 const DIAG_SCHEDULES = new Set(['manual', 'daily', 'weekly', 'monthly']);
-// ponytail: Combo preset implies all three modes. Mirrors COMBO_LEVELS in
-// extensions/shared/session-state.ts (rtk as boolean here). Single source.
+// Mirrors COMBO_LEVELS in extensions/shared/session-state.ts (rtk as a
+// boolean here), so the combo preset implies all three modes.
 const COMBO_PRESET_MODES = {
     off: { caveman: 'off', rtk: false, ponytail: 'off' },
     medium: { caveman: 'lite', rtk: true, ponytail: 'lite' },
@@ -48,13 +47,12 @@ function flagValue(name) {
     return args.find((arg) => arg.startsWith(prefix))?.slice(prefix.length);
 }
 // --- CLI flags ---
-const COMMANDS = { install: true, update: true, reinstall: true, doctor: true, uninstall: true, usage: true, dashboard: true, reset: true, settings: true, version: true, help: true };
+const COMMANDS = { install: true, update: true, doctor: true, uninstall: true, usage: true, dashboard: true, reset: true, settings: true, version: true, help: true };
 const args = process.argv.slice(2);
 const commandArg = args.find((arg) => !arg.startsWith('-'));
 const command = commandArg?.toLowerCase() || null;
-// Positional after the command: `tersio settings diagnosis` jumps straight
-// to one prompt instead of walking the whole chain. Index-based so flag
-// values (e.g. `--combo-default balanced`) never count as positionals.
+// `tersio settings diagnosis` jumps to one prompt instead of walking the
+// chain. Index-based, so flag values never count as positionals.
 const settingArg = (() => {
     const at = args.indexOf(commandArg ?? '');
     const next = at >= 0 ? args[at + 1] : undefined;
@@ -65,12 +63,11 @@ const settingArg = (() => {
 const unknownCommand = command !== null && !COMMANDS[command];
 const install = command === 'install';
 const update = command === 'update';
-const reinstall = command === 'reinstall';
 const showVersion = command === 'version' || args.includes('--version') || args.includes('-v');
 const showHelp = command === 'help' || args.includes('--help') || args.includes('-h');
 const applyUpdate = args.includes('--apply-update');
 const dryRun = args.includes('--dry-run');
-const yes = args.includes('--yes') || args.includes('-y') || install || update || reinstall || applyUpdate;
+const yes = args.includes('--yes') || args.includes('-y') || install || update || applyUpdate;
 const verbose = args.includes('--verbose');
 const doctor = command === 'doctor' || args.includes('--doctor');
 const fix = args.includes('--fix') || args.some((a) => a.startsWith('--fix='));
@@ -81,16 +78,15 @@ const settings = command === 'settings';
 const dashboardPort = Number.parseInt(flagValue('--port') ?? '', 10) || 0;
 const dashboardOpen = args.includes('--open');
 const dashboardExport = flagValue('--export') ?? null;
-// Display currency for usage/dashboard: --currency flag wins, then the stored
-// plugin default (tersio settings), then USD.
+// Display currency: --currency flag, then the stored default, then USD.
 const currency = parseCurrencyFlag(flagValue('--currency')) ?? readStoredCurrency();
 const currencyGiven = flagValue('--currency') !== undefined;
 const uninstall = command === 'uninstall' || args.includes('--uninstall');
 const removePonytail = args.includes('--remove-ponytail');
 const keepPonytail = args.includes('--keep-ponytail');
 const removeRtk = args.includes('--remove-rtk');
-// Project scope was removed: tersio installs user-level (all OMP sessions)
-// only. A bare `--scope user` still parses (old scripts); anything else fails.
+// User scope only (all OMP sessions). A bare `--scope user` still parses for
+// old scripts; anything else fails.
 const legacyScope = flagValue('--scope')?.toLowerCase() ?? null;
 if (legacyScope !== null && legacyScope !== 'user') {
     console.error(`[fail] Invalid --scope: ${legacyScope}. Project scope was removed; tersio installs user-level only.`);
@@ -137,8 +133,8 @@ async function writeIfChanged(dest, content, options = {}) {
         console.log(`  [write] ${dest}`);
     return true;
 }
-// omp ships 'extensions: null'; appending list items under a null scalar breaks
-// YAML parsing. Normalize null/~/[]/empty to a mapping key first.
+// omp ships 'extensions: null', and list items under a null scalar break YAML:
+// normalize null/~/[]/empty to a mapping key first.
 const EXTENSIONS_KEY_RE = /^\s*extensions\s*:/i;
 const EXTENSIONS_NULL_RE = /^\s*extensions\s*:\s*(?:\[\s*\]|null|~)?\s*$/i;
 function normalizeExtensionsKey(lines) {
@@ -160,14 +156,32 @@ async function ensureExtensionInConfig(configPath, extensionPath, label, options
     const line = `  - ${normalizedPath}`;
     const raw = await readTextIfExists(configPath);
     let lines = (raw || '').split('\n');
-    // Drop legacy compiled twins (same dir/name, .js): previous installs
-    // registered them, and OMP must never load both copies. Match on the
-    // trailing path so absolute and relative entries are both caught.
+    // Drop legacy compiled twins (same dir/name, .js): OMP must never load both
+    // copies. Matched on the trailing path so relative entries are caught too.
     if (normalizedPath.endsWith('.ts')) {
         const legacyTail = `${normalizedPath.slice(0, -3)}.js`.split('/').slice(-2).join('/');
         lines = lines.filter((l) => !l.trim().replace(/^\.\//, '').endsWith(legacyTail));
     }
-    if (lines.some((l) => l.includes(normalizedPath))) {
+    const matchingEntries = lines.filter((entry) => entry.trim().replace(/^-\s*/, '').replace(/^['"]|['"]$/g, '') === normalizedPath);
+    if (matchingEntries.length > 1) {
+        if (options.dryRun) {
+            if (verbose && !options.quiet)
+                console.log(`  [dry-run] would remove ${matchingEntries.length - 1} duplicate ${label} registration${matchingEntries.length === 2 ? '' : 's'}`);
+            return true;
+        }
+        let kept = false;
+        lines = lines.filter((entry) => {
+            if (entry.trim().replace(/^-\s*/, '').replace(/^['"]|['"]$/g, '') !== normalizedPath)
+                return true;
+            if (kept)
+                return false;
+            kept = true;
+            return true;
+        });
+        await writeConfigLines(configPath, lines, `  [write] Removed duplicate ${label} registration`, options);
+        return true;
+    }
+    if (matchingEntries.length === 1) {
         debug(`${label} already in config.yml`);
         return false;
     }
@@ -191,43 +205,29 @@ async function ensureExtensionInConfig(configPath, extensionPath, label, options
     await writeConfigLines(configPath, lines, `  [write] Added ${label} to config.yml`, options);
     return true;
 }
-async function ensureExtensionAfterConfigEntry(configPath, extensionPath, afterPath, label, options = {}) {
-    const normalizedPath = extensionPath.replace(/\\/g, '/');
-    const normalizedAfterPath = afterPath.replace(/\\/g, '/');
-    const line = `  - ${normalizedPath}`;
+async function removeExtensionFromConfig(configPath, extensionPath, label, options = {}) {
+    const normalizedPath = extensionPath.replace(/\\/g, '/').replace(/^\.\//, '');
+    const extensionSuffix = normalizedPath.includes('/extensions/')
+        ? normalizedPath.slice(normalizedPath.indexOf('/extensions/') + 1)
+        : normalizedPath;
     const raw = await readTextIfExists(configPath);
-    let lines = (raw || '').split('\n');
-    // Same legacy-twin drop as ensureExtensionInConfig above.
-    if (normalizedPath.endsWith('.ts')) {
-        const legacyTail = `${normalizedPath.slice(0, -3)}.js`.split('/').slice(-2).join('/');
-        lines = lines.filter((l) => !l.trim().replace(/^\.\//, '').endsWith(legacyTail));
-    }
-    const existingIndex = lines.findIndex((entry) => entry.includes(normalizedPath));
-    const afterIndex = lines.findIndex((entry) => entry.includes(normalizedAfterPath));
-    if (existingIndex !== -1 && afterIndex !== -1 && existingIndex === afterIndex + 1)
+    if (raw === null)
+        return false;
+    const lines = raw.split('\n');
+    const matches = (entry) => {
+        const value = entry.trim().replace(/^-\s*/, '').replace(/^['"]|['"]$/g, '').replace(/\\/g, '/').replace(/^\.\//, '');
+        return value === extensionSuffix || value.endsWith(`/${extensionSuffix}`);
+    };
+    const kept = lines.filter((entry) => !matches(entry));
+    const removed = lines.length - kept.length;
+    if (removed === 0)
         return false;
     if (options.dryRun) {
         if (verbose && !options.quiet)
-            console.log(`  [dry-run] would place ${label} after Ponytail in config.yml`);
+            console.log(`  [dry-run] would remove ${label} registration${removed === 1 ? '' : 's'}`);
         return true;
     }
-    if (existingIndex !== -1)
-        lines.splice(existingIndex, 1);
-    const refreshedAfterIndex = lines.findIndex((entry) => entry.includes(normalizedAfterPath));
-    if (refreshedAfterIndex !== -1) {
-        lines.splice(refreshedAfterIndex + 1, 0, line);
-    }
-    else {
-        const extensionsIndex = lines.findIndex((entry) => EXTENSIONS_KEY_RE.test(entry));
-        if (extensionsIndex === -1) {
-            lines.push('extensions:', line, '');
-        }
-        else {
-            normalizeExtensionsKey(lines);
-            lines.splice(extensionsIndex + 1, 0, line);
-        }
-    }
-    await writeConfigLines(configPath, lines, `  [write] Placed ${label} after Ponytail in config.yml`, options);
+    await writeConfigLines(configPath, kept, `  [write] Removed duplicate ${label} registration`, options);
     return true;
 }
 function readPonytailConfig() {
@@ -277,8 +277,8 @@ async function ensurePonytailConfigValue(key, value, options = {}) {
         return true;
     }, `would set Ponytail ${key}=${value}`, `Set Ponytail ${key}=${value}`, options);
 }
-// Read plugins/package.json tolerantly; corrupt or missing files start fresh
-// with installer-managed defaults.
+// Read plugins/package.json tolerantly; a corrupt or missing file starts
+// fresh with installer-managed defaults.
 async function readPluginsPackage(pkgPath) {
     const pkg = parseJsonObject(await readTextIfExists(pkgPath)) ?? {};
     return { ...pkg, name: pkg.name || 'omp-plugins', private: true, dependencies: pkg.dependencies || {} };
@@ -298,4 +298,4 @@ function relTime(ageMs) {
     const weeks = Math.floor(days / 7);
     return `${weeks} week${weeks === 1 ? '' : 's'} ago`;
 }
-export { IS_WINDOWS, RTK_BINARY_NAME, HOME, OMP_AGENT_DIR, OMP_PLUGINS_DIR, BUN_BIN_DIR, OMP_BIN, PACKAGE_NAME, PACKAGE_BIN, PACKAGE_VERSION, CAVEMAN_DEFAULTS, PONYTAIL_DEFAULTS, RTK_DEFAULTS, DIAG_SCHEDULES, COMBO_PRESET_MODES, COMBO_DEFAULTS, parseEnum, flagValue, COMMANDS, commandArg, command, settingArg, unknownCommand, install, update, reinstall, showVersion, showHelp, applyUpdate, args, dryRun, yes, verbose, doctor, fix, uninstall, usage, dashboard, reset, settings, dashboardPort, dashboardOpen, dashboardExport, currency, currencyGiven, removePonytail, keepPonytail, removeRtk, comboDefaultFlag, cavemanDefaultFlag, ponytailDefaultFlag, rtkDefaultFlag, diagScheduleFlag, profileFlagsGiven, debug, execFileP, execP, writeIfChanged, normalizeExtensionsKey, EXTENSIONS_KEY_RE, writeConfigLines, ensureExtensionInConfig, ensureExtensionAfterConfigEntry, readPonytailConfig, parseJsonObject, parsePonytailConfig, patchPonytailConfig, ensurePonytailConfigValue, readPluginsPackage, relTime, };
+export { IS_WINDOWS, RTK_BINARY_NAME, HOME, OMP_AGENT_DIR, OMP_PLUGINS_DIR, BUN_BIN_DIR, OMP_BIN, PACKAGE_NAME, PACKAGE_BIN, PACKAGE_VERSION, CAVEMAN_DEFAULTS, PONYTAIL_DEFAULTS, RTK_DEFAULTS, DIAG_SCHEDULES, COMBO_PRESET_MODES, COMBO_DEFAULTS, parseEnum, flagValue, COMMANDS, commandArg, command, settingArg, unknownCommand, install, update, showVersion, showHelp, applyUpdate, args, dryRun, yes, verbose, doctor, fix, uninstall, usage, dashboard, reset, settings, dashboardPort, dashboardOpen, dashboardExport, currency, currencyGiven, removePonytail, keepPonytail, removeRtk, comboDefaultFlag, cavemanDefaultFlag, ponytailDefaultFlag, rtkDefaultFlag, diagScheduleFlag, profileFlagsGiven, debug, execFileP, execP, writeIfChanged, normalizeExtensionsKey, EXTENSIONS_KEY_RE, writeConfigLines, ensureExtensionInConfig, removeExtensionFromConfig, readPonytailConfig, parseJsonObject, parsePonytailConfig, patchPonytailConfig, ensurePonytailConfigValue, readPluginsPackage, relTime, };

package/cli/currency.js

@@ -3,8 +3,7 @@
 // converter (dashboard/app); the dashboard replaces rates with live
 // frankfurter figures when reachable, the CLI always uses the snapshot.
 import { existsSync, readFileSync } from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
+import { tersioSettingsFile } from '../extensions/shared/plugin-settings.js';
 const CURRENCY_CODES = [
     'USD',
     'PHP',
@@ -36,18 +35,14 @@ function parseCurrencyFlag(raw) {
 // USD when missing, corrupt, or holding an unknown code. Sync so flag
 // parsing in cli/common.ts can fall back to it at startup.
 function readStoredCurrency() {
-    const home = process.env.HOME || process.env.USERPROFILE || os.homedir();
-    const lock = path.join(home, '.omp', 'plugins', 'omp-plugins.lock.json');
-    if (!existsSync(lock))
+    const file = tersioSettingsFile();
+    if (!existsSync(file))
         return DEFAULT_CURRENCY;
     try {
-        const parsed = JSON.parse(readFileSync(lock, 'utf8'));
-        if (!parsed || typeof parsed !== 'object' || !('settings' in parsed))
+        const parsed = JSON.parse(readFileSync(file, 'utf8'));
+        if (!parsed || typeof parsed !== 'object')
             return DEFAULT_CURRENCY;
-        const entry = parsed.settings?.['@krtclcdy/tersio'];
-        if (!entry || typeof entry !== 'object')
-            return DEFAULT_CURRENCY;
-        const raw = entry.currency;
+        const raw = parsed.currency;
         return typeof raw === 'string' && isCurrencyCode(raw.trim().toUpperCase())
             ? raw.trim().toUpperCase()
             : DEFAULT_CURRENCY;
@@ -68,7 +63,8 @@ const CURRENCY_META = {
     CAD: { symbol: 'C$', decimals: 2 },
     INR: { symbol: '₹', decimals: 2 },
 };
-const FX_SNAPSHOT_RATES = {
+// Exported so a test can hold it against the Dashboard's copy of the same table.
+export const FX_SNAPSHOT_RATES = {
     USD: 1,
     PHP: 58.7,
     EUR: 0.92,

package/cli/dashboard.js

@@ -1,7 +1,5 @@
-// cli/dashboard.ts — serves the built shadcn Dashboard with local usage APIs.
-// Binds 127.0.0.1 only; --export writes a file://-ready file instead.
-// Note: no Promise.withResolvers here — engines still allow Node 20.12,
-// which lacks it; the listening server itself keeps the process alive.
+// Serves the built Dashboard with local usage APIs. Binds 127.0.0.1 only;
+// --export writes a file instead. No Promise.withResolvers: Node 20.12 lacks it.
 import { spawn } from 'node:child_process';
 import { execFileSync } from 'node:child_process';
 import { existsSync, mkdirSync, readFileSync, statSync, writeFileSync } from 'node:fs';
@@ -13,8 +11,10 @@ import { clearUsageLedger, markReset, readUsage } from '../extensions/shared/usa
 import { pricesCachePath } from '../extensions/shared/pricing.js';
 import { summarizeUsage } from './usage.js';
 import { isCurrencyCode } from './currency.js';
-import { BUN_BIN_DIR, OMP_AGENT_DIR, OMP_PLUGINS_DIR, PACKAGE_VERSION, RTK_BINARY_NAME, } from './common.js';
+import { OMP_AGENT_DIR, OMP_PLUGINS_DIR, PACKAGE_VERSION, } from './common.js';
 import { storedProfile, writePluginSettings } from './profile.js';
+import { PACKAGE_NAME } from './common.js';
+import { resolveRtkBinary } from '../extensions/lib/utils.js';
 const DASHBOARD_DIST = path.join(path.dirname(fileURLToPath(import.meta.url)), '..', 'dashboard', 'dist');
 const DASHBOARD_INDEX = path.join(DASHBOARD_DIST, 'index.html');
 const DASHBOARD_BRAND = path.join(DASHBOARD_DIST, 'brand.webp');
@@ -50,10 +50,9 @@ async function brandDataUri() {
 function dataJson() {
     return JSON.stringify(summarizeUsage(readUsage()));
 }
-// Local-only health + diagnosis for the settings modal. No network: version
-// probes run with short timeouts, file checks are existsSync.
-function ompPath() {
-    const names = process.platform === 'win32' ? ['omp.cmd', 'omp.exe', 'omp.bat', 'omp'] : ['omp'];
+// Local-only health: find each agent on PATH, run `--version`, report both.
+function binOnPath(bin) {
+    const names = process.platform === 'win32' ? [`${bin}.cmd`, `${bin}.exe`, `${bin}.bat`, bin] : [bin];
     for (const dir of (process.env.PATH || '').split(path.delimiter).filter(Boolean)) {
         for (const name of names) {
             const candidate = path.join(dir, name);
@@ -66,15 +65,28 @@ function ompPath() {
     }
     return null;
 }
-function ompVersion(bin = ompPath()) {
+function ompPath() {
+    return binOnPath('omp');
+}
+function piPath() {
+    return binOnPath('pi');
+}
+// omp prints `omp/18.4.1`, pi prints a bare `0.87.1`; both read as name/version.
+function versionLabel(bin, raw) {
+    const out = raw.trim();
+    return out.includes('/') ? out : `${bin}/${out}`;
+}
+function agentVersion(bin) {
     if (!bin)
         return null;
     try {
         if (process.platform === 'win32') {
             const exe = process.env.ComSpec || 'cmd.exe';
-            return execFileSync(exe, ['/d', '/s', '/c', bin, '--version'], { encoding: 'utf8', timeout: 5000, windowsHide: true }).trim() || null;
+            const out = execFileSync(exe, ['/d', '/s', '/c', bin, '--version'], { encoding: 'utf8', timeout: 5000, windowsHide: true });
+            return out.trim() ? versionLabel(path.basename(bin), out) : null;
         }
-        return execFileSync(bin, ['--version'], { encoding: 'utf8', timeout: 5000, windowsHide: true }).trim() || null;
+        const out = execFileSync(bin, ['--version'], { encoding: 'utf8', timeout: 5000, windowsHide: true });
+        return out.trim() ? versionLabel(path.basename(bin), out) : null;
     }
     catch {
         return null;
@@ -99,17 +111,20 @@ function ompDefaultModel() {
     }
 }
 function healthJson() {
-    const rtkBin = path.join(BUN_BIN_DIR, RTK_BINARY_NAME);
-    const rtkPresent = existsSync(rtkBin);
+    const rtkBin = resolveRtkBinary();
+    const rtkPresent = rtkBin !== null;
     const detectedOmpPath = ompPath();
+    const detectedPiPath = piPath();
     return JSON.stringify({
         tersio: PACKAGE_VERSION,
         node: process.version,
         platform: `${process.platform}/${process.arch}`,
-        omp: ompVersion(detectedOmpPath),
+        omp: agentVersion(detectedOmpPath),
         ompPath: detectedOmpPath,
+        pi: agentVersion(detectedPiPath),
+        piPath: detectedPiPath,
         provider: ompDefaultModel(),
-        rtk: { present: rtkPresent, version: rtkPresent ? rtkVersion(rtkBin) : null, path: rtkBin },
+        rtk: { present: rtkPresent, version: rtkPresent ? rtkVersion(rtkBin) : null, path: rtkBin || 'not found in PATH' },
         home: tersioHomePath(),
     });
 }
@@ -123,8 +138,7 @@ const DIAG_TTL_MS = {
     monthly: 30 * 24 * 3600 * 1000,
 };
 function diagPaths() {
-    // Hermetic env (tests) overrides the DB path: keep the report next to it
-    // so test runs never touch the real ~/.tersio/diag.json.
+    // Keep the report beside the overridden DB so tests never touch the real one.
     const dbOverride = process.env.TERSIO_USAGE_DB;
     if (dbOverride)
         return { report: path.join(path.dirname(dbOverride), 'diag.json') };
@@ -183,29 +197,39 @@ function relAge(ms) {
 }
 function computeDoctorRows() {
     const rows = [];
-    const extDir = path.join(OMP_AGENT_DIR, 'extensions');
-    const rtkBin = path.join(BUN_BIN_DIR, RTK_BINARY_NAME);
+    const rtkBin = resolveRtkBinary();
     const ponytailPkg = path.join(OMP_PLUGINS_DIR, 'node_modules', '@dietrichgebert', 'ponytail', 'package.json');
+    const tersioPluginDir = path.join(OMP_PLUGINS_DIR, 'node_modules', ...PACKAGE_NAME.split('/'));
     const ok = (p) => existsSync(p);
+    const configPath = path.join(OMP_AGENT_DIR, 'config.yml');
     const ext = (label, p) => {
         rows.push({ label, ok: ok(p), detail: ok(p) ? 'installed' : p, group: 'Extensions' });
     };
     const addon = (label, present, detail, group = 'Add-ons') => {
         rows.push({ label, ok: present, detail, group });
     };
-    ext('Caveman extension', path.join(extDir, 'caveman-session', 'index.ts'));
-    ext('RTK extension', path.join(extDir, 'rtk-session', 'index.ts'));
+    let explicitEntries = [];
+    try {
+        explicitEntries = readFileSync(configPath, 'utf8').split('\n')
+            .map((line) => line.trim().replace(/^-\s*/, '').replace(/^['"]|['"]$/g, ''))
+            .filter((line) => line.startsWith('/') || line.startsWith('.'));
+    }
+    catch { /* missing config is reported by extension rows */ }
+    const duplicateExtensions = [...new Set(explicitEntries.filter((entry, index) => explicitEntries.indexOf(entry) !== index))];
+    addon('Unique config registrations', duplicateExtensions.length === 0, duplicateExtensions.length ? duplicateExtensions.join(', ') : 'ok', 'Extensions');
+    const retiredReinforcement = explicitEntries.filter((entry) => entry.endsWith('/shared/mode-reinforcement.ts')).length;
+    addon('No retired reinforcement', retiredReinforcement === 0, retiredReinforcement ? `${retiredReinforcement} registration(s)` : 'ok', 'Extensions');
+    ext('Caveman extension', path.join(tersioPluginDir, 'extensions', 'caveman-session', 'index.ts'));
+    ext('RTK extension', path.join(tersioPluginDir, 'extensions', 'rtk-session', 'index.ts'));
     ext('Ponytail extension', path.join(OMP_PLUGINS_DIR, 'node_modules', '@dietrichgebert', 'ponytail', 'pi-extension', 'index.js'));
-    const rule = path.join(extDir, 'caveman-session', 'rule.md');
+    const rule = path.join(tersioPluginDir, 'extensions', 'caveman-session', 'rule.md');
     const ruleAge = ageStr(rule);
     const ruleAt = absTime(rule);
     addon('Caveman rule', ruleAge !== null, ruleAge && ruleAt ? `(updated ${ruleAge} · ${ruleAt})` : rule);
-    const rtkVer = ok(rtkBin) ? rtkVersion(rtkBin) : null;
-    const rtkAge = ageStr(rtkBin);
-    const rtkAt = absTime(rtkBin);
-    addon('RTK binary', rtkVer !== null, rtkVer && rtkAge && rtkAt ? `${rtkVer} (updated ${rtkAge} · ${rtkAt})` : rtkBin);
-    const wiring = path.join(extDir, 'rtk.ts');
-    addon('RTK OMP wiring (rtk.ts)', ok(wiring), ok(wiring) ? 'loaded' : wiring);
+    const rtkVer = rtkBin ? rtkVersion(rtkBin) : null;
+    const rtkAge = rtkBin ? ageStr(rtkBin) : null;
+    const rtkAt = rtkBin ? absTime(rtkBin) : null;
+    addon('RTK binary', rtkVer !== null, rtkVer && rtkAge && rtkAt ? `${rtkVer} (updated ${rtkAge} · ${rtkAt})` : rtkBin || 'not found in PATH');
     let ponytailVer = null;
     try {
         ponytailVer = JSON.parse(readFileSync(ponytailPkg, 'utf8')).version ?? null;
@@ -222,10 +246,8 @@ function computeDoctorRows() {
     addon('Prices feed', pricesAge !== null, pricesAge && pricesAt ? `(pulled ${pricesAge} · ${pricesAt})` : prices, 'Usage');
     return rows;
 }
-// Retained diagnosis: recompute when forced, missing, or older than the
-// schedule. Otherwise return the saved report so users never re-run.
-// Labels retired from the report. A saved report containing any of them
-// is stale by definition and always recomputes.
+// Recompute when forced, missing, or past the schedule; a report holding a
+// retired label is stale by definition.
 const RETIRED_DIAG_LABELS = new Set([
     'OMP CLI',
     'Tersio CLI',
@@ -240,13 +262,11 @@ const RETIRED_DIAG_LABELS = new Set([
 function isRetiredReport(report) {
     return report.rows.some((r) => RETIRED_DIAG_LABELS.has(r.label));
 }
-// Retained diagnosis: recompute when forced, missing, retired, or older
-// than the schedule. Otherwise return the saved report.
+// Recompute when forced, missing, retired, or past the schedule.
 function getDoctorReport(force) {
     const saved = readDiagReport();
     const ttl = saved ? DIAG_TTL_MS[saved.schedule] : 0;
-    // Manual always recomputes on open; dated schedules reuse the saved
-    // report until stale. Otherwise a retained snapshot hides new rows.
+    // Manual recomputes on open; a dated schedule reuses the saved report.
     if (!force && saved && !isRetiredReport(saved) && ttl > 0 && Date.now() - saved.checkedAt < ttl)
         return saved;
     const report = { rows: computeDoctorRows(), checkedAt: Date.now(), schedule: saved?.schedule ?? 'manual' };
@@ -262,10 +282,8 @@ function setDiagSchedule(schedule) {
 function readDiagSchedule() {
     return readDiagReport()?.schedule ?? 'manual';
 }
-// Persist the dashboard's picker choice so close → reopen keeps it: each
-// `tersio dashboard` run serves a fresh ephemeral port (a new origin), so the
-// browser's localStorage alone cannot survive a restart. The stored default
-// feeds data.json and `tersio usage` on the next run.
+// Persist the picker choice: every run serves a fresh port, so localStorage
+// alone cannot survive a restart.
 async function saveDashboardCurrency(raw) {
     if (typeof raw !== 'string')
         return null;
@@ -289,21 +307,30 @@ function openBrowser(url) {
     const cmd = process.platform === 'darwin' ? 'open' : process.platform === 'win32' ? 'start' : 'xdg-open';
     spawn(cmd, [url], { detached: true, stdio: 'ignore' }).unref();
 }
-// Replacer functions throughout: session data routinely contains `$'`
-// sequences (shell quoting in tool details), which String.replace would
-// expand as match-suffix patterns and corrupt the file.
-// RTK-metered command lines routinely contain literal `</script>` (Vue
-// SFC probes), which would close the inlined <script> early and dump the
-// rest of the JSON as page text. Escape it; JSON.parse never sees the
-// backslash form inside a string literal, the browser decodes it first.
+// Replacers throughout: session data holds `$'` (shell quoting) and literal
+// `</script>`, which String.replace would mangle.
 function escapeInline(json) {
     return json.replace(/<\/(script)/gi, '<\\/$1');
 }
+// Our own JSON.stringify output, so a parse failure means a bad value: name it.
+function parseOwnJson(json, field) {
+    let parsed;
+    try {
+        parsed = JSON.parse(json);
+    }
+    catch {
+        throw new Error(`dashboard snapshot ${field} is not valid JSON`);
+    }
+    if (!parsed || typeof parsed !== 'object' || Array.isArray(parsed)) {
+        throw new Error(`dashboard snapshot ${field} is not an object`);
+    }
+    return parsed;
+}
 async function exportDashboard(exportFile) {
     requireDashboardBundle();
     const [bundle, icon] = await Promise.all([readSegment(DASHBOARD_INDEX), brandDataUri()]);
     const inline = bundle
-        .replace('window.__TERSIO_SNAP = null;', () => `window.__TERSIO_SNAP = ${escapeInline(JSON.stringify({ data: JSON.parse(dataJson()), health: JSON.parse(healthJson()), doctor: getDoctorReport(false) }))};`)
+        .replace('window.__TERSIO_SNAP = null;', () => `window.__TERSIO_SNAP = ${escapeInline(JSON.stringify({ data: parseOwnJson(dataJson(), 'usage data'), health: parseOwnJson(healthJson(), 'health'), doctor: getDoctorReport(false) }))};`)
         .replace(/href="brand\.webp"/g, () => `href="${icon}"`)
         .replace(/src="brand\.webp"/g, () => `src="${icon}"`)
         .replace('fetch("brand.webp")', () => `Promise.resolve({ ok: true, blob: async () => new Blob([atob("${icon.split(',')[1]}")], { type: "image/webp" }) })`);