chore(api): Use uuidv7 as PK for provider_model_credentials and provider_credentials

Author: QuantumGhost

api/migrations/versions/2025_08_09_1553-e8446f481c1e_add_provider_credential_pool_support.py

@@ -6,10 +6,10 @@
 
 """
 from alembic import op
+from libs.uuid_utils import uuidv7
 import models as models
 import sqlalchemy as sa
 from sqlalchemy.sql import table, column
-import uuid
 
 # revision identifiers, used by Alembic.
 revision = 'e8446f481c1e'
@@ -21,7 +21,7 @@
 def upgrade():
     # Create provider_credentials table
     op.create_table('provider_credentials',
-    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuid_generate_v4()'), nullable=False),
+    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuidv7()'), nullable=False),
     sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
     sa.Column('provider_name', sa.String(length=255), nullable=False),
     sa.Column('credential_name', sa.String(length=255), nullable=False),
@@ -63,7 +63,7 @@ def migrate_existing_providers_data():
         column('updated_at', sa.DateTime()),
         column('credential_id', models.types.StringUUID()),
     )
-    
+
     provider_credential_table = table('provider_credentials',
         column('id', models.types.StringUUID()),
         column('tenant_id', models.types.StringUUID()),
@@ -79,15 +79,15 @@ def migrate_existing_providers_data():
 
     # Query all existing providers data
     existing_providers = conn.execute(
-        sa.select(providers_table.c.id, providers_table.c.tenant_id, 
+        sa.select(providers_table.c.id, providers_table.c.tenant_id,
                  providers_table.c.provider_name, providers_table.c.encrypted_config,
                  providers_table.c.created_at, providers_table.c.updated_at)
         .where(providers_table.c.encrypted_config.isnot(None))
     ).fetchall()
-    
+
     # Iterate through each provider and insert into provider_credentials
     for provider in existing_providers:
-        credential_id = str(uuid.uuid4())
+        credential_id = str(uuidv7())
         if not provider.encrypted_config or provider.encrypted_config.strip() == '':
             continue
 
@@ -134,7 +134,7 @@ def downgrade():
 
 def migrate_data_back_to_providers():
     """Migrate data back from provider_credentials to providers table for downgrade"""
-    
+
     # Define table structure for data manipulation
     providers_table = table('providers',
         column('id', models.types.StringUUID()),
@@ -143,7 +143,7 @@ def migrate_data_back_to_providers():
         column('encrypted_config', sa.Text()),
         column('credential_id', models.types.StringUUID()),
     )
-    
+
     provider_credential_table = table('provider_credentials',
         column('id', models.types.StringUUID()),
         column('tenant_id', models.types.StringUUID()),
@@ -160,18 +160,18 @@ def migrate_data_back_to_providers():
         sa.select(providers_table.c.id, providers_table.c.credential_id)
         .where(providers_table.c.credential_id.isnot(None))
     ).fetchall()
-    
+
     # For each provider, get the credential data and update providers table
     for provider in providers_with_credentials:
         credential = conn.execute(
             sa.select(provider_credential_table.c.encrypted_config)
             .where(provider_credential_table.c.id == provider.credential_id)
         ).fetchone()
-        
+
         if credential:
             # Update providers table with encrypted_config from credential
             conn.execute(
                 providers_table.update()
                 .where(providers_table.c.id == provider.id)
                 .values(encrypted_config=credential.encrypted_config)
-            )
+            )

api/migrations/versions/2025_08_13_1605-0e154742a5fa_add_provider_model_multi_credential.py

@@ -5,9 +5,9 @@
 Create Date: 2025-08-13 16:05:42.657730
 
 """
-import uuid
 
 from alembic import op
+from libs.uuid_utils import uuidv7
 import models as models
 import sqlalchemy as sa
 from sqlalchemy.sql import table, column
@@ -23,7 +23,7 @@
 def upgrade():
     # Create provider_model_credentials table
     op.create_table('provider_model_credentials',
-    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuid_generate_v4()'), nullable=False),
+    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuidv7()'), nullable=False),
     sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
     sa.Column('provider_name', sa.String(length=255), nullable=False),
     sa.Column('model_name', sa.String(length=255), nullable=False),
@@ -71,7 +71,7 @@ def migrate_existing_provider_models_data():
         column('updated_at', sa.DateTime()),
         column('credential_id', models.types.StringUUID()),
     )
-    
+
     provider_model_credentials_table = table('provider_model_credentials',
         column('id', models.types.StringUUID()),
         column('tenant_id', models.types.StringUUID()),
@@ -90,19 +90,19 @@ def migrate_existing_provider_models_data():
 
     # Query all existing provider_models data with encrypted_config
     existing_provider_models = conn.execute(
-        sa.select(provider_models_table.c.id, provider_models_table.c.tenant_id, 
+        sa.select(provider_models_table.c.id, provider_models_table.c.tenant_id,
                  provider_models_table.c.provider_name, provider_models_table.c.model_name,
                  provider_models_table.c.model_type, provider_models_table.c.encrypted_config,
                  provider_models_table.c.created_at, provider_models_table.c.updated_at)
         .where(provider_models_table.c.encrypted_config.isnot(None))
     ).fetchall()
-    
+
     # Iterate through each provider_model and insert into provider_model_credentials
     for provider_model in existing_provider_models:
         if not provider_model.encrypted_config or provider_model.encrypted_config.strip() == '':
             continue
 
-        credential_id = str(uuid.uuid4())
+        credential_id = str(uuidv7())
 
         # Insert into provider_model_credentials table
         conn.execute(
@@ -148,14 +148,14 @@ def downgrade():
 
 def migrate_data_back_to_provider_models():
     """Migrate data back from provider_model_credentials to provider_models table for downgrade"""
-    
+
     # Define table structure for data manipulation
     provider_models_table = table('provider_models',
         column('id', models.types.StringUUID()),
         column('encrypted_config', sa.Text()),
         column('credential_id', models.types.StringUUID()),
     )
-    
+
     provider_model_credentials_table = table('provider_model_credentials',
         column('id', models.types.StringUUID()),
         column('encrypted_config', sa.Text()),
@@ -169,14 +169,14 @@ def migrate_data_back_to_provider_models():
         sa.select(provider_models_table.c.id, provider_models_table.c.credential_id)
         .where(provider_models_table.c.credential_id.isnot(None))
     ).fetchall()
-    
+
     # For each provider_model, get the credential data and update provider_models table
     for provider_model in provider_models_with_credentials:
         credential = conn.execute(
             sa.select(provider_model_credentials_table.c.encrypted_config)
             .where(provider_model_credentials_table.c.id == provider_model.credential_id)
         ).fetchone()
-        
+
         if credential:
             # Update provider_models table with encrypted_config from credential
             conn.execute(

api/models/provider.py

@@ -274,7 +274,7 @@ class ProviderCredential(Base):
         sa.Index("provider_credential_tenant_provider_idx", "tenant_id", "provider_name"),
     )
 
-    id: Mapped[str] = mapped_column(StringUUID, server_default=text("uuid_generate_v4()"))
+    id: Mapped[str] = mapped_column(StringUUID, server_default=text("uuidv7()"))
     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)
@@ -300,7 +300,7 @@ class ProviderModelCredential(Base):
         ),
     )
 
-    id: Mapped[str] = mapped_column(StringUUID, server_default=text("uuid_generate_v4()"))
+    id: Mapped[str] = mapped_column(StringUUID, server_default=text("uuidv7()"))
     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)