$sqlReader->withFilter(new SqlFilter(new Criteria()))->withLimit(100)

фильтр знает критерии и умеет их применять

ридер знает что фильтр умеет что то применять(build)

Проблема только в том, что SqlReader не сможет работать ни с каким фильтром кромк SqlFilter
Значит надо будет делать проверку в SqlReader

public function read(): iterable
{
    // ...
    if ($this->filter instanceof SqlFilter) {
        $condition = $this->filter->build();
    }
    // ...
}

Если зависимость от SqlFilter уж есть, не проще ли для использования бует все-таки сделать
$sqlReader->withCriteria(new Criteria())->withLimit(100)
И в SqlReader

public function read(): iterable
{
    // ...
    if ($this->criteria !== null) {
        $condition = (new SqlFilter($this->criteria))->build();
    }
    // ...
}

вероятно

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

а withFilter будет его применять

хотя тоже самое получится

<?php

use yii\db\Query;

interface FilterBuilder
{
   public function addEquals(string $field, $value);
}

class SqlFilterBuild implements FilterBuilder
{
   /
    * @var Query
    */
   private $query;

   public function __construct(Query $query)
   {
       $this->query = $query;
   }

   public function addEquals(string $field, $value)
   {
       $this->query->andWhere([$field => $value]);
   }

   public function getProduct() : Query
   {
       return $this->query;
   }
}

interface Criteria
{
   public function buildFilter(FilterBuilder $builder);
}

class EqualsCriteria implements Criteria
{
   /
    * @var string
    */
   private $field;
   private $value;

   public function __construct(string $field, $value)
   {
       $this->field = $field;
       $this->value = $value;
   }


   public function buildFilter(FilterBuilder $builder)
   {
       $builder->addEquals($this->field, $this->value);
   }
}

смысл в том, чтобы сделать Filter, уж коли он специфичен для Reader'a, ещё и мостом. Методы вроде addFilter в Reader'e будут нарушать LSP если окажутся в интерфейсе, такую зависимость можно добавлять только в конструктор

Довёл https://github.com/yiisoft/data/pull/3 до рабочего состояния. Примеры в readme. Как вам?

огонь. Фильтр крутой получился.

что осталось? чем помочь?

оценить, норм ли

нет ли косяков

Ок, смотреть только с точки зрения архитектуры или нейминг тоже?

всё что покажется не очень

буду рад любым комментам

Ок

Просмотрел дата ридер. Подскажите правильно ли я понял. Для работы с базой данных надо будет писать свой ридер и переопределять методы ридера, чтоб в зависимости от фильтров и сортировки, подставлять условия в запрос, и в методе read вызывать метод all() в случае использования Актив Рекорд?