code: costs.ts is a god module — 1118 lines, 14 fetch endpoints, 13 interfaces

[kokevidaurre]

Problem

src/lib/costs.ts (1118 lines) mixes cost calculation, bridge API client, quota management, npm stats, Claude Code capacity detection, ROI metrics, and rate limiting into a single file. It exports 13 interfaces and 10+ functions spanning unrelated domains.

Measurements

• 1118 lines total
• 14 fetch/fetchWithTimeout calls to different API endpoints
• 13 exported interfaces (CostSummary, BridgeStats, QuotaInfo, RateLimits, Insights, NpmStats, ClaudeCodeCapacity, ROIMetrics, etc.)
• Legacy _MODEL_PRICING table duplicates providers.ts pricing

Location

src/lib/costs.ts

Suggested Fix

Split into focused modules:

• costs.ts — CostSummary, fetchCostSummary, calcCost, formatCostBar (~150 lines)
• bridge-client.ts — BridgeStats, fetchBridgeStats, fetchWithTimeout, bridge URL config
• quota.ts — QuotaInfo, RateLimits, fetchQuotaInfo, fetchRateLimits
• insights.ts — Insights, TaskMetrics, ToolMetrics, QualityMetrics, fetchInsights
• roi.ts — ROIMetrics, SquadCostProjection, calculateROIMetrics, calculateSquadCostProjections

Remove legacy _MODEL_PRICING (already marked as deprecated in favor of providers.ts).

[kokevidaurre]

Checked current state: cost.ts is now 369 lines (down from 1118 mentioned in this issue). May already be partially resolved by previous refactors. Skipping for now — complex enough to risk scope creep in a time-constrained session. Will re-evaluate in a future run.