Skip to content

T-11.3: Add spend data layer - #185

Merged
Mkohler4 merged 1 commit into
mainfrom
codex/t-11-3-spend-data-layer
Jun 17, 2026
Merged

T-11.3: Add spend data layer#185
Mkohler4 merged 1 commit into
mainfrom
codex/t-11-3-spend-data-layer

Conversation

@Mkohler4

Copy link
Copy Markdown
Owner

Summary

  • Adds explicit-column Store read APIs for llm_runs, including list, source-filtered list, and provider/model aggregation.
  • Aggregates token totals with NULL token columns treated as 0 while exposing hasUnknownTokens.
  • Adds a bounded SpendModel that attaches T-11.2 CostEstimate rows, totals known-price USD only, and carries the price table stamp.
  • Adds in-memory Store and SpendModel tests for grouping, NULL token handling, unknown-price totals, empty model state, and bounded rows.

Guardrails

  • No UI changes.
  • No schema migration and no Store/Schema.swift changes.
  • No engine/ or mcp/ changes.
  • Store read paths do not select cost_usd; app-side recordLLMRun leaves cost_usd untouched.

Verification

  • swift build
  • swift test (275 tests, 1 skipped, 0 failures)

@netlify

netlify Bot commented Jun 17, 2026

Copy link
Copy Markdown

Deploy Preview for velvety-moxie-c6d22b canceled.

Name Link
🔨 Latest commit 4e57d86
🔍 Latest deploy log https://app.netlify.com/projects/velvety-moxie-c6d22b/deploys/6a330448782a4b0008a69897

@Mkohler4
Mkohler4 merged commit 333dcca into main Jun 17, 2026
13 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant