Skip to content

Conversation

@dragon-ai-agent
Copy link

This PR adds a CLI entry point to enable installation and running via uvx from git source.

Changes:

  • Added [project.scripts] section to pyproject.toml pointing to paper_search_mcp.server:main
  • Added main() function to server.py as entry point
  • This enables installation via: uvx --from git+https://github.com/openags/paper-search-mcp paper-search-mcp

Resolves #16

@dragon-ai-agent

- Added [project.scripts] section to pyproject.toml pointing to paper_search_mcp.server:main
- Added main() function to server.py as entry point
- This enables installation via uvx from git source

Resolves openags#16

@dragon-ai-agent
@chriscarrollsmith
Copy link

It's a good idea, but... when I try to install from your repo, I get "No executables are provided by paper-search-mcp"

@universea universea requested a review from Copilot August 25, 2025 06:37
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds CLI entry point functionality to enable uvx compatibility for the paper-search-mcp package. The changes allow users to install and run the MCP server directly from git source using uvx.

  • Added a CLI script entry point in pyproject.toml
  • Refactored the main execution logic into a dedicated main() function

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
pyproject.toml Added project.scripts section with CLI entry point
paper_search_mcp/server.py Added main() function as entry point and updated main block

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines +344 to +348
def main():
"""Main entry point for the MCP server."""
mcp.run(transport="stdio")


Copy link

Copilot AI Aug 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] The extra blank line at line 347 creates inconsistent spacing. Consider removing one blank line to maintain consistent formatting with the rest of the codebase.

Suggested change
def main():
"""Main entry point for the MCP server."""
mcp.run(transport="stdio")

Copilot uses AI. Check for mistakes.
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.

Add CLI entry point for uvx compatibility

2 participants