fix: update migration to follow repo patterns and use /api/forms prefix

- Use postgresql.UUID(as_uuid=True) for UUID columns
- Use server_default=sa.text('CURRENT_TIMESTAMP') for timestamps
- Use server_default=sa.text("'pending'") for string defaults
- Add explicit FK constraint name
- Change router prefix from /api/v1/forms to /api/forms

Co-authored-by: openhands <openhands@all-hands.dev>

enterprise/migrations/versions/102_create_form_submission_table.py

@@ -3,6 +3,7 @@
 Revision ID: 102
 Revises: 101
 Create Date: 2025-03-19 00:00:00.000000
+
 """
 
 from typing import Sequence, Union
@@ -26,19 +27,35 @@ def upgrade() -> None:
     """
     op.create_table(
         'form_submission',
-        sa.Column('id', sa.UUID(), nullable=False),
+        sa.Column('id', postgresql.UUID(as_uuid=True), nullable=False),
         sa.Column('form_type', sa.String(50), nullable=False),
         sa.Column('answers', postgresql.JSONB(astext_type=sa.Text()), nullable=False),
-        sa.Column('status', sa.String(20), nullable=False, server_default='pending'),
-        sa.Column('user_id', sa.UUID(), nullable=True),
         sa.Column(
-            'created_at', sa.DateTime(), nullable=False, server_default=sa.text('now()')
+            'status',
+            sa.String(20),
+            nullable=False,
+            server_default=sa.text("'pending'"),
+        ),
+        sa.Column('user_id', postgresql.UUID(as_uuid=True), nullable=True),
+        sa.Column(
+            'created_at',
+            sa.DateTime(),
+            nullable=False,
+            server_default=sa.text('CURRENT_TIMESTAMP'),
         ),
         sa.Column(
-            'updated_at', sa.DateTime(), nullable=False, server_default=sa.text('now()')
+            'updated_at',
+            sa.DateTime(),
+            nullable=False,
+            server_default=sa.text('CURRENT_TIMESTAMP'),
         ),
         sa.PrimaryKeyConstraint('id'),
-        sa.ForeignKeyConstraint(['user_id'], ['user.id'], ondelete='SET NULL'),
+        sa.ForeignKeyConstraint(
+            ['user_id'],
+            ['user.id'],
+            name='form_submission_user_fkey',
+            ondelete='SET NULL',
+        ),
     )
     op.create_index(
         op.f('ix_form_submission_form_type'),

enterprise/server/middleware.py

@@ -168,7 +168,7 @@ class SetAuthCookieMiddleware:
             '/oauth/device/authorize',
             '/oauth/device/token',
             '/api/v1/web-client/config',
-            '/api/v1/forms/submit',
+            '/api/forms/submit',
         )
         if path in ignore_paths:
             return False

enterprise/server/routes/form_submission.py

@@ -17,7 +17,7 @@ from storage.form_submission import FormSubmission
 from openhands.core.logger import openhands_logger as logger
 from openhands.server.user_auth.user_auth import UserAuth
 
-router = APIRouter(prefix='/api/v1/forms', tags=['forms'])
+router = APIRouter(prefix='/api/forms', tags=['forms'])
 
 
 class FormSubmissionRequest(BaseModel):