Fix paper-search skill to use connected MCP servers - #2
Open
conormccallion1-sys wants to merge 1 commit into
Open
Fix paper-search skill to use connected MCP servers#2conormccallion1-sys wants to merge 1 commit into
conormccallion1-sys wants to merge 1 commit into
Conversation
The skill pointed to a `paper-search` CLI run via `uv run`, but no such package exists in the repo and sandboxed sessions block egress to academic APIs, so a local HTTP client could never work. Rewrite SKILL.md to drive the already-connected research MCP servers (Consensus, PubMed, bioRxiv/medRxiv, Scholar Gateway, Exa), which cover the same sources without needing egress. Map each job (search / read full text / related) to the right MCP tool and document the workflow. Update README to match. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EXmEUy9tbicVu62VatGrE3
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
2 participants
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.
Problem
The
paper-searchskill was broken.SKILL.mdinstructed Claude to run a CLI viauv run --directory <REPO_PATH> paper-search …, but:README.mdandSKILL.md, nopyproject.tomlor source.uv runsubprocess can't reach the session's MCP servers either.Meanwhile, the connected research MCP servers already cover the same sources and work (verified live: Consensus returned 20 papers across Semantic Scholar/PubMed/Scopus/arXiv; PubMed matched 11k+ articles).
Fix
Rewrite the skill to drive the working MCP tools instead of the phantom CLI:
SKILL.md— routing table mapping each job to a real tool:Consensus,Scholar_Gateway,PubMed,bioRxiv,ExaPubMed get_full_text_article/get_article_metadata,bioRxiv get_preprint,Exa web_fetchPubMed find_related_articles/lookup_article_by_citation/convert_article_idsREADME.md— now describes the repo and usage instead of placeholder text.No egress required: the MCP servers run outside the sandbox's network boundary.
🤖 Generated with Claude Code
https://claude.ai/code/session_01EXmEUy9tbicVu62VatGrE3
Generated by Claude Code