Skip to content

server:start: Doppar banner and per-request log with execution time - #346

Open
techmahedy wants to merge 1 commit into
doppar:4.xfrom
techmahedy:techmahedy-4.x
Open

techmahedy wants to merge 1 commit into
doppar:4.xfrom
techmahedy:techmahedy-4.x

Conversation

@techmahedy

@techmahedy techmahedy commented Oct 2, 2026 •

Copy link
Copy Markdown
Member

Pull Request Checklist

Q A
Branch? 4.x
Bug fix? no
New feature? yes
Deprecations? no
Issues -
License MIT

Summary

php pool server:start now prints the Doppar logo with the framework version on startup, and shows one clean line per request: status and request on the left, execution time on the right.

Before, the output was PHP's raw built-in server log, with three lines per connection and no timing:

[Fri Oct  2 21:52:11 2026] 127.0.0.1:40064 Accepted
[Fri Oct  2 21:52:11 2026] 127.0.0.1:40072 [200]: GET /robots.txt
[Fri Oct  2 21:52:11 2026] 127.0.0.1:40072 Closing

After:

    ____
   / __ \____  ____  ____  ____ ______
  / / / / __ \/ __ \/ __ \/ __ `/ ___/
 / /_/ / /_/ / /_/ / /_/ / /_/ / /
/_____/\____/ .___/ .___/\__,_/_/
           /_/   /_/
                          v4.2.0
------------------------------------------------

 SUCCESS  Server started on http://localhost:8000

  Press Ctrl+C to stop the server

  200 GET / ......................................................... 68.46ms
  404 GET /nope ...................................................... 10.14ms
  200 GET /favicon.ico ................................................ 0.10ms

Screenshot

Screenshot from 2026-10-02 22-02-39

Files

  • src/Phaseolies/Console/Commands/Server/ServerStartCommand.php: banner, output parsing, request rendering.
  • src/Phaseolies/Console/Commands/Server/request_logger.stub: new, the prepended per-request script.

Compatibility

  • No public API, config or signature changes. server:start arguments and options are the same.
  • Output is formatted for a terminal. If you scrape server:start stdout, the format changed.
  • Requires nothing new: Symfony\Component\Console\Terminal and OutputFormatter are already dependencies.

Checklist

  • Tests have been added or updated
  • Documentation has been updated if necessary
  • Code follows the project coding standards
  • All tests pass locally

@techmahedy
techmahedy requested a review from rrr63 October 2, 2026 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant