FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

refactor(mcp): tighten connection result types by rekram1-node · Pull Request #31550 · anomalyco/opencode · GitHub

refactor(mcp): tighten connection result types - #31550

Closed
rekram1-node wants to merge 3 commits into
refactor/mcp-simplify-service-stackfrom
refactor/mcp-connection-results
Closed

refactor(mcp): tighten connection result types#31550
rekram1-node wants to merge 3 commits into
refactor/mcp-simplify-service-stackfrom
refactor/mcp-connection-results

Conversation

rekram1-node commented Jun 9, 2026
edited
Loading

Copy link
Copy Markdown
Collaborator

Summary

  • use explicit connected and unavailable tags for MCP connection outcomes
  • use the existing local and remote MCP config variants without casts
  • require clients and tool definitions on connected creation results
  • remove unchecked type assertions, undefined-client placeholders, and non-null assertions
  • validate tool arguments at the SDK boundary instead of casting them

Validation

  • bun test test/mcp/lifecycle.test.ts
  • bun typecheck

Stacked on: #31549
Relates to: #28567

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

1 participant


Back | FazBrowse Home | New Git URL