fix(db): give the session revoke migration its own version

[JOY (JOY)]

Problem

main is broken for every fresh install. #182 (20261016200000_crove_kernel_inbound_drop_count.sql) and #186 (20261016200000_crove_kernel_session_revoke_rank_revoked.sql) share a version. The runner keys crove_meta.schema_migrations by version, so applying migrations stops at the second file:

migration 20261016200000_crove_kernel_session_revoke_rank_revoked.sql failed:
duplicate key value violates unique constraint "schema_migrations_pkey"

This breaks docker compose up (the migrate service exits 1), the CI db job and the invariant suite on main at 10b21d7. I reproduced it locally with vitest run --project db.

Fix

• 20261016200000_crove_kernel_session_revoke_rank_revoked.sql → 20261016200100_..., content unchanged.
  • fix(auth): pick a session not already revoked to carry the global sign-out #186's file sorts after feat(inbound): show the owner how many messages to a retired address were dropped #182's, so no database can have applied it under 20261016200000.
  • A database that applied feat(inbound): show the owner how many messages to a retired address were dropped #182's file keeps that row, and gets fix(auth): pick a session not already revoked to carry the global sign-out #186's change under the new version on the next run.
• tools/test/migration-versions.test.ts: a unit test that fails when two files in supabase/migrations share a 14-digit version. I checked that it fails on main's layout and passes after the rename.

Tests

Local, on main at 10b21d7 plus this change, PostgreSQL 17:

• pnpm lint, pnpm typecheck and pnpm docs:check pass.
• Unit tests: 309 passed, 6 skipped. Database tests: 300 passed, 5 skipped. Both failed before this change.
• Invariant suite: 261/261.

Touches supabase/migrations/ (a protected path), so the invariant job from the base runs. Because the base itself cannot migrate, that job may fail until this merges. It is not the PR's own failure.

🤖 Generated with Claude Code

https://claude.ai/code/session_013LrvuThKGaPX1uDwUiwmGQ

---

Generated by Claude Code

Summary by CodeRabbit

• Tests
  • Added automated checks to catch missing or incorrectly formatted database migration versions, as well as duplicate versions that could disrupt fresh installations. The checks also help ensure new migrations use versions later than those already on the main branch. These safeguards help identify migration issues before release.

[claude]

⚠️ Code review skipped — your organization has no extra usage available to pay for this review.

If your organization's extra usage balance is empty, an organization admin can add extra usage credits at claude.ai/admin-settings/usage. If its monthly spend limit was reached, an admin can raise it on the same page. If neither applies, contact Anthropic support.

Once extra usage is available, someone with write access to this repository can comment @claude review on this pull request to trigger a review.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
crove	Ready	Preview	Oct 15, 2026 8:11pm UTC

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 31 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

• Configuration used: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: 097755d7-4f42-43a8-bd4e-4d5526de7958

📥 Commits

Reviewing files that changed from the base of the PR and between 6028295 and b9738f6.

📒 Files selected for processing: 1

• tools/test/migration-versions.test.ts

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: 117ddb9a-08e2-4397-b4fe-3cc94b182d35

📥 Commits

Reviewing files that changed from the base of the PR and between 10b21d7 and 6028295.

📒 Files selected for processing: 2

• supabase/migrations/20261016200100_crove_kernel_session_revoke_rank_revoked.sql
• tools/test/migration-versions.test.ts

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Walkthrough

Adds a Vitest test that checks SQL migration filenames for a 14-digit version prefix and unique versions. The test also requires the migration directory to contain at least one file.

Changes: Migration version validation

Layer / File(s)	Summary
Validate migration filenames tools/test/migration-versions.test.ts	The test checks that the migration directory is nonempty, each SQL filename starts with a 14-digit version, and no version is shared. Comments describe duplicate-version behavior and timestamp requirements.

Priority: ⬆️ High

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 60282

The migration version collision that blocked fresh installs is corrected, and CI runs a test to catch duplicate migration versions. No actionable merge risk remains in the reviewed changes.

Pre-merge checks: 5

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly and concisely describes the main change: assigning the session revoke migration a unique version.
Docstring Coverage		No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[gemini-code-assist]

Code Review

This pull request introduces a new test suite in tools/test/migration-versions.test.ts to verify that all Supabase migration files have unique 14-digit version prefixes. Feedback suggests enhancing this test to validate the entire filename against the strict naming convention pattern used by the migration runner, which would prevent runtime migration failures and allow for cleaner, safer TypeScript code.