@supabase/pg-delta 1.0.0-alpha.4 → 1.0.0-alpha.5

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

@@ -14,10 +14,10 @@ declare const viewPropsSchema: z.ZodObject<{
     has_subclasses: z.ZodBoolean;
     is_populated: z.ZodBoolean;
     replica_identity: z.ZodEnum<{
-        f: "f";
         n: "n";
         i: "i";
         d: "d";
+        f: "f";
     }>;
     is_partition: z.ZodBoolean;
     options: z.ZodNullable<z.ZodArray<z.ZodString>>;
@@ -85,7 +85,7 @@ export declare class View extends BasePgModel implements TableLikeObject {
         has_triggers: boolean;
         has_subclasses: boolean;
         is_populated: boolean;
-        replica_identity: "f" | "n" | "i" | "d";
+        replica_identity: "n" | "i" | "d" | "f";
         is_partition: boolean;
         options: string[] | null;
         partition_bound: string | null;
@@ -123,7 +123,21 @@ export declare class View extends BasePgModel implements TableLikeObject {
         };
         data: {
             columns: {
-                [x: string]: unknown;
+                name: string;
+                data_type: string;
+                data_type_str: string;
+                is_custom_type: boolean;
+                custom_type_type: string | null;
+                custom_type_category: string | null;
+                custom_type_schema: string | null;
+                custom_type_name: string | null;
+                not_null: boolean;
+                is_identity: boolean;
+                is_identity_always: boolean;
+                is_generated: boolean;
+                collation: string | null;
+                default: string | null;
+                comment: string | null;
             }[];
             definition: string;
             row_security: boolean;
@@ -133,7 +147,7 @@ export declare class View extends BasePgModel implements TableLikeObject {
             has_triggers: boolean;
             has_subclasses: boolean;
             is_populated: boolean;
-            replica_identity: "f" | "n" | "i" | "d";
+            replica_identity: "n" | "i" | "d" | "f";
             is_partition: boolean;
             options: string[] | null;
             partition_bound: string | null;

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

@@ -1,6 +1,6 @@
 import { sql } from "@ts-safeql/sql-tag";
 import z from "zod";
-import { BasePgModel, columnPropsSchema, } from "../base.model.js";
+import { BasePgModel, columnPropsSchema, normalizeColumns, } from "../base.model.js";
 import { privilegePropsSchema, } from "../base.privilege-diff.js";
 import { ReplicaIdentitySchema } from "../table/table.model.js";
 const viewPropsSchema = z.object({
@@ -95,21 +95,11 @@ export class View extends BasePgModel {
         };
     }
     stableSnapshot() {
-        const normalizeColumns = () => [...this.columns]
-            .map((col) => {
-            const { position: _pos, ...rest } = col;
-            return rest;
-        })
-            .sort((a, b) => {
-            const nameA = a.name ?? "";
-            const nameB = b.name ?? "";
-            return nameA.localeCompare(nameB);
-        });
         return {
             identity: this.identityFields,
             data: {
                 ...this.dataFields,
-                columns: normalizeColumns(),
+                columns: normalizeColumns(this.columns),
             },
         };
     }
@@ -218,7 +208,7 @@ select
       from (
         -- one row for object ACL + one row per column ACL
         select null::name as attname, v.oid as relacl_oid, (
-          select c_rel.relacl from pg_class c_rel where c_rel.oid = v.oid
+          select COALESCE(c_rel.relacl, acldefault('r', c_rel.relowner)) from pg_class c_rel where c_rel.oid = v.oid
         ) as acl
         union all
         select a2.attname, v.oid as relacl_oid, a2.attacl

package/dist/core/plan/apply.js

@@ -1,12 +1,11 @@
 /**
  * Plan application - execute migration plans against target databases.
  */
-import { escapeIdentifier } from "pg";
 import { diffCatalogs } from "../catalog.diff.js";
 import { extractCatalog } from "../catalog.model.js";
 import { buildPlanScopeFingerprint, hashStableIds } from "../fingerprint.js";
 import { compileFilterDSL } from "../integrations/filter/dsl.js";
-import { createPool, endPool } from "../postgres-config.js";
+import { createManagedPool, endPool } from "../postgres-config.js";
 import { sortChanges } from "../sort/sort-changes.js";
 /**
  * Check if a statement is a session configuration statement (standalone SET statements).
@@ -30,39 +29,23 @@ export async function applyPlan(plan, source, target, options = {}) {
     let desiredPool;
     let shouldCloseCurrent = false;
     let shouldCloseDesired = false;
-    // Suppress expected shutdown errors from idle pool connections (57P01 = admin_shutdown)
-    const onError = (err) => {
-        if (err.code !== "57P01") {
-            console.error("Pool error:", err);
-        }
-    };
     if (typeof source === "string") {
-        currentPool = createPool(source, {
-            onError,
-            onConnect: async (client) => {
-                // Force fully qualified names in catalog queries for fingerprint verification
-                await client.query("SET search_path = ''");
-                if (plan.role) {
-                    await client.query(`SET ROLE ${escapeIdentifier(plan.role)}`);
-                }
-            },
+        const managed = await createManagedPool(source, {
+            role: plan.role,
+            label: "source",
         });
+        currentPool = managed.pool;
         shouldCloseCurrent = true;
     }
     else {
         currentPool = source;
     }
     if (typeof target === "string") {
-        desiredPool = createPool(target, {
-            onError,
-            onConnect: async (client) => {
-                // Force fully qualified names in catalog queries for fingerprint verification
-                await client.query("SET search_path = ''");
-                if (plan.role) {
-                    await client.query(`SET ROLE ${escapeIdentifier(plan.role)}`);
-                }
-            },
+        const managed = await createManagedPool(target, {
+            role: plan.role,
+            label: "target",
         });
+        desiredPool = managed.pool;
         shouldCloseDesired = true;
     }
     else {

package/dist/core/plan/create.d.ts

@@ -2,21 +2,34 @@
  * Plan creation - the main entry point for creating migration plans.
  */
 import type { Pool } from "pg";
+import { Catalog } from "../catalog.model.ts";
 import type { Change } from "../change.types.ts";
 import type { DiffContext } from "../context.ts";
 import type { CreatePlanOptions, Plan } from "./types.ts";
 /**
- * Create a migration plan by comparing two databases.
+ * Input for source/target: a postgres connection URL, an existing Pool, or
+ * an already-resolved Catalog (e.g. deserialized from a snapshot file).
+ */
+export type CatalogInput = string | Pool | Catalog;
+/**
+ * Create a migration plan by comparing two catalog states.
+ *
+ * Each input can be:
+ * - A postgres connection URL (string) -- a pool is created and catalog extracted
+ * - An existing pg Pool -- catalog is extracted directly
+ * - A Catalog instance -- used as-is (e.g. from a deserialized snapshot)
+ *
+ * When `source` is `null`, a minimal empty catalog (`createEmptyCatalog`) is
+ * used as the baseline. For a more accurate baseline, pass a Catalog
+ * deserialized from a snapshot of `template1` or another reference database.
  *
- * @param fromUrl - Source database connection URL (current state)
- * @param toUrl - Target database connection URL (desired state)
+ * @param source - Source catalog input (current state), or null for empty baseline
+ * @param target - Target catalog input (desired state)
  * @param options - Optional configuration
  * @returns A Plan if there are changes, null if databases are identical
  */
-type ConnectionInput = string | Pool;
-export declare function createPlan(source: ConnectionInput, target: ConnectionInput, options?: CreatePlanOptions): Promise<{
+export declare function createPlan(source: CatalogInput | null, target: CatalogInput, options?: CreatePlanOptions): Promise<{
     plan: Plan;
     sortedChanges: Change[];
     ctx: DiffContext;
 } | null>;
-export {};

package/dist/core/plan/create.js

@@ -1,196 +1,68 @@
 /**
  * Plan creation - the main entry point for creating migration plans.
  */
-import { readFile } from "node:fs/promises";
 import { escapeIdentifier } from "pg";
 import { diffCatalogs } from "../catalog.diff.js";
-import { extractCatalog } from "../catalog.model.js";
+import { Catalog, createEmptyCatalog, extractCatalog, } from "../catalog.model.js";
 import { buildPlanScopeFingerprint, hashStableIds } from "../fingerprint.js";
 import { compileFilterDSL, } from "../integrations/filter/dsl.js";
 import { compileSerializeDSL, } from "../integrations/serialize/dsl.js";
-import { createPool, endPool } from "../postgres-config.js";
+import { createManagedPool, endPool } from "../postgres-config.js";
 import { sortChanges } from "../sort/sort-changes.js";
 import { classifyChangesRisk } from "./risk.js";
 /**
- * Parse SSL configuration from a PostgreSQL connection URL.
- * Supports sslmode (require, verify-ca, verify-full, prefer, disable).
- * Certificates can be provided via:
- * - Query string parameters (file paths): sslrootcert, sslcert, sslkey (preferred)
- * - Environment variables (content): PGDELTA_SOURCE_SSLROOTCERT/SSLCERT/SSLKEY or PGDELTA_TARGET_SSLROOTCERT/SSLCERT/SSLKEY
- * Returns SSL options for the postgres.js library and a cleaned URL without SSL-related query parameters.
+ * Create a migration plan by comparing two catalog states.
+ *
+ * Each input can be:
+ * - A postgres connection URL (string) -- a pool is created and catalog extracted
+ * - An existing pg Pool -- catalog is extracted directly
+ * - A Catalog instance -- used as-is (e.g. from a deserialized snapshot)
+ *
+ * When `source` is `null`, a minimal empty catalog (`createEmptyCatalog`) is
+ * used as the baseline. For a more accurate baseline, pass a Catalog
+ * deserialized from a snapshot of `template1` or another reference database.
+ *
+ * @param source - Source catalog input (current state), or null for empty baseline
+ * @param target - Target catalog input (desired state)
+ * @param options - Optional configuration
+ * @returns A Plan if there are changes, null if databases are identical
  */
-async function parseSslConfig(url, connectionType) {
-    const urlObj = new URL(url);
-    const sslmode = urlObj.searchParams.get("sslmode");
-    const sslrootcert = urlObj.searchParams.get("sslrootcert");
-    const sslcert = urlObj.searchParams.get("sslcert");
-    const sslkey = urlObj.searchParams.get("sslkey");
-    // Remove SSL-related query parameters since we parse them ourselves
-    urlObj.searchParams.delete("sslmode");
-    urlObj.searchParams.delete("sslrootcert");
-    urlObj.searchParams.delete("sslcert");
-    urlObj.searchParams.delete("sslkey");
-    const cleanedUrl = urlObj.toString();
-    // Handle different SSL modes
-    if (sslmode === "disable") {
-        return { cleanedUrl, ssl: false };
-    }
-    if (sslmode === "require" ||
-        sslmode === "prefer" ||
-        sslmode === "verify-ca" ||
-        sslmode === "verify-full") {
-        // Helper function to get certificate value: query param (file path) takes precedence over env var (content)
-        const getCertValue = async (queryParam, envVarName) => {
-            // Prefer query parameter (file path)
-            if (queryParam) {
-                try {
-                    return await readFile(queryParam, "utf-8");
-                }
-                catch (error) {
-                    throw new Error(`Failed to read certificate file '${queryParam}': ${error instanceof Error ? error.message : String(error)}`);
-                }
-            }
-            // Fallback to environment variable (content)
-            const envValue = process.env[envVarName];
-            return envValue || undefined;
-        };
-        const hasExplicitVerification = sslmode === "verify-ca" || sslmode === "verify-full";
-        // Get CA certificate value.
-        // - verify-ca/verify-full: check query param first, then env var
-        // - require/prefer: only check query param (libpq backward compatibility
-        //   requires an explicit root CA *file*, not a global env var)
-        const caEnvVar = connectionType === "source"
-            ? "PGDELTA_SOURCE_SSLROOTCERT"
-            : "PGDELTA_TARGET_SSLROOTCERT";
-        let caValue;
-        if (sslrootcert) {
-            // Explicit file path in query param — always honour it
-            caValue = await getCertValue(sslrootcert, caEnvVar);
-        }
-        else if (hasExplicitVerification) {
-            // verify-ca / verify-full without file path — fall back to env var
-            caValue = await getCertValue(null, caEnvVar);
-        }
-        // require/prefer without sslrootcert: no CA cert, no verification
-        // Determine if we should verify the CA chain
-        //   From PostgreSQL docs: "if a root CA file exists, the behavior of sslmode=require
-        //   will be the same as that of verify-ca"
-        const hasLibpqCompatibility = (sslmode === "require" || sslmode === "prefer") && caValue !== undefined;
-        const shouldVerifyCa = hasExplicitVerification || hasLibpqCompatibility;
-        // Determine if we should verify hostname
-        // - verify-full: verify both CA and hostname
-        // - verify-ca: verify CA only (skip hostname)
-        // - require/prefer with CA (libpq compat): behaves like verify-ca (skip hostname)
-        const shouldVerifyHostname = sslmode === "verify-full";
-        const ssl = {
-            rejectUnauthorized: shouldVerifyCa,
-        };
-        // Add CA certificate if verifying
-        if (shouldVerifyCa && caValue) {
-            ssl.ca = caValue;
-        }
-        // For verify-ca and libpq compatibility mode: skip hostname verification
-        // This matches PostgreSQL semantics where verify-ca only checks the CA chain
-        if (shouldVerifyCa && !shouldVerifyHostname) {
-            ssl.checkServerIdentity = () => undefined;
-        }
-        // Get client certificate (optional, for mutual TLS)
-        const certEnvVar = connectionType === "source"
-            ? "PGDELTA_SOURCE_SSLCERT"
-            : "PGDELTA_TARGET_SSLCERT";
-        const certValue = await getCertValue(sslcert, certEnvVar);
-        if (certValue) {
-            ssl.cert = certValue;
-        }
-        // Get client key (optional, for mutual TLS, required if cert is provided)
-        const keyEnvVar = connectionType === "source"
-            ? "PGDELTA_SOURCE_SSLKEY"
-            : "PGDELTA_TARGET_SSLKEY";
-        const keyValue = await getCertValue(sslkey, keyEnvVar);
-        if (keyValue) {
-            ssl.key = keyValue;
-        }
-        // Warn if cert is provided without key (or vice versa)
-        if ((ssl.cert && !ssl.key) || (!ssl.cert && ssl.key)) {
-            throw new Error("Both client certificate and key must be provided together for mutual TLS");
-        }
-        return { ssl, cleanedUrl };
-    }
-    // No sslmode specified or invalid value - explicitly disable SSL
-    return { cleanedUrl, ssl: false };
-}
 export async function createPlan(source, target, options = {}) {
-    let sourcePool;
-    let targetPool;
-    let shouldCloseSource = false;
-    let shouldCloseTarget = false;
-    // Suppress expected shutdown errors from idle pool connections (57P01 = admin_shutdown)
-    const onError = (err) => {
-        if (err.code !== "57P01") {
-            console.error("Pool error:", err);
+    const resolvePool = async (input, label) => {
+        if (typeof input === "string") {
+            const managed = await createManagedPool(input, {
+                role: options.role,
+                label,
+            });
+            return { pool: managed.pool, shouldClose: true };
         }
+        return { pool: input, shouldClose: false };
     };
-    if (typeof source === "string") {
-        const sslConfig = await parseSslConfig(source, "source");
-        sourcePool = createPool(sslConfig.cleanedUrl, {
-            ...(sslConfig.ssl !== undefined ? { ssl: sslConfig.ssl } : {}),
-            onError,
-            onConnect: async (client) => {
-                // Force fully qualified names in catalog queries
-                await client.query("SET search_path = ''");
-                if (options.role) {
-                    await client.query(`SET ROLE ${escapeIdentifier(options.role)}`);
-                }
-            },
-        });
-        shouldCloseSource = true;
-    }
-    else {
-        sourcePool = source;
-    }
-    if (typeof target === "string") {
-        const sslConfig = await parseSslConfig(target, "target");
-        targetPool = createPool(sslConfig.cleanedUrl, {
-            ...(sslConfig.ssl !== undefined ? { ssl: sslConfig.ssl } : {}),
-            onError,
-            onConnect: async (client) => {
-                // Force fully qualified names in catalog queries
-                await client.query("SET search_path = ''");
-                if (options.role) {
-                    await client.query(`SET ROLE ${escapeIdentifier(options.role)}`);
-                }
-            },
-        });
-        shouldCloseTarget = true;
-    }
-    else {
-        targetPool = target;
-    }
-    const sourceExtraction = extractCatalog(sourcePool);
-    const targetExtraction = extractCatalog(targetPool);
+    /**
+     * Resolve a CatalogInput to a Catalog, tracking pools that need cleanup.
+     */
+    const resolveCatalog = async (input, label, pools) => {
+        if (input instanceof Catalog) {
+            return input;
+        }
+        const resolved = await resolvePool(input, label);
+        pools.push(resolved);
+        return extractCatalog(resolved.pool);
+    };
+    const pools = [];
     try {
-        const [fromCatalog, toCatalog] = await Promise.all([
-            sourceExtraction,
-            targetExtraction,
-        ]);
+        const toCatalog = await resolveCatalog(target, "target", pools);
+        const fromCatalog = source !== null
+            ? await resolveCatalog(source, "source", pools)
+            : await createEmptyCatalog(toCatalog.version, toCatalog.currentUser);
         return buildPlanForCatalogs(fromCatalog, toCatalog, options);
     }
-    catch (error) {
-        // When one extraction fails, the other may still have in-flight queries.
-        // Wait for both to settle before pool cleanup to prevent unhandled
-        // rejections from connections being terminated mid-flight.
-        await Promise.allSettled([sourceExtraction, targetExtraction]);
-        throw error;
-    }
     finally {
-        const closers = [];
-        if (shouldCloseSource)
-            closers.push(endPool(sourcePool));
-        if (shouldCloseTarget)
-            closers.push(endPool(targetPool));
-        if (closers.length) {
+        const closers = pools
+            .filter((p) => p.shouldClose)
+            .map((p) => endPool(p.pool));
+        if (closers.length)
             await Promise.all(closers);
-        }
     }
 }
 /**
@@ -199,6 +71,7 @@ export async function createPlan(source, target, options = {}) {
 function buildPlanForCatalogs(fromCatalog, toCatalog, options = {}) {
     const changes = diffCatalogs(fromCatalog, toCatalog, {
         role: options.role,
+        skipDefaultPrivilegeSubtraction: options.skipDefaultPrivilegeSubtraction,
     });
     const filterOption = options.filter;
     const serializeOption = options.serialize;
@@ -231,9 +104,18 @@ function buildPlanForCatalogs(fromCatalog, toCatalog, options = {}) {
     }
     // Use filter from final integration
     const filterFn = finalIntegration?.filter;
-    const filteredChanges = filterFn
+    let filteredChanges = filterFn
         ? changes.filter((change) => filterFn(change))
         : changes;
+    // Cascade dependency exclusions: when a change is excluded by the filter,
+    // also exclude changes that depend on it (via requires or pg_depend).
+    // DSL filters: cascade only if explicitly opted in (cascade: true). Function filters: cascade by default.
+    const shouldCascade = isFilterDSL
+        ? filterDSL?.cascade === true
+        : true;
+    if (filterFn && filteredChanges.length < changes.length && shouldCascade) {
+        filteredChanges = cascadeExclusions(filteredChanges, changes, toCatalog.depends);
+    }
     if (filteredChanges.length === 0) {
         return null;
     }
@@ -242,6 +124,84 @@ function buildPlanForCatalogs(fromCatalog, toCatalog, options = {}) {
     return { plan, sortedChanges, ctx };
 }
 // ============================================================================
+// Dependency Cascading
+// ============================================================================
+/**
+ * Cascade exclusions through dependency relationships.
+ *
+ * When a change is excluded by the filter, any change that depends on it
+ * (via explicit `requires` or via catalog `pg_depend`) should also be excluded.
+ * This runs as a fixpoint loop, bounded by the total number of changes to
+ * guarantee deterministic termination.
+ *
+ * @param filteredChanges - Changes that passed the initial filter
+ * @param allChanges - All changes before filtering
+ * @param catalogDepends - Dependency rows from the target catalog (pg_depend)
+ * @returns The filtered changes with cascading exclusions applied
+ */
+function cascadeExclusions(filteredChanges, allChanges, catalogDepends) {
+    // Collect stableIds created by initially-excluded changes
+    const filteredSet = new Set(filteredChanges);
+    const excludedIds = new Set();
+    for (const change of allChanges) {
+        if (!filteredSet.has(change)) {
+            for (const id of change.creates ?? []) {
+                excludedIds.add(id);
+            }
+        }
+    }
+    if (excludedIds.size === 0) {
+        return filteredChanges;
+    }
+    // Build reverse dependency map: referenced_stable_id -> Set(dependent_stable_ids)
+    const catalogDependents = new Map();
+    for (const dep of catalogDepends) {
+        const existing = catalogDependents.get(dep.referenced_stable_id);
+        if (existing) {
+            existing.add(dep.dependent_stable_id);
+        }
+        else {
+            catalogDependents.set(dep.referenced_stable_id, new Set([dep.dependent_stable_id]));
+        }
+    }
+    // Fixpoint loop: bounded by total changes to guarantee termination.
+    // Each iteration must remove at least one change, otherwise we break.
+    let result = filteredChanges;
+    for (let i = 0; i < allChanges.length; i++) {
+        const beforeLength = result.length;
+        result = result.filter((change) => {
+            // Check explicit requirements: does this change require an excluded id?
+            const requires = change.requires ?? [];
+            if (requires.some((dep) => excludedIds.has(dep))) {
+                for (const id of change.creates ?? []) {
+                    excludedIds.add(id);
+                }
+                return false;
+            }
+            // Check catalog dependencies: does anything this change creates
+            // depend on an excluded id via pg_depend?
+            const creates = change.creates ?? [];
+            for (const createdId of creates) {
+                for (const excludedId of excludedIds) {
+                    const dependents = catalogDependents.get(excludedId);
+                    if (dependents?.has(createdId)) {
+                        for (const id of creates) {
+                            excludedIds.add(id);
+                        }
+                        return false;
+                    }
+                }
+            }
+            return true;
+        });
+        // No changes removed this iteration — fixpoint reached
+        if (result.length === beforeLength) {
+            break;
+        }
+    }
+    return result;
+}
+// ============================================================================
 // Plan Building
 // ============================================================================
 /**

package/dist/core/plan/serialize.js

@@ -139,10 +139,22 @@ export function getParentInfo(change) {
             const parentType = change.index.table_relkind === "m" ? "materialized_view" : "table";
             return { type: parentType, name: change.index.table_name };
         }
-        case "trigger":
-            return { type: "table", name: change.trigger.table_name };
-        case "rule":
-            return { type: "table", name: change.rule.table_name };
+        case "trigger": {
+            const parentType = change.trigger.table_relkind === "v"
+                ? "view"
+                : change.trigger.table_relkind === "m"
+                    ? "materialized_view"
+                    : "table";
+            return { type: parentType, name: change.trigger.table_name };
+        }
+        case "rule": {
+            const parentType = change.rule.relation_kind === "v"
+                ? "view"
+                : change.rule.relation_kind === "m"
+                    ? "materialized_view"
+                    : "table";
+            return { type: parentType, name: change.rule.table_name };
+        }
         case "rls_policy":
             return { type: "table", name: change.policy.table_name };
         case "aggregate":

package/dist/core/plan/sql-format/fixtures.js

@@ -84,7 +84,7 @@ import { CreateProcedure } from "../../objects/procedure/changes/procedure.creat
 import { DropProcedure } from "../../objects/procedure/changes/procedure.drop.js";
 import { GrantProcedurePrivileges, RevokeGrantOptionProcedurePrivileges, RevokeProcedurePrivileges, } from "../../objects/procedure/changes/procedure.privilege.js";
 import { Procedure } from "../../objects/procedure/procedure.model.js";
-import { AlterPublicationAddSchemas, AlterPublicationAddTables, AlterPublicationDropSchemas, AlterPublicationDropTables, AlterPublicationSetForAllTables, AlterPublicationSetList, AlterPublicationSetOptions, AlterPublicationSetOwner, } from "../../objects/publication/changes/publication.alter.js";
+import { AlterPublicationAddSchemas, AlterPublicationAddTables, AlterPublicationDropSchemas, AlterPublicationDropTables, AlterPublicationSetList, AlterPublicationSetOptions, AlterPublicationSetOwner, } from "../../objects/publication/changes/publication.alter.js";
 import { CreateCommentOnPublication, DropCommentOnPublication, } from "../../objects/publication/changes/publication.comment.js";
 // ── Publication changes ─────────────────────────────────────────────────────
 import { CreatePublication } from "../../objects/publication/changes/publication.create.js";
@@ -769,6 +769,7 @@ const trigger = new Trigger({
     schema: "public",
     name: "trg_audit",
     table_name: "table_with_very_long_name_for_formatting_and_wrapping_test",
+    table_relkind: "r",
     function_schema: "public",
     function_name: "audit_trigger_fn",
     trigger_type: 7,
@@ -931,6 +932,7 @@ const role = new Role({
             objtype: "r",
             grantee: "app_reader",
             privileges: [{ privilege: "SELECT", grantable: false }],
+            is_implicit: false,
         },
     ],
 });
@@ -1572,10 +1574,6 @@ const changeCases = [
             setPublishViaPartitionRoot: true,
         }),
     },
-    {
-        label: "publication.alter.set_all_tables",
-        change: new AlterPublicationSetForAllTables({ publication }),
-    },
     {
         label: "publication.alter.set_list",
         change: new AlterPublicationSetList({ publication }),