Skip to content

Migrate to plumcp#355

Open
ericdallo wants to merge 7 commits intomasterfrom
plumcp
Open

Migrate to plumcp#355
ericdallo wants to merge 7 commits intomasterfrom
plumcp

Conversation

@ericdallo
Copy link
Member

Fixes #316

ericdallo and others added 6 commits March 11, 2026 18:29
plumcp's default list_changed notification handlers (fetch-prompts,
fetch-resources, fetch-tools) fire immediately when the server sends
notifications before the initialize handshake completes. Since MCP
servers ignore requests before initialize, these handlers block forever
waiting for responses that never come, preventing the client from
completing initialization.

Override the three list_changed handlers with guarded versions that
only dispatch after initialize-and-notify! succeeds.

🤖 Generated with [eca](https://eca.dev)

Co-Authored-By: eca <noreply@eca.dev>
- Remove flushing-stdio-transport wrapper (flush bug fixed in beta3)
- Replace manual initialized? atom with wrap-initialized-check
- Exclude plumcp .cljc sources from uberjar to fix AOT classloader bug
  where DynamicClassLoader re-evaluates defprotocol, creating incompatible
  interface classes vs AppClassLoader-loaded AOT reify classes

🤖 Generated with [eca](https://eca.dev)

Co-Authored-By: eca <noreply@eca.dev>
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.

Integrate with plumcp for MCP functionality

1 participant