[Bug] supabaseAdmin typed as any allowing silent null crashes across all API routes

[akhilmodi29]

Bug Description

supabaseAdmin in src/lib/supabase.ts is typed as any despite
being explicitly set to null when environment variables are missing.
This means TypeScript provides zero protection — any API route calling
supabaseAdmin.from(...) without a null check will throw an unhandled
TypeError: Cannot read properties of null at runtime, crashing the
entire route handler.

Steps to Reproduce

1. Remove or leave empty NEXT_PUBLIC_SUPABASE_URL in .env.local
2. Hit any API route that uses supabaseAdmin directly
3. Observe: unhandled TypeError crashes the route instead of a
   clean error response

Expected Behavior

TypeScript should catch unguarded supabaseAdmin calls at compile
time. The type should reflect that it can be null so developers
are forced to handle that case.

Root Cause

In src/lib/supabase.ts:

export const supabaseAdmin: any = ...

Typing it as any disables all TypeScript checks. The correct type
is SupabaseClient | null which forces null checks at every call site.

Fix

Import SupabaseClient and use it as the type:

import { createClient, SupabaseClient } from "@supabase/supabase-js";

export const supabaseAdmin: SupabaseClient | null =
  supabaseUrl && serviceRoleKey && !supabaseUrl.includes("placeholder")
    ? createClient(supabaseUrl, serviceRoleKey)
    : null;

File Affected

src/lib/supabase.ts

Impact

Every API route in the app that uses supabaseAdmin is affected.

[akhilmodi29]

can i be assigned to this issue ?

[mittalsonal]

I’m taking a focused pass on this. Plan: replace the nullable �ny admin client with a typed guarded client path, make the missing-config failure explicit instead of a null property crash, and add a narrow regression test around the guard behavior.

[mittalsonal]

Implemented this in PR #1263. The fix replaces the nullable �ny admin client path with an explicit typed guard so missing Supabase config fails predictably instead of crashing on
ull.from(...), and it adds focused regression coverage for that guard behavior.

[akhilmodi29]

Please assign it to me i already worked on it and attached a pr

[github-actions]

✅ Closed as completed!

If DevTrack has been useful, a ⭐ star on the repo helps the project grow and reach more developers.