Skip to content

refactor: remove unused _logger fields from handlers#269

Merged
ziagham merged 1 commit into
cortexiumlabs:mainfrom
yahyasaqban-lab:fix/remove-unused-logger-fields
May 17, 2026
Merged

refactor: remove unused _logger fields from handlers#269
ziagham merged 1 commit into
cortexiumlabs:mainfrom
yahyasaqban-lab:fix/remove-unused-logger-fields

Conversation

@yahyasaqban-lab
Copy link
Copy Markdown
Contributor

Summary

Removed unused ILogger fields and their constructor injections from three handlers:

  • CreateAgentHandler - removed _logger field and constructor param
  • GenerateTokenHandler - removed _logger field and constructor param
  • GetAllPendingHandler - removed _logger field and constructor param

Also removed the now-unused Microsoft.Extensions.Logging import from each file.

Test plan

  • Verified each handler compiles without the logger
  • Run existing tests

Fixes #265, #266, #267

🤖 Generated with Claude Code

Removed unused ILogger fields and their constructor injections from:
- CreateAgentHandler
- GenerateTokenHandler
- GetAllPendingHandler

Fixes cortexiumlabs#265, cortexiumlabs#266, cortexiumlabs#267

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@ziagham ziagham merged commit fd8fdc3 into cortexiumlabs:main May 17, 2026
2 of 3 checks passed
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.

Remove unused _logge` field from CreateAgentHandler

2 participants