Skip to content

Search PubMed under a topic - #150

Merged
thavelin merged 1 commit into
mainfrom
snowpile/api-136-ncbi-search
Nov 12, 2026
Merged

thavelin merged 1 commit into
mainfrom
snowpile/api-136-ncbi-search

Conversation

@Snowpile

Copy link
Copy Markdown
Contributor

Closes #136.

  • GET /topic/{topicId}/hit?term= → HitList (ListHits, in PublicationRoutes): up to 100 typed PubMed hits, best match first, no paging, nothing recorded.
  • Each Hit is the stored Publication if one holds the work (DOI, then PMID; Add a paper to a topic from a DOI or PMID (C12) #133's rule), else the provider's record (no id), plus the topic's paperId if already included. Abstract rides on the publication, so B2's route goes.
  • service.Searcher + Config.Searcher (503 without one); pubmed.Client implements it and caches hits for the following add. No store.Store change.
  • TS client now sends query fields (first route with one); manifest-agreement test covers them.
  • AGENTS.md: request-standard row for a provider search.
  • Holds under either answer to Spike: is a recorded search strategy (PRISMA) a product goal? #103: a recorded search would be a separate /topic/{topicId}/search resource, additive.

Services must set Config.Searcher at their bump to serve it.

🤖 Generated with Claude Code

GET /topic/{topicId}/hit?term= (PublicationRoutes.ListHits) searches the
provider and answers up to 100 typed hits, each the stored Publication when
one holds the work (DOI, then PMID) and the topic's paper id when it includes
it. Nothing is recorded; the abstract rides on the hit's Publication.

- service.Searcher, Config.Searcher; pubmed.Client implements it.
- The TypeScript client sends query fields, the first route to have one.
- AGENTS.md: the request-standard row for a provider search.

Closes #136

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@thavelin
thavelin requested a review from ptmo516 November 12, 2026 03:05
@thavelin
thavelin added this pull request to the merge queue Nov 12, 2026
Merged via the queue into main with commit c3ec21d Nov 12, 2026
2 checks passed
@thavelin
thavelin deleted the snowpile/api-136-ncbi-search branch November 12, 2026 03: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.

Search NCBI through the server (C10, B2)

2 participants