From c67a9860854ee33c6d41ad1e4618977b3ad53262 Mon Sep 17 00:00:00 2001 From: Dan Lynch Date: Tue, 31 Mar 2026 17:37:53 +0000 Subject: [PATCH] feat: remove include_id from Data* modules and blueprint types --- graphql/node-type-registry/README.md | 2 +- .../src/blueprint-types.generated.ts | 14 -------------- .../src/data/data-direct-owner.ts | 5 ----- .../src/data/data-entity-membership.ts | 5 ----- .../src/data/data-ownership-in-entity.ts | 5 ----- .../src/data/data-peoplestamps.ts | 5 ----- .../src/data/data-publishable.ts | 8 +------- .../src/data/data-soft-delete.ts | 8 +------- .../node-type-registry/src/data/data-timestamps.ts | 8 +------- 9 files changed, 4 insertions(+), 56 deletions(-) diff --git a/graphql/node-type-registry/README.md b/graphql/node-type-registry/README.md index d9f7900b3..4e6b21632 100644 --- a/graphql/node-type-registry/README.md +++ b/graphql/node-type-registry/README.md @@ -35,7 +35,7 @@ const definition: BlueprintDefinition = { nodes: [ 'DataId', 'DataTimestamps', - { $type: 'DataDirectOwner', data: { include_id: false } }, + 'DataDirectOwner', ], fields: [ { name: 'title', type: 'text', is_not_null: true }, diff --git a/graphql/node-type-registry/src/blueprint-types.generated.ts b/graphql/node-type-registry/src/blueprint-types.generated.ts index b5a5f68b5..685b32cb7 100644 --- a/graphql/node-type-registry/src/blueprint-types.generated.ts +++ b/graphql/node-type-registry/src/blueprint-types.generated.ts @@ -21,8 +21,6 @@ export interface DataIdParams { export interface DataDirectOwnerParams { /* Column name for owner ID */ owner_field_name?: string; - /* If true, also adds a UUID primary key column with auto-generation */ - include_id?: boolean; /* If true, adds a foreign key constraint from owner_id to the users table */ include_user_fk?: boolean; } @@ -30,39 +28,27 @@ export interface DataDirectOwnerParams { export interface DataEntityMembershipParams { /* Column name for entity ID */ entity_field_name?: string; - /* If true, also adds a UUID primary key column with auto-generation */ - include_id?: boolean; /* If true, adds a foreign key constraint from entity_id to the users table */ include_user_fk?: boolean; } /** Combines direct ownership with entity scoping. Adds both owner_id and entity_id columns. Enables AuthzDirectOwner, AuthzEntityMembership, and AuthzOrgHierarchy authorization. Particularly useful for OrgHierarchy where a user owns a row (owner_id) within an entity (entity_id), and managers above can see subordinate-owned records via the hierarchy closure table. */ export interface DataOwnershipInEntityParams { - /* If true, also adds a UUID primary key column with auto-generation */ - include_id?: boolean; /* If true, adds foreign key constraints from owner_id and entity_id to the users table */ include_user_fk?: boolean; } /** Adds automatic timestamp tracking with created_at and updated_at columns. */ export interface DataTimestampsParams { - /* If true, also adds a UUID primary key column with auto-generation */ - include_id?: boolean; } /** Adds user tracking for creates/updates with created_by and updated_by columns. */ export interface DataPeoplestampsParams { - /* If true, also adds a UUID primary key column with auto-generation */ - include_id?: boolean; /* If true, adds foreign key constraints from created_by and updated_by to the users table */ include_user_fk?: boolean; } /** Adds publish state columns (is_published, published_at) for content visibility. Enables AuthzPublishable and AuthzTemporal authorization. */ export interface DataPublishableParams { - /* If true, also adds a UUID primary key column with auto-generation */ - include_id?: boolean; } /** Adds soft delete support with deleted_at and is_deleted columns. */ export interface DataSoftDeleteParams { - /* If true, also adds a UUID primary key column with auto-generation */ - include_id?: boolean; } /** Adds a vector embedding column with HNSW or IVFFlat index for similarity search. Supports configurable dimensions, distance metrics (cosine, l2, ip), stale tracking strategies (column, null, hash), and automatic job enqueue triggers for embedding generation. */ export interface DataEmbeddingParams { diff --git a/graphql/node-type-registry/src/data/data-direct-owner.ts b/graphql/node-type-registry/src/data/data-direct-owner.ts index 335e9ef05..283b22258 100644 --- a/graphql/node-type-registry/src/data/data-direct-owner.ts +++ b/graphql/node-type-registry/src/data/data-direct-owner.ts @@ -14,11 +14,6 @@ export const DataDirectOwner: NodeTypeDefinition = { "description": "Column name for owner ID", "default": "owner_id" }, - "include_id": { - "type": "boolean", - "description": "If true, also adds a UUID primary key column with auto-generation", - "default": true - }, "include_user_fk": { "type": "boolean", "description": "If true, adds a foreign key constraint from owner_id to the users table", diff --git a/graphql/node-type-registry/src/data/data-entity-membership.ts b/graphql/node-type-registry/src/data/data-entity-membership.ts index b89bc9a68..65eef4145 100644 --- a/graphql/node-type-registry/src/data/data-entity-membership.ts +++ b/graphql/node-type-registry/src/data/data-entity-membership.ts @@ -14,11 +14,6 @@ export const DataEntityMembership: NodeTypeDefinition = { "description": "Column name for entity ID", "default": "entity_id" }, - "include_id": { - "type": "boolean", - "description": "If true, also adds a UUID primary key column with auto-generation", - "default": true - }, "include_user_fk": { "type": "boolean", "description": "If true, adds a foreign key constraint from entity_id to the users table", diff --git a/graphql/node-type-registry/src/data/data-ownership-in-entity.ts b/graphql/node-type-registry/src/data/data-ownership-in-entity.ts index 23f4ac47b..24c4b6a5d 100644 --- a/graphql/node-type-registry/src/data/data-ownership-in-entity.ts +++ b/graphql/node-type-registry/src/data/data-ownership-in-entity.ts @@ -9,11 +9,6 @@ export const DataOwnershipInEntity: NodeTypeDefinition = { "parameter_schema": { "type": "object", "properties": { - "include_id": { - "type": "boolean", - "description": "If true, also adds a UUID primary key column with auto-generation", - "default": true - }, "include_user_fk": { "type": "boolean", "description": "If true, adds foreign key constraints from owner_id and entity_id to the users table", diff --git a/graphql/node-type-registry/src/data/data-peoplestamps.ts b/graphql/node-type-registry/src/data/data-peoplestamps.ts index cf547c3d1..5f0337674 100644 --- a/graphql/node-type-registry/src/data/data-peoplestamps.ts +++ b/graphql/node-type-registry/src/data/data-peoplestamps.ts @@ -9,11 +9,6 @@ export const DataPeoplestamps: NodeTypeDefinition = { "parameter_schema": { "type": "object", "properties": { - "include_id": { - "type": "boolean", - "description": "If true, also adds a UUID primary key column with auto-generation", - "default": true - }, "include_user_fk": { "type": "boolean", "description": "If true, adds foreign key constraints from created_by and updated_by to the users table", diff --git a/graphql/node-type-registry/src/data/data-publishable.ts b/graphql/node-type-registry/src/data/data-publishable.ts index f60363f4c..7ea81fe3e 100644 --- a/graphql/node-type-registry/src/data/data-publishable.ts +++ b/graphql/node-type-registry/src/data/data-publishable.ts @@ -8,13 +8,7 @@ export const DataPublishable: NodeTypeDefinition = { "description": "Adds publish state columns (is_published, published_at) for content visibility. Enables AuthzPublishable and AuthzTemporal authorization.", "parameter_schema": { "type": "object", - "properties": { - "include_id": { - "type": "boolean", - "description": "If true, also adds a UUID primary key column with auto-generation", - "default": true - } - } + "properties": {} }, "tags": [ "publishing", diff --git a/graphql/node-type-registry/src/data/data-soft-delete.ts b/graphql/node-type-registry/src/data/data-soft-delete.ts index e3f5a1e4b..a08559821 100644 --- a/graphql/node-type-registry/src/data/data-soft-delete.ts +++ b/graphql/node-type-registry/src/data/data-soft-delete.ts @@ -8,13 +8,7 @@ export const DataSoftDelete: NodeTypeDefinition = { "description": "Adds soft delete support with deleted_at and is_deleted columns.", "parameter_schema": { "type": "object", - "properties": { - "include_id": { - "type": "boolean", - "description": "If true, also adds a UUID primary key column with auto-generation", - "default": true - } - } + "properties": {} }, "tags": [ "schema" diff --git a/graphql/node-type-registry/src/data/data-timestamps.ts b/graphql/node-type-registry/src/data/data-timestamps.ts index 3fca3f769..3c2558cd5 100644 --- a/graphql/node-type-registry/src/data/data-timestamps.ts +++ b/graphql/node-type-registry/src/data/data-timestamps.ts @@ -8,13 +8,7 @@ export const DataTimestamps: NodeTypeDefinition = { "description": "Adds automatic timestamp tracking with created_at and updated_at columns.", "parameter_schema": { "type": "object", - "properties": { - "include_id": { - "type": "boolean", - "description": "If true, also adds a UUID primary key column with auto-generation", - "default": true - } - } + "properties": {} }, "tags": [ "timestamps",