Repository navigation
fix(mcp): guide Connector Builder agents through the credentials flow - #1410
Conversation
Co-Authored-By: AJ Steers <aj@airbyte.io>
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/1792438676-connector-builder-credentials-flow' pyairbyte --help
# Install PyAirbyte from this branch for development:
pip install 'git+https://github.com/airbytehq/PyAirbyte.git@devin/1792438676-connector-builder-credentials-flow'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-creds
View coverage of the 10 most impacted files.
Python / code-coverage/pytest
View coverage of the 10 most impacted files.
Python / code-coverage/pytest-unit
Line coverage data for the devin/1792350894-con... branch is not yet available. View coverage of the 10 most covered files.
ℹ️ Calculations represent line coverage. Results may change as calculations are updated. Updated |
Overview
👉 TL;DR: Guides agents building a connector that needs credentials. They create a draft test source up front, and they ask the user to enter credentials in both the Connector Builder testing values and that test source, never in chat.
Specifically, this updates the
create_connector_builder_projectdocstring, returnsnext_stepsfrom it, and makes the readiness testing-values check fail while a required field has no value.Stacked on #1407:
Changes
create_connector_builder_projectdocstring: for credentialed connectors, put the spec (secrets markedairbyte_secret) in the initial manifest and setcreate_test_source. The test source is saved as a draft (deferred credentials), so credentials nobody has tested yet don't block it.credentialsobject whoseoneOfoptions each have a constantauth_type(api_key,access_token,oauth2.0), andSelectiveAuthenticatorwithauthenticator_selection_path: ["credentials", "auth_type"]when there is more than one option.next_steps. With a test source, it asks the user to enter credentials atconnector_builder_project_urland attest_source_settings_url(then test and save).Test plan
next_stepswith and without a test source, and for the testing-values check (missing required, masked secret, no inputs, required with default).cloud-mcp-previewwith a credentialed source: pending.Link to Devin session: https://app.devin.ai/sessions/37c00a83f788401489bfd0e37c0b7440
Open in Devin Desktop: https://app.devin.ai/desktop/session/37c00a83f788401489bfd0e37c0b7440?variant=devin
Requested by: Aaron ("AJ") Steers (@aaronsteers)