Основной код допишу и сделаю папку в каждом приложении "tests" и там размещу тесты

Спасибо)

Если собираешься пилить проект с кем-то то коротенькое README обязательно

Это дело в процессе

и докер собери, что ль

Как же без этого

Сокеты допишу и надо собирать

1) Это ужасно читается (не читается вообще)
   
Разбей на строки, раскидай по переменным


def _get_image_args(self, validated_data) -> Tuple[str, Dict[str, Any]]:
       """
       :return ("image", {"image": InMemoryUploadedFile <...> or TemporaryUploadedFile <...>})
       """

       return tuple(*{
           field_name: value for field_name, value in validated_data.items()
           if isinstance(value, dict) and self._get_image_args(value)
           or any([isinstance(value, InMemoryUploadedFile), isinstance(value, TemporaryUploadedFile)])
       }.i

2) def _get_user(self) -> Any or None:

Во-первых, Optional[Any]
Во-вторых это неправда
В-третьих, зачем нужны типы, если возращаешь Optinal Any? Шоб було?

3)    def _common_related(self):
       model = self.model
       qs = self.all()

       if getattr(model, 'COMMON_SELECT_RELATED', None):
           qs = qs.select_related(*model.COMMON_SELECT_RELATED)
       if getattr(model, 'COMMON_PREFETCH_RELATED', None):
           qs = qs.prefetch_related(*model.COMMON_PREFETCH_RELATED)
       return qs

   def with_common_related(self):
       return self._common_related()

зачем нужен with_common_related метод?
Мусорный код

почему модель должна знать о префетчах и селектах?
это должно быть в сериалайзерах

4) ChatGroupViewSet(mixins.CreateModelMixin,
                      mixins.ListModelMixin,
                      mixins.DestroyModelMixin,
                      mixins.UpdateModelMixin,
                      viewsets.GenericViewSet)

если я не ошибаюсь, это уже полноценный ModelViewSet
точно ли есть смысл отдельно миксины прописывать?

1) Ну на счёт Any or None:
Видел в описании Any, что Any применятся при возвращении объекта.

2) Не совсем понял, как такую логику (как в моделях) реализовать в сериализоторах, но во всяком случае тогда перенесу во вьюшки

3) Не совсем) Там нет mixins.RetrieveModelMixin, поскольку он там не нужен

На счёт того, что ужасно читается - уже давно думал

а толку от такой типизации?
по идее, надо бы указать Optional[User]

Он может вернуть None

Естественно, поэтому Optional[]

Понял, спасибо)

Ещё что-нибудь?)

2) Не совсем понял, как такую логику (как в моделях) реализовать в сериализоторах, но во всяком случае тогда перенесу во вьюшки

Не надо во вьюшки
У тебя динамический сериализатор

Пишешь префетчи и пре-селекты - здорово
Но их надо держать в том месте где происходит сериализация

У меня в проекте каждый ViewSet имеет по 3-8 сериализаторов, поверь лучше сразу хранить по месту применения