Skip to content

Conversation

@kzu
Copy link
Member

@kzu kzu commented Jul 16, 2025

Clients will typically invoke ListTools right after client initialization is complete. We don't attempt to run this code as a notification handler because those don't have access to the server instance and therefore are not very useful.

Running just before tools list is a good spot for post-initialized and before things can actually start running.

Note that doing elicitation in this context won't work, but other notifications/logging to the client should be perfectly functional at this stage.

Clients will typically invoke ListTools right after client initialization is complete. We don't attempt to run this code as a notification handler because those don't have access to the server instance and therefore are not very useful.

Running just before tools list is a good spot for post-initialized and before things can actually start running.

Note that doing elicitation in this context won't work, but other notifications/logging to the client should be perfectly functional at this stage.
@kzu kzu added the enhancement New feature or request label Jul 16, 2025
@kzu kzu enabled auto-merge (rebase) July 16, 2025 23:07
@kzu kzu merged commit 01e588e into main Jul 16, 2025
7 checks passed
@kzu kzu deleted the dev/initializer branch July 16, 2025 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants