Skip to content

Conversation

@oshchyhol
Copy link
Collaborator

@oshchyhol oshchyhol commented May 19, 2025

⚠️ Depends on https://github.com/railsware/coupler-io-web/pull/6152

Changes

  • send x-coupler-client header with app version set to commit SHA
  • upgrade deps to latest
  • add list-dataflows tool
  • set default fetch timeout to 15s
  • add notes on running MCP inspector in CLI mode

How to test

  • run MCP inspector in UI mode
  • run tool with MCP inspector in CLI mode: npx @modelcontextprotocol/inspector --cli npm run dev --method tools/call --tool-name list-dataflows
  • test the PR image: ghcr.io/railsware/coupler-io-mcp-server:pr-6

@oshchyhol oshchyhol requested a review from MorozovaLiuda May 20, 2025 06:57
npx @modelcontextprotocol/inspector --cli npm run dev --method tools/call --tool-name list-dataflows

# Call get-schema tool
npx @modelcontextprotocol/inspector --cli npm run dev --method tools/call --tool-name get-schema --tool-arg dataflowId=<your data flow ID>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥

import { CouplerioClient } from '@/lib/couplerio-client'

export const handler = async (): Promise<CallToolResult> => {
const coupler = new CouplerioClient({ auth: COUPLER_ACCESS_TOKEN })

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does it make sense to move COUPLER_ACCESS_TOKEN to CouplerioClient?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

might be a good idea to move it, once it's more final. I kept it this way in case we have any other auth types. Included this in a task for API SDK improvements https://railsware.atlassian.net/browse/CPL-20383

@oshchyhol oshchyhol requested a review from MorozovaLiuda May 20, 2025 14:36
@oshchyhol oshchyhol merged commit 3e51aaa into main May 21, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants