Skip to content

Conversation

@matthewgoodman13
Copy link
Contributor

@matthewgoodman13 matthewgoodman13 commented Jul 31, 2025

Summary

Customers are unable to install the ECS MCP Server using uvx. When configuring the ECS MCP Server and starting it, they get the following error:

Traceback (most recent call last):
  File "/Users/asdf/.cache/uv/archive-v0/fM-nIq6nkiZfB0W5xjMll/bin/ecs-mcp-server", line 6, in <module>
    from awslabs.ecs_mcp_server.main import main
  File "/Users/asdf/.cache/uv/archive-v0/fM-nIq6nkiZfB0W5xjMll/lib/python3.13/site-packages/awslabs/ecs_mcp_server/main.py", line 72, in <module>
    mcp = FastMCP(
        name="AWS ECS MCP Server",
    ...<29 lines>...
    """,
    )
TypeError: FastMCP.__init__() got an unexpected keyword argument 'description'

This is related to a non-backwards compatible change that is being tracked here.

Changes

  • remove the unsupported parameters
  • update readme instructions with pre-reqs

User experience

  • customers can install the ecs mcp server again successfully

Checklist

If your change doesn't seem to apply, please leave them unchecked.

  • [Y] I have reviewed the contributing guidelines
  • [Y] I have performed a self-review of this change
  • [Y] Changes have been tested
  • [Y] Changes are documented

Is this a breaking change? (Y/N) No.

RFC issue number:

Checklist:

  • [NA] Migration process documented
  • [NA] Implement warnings (if it can live side by side)

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

@matthewgoodman13 matthewgoodman13 requested review from a team and karanbokil as code owners July 31, 2025 18:43
@matthewgoodman13 matthewgoodman13 changed the title [docs] update ecs mcp server installation instructions (readme) docs: update ecs mcp server installation instructions (readme) Jul 31, 2025
alexa-perlov
alexa-perlov previously approved these changes Aug 1, 2025
@codecov
Copy link

codecov bot commented Aug 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.12%. Comparing base (ab9bb07) to head (3e0d209).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #994      +/-   ##
==========================================
+ Coverage   88.72%   89.12%   +0.39%     
==========================================
  Files         574      527      -47     
  Lines       39156    35420    -3736     
  Branches     6019     5410     -609     
==========================================
- Hits        34743    31568    -3175     
+ Misses       2912     2550     -362     
+ Partials     1501     1302     -199     

☔ 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.

@matthewgoodman13 matthewgoodman13 changed the title docs: update ecs mcp server installation instructions (readme) fix: remove unsupported parameter from initialization & update readme installation instructions Aug 7, 2025
@matthewgoodman13 matthewgoodman13 removed the request for review from karanbokil August 8, 2025 01:10
@matthewgoodman13 matthewgoodman13 added this pull request to the merge queue Aug 8, 2025
Merged via the queue into main with commit e5845ea Aug 8, 2025
179 checks passed
@matthewgoodman13 matthewgoodman13 deleted the update-readme-jul branch August 8, 2025 01:58
@github-project-automation github-project-automation bot moved this from To triage to Done in awslabs/mcp Project Aug 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants