Skip to content

Conversation

@uriyyo
Copy link
Contributor

@uriyyo uriyyo commented Nov 22, 2025

Hi,

Thanks for this library!

In the scope of this PR, added [project.scripts] entrypoint to pyproject.toml so 2 commands should be part of telagram-mcp package:

  1. telegram-mcp to run MCP stdio server.
  2. telegram-mcp-generate-session to generate session config.

It will allow running it using uvx without cloning the repo.

To generate session key:

uvx --from git+https://github.com/chigwell/telegram-mcp telegram-mcp-generate-session

To start MCP server:

uvx --from git+https://github.com/chigwell/telegram-mcp telegram-mcp

Copilot AI review requested due to automatic review settings November 22, 2025 13:21
Copy link
Contributor

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 the ability to run telegram-mcp using uvx by defining script entrypoints in pyproject.toml. The changes enable users to execute the MCP server and session generator without cloning the repository.

Key changes:

  • Added [project.scripts] entrypoints for telegram-mcp and telegram-mcp-generate-session commands
  • Refactored main.py and session_string_generator.py to expose callable main() functions
  • Updated MCP dependency from >=1.4.1 to >=1.8.0

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
pyproject.toml Removed package-mode = false, added setuptools configuration with py-modules, defined script entrypoints, and bumped mcp dependency
main.py Refactored execution logic by extracting async code into _main() function and creating synchronous main() entrypoint
session_string_generator.py Wrapped existing code in main() function to enable it as a script entrypoint
init.py Added root-level package marker file with comment
uv.lock Updated dependencies including mcp upgrade to v1.22.0 with additional transitive dependencies

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@iqdoctor
Copy link
Contributor

pls apply black

@uriyyo
Copy link
Contributor Author

uriyyo commented Dec 18, 2025

@iqdoctor Done, it should be good now

@chigwell chigwell self-requested a review December 19, 2025 09:08
@chigwell chigwell merged commit 83c2c73 into chigwell:main Dec 19, 2025
3 checks passed
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