Conversation
📝 WalkthroughWalkthroughAdded an MCP server manifest ( Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
server.json (1)
12-12: Keep manifest version synchronized with runtime MCP version.Line 12 hardcodes
"version": "1.0.0", but runtime MCP server info is sourced fromVersionProvider(backend/app/src/main/kotlin/io/tolgee/mcp/McpConfig.kt:26-31). If these drift, registry metadata becomes stale. Consider tying this field to the release pipeline or documenting a required manual bump step.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@server.json` at line 12, The manifest currently hardcodes "version": "1.0.0" which can drift from the runtime MCP version provided by VersionProvider/McpConfig; update the build so server.json's version is produced at release-time from the same source as VersionProvider (e.g., inject the MCP version into server.json as a build artifact or generate server.json from a template during CI/CD), or alternatively add a clear release-note step that documents manually bumping server.json whenever McpConfig's MCP version changes—ensure the symbol VersionProvider (or the MCP_VERSION constant in McpConfig) is the canonical source and wire the pipeline/template to read it and replace the manifest entry.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@server.json`:
- Line 12: The manifest currently hardcodes "version": "1.0.0" which can drift
from the runtime MCP version provided by VersionProvider/McpConfig; update the
build so server.json's version is produced at release-time from the same source
as VersionProvider (e.g., inject the MCP version into server.json as a build
artifact or generate server.json from a template during CI/CD), or alternatively
add a clear release-note step that documents manually bumping server.json
whenever McpConfig's MCP version changes—ensure the symbol VersionProvider (or
the MCP_VERSION constant in McpConfig) is the canonical source and wire the
pipeline/template to read it and replace the manifest entry.
Publishes the Tolgee MCP server to the official MCP Registry (https://registry.modelcontextprotocol.io) as a remote server entry pointing at https://app.tolgee.io/mcp/developer, authenticated via an X-API-Key header (PAT or PAK).
ce23250 to
49bff6a
Compare
Publishes the Tolgee MCP server to the official MCP Registry (https://registry.modelcontextprotocol.io) as a remote server entry pointing at https://app.tolgee.io/mcp/developer, authenticated via an X-API-Key header (PAT or PAK).
Summary by CodeRabbit
Chores
Documentation