Skip to content

Remove ext supabase command and Supabase launch support#4997

Merged
stahnma merged 1 commit into
masterfrom
remove-ext-supabase
Jul 13, 2026
Merged

Remove ext supabase command and Supabase launch support#4997
stahnma merged 1 commit into
masterfrom
remove-ext-supabase

Conversation

@stahnma

@stahnma stahnma commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Summary

Supabase is no longer supported, so this removes the fly ext supabase command and all Supabase references from the codebase.

  • Delete the internal/command/extensions/supabase/ package (supabase, list, status, dashboard, destroy) and unregister it in extensions.go.
  • Remove Supabase from the launch planner:
    • plan/postgres.go — drop the SupabasePostgres field, its Provider() branch, and the SupabasePostgresPlan type + methods.
    • describe_plan.go — remove the plan case and describeSupabasePostgresPlan.
    • launch_databases.go — remove the now-defunct "Supabase Postgres is no longer supported" notice.
    • cmd.go — drop SupabasePostgres from the HasPostgres check.
    • state.govalidateExtensions only did Supabase plan-limit validation, so it's now a no-op stub; dropped the newly-unused errors, prompt, extensions_core, and flag imports.

Notes

  • The generated gql AddOnType enum still carries the server-side supabase value and is left untouched (it's regenerated from the server schema; nothing in flyctl references it anymore).
  • Behavioral change: if a launch manifest ever contains a supabase_postgres block, 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 ./... — passes
  • go vet ./internal/command/launch/... ./internal/command/extensions/... — clean

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.
@stahnma stahnma requested a review from dangra July 10, 2026 17:11

@dangra dangra left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yay!

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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/supabase command 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 validateExtensions as 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.

@stahnma stahnma merged commit ed8991a into master Jul 13, 2026
61 of 64 checks passed
@stahnma stahnma deleted the remove-ext-supabase branch July 13, 2026 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants