fix: skip config diff to reduce output noise (#4036)

Author: sweatybridge

internal/link/link.go

@@ -19,17 +19,11 @@ import (
 	"github.com/supabase/cli/pkg/api"
 	"github.com/supabase/cli/pkg/cast"
 	cliConfig "github.com/supabase/cli/pkg/config"
-	"github.com/supabase/cli/pkg/diff"
 	"github.com/supabase/cli/pkg/migration"
 )
 
 func Run(ctx context.Context, projectRef string, fsys afero.Fs, options ...func(*pgx.ConnConfig)) error {
-	copy := utils.Config.Clone()
-	original, err := cliConfig.ToTomlBytes(copy)
-	if err != nil {
-		fmt.Fprintln(utils.GetDebugLogger(), err)
-	}
-
+	majorVersion := utils.Config.Db.MajorVersion
 	if err := checkRemoteProjectStatus(ctx, projectRef, fsys); err != nil {
 		return err
 	}
@@ -54,14 +48,12 @@ func Run(ctx context.Context, projectRef string, fsys afero.Fs, options ...func(
 	fmt.Fprintln(os.Stdout, "Finished "+utils.Aqua("supabase link")+".")
 
 	// 4. Suggest config update
-	updated, err := cliConfig.ToTomlBytes(utils.Config.Clone())
-	if err != nil {
-		fmt.Fprintln(utils.GetDebugLogger(), err)
-	}
-
-	if lineDiff := diff.Diff(utils.ConfigPath, original, projectRef, updated); len(lineDiff) > 0 {
-		fmt.Fprintln(os.Stderr, utils.Yellow("WARNING:"), "Local config differs from linked project. Try updating", utils.Bold(utils.ConfigPath))
-		fmt.Println(string(lineDiff))
+	if utils.Config.Db.MajorVersion != majorVersion {
+		fmt.Fprintln(os.Stderr, utils.Yellow("WARNING:"), "Local database version differs from the linked project.")
+		fmt.Fprintf(os.Stderr, `Update your %s to fix it:
+[db]
+major_version = %d
+`, utils.Bold(utils.ConfigPath), utils.Config.Db.MajorVersion)
 	}
 	return nil
 }

pkg/pgxv5/connect.go

@@ -24,7 +24,9 @@ func Connect(ctx context.Context, connString string, options ...func(*pgx.ConnCo
 		return nil, errors.Errorf("failed to parse connection string: %w", err)
 	}
 	config.OnNotice = func(pc *pgconn.PgConn, n *pgconn.Notice) {
-		fmt.Fprintf(os.Stderr, "%s (%s): %s\n", n.Severity, n.Code, n.Message)
+		if !shouldIgnore(n.Message) {
+			fmt.Fprintf(os.Stderr, "%s (%s): %s\n", n.Severity, n.Code, n.Message)
+		}
 	}
 	if strings.HasPrefix(config.User, CLI_LOGIN_ROLE) {
 		config.AfterConnect = func(ctx context.Context, pgconn *pgconn.PgConn) error {
@@ -42,3 +44,9 @@ func Connect(ctx context.Context, connString string, options ...func(*pgx.ConnCo
 	}
 	return conn, nil
 }
+
+func shouldIgnore(msg string) bool {
+	return strings.Contains(msg, `schema "supabase_migrations" already exists`) ||
+		strings.Contains(msg, `relation "schema_migrations" already exists`) ||
+		strings.Contains(msg, `relation "seed_files" already exists`)
+}