@pellux/goodvibes-tui 1.16.1 → 1.18.0

package/src/panels/cost-tracker-panel.ts

@@ -22,15 +22,15 @@ import {
   DEFAULT_PANEL_PALETTE,
   type PanelWorkspaceSection,
 } from './polish.ts';
-import { calcSessionCost, isModelPriced, computeBudgetBreach, readBudgetAlertUsd, BUDGET_ALERT_USD_CONFIG_KEY, type BudgetAlertConfigAccess } from '../export/cost-utils.ts';
+import { calcSessionCost, describePricingSource, isModelPriced, computeBudgetBreach, readBudgetAlertUsd, writeManualModelPrice, BUDGET_ALERT_USD_CONFIG_KEY, type BudgetAlertConfigAccess } from '../export/cost-utils.ts';
 import { abbreviateCount } from '../utils/format-number.ts';
 import { isTextBackspace } from '../input/delete-key-policy.ts';
 
 // Pricing lookups are provided by ../export/cost-utils.ts (single source of truth).
 
-/** unpriced=true renders an honest "unpriced" marker instead of a $0.00 that could be mistaken for a real zero cost. */
+/** unpriced=true renders the explicit "price unknown" marker instead of a $0.00 that could be mistaken for a real zero cost. */
 function formatCost(usd: number, unpriced = false): string {
-  if (unpriced) return 'unpriced';
+  if (unpriced) return 'price unknown';
   if (usd === 0) return '$0.00';
   if (usd < 0.0001) return '<$0.0001';
   if (usd < 0.01) return `$${usd.toFixed(4)}`;
@@ -127,6 +127,12 @@ export class CostTrackerPanel extends BasePanel {
   // secret so it is echoed directly.
   private budgetEntry: string | null = null;
 
+  // Draft buffer for the in-panel manual-price entry mode ('p' key): the one
+  // action away from any price display (and from the "price unknown" marker).
+  // Format: "input,output" in USD per 1M tokens; persists to the
+  // pricing.modelPrices settings key live on Enter.
+  private priceEntry: string | null = null;
+
   // Scroll offset for agent list
   private scrollOffset = 0;
 
@@ -347,11 +353,12 @@ export class CostTrackerPanel extends BasePanel {
    * `Panel.isCapturingTextBurst`.
    */
   isCapturingTextBurst(): boolean {
-    return this.budgetEntry !== null;
+    return this.budgetEntry !== null || this.priceEntry !== null;
   }
 
   handleInput(key: string): boolean {
     if (this.budgetEntry !== null) return this.handleBudgetEntryInput(key);
+    if (this.priceEntry !== null) return this.handlePriceEntryInput(key);
 
     if (key === 'b') {
       const threshold = this.currentBudgetThreshold();
@@ -360,6 +367,16 @@ export class CostTrackerPanel extends BasePanel {
       return true;
     }
 
+    if (key === 'p') {
+      // One action from any price display: edit the manual price for the
+      // session's provider:model. Needs a wired config (production path) —
+      // without one the key falls through rather than opening a dead editor.
+      if (!this.configAccess) return false;
+      this.priceEntry = '';
+      this.markDirty();
+      return true;
+    }
+
     switch (key) {
       case 'up':   return this.scroll(-1);
       case 'down': return this.scroll(1);
@@ -402,6 +419,52 @@ export class CostTrackerPanel extends BasePanel {
     return true; // absorb everything else while entry is active
   }
 
+  /** True when the session model can carry a manual price (pricing.modelPrices keys are `provider:model`). */
+  private canEditSessionModelPrice(): boolean {
+    return this.sessionModel !== 'unknown' && this.sessionModel.includes(':');
+  }
+
+  private handlePriceEntryInput(key: string): boolean {
+    if (key === 'escape') {
+      this.priceEntry = null;
+      this.markDirty();
+      return true;
+    }
+    if (key === 'enter' || key === 'return') {
+      const raw = (this.priceEntry ?? '').trim();
+      if (!this.canEditSessionModelPrice()) {
+        // The prompt already explains why; Enter just closes it.
+        this.priceEntry = null;
+        this.markDirty();
+        return true;
+      }
+      const parts = raw.split(/[\s,]+/).filter((part) => part.length > 0);
+      const input = Number(parts[0]);
+      const output = Number(parts[1]);
+      if (parts.length === 2 && Number.isFinite(input) && input >= 0 && Number.isFinite(output) && output >= 0) {
+        writeManualModelPrice(this.configAccess!, this.sessionModel, { input, output });
+        this.priceEntry = null;
+      }
+      // Malformed input keeps the editor open so the draft can be corrected.
+      this.markDirty();
+      return true;
+    }
+    if (isTextBackspace(key)) {
+      if (this.priceEntry && this.priceEntry.length > 0) {
+        this.priceEntry = this.priceEntry.slice(0, -1);
+        this.markDirty();
+      }
+      return true;
+    }
+    // Two USD amounts separated by a comma (or space) — digits, dots, comma, space.
+    if (key.length === 1 && /[0-9., ]/.test(key)) {
+      this.priceEntry = (this.priceEntry ?? '') + key;
+      this.markDirty();
+      return true;
+    }
+    return true; // absorb everything else while entry is active
+  }
+
   /**
    * Scroll the agent list. Over-consumption fix: only absorbs the key (and
    * only advances) when the agent list is actually long enough to scroll —
@@ -427,20 +490,29 @@ export class CostTrackerPanel extends BasePanel {
   render(width: number, height: number): Line[] {
     if (height <= 0 || width <= 0) return [];
     if (this.budgetEntry !== null) return this.renderBudgetEntryPrompt(width, height);
+    if (this.priceEntry !== null) return this.renderPriceEntryPrompt(width, height);
 
     const totalInputTokens = this.sessionUsage.input + this.sessionUsage.cacheRead + this.sessionUsage.cacheWrite;
     const sessionCost = calcSessionCost(this.sessionUsage.input, this.sessionUsage.output, this.sessionUsage.cacheRead, this.sessionUsage.cacheWrite, this.sessionModel);
     const budgetThreshold = this.currentBudgetThreshold();
     const overBudget = computeBudgetBreach(sessionCost, budgetThreshold);
     const sparkline = buildSparkline(this.costHistory);
-    const costStr = formatCost(sessionCost, !isModelPriced(this.sessionModel));
+    const sessionPriced = isModelPriced(this.sessionModel);
+    const costStr = formatCost(sessionCost, !sessionPriced);
     const costFg = overBudget ? C.bad : C.cost;
+    // Dollars carry their source: "your price" for a manual entry,
+    // "catalog price, as of <date>" for the dated catalog, etc. The
+    // "price unknown" marker carries the one-key action to fix it instead.
+    const sourceDescription = sessionPriced ? describePricingSource(this.sessionModel) : null;
+    const sourceStr = sessionPriced
+      ? (sourceDescription ? ` (${sourceDescription})` : '')
+      : ' — press p to set a price';
     const budgetStr = budgetThreshold > 0
       ? ` / ${formatCost(budgetThreshold)}`
       : '';
     const alertStr = overBudget ? ' ! OVER BUDGET' : '';
     const sessionLines: Line[] = [
-      this.renderKeyValue(width, ' Total', `${costStr}${budgetStr}${alertStr}`, costFg),
+      this.renderKeyValue(width, ' Total', `${costStr}${budgetStr}${alertStr}${sourceStr}`, costFg),
     ];
     // Budget meter — the single most important glance for this panel: how much
     // of the configured budget the session has consumed. Only shown when a
@@ -455,6 +527,13 @@ export class CostTrackerPanel extends BasePanel {
         { filled: meterFg, empty: C.separator, label: C.label },
         { prefix: ' Budget [', suffix: `] ${pctStr}` },
       ));
+      // Unpriced-spend honesty wherever budget state shows: a meter over a
+      // partially-unpriced session must say the % undercounts real spend.
+      const anyUnpricedSpend = (!sessionPriced && (totalInputTokens > 0 || this.sessionUsage.output > 0))
+        || Array.from(this.agents.values()).some((agent) => agent.inputTokens > 0 && !isModelPriced(agent.model));
+      if (anyUnpricedSpend) {
+        sessionLines.push(this.renderKeyValue(width, '', 'some spend has no known price and is not counted above', C.warn));
+      }
     }
     if (sparkline.length > 0) sessionLines.push(this.renderLabeledLine(width, ' Trend', sparkline, C.good));
     sessionLines.push(this.renderKeyValue(width, ' Input',  formatTokens(this.sessionUsage.input),  C.label));
@@ -481,9 +560,11 @@ export class CostTrackerPanel extends BasePanel {
           { keys: 'Up/Down', label: 'scroll agents' },
           { keys: 'PgUp/PgDn', label: 'page' },
           { keys: 'b', label: 'set budget' },
+          { keys: 'p', label: 'set model price' },
         ], DEFAULT_PANEL_PALETTE)
       : buildKeyboardHints(width, [
           { keys: 'b', label: 'set budget alert' },
+          { keys: 'p', label: 'set model price' },
         ], DEFAULT_PANEL_PALETTE);
     if (agentList.length > 0) {
       const planCost = agentList.reduce((sum, a) => sum + a.cost, 0);
@@ -562,7 +643,7 @@ export class CostTrackerPanel extends BasePanel {
           width,
           ' No agents spawned this session',
           'Agent-level cost estimates appear here once delegated or background agents start running.',
-          // WO-160: 'b' already opens the in-panel budget-entry field (see
+          // 'b' already opens the in-panel budget-entry field (see
           // handleInput/handleBudgetEntryInput above) and is advertised in
           // the footer's 'b: set budget' hint, so the printed
           // '/cost budget <usd>' command was a redundant action substitute.
@@ -626,4 +707,40 @@ export class CostTrackerPanel extends BasePanel {
     return workspace.slice(0, height);
   }
 
+  // Non-masked entry for the 'p' manual-price key. Two USD-per-1M-token
+  // amounts (input,output); persists to pricing.modelPrices for the session's
+  // provider:model live. A model without a provider prefix cannot carry a
+  // manual price (the settings key is keyed provider:model) — the prompt says
+  // so instead of accepting a value it could not store.
+  private renderPriceEntryPrompt(width: number, height: number): Line[] {
+    const editable = this.canEditSessionModelPrice();
+    const draft = this.priceEntry ?? '';
+    const display = `${draft}█`;
+    const promptLines: Line[] = editable
+      ? [
+          buildPanelLine(width, [[` Set your manual price for ${this.sessionModel} (USD per 1M tokens).`, C.label]]),
+          buildPanelLine(width, [[' Format: input,output — for example 3.00,15.00', C.dim]]),
+          buildPanelLine(width, [['', C.label]]),
+          buildPanelLine(width, [[' Price  ', C.label], [display, C.cost]]),
+          buildPanelLine(width, [['', C.label]]),
+          buildPanelLine(width, [[' [Enter] Save   [Esc] Cancel   [Backspace] Delete char', C.dim]]),
+        ]
+      : [
+          buildPanelLine(width, [[` The current model (${this.sessionModel}) has no provider prefix, so a`, C.label]]),
+          buildPanelLine(width, [[' manual price cannot be stored for it (prices are keyed provider:model).', C.label]]),
+          buildPanelLine(width, [['', C.label]]),
+          buildPanelLine(width, [[' [Esc] Close', C.dim]]),
+        ];
+    const workspace = buildPanelWorkspace(width, height, {
+      title: ' Cost Tracker — Model Price',
+      intro: editable
+        ? 'Your price wins over provider and catalog prices, immediately and everywhere.'
+        : 'Manual prices are stored per provider:model.',
+      sections: [{ lines: promptLines }],
+      palette: DEFAULT_PANEL_PALETTE,
+    });
+    while (workspace.length < height) workspace.push(createEmptyLine(width));
+    return workspace.slice(0, height);
+  }
+
 }

package/src/panels/diff-panel.ts

@@ -5,7 +5,7 @@
 import type { Line } from '../types/grid.ts';
 import { createStyledCell, createEmptyLine } from '../types/grid.ts';
 import { truncateDisplay, getDisplayWidth } from '../utils/terminal-width.ts';
-import { GitService } from '@pellux/goodvibes-sdk/platform/git';
+import { GitService, type StructuredDiff, type StructuredDiffFile } from '@pellux/goodvibes-sdk/platform/git';
 import { BasePanel } from './base-panel.ts';
 import { UI_TONES, DIFF_TONES } from '../renderer/ui-primitives.ts';
 import { logger } from '@pellux/goodvibes-sdk/platform/utils';
@@ -27,10 +27,10 @@ import {
 // that shares it, including the workspace-chrome aliases (info/dim/value/
 // empty) so the raw-hex count never grows. The title band itself is NOT
 // overridden — buildPanelWorkspace always falls back to the canonical
-// DEFAULT_PANEL_PALETTE.headerBg (WO-002: one title band everywhere).
+// DEFAULT_PANEL_PALETTE.headerBg (one title band everywhere).
 // ---------------------------------------------------------------------------
 
-// Hunk blue is the shared DIFF_TONES token (WO-204) — diff-view.ts (conversation)
+// Hunk blue is the shared DIFF_TONES token — diff-view.ts (conversation)
 // and git-panel.ts's inline diff converge onto this file's pre-existing value.
 const HUNK_BLUE: string = DIFF_TONES.hunk;
 // Context rows and line-number gutter use the shared theme's muted/dim
@@ -39,7 +39,7 @@ const CONTEXT_GRAY = UI_TONES.fg.muted;
 const FILENAME_WHITE = '#ffffff';
 // Add/del text colors are the shared DIFF_TONES tokens, whose values ARE this
 // panel's shipped colors — this panel is the reference diff look (diff-view
-// was unwired dead code until WO-204, so "majority of surfaces" was a mirage);
+// was unwired dead code until, so "majority of surfaces" was a mirage);
 // the conversation surface converges onto these, not the reverse.
 const ADD_GREEN: string = DIFF_TONES.add;
 const ADD_BG = '#001a0d';
@@ -199,6 +199,19 @@ function extractDiffFilePath(trimmed: string): string {
   return 'unknown';
 }
 
+/** Reconstruct one file's unified-diff text from a StructuredDiffFile (header + hunks, prefixes re-attached; no size cap). */
+function reconstructStructuredFile(file: StructuredDiffFile): string {
+  const out: string[] = [...file.headerLines];
+  for (const hunk of file.hunks) {
+    out.push(hunk.header);
+    for (const line of hunk.lines) {
+      const prefix = line.kind === 'add' ? '+' : line.kind === 'del' ? '-' : ' ';
+      out.push(`${prefix}${line.text}`);
+    }
+  }
+  return out.join('\n');
+}
+
 function splitIntoDiffEntries(raw: string): DiffEntry[] {
   const entries: DiffEntry[] = [];
   // Split on "diff --git" / "diff --cc" / "diff --combined" lines
@@ -306,6 +319,26 @@ export class DiffPanel extends BasePanel {
     this.markDirty();
   }
 
+  /**
+   * Ingest a StructuredDiff (GitService.diffStructured — the FULL, uncapped
+   * working-tree diff). Each file becomes one DiffEntry (unified text
+   * reconstructed) so the existing tabs/renderer work unchanged; the old
+   * 4,000-char slice is gone and a diff of any length renders complete.
+   */
+  loadStructuredDiff(diff: StructuredDiff): void {
+    this.entries = diff.files.map((file) => {
+      const raw = reconstructStructuredFile(file);
+      return {
+        filePath: file.newPath ?? file.oldPath ?? 'unknown',
+        raw,
+        lines: parseDiff(raw),
+      };
+    });
+    this.selectedFile = 0;
+    this.scrollOffset = 0;
+    this.markDirty();
+  }
+
   /**
    * Defensive gate mirroring GitPanel's own notGitRepo messaging — short-
    * circuits before any of this panel's self-load git spawns run, instead of
@@ -521,7 +554,7 @@ export class DiffPanel extends BasePanel {
     }
   }
 
-  // W6.1 (the purge): 'o' used to open the currently selected file in the
+  // (the purge): 'o' used to open the currently selected file in the
   // preview panel via a staged pendingOpenPreview flag + a
   // handlePanelIntegrationAction cross-panel hook (the same bridge
   // FileExplorerPanel used). 'preview' is DELETE-disposition with no
@@ -597,7 +630,7 @@ export class DiffPanel extends BasePanel {
             width,
             ' No diff to display.',
             'Load a git diff or select a changed file to populate the workspace. w=working h=HEAD s=staged self-load right here.',
-            // WO-160: corrected — bare /diff loads files changed *this
+            // corrected — bare /diff loads files changed *this
             // session* (falling back to the full HEAD diff when none are
             // tracked yet), not the working-tree diff the w key already
             // loads in-panel; the old summary text described the wrong

package/src/panels/eval-registry.ts

@@ -1,12 +1,12 @@
 /**
  * EvalRegistry — holds the latest evaluation harness run state.
  *
- * W6.1 (the purge): extracted out of eval-panel.ts. 'eval' was
+ * (the purge): extracted out of eval-panel.ts. 'eval' was
  * DELETE-disposition (no surviving human surface — the evaluation harness
  * is driven and reviewed via the `/eval` CLI command, input/commands/eval.ts)
  * but this registry is still a live consumer of `/eval run` output, so it
  * survives as a standalone read-model rather than being deleted with the
- * view. See .goodvibes/audit/2026-07-04-wave6-briefs.json (W6.1).
+ * view. See .goodvibes/audit/2026-07-04-wave6-briefs.json.
  */
 
 import type { EvalSuiteResult, EvalGateResult } from '@/runtime/index.ts';