chore(broker): comment migrate skip flag as break-glass only
Some checks failed
CI / Lint (push) Has been cancelled
CI / Typecheck (push) Has been cancelled
CI / Broker tests (Postgres) (push) Has been cancelled
CI / Docker build (linux/amd64) (push) Has been cancelled

Now that the filename-tracked runner is in place and prod is bootstrapped,
BROKER_SKIP_MIGRATE=1 is no longer needed. Removed from Coolify env;
the comment is updated to reflect that the flag is a break-glass for
ops, not the steady-state.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
Alejandro Gutiérrez
2026-05-02 16:45:36 +01:00
parent c2cd67a885
commit 6de5e275fa

View File

@@ -4875,10 +4875,10 @@ async function recoverScheduledMessages(): Promise<void> {
}
async function main(): Promise<void> {
// Auto-migrate: skip when BROKER_SKIP_MIGRATE=1 (e.g. when the prod
// DB was already manually migrated and the drizzle tracking table is
// out of sync). Once drizzle's __drizzle_migrations is caught up,
// remove the flag.
// Auto-migrate via the filename-tracked runner (mesh.__cmh_migrations).
// The legacy BROKER_SKIP_MIGRATE=1 escape hatch is preserved as a
// break-glass for ops; under normal operation the runner is fast (<1s
// when up-to-date) and idempotent so the flag should stay unset.
if (process.env.BROKER_SKIP_MIGRATE !== "1") {
const { runMigrationsOnStartup } = await import("./migrate");
await runMigrationsOnStartup();