Conversation
Merge duplicate Docs and Reports tools into shared entrypoints, keep legacy aliases working in dispatch, remove apiGuidance injection from tool responses, and stop advertising per-tool verbose schema flags. Also update server instructions, manifests, and tool metadata to match the smaller public tool surface.
Extend tool tests to cover the new getReport entrypoint and legacy report aliases, and add a dedicated docs-tools test file covering listDocsArticles, getDocsEntity, updateDocsEntity, and their legacy Docs aliases through the top-level ToolHandler.
Update the main README to reflect the merged tool surface, documented response behavior, and current examples, add an AGENTS.md file with repo-specific maintenance guidance, and delete the orphaned README_REPORTS_FEATURE.md implementation note.
Docs and reports tools were already compacted by their own handlers (docs-tools.ts, reports-tools.ts), but listTools() re-applied compactTool with CONVERSATION_TOOL_DESCRIPTIONS — which had no entries for those tools, causing redundant compaction passes. - Only apply CONVERSATION_TOOL_DESCRIPTIONS to conversation tools - Pass docs and reports tools through as-is since they're pre-compacted
Claude Code can install the MCP server with a single command, making it the fastest onboarding path for developers already using it. - Add Option 1: Claude Code with `claude mcp add` instructions - Renumber existing options (MCPB → 2, Manual → 3, CLI → 4) - Update prerequisites to mention Claude Code
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (16)
WalkthroughConsolidates the Help Scout MCP server's tool surface by unifying multiple Docs API endpoints into entity-type-aware handlers, merging Reports tools into a single Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~50 minutes ✨ Finishing Touches
🧪 Generate unit tests (beta)
Comment |
jgalea
pushed a commit
to jgalea/help-scout-mcp
that referenced
this pull request
Apr 25, 2026
Release v2.3.0
Open
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
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.
Summary
Summary by CodeRabbit
New Features
Removed/Deprecated
Documentation