feat: MCP Desktop Extension - user friendly installation of qsv MCP Server #3296
+1,853
−12
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
Contributor
There was a problem hiding this comment.
Pull request overview
This PR introduces a user-friendly Desktop Extension installation method for the qsv MCP Server, making it accessible to non-technical users through a packaged .mcpb file format. The extension provides identical functionality to the legacy MCP server but with a simplified GUI-based installation and configuration experience.
Key changes:
- New packaging script to bundle the MCP server as a Desktop Extension (.mcpb file)
- Extension mode detection via environment variable to skip MCP server version checks
- Comprehensive documentation for Desktop Extension installation, usage, and troubleshooting
Reviewed changes
Copilot reviewed 6 out of 8 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
.claude/skills/scripts/package-mcpb.js |
New packaging script that creates .mcpb archives with compiled code, dependencies, and skill definitions |
.claude/skills/src/update-checker.ts |
Adds isExtensionMode flag to skip MCP server version checks when running as a Desktop Extension |
.claude/skills/src/config.ts |
Adds getBooleanEnv helper and isExtensionMode detection function for Desktop Extension mode |
.claude/skills/package.json |
Adds mcpb:package npm script for building Desktop Extensions |
.claude/skills/package-lock.json |
Adds archiver dependency and its transitive dependencies |
.claude/skills/README.md |
Updates installation section with Desktop Extension option and reorganizes documentation links |
.claude/skills/DESKTOP_EXTENSION.md |
Comprehensive 565-line guide covering installation, configuration, troubleshooting, and FAQs |
Files not reviewed (1)
- .claude/skills/package-lock.json: Language not supported
Fixed three issues identified in PR #3296 review: 1. Division by zero protection in progress calculation 2. Case-insensitive file filtering for node_modules 3. Safe error message access with optional chaining Also regenerated qsv-mcp-server.mcpb with the fixes applied. Addresses: #3296 (review) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.