Remove ext supabase command and Supabase launch support#4997
Merged
Conversation
Supabase is no longer supported. This removes the fly ext supabase command package and all Supabase references from the launch planner (the SupabasePostgres plan type, its describe/validate paths, and the now-defunct deprecation notice). The generated gql AddOnType enum still carries the server-side supabase value and is left untouched.
There was a problem hiding this comment.
Pull request overview
Removes all CLI and launch-planner support for Supabase now that Supabase is no longer supported, including deleting the fly ext supabase command tree and eliminating Supabase-specific launch planning/validation paths.
Changes:
- Deleted the
internal/command/extensions/supabasecommand package and unregistered it from the extensions root command. - Removed Supabase Postgres support from the launch plan model and its plan description output.
- Removed Supabase-specific launch-time messaging and validation (leaving
validateExtensionsas a no-op stub).
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| internal/command/launch/state.go | Removes Supabase plan-limit validation logic; validateExtensions now returns nil. |
| internal/command/launch/plan/postgres.go | Drops SupabasePostgres from PostgresPlan and removes SupabasePostgresPlan type/methods. |
| internal/command/launch/launch_databases.go | Removes the “Supabase Postgres is no longer supported” notice during database creation. |
| internal/command/launch/describe_plan.go | Removes Supabase plan description branch/function; unrecognized providers now fall back to <none>. |
| internal/command/launch/cmd.go | Updates HasPostgres to exclude removed Supabase Postgres plan field. |
| internal/command/extensions/supabase/supabase.go | Deleted: removes the fly ext supabase root command. |
| internal/command/extensions/supabase/status.go | Deleted: removes Supabase status subcommand. |
| internal/command/extensions/supabase/list.go | Deleted: removes Supabase list subcommand. |
| internal/command/extensions/supabase/destroy.go | Deleted: removes Supabase destroy subcommand. |
| internal/command/extensions/supabase/dashboard.go | Deleted: removes Supabase dashboard subcommand. |
| internal/command/extensions/extensions.go | Unregisters Supabase extension command and removes its import. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Supabase is no longer supported, so this removes the
fly ext supabasecommand and all Supabase references from the codebase.internal/command/extensions/supabase/package (supabase,list,status,dashboard,destroy) and unregister it inextensions.go.plan/postgres.go— drop theSupabasePostgresfield, itsProvider()branch, and theSupabasePostgresPlantype + methods.describe_plan.go— remove the plan case anddescribeSupabasePostgresPlan.launch_databases.go— remove the now-defunct "Supabase Postgres is no longer supported" notice.cmd.go— dropSupabasePostgresfrom theHasPostgrescheck.state.go—validateExtensionsonly did Supabase plan-limit validation, so it's now a no-op stub; dropped the newly-unusederrors,prompt,extensions_core, andflagimports.Notes
gqlAddOnTypeenum still carries the server-sidesupabasevalue and is left untouched (it's regenerated from the server schema; nothing in flyctl references it anymore).supabase_postgresblock, flyctl now silently ignores it (Go drops unknown JSON fields) instead of printing the deprecation message — consistent with "no longer supported."Test plan
go build ./...— passesgo vet ./internal/command/launch/... ./internal/command/extensions/...— clean