WIP checkpoint: Trust & Safety: Moderation data model + Alembic migration
This commit is contained in:
@@ -69,8 +69,7 @@ def upgrade() -> None:
|
||||
sa.Column(
|
||||
"visibility",
|
||||
postgresql.ENUM(
|
||||
"public", "private", "friends_only", name="world_visibility",
|
||||
create_type=False,
|
||||
"public", "private", "friends_only", name="world_visibility"
|
||||
),
|
||||
nullable=True,
|
||||
),
|
||||
@@ -90,7 +89,7 @@ def upgrade() -> None:
|
||||
"asset_type",
|
||||
postgresql.ENUM(
|
||||
"image", "audio", "video", "document", "model_3d", "other",
|
||||
name="asset_type", create_type=False,
|
||||
name="asset_type",
|
||||
),
|
||||
nullable=False,
|
||||
),
|
||||
|
||||
@@ -0,0 +1,272 @@
|
||||
"""add moderation tables and user roles
|
||||
|
||||
Revision ID: 0003_add_moderation_tables_and_user_roles
|
||||
Revises: 0002_add_date_of_birth_to_users
|
||||
Create Date: 2025-01-02 00:00:00.000000
|
||||
|
||||
"""
|
||||
from typing import Sequence, Union
|
||||
|
||||
from alembic import op
|
||||
import sqlalchemy as sa
|
||||
from sqlalchemy.dialects import postgresql
|
||||
|
||||
# revision identifiers, used by Alembic.
|
||||
revision: str = "0003_add_moderation_tables_and_user_roles"
|
||||
down_revision: Union[str, None] = "0002_add_date_of_birth_to_users"
|
||||
branch_labels: Union[str, Sequence[str], None] = None
|
||||
depends_on: Union[str, Sequence[str], None] = None
|
||||
|
||||
|
||||
def upgrade() -> None:
|
||||
"""Add moderation tables and user role/ban/suspend columns."""
|
||||
# --- ENUM types ---
|
||||
|
||||
user_role = postgresql.ENUM("user", "moderator", "admin", name="user_role")
|
||||
user_role.create(op.get_bind(), checkfirst=True)
|
||||
|
||||
target_type = postgresql.ENUM(
|
||||
"user", "world", "asset", "chat_message", name="target_type"
|
||||
)
|
||||
target_type.create(op.get_bind(), checkfirst=True)
|
||||
|
||||
reason_category = postgresql.ENUM(
|
||||
"harassment",
|
||||
"hate_speech",
|
||||
"sexual_content",
|
||||
"violence",
|
||||
"spam",
|
||||
"self_harm",
|
||||
"other",
|
||||
name="reason_category",
|
||||
)
|
||||
reason_category.create(op.get_bind(), checkfirst=True)
|
||||
|
||||
report_status = postgresql.ENUM(
|
||||
"open", "in_review", "resolved", "dismissed", name="report_status"
|
||||
)
|
||||
report_status.create(op.get_bind(), checkfirst=True)
|
||||
|
||||
resolution = postgresql.ENUM(
|
||||
"action_taken", "dismissed", "no_violation", name="resolution"
|
||||
)
|
||||
resolution.create(op.get_bind(), checkfirst=True)
|
||||
|
||||
action_type = postgresql.ENUM(
|
||||
"warn", "suspend", "ban", "restrict", "remove", "restore",
|
||||
name="action_type",
|
||||
)
|
||||
action_type.create(op.get_bind(), checkfirst=True)
|
||||
|
||||
appeal_status = postgresql.ENUM(
|
||||
"pending", "approved", "rejected", name="appeal_status"
|
||||
)
|
||||
appeal_status.create(op.get_bind(), checkfirst=True)
|
||||
|
||||
# --- Add columns to users table ---
|
||||
|
||||
op.add_column(
|
||||
"users",
|
||||
sa.Column(
|
||||
"role",
|
||||
postgresql.ENUM("user", "moderator", "admin", name="user_role"),
|
||||
nullable=False,
|
||||
server_default="user",
|
||||
),
|
||||
)
|
||||
op.add_column("users", sa.Column("banned_until", sa.DateTime(timezone=True), nullable=True))
|
||||
op.add_column("users", sa.Column("suspended_until", sa.DateTime(timezone=True), nullable=True))
|
||||
|
||||
# --- reports table ---
|
||||
|
||||
op.create_table(
|
||||
"reports",
|
||||
sa.Column("id", sa.Integer(), primary_key=True),
|
||||
sa.Column("reporter_id", sa.Integer(), nullable=False),
|
||||
sa.Column(
|
||||
"target_type",
|
||||
postgresql.ENUM(
|
||||
"user", "world", "asset", "chat_message", name="target_type"
|
||||
),
|
||||
nullable=False,
|
||||
),
|
||||
sa.Column("target_id", sa.String(length=64), nullable=False),
|
||||
sa.Column(
|
||||
"reason_category",
|
||||
postgresql.ENUM(
|
||||
"harassment",
|
||||
"hate_speech",
|
||||
"sexual_content",
|
||||
"violence",
|
||||
"spam",
|
||||
"self_harm",
|
||||
"other",
|
||||
name="reason_category",
|
||||
),
|
||||
nullable=False,
|
||||
),
|
||||
sa.Column("details", sa.Text(), nullable=True),
|
||||
sa.Column(
|
||||
"status",
|
||||
postgresql.ENUM(
|
||||
"open", "in_review", "resolved", "dismissed", name="report_status"
|
||||
),
|
||||
nullable=False,
|
||||
server_default="open",
|
||||
),
|
||||
sa.Column("resolved_by", sa.Integer(), nullable=True),
|
||||
sa.Column(
|
||||
"resolution",
|
||||
postgresql.ENUM(
|
||||
"action_taken", "dismissed", "no_violation", name="resolution"
|
||||
),
|
||||
nullable=True,
|
||||
),
|
||||
sa.Column("resolved_at", sa.DateTime(timezone=True), nullable=True),
|
||||
sa.Column(
|
||||
"created_at",
|
||||
sa.DateTime(timezone=True),
|
||||
server_default=sa.func.now(),
|
||||
),
|
||||
sa.Column(
|
||||
"updated_at",
|
||||
sa.DateTime(timezone=True),
|
||||
server_default=sa.func.now(),
|
||||
),
|
||||
sa.ForeignKeyConstraint(["reporter_id"], ["users.id"], ondelete="CASCADE"),
|
||||
sa.ForeignKeyConstraint(["resolved_by"], ["users.id"], ondelete="SET NULL"),
|
||||
)
|
||||
op.create_index(op.f("ix_reports_id"), "reports", ["id"], unique=False)
|
||||
# Index on reports.status for filtering by status
|
||||
op.create_index(op.f("ix_reports_status"), "reports", ["status"], unique=False)
|
||||
# Composite index for duplicate detection on target
|
||||
op.create_index(
|
||||
op.f("ix_reports_target"), "reports", ["target_type", "target_id"], unique=False
|
||||
)
|
||||
|
||||
# --- moderation_actions table ---
|
||||
|
||||
op.create_table(
|
||||
"moderation_actions",
|
||||
sa.Column("id", sa.Integer(), primary_key=True),
|
||||
sa.Column("moderator_id", sa.Integer(), nullable=False),
|
||||
sa.Column("target_user_id", sa.Integer(), nullable=False),
|
||||
sa.Column(
|
||||
"target_type",
|
||||
postgresql.ENUM(
|
||||
"user", "world", "asset", "chat_message", name="target_type"
|
||||
),
|
||||
nullable=False,
|
||||
),
|
||||
sa.Column("target_id", sa.String(length=64), nullable=False),
|
||||
sa.Column(
|
||||
"action",
|
||||
postgresql.ENUM(
|
||||
"warn", "suspend", "ban", "restrict", "remove", "restore",
|
||||
name="action_type",
|
||||
),
|
||||
nullable=False,
|
||||
),
|
||||
sa.Column("reason", sa.Text(), nullable=True),
|
||||
sa.Column("report_id", sa.Integer(), nullable=True),
|
||||
sa.Column("expires_at", sa.DateTime(timezone=True), nullable=True),
|
||||
sa.Column("revoked_at", sa.DateTime(timezone=True), nullable=True),
|
||||
sa.Column(
|
||||
"created_at",
|
||||
sa.DateTime(timezone=True),
|
||||
server_default=sa.func.now(),
|
||||
),
|
||||
sa.ForeignKeyConstraint(["moderator_id"], ["users.id"], ondelete="SET NULL"),
|
||||
sa.ForeignKeyConstraint(["target_user_id"], ["users.id"], ondelete="CASCADE"),
|
||||
sa.ForeignKeyConstraint(["report_id"], ["reports.id"], ondelete="SET NULL"),
|
||||
)
|
||||
op.create_index(op.f("ix_moderation_actions_id"), "moderation_actions", ["id"], unique=False)
|
||||
# Index on target_user_id for looking up actions against a user
|
||||
op.create_index(
|
||||
op.f("ix_moderation_actions_target_user_id"),
|
||||
"moderation_actions",
|
||||
["target_user_id"],
|
||||
unique=False,
|
||||
)
|
||||
# Index on revoked_at for finding expired/revoked actions
|
||||
op.create_index(
|
||||
op.f("ix_moderation_actions_revoked_at"),
|
||||
"moderation_actions",
|
||||
["revoked_at"],
|
||||
unique=False,
|
||||
)
|
||||
|
||||
# --- appeals table ---
|
||||
|
||||
op.create_table(
|
||||
"appeals",
|
||||
sa.Column("id", sa.Integer(), primary_key=True),
|
||||
sa.Column(
|
||||
"moderation_action_id", sa.Integer(), nullable=False
|
||||
),
|
||||
sa.Column("user_id", sa.Integer(), nullable=False),
|
||||
sa.Column("reason", sa.Text(), nullable=False),
|
||||
sa.Column(
|
||||
"status",
|
||||
postgresql.ENUM(
|
||||
"pending", "approved", "rejected", name="appeal_status"
|
||||
),
|
||||
nullable=False,
|
||||
server_default="pending",
|
||||
),
|
||||
sa.Column("reviewed_by", sa.Integer(), nullable=True),
|
||||
sa.Column("reviewed_at", sa.DateTime(timezone=True), nullable=True),
|
||||
sa.Column(
|
||||
"created_at",
|
||||
sa.DateTime(timezone=True),
|
||||
server_default=sa.func.now(),
|
||||
),
|
||||
sa.ForeignKeyConstraint(
|
||||
["moderation_action_id"],
|
||||
["moderation_actions.id"],
|
||||
ondelete="CASCADE",
|
||||
),
|
||||
sa.ForeignKeyConstraint(["user_id"], ["users.id"], ondelete="CASCADE"),
|
||||
sa.ForeignKeyConstraint(["reviewed_by"], ["users.id"], ondelete="SET NULL"),
|
||||
)
|
||||
op.create_index(op.f("ix_appeals_id"), "appeals", ["id"], unique=False)
|
||||
# Index on appeals.status for filtering by status
|
||||
op.create_index(op.f("ix_appeals_status"), "appeals", ["status"], unique=False)
|
||||
# Index on moderation_action_id for looking up appeals against an action
|
||||
op.create_index(
|
||||
op.f("ix_appeals_moderation_action_id"),
|
||||
"appeals",
|
||||
["moderation_action_id"],
|
||||
unique=False,
|
||||
)
|
||||
|
||||
|
||||
def downgrade() -> None:
|
||||
"""Downgrade database schema."""
|
||||
op.drop_index(op.f("ix_appeals_moderation_action_id"), table_name="appeals")
|
||||
op.drop_index(op.f("ix_appeals_status"), table_name="appeals")
|
||||
op.drop_index(op.f("ix_appeals_id"), table_name="appeals")
|
||||
op.drop_table("appeals")
|
||||
|
||||
op.drop_index(op.f("ix_moderation_actions_revoked_at"), table_name="moderation_actions")
|
||||
op.drop_index(op.f("ix_moderation_actions_target_user_id"), table_name="moderation_actions")
|
||||
op.drop_index(op.f("ix_moderation_actions_id"), table_name="moderation_actions")
|
||||
op.drop_table("moderation_actions")
|
||||
|
||||
op.drop_index(op.f("ix_reports_target"), table_name="reports")
|
||||
op.drop_index(op.f("ix_reports_status"), table_name="reports")
|
||||
op.drop_index(op.f("ix_reports_id"), table_name="reports")
|
||||
op.drop_table("reports")
|
||||
|
||||
op.drop_column("users", "suspended_until")
|
||||
op.drop_column("users", "banned_until")
|
||||
op.drop_column("users", "role")
|
||||
|
||||
# Drop ENUM types
|
||||
postgresql.ENUM(name="appeal_status").drop(op.get_bind(), checkfirst=True)
|
||||
postgresql.ENUM(name="action_type").drop(op.get_bind(), checkfirst=True)
|
||||
postgresql.ENUM(name="resolution").drop(op.get_bind(), checkfirst=True)
|
||||
postgresql.ENUM(name="report_status").drop(op.get_bind(), checkfirst=True)
|
||||
postgresql.ENUM(name="reason_category").drop(op.get_bind(), checkfirst=True)
|
||||
postgresql.ENUM(name="target_type").drop(op.get_bind(), checkfirst=True)
|
||||
postgresql.ENUM(name="user_role").drop(op.get_bind(), checkfirst=True)
|
||||
Reference in New Issue
Block a user