Skip to content

B4 - ADR - error responder, application, emitter, provider, kernel#17376

Merged
niden merged 8 commits into
CP-17341-add-adr-implementationfrom
CP-17341-b4-app
Jul 18, 2026
Merged

B4 - ADR - error responder, application, emitter, provider, kernel#17376
niden merged 8 commits into
CP-17341-add-adr-implementationfrom
CP-17341-b4-app

Conversation

@niden

@niden niden commented Jul 18, 2026

Copy link
Copy Markdown
Member
  • Phalcon\ADR\ErrorResponder - maps a thrown exception to a response through the responder chain (exact->ancestor status match, generic client message, full log + correlation ref)
  • Phalcon\Contracts\ADR\Application + Phalcon\ADR\Application - routes, applies route attributes, dispatches the action, fires application:beforeHandle/afterHandle, falls back to the error responder
  • Phalcon\Contracts\ADR\Emitter\Emitter + Phalcon\ADR\Emitter\SapiEmitter - sends the response via SAPI (guards headers-sent); adds HeadersAlreadySent / OutputAlreadySent
  • Phalcon\ADR\Container\AdrProvider - registers the ADR seams (request, response, logger, responder, dispatcher, router, emitter); used instead of Phalcon\Container\Provider\Web
  • Phalcon\Contracts\ADR\Kernel\Kernel + Phalcon\ADR\Kernel\AbstractHttpKernel + HttpKernel - boots the container, handles the request, emits the response, returns an exit code
  • Phalcon\ADR\Middleware\RequestIdMiddleware - missing use for AttributeBag
  • Phalcon\ADR\Payload\Payload - contract alias renamed to PayloadContract

Refs #17341

niden added 8 commits July 17, 2026 21:01
Assisted-by: Claude Code
Assisted-by: Claude Code
Assisted-by: Claude Code
Assisted-by: Claude Code
Assisted-by: Claude Code
Assisted-by: Claude Code
Assisted-by: Claude Code
@niden
niden merged commit 60dc7c5 into CP-17341-add-adr-implementation Jul 18, 2026
0 of 30 checks passed
@niden
niden deleted the CP-17341-b4-app branch July 18, 2026 02:15
@niden niden changed the title ADR - error responder, application, emitter, provider, kernel B4 - ADR - error responder, application, emitter, provider, kernel Jul 18, 2026
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