Skip to content

Fix/mcp server path error#111

Closed
httpdss wants to merge 4 commits intomainfrom
fix/mcp-server-path-error
Closed

Fix/mcp server path error#111
httpdss wants to merge 4 commits intomainfrom
fix/mcp-server-path-error

Conversation

@httpdss
Copy link
Owner

@httpdss httpdss commented Aug 22, 2025

Issue

Description

Checklist

  • I have read the contributing guidelines.
  • My code follows the code style of this project.
  • I have performed a self-review of my own code.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.
  • I have updated the documentation accordingly.

Screenshots (if applicable)

Additional Comments

…ser issues

- Add HTTP MCP server support alongside existing stdio transport
- Integrate HTTP MCP server into CLI with 'struct mcp --server --transport http'
- Fix ArgumentParser issues in generate_structure and validate_structure commands
- Add mock parser classes for proper MCP tool instantiation
- Update MCP command with transport selection (--transport, --host, --port options)
- Enhance error handling and logging for both stdio and HTTP transports
- Add FastAPI dependency for HTTP MCP server functionality
- Update comprehensive documentation in mcp-integration.md with:
  - HTTP transport usage examples and benefits
  - Client integration for both transports
  - Testing instructions and troubleshooting guide
  - Updated quick start guide

Fixes ArgumentParser initialization errors that were preventing MCP tools from working.
Maintains backward compatibility while adding HTTP transport flexibility.
@httpdss httpdss added the bug Something isn't working label Aug 22, 2025
- Updated fake_start function signature to accept args parameter
- Fixed assertion to verify method called with correct argument
- All 113 tests now passing
@codecov
Copy link

codecov bot commented Aug 22, 2025

Codecov Report

❌ Patch coverage is 34.67742% with 81 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (main@e3973e2). Learn more about missing BASE report.

Files with missing lines Patch % Lines
struct_module/commands/mcp.py 10.84% 74 Missing ⚠️
struct_module/mcp_server.py 82.92% 6 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #111   +/-   ##
=======================================
  Coverage        ?   67.63%           
=======================================
  Files           ?       20           
  Lines           ?     1585           
  Branches        ?      276           
=======================================
  Hits            ?     1072           
  Misses          ?      434           
  Partials        ?       79           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@httpdss httpdss closed this Aug 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant