Repository navigation
docs(mcp): point deploy_connector_to_cloud and publish_custom_source_definition at each other - #1371
Conversation
…definition at each other Co-Authored-By: AJ Steers <aj@airbyte.io>
|
I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".
|
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. 💡 Show Tips and TricksTesting This PyAirbyte VersionYou can test this version of PyAirbyte using the following: # Run PyAirbyte CLI from this branch:
uvx --from 'git+https://github.com/airbytehq/PyAirbyte.git@devin/1791941748-custom-connector-tool-pointers' pyairbyte --help
# Install PyAirbyte from this branch for development:
pip install 'git+https://github.com/airbytehq/PyAirbyte.git@devin/1791941748-custom-connector-tool-pointers'PR Slash CommandsAirbyte Maintainers can execute the following slash commands on your PR:
📚 Show Repo GuidanceHelpful ResourcesCommunity SupportQuestions? Join the #pyairbyte channel in our Slack workspace. |
Code Coverage OverviewLanguages: Python Python / code-coverage/pytest-no-credsThe overall line coverage in commit 65e42cd in the Show a line coverage summary of the most impacted files.
Python / code-coverage/pytestThe overall line coverage in commit 65e42cd in the Show a line coverage summary of the most impacted files.
Python / code-coverage/pytest-unitThe overall line coverage in commit 65e42cd in the Show a line coverage summary of the most covered files.
Updated |
|
Devin, check for newly merged tool updates for supporting custom connectors' pointers in these tools. Based on that, decide if this should be closed as no longer needed. |
Overview
👉 TL;DR: Tells agents which Cloud MCP tool to use for custom connectors versus connectors from the Airbyte registry. In prod telemetry, agents keep calling the registry-only deploy tool with custom connector names.
Specifically, this changes only the tool descriptions: the
deploy_connector_to_clouddocstring and itsconnector_namefield, plus thepublish_custom_source_definitiondocstring.Changes
deploy_connector_to_clouddocstring: says it works only for registry connectors, and for custom (YAML manifest) source connectors points topublish_custom_source_definition, or tolist_custom_source_definitionsandupdate_custom_source_definitionwhen the definition already exists.deploy_connector_to_cloudconnector_namefield description: says it takes the canonical registry name, not a custom definition name or ID.publish_custom_source_definitiondocstring: points back todeploy_connector_to_cloudfor registry connectors.Why
In the last 24h of hosted Cloud MCP traffic, 4
deploy_connector_to_cloudcalls failed withAirbyteConnectorNotRegisteredError. Theconnector_namevalues were custom connector names or a definition UUID. The tool descriptions never said which tool to use instead.Test plan
ruff format/ruff checkonairbyte/mcp/cloud.pypytest tests/unit_tests/test_mcp_tool_utils.py tests/unit_tests/test_mcp_cloud.py tests/unit_tests/test_mcp_docs_includes.py tests/unit_tests/test_deferred_cloud_credentials.py: 270 passedLink to Devin session: https://app.devin.ai/sessions/8fbdc1e4380941958b0bed24914614f2
Open in Devin Desktop: https://app.devin.ai/desktop/session/8fbdc1e4380941958b0bed24914614f2?variant=devin
Requested by: Aaron ("AJ") Steers (@aaronsteers)