Fix migration issues for VARCHAR(191) column resize

- Update db.py SQLAlchemy model to match migration (String(191))
- Add dialect-specific SQL handling in migration (SQLite, PostgreSQL, MySQL)
- Use CHAR_LENGTH for character-based truncation (not byte-based LENGTH in MySQL)
- Ensure truncation works correctly across all supported database backends

Author: crivetimihai

mcpgateway/alembic/versions/878e287d2366_resize_url_and_slug_columns_to_191.py

@@ -19,13 +19,43 @@
 
 def upgrade() -> None:
     """Upgrade schema."""
-    # Truncate existing values longer than 191 chars
-    op.execute("""
-        UPDATE gateways
-        SET slug = LEFT(slug, 191),
-            url = LEFT(url, 191)
-        WHERE LENGTH(slug) > 191 OR LENGTH(url) > 191;
-    """)
+    # Get database dialect for dialect-specific operations
+    bind = op.get_bind()
+    dialect_name = bind.dialect.name
+
+    # Truncate existing values longer than 191 chars using dialect-appropriate functions
+    if dialect_name == 'sqlite':
+        # SQLite uses SUBSTR and LENGTH
+        op.execute("""
+            UPDATE gateways
+            SET slug = SUBSTR(slug, 1, 191),
+                url = SUBSTR(url, 1, 191)
+            WHERE LENGTH(slug) > 191 OR LENGTH(url) > 191;
+        """)
+    elif dialect_name == 'postgresql':
+        # PostgreSQL supports LEFT and CHAR_LENGTH
+        op.execute("""
+            UPDATE gateways
+            SET slug = LEFT(slug, 191),
+                url = LEFT(url, 191)
+            WHERE CHAR_LENGTH(slug) > 191 OR CHAR_LENGTH(url) > 191;
+        """)
+    elif dialect_name == 'mysql':
+        # MySQL supports LEFT and CHAR_LENGTH (character-based, not byte-based)
+        op.execute("""
+            UPDATE gateways
+            SET slug = LEFT(slug, 191),
+                url = LEFT(url, 191)
+            WHERE CHAR_LENGTH(slug) > 191 OR CHAR_LENGTH(url) > 191;
+        """)
+    else:
+        # Fallback for other databases
+        op.execute("""
+            UPDATE gateways
+            SET slug = SUBSTR(slug, 1, 191),
+                url = SUBSTR(url, 1, 191)
+            WHERE LENGTH(slug) > 191 OR LENGTH(url) > 191;
+        """)
 
     # Resize columns to String(191)
     op.alter_column(

mcpgateway/db.py

@@ -2430,8 +2430,8 @@ class Gateway(Base):
 
     id: Mapped[str] = mapped_column(String(36), primary_key=True, default=lambda: uuid.uuid4().hex)
     name: Mapped[str] = mapped_column(String(255), nullable=False)
-    slug: Mapped[str] = mapped_column(String(255), nullable=False)
-    url: Mapped[str] = mapped_column(String(767), nullable=False)
+    slug: Mapped[str] = mapped_column(String(191), nullable=False)
+    url: Mapped[str] = mapped_column(String(191), nullable=False)
     description: Mapped[Optional[str]] = mapped_column(Text, nullable=True)
     transport: Mapped[str] = mapped_column(String(20), default="SSE")
     capabilities: Mapped[Dict[str, Any]] = mapped_column(JSON)