mcp icon indicating copy to clipboard operation
mcp copied to clipboard

fix: no wrappers

Open scottschreckengaust opened this issue 2 months ago • 2 comments

Added warning about merging new MCP servers that are primarily wrappers around an AWS SDK.

Fixes

Summary

Changes

Please provide a summary of what's being changed

User experience

Please share what the user experience looks like before and after this change

Checklist

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

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

Is this a breaking change? (Y/N)

RFC issue number:

Checklist:

  • [ ] Migration process documented
  • [ ] 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.

scottschreckengaust avatar Oct 15 '25 19:10 scottschreckengaust

Do we need to include more details of what a wrapper is in the contributing or developers guide?

scottschreckengaust avatar Oct 15 '25 19:10 scottschreckengaust

Yeah, I would expand on the definition (or provide some sort of checklist). For each tool exposed:

  • can this action be achieved through the api mcp server ? If yes, then we don't take it
  • Is it simply a wrapper around a boto3 call, a.k.a it does not add any logic or enhanced functionality ? If yes, we don't take it

krokoko avatar Oct 17 '25 17:10 krokoko

This pull request is now marked as stale because it hasn't seen activity for a while. Add a comment or it will be closed soon. If you wish to exclude this issue from being marked as stale, add the "backlog" label.

github-actions[bot] avatar Nov 12 '25 01:11 github-actions[bot]

Closing this pull request as it hasn't seen activity for a while. Please add a comment @mentioning a maintainer to reopen. If you wish to exclude this issue from being marked as stale, add the "backlog" label.

github-actions[bot] avatar Nov 14 '25 01:11 github-actions[bot]