singularity/singularity-forge
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: guard formatCost against non-number cost values (#74)

Browse source 
Handle both plain number and { total: number } shapes for msg.usage.cost
in snapshotUnitMetrics, and coerce formatCost input to prevent crashes
when cost is null/undefined/NaN from corrupted ledger data.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Lex Christopherson 2026-03-11 18:26:02 -06:00
parent 7b049ea539
commit b10b78cb75
2 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
src/resources/extensions/gsd/metrics.ts
View file

@ -129,8 +129,9 @@ export function snapshotUnitMetrics(
tokens.cacheRead += msg.usage.cacheRead ?? 0;
tokens.cacheWrite += msg.usage.cacheWrite ?? 0;
tokens.total += msg.usage.totalTokens ?? 0;
if (msg.usage.cost) {
cost += msg.usage.cost.total ?? 0;
if (msg.usage.cost != null) {
const c = msg.usage.cost;
cost += typeof c === "number" ? c : (c.total ?? 0);
}
}
// Count tool calls in this message
@ -296,9 +297,10 @@ export function getProjectTotals(units: UnitMetrics[]): ProjectTotals {
// ─── Formatting helpers ───────────────────────────────────────────────────────
export function formatCost(cost: number): string {
if (cost < 0.01) return `$${cost.toFixed(4)}`;
if (cost < 1) return `$${cost.toFixed(3)}`;
return `$${cost.toFixed(2)}`;
const n = Number(cost) || 0;
if (n < 0.01) return `$${n.toFixed(4)}`;
if (n < 1) return `$${n.toFixed(3)}`;
return `$${n.toFixed(2)}`;
}
/**

2
src/resources/extensions/subagent/index.ts
View file

@ -53,7 +53,7 @@ function formatUsageStats(
if (usage.output) parts.push(`↓${formatTokens(usage.output)}`);
if (usage.cacheRead) parts.push(`R${formatTokens(usage.cacheRead)}`);
if (usage.cacheWrite) parts.push(`W${formatTokens(usage.cacheWrite)}`);
if (usage.cost) parts.push(`$${usage.cost.toFixed(4)}`);
if (usage.cost) parts.push(`$${(Number(usage.cost) || 0).toFixed(4)}`);
if (usage.contextTokens && usage.contextTokens > 0) {
parts.push(`ctx:${formatTokens(usage.contextTokens)}`);
}