Skip to content

Add OpenAI cost reconciliation - #35

Merged
Romonaga merged 1 commit into
mainfrom
feature/circuit-shelf-20
Jun 14, 2026
Merged

Romonaga merged 1 commit into
mainfrom
feature/circuit-shelf-20

Conversation

@Romonaga

Copy link
Copy Markdown
Owner

Change Type

  • Feature
  • Bug fix
  • Refactor
  • Documentation / workflow artifact
  • Repo organization / cleanup
  • Other: ___

Linked Workstream Change

  • Linked Verlyn change: circuit-shelf-20
  • No change applies - reason: ___

Verlyn PR Metadata

Verlyn PR creator: ___

Acceptance Criteria

  • AI assist events can store estimated and final reconciled cost state without losing existing estimated-cost budget behavior.
  • A reconciliation service allocates verified OpenAI organization cost buckets across recorded OpenAI assist events and ignores local Ollama events.
  • A backfill/admin command can fetch OpenAI organization costs for a date range and persist a reconciliation run.
  • AI usage reports and CSV exports expose final cost/reconciliation status when available.

Verification Runs Completed

  • Session baseline
  • Architecture check (if configured)
  • Workstream validation: python scripts/repo_helpers/workflow/validate.py <id> --strict
  • Other:
    • ___

Coverage Summary

  • 4 of 4 tracked work items are complete.

AI Assistance Disclosure

  • AI-assisted - tool(s) used: Codex
  • Human-authored only

Accountable human owner: hellweek

Seeded by Verlyn for Add OpenAI cost reconciliation: this tracked change is being managed through Verlyn's solo workflow path. Replace this note with the concrete AI-assisted tools and scope before merge if implementation, review, or packet authoring used AI, or replace it with 'Human-authored only.' when no AI assistance materially shaped the final change.

Independent Cross-Agent Review

  • Not required for this change - reason: ___
  • Completed - reviewer tool: Verlyn workflow gate | findings summary: No blocking findings recorded.
    • Unresolved findings:
      • ___

Rollback Plan

  • Revert the migration and service/API/report changes; existing estimated_cost fields remain the pre-change source of truth.

Handoff Status

  • All acceptance criteria satisfied
  • Verlyn-managed work items updated
  • Outstanding blockers or deferred items noted
  • Ready to merge

@Romonaga
Romonaga merged commit ba9df27 into main Jun 14, 2026
1 check passed
@Romonaga
Romonaga deleted the feature/circuit-shelf-20 branch June 14, 2026 02:37
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