@supabase/pg-delta 1.0.0-alpha.16 → 1.0.0-alpha.17

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

@@ -44,6 +44,11 @@ export function diffTriggers(main, branch, branchIndexableObjects) {
         if (mainTrigger.is_partition_clone || branchTrigger.is_partition_clone) {
             continue;
         }
+        // Note: column_numbers is excluded because it contains pg_trigger.tgattr
+        // attnums that differ between databases when physical column layouts
+        // diverge but logical (named) columns match. The definition field
+        // (pg_get_triggerdef) already captures the UPDATE OF column list by name,
+        // so we compare by definition instead.
         const NON_ALTERABLE_FIELDS = [
             "function_schema",
             "function_name",
@@ -53,14 +58,14 @@ export function diffTriggers(main, branch, branchIndexableObjects) {
             "deferrable",
             "initially_deferred",
             "argument_count",
-            "column_numbers",
             "arguments",
             "when_condition",
             "old_table",
             "new_table",
             "owner",
+            "definition", // Compare by definition instead of column_numbers (tgattr)
         ];
-        const shouldReplace = hasNonAlterableChanges(mainTrigger, branchTrigger, NON_ALTERABLE_FIELDS, { column_numbers: deepEqual, arguments: deepEqual });
+        const shouldReplace = hasNonAlterableChanges(mainTrigger, branchTrigger, NON_ALTERABLE_FIELDS, { arguments: deepEqual });
         if (shouldReplace) {
             const tableStableId = stableId.table(branchTrigger.schema, branchTrigger.table_name);
             changes.push(new ReplaceTrigger({

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

@@ -83,7 +83,6 @@ export declare class Trigger extends BasePgModel {
         deferrable: boolean;
         initially_deferred: boolean;
         argument_count: number;
-        column_numbers: number[] | null;
         arguments: string[];
         when_condition: string | null;
         old_table: string | null;
@@ -94,6 +93,7 @@ export declare class Trigger extends BasePgModel {
         parent_table_name: string | null;
         is_on_partitioned_table: boolean;
         owner: string;
+        definition: string;
         comment: string | null;
     };
 }

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

@@ -120,7 +120,10 @@ export class Trigger extends BasePgModel {
             deferrable: this.deferrable,
             initially_deferred: this.initially_deferred,
             argument_count: this.argument_count,
-            column_numbers: this.column_numbers,
+            // column_numbers excluded: contains pg_trigger.tgattr attnums that differ
+            // between databases when physical column layouts diverge but logical
+            // (named) columns match. The definition field (pg_get_triggerdef) captures
+            // the UPDATE OF column list by name, so we compare by definition instead.
             arguments: this.arguments,
             when_condition: this.when_condition,
             old_table: this.old_table,
@@ -131,6 +134,7 @@ export class Trigger extends BasePgModel {
             parent_table_name: this.parent_table_name,
             is_on_partitioned_table: this.is_on_partitioned_table,
             owner: this.owner,
+            definition: this.definition,
             comment: this.comment,
         };
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@supabase/pg-delta",
-  "version": "1.0.0-alpha.16",
+  "version": "1.0.0-alpha.17",
   "description": "PostgreSQL migrations made easy",
   "type": "module",
   "sideEffects": false,

package/src/core/objects/trigger/trigger.diff.ts

@@ -67,6 +67,11 @@ export function diffTriggers(
       continue;
     }
 
+    // Note: column_numbers is excluded because it contains pg_trigger.tgattr
+    // attnums that differ between databases when physical column layouts
+    // diverge but logical (named) columns match. The definition field
+    // (pg_get_triggerdef) already captures the UPDATE OF column list by name,
+    // so we compare by definition instead.
     const NON_ALTERABLE_FIELDS: Array<keyof Trigger> = [
       "function_schema",
       "function_name",
@@ -76,18 +81,18 @@ export function diffTriggers(
       "deferrable",
       "initially_deferred",
       "argument_count",
-      "column_numbers",
       "arguments",
       "when_condition",
       "old_table",
       "new_table",
       "owner",
+      "definition", // Compare by definition instead of column_numbers (tgattr)
     ];
     const shouldReplace = hasNonAlterableChanges(
       mainTrigger,
       branchTrigger,
       NON_ALTERABLE_FIELDS,
-      { column_numbers: deepEqual, arguments: deepEqual },
+      { arguments: deepEqual },
     );
     if (shouldReplace) {
       const tableStableId = stableId.table(

package/src/core/objects/trigger/trigger.model.ts

@@ -134,7 +134,10 @@ export class Trigger extends BasePgModel {
       deferrable: this.deferrable,
       initially_deferred: this.initially_deferred,
       argument_count: this.argument_count,
-      column_numbers: this.column_numbers,
+      // column_numbers excluded: contains pg_trigger.tgattr attnums that differ
+      // between databases when physical column layouts diverge but logical
+      // (named) columns match. The definition field (pg_get_triggerdef) captures
+      // the UPDATE OF column list by name, so we compare by definition instead.
       arguments: this.arguments,
       when_condition: this.when_condition,
       old_table: this.old_table,
@@ -145,6 +148,7 @@ export class Trigger extends BasePgModel {
       parent_table_name: this.parent_table_name,
       is_on_partitioned_table: this.is_on_partitioned_table,
       owner: this.owner,
+      definition: this.definition,
       comment: this.comment,
     };
   }