Skip to content

Conversation

@tonxxd
Copy link
Contributor

@tonxxd tonxxd commented Oct 14, 2025

Description

This PR includes several fixes and improvements:

  • Fix inspector connector status display issues
  • Fix transport handling (1 transport per request)
  • Fix status dot display
  • Add non-blocking structured output
  • Export MCPAgentExecutionEventData as type export
  • Update examples and templates

Fixes connector status display issues in the inspector

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Internal change (refactoring, tests, tooling)

📦 Changeset Required?

Does this PR need a changeset?

  • ✅ Yes - I have created a changeset (pnpm changeset)
  • ❌ No - This is an internal-only change (docs, tests, tooling)

Quick reminder:

pnpm changeset  # Creates a changeset interactively

📚 Full guide: Changeset Workflow Guide
Need help? See when to create changesets and examples

🚀 Release Target

  • main branch → Stable release (default)
  • beta branch → Beta/prerelease version
  • Other feature branch → No release

Releasing a beta? See the Beta Release Workflow Guide for instructions.

Checklist

Code Quality

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have added tests that prove my fix is effective or that my feature works
  • Existing tests pass locally (pnpm test)
  • Linting passes (pnpm lint)
  • Build succeeds (pnpm build)

Documentation

  • The title of my pull request follows the conventional commits standard
  • Changes have been documented in the README/documentation (if applicable)
  • Breaking changes are clearly documented with migration steps

Release Preparation

  • I have created a changeset (pnpm changeset) - if this affects published packages
  • The changeset has a clear, user-focused description
  • Version bump type is appropriate (patch/minor/major)
  • All affected packages are included in the changeset

Additional Notes

This PR focuses on fixing connector status display issues in the inspector and improving transport handling. The changes are primarily bug fixes that improve the user experience without breaking existing functionality.

@tonxxd tonxxd requested a review from Pederzh October 14, 2025 14:52
@tonxxd tonxxd merged commit 1bbe767 into main Oct 14, 2025
4 checks passed
@tonxxd tonxxd deleted the fix-inspector-connector-status branch October 14, 2025 14:53
tonxxd added a commit that referenced this pull request Oct 14, 2025
* fix status dot

* fix: 1 transport per request

* feat: add @radix-ui/react-switch and implement auto-connect feature

- Added @radix-ui/react-switch dependency to manage auto-connect setting.
- Updated InspectorDashboard to include a switch for enabling/disabling auto-connect.
- Enhanced McpContext to handle auto-connect state and connect/disconnect servers accordingly.
- Implemented auto-connection logic in ServerDetail for seamless user experience.

* refactor: replace console.log with console.warn for connection status updates

- Updated logging in InspectorDashboard and ServerDetail components to use console.warn for better visibility of connection state changes.
- Enhanced pending navigation checks to include data presence before navigating or canceling navigation.

* fix: inspector connector status and transport improvements (#75)

* fix status dot

* fix: 1 transport per request

* chore: version packages and update changelogs

* feat: implement connection tester and auto-switch feature in InspectorDashboard

- Added a temporary ConnectionTester component to handle connection attempts with success and failure callbacks.
- Introduced an auto-switch feature that allows users to automatically switch between direct and proxy connections if one fails.
- Updated the InspectorDashboard to manage pending connection configurations and handle connection success/failure logic.
- Enhanced UI to include a switch for enabling/disabling the auto-switch feature, with state persistence in localStorage.
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