Skip to content

feat: add token counts and cost to audit log, report usage on all Cla…#106

Open
jee7s wants to merge 2 commits into
mainfrom
feat/audit-token-cost
Open

feat: add token counts and cost to audit log, report usage on all Cla…#106
jee7s wants to merge 2 commits into
mainfrom
feat/audit-token-cost

Conversation

@jee7s

@jee7s jee7s commented Apr 14, 2026

Copy link
Copy Markdown
Collaborator

…ude actions

  • Add input_tokens, output_tokens, cache_read_tokens, cache_creation_tokens, cost_usd columns to audit_log in Postgres schema, SQLite schema, and migration
  • Update AuditRecord/NewAuditRecord types and all store write/map methods
  • Change runAgent return type to AgentResult struct capturing SDK usage fields
  • Pass token/cost data through processEvent to auditLogger.log()
  • Add "Report token usage" post-step to all 5 claude.yml jobs, parsing the action execution_file and commenting on the relevant issue or PR

…ude actions

- Add input_tokens, output_tokens, cache_read_tokens, cache_creation_tokens,
  cost_usd columns to audit_log in Postgres schema, SQLite schema, and migration
- Update AuditRecord/NewAuditRecord types and all store write/map methods
- Change runAgent return type to AgentResult struct capturing SDK usage fields
- Pass token/cost data through processEvent to auditLogger.log()
- Add "Report token usage" post-step to all 5 claude.yml jobs, parsing the
  action execution_file and commenting on the relevant issue or PR

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@jee7s jee7s requested review from dkaygithub and thpr June 10, 2026 03:27
- Register 0001_audit_token_cost.sql in the drizzle journal (with
  statement breakpoints) — unjournaled migrations never run, which would
  have broken every audit write after deploy.
- Read the SDK result usage with its real snake_case keys (input_tokens,
  cache_read_input_tokens, ...); the camelCase reads recorded 0 for every
  token count, and the test mocks were reinforcing the bug.
- Extract the 5x-duplicated workflow report block into
  scripts/report-token-usage.sh and read total_cost_usd (with cost_usd
  fallback) — the old .cost_usd extraction always reported $0.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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