Skip to content

Revert "fix(cost-insights): decouple usage rollups and gate plan suggestions"#4594

Merged
jeanduplessis merged 1 commit into
mainfrom
revert-4588-fix/usage-rollup-contention
Jul 16, 2026
Merged

Revert "fix(cost-insights): decouple usage rollups and gate plan suggestions"#4594
jeanduplessis merged 1 commit into
mainfrom
revert-4588-fix/usage-rollup-contention

Conversation

@jeanduplessis

Copy link
Copy Markdown
Contributor

Reverts #4588

@jeanduplessis
jeanduplessis merged commit 4567f02 into main Jul 16, 2026
9 checks passed
@jeanduplessis
jeanduplessis deleted the revert-4588-fix/usage-rollup-contention branch July 16, 2026 15:42
@kilo-code-bot

kilo-code-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Verified this is a byte-perfect revert of #4588 (confirmed the full repo tree at the revert commit is identical to the pre-#4588 baseline), so no new bugs, security issues, or migration inconsistencies are introduced.

Files Reviewed (16 files)
  • apps/web/src/app/api/cron/cost-insights-hourly/route.test.ts
  • apps/web/src/app/api/cron/cost-insights-hourly/route.ts
  • apps/web/src/lib/ai-gateway/processUsage.test.ts
  • apps/web/src/lib/ai-gateway/processUsage.ts
  • apps/web/src/lib/ai-gateway/usage-daily-rollup-repairs.test.ts (deleted)
  • apps/web/src/lib/ai-gateway/usage-daily-rollup-repairs.ts (deleted)
  • apps/web/src/lib/coding-plans/index.test.ts
  • apps/web/src/lib/coding-plans/pricing.ts
  • apps/web/src/lib/cost-insights/evaluation.ts
  • apps/web/src/lib/cost-insights/jobs.ts
  • apps/web/src/lib/cost-insights/suggestion-eligibility.test.ts (deleted)
  • apps/web/src/lib/cost-insights/suggestion-eligibility.ts (deleted)
  • dev/seed/cost-insights/spend-evidence.ts
  • packages/db/src/migrations/0188_unusual_titanium_man.sql (deleted)
  • packages/db/src/migrations/meta/_journal.json (generated, skipped per review rules)
  • packages/db/src/schema.ts

Reviewed by claude-sonnet-5 · Input: 50 · Output: 13.2K · Cached: 2M

Review guidance: REVIEW.md from base branch main

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.

2 participants