[quote Calabonga]skyANACalabongaskyANA,

Суть репозитория в том, что он скрывает за собой хранилище.
То есть это может быть и SQL Server, и MongoDB, и Couchbase, и сторонний сервис, и вообще несколько хранилишь последовательно, или параллельно.

У Вас же это выходит не репозиторий, а обёртка над EF.

Репозиторий - как абстракция на доступ к БД (DAL). Его задача отвязаться от конкретной БД. То есть вашему репозиторию должно быть абсолютно фиалетово, к какой базе данных цепляться (MS SQL, PostgreSQL, SqlLite, Oracle, InMemory, etc). Таким образом, EntityFramework делает тоже самое только на основании набора Provider'ов. Именно поэтому я и говорю, что ваша реализация просто дублирует функционал, который уже делает EntityFramework. И я считаю, что в этом очень мало смысла. Поэтому я в свой репозиторий привнес немного больше обязанностей, как то: Mapping, Pagination and Secutiry (в частности, Row Level Security).
Вы не улавливаете сути. Не только БД. Конкретная реализация репозитория вполне может работать со сторонним сервисом.
Или не только с SQL БД, но и с MongoDB. Или с Key/Value storage: Couchbase, Redis.

Можно заинжектить такую реализацию IRepository, что данные сначала будут искаться в локальной памяти, потом в распределённом кэше, в БД и если нигде нет, то запрашивать сторонний сервис, класть в БД, распределённый кэш и память

И при этом принцип единственности ответственности не будет нарушаться