Skip to content

Conversation

@movinam
Copy link
Collaborator

@movinam movinam commented Jul 21, 2025

Summary:

This PR adds an additional endpoint to the SemanticScholarSearchTool. It also adds support for additional Semantic Scholar APIs in the future.

Details:

  1. Add a field named endpoint to allow arun to direct Semantic Scholar tool calls to:
    a. search by default (previous functionality)
    b. get paper details using DOI via the /graph/v1/paper/DOI endpoint

  2. Add PaperDataItem to structures.py. It represents a schema replicating the response from semantic scholar for a single paper (ref).

  3. Add SearchItem as an abstraction to support different types of search results instead of restricting results to SearchResultItem. This way we can inherit from SearchItem and create result schemas based on the search. This change has been made only for SemanticScholarSearchTool as it has different endpoints that produce different results. SearchResultItem and PaperDataItem will inherit from SearchItem.

Usage:

Will allow users to fetch paper details for a given DOI or list of DOIs by calling Semantic Scholar's DOI endpoint.

Checks

  • Tested Changes
  • Stakeholder Approval

@movinam movinam changed the title Semantic Scholar API PR Additional Semantic Scholar API Jul 21, 2025
@movinam
Copy link
Collaborator Author

movinam commented Jul 21, 2025

@NISH1001 Please test and make the necessary changes required to make the PR adhere the current abstraction.

@NISH1001
Copy link
Collaborator

ONce these changes are addressed, i can have a final look and merge. Thanks for the PR.

@NISH1001
Copy link
Collaborator

NISH1001 commented Jul 22, 2025

@movinam also please add test cases in tests/ as well.

And in this PR, add the usage section as well. This will give idea what the new feature is doing.

Example PR:
#77

@NISH1001 NISH1001 merged commit df19b23 into develop Jul 22, 2025
@NISH1001 NISH1001 deleted the feature/semantic-apis branch July 22, 2025 19:06
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.

3 participants