Skip to content

feat(cli): add mcp subcommand to launch Beacon MCP server - #948

Open
jmatson36097-blip wants to merge 1 commit into
Scottcjn:mainfrom
jmatson36097-blip:fix/issue-942-cli-mcp-subcommand
Open

jmatson36097-blip wants to merge 1 commit into
Scottcjn:mainfrom
jmatson36097-blip:fix/issue-942-cli-mcp-subcommand

Conversation

@jmatson36097-blip

Copy link
Copy Markdown
Contributor

Summary of Changes

Fixes #942

  • Added mcp parser subcommand to beacon_skill/cli.py aligning the CLI interface with the README documentation.
  • Invokes mcp_server.server.main() over stdio transport.
  • Displays clear instructions if optional mcp dependencies are not installed.
  • Added test in tests/test_cli_mcp.py.

Verification

  • Tested syntax and argument parsing.
  • Verified test suite passes.

Closes Scottcjn#942
- Register mcp subcommand in CLI parser
- Invoke mcp_server.server.main via asyncio.run()
- Provide helpful error message when mcp package is missing
@github-actions

Copy link
Copy Markdown

Welcome to beacon-skill! Thanks for your first pull request.

Quick checklist:

  • Tests pass (pytest tests/)
  • New code includes SPDX license header
  • Focused on a single change

Bounty tiers: Micro (1-10 RTC) | Standard (20-50) | Major (75-100)

A maintainer will review your PR soon!

@github-actions github-actions Bot added the size/S PR: 11-50 lines label Sep 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S PR: 11-50 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

README documents beacon mcp, but the CLI has no mcp subcommand

1 participant