Skip to content

Conversation

@roomote
Copy link
Collaborator

@roomote roomote commented Jun 30, 2025

Important

Add VoiceVox MCP Server to marketplace with local fallback support and comprehensive tests.

  • Marketplace Data:
    • Add VoiceVox MCP Server to mcps.yaml with installation methods and prerequisites.
    • Update README.md to document the structure and details of the VoiceVox MCP Server.
  • RemoteConfigLoader:
    • Add local fallback methods fetchLocalMcps() and fetchLocalModes() to load data from marketplace-data directory if remote fetch fails.
    • Update fetchModes() and fetchMcps() to use local fallbacks on failure.
  • Tests:
    • Add voicevox-mcp.spec.ts to test loading of VoiceVox MCP Server, including tags, installation methods, prerequisites, and JSON configuration validation.

This description was created by Ellipsis for 564df5f. You can customize this summary. It will automatically update as commits are pushed.

@roomote roomote requested review from cte, jr and mrubens as code owners June 30, 2025 09:58
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. documentation Improvements or additions to documentation enhancement New feature or request labels Jun 30, 2025
@delve-auditor
Copy link

delve-auditor bot commented Jun 30, 2025

No security or compliance issues detected. Reviewed everything up to 564df5f.

Security Overview
  • 🔎 Scanned files: 4 changed file(s)
Detected Code Changes

The diff is too large to display a summary of code changes.

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.

{
"voicevox": {
"command": "uvx",
"args": ["mcp-server-voicevox", "--voicevox-url=http://localhost:50021"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's an inconsistency in the module naming. In the 'uvx (Recommended)' and 'Custom Configuration' sections, the command argument is 'mcp-server-voicevox', but in the 'pip install' section, it appears as 'mcp_server_voicevox'. Please confirm if this is intentional or if it should be standardized.

Suggested change
"args": ["mcp-server-voicevox", "--voicevox-url=http://localhost:50021"]
"args": ["mcp_server_voicevox", "--voicevox-url=http://localhost:50021"]

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jun 30, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jul 7, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jul 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants