Skip to content

Conversation

ashwin-ant
Copy link
Member

Mostly copying from how the Python SDK does it. Not actually seeing how we'd infer what notification types a server offers though.

#11

@ashwin-ant ashwin-ant marked this pull request as ready for review October 16, 2024 00:23
Copy link
Member

@jspahrsummers jspahrsummers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not actually seeing how we'd infer what notification types a server offers though.

I think we'd have to get this information from the caller.

This is at least a step forward, thank you!

@ashwin-ant ashwin-ant merged commit 36ba802 into main Oct 16, 2024
3 checks passed
@ashwin-ant ashwin-ant deleted the ashwin/capabilities branch October 16, 2024 15:22
Pizzaface pushed a commit to RewstApp/mcp-inspector that referenced this pull request May 2, 2025
e665be0 Merge pull request modelcontextprotocol#14 from modelcontextprotocol/justin/fix-race-condition
be9c1fe Merge branch 'main' into justin/fix-race-condition
36ba802 Merge pull request modelcontextprotocol#17 from modelcontextprotocol/ashwin/capabilities
a1b8886 yarn build
7b718de Report errors when start() is called multiple times
83a43fd Protocol.connect() can automatically call Transport.start()
8d80cc5 Merge branch 'main' into justin/fix-race-condition
bcb4b21 add server capabilities derivation
e9c64fb Update tests and CLI
2b0599e Update server SSE transport
c39f808 Update client transports
87ae7ff Add start() to Transport interface

git-subtree-dir: packages/mcp-typescript
git-subtree-split: e665be0
Pizzaface pushed a commit to RewstApp/mcp-inspector that referenced this pull request May 2, 2025
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.

2 participants