chore: libpg-query → sql-parser migration cleanup#170
Conversation
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
@lucassalatiel is attempting to deploy a commit to the Supabase Team on Vercel. A member of the Team first needs to authorize it. |
|
@lucassalatiel thanks for the PR! To be honest, I don't think
Can you try swapping in |
Summary
Swaps
libpg-queryforsql-parseracross the web app and updates related modules. Includes a smalllibpg-query-compat.tsshim so call sites that still expect the legacy API keep working during the transition.Changes
apps/web/package.json— replacelibpg-querydependency withsql-parserapps/web/lib/libpg-query-compat.ts— new compat shim mapping the old surface tosql-parserapps/web/lib/sql-util.ts— switch parser importapps/web/components/ide.tsx,executed-sql.tsx,deploy-info-dialog.tsx— update parser usageapps/web/app/api/integrations/[id]/details/route.ts,apps/web/app/api/oauth/supabase/callback/route.ts— update parser usagepackage-lock.json— regenerated after dependency swapWhy
libpg-queryships native bindings that are awkward to bundle in Next.js / edge runtimes.sql-parseris pure JS and avoids the native build step while keeping the parsing API we rely on.Test plan
pnpm --filter web buildsucceeds