Skip to content

Conversation

aaronsteers
Copy link
Contributor

No description provided.

@Copilot Copilot AI review requested due to automatic review settings August 8, 2025 19:00
Copy link

github-actions bot commented Aug 8, 2025

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

Testing This Branch via MCP

To test the changes in this specific branch with an MCP client like Claude Desktop, use the following configuration:

{
  "mcpServers": {
    "connector-builder-mcp-dev": {
      "command": "uvx",
      "args": ["--from", "git+https://github.com/airbytehq/connector-builder-mcp.git@aj/fix/misc", "connector-builder-mcp"]
    }
  }
}

Testing This Branch via CLI

You can test this version of the MCP Server using the following CLI snippet:

# Run the CLI from this branch:
uvx 'git+https://github.com/airbytehq/connector-builder-mcp.git@aj/fix/misc#egg=airbyte-connector-builder-mcp' --help

PR Slash Commands

Airbyte Maintainers can execute the following slash commands on your PR:

  • /autofix - Fixes most formatting and linting issues
  • /poe <command> - Runs any poe command in the uv virtual environment

📝 Edit this welcome message.

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes an issue where the manifest retrieval function was incorrectly returning metadata.yaml files instead of always returning manifest.yaml files for connector manifests.

  • Simplifies the function to always fetch manifest.yaml files only
  • Adds validation to return an error for non-manifest-only connectors
  • Updates error messages and documentation examples to be more consistent

Copy link

github-actions bot commented Aug 8, 2025

PyTest Results (Full)

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 4111ac8. ± Comparison against base commit 620ad19.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Aug 8, 2025

PyTest Results (Fast)

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 4111ac8. ± Comparison against base commit 620ad19.

♻️ This comment has been updated with latest results.

@aaronsteers
Copy link
Contributor Author

/format-fix

@aaronsteers
Copy link
Contributor Author

aaronsteers commented Aug 8, 2025

/autofix

Auto-Fix Job Info

This job attempts to auto-fix any linting or formating issues. If any fixes are made,
those changes will be automatically committed and pushed back to the PR.

Note: This job can only be run by maintainers. On PRs from forks, this command requires
that the PR author has enabled the Allow edits from maintainers option.

PR auto-fix job started... Check job output.

✅ Changes applied successfully.

@aaronsteers aaronsteers merged commit 5302914 into main Aug 9, 2025
13 checks passed
@aaronsteers aaronsteers deleted the aj/fix/misc branch August 9, 2025 00:05
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.

1 participant