feat(cli)!: remove db remote commit and the low-use inspect db aliases

[Coly010]

Summary

This removes four commands that have been deprecated for at least one stable cycle and have a direct replacement. They stay registered but are hidden from help. Running any of them now exits 1 and names the replacement. This uses the same removedCommand() tombstone as db remote changes and gen keys.

Command	Replacement
db remote commit	db pull
inspect db total-index-size	inspect db index-stats
inspect db total-table-sizes	inspect db table-stats
inspect db table-index-sizes	inspect db table-stats

• The tombstones still accept their old flags, so existing invocations reach the removal error instead of a parse error. The inspect tombstones open no DB connection.
• Each tombstone records a RemovedSurfaceError telemetry fingerprint (removed_command), so remaining usage stays visible.
• Their handlers, handler tests and per-command docs pages are deleted. SIDE_EFFECTS.md and the DOCS_EXCLUDED table are updated.
• The pg-delta config description no longer mentions db remote commit, and config.schema.json is regenerated.
• The higher-use inspect db aliases (table-sizes, index-sizes, cache-hit, …) are unchanged.

The matching "removed in v3" warning text for stable is in #7108 (develop).

Linked issue

Part of CLI-2595

Checklist

• The PR title follows Conventional Commits.
• Tests added or updated: hidden-flag.unit.test.ts runs all four tombstones and asserts the removal message and suggestion, and checks they're absent from inspect db --help.
• From the repository root, pnpm check:all passes.

🤖 Generated with Claude Code

BREAKING CHANGE: supabase db remote commit is removed; use supabase db pull. supabase inspect db total-index-size is removed; use supabase inspect db index-stats. supabase inspect db total-table-sizes and supabase inspect db table-index-sizes are removed; use supabase inspect db table-stats. Each removed command exits 1 and prints its replacement.

[Coly010]

/ai-review

[github-actions]

🤖 AI Review

Confirmed both Claude findings as non-blocking nits: an unused pull option and a stale command count. Codex reported no findings. Inspection of the checked-out code found no additional concrete issues. Tests were not run.

Findings

Severity	Location	Category	Sources	Claim
⚪ NIT	apps/cli/src/commands/db/pull/pull.handler.ts:50	dead-code	claude	The skipFinishedLine option is obsolete after removal of the db remote commit handler; no remaining caller sets it.
⚪ NIT	apps/cli/src/commands/inspect/db/inspect-db-command.ts:37	documentation	claude	The inspectDbCommandHandler JSDoc still says it is shared by all 25 command files, but only 22 use it after the three aliases become tombstones.

Findings outside the diff

• ⚪ NIT apps/cli/src/commands/db/pull/pull.handler.ts:50 — The skipFinishedLine option is obsolete after removal of the db remote commit handler; no remaining caller sets it.
• ⚪ NIT apps/cli/src/commands/inspect/db/inspect-db-command.ts:37 — The inspectDbCommandHandler JSDoc still says it is shared by all 25 command files, but only 22 use it after the three aliases become tombstones.

Stats

Claude findings: 2 · Codex findings: 0 · Confirmed: 2 · Refuted: 0 · Uncertain: 0

---

Models: claude-opus-5-5 + gpt-6.1-sol · Trigger: manual · Workflow run

This review runs once per PR. A maintainer can request another with a /ai-review comment.