Skip to content

refactor(mcp): shorten product tool descriptions - #63

Merged
Kikobeats merged 2 commits into
masterfrom
refactor/mcp-short-tool-descriptions
Sep 17, 2026
Merged

Kikobeats merged 2 commits into
masterfrom
refactor/mcp-short-tool-descriptions

Conversation

@Kikobeats

Copy link
Copy Markdown
Member

Summary

  • Cut product tool descriptions to the job, return shape, and sibling disambiguation (video/videos, extract vs markdown).
  • Point agents at microlink_docs plus JSON Schema for parameters instead of repeating flags on every tool.
  • Group the three tool families (products, search/extract, checkout) in server instructions so the catalog is less of a flat dump.

Does not remove tools or split billing into another server.

Test plan

  • pnpm --filter @microlink/mcp test
  • Confirm screenshot/pdf descriptions no longer list fullPage / pageRanges
  • In an MCP client, ask for a full-page screenshot and check the model calls microlink_docs or uses the schema rather than guessing from the old prose

Made with Cursor

Parameters already live in JSON Schema and microlink_docs; repeating them on every tool burns tokens and undercuts the docs tool.

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai

coderabbitai Bot commented Sep 17, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 37 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 9d716ef7-83c1-4c74-910a-1b556dd2ff25

📥 Commits

Reviewing files that changed from the base of the PR and between 99f6d96 and 41eee3a.

📒 Files selected for processing (24)
  • packages/mcp/src/index.js
  • packages/mcp/src/tools/audio.js
  • packages/mcp/src/tools/audios.js
  • packages/mcp/src/tools/docs.js
  • packages/mcp/src/tools/emails.js
  • packages/mcp/src/tools/embed.js
  • packages/mcp/src/tools/extract.js
  • packages/mcp/src/tools/function.js
  • packages/mcp/src/tools/html.js
  • packages/mcp/src/tools/images.js
  • packages/mcp/src/tools/lighthouse.js
  • packages/mcp/src/tools/links.js
  • packages/mcp/src/tools/logo.js
  • packages/mcp/src/tools/markdown.js
  • packages/mcp/src/tools/metadata.js
  • packages/mcp/src/tools/pdf.js
  • packages/mcp/src/tools/screenshot.js
  • packages/mcp/src/tools/search.js
  • packages/mcp/src/tools/technologies.js
  • packages/mcp/src/tools/text.js
  • packages/mcp/src/tools/video.js
  • packages/mcp/src/tools/videos.js
  • packages/mcp/test/stdio-server.test.js
  • packages/mcp/test/tools.test.js

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coveralls

coveralls commented Sep 17, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 35204932493

Warning

No base build found for commit 99f6d96 on master.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 80.496%

Details

  • Patch coverage: 25 of 25 lines across 22 files are fully covered (100%).

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 5582
Covered Lines: 4511
Line Coverage: 80.81%
Relevant Branches: 955
Covered Branches: 751
Branch Coverage: 78.64%
Branches in Coverage %: Yes
Coverage Strength: 28.32 hits per line

💛 - Coveralls

@Kikobeats
Kikobeats merged commit 704e7a2 into master Sep 17, 2026
9 checks passed
@Kikobeats
Kikobeats deleted the refactor/mcp-short-tool-descriptions branch September 17, 2026 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants