[RSO]

Final PR of the Cost Insights shutdown, after #4993 (UI and API) and #4996 (all reads and writes). The twelve tables have been unused since #4996 merged.

The migration

0204_drop_cost_insight_tables.sql — twelve DROP TABLE ... CASCADE statements and nothing else. Generated by pnpm drizzle generate; no hand-editing of DDL, snapshots, or the journal.

Deploy ordering is a non-issue. I verified against 0203_snapshot.json that no non-Cost-Insights table has a foreign key to any of these tables, and no view references them. All 27 foreign keys involved originate from the dropped tables. Combined with #4996 removing every read and write, the migration is safe to run before, during, or after the code deploy.

CASCADE therefore only tears down the feature's own intra-table foreign keys, indexes, and constraints — it cannot reach outside the set.

One incidental improvement: cost_insight_owner_hour_driver_buckets.actor_user_id was an ON DELETE NO ACTION foreign key to kilocode_users, so it could previously block hard user deletion. That blocker is gone.

Schema and code removed

• packages/db/src/schema.ts (−921): 12 table definitions, 16 inferred row types, the CostInsightEventSnapshot Zod schemas, the schema-types import, and the SCHEMA_CHECK_ENUMS registry entries.
• packages/db/src/schema-types.ts (−69): the 7 Cost Insights enums. I confirmed all 7 have zero consumers outside packages/db.
• packages/db/src/schema.test.ts (−19): the expected enum values. This must land with the registry change — removing one without the other fails the registry test with Missing enums.
• packages/db/src/cost-insight-event-snapshot.test.ts: deleted; its only subject was CostInsightEventSnapshotSchema.
• .specs/cost-insights.md: deleted. It was fully orphaned — chore(cost-insights): stop all writes to the cost_insight_* tables #4996 removed the last CONTEXT.md reference, and contrary to what I assumed, it was never listed in the specs skill index, so no index entry needed updating.
• CONTEXT.md: the "scheduled to be dropped" language is now inaccurate.

Deliberately kept

The discontinued-notice routes from #4993, so old bookmarks and previously sent alert emails still land on an explanation.

Two untracked production indexes. I told you in #4996 that these would be dropped in this migration. That was wrong, and worth correcting:

Index	Table
IDX_coding_plan_terms_credit_transaction	coding_plan_terms(credit_transaction_id)
idx_microdollar_usage_org_created_at	microdollar_usage(organization_id, created_at) WHERE organization_id IS NOT NULL

Neither is known to Drizzle — absent from schema.ts, all 204 prior migrations, and every snapshot. They exist only where the now-deleted maintenance script was run, so drizzle generate will not emit drops for them. DROP INDEX CONCURRENTLY cannot run inside Drizzle's transactional migration, and a plain DROP INDEX would take an ACCESS EXCLUSIVE lock on microdollar_usage, one of the hottest tables in the system. The second index also plausibly still helps organization usage queries over a time range. Left for a separate operational decision informed by real production index-usage stats rather than a guess from code.

Verification

• pnpm drizzle generate produced only the intended drops; pnpm drizzle check reports Everything's fine.
• pnpm drizzle migrate applied locally, then confirmed via psql: 0 cost_insight% tables remain, 214 public tables remain, and kilocode_users, organizations, microdollar_usage, coding_plan_terms, credit_transactions are all intact.
• pnpm drizzle:verify-bootstrap passes, proving the full 0000 → 0204 chain replays on an empty database (0179 and 0183 create these tables, 0204 drops them).
• scripts/typecheck-all.sh and scripts/lint-all.sh: 0 errors across all workspaces. Repo-wide pnpm format:check clean.
• schema.test.ts + schema-types.test.ts: 42 tests pass, covering both the schema-versus-snapshot parity check and the enum registry.
• Full apps/web suite: 700 suites, 8,943 tests pass. services/kiloclaw-billing: 184 tests pass.

Pre-existing gap found while verifying

packages/db tests do not run anywhere. apps/web/jest.config.ts:38 lists <rootDir>/../../packages/db/src/**/*.test.ts in testMatch, but roots is unset so it defaults to apps/web and Jest never crawls that directory. packages/db/package.json has only typecheck and lint, and no other Jest config covers it.

So schema.test.ts — which exists specifically to fail with "Schema is out of date! Run 'pnpm drizzle generate'" — is never executed by CI. I ran it explicitly via --roots to validate this change. Not fixed here since it is unrelated to this PR and would newly enforce assertions across the whole schema, but it should be fixed separately.

Also note packages/db/src/client.test.ts fails under that ad-hoc invocation. I verified by stashing this branch that it fails identically on clean origin/main, so it is pre-existing and unrelated.

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Incremental review of the follow-up CONTEXT.md cleanup (removal of the Cost Insights domain row and the "fully removed" bullet, applying the author's own two line-deletion suggestions): both deletions match the PR's stated scope, the two remaining Cost Insights mentions are the deliberately kept behavioral notes about spend recording and the discontinued-notice routes, and the table and list structure remain intact.

Files Reviewed (1 file)

• CONTEXT.md

Previous Review Summary (commit d8363fd)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit d8363fd)

Status: No Issues Found | Recommendation: Merge

Executive Summary

Verified the Cost Insights table shutdown end-to-end: the migration drops exactly the 12 cost_insight_* tables present in the 0203 snapshot, no external table or view references them (all FKs to the set are intra-set, so CASCADE is self-contained), the snapshot chain is intact (0204 prevId matches 0203 id), and zero code references to the removed tables, enums, or types remain anywhere in the repo, including the deliberately kept discontinued-notice routes.

Files Reviewed (9 files)

• .specs/cost-insights.md (deleted)
• CONTEXT.md
• packages/db/src/cost-insight-event-snapshot.test.ts (deleted)
• packages/db/src/migrations/0204_drop_cost_insight_tables.sql
• packages/db/src/migrations/meta/0204_snapshot.json (generated)
• packages/db/src/migrations/meta/_journal.json (generated)
• packages/db/src/schema-types.ts
• packages/db/src/schema.test.ts
• packages/db/src/schema.ts

---

_{Reviewed by kimi-k3 · Input: 53.4K · Output: 4K · Cached: 185.3K}

_{Review guidance: REVIEW.md from base branch main}