From 38522e5dfa38831d44655faef068a525852f7ea2 Mon Sep 17 00:00:00 2001 From: -LAN- Date: Thu, 27 Nov 2025 08:39:49 +0800 Subject: [PATCH] fix: use default_factory for callable defaults in ORM dataclasses (#28730) --- api/models/account.py | 24 +++++-- api/models/api_based_extension.py | 4 +- api/models/dataset.py | 106 +++++++++++++++++++++++++----- api/models/model.py | 52 +++++++++++---- api/models/oauth.py | 12 +++- api/models/provider.py | 40 ++++++++--- api/models/source.py | 8 ++- api/models/task.py | 7 +- api/models/tools.py | 44 +++++++++---- api/models/trigger.py | 36 +++++++--- api/models/web.py | 8 ++- api/models/workflow.py | 4 +- 12 files changed, 269 insertions(+), 76 deletions(-) diff --git a/api/models/account.py b/api/models/account.py index b1dafed0ed..420e6adc6c 100644 --- a/api/models/account.py +++ b/api/models/account.py @@ -88,7 +88,9 @@ class Account(UserMixin, TypeBase): __tablename__ = "accounts" __table_args__ = (sa.PrimaryKeyConstraint("id", name="account_pkey"), sa.Index("account_email_idx", "email")) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) name: Mapped[str] = mapped_column(String(255)) email: Mapped[str] = mapped_column(String(255)) password: Mapped[str | None] = mapped_column(String(255), default=None) @@ -235,7 +237,9 @@ class Tenant(TypeBase): __tablename__ = "tenants" __table_args__ = (sa.PrimaryKeyConstraint("id", name="tenant_pkey"),) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) name: Mapped[str] = mapped_column(String(255)) encrypt_public_key: Mapped[str | None] = mapped_column(LongText, default=None) plan: Mapped[str] = mapped_column(String(255), server_default=sa.text("'basic'"), default="basic") @@ -275,7 +279,9 @@ class TenantAccountJoin(TypeBase): sa.UniqueConstraint("tenant_id", "account_id", name="unique_tenant_account_join"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID) account_id: Mapped[str] = mapped_column(StringUUID) current: Mapped[bool] = mapped_column(sa.Boolean, server_default=sa.text("false"), default=False) @@ -297,7 +303,9 @@ class AccountIntegrate(TypeBase): sa.UniqueConstraint("provider", "open_id", name="unique_provider_open_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) account_id: Mapped[str] = mapped_column(StringUUID) provider: Mapped[str] = mapped_column(String(16)) open_id: Mapped[str] = mapped_column(String(255)) @@ -348,7 +356,9 @@ class TenantPluginPermission(TypeBase): sa.UniqueConstraint("tenant_id", name="unique_tenant_plugin"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) install_permission: Mapped[InstallPermission] = mapped_column( String(16), nullable=False, server_default="everyone", default=InstallPermission.EVERYONE @@ -375,7 +385,9 @@ class TenantPluginAutoUpgradeStrategy(TypeBase): sa.UniqueConstraint("tenant_id", name="unique_tenant_plugin_auto_upgrade_strategy"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) strategy_setting: Mapped[StrategySetting] = mapped_column( String(16), nullable=False, server_default="fix_only", default=StrategySetting.FIX_ONLY diff --git a/api/models/api_based_extension.py b/api/models/api_based_extension.py index 99d33908f8..b5acab5a75 100644 --- a/api/models/api_based_extension.py +++ b/api/models/api_based_extension.py @@ -24,7 +24,9 @@ class APIBasedExtension(TypeBase): sa.Index("api_based_extension_tenant_idx", "tenant_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) name: Mapped[str] = mapped_column(String(255), nullable=False) api_endpoint: Mapped[str] = mapped_column(String(255), nullable=False) diff --git a/api/models/dataset.py b/api/models/dataset.py index 2ea6d98b5f..e072711b82 100644 --- a/api/models/dataset.py +++ b/api/models/dataset.py @@ -920,7 +920,12 @@ class AppDatasetJoin(TypeBase): ) id: Mapped[str] = mapped_column( - StringUUID, primary_key=True, nullable=False, default=lambda: str(uuid4()), init=False + StringUUID, + primary_key=True, + nullable=False, + insert_default=lambda: str(uuid4()), + default_factory=lambda: str(uuid4()), + init=False, ) app_id: Mapped[str] = mapped_column(StringUUID, nullable=False) dataset_id: Mapped[str] = mapped_column(StringUUID, nullable=False) @@ -941,7 +946,12 @@ class DatasetQuery(TypeBase): ) id: Mapped[str] = mapped_column( - StringUUID, primary_key=True, nullable=False, default=lambda: str(uuid4()), init=False + StringUUID, + primary_key=True, + nullable=False, + insert_default=lambda: str(uuid4()), + default_factory=lambda: str(uuid4()), + init=False, ) dataset_id: Mapped[str] = mapped_column(StringUUID, nullable=False) content: Mapped[str] = mapped_column(LongText, nullable=False) @@ -961,7 +971,13 @@ class DatasetKeywordTable(TypeBase): sa.Index("dataset_keyword_table_dataset_id_idx", "dataset_id"), ) - id: Mapped[str] = mapped_column(StringUUID, primary_key=True, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, + primary_key=True, + insert_default=lambda: str(uuid4()), + default_factory=lambda: str(uuid4()), + init=False, + ) dataset_id: Mapped[str] = mapped_column(StringUUID, nullable=False, unique=True) keyword_table: Mapped[str] = mapped_column(LongText, nullable=False) data_source_type: Mapped[str] = mapped_column( @@ -1012,7 +1028,13 @@ class Embedding(TypeBase): sa.Index("created_at_idx", "created_at"), ) - id: Mapped[str] = mapped_column(StringUUID, primary_key=True, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, + primary_key=True, + insert_default=lambda: str(uuid4()), + default_factory=lambda: str(uuid4()), + init=False, + ) model_name: Mapped[str] = mapped_column( String(255), nullable=False, server_default=sa.text("'text-embedding-ada-002'") ) @@ -1037,7 +1059,13 @@ class DatasetCollectionBinding(TypeBase): sa.Index("provider_model_name_idx", "provider_name", "model_name"), ) - id: Mapped[str] = mapped_column(StringUUID, primary_key=True, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, + primary_key=True, + insert_default=lambda: str(uuid4()), + default_factory=lambda: str(uuid4()), + init=False, + ) provider_name: Mapped[str] = mapped_column(String(255), nullable=False) model_name: Mapped[str] = mapped_column(String(255), nullable=False) type: Mapped[str] = mapped_column(String(40), server_default=sa.text("'dataset'"), nullable=False) @@ -1073,7 +1101,13 @@ class Whitelist(TypeBase): sa.PrimaryKeyConstraint("id", name="whitelists_pkey"), sa.Index("whitelists_tenant_idx", "tenant_id"), ) - id: Mapped[str] = mapped_column(StringUUID, primary_key=True, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, + primary_key=True, + insert_default=lambda: str(uuid4()), + default_factory=lambda: str(uuid4()), + init=False, + ) tenant_id: Mapped[str | None] = mapped_column(StringUUID, nullable=True) category: Mapped[str] = mapped_column(String(255), nullable=False) created_at: Mapped[datetime] = mapped_column( @@ -1090,7 +1124,13 @@ class DatasetPermission(TypeBase): sa.Index("idx_dataset_permissions_tenant_id", "tenant_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), primary_key=True, init=False) + id: Mapped[str] = mapped_column( + StringUUID, + insert_default=lambda: str(uuid4()), + default_factory=lambda: str(uuid4()), + primary_key=True, + init=False, + ) dataset_id: Mapped[str] = mapped_column(StringUUID, nullable=False) account_id: Mapped[str] = mapped_column(StringUUID, nullable=False) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) @@ -1110,7 +1150,13 @@ class ExternalKnowledgeApis(TypeBase): sa.Index("external_knowledge_apis_name_idx", "name"), ) - id: Mapped[str] = mapped_column(StringUUID, nullable=False, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, + nullable=False, + insert_default=lambda: str(uuid4()), + default_factory=lambda: str(uuid4()), + init=False, + ) name: Mapped[str] = mapped_column(String(255), nullable=False) description: Mapped[str] = mapped_column(String(255), nullable=False) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) @@ -1167,7 +1213,13 @@ class ExternalKnowledgeBindings(TypeBase): sa.Index("external_knowledge_bindings_external_knowledge_api_idx", "external_knowledge_api_id"), ) - id: Mapped[str] = mapped_column(StringUUID, nullable=False, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, + nullable=False, + insert_default=lambda: str(uuid4()), + default_factory=lambda: str(uuid4()), + init=False, + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) external_knowledge_api_id: Mapped[str] = mapped_column(StringUUID, nullable=False) dataset_id: Mapped[str] = mapped_column(StringUUID, nullable=False) @@ -1191,7 +1243,9 @@ class DatasetAutoDisableLog(TypeBase): sa.Index("dataset_auto_disable_log_created_atx", "created_at"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) dataset_id: Mapped[str] = mapped_column(StringUUID, nullable=False) document_id: Mapped[str] = mapped_column(StringUUID, nullable=False) @@ -1209,7 +1263,9 @@ class RateLimitLog(TypeBase): sa.Index("rate_limit_log_operation_idx", "operation"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) subscription_plan: Mapped[str] = mapped_column(String(255), nullable=False) operation: Mapped[str] = mapped_column(String(255), nullable=False) @@ -1226,7 +1282,9 @@ class DatasetMetadata(TypeBase): sa.Index("dataset_metadata_dataset_idx", "dataset_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) dataset_id: Mapped[str] = mapped_column(StringUUID, nullable=False) type: Mapped[str] = mapped_column(String(255), nullable=False) @@ -1255,7 +1313,9 @@ class DatasetMetadataBinding(TypeBase): sa.Index("dataset_metadata_binding_document_idx", "document_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) dataset_id: Mapped[str] = mapped_column(StringUUID, nullable=False) metadata_id: Mapped[str] = mapped_column(StringUUID, nullable=False) @@ -1270,7 +1330,9 @@ class PipelineBuiltInTemplate(TypeBase): __tablename__ = "pipeline_built_in_templates" __table_args__ = (sa.PrimaryKeyConstraint("id", name="pipeline_built_in_template_pkey"),) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuidv7()), default_factory=lambda: str(uuidv7()), init=False + ) name: Mapped[str] = mapped_column(sa.String(255), nullable=False) description: Mapped[str] = mapped_column(LongText, nullable=False) chunk_structure: Mapped[str] = mapped_column(sa.String(255), nullable=False) @@ -1300,7 +1362,9 @@ class PipelineCustomizedTemplate(TypeBase): sa.Index("pipeline_customized_template_tenant_idx", "tenant_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuidv7()), default_factory=lambda: str(uuidv7()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) name: Mapped[str] = mapped_column(sa.String(255), nullable=False) description: Mapped[str] = mapped_column(LongText, nullable=False) @@ -1335,7 +1399,9 @@ class Pipeline(TypeBase): __tablename__ = "pipelines" __table_args__ = (sa.PrimaryKeyConstraint("id", name="pipeline_pkey"),) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuidv7()), default_factory=lambda: str(uuidv7()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) name: Mapped[str] = mapped_column(sa.String(255), nullable=False) description: Mapped[str] = mapped_column(LongText, nullable=False, default=sa.text("''")) @@ -1368,7 +1434,9 @@ class DocumentPipelineExecutionLog(TypeBase): sa.Index("document_pipeline_execution_logs_document_id_idx", "document_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuidv7()), default_factory=lambda: str(uuidv7()), init=False + ) pipeline_id: Mapped[str] = mapped_column(StringUUID, nullable=False) document_id: Mapped[str] = mapped_column(StringUUID, nullable=False) datasource_type: Mapped[str] = mapped_column(sa.String(255), nullable=False) @@ -1385,7 +1453,9 @@ class PipelineRecommendedPlugin(TypeBase): __tablename__ = "pipeline_recommended_plugins" __table_args__ = (sa.PrimaryKeyConstraint("id", name="pipeline_recommended_plugin_pkey"),) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuidv7()), default_factory=lambda: str(uuidv7()), init=False + ) plugin_id: Mapped[str] = mapped_column(LongText, nullable=False) provider_name: Mapped[str] = mapped_column(LongText, nullable=False) position: Mapped[int] = mapped_column(sa.Integer, nullable=False, default=0) diff --git a/api/models/model.py b/api/models/model.py index 33a94628f0..1731ff5699 100644 --- a/api/models/model.py +++ b/api/models/model.py @@ -572,7 +572,9 @@ class InstalledApp(TypeBase): sa.UniqueConstraint("tenant_id", "app_id", name="unique_tenant_app"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) app_id: Mapped[str] = mapped_column(StringUUID, nullable=False) app_owner_tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) @@ -606,7 +608,9 @@ class OAuthProviderApp(TypeBase): sa.Index("oauth_provider_app_client_id_idx", "client_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuidv7()), default_factory=lambda: str(uuidv7()), init=False + ) app_icon: Mapped[str] = mapped_column(String(255), nullable=False) client_id: Mapped[str] = mapped_column(String(255), nullable=False) client_secret: Mapped[str] = mapped_column(String(255), nullable=False) @@ -1311,7 +1315,9 @@ class MessageFeedback(TypeBase): sa.Index("message_feedback_conversation_idx", "conversation_id", "from_source", "rating"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) app_id: Mapped[str] = mapped_column(StringUUID, nullable=False) conversation_id: Mapped[str] = mapped_column(StringUUID, nullable=False) message_id: Mapped[str] = mapped_column(StringUUID, nullable=False) @@ -1360,7 +1366,9 @@ class MessageFile(TypeBase): sa.Index("message_file_created_by_idx", "created_by"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) message_id: Mapped[str] = mapped_column(StringUUID, nullable=False) type: Mapped[str] = mapped_column(String(255), nullable=False) transfer_method: Mapped[FileTransferMethod] = mapped_column(String(255), nullable=False) @@ -1452,7 +1460,9 @@ class AppAnnotationSetting(TypeBase): sa.Index("app_annotation_settings_app_idx", "app_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) app_id: Mapped[str] = mapped_column(StringUUID, nullable=False) score_threshold: Mapped[float] = mapped_column(Float, nullable=False, server_default=sa.text("0")) collection_binding_id: Mapped[str] = mapped_column(StringUUID, nullable=False) @@ -1488,7 +1498,9 @@ class OperationLog(TypeBase): sa.Index("operation_log_account_action_idx", "tenant_id", "account_id", "action"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) account_id: Mapped[str] = mapped_column(StringUUID, nullable=False) action: Mapped[str] = mapped_column(String(255), nullable=False) @@ -1554,7 +1566,9 @@ class AppMCPServer(TypeBase): sa.UniqueConstraint("tenant_id", "app_id", name="unique_app_mcp_server_tenant_app_id"), sa.UniqueConstraint("server_code", name="unique_app_mcp_server_server_code"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) app_id: Mapped[str] = mapped_column(StringUUID, nullable=False) name: Mapped[str] = mapped_column(String(255), nullable=False) @@ -1764,7 +1778,9 @@ class ApiRequest(TypeBase): sa.Index("api_request_token_idx", "tenant_id", "api_token_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) api_token_id: Mapped[str] = mapped_column(StringUUID, nullable=False) path: Mapped[str] = mapped_column(String(255), nullable=False) @@ -1783,7 +1799,9 @@ class MessageChain(TypeBase): sa.Index("message_chain_message_id_idx", "message_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) message_id: Mapped[str] = mapped_column(StringUUID, nullable=False) type: Mapped[str] = mapped_column(String(255), nullable=False) input: Mapped[str | None] = mapped_column(LongText, nullable=True) @@ -1914,7 +1932,9 @@ class DatasetRetrieverResource(TypeBase): sa.Index("dataset_retriever_resource_message_id_idx", "message_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) message_id: Mapped[str] = mapped_column(StringUUID, nullable=False) position: Mapped[int] = mapped_column(sa.Integer, nullable=False) dataset_id: Mapped[str] = mapped_column(StringUUID, nullable=False) @@ -1946,7 +1966,9 @@ class Tag(TypeBase): TAG_TYPE_LIST = ["knowledge", "app"] - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str | None] = mapped_column(StringUUID, nullable=True) type: Mapped[str] = mapped_column(String(16), nullable=False) name: Mapped[str] = mapped_column(String(255), nullable=False) @@ -1964,7 +1986,9 @@ class TagBinding(TypeBase): sa.Index("tag_bind_tag_id_idx", "tag_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str | None] = mapped_column(StringUUID, nullable=True) tag_id: Mapped[str | None] = mapped_column(StringUUID, nullable=True) target_id: Mapped[str | None] = mapped_column(StringUUID, nullable=True) @@ -1981,7 +2005,9 @@ class TraceAppConfig(TypeBase): sa.Index("trace_app_config_app_id_idx", "app_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) app_id: Mapped[str] = mapped_column(StringUUID, nullable=False) tracing_provider: Mapped[str | None] = mapped_column(String(255), nullable=True) tracing_config: Mapped[dict | None] = mapped_column(sa.JSON, nullable=True) diff --git a/api/models/oauth.py b/api/models/oauth.py index 2fce67c998..1db2552469 100644 --- a/api/models/oauth.py +++ b/api/models/oauth.py @@ -17,7 +17,9 @@ class DatasourceOauthParamConfig(TypeBase): sa.UniqueConstraint("plugin_id", "provider", name="datasource_oauth_config_datasource_id_provider_idx"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuidv7()), default_factory=lambda: str(uuidv7()), init=False + ) plugin_id: Mapped[str] = mapped_column(sa.String(255), nullable=False) provider: Mapped[str] = mapped_column(sa.String(255), nullable=False) system_credentials: Mapped[dict] = mapped_column(AdjustedJSON, nullable=False) @@ -30,7 +32,9 @@ class DatasourceProvider(TypeBase): sa.UniqueConstraint("tenant_id", "plugin_id", "provider", "name", name="datasource_provider_unique_name"), sa.Index("datasource_provider_auth_type_provider_idx", "tenant_id", "plugin_id", "provider"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuidv7()), default_factory=lambda: str(uuidv7()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) name: Mapped[str] = mapped_column(sa.String(255), nullable=False) provider: Mapped[str] = mapped_column(sa.String(128), nullable=False) @@ -60,7 +64,9 @@ class DatasourceOauthTenantParamConfig(TypeBase): sa.UniqueConstraint("tenant_id", "plugin_id", "provider", name="datasource_oauth_tenant_config_unique"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuidv7()), default_factory=lambda: str(uuidv7()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) provider: Mapped[str] = mapped_column(sa.String(255), nullable=False) plugin_id: Mapped[str] = mapped_column(sa.String(255), nullable=False) diff --git a/api/models/provider.py b/api/models/provider.py index 577e098a2e..2afd8c5329 100644 --- a/api/models/provider.py +++ b/api/models/provider.py @@ -58,7 +58,13 @@ class Provider(TypeBase): ), ) - id: Mapped[str] = mapped_column(StringUUID, primary_key=True, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, + primary_key=True, + insert_default=lambda: str(uuidv7()), + default_factory=lambda: str(uuidv7()), + init=False, + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) provider_name: Mapped[str] = mapped_column(String(255), nullable=False) provider_type: Mapped[str] = mapped_column( @@ -132,7 +138,9 @@ class ProviderModel(TypeBase): ), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) provider_name: Mapped[str] = mapped_column(String(255), nullable=False) model_name: Mapped[str] = mapped_column(String(255), nullable=False) @@ -173,7 +181,9 @@ class TenantDefaultModel(TypeBase): sa.Index("tenant_default_model_tenant_id_provider_type_idx", "tenant_id", "provider_name", "model_type"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) provider_name: Mapped[str] = mapped_column(String(255), nullable=False) model_name: Mapped[str] = mapped_column(String(255), nullable=False) @@ -193,7 +203,9 @@ class TenantPreferredModelProvider(TypeBase): sa.Index("tenant_preferred_model_provider_tenant_provider_idx", "tenant_id", "provider_name"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) provider_name: Mapped[str] = mapped_column(String(255), nullable=False) preferred_provider_type: Mapped[str] = mapped_column(String(40), nullable=False) @@ -212,7 +224,9 @@ class ProviderOrder(TypeBase): sa.Index("provider_order_tenant_provider_idx", "tenant_id", "provider_name"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) provider_name: Mapped[str] = mapped_column(String(255), nullable=False) account_id: Mapped[str] = mapped_column(StringUUID, nullable=False) @@ -245,7 +259,9 @@ class ProviderModelSetting(TypeBase): sa.Index("provider_model_setting_tenant_provider_model_idx", "tenant_id", "provider_name", "model_type"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) provider_name: Mapped[str] = mapped_column(String(255), nullable=False) model_name: Mapped[str] = mapped_column(String(255), nullable=False) @@ -273,7 +289,9 @@ class LoadBalancingModelConfig(TypeBase): sa.Index("load_balancing_model_config_tenant_provider_model_idx", "tenant_id", "provider_name", "model_type"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) provider_name: Mapped[str] = mapped_column(String(255), nullable=False) model_name: Mapped[str] = mapped_column(String(255), nullable=False) @@ -302,7 +320,9 @@ class ProviderCredential(TypeBase): sa.Index("provider_credential_tenant_provider_idx", "tenant_id", "provider_name"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuidv7()), default_factory=lambda: str(uuidv7()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) provider_name: Mapped[str] = mapped_column(String(255), nullable=False) credential_name: Mapped[str] = mapped_column(String(255), nullable=False) @@ -332,7 +352,9 @@ class ProviderModelCredential(TypeBase): ), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuidv7()), default_factory=lambda: str(uuidv7()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) provider_name: Mapped[str] = mapped_column(String(255), nullable=False) model_name: Mapped[str] = mapped_column(String(255), nullable=False) diff --git a/api/models/source.py b/api/models/source.py index f093048c00..a8addbe342 100644 --- a/api/models/source.py +++ b/api/models/source.py @@ -18,7 +18,9 @@ class DataSourceOauthBinding(TypeBase): adjusted_json_index("source_info_idx", "source_info"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) access_token: Mapped[str] = mapped_column(String(255), nullable=False) provider: Mapped[str] = mapped_column(String(255), nullable=False) @@ -44,7 +46,9 @@ class DataSourceApiKeyAuthBinding(TypeBase): sa.Index("data_source_api_key_auth_binding_provider_idx", "provider"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) category: Mapped[str] = mapped_column(String(255), nullable=False) provider: Mapped[str] = mapped_column(String(255), nullable=False) diff --git a/api/models/task.py b/api/models/task.py index 539945b251..d98d99ca2c 100644 --- a/api/models/task.py +++ b/api/models/task.py @@ -24,7 +24,8 @@ class CeleryTask(TypeBase): result: Mapped[bytes | None] = mapped_column(BinaryData, nullable=True, default=None) date_done: Mapped[datetime | None] = mapped_column( DateTime, - default=naive_utc_now, + insert_default=naive_utc_now, + default=None, onupdate=naive_utc_now, nullable=True, ) @@ -47,4 +48,6 @@ class CeleryTaskSet(TypeBase): ) taskset_id: Mapped[str] = mapped_column(String(155), unique=True) result: Mapped[bytes | None] = mapped_column(BinaryData, nullable=True, default=None) - date_done: Mapped[datetime | None] = mapped_column(DateTime, default=naive_utc_now, nullable=True) + date_done: Mapped[datetime | None] = mapped_column( + DateTime, insert_default=naive_utc_now, default=None, nullable=True + ) diff --git a/api/models/tools.py b/api/models/tools.py index 0a79f95a70..e4f9bcb582 100644 --- a/api/models/tools.py +++ b/api/models/tools.py @@ -30,7 +30,9 @@ class ToolOAuthSystemClient(TypeBase): sa.UniqueConstraint("plugin_id", "provider", name="tool_oauth_system_client_plugin_id_provider_idx"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) plugin_id: Mapped[str] = mapped_column(String(512), nullable=False) provider: Mapped[str] = mapped_column(String(255), nullable=False) # oauth params of the tool provider @@ -45,7 +47,9 @@ class ToolOAuthTenantClient(TypeBase): sa.UniqueConstraint("tenant_id", "plugin_id", "provider", name="unique_tool_oauth_tenant_client"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) # tenant id tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) plugin_id: Mapped[str] = mapped_column(String(255), nullable=False) @@ -71,7 +75,9 @@ class BuiltinToolProvider(TypeBase): ) # id of the tool provider - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) name: Mapped[str] = mapped_column( String(256), nullable=False, @@ -120,7 +126,9 @@ class ApiToolProvider(TypeBase): sa.UniqueConstraint("name", "tenant_id", name="unique_api_tool_provider"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) # name of the api provider name: Mapped[str] = mapped_column( String(255), @@ -192,7 +200,9 @@ class ToolLabelBinding(TypeBase): sa.UniqueConstraint("tool_id", "label_name", name="unique_tool_label_bind"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) # tool id tool_id: Mapped[str] = mapped_column(String(64), nullable=False) # tool type @@ -213,7 +223,9 @@ class WorkflowToolProvider(TypeBase): sa.UniqueConstraint("tenant_id", "app_id", name="unique_workflow_tool_provider_app_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) # name of the workflow provider name: Mapped[str] = mapped_column(String(255), nullable=False) # label of the workflow provider @@ -279,7 +291,9 @@ class MCPToolProvider(TypeBase): sa.UniqueConstraint("tenant_id", "server_identifier", name="unique_mcp_provider_server_identifier"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) # name of the mcp provider name: Mapped[str] = mapped_column(String(40), nullable=False) # server identifier of the mcp provider @@ -360,7 +374,9 @@ class ToolModelInvoke(TypeBase): __tablename__ = "tool_model_invokes" __table_args__ = (sa.PrimaryKeyConstraint("id", name="tool_model_invoke_pkey"),) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) # who invoke this tool user_id: Mapped[str] = mapped_column(StringUUID, nullable=False) # tenant id @@ -413,7 +429,9 @@ class ToolConversationVariables(TypeBase): sa.Index("conversation_id_idx", "conversation_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) # conversation user id user_id: Mapped[str] = mapped_column(StringUUID, nullable=False) # tenant id @@ -450,7 +468,9 @@ class ToolFile(TypeBase): sa.Index("tool_file_conversation_id_idx", "conversation_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) # conversation user id user_id: Mapped[str] = mapped_column(StringUUID) # tenant id @@ -481,7 +501,9 @@ class DeprecatedPublishedAppTool(TypeBase): sa.UniqueConstraint("app_id", "user_id", name="unique_published_app_tool"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) # id of the app app_id: Mapped[str] = mapped_column(StringUUID, ForeignKey("apps.id"), nullable=False) diff --git a/api/models/trigger.py b/api/models/trigger.py index 088e797f82..87e2a5ccfc 100644 --- a/api/models/trigger.py +++ b/api/models/trigger.py @@ -41,7 +41,9 @@ class TriggerSubscription(TypeBase): UniqueConstraint("tenant_id", "provider_id", "name", name="unique_trigger_provider"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) name: Mapped[str] = mapped_column(String(255), nullable=False, comment="Subscription instance name") tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) user_id: Mapped[str] = mapped_column(StringUUID, nullable=False) @@ -111,7 +113,9 @@ class TriggerOAuthSystemClient(TypeBase): sa.UniqueConstraint("plugin_id", "provider", name="trigger_oauth_system_client_plugin_id_provider_idx"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) plugin_id: Mapped[str] = mapped_column(String(255), nullable=False) provider: Mapped[str] = mapped_column(String(255), nullable=False) # oauth params of the trigger provider @@ -136,7 +140,9 @@ class TriggerOAuthTenantClient(TypeBase): sa.UniqueConstraint("tenant_id", "plugin_id", "provider", name="unique_trigger_oauth_tenant_client"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) # tenant id tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) plugin_id: Mapped[str] = mapped_column(String(255), nullable=False) @@ -202,7 +208,9 @@ class WorkflowTriggerLog(TypeBase): sa.Index("workflow_trigger_log_workflow_id_idx", "workflow_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuidv7()), default_factory=lambda: str(uuidv7()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) app_id: Mapped[str] = mapped_column(StringUUID, nullable=False) workflow_id: Mapped[str] = mapped_column(StringUUID, nullable=False) @@ -294,7 +302,9 @@ class WorkflowWebhookTrigger(TypeBase): sa.UniqueConstraint("webhook_id", name="uniq_webhook_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuidv7()), default_factory=lambda: str(uuidv7()), init=False + ) app_id: Mapped[str] = mapped_column(StringUUID, nullable=False) node_id: Mapped[str] = mapped_column(String(64), nullable=False) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) @@ -351,7 +361,9 @@ class WorkflowPluginTrigger(TypeBase): sa.UniqueConstraint("app_id", "node_id", name="uniq_app_node_subscription"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) app_id: Mapped[str] = mapped_column(StringUUID, nullable=False) node_id: Mapped[str] = mapped_column(String(64), nullable=False) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) @@ -395,7 +407,9 @@ class AppTrigger(TypeBase): sa.Index("app_trigger_tenant_app_idx", "tenant_id", "app_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuidv7()), default_factory=lambda: str(uuidv7()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) app_id: Mapped[str] = mapped_column(StringUUID, nullable=False) node_id: Mapped[str | None] = mapped_column(String(64), nullable=False) @@ -443,7 +457,13 @@ class WorkflowSchedulePlan(TypeBase): sa.Index("workflow_schedule_plan_next_idx", "next_run_at"), ) - id: Mapped[str] = mapped_column(StringUUID, primary_key=True, default=lambda: str(uuidv7()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, + primary_key=True, + insert_default=lambda: str(uuidv7()), + default_factory=lambda: str(uuidv7()), + init=False, + ) app_id: Mapped[str] = mapped_column(StringUUID, nullable=False) node_id: Mapped[str] = mapped_column(String(64), nullable=False) tenant_id: Mapped[str] = mapped_column(StringUUID, nullable=False) diff --git a/api/models/web.py b/api/models/web.py index 4f0bf7c7da..b2832aa163 100644 --- a/api/models/web.py +++ b/api/models/web.py @@ -18,7 +18,9 @@ class SavedMessage(TypeBase): sa.Index("saved_message_message_idx", "app_id", "message_id", "created_by_role", "created_by"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) app_id: Mapped[str] = mapped_column(StringUUID, nullable=False) message_id: Mapped[str] = mapped_column(StringUUID, nullable=False) created_by_role: Mapped[str] = mapped_column(String(255), nullable=False, server_default=sa.text("'end_user'")) @@ -42,7 +44,9 @@ class PinnedConversation(TypeBase): sa.Index("pinned_conversation_conversation_idx", "app_id", "conversation_id", "created_by_role", "created_by"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) app_id: Mapped[str] = mapped_column(StringUUID, nullable=False) conversation_id: Mapped[str] = mapped_column(StringUUID) created_by_role: Mapped[str] = mapped_column( diff --git a/api/models/workflow.py b/api/models/workflow.py index 4efa829692..42ee8a1f2b 100644 --- a/api/models/workflow.py +++ b/api/models/workflow.py @@ -1103,7 +1103,9 @@ class WorkflowAppLog(TypeBase): sa.Index("workflow_app_log_workflow_run_id_idx", "workflow_run_id"), ) - id: Mapped[str] = mapped_column(StringUUID, default=lambda: str(uuid4()), init=False) + id: Mapped[str] = mapped_column( + StringUUID, insert_default=lambda: str(uuid4()), default_factory=lambda: str(uuid4()), init=False + ) tenant_id: Mapped[str] = mapped_column(StringUUID) app_id: Mapped[str] = mapped_column(StringUUID) workflow_id: Mapped[str] = mapped_column(StringUUID, nullable=False)