ну хендлер то ладно

мидлварь лучше бы оставить

А зачем он,

Для каких целей?

чтобы пайплайны совмещать

Диспетчер - это самый верх, он вызывается непосредственно Application, с чем ты его хочешь совмещать?

переименуй его в PipeLine и ничего не изменится

Чета ты не догоняешь по ходу) После того как Application выполнит метод dispatch() диспетчера, экземпляр диспетчера больше никому не будет доступен.

а он и не должен быть никому доступен

Ну так зачем тебе его метод process()?

это метод интерфейса мидлвари

Там нет сейчас интерфейса мидлтвари у него, я убрал.

Именно так :) Вы норм штуки предлагаете. Не надо каждый раз спрашивать. Особенно если pull request занимает 10 минут.

И на совместимость пока относительно пофиг. Главное чтобы демо не ломалось больше чем на пару часов.

Так что там с process делаем? Он нужен только для возможности совмещения двух отдельных приложений, я не знаю, буде ли мы такой кейс предлагать пользователям фреймворка.

В общем:

process() оставить, добавить MiddlewareInterface
Выбрасывает исключение, если стек пуст

dispatch() переименовать на handle(), вернув интерфейс RequestHandlerInterface
выбрасывает исключение, если нет $nextHandler

addMiddleware() переименовать в add()
из конструктора убрать массив мидлварей, он только путает

зачем нужен MiddlewareInterface? чтобы помещать один трубопровод в другой, достраивать трубопровод из отрезков
Откуда они берутся?
У меня такой юзкейс: есть два приложения, отличающиеся доменом на 3-м уровне. Для каждого из них отличается низ стека пайплайна, а начало общее. Поэтому в зависимости от домена берётся нужный пайплайн и к нему достраивается пайплайн общий

зачем нужен RequestHandlerInterface? Да потому что по факту диспетчер это и есть RequestHandlerInterface - ты туда даёшь запрос, получаешь ответ

Все неправильно, садись, два😏

ну скрываем

Ок, тогда пока скрываю, из конструктора мидлтвари убираю.