Add multi-client support #38
Conversation
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
- Add supportedClients and supportedTransports fields to package schema - Create client adapters for Claude, Zed, Continue, and Firebase - Update package list with client compatibility information - Add validation utilities for client configurations Co-Authored-By: Michael Latman <[email protected]>
- Add Preferences class for managing client selection - Use config files to detect installed clients - Store and persist client preferences - Auto-select single client installations - Fix ClientType definition and usage Co-Authored-By: Michael Latman <[email protected]>
- Add Jest configuration - Create test setup with mocks - Add unit tests for all client adapters - Add integration tests for config manager - Test error handling and validation Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
- Add client configuration abstractions and adapters - Implement automatic client detection and selection - Add support for multiple client configuration - Update documentation with multi-client support details - Add comprehensive test suite for client adapters - Improve transport validation and error handling Co-Authored-By: Michael Latman <[email protected]>
devin-ai-integration
bot
force-pushed
the
devin/1733898531-multi-client-support
branch
from
b7864ee to
8ebec03
Compare
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
⚙️ Control Options:
Add "(aside)" to your comment to have me ignore it. |
- Add supportedClients and supportedTransports fields to package schema - Create client adapters for Claude, Zed, Continue, and Firebase - Update package list with client compatibility information - Add validation utilities for client configurations Co-Authored-By: Michael Latman <[email protected]>
- Add Preferences class for managing client selection - Use config files to detect installed clients - Store and persist client preferences - Auto-select single client installations - Fix ClientType definition and usage Co-Authored-By: Michael Latman <[email protected]>
devin-ai-integration
bot
force-pushed
the
devin/1733898531-multi-client-support
branch
from
8ebec03 to
c977796
Compare
|
❌ PR validation failed. Please check the workflow logs for details. |
Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
|
❌ PR validation failed. Please check the workflow logs for details. |
Co-Authored-By: Michael Latman <[email protected]>
|
❌ PR validation failed. Please check the workflow logs for details. |
Co-Authored-By: Michael Latman <[email protected]>
|
❌ PR validation failed. Please check the workflow logs for details. |
Co-Authored-By: Michael Latman <[email protected]>
|
❌ PR validation failed. Please check the workflow logs for details. |
Co-Authored-By: Michael Latman <[email protected]>
|
❌ PR validation failed. Please check the workflow logs for details. |
Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
|
❌ No new packages found or validation failed. |
|
✅ PR validation passed successfully! |
|
All clients should be supported by all packages so drop that from the package list. But do we actually need to track stdio versus other transports in packages list? I'd rather not. Which client requires that? Also we should only allow .json for zed not their other style unless I'm misunderstanding |
…dation Co-Authored-By: Michael Latman <[email protected]>
|
❌ No new packages found or validation failed. |
|
✅ PR validation passed successfully! |
- Add comprehensive documentation and examples to all client adapters - Remove legacy supportedClients and supportedTransports fields - Add multi-client selection for package installation/uninstallation - Update configuration validation per client requirements - Add source attribution and official documentation links Co-Authored-By: Michael Latman <[email protected]>
|
❌ No new packages found or validation failed. |
|
✅ PR validation passed successfully! |
- Add new clients command to list installed MCP clients - Add test environment setup script - Update Zed adapter to properly detect config files - Add comprehensive client documentation - Update package.json with new test commands Co-Authored-By: Michael Latman <[email protected]>
|
❌ PR validation failed. Please check the workflow logs for details. |
Co-Authored-By: Michael Latman <[email protected]>
|
❌ PR validation failed. Please check the workflow logs for details. |
- Update config path to use ~/.config/zed/settings.json - Remove transport and runtime fields (not required per docs) - Match official Zed context server configuration format - Fix TypeScript errors in configuration handling - Add comprehensive documentation with examples Co-Authored-By: Michael Latman <[email protected]>
|
❌ PR validation failed. Please check the workflow logs for details. |
Co-Authored-By: Michael Latman <[email protected]>
|
❌ No new packages found or validation failed. |
|
✅ PR validation passed successfully! |
Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
|
❌ PR validation failed. Please check the workflow logs for details. |
|
Devin please fix |
Enhances multi-client support and improves configuration management:
Key Changes:
All changes have been tested:
Link to Devin run: https://app.devin.ai/sessions/2c5aea9860bd446f9896569e1ea2b4b0