@supabase/pg-delta 1.0.0-alpha.22 → 1.0.0-alpha.23

package/src/core/objects/type/range/range.model.ts

@@ -6,6 +6,10 @@ import {
   type PrivilegeProps,
   privilegePropsSchema,
 } from "../../base.privilege-diff.ts";
+import {
+  type SecurityLabelProps,
+  securityLabelPropsSchema,
+} from "../../security-label.types.ts";
 
 const rangePropsSchema = z.object({
   schema: z.string(),
@@ -30,6 +34,7 @@ const rangePropsSchema = z.object({
   subtype_opclass_schema: z.string().nullable(),
   subtype_opclass_name: z.string().nullable(),
   privileges: z.array(privilegePropsSchema),
+  security_labels: z.array(securityLabelPropsSchema).default([]).optional(),
 });
 
 type RangePrivilegeProps = PrivilegeProps;
@@ -54,6 +59,7 @@ export class Range extends BasePgModel {
   public readonly subtype_opclass_schema: RangeProps["subtype_opclass_schema"];
   public readonly subtype_opclass_name: RangeProps["subtype_opclass_name"];
   public readonly privileges: RangePrivilegeProps[];
+  public readonly security_labels: SecurityLabelProps[];
 
   constructor(props: RangeProps) {
     super();
@@ -75,6 +81,7 @@ export class Range extends BasePgModel {
     this.subtype_opclass_schema = props.subtype_opclass_schema;
     this.subtype_opclass_name = props.subtype_opclass_name;
     this.privileges = props.privileges;
+    this.security_labels = props.security_labels ?? [];
   }
 
   get stableId(): `type:${string}` {
@@ -102,6 +109,7 @@ export class Range extends BasePgModel {
       subtype_opclass_name: this.subtype_opclass_name,
       comment: this.comment,
       privileges: this.privileges,
+      security_labels: this.security_labels,
     };
   }
 }
@@ -166,7 +174,20 @@ select
       )
       from lateral aclexplode(COALESCE(t.typacl, acldefault('T', t.typowner))) as x(grantor, grantee, privilege_type, is_grantable)
     ), '[]'
-  ) as privileges
+  ) as privileges,
+  coalesce(
+    (
+      select json_agg(
+        json_build_object('provider', sl.provider, 'label', sl.label)
+        order by sl.provider
+      )
+      from pg_catalog.pg_seclabel sl
+      where sl.objoid = t.oid
+        and sl.classoid = 'pg_type'::regclass
+        and sl.objsubid = 0
+    ),
+    '[]'::json
+  ) as security_labels
 from pg_catalog.pg_range r
 join pg_catalog.pg_type t on t.oid = r.rngtypid
 join pg_catalog.pg_type subt on subt.oid = r.rngsubtype

package/src/core/objects/utils.ts

@@ -77,6 +77,9 @@ export const stableId = {
   comment(objectStableId: string) {
     return `comment:${objectStableId}` as const;
   },
+  securityLabel(objectStableId: string, provider: string) {
+    return `securityLabel:${objectStableId}::provider:${provider}` as const;
+  },
   role(role: string) {
     return `role:${role}` as const;
   },

package/src/core/objects/view/changes/view.base.ts

@@ -3,7 +3,11 @@ import type { View } from "../view.model.ts";
 
 abstract class BaseViewChange extends BaseChange {
   abstract readonly view: View;
-  abstract readonly scope: "object" | "comment" | "privilege";
+  abstract readonly scope:
+    | "object"
+    | "comment"
+    | "privilege"
+    | "security_label";
   readonly objectType: "view" = "view";
 }
 

package/src/core/objects/view/changes/view.security-label.test.ts

@@ -0,0 +1,64 @@
+import { describe, expect, test } from "bun:test";
+import { assertValidSql } from "../../../test-utils/assert-valid-sql.ts";
+import { stableId } from "../../utils.ts";
+import { View, type ViewProps } from "../view.model.ts";
+import {
+  CreateSecurityLabelOnView,
+  DropSecurityLabelOnView,
+} from "./view.security-label.ts";
+
+const makeView = (): View =>
+  new View({
+    schema: "public",
+    name: "v",
+    definition: "SELECT 1",
+    row_security: false,
+    force_row_security: false,
+    has_indexes: false,
+    has_rules: false,
+    has_triggers: false,
+    has_subclasses: false,
+    is_populated: true,
+    replica_identity: "d",
+    is_partition: false,
+    options: null,
+    partition_bound: null,
+    owner: "postgres",
+    comment: null,
+    columns: [],
+    privileges: [],
+  } as ViewProps);
+
+describe("view.security-label", () => {
+  test("create serializes and tracks dependencies", async () => {
+    const view = makeView();
+    const change = new CreateSecurityLabelOnView({
+      view,
+      securityLabel: { provider: "dummy", label: "classified" },
+    });
+    expect(change.scope).toBe("security_label");
+    expect(change.creates).toEqual([
+      stableId.securityLabel(view.stableId, "dummy"),
+    ]);
+    expect(change.requires).toEqual([view.stableId]);
+    await assertValidSql(change.serialize());
+    expect(change.serialize()).toBe(
+      "SECURITY LABEL FOR dummy ON VIEW public.v IS 'classified'",
+    );
+  });
+
+  test("drop serializes to IS NULL", async () => {
+    const view = makeView();
+    const change = new DropSecurityLabelOnView({
+      view,
+      securityLabel: { provider: "dummy", label: "classified" },
+    });
+    expect(change.drops).toEqual([
+      stableId.securityLabel(view.stableId, "dummy"),
+    ]);
+    await assertValidSql(change.serialize());
+    expect(change.serialize()).toBe(
+      "SECURITY LABEL FOR dummy ON VIEW public.v IS NULL",
+    );
+  });
+});

package/src/core/objects/view/changes/view.security-label.ts

@@ -0,0 +1,77 @@
+import { quoteLiteral } from "../../base.change.ts";
+import type { SecurityLabelProps } from "../../security-label.types.ts";
+import { stableId } from "../../utils.ts";
+import type { View } from "../view.model.ts";
+import { CreateViewChange, DropViewChange } from "./view.base.ts";
+
+export type SecurityLabelView =
+  | CreateSecurityLabelOnView
+  | DropSecurityLabelOnView;
+
+export class CreateSecurityLabelOnView extends CreateViewChange {
+  public readonly view: View;
+  public readonly securityLabel: SecurityLabelProps;
+  public readonly scope = "security_label" as const;
+
+  constructor(props: { view: View; securityLabel: SecurityLabelProps }) {
+    super();
+    this.view = props.view;
+    this.securityLabel = props.securityLabel;
+  }
+
+  get creates() {
+    return [
+      stableId.securityLabel(this.view.stableId, this.securityLabel.provider),
+    ];
+  }
+
+  get requires() {
+    return [this.view.stableId];
+  }
+
+  serialize(): string {
+    return [
+      "SECURITY LABEL FOR",
+      this.securityLabel.provider,
+      "ON VIEW",
+      `${this.view.schema}.${this.view.name}`,
+      "IS",
+      quoteLiteral(this.securityLabel.label),
+    ].join(" ");
+  }
+}
+
+export class DropSecurityLabelOnView extends DropViewChange {
+  public readonly view: View;
+  public readonly securityLabel: SecurityLabelProps;
+  public readonly scope = "security_label" as const;
+
+  constructor(props: { view: View; securityLabel: SecurityLabelProps }) {
+    super();
+    this.view = props.view;
+    this.securityLabel = props.securityLabel;
+  }
+
+  get drops() {
+    return [
+      stableId.securityLabel(this.view.stableId, this.securityLabel.provider),
+    ];
+  }
+
+  get requires() {
+    return [
+      stableId.securityLabel(this.view.stableId, this.securityLabel.provider),
+      this.view.stableId,
+    ];
+  }
+
+  serialize(): string {
+    return [
+      "SECURITY LABEL FOR",
+      this.securityLabel.provider,
+      "ON VIEW",
+      `${this.view.schema}.${this.view.name}`,
+      "IS NULL",
+    ].join(" ");
+  }
+}

package/src/core/objects/view/changes/view.types.ts

@@ -3,6 +3,7 @@ import type { CommentView } from "./view.comment.ts";
 import type { CreateView } from "./view.create.ts";
 import type { DropView } from "./view.drop.ts";
 import type { ViewPrivilege } from "./view.privilege.ts";
+import type { SecurityLabelView } from "./view.security-label.ts";
 
 /** Union of all view-related change variants (`objectType: "view"`). @category Change Types */
 export type ViewChange =
@@ -10,4 +11,5 @@ export type ViewChange =
   | CommentView
   | CreateView
   | DropView
-  | ViewPrivilege;
+  | ViewPrivilege
+  | SecurityLabelView;

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

@@ -5,6 +5,7 @@ import {
   emitColumnPrivilegeChanges,
 } from "../base.privilege-diff.ts";
 import type { ObjectDiffContext } from "../diff-context.ts";
+import { diffSecurityLabels } from "../security-label.types.ts";
 import { deepEqual, hasNonAlterableChanges } from "../utils.ts";
 import {
   AlterViewChangeOwner,
@@ -22,6 +23,10 @@ import {
   RevokeGrantOptionViewPrivileges,
   RevokeViewPrivileges,
 } from "./changes/view.privilege.ts";
+import {
+  CreateSecurityLabelOnView,
+  DropSecurityLabelOnView,
+} from "./changes/view.security-label.ts";
 import type { ViewChange } from "./changes/view.types.ts";
 import type { View } from "./view.model.ts";
 
@@ -57,6 +62,12 @@ export function diffViews(
       changes.push(new CreateCommentOnView({ view }));
     }
 
+    for (const label of view.security_labels) {
+      changes.push(
+        new CreateSecurityLabelOnView({ view, securityLabel: label }),
+      );
+    }
+
     // PRIVILEGES: For created objects, compare against default privileges state
     // The migration script will run ALTER DEFAULT PRIVILEGES before CREATE (via constraint spec),
     // so objects are created with the default privileges state in effect.
@@ -195,6 +206,26 @@ export function diffViews(
         }
       }
 
+      // SECURITY LABELS
+      changes.push(
+        ...diffSecurityLabels<
+          CreateSecurityLabelOnView | DropSecurityLabelOnView
+        >(
+          mainView.security_labels,
+          branchView.security_labels,
+          (securityLabel) =>
+            new CreateSecurityLabelOnView({
+              view: branchView,
+              securityLabel,
+            }),
+          (securityLabel) =>
+            new DropSecurityLabelOnView({
+              view: mainView,
+              securityLabel,
+            }),
+        ),
+      );
+
       // Note: View renaming would also use ALTER VIEW ... RENAME TO ...
       // But since our View model uses 'name' as the identity field,
       // a name change would be handled as drop + create by diffObjects()

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

@@ -15,6 +15,11 @@ import {
   type ExtractRetryOptions,
   extractWithDefinitionRetry,
 } from "../extract-with-retry.ts";
+import {
+  normalizeSecurityLabels,
+  type SecurityLabelProps,
+  securityLabelPropsSchema,
+} from "../security-label.types.ts";
 import { ReplicaIdentitySchema } from "../table/table.model.ts";
 
 const viewPropsSchema = z.object({
@@ -36,6 +41,7 @@ const viewPropsSchema = z.object({
   comment: z.string().nullable(),
   columns: z.array(columnPropsSchema),
   privileges: z.array(privilegePropsSchema),
+  security_labels: z.array(securityLabelPropsSchema).default([]).optional(),
 });
 
 // pg_get_viewdef(oid) can return NULL when the underlying view (or its
@@ -69,6 +75,7 @@ export class View extends BasePgModel implements TableLikeObject {
   public readonly comment: ViewProps["comment"];
   public readonly columns: ViewProps["columns"];
   public readonly privileges: ViewPrivilegeProps[];
+  public readonly security_labels: SecurityLabelProps[];
 
   constructor(props: ViewProps) {
     super();
@@ -94,6 +101,7 @@ export class View extends BasePgModel implements TableLikeObject {
     this.comment = props.comment;
     this.columns = props.columns;
     this.privileges = props.privileges;
+    this.security_labels = props.security_labels ?? [];
   }
 
   get stableId(): `view:${string}` {
@@ -125,6 +133,7 @@ export class View extends BasePgModel implements TableLikeObject {
       comment: this.comment,
       columns: this.columns,
       privileges: this.privileges,
+      security_labels: this.security_labels,
     };
   }
 
@@ -134,6 +143,7 @@ export class View extends BasePgModel implements TableLikeObject {
       data: {
         ...this.dataFields,
         columns: normalizeColumns(this.columns),
+        security_labels: normalizeSecurityLabels(this.security_labels),
       },
     };
   }
@@ -264,7 +274,20 @@ select
       join lateral aclexplode(src.acl) as x(grantor, grantee, privilege_type, is_grantable) on true
       group by x.grantee, x.privilege_type
     ) as grp
-  ), '[]') as privileges
+  ), '[]') as privileges,
+  coalesce(
+    (
+      select json_agg(
+        json_build_object('provider', sl.provider, 'label', sl.label)
+        order by sl.provider
+      )
+      from pg_catalog.pg_seclabel sl
+      where sl.objoid = v.oid
+        and sl.classoid = 'pg_class'::regclass
+        and sl.objsubid = 0
+    ),
+    '[]'::json
+  ) as security_labels
 from
   views v
   left join pg_attribute a on a.attrelid = v.oid and a.attnum > 0 and not a.attisdropped
@@ -281,5 +304,5 @@ order by
   const validatedRows = viewRows.filter(
     (row): row is ViewProps => row.definition !== null,
   );
-  return validatedRows.map((row: ViewProps) => new View(row));
+  return validatedRows.map((row) => new View(row));
 }

package/src/core/plan/sql-format/fixtures.ts

@@ -1119,6 +1119,7 @@ const schema = new Schema({
   owner: "admin",
   comment: "application schema",
   privileges: [],
+  security_labels: [],
 });
 
 const extension = new Extension({