Fix DB models cross-references

Author: dolfinus

syncmaster/db/models/connection.py

@@ -41,7 +41,7 @@ class Connection(Base, ResourceMixin, TimestampMixin):
     type: Mapped[ConnectionType] = mapped_column(ChoiceType(ConnectionType, impl=String(32)), nullable=False)
     data: Mapped[dict[str, Any]] = mapped_column(JSON, nullable=False, default={})
 
-    group: Mapped[Group] = relationship("Group")
+    group: Mapped[Group] = relationship()
 
     search_vector: Mapped[str] = mapped_column(
         TSVECTOR,

syncmaster/db/models/group.py

@@ -71,7 +71,7 @@ class Group(Base, TimestampMixin):
     description: Mapped[str] = mapped_column(String(512), nullable=False, default="")
     owner_id: Mapped[int] = mapped_column(ForeignKey("user.id", ondelete="CASCADE"), nullable=False, index=True)
 
-    owner: Mapped[User] = relationship(User)
+    owner: Mapped[User] = relationship(viewonly=True)
     search_vector: Mapped[str] = mapped_column(
         TSVECTOR,
         Computed(

syncmaster/db/models/queue.py

@@ -15,7 +15,7 @@ class Queue(Base, ResourceMixin, TimestampMixin):
     name: Mapped[str] = mapped_column(String(128), nullable=False)
     slug: Mapped[str] = mapped_column(String(256), nullable=False, unique=True)
 
-    group: Mapped[Group] = relationship(Group)
+    group: Mapped[Group] = relationship(viewonly=True)
 
     search_vector: Mapped[str] = mapped_column(
         TSVECTOR,

syncmaster/db/models/run.py

@@ -12,7 +12,6 @@
 
 from syncmaster.db.mixins import TimestampMixin
 from syncmaster.db.models.base import Base
-from syncmaster.db.models.group import Group
 from syncmaster.db.models.transfer import Transfer
 
 
@@ -66,8 +65,7 @@ class Run(Base, TimestampMixin):
     log_url: Mapped[str] = mapped_column(String(512), nullable=True)
     transfer_dump: Mapped[dict[str, Any]] = mapped_column(JSON, nullable=False, default={})
 
-    transfer: Mapped[Transfer] = relationship(Transfer)
-    group: Mapped[Group] = relationship(Group)
+    transfer: Mapped[Transfer] = relationship(viewonly=True)
 
     def __repr__(self):
         return (

syncmaster/db/models/transfer.py

@@ -2,7 +2,7 @@
 # SPDX-License-Identifier: Apache-2.0
 from __future__ import annotations
 
-from typing import TYPE_CHECKING, Any
+from typing import Any
 
 from sqlalchemy import (
     JSON,
@@ -19,11 +19,9 @@
 
 from syncmaster.db.mixins import ResourceMixin, TimestampMixin
 from syncmaster.db.models.base import Base
+from syncmaster.db.models.connection import Connection
 from syncmaster.db.models.group import Group
-
-if TYPE_CHECKING:
-    from syncmaster.db.models.connection import Connection
-    from syncmaster.db.models.queue import Queue
+from syncmaster.db.models.queue import Queue
 
 
 class Transfer(
@@ -61,10 +59,10 @@ class Transfer(
         nullable=False,
     )
 
-    group: Mapped[Group] = relationship(Group)
-    source_connection: Mapped[Connection] = relationship(foreign_keys=source_connection_id)
-    target_connection: Mapped[Connection] = relationship(foreign_keys=target_connection_id)
-    queue: Mapped[Queue] = relationship(Queue)
+    group: Mapped[Group] = relationship(viewonly=True)
+    source_connection: Mapped[Connection] = relationship(foreign_keys=source_connection_id, viewonly=True)
+    target_connection: Mapped[Connection] = relationship(foreign_keys=target_connection_id, viewonly=True)
+    queue: Mapped[Queue] = relationship(viewonly=True)
 
     search_vector: Mapped[str] = mapped_column(
         TSVECTOR,