Implement multi-database support and insights management #8
Conversation
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
- Added support for multiple databases via a JSON configuration file, allowing simultaneous access to SQLite, SQL Server, PostgreSQL, and MySQL. - Introduced an insights database for storing business insights, with functionality to append and list insights. - Updated various handlers and tools to require a `dbId` parameter for database operations. - Enhanced README documentation to reflect new features and usage instructions for multi-database configuration and insights management. - Version updated to 1.1.0 in package.json and package-lock.json.
| @dale-stewart - Love the idea, can you please help me update the Documentation of the change like how you mentioned in the Thanks, |
| @dale-stewart with this would I able to run 2 mcp servers for two different MySQL connections? |
| It would manage two connections from a single MCP server instance. My original motivation for doing this is that I could not get multiple instances of this MCP server working correctly with Cursor at the same time - it would get confused about which instance it was talking to. I haven't had any more time to work on this, so I've just been using the existing MCP server version one database at a time instead. |
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
dbIdparameter for database operations.I've found this tool very useful in my work, but I am only able to use one instance of the server at a time. Because I wanted to access multiple databases simultaneously, I thought I would add that capability. I'm sure this could use some refinement, but I thought I would offer this pull request to start the conversation.