fix: lazy-import db in test-data cleanup route to prevent build-time crash

[timothyfroehlich]

/api/test-data/cleanup imported db at module scope, causing next build to throw when POSTGRES_URL is unset. The dynamic = "force-dynamic" export added in a prior commit prevented static rendering but didn't stop the module from being loaded — src/server/db/index.ts throws during initialization regardless.

Changes

• Lazy db import — replaced top-level import { db } with const { db } = await import("~/server/db") inside POST(), placed after the NODE_ENV production guard. Mirrors the pattern in test-setup/route.ts.
• Import order fix — moved export const dynamic = "force-dynamic" below the import block to match sibling routes (client-logs, cron/cleanup-blobs) and avoid lint violations.

export async function POST(request: Request): Promise<Response> {
  if (process.env.NODE_ENV === "production") {
    return NextResponse.json({ error: "Forbidden" }, { status: 403 });
  }

  // Lazy import to avoid loading db module during build
  const { db } = await import("~/server/db");
  // ...
}

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
pin-point	Ready	Preview, Comment	Apr 8, 2026 3:25am

[supabase]

Updates to Preview Branch (worktree/transient-enchanting-giraffe) ↗︎

Deployments	Status	Updated
Database	✅	Wed, 08 Apr 2026 03:24:40 UTC
Services	✅	Wed, 08 Apr 2026 03:24:40 UTC
APIs	✅	Wed, 08 Apr 2026 03:24:40 UTC

Tasks are run on every commit but only new migration files are pushed.
Close and reopen this PR if you want to apply changes from existing seed or migration files.

Tasks	Status	Updated
Configurations	✅	Wed, 08 Apr 2026 03:24:41 UTC
Migrations	✅	Wed, 08 Apr 2026 03:24:41 UTC
Seeding	✅	Wed, 08 Apr 2026 03:24:41 UTC
Edge Functions	✅	Wed, 08 Apr 2026 03:24:41 UTC

---

View logs for this Workflow Run ↗︎.
Learn more about Supabase for Git ↗︎.

[Copilot]

Pull request overview

Ensures the /api/test-data/cleanup route is treated as dynamic so next build won’t eagerly evaluate server-only DB setup when environment variables aren’t present.

Changes:

• Adds export const dynamic = "force-dynamic" to the test-data cleanup route.

[Copilot]

export const dynamic = "force-dynamic" is currently placed between import statements, leaving import { log ... } (and the schema import) after a non-import statement. This diverges from the pattern used in other route handlers (e.g., src/app/api/client-logs/route.ts:1-6, src/app/api/cron/cleanup-blobs/route.ts:1-9) and can also trip common lint rules that require imports to come first. Move the dynamic export below the last import so the import block stays contiguous.

[Copilot]

If the goal is to prevent next build from failing when POSTGRES_URL is unset, note that this route still imports db at module scope (import { db } from "~/server/db";). src/server/db/index.ts:5-11 throws during module initialization when POSTGRES_URL is missing, so any build-time code path that ends up loading this route module will still crash regardless of the dynamic export. Consider mirroring src/app/api/test-setup/route.ts:12-15 by lazily importing db inside POST() (after the NODE_ENV guard) so the module can be safely parsed/loaded during build without requiring database configuration.

[timothyfroehlich]

@copilot apply changes based on the comments in this thread

[Copilot]

@copilot apply changes based on the comments in this thread

Applied both fixes in b963910:

1. Moved export const dynamic = "force-dynamic" below all imports so the import block is contiguous.
2. Removed the top-level import { db } and replaced it with a lazy const { db } = await import("~/server/db") inside POST() after the NODE_ENV guard — mirroring the pattern in test-setup/route.ts so the module can be safely loaded during build without POSTGRES_URL.