Skip to content

Expose the server logger through route dependency injection - #11

Merged
DrewImm merged 2 commits into
mainfrom
copilot/add-log-instance-to-hidi-container
Oct 10, 2026
Merged

DrewImm merged 2 commits into
mainfrom
copilot/add-log-instance-to-hidi-container

Conversation

Copilot AI commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

Routers and endpoints need access to the configured server logger through Hidi’s dependency hierarchy. Register the logger in the server container so route containers inherit it.

  • Container registration: Register the configured logger under log in the server’s root container.

  • Route access: Routers and endpoints can resolve the inherited instance:

    const log = this.container.get('log');
  • Regression coverage: Verify the router and endpoint resolve the same logger instance configured on the server.

Co-authored-by: DrewImm <30954849+DrewImm@users.noreply.github.com>
Copilot AI changed the title [WIP] Add log instance to hidi container for routers and endpoints Expose the server logger through route dependency injection Oct 10, 2026
Copilot AI requested a review from DrewImm October 10, 2026 20:39
@DrewImm
DrewImm marked this pull request as ready for review October 10, 2026 20:48
@DrewImm
DrewImm merged commit b946e8b into main Oct 10, 2026
2 checks passed
@DrewImm
DrewImm deleted the copilot/add-log-instance-to-hidi-container branch October 10, 2026 20:48
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.

Add log instance to hidi container so it can be inherited by routers & endpoints

2 participants