Это плохая практика, чтобы функция конструктора возвращала Promise?

Когда вы выполняете задание в PHP, присваивание возвращает значение, которое было назначено. Это позволяет вам оценить назначение в условных выражениях или циклах или других выражениях.

var_dump ($var = 'test');

будет отображаться результат «test».

В приведенном выше примере кода выполняется присвоение и тест в одной и той же операции, что является довольно короткой стенографией программиста. fgets () возвращает строку из открытого файла или false, если нет строки для возврата. Вы могли бы сделать

while ($buffer !== false) {
    echo $buffer;
    $buffer = fgets($handle, 4096);
}

, но вам, разумеется, пришлось бы установить буфер в значение, отличное от ложного, перед входом в цикл (путем выполнения fgets перед циклом или назначения значения, которое не будет получено отображается, но не является ложным, например пустая строка).