External MCP: Claude Desktop extension export and lock-safe writes - #1596
Salomon-Mazatlan wants to merge 30 commits into
Conversation
|
Hi Lorenzo Great that you have a chance to test the MCP access with Claude Desktop (and maybe Claude Code as well), as I can only test with Codex here. It seems that the MCP-protocol has evolved since the version I was using, and that Claude Desktop is especially picky in some places (e.g., regarding tool naming conventions). But in order to make QualCoder as widely compatible as possible, we should fix some of the issues in QualCoder itself rather than only in the Claude Desktop bridge:
Other than that, a few comments I would prefer not to add a Claude-specific export button to QualCoder. There are so many agentic frameworks out there; we cannot add a separate button for each. Could we instead build a generic, versioned QualCoder.mcpb as a release artifact and offer it through the QualCoder website and GitHub Releases? The default port can be included in the manifest and remain editable through the extension’s user_config. We could even consider submitting it to Claude Desktop’s extension directory later. "External operations moved from the GUI thread to a dedicated worker thread..." -> This sounds like a very good idea. The database-access optimizations: Also very good Quote matching: As far as I understand, this does not add another MCP search tool but improves QualCoder’s internal quote matching, right? That's would be good, yes. |
|
Regarding the internal implementation of the bridge, here are some recommendations from Codex: "
BTW: I wonder if we could transform this into a small generic stdio <> http bridge that would be shipped with every QualCoder installation and could be invoked by |
|
Hi @kaixxx, I applied your observations. In QualCoder:
.mcpb
what I did not do:
|
|
Hi Lorenzo, I have only a few remaining recommendations:
|
|
Hi Lorenzo, I didn't realize that this is still a draft. I think we should include it into the final version 4.0 release. You've made very good improvements to the current MCP-setup, extended compatibility with Claude, and corrected some issues along the way. The ideas from my last comment can be left for the future. The PR seems good as it is. |
Hi @kaixxx I don't have access to my computer right now to work on this proposal. Would you mind making a PR with the adjustments from your fork? The GitHub app is only letting me leave comments. |
Yes, of course. I'll see what I can do. |
Muchas gracias @kaixxx |
Hi @kaixxx, I have been testing the MCP connection you added in the latest update. I was asked to adapt it to a desktop application such as Claude Desktop, so I made some adjustments on top of your work. So far I have only tested it with Claude Desktop. I would like to hear your opinion before proposing it as a PR.
Button and installation file
QualCoder.mcpb, which installs in Claude Desktop.Bridge inside the extension (version 1.0.2)
Execution of external requests
isErrorand a readable message, instead of a JSON-RPC error. The long SDK traceback no longer shows up in the log. Claude Code and Codex would see this change too.Database
Quote matching