@supabase/pg-delta 1.0.0-alpha.2 → 1.0.0-alpha.21

package/dist/core/objects/procedure/procedure.diff.js

@@ -1,5 +1,5 @@
 import { diffObjects } from "../base.diff.js";
-import { diffPrivileges, filterPublicBuiltInDefaults, groupPrivilegesByGrantable, } from "../base.privilege-diff.js";
+import { diffPrivileges, emitObjectPrivilegeChanges, filterPublicBuiltInDefaults, } from "../base.privilege-diff.js";
 import { deepEqual, hasNonAlterableChanges } from "../utils.js";
 import { AlterProcedureChangeOwner, AlterProcedureSetConfig, AlterProcedureSetLeakproof, AlterProcedureSetParallel, AlterProcedureSetSecurity, AlterProcedureSetStrictness, AlterProcedureSetVolatility, } from "./changes/procedure.alter.js";
 import { CreateCommentOnProcedure, DropCommentOnProcedure, } from "./changes/procedure.comment.js";
@@ -17,8 +17,7 @@ import { GrantProcedurePrivileges, RevokeGrantOptionProcedurePrivileges, RevokeP
 export function diffProcedures(ctx, main, branch) {
     const { created, dropped, altered } = diffObjects(main, branch);
     const changes = [];
-    for (const procedureId of created) {
-        const proc = branch[procedureId];
+    const appendCreateProcedureChanges = (proc) => {
         changes.push(new CreateProcedure({ procedure: proc }));
         // OWNER: If the procedure should be owned by someone other than the current user,
         // emit ALTER FUNCTION/PROCEDURE ... OWNER TO after creation
@@ -37,6 +36,9 @@ export function diffProcedures(ctx, main, branch) {
         // We compare default privileges against desired privileges to generate REVOKE/GRANT statements
         // needed to reach the final desired state.
         const effectiveDefaults = ctx.defaultPrivilegeState.getEffectiveDefaults(ctx.currentUser, "procedure", proc.schema ?? "");
+        const creatorFilteredDefaults = proc.owner !== ctx.currentUser
+            ? effectiveDefaults.filter((p) => p.grantee !== ctx.currentUser)
+            : effectiveDefaults;
         // Filter out PUBLIC's built-in default EXECUTE privilege (PostgreSQL grants it automatically)
         // Reference: https://www.postgresql.org/docs/17/ddl-priv.html Table 5.2
         // This prevents generating unnecessary "GRANT EXECUTE TO PUBLIC" statements
@@ -44,44 +46,15 @@ export function diffProcedures(ctx, main, branch) {
         // Filter out owner privileges - owner always has ALL privileges implicitly
         // and shouldn't be compared. Note: we use the final owner (proc.owner), not the
         // current user, because ownership change happens before privilege diffing.
-        const privilegeResults = diffPrivileges(effectiveDefaults, desiredPrivileges, proc.owner, ctx.mainRoles);
-        // Generate grant changes
-        for (const [grantee, result] of privilegeResults) {
-            if (result.grants.length > 0) {
-                const grantGroups = groupPrivilegesByGrantable(result.grants);
-                for (const [grantable, list] of grantGroups) {
-                    void grantable;
-                    changes.push(new GrantProcedurePrivileges({
-                        procedure: proc,
-                        grantee,
-                        privileges: list,
-                        version: ctx.version,
-                    }));
-                }
-            }
-            // Generate revoke changes
-            if (result.revokes.length > 0) {
-                const revokeGroups = groupPrivilegesByGrantable(result.revokes);
-                for (const [grantable, list] of revokeGroups) {
-                    void grantable;
-                    changes.push(new RevokeProcedurePrivileges({
-                        procedure: proc,
-                        grantee,
-                        privileges: list,
-                        version: ctx.version,
-                    }));
-                }
-            }
-            // Generate revoke grant option changes
-            if (result.revokeGrantOption.length > 0) {
-                changes.push(new RevokeGrantOptionProcedurePrivileges({
-                    procedure: proc,
-                    grantee,
-                    privilegeNames: result.revokeGrantOption,
-                    version: ctx.version,
-                }));
-            }
-        }
+        const privilegeResults = diffPrivileges(filterPublicBuiltInDefaults("procedure", creatorFilteredDefaults), desiredPrivileges, proc.owner);
+        changes.push(...emitObjectPrivilegeChanges(privilegeResults, proc, proc, "procedure", {
+            Grant: GrantProcedurePrivileges,
+            Revoke: RevokeProcedurePrivileges,
+            RevokeGrantOption: RevokeGrantOptionProcedurePrivileges,
+        }, ctx.version));
+    };
+    for (const procedureId of created) {
+        appendCreateProcedureChanges(branch[procedureId]);
     }
     for (const procedureId of dropped) {
         changes.push(new DropProcedure({ procedure: main[procedureId] }));
@@ -89,22 +62,18 @@ export function diffProcedures(ctx, main, branch) {
     for (const procedureId of altered) {
         const mainProcedure = main[procedureId];
         const branchProcedure = branch[procedureId];
-        // Check if non-alterable properties have changed
-        // These require dropping and recreating the procedure
-        const NON_ALTERABLE_FIELDS = [
+        // Fields that are part of the function's identity/signature. PostgreSQL
+        // rejects `CREATE OR REPLACE FUNCTION` for any of these changes with
+        // errors such as:
+        //   - cannot change return type of existing function
+        //   - cannot change name of input parameter "..."
+        //   - cannot change whether a procedure has output parameters
+        //   - cannot remove parameter defaults from existing function
+        // These require `DROP FUNCTION` followed by `CREATE FUNCTION`.
+        const SIGNATURE_BREAKING_FIELDS = [
             "kind",
             "return_type",
             "return_type_schema",
-            "language",
-            // The following properties are alterable in SQL, but our generator may choose
-            // to replace on changes not covered by explicit ALTER actions. Keep them out here
-            // to allow ALTER for those we implement below.
-            // security_definer,
-            // volatility,
-            // parallel_safety,
-            // is_strict,
-            // leakproof,
-            // Returns-set is part of the signature and not alterable
             "returns_set",
             "argument_count",
             "argument_default_count",
@@ -113,21 +82,44 @@ export function diffProcedures(ctx, main, branch) {
             "all_argument_types",
             "argument_modes",
             "argument_defaults",
+        ];
+        // Fields where `CREATE OR REPLACE` is sufficient - body replacement only.
+        // Other fields (security_definer, volatility, parallel_safety, is_strict,
+        // leakproof, config) are alterable via dedicated ALTER actions below.
+        const OR_REPLACEABLE_NON_ALTERABLE_FIELDS = [
+            "language",
             "source_code",
             "binary_path",
             "sql_body",
-            // config is alterable via SET/RESET
         ];
-        const nonAlterablePropsChanged = hasNonAlterableChanges(mainProcedure, branchProcedure, NON_ALTERABLE_FIELDS, {
+        const signatureChanged = hasNonAlterableChanges(mainProcedure, branchProcedure, SIGNATURE_BREAKING_FIELDS, {
             argument_names: deepEqual,
             argument_types: deepEqual,
             all_argument_types: deepEqual,
             argument_modes: deepEqual,
-            config: deepEqual,
         });
-        if (nonAlterablePropsChanged) {
-            // Replace the entire procedure
+        const nonAlterablePropsChanged = signatureChanged ||
+            hasNonAlterableChanges(mainProcedure, branchProcedure, OR_REPLACEABLE_NON_ALTERABLE_FIELDS);
+        if (signatureChanged) {
+            // PostgreSQL cannot change an existing function's signature via
+            // `CREATE OR REPLACE`. Drop the old signature, then recreate.
+            // `expandReplaceDependencies` will cascade the replacement to dependent
+            // objects (views, triggers, column defaults) via pg_depend edges.
+            changes.push(new DropProcedure({ procedure: mainProcedure }));
+            appendCreateProcedureChanges(branchProcedure);
+        }
+        else if (nonAlterablePropsChanged) {
+            // Body-only non-alterable change - `CREATE OR REPLACE` preserves the
+            // function OID and keeps dependent objects attached.
             changes.push(new CreateProcedure({ procedure: branchProcedure, orReplace: true }));
+            if (mainProcedure.comment !== branchProcedure.comment) {
+                if (branchProcedure.comment === null) {
+                    changes.push(new DropCommentOnProcedure({ procedure: mainProcedure }));
+                }
+                else {
+                    changes.push(new CreateCommentOnProcedure({ procedure: branchProcedure }));
+                }
+            }
         }
         else {
             // Only alterable properties changed - check each one
@@ -238,44 +230,12 @@ export function diffProcedures(ctx, main, branch) {
             const branchPrivilegesFiltered = filterPublicBuiltInDefaults("procedure", branchProcedure.privileges);
             // Filter out owner privileges - owner always has ALL privileges implicitly
             // and shouldn't be compared. Use branch owner as the reference.
-            const privilegeResults = diffPrivileges(mainPrivilegesFiltered, branchPrivilegesFiltered, branchProcedure.owner, ctx.mainRoles);
-            for (const [grantee, result] of privilegeResults) {
-                // Generate grant changes
-                if (result.grants.length > 0) {
-                    const grantGroups = groupPrivilegesByGrantable(result.grants);
-                    for (const [grantable, list] of grantGroups) {
-                        void grantable;
-                        changes.push(new GrantProcedurePrivileges({
-                            procedure: branchProcedure,
-                            grantee,
-                            privileges: list,
-                            version: ctx.version,
-                        }));
-                    }
-                }
-                // Generate revoke changes
-                if (result.revokes.length > 0) {
-                    const revokeGroups = groupPrivilegesByGrantable(result.revokes);
-                    for (const [grantable, list] of revokeGroups) {
-                        void grantable;
-                        changes.push(new RevokeProcedurePrivileges({
-                            procedure: mainProcedure,
-                            grantee,
-                            privileges: list,
-                            version: ctx.version,
-                        }));
-                    }
-                }
-                // Generate revoke grant option changes
-                if (result.revokeGrantOption.length > 0) {
-                    changes.push(new RevokeGrantOptionProcedurePrivileges({
-                        procedure: mainProcedure,
-                        grantee,
-                        privilegeNames: result.revokeGrantOption,
-                        version: ctx.version,
-                    }));
-                }
-            }
+            const privilegeResults = diffPrivileges(mainPrivilegesFiltered, branchPrivilegesFiltered, branchProcedure.owner);
+            changes.push(...emitObjectPrivilegeChanges(privilegeResults, branchProcedure, mainProcedure, "procedure", {
+                Grant: GrantProcedurePrivileges,
+                Revoke: RevokeProcedurePrivileges,
+                RevokeGrantOption: RevokeGrantOptionProcedurePrivileges,
+            }, ctx.version));
         }
     }
     return changes;

package/dist/core/objects/procedure/procedure.model.d.ts

@@ -6,24 +6,24 @@ declare const procedurePropsSchema: z.ZodObject<{
     schema: z.ZodString;
     name: z.ZodString;
     kind: z.ZodEnum<{
-        f: "f";
-        w: "w";
         a: "a";
+        w: "w";
         p: "p";
+        f: "f";
     }>;
     return_type: z.ZodString;
     return_type_schema: z.ZodString;
     language: z.ZodString;
     security_definer: z.ZodBoolean;
     volatility: z.ZodEnum<{
-        s: "s";
         i: "i";
+        s: "s";
         v: "v";
     }>;
     parallel_safety: z.ZodEnum<{
-        r: "r";
         u: "u";
         s: "s";
+        r: "r";
     }>;
     execution_cost: z.ZodNumber;
     result_rows: z.ZodNumber;
@@ -36,8 +36,8 @@ declare const procedurePropsSchema: z.ZodObject<{
     argument_types: z.ZodNullable<z.ZodArray<z.ZodString>>;
     all_argument_types: z.ZodNullable<z.ZodArray<z.ZodString>>;
     argument_modes: z.ZodNullable<z.ZodArray<z.ZodEnum<{
-        o: "o";
         i: "i";
+        o: "o";
         b: "b";
         v: "v";
         t: "t";
@@ -96,13 +96,13 @@ export declare class Procedure extends BasePgModel {
         name: string;
     };
     get dataFields(): {
-        kind: "f" | "w" | "a" | "p";
+        kind: "a" | "w" | "p" | "f";
         return_type: string;
         return_type_schema: string;
         language: string;
         security_definer: boolean;
-        volatility: "s" | "i" | "v";
-        parallel_safety: "r" | "u" | "s";
+        volatility: "i" | "s" | "v";
+        parallel_safety: "u" | "s" | "r";
         is_strict: boolean;
         leakproof: boolean;
         returns_set: boolean;
@@ -111,7 +111,7 @@ export declare class Procedure extends BasePgModel {
         argument_names: string[] | null;
         argument_types: string[] | null;
         all_argument_types: string[] | null;
-        argument_modes: ("o" | "i" | "b" | "v" | "t")[] | null;
+        argument_modes: ("i" | "o" | "b" | "v" | "t")[] | null;
         argument_defaults: string | null;
         source_code: string | null;
         binary_path: string | null;

package/dist/core/objects/procedure/procedure.model.js

@@ -222,7 +222,7 @@ select
         )
         order by x.grantee, x.privilege_type
       )
-      from lateral aclexplode(p.proacl) as x(grantor, grantee, privilege_type, is_grantable)
+      from lateral aclexplode(COALESCE(p.proacl, acldefault('f', p.proowner))) as x(grantor, grantee, privilege_type, is_grantable)
     ), '[]'
   ) as privileges
 from

package/dist/core/objects/publication/changes/publication.alter.d.ts

@@ -1,3 +1,4 @@
+import type { SerializeOptions } from "../../../integrations/serialize/serialize.types.ts";
 import type { Publication, PublicationTableProps } from "../publication.model.ts";
 import { AlterPublicationChange } from "./publication.base.ts";
 export declare class AlterPublicationSetOptions extends AlterPublicationChange {
@@ -11,16 +12,7 @@ export declare class AlterPublicationSetOptions extends AlterPublicationChange {
         setPublishViaPartitionRoot: boolean;
     });
     get requires(): `publication:${string}`[];
-    serialize(): string;
-}
-export declare class AlterPublicationSetForAllTables extends AlterPublicationChange {
-    readonly publication: Publication;
-    readonly scope: "object";
-    constructor(props: {
-        publication: Publication;
-    });
-    get requires(): `publication:${string}`[];
-    serialize(): string;
+    serialize(_options?: SerializeOptions): string;
 }
 export declare class AlterPublicationSetList extends AlterPublicationChange {
     readonly publication: Publication;
@@ -29,7 +21,7 @@ export declare class AlterPublicationSetList extends AlterPublicationChange {
         publication: Publication;
     });
     get requires(): string[];
-    serialize(): string;
+    serialize(_options?: SerializeOptions): string;
 }
 export declare class AlterPublicationAddTables extends AlterPublicationChange {
     readonly publication: Publication;
@@ -40,18 +32,19 @@ export declare class AlterPublicationAddTables extends AlterPublicationChange {
         tables: PublicationTableProps[];
     });
     get requires(): string[];
-    serialize(): string;
+    serialize(_options?: SerializeOptions): string;
 }
 export declare class AlterPublicationDropTables extends AlterPublicationChange {
     readonly publication: Publication;
     readonly scope: "object";
-    private readonly tables;
+    readonly tables: PublicationTableProps[];
     constructor(props: {
         publication: Publication;
         tables: PublicationTableProps[];
     });
     get requires(): string[];
-    serialize(): string;
+    get drops(): `table:${string}.${string}`[];
+    serialize(_options?: SerializeOptions): string;
 }
 export declare class AlterPublicationAddSchemas extends AlterPublicationChange {
     readonly publication: Publication;
@@ -62,7 +55,7 @@ export declare class AlterPublicationAddSchemas extends AlterPublicationChange {
         schemas: string[];
     });
     get requires(): (`schema:${string}` | `publication:${string}`)[];
-    serialize(): string;
+    serialize(_options?: SerializeOptions): string;
 }
 export declare class AlterPublicationDropSchemas extends AlterPublicationChange {
     readonly publication: Publication;
@@ -73,7 +66,7 @@ export declare class AlterPublicationDropSchemas extends AlterPublicationChange
         schemas: string[];
     });
     get requires(): (`schema:${string}` | `publication:${string}`)[];
-    serialize(): string;
+    serialize(_options?: SerializeOptions): string;
 }
 export declare class AlterPublicationSetOwner extends AlterPublicationChange {
     readonly publication: Publication;
@@ -84,5 +77,5 @@ export declare class AlterPublicationSetOwner extends AlterPublicationChange {
         owner: string;
     });
     get requires(): (`role:${string}` | `publication:${string}`)[];
-    serialize(): string;
+    serialize(_options?: SerializeOptions): string;
 }

package/dist/core/objects/publication/changes/publication.alter.js

@@ -15,7 +15,7 @@ export class AlterPublicationSetOptions extends AlterPublicationChange {
     get requires() {
         return [this.publication.stableId];
     }
-    serialize() {
+    serialize(_options) {
         const assignments = [];
         if (this.setPublish) {
             const operations = getPublicationOperations(this.publication);
@@ -27,20 +27,6 @@ export class AlterPublicationSetOptions extends AlterPublicationChange {
         return `ALTER PUBLICATION ${this.publication.name} SET (${assignments.join(", ")})`;
     }
 }
-export class AlterPublicationSetForAllTables extends AlterPublicationChange {
-    publication;
-    scope = "object";
-    constructor(props) {
-        super();
-        this.publication = props.publication;
-    }
-    get requires() {
-        return [this.publication.stableId];
-    }
-    serialize() {
-        return `ALTER PUBLICATION ${this.publication.name} SET FOR ALL TABLES`;
-    }
-}
 export class AlterPublicationSetList extends AlterPublicationChange {
     publication;
     scope = "object";
@@ -64,7 +50,7 @@ export class AlterPublicationSetList extends AlterPublicationChange {
         }
         return Array.from(dependencies);
     }
-    serialize() {
+    serialize(_options) {
         const clauses = formatPublicationObjects(this.publication.tables, this.publication.schemas);
         return `ALTER PUBLICATION ${this.publication.name} SET ${clauses.join(", ")}`;
     }
@@ -91,7 +77,7 @@ export class AlterPublicationAddTables extends AlterPublicationChange {
         }
         return Array.from(dependencies);
     }
-    serialize() {
+    serialize(_options) {
         const clauses = this.tables.map((table) => formatPublicationTable(table));
         return `ALTER PUBLICATION ${this.publication.name} ADD ${clauses.join(", ")}`;
     }
@@ -113,7 +99,12 @@ export class AlterPublicationDropTables extends AlterPublicationChange {
         }
         return Array.from(dependencies);
     }
-    serialize() {
+    get drops() {
+        // Treat ALTER PUBLICATION ... DROP TABLE as a destructive change so it runs
+        // in the drop phase before DROP TABLE removes the referenced relation.
+        return this.tables.map((table) => stableId.table(table.schema, table.name));
+    }
+    serialize(_options) {
         const targets = this.tables.map((table) => `${table.schema}.${table.name}`);
         return `ALTER PUBLICATION ${this.publication.name} DROP TABLE ${targets.join(", ")}`;
     }
@@ -133,7 +124,7 @@ export class AlterPublicationAddSchemas extends AlterPublicationChange {
             ...this.schemas.map((schema) => stableId.schema(schema)),
         ];
     }
-    serialize() {
+    serialize(_options) {
         const clauses = this.schemas.map((schema) => `TABLES IN SCHEMA ${schema}`);
         return `ALTER PUBLICATION ${this.publication.name} ADD ${clauses.join(", ")}`;
     }
@@ -153,7 +144,7 @@ export class AlterPublicationDropSchemas extends AlterPublicationChange {
             ...this.schemas.map((schema) => stableId.schema(schema)),
         ];
     }
-    serialize() {
+    serialize(_options) {
         const clauses = this.schemas.map((schema) => `TABLES IN SCHEMA ${schema}`);
         return `ALTER PUBLICATION ${this.publication.name} DROP ${clauses.join(", ")}`;
     }
@@ -170,7 +161,7 @@ export class AlterPublicationSetOwner extends AlterPublicationChange {
     get requires() {
         return [this.publication.stableId, stableId.role(this.owner)];
     }
-    serialize() {
+    serialize(_options) {
         return `ALTER PUBLICATION ${this.publication.name} OWNER TO ${this.owner}`;
     }
 }

package/dist/core/objects/publication/changes/publication.comment.d.ts

@@ -1,3 +1,4 @@
+import type { SerializeOptions } from "../../../integrations/serialize/serialize.types.ts";
 import type { Publication } from "../publication.model.ts";
 import { CreatePublicationChange, DropPublicationChange } from "./publication.base.ts";
 export type CommentPublication = CreateCommentOnPublication | DropCommentOnPublication;
@@ -9,7 +10,7 @@ export declare class CreateCommentOnPublication extends CreatePublicationChange
     });
     get creates(): `comment:${string}`[];
     get requires(): `publication:${string}`[];
-    serialize(): string;
+    serialize(_options?: SerializeOptions): string;
 }
 export declare class DropCommentOnPublication extends DropPublicationChange {
     readonly publication: Publication;
@@ -19,5 +20,5 @@ export declare class DropCommentOnPublication extends DropPublicationChange {
     });
     get drops(): `comment:${string}`[];
     get requires(): (`comment:${string}` | `publication:${string}`)[];
-    serialize(): string;
+    serialize(_options?: SerializeOptions): string;
 }

package/dist/core/objects/publication/changes/publication.comment.js

@@ -14,7 +14,7 @@ export class CreateCommentOnPublication extends CreatePublicationChange {
     get requires() {
         return [this.publication.stableId];
     }
-    serialize() {
+    serialize(_options) {
         return [
             "COMMENT ON PUBLICATION",
             this.publication.name,
@@ -40,7 +40,7 @@ export class DropCommentOnPublication extends DropPublicationChange {
             this.publication.stableId,
         ];
     }
-    serialize() {
+    serialize(_options) {
         return `COMMENT ON PUBLICATION ${this.publication.name} IS NULL`;
     }
 }

package/dist/core/objects/publication/changes/publication.create.d.ts

@@ -1,3 +1,4 @@
+import type { SerializeOptions } from "../../../integrations/serialize/serialize.types.ts";
 import type { Publication } from "../publication.model.ts";
 import { CreatePublicationChange } from "./publication.base.ts";
 /**
@@ -13,5 +14,5 @@ export declare class CreatePublication extends CreatePublicationChange {
     });
     get creates(): `publication:${string}`[];
     get requires(): string[];
-    serialize(): string;
+    serialize(_options?: SerializeOptions): string;
 }

package/dist/core/objects/publication/changes/publication.create.js

@@ -34,7 +34,7 @@ export class CreatePublication extends CreatePublicationChange {
         }
         return Array.from(dependencies);
     }
-    serialize() {
+    serialize(_options) {
         const parts = ["CREATE PUBLICATION", this.publication.name];
         if (this.publication.all_tables) {
             parts.push("FOR ALL TABLES");

package/dist/core/objects/publication/changes/publication.drop.d.ts

@@ -1,3 +1,4 @@
+import type { SerializeOptions } from "../../../integrations/serialize/serialize.types.ts";
 import type { Publication } from "../publication.model.ts";
 import { DropPublicationChange } from "./publication.base.ts";
 /**
@@ -13,5 +14,5 @@ export declare class DropPublication extends DropPublicationChange {
     });
     get drops(): `publication:${string}`[];
     get requires(): `publication:${string}`[];
-    serialize(): string;
+    serialize(_options?: SerializeOptions): string;
 }

package/dist/core/objects/publication/changes/publication.drop.js

@@ -17,7 +17,7 @@ export class DropPublication extends DropPublicationChange {
     get requires() {
         return [this.publication.stableId];
     }
-    serialize() {
+    serialize(_options) {
         return `DROP PUBLICATION ${this.publication.name}`;
     }
 }

package/dist/core/objects/publication/changes/publication.types.d.ts

@@ -1,5 +1,6 @@
-import type { AlterPublicationAddSchemas, AlterPublicationAddTables, AlterPublicationDropSchemas, AlterPublicationDropTables, AlterPublicationSetForAllTables, AlterPublicationSetList, AlterPublicationSetOptions, AlterPublicationSetOwner } from "./publication.alter.ts";
+import type { AlterPublicationAddSchemas, AlterPublicationAddTables, AlterPublicationDropSchemas, AlterPublicationDropTables, AlterPublicationSetList, AlterPublicationSetOptions, AlterPublicationSetOwner } from "./publication.alter.ts";
 import type { CommentPublication } from "./publication.comment.ts";
 import type { CreatePublication } from "./publication.create.ts";
 import type { DropPublication } from "./publication.drop.ts";
-export type PublicationChange = AlterPublicationAddSchemas | AlterPublicationAddTables | AlterPublicationDropSchemas | AlterPublicationDropTables | AlterPublicationSetForAllTables | AlterPublicationSetList | AlterPublicationSetOptions | AlterPublicationSetOwner | CommentPublication | CreatePublication | DropPublication;
+/** Union of all publication-related change variants (`objectType: "publication"`). @category Change Types */
+export type PublicationChange = AlterPublicationAddSchemas | AlterPublicationAddTables | AlterPublicationDropSchemas | AlterPublicationDropTables | AlterPublicationSetList | AlterPublicationSetOptions | AlterPublicationSetOwner | CommentPublication | CreatePublication | DropPublication;

package/dist/core/objects/publication/publication.diff.d.ts

@@ -1,5 +1,4 @@
+import type { ObjectDiffContext } from "../diff-context.ts";
 import type { PublicationChange } from "./changes/publication.types.ts";
 import type { Publication } from "./publication.model.ts";
-export declare function diffPublications(ctx: {
-    currentUser: string;
-}, main: Record<string, Publication>, branch: Record<string, Publication>): PublicationChange[];
+export declare function diffPublications(ctx: Pick<ObjectDiffContext, "currentUser">, main: Record<string, Publication>, branch: Record<string, Publication>): PublicationChange[];

package/dist/core/objects/publication/publication.diff.js

@@ -1,6 +1,6 @@
 import { diffObjects } from "../base.diff.js";
 import { deepEqual } from "../utils.js";
-import { AlterPublicationAddSchemas, AlterPublicationAddTables, AlterPublicationDropSchemas, AlterPublicationDropTables, AlterPublicationSetForAllTables, AlterPublicationSetOptions, AlterPublicationSetOwner, } from "./changes/publication.alter.js";
+import { AlterPublicationAddSchemas, AlterPublicationAddTables, AlterPublicationDropSchemas, AlterPublicationDropTables, AlterPublicationSetOptions, AlterPublicationSetOwner, } from "./changes/publication.alter.js";
 import { CreateCommentOnPublication, DropCommentOnPublication, } from "./changes/publication.comment.js";
 import { CreatePublication } from "./changes/publication.create.js";
 import { DropPublication } from "./changes/publication.drop.js";
@@ -52,21 +52,16 @@ export function diffPublications(ctx, main, branch) {
         let handledObjectLists = false;
         if (mainPublication.all_tables !== branchPublication.all_tables) {
             handledObjectLists = true;
-            if (branchPublication.all_tables) {
-                changes.push(new AlterPublicationSetForAllTables({
+            // Changing the all_tables mode requires DROP + CREATE because
+            // ALTER PUBLICATION does not support SET ALL TABLES.
+            changes.push(new DropPublication({ publication: mainPublication }));
+            changes.push(new CreatePublication({ publication: branchPublication }));
+            if (branchPublication.comment !== null) {
+                changes.push(new CreateCommentOnPublication({
                     publication: branchPublication,
                 }));
             }
-            else {
-                changes.push(new DropPublication({ publication: mainPublication }));
-                changes.push(new CreatePublication({ publication: branchPublication }));
-                if (branchPublication.comment !== null) {
-                    changes.push(new CreateCommentOnPublication({
-                        publication: branchPublication,
-                    }));
-                }
-                continue;
-            }
+            continue;
         }
         if (!handledObjectLists && !branchPublication.all_tables) {
             const tableDiff = diffPublicationTables(mainPublication.tables, branchPublication.tables);

package/dist/core/objects/rls-policy/changes/rls-policy.alter.d.ts

@@ -1,3 +1,4 @@
+import type { SerializeOptions } from "../../../integrations/serialize/serialize.types.ts";
 import type { RlsPolicy } from "../rls-policy.model.ts";
 import { AlterRlsPolicyChange } from "./rls-policy.base.ts";
 /**
@@ -26,7 +27,7 @@ export declare class AlterRlsPolicySetRoles extends AlterRlsPolicyChange {
         roles: string[];
     });
     get requires(): `rlsPolicy:${string}`[];
-    serialize(): string;
+    serialize(_options?: SerializeOptions): string;
 }
 /**
  * ALTER POLICY ... USING (...)
@@ -40,7 +41,7 @@ export declare class AlterRlsPolicySetUsingExpression extends AlterRlsPolicyChan
         usingExpression: string | null;
     });
     get requires(): `rlsPolicy:${string}`[];
-    serialize(): string;
+    serialize(_options?: SerializeOptions): string;
 }
 /**
  * ALTER POLICY ... WITH CHECK (...)
@@ -54,7 +55,7 @@ export declare class AlterRlsPolicySetWithCheckExpression extends AlterRlsPolicy
         withCheckExpression: string | null;
     });
     get requires(): `rlsPolicy:${string}`[];
-    serialize(): string;
+    serialize(_options?: SerializeOptions): string;
 }
 /**
  * Replace an RLS policy by dropping and recreating it.