Repository navigation
feat(adapters): add SumUp payments MCP adapter - #1367
Conversation
Add pre-built MCP adapter for SumUp payment gateway exposing 6 tools: - sumup_get_merchant_profile: retrieve authenticated merchant details - sumup_list_transactions: list transaction history with status/date filters - sumup_get_transaction: retrieve specific transaction details - sumup_list_payouts: list payout settlement batches - sumup_list_checkouts: list online checkouts and customer payment links - sumup_list_subaccounts: list operator subaccounts Regenerate catalog.ts and validate via scripts/validate-adapters.mjs. Part of HelpCode-ai#982
|
👋 Welcome, @ak-bharadwaj, and thanks for opening your first PR on AnythingMCP! A few quick pointers:
Someone from the core team will look at this within ~48h. If you don't hear back, please ping us in Discussions / Q&A. ⭐ While you wait — if you find AnythingMCP useful, a star helps others discover it. |
|
All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
|
recheck |
helpcodeai
left a comment
There was a problem hiding this comment.
Thanks for the SumUp adapter. The host and the bearer token are fine, but the tools cannot work yet:
- Parameters are dropped. None of the GET tools has
endpointMapping.queryParams. The REST engine builds the query string only fromqueryParams, solimit,statuses, the date filters and so on are never sent.sumup_get_transactionnever sends its required id, so it cannot find a transaction. Please map every parameter, for example"queryParams": { "transaction_code": "$transaction_code", "limit": "$limit" }.docs/tool-definition.mdshows the format. - Paths. Please check
/v0.1/me/payoutsand/v0.1/me/subaccountsagainst the SumUp API reference. As far as I can see payouts are under/v0.1/me/financials/payouts(or the newer/v1.0/merchants/{merchant_code}/payouts), and sub-accounts under/v0.1/me/accounts. - Missing pieces from the adapter checklist (AGENTS.md, "Add an adapter: six steps"): a
sumup.live.spec.tswith static checks and a live block behindRUN_SUMUP_LIVE=1, regeneratedcatalog.ts, updated counts (node scripts/adapter-count.mjs --checkwill fail now), and the logo inpackages/frontend/public/logos/connectors/. - Please claim SumUp in #982 first, so nobody else works on it at the same time, and remove the
ccto the unrelated account from the description.
I will approve the CI runs once these are in.
|
@helpcodeai Thank you for the detailed feedback! All 4 requested points have been completed in commit \�bb5fb6:
Ready for CI approval and review. Thanks! |
helpcodeai
left a comment
There was a problem hiding this comment.
Thanks, this is much better: parameters are mapped, the paths are fixed, and the counts, catalog and logo are in. I approved the CI runs.
One last thing: the live test only checks that SUMUP_API_KEY is set. Please make one real call when RUN_SUMUP_LIVE=1, for example GET https://api.sumup.com/v0.1/me with the bearer token, and check that the answer has a merchant_profile. intl/todoist.live.spec.ts shows the pattern. After that and a green CI we can merge.
|
@helpcodeai Updated in commit \cefbe0a! Added the live \GET https://api.sumup.com/v0.1/me\ request asserting that \merchant_profile\ is present in the response when \RUN_SUMUP_LIVE=1\ and \SUMUP_API_KEY\ is configured. Thank you for approving the CI runs! |
|
Merged, thank you @ak-bharadwaj. I merged main into your branch to fix the adapter counts after the Pipefy merge. SumUp will be in the catalog on AnythingMCP Cloud with one of the next updates. |
Summary
Adds a pre-built MCP adapter for the SumUp payments platform (\packages/backend/src/adapters/intl/sumup.json), exposing 6 tools:
Review Updates Completed
ode scripts/regenerate-catalog.mjs.
ode scripts/adapter-count.mjs --check\ across all quoted docs, package, and citation files.
ode scripts/generate-llms-txt.mjs.
Part of #982