Merge pull request #9 from pntech-dev/hotfix/department-deletion

fix(api): resolve critical bugs affecting API functionality

Author: pntech-dev

alembic/versions/ae2f6293ca2d_add_cascade_delete_to_document_files.py

@@ -0,0 +1,34 @@
+"""Add cascade delete to document_files
+
+Revision ID: ae2f6293ca2d
+Revises: d94fac64bccf
+Create Date: 2026-02-26 15:55:23.114743
+
+"""
+from typing import Sequence, Union
+
+from alembic import op
+import sqlalchemy as sa
+
+
+# revision identifiers, used by Alembic.
+revision: str = 'ae2f6293ca2d'
+down_revision: Union[str, Sequence[str], None] = 'd94fac64bccf'
+branch_labels: Union[str, Sequence[str], None] = None
+depends_on: Union[str, Sequence[str], None] = None
+
+
+def upgrade() -> None:
+    """Upgrade schema."""
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_constraint(op.f('document_files_document_id_fkey'), 'document_files', type_='foreignkey')
+    op.create_foreign_key(None, 'document_files', 'documents', ['document_id'], ['id'], ondelete='CASCADE')
+    # ### end Alembic commands ###
+
+
+def downgrade() -> None:
+    """Downgrade schema."""
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_constraint(None, 'document_files', type_='foreignkey')
+    op.create_foreign_key(op.f('document_files_document_id_fkey'), 'document_files', 'documents', ['document_id'], ['id'])
+    # ### end Alembic commands ###

alembic/versions/b3c1d8e9f012_add_trgm_search_indexes.py

@@ -0,0 +1,44 @@
+"""Add pg_trgm search indexes for fast ILIKE search
+
+Revision ID: b3c1d8e9f012
+Revises: ae2f6293ca2d
+Create Date: 2026-02-26 17:00:00.000000
+
+"""
+from typing import Sequence, Union
+
+from alembic import op
+import sqlalchemy as sa
+
+
+# revision identifiers, used by Alembic.
+revision: str = 'b3c1d8e9f012'
+down_revision: Union[str, Sequence[str], None] = 'ae2f6293ca2d'
+branch_labels: Union[str, Sequence[str], None] = None
+depends_on: Union[str, Sequence[str], None] = None
+
+
+def upgrade() -> None:
+    # Enable pg_trgm extension (required for GIN trigram indexes)
+    op.execute("CREATE EXTENSION IF NOT EXISTS pg_trgm")
+
+    # GIN trigram indexes — make ILIKE '%query%' use index instead of full scan
+    op.execute("CREATE INDEX IF NOT EXISTS idx_documents_code_trgm ON documents USING GIN (code gin_trgm_ops)")
+    op.execute("CREATE INDEX IF NOT EXISTS idx_documents_name_trgm ON documents USING GIN (name gin_trgm_ops)")
+    op.execute("CREATE INDEX IF NOT EXISTS idx_pages_designation_trgm ON pages USING GIN (designation gin_trgm_ops)")
+    op.execute("CREATE INDEX IF NOT EXISTS idx_pages_name_trgm ON pages USING GIN (name gin_trgm_ops)")
+
+    # Standard B-tree indexes for joins and filtering
+    op.execute("CREATE INDEX IF NOT EXISTS idx_documents_category_id ON documents (category_id)")
+    op.execute("CREATE INDEX IF NOT EXISTS idx_pages_document_id ON pages (document_id)")
+    op.execute("CREATE INDEX IF NOT EXISTS idx_categories_group_id ON categories (group_id)")
+
+
+def downgrade() -> None:
+    op.execute("DROP INDEX IF EXISTS idx_documents_code_trgm")
+    op.execute("DROP INDEX IF EXISTS idx_documents_name_trgm")
+    op.execute("DROP INDEX IF EXISTS idx_pages_designation_trgm")
+    op.execute("DROP INDEX IF EXISTS idx_pages_name_trgm")
+    op.execute("DROP INDEX IF EXISTS idx_documents_category_id")
+    op.execute("DROP INDEX IF EXISTS idx_pages_document_id")
+    op.execute("DROP INDEX IF EXISTS idx_categories_group_id")

alembic/versions/d94fac64bccf_add_cascade_delete_to_document_tags.py

@@ -0,0 +1,38 @@
+"""Add cascade delete to document_tags
+
+Revision ID: d94fac64bccf
+Revises: 1f43c69cbade
+Create Date: 2026-02-26 15:52:44.282113
+
+"""
+from typing import Sequence, Union
+
+from alembic import op
+import sqlalchemy as sa
+
+
+# revision identifiers, used by Alembic.
+revision: str = 'd94fac64bccf'
+down_revision: Union[str, Sequence[str], None] = '1f43c69cbade'
+branch_labels: Union[str, Sequence[str], None] = None
+depends_on: Union[str, Sequence[str], None] = None
+
+
+def upgrade() -> None:
+    """Upgrade schema."""
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_constraint(op.f('document_tags_tag_id_fkey'), 'document_tags', type_='foreignkey')
+    op.drop_constraint(op.f('document_tags_document_id_fkey'), 'document_tags', type_='foreignkey')
+    op.create_foreign_key(None, 'document_tags', 'documents', ['document_id'], ['id'], ondelete='CASCADE')
+    op.create_foreign_key(None, 'document_tags', 'tags', ['tag_id'], ['id'], ondelete='CASCADE')
+    # ### end Alembic commands ###
+
+
+def downgrade() -> None:
+    """Downgrade schema."""
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_constraint(None, 'document_tags', type_='foreignkey')
+    op.drop_constraint(None, 'document_tags', type_='foreignkey')
+    op.create_foreign_key(op.f('document_tags_document_id_fkey'), 'document_tags', 'documents', ['document_id'], ['id'])
+    op.create_foreign_key(op.f('document_tags_tag_id_fkey'), 'document_tags', 'tags', ['tag_id'], ['id'])
+    # ### end Alembic commands ###

models/document_file_model.py

@@ -8,7 +8,7 @@ class DocumentFile(Base):
     __tablename__ = 'document_files'
 
     id = Column(Integer, primary_key=True)
-    document_id = Column(Integer, ForeignKey('documents.id'), nullable=False)
+    document_id = Column(Integer, ForeignKey('documents.id', ondelete="CASCADE"), nullable=False)
 
     file_path = Column(String, nullable=False) # Path (Key) in MinIO
     filename = Column(String, nullable=False)  # Original filename

models/document_model.py

@@ -6,8 +6,8 @@
 document_tags = Table(
     'document_tags',
     Base.metadata,
-    Column('document_id', Integer, ForeignKey('documents.id'), primary_key=True),
-    Column('tag_id', Integer, ForeignKey('tags.id'), primary_key=True)
+    Column('document_id', Integer, ForeignKey('documents.id', ondelete="CASCADE"), primary_key=True),
+    Column('tag_id', Integer, ForeignKey('tags.id', ondelete="CASCADE"), primary_key=True)
 )