-
Notifications
You must be signed in to change notification settings - Fork 2
feat: add reporting tools and misc fix bugs #88
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. Testing This Branch via MCPTo 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/feat/add-mcp-tool-issue-report-tool", "connector-builder-mcp"]
}
}
} Testing This Branch via CLIYou 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/feat/add-mcp-tool-issue-report-tool#egg=airbyte-connector-builder-mcp' --help PR Slash CommandsAirbyte Maintainers can execute the following slash commands on your PR:
|
PyTest Results (Full)0 tests 0 ✅ 0s ⏱️ Results for commit 519e817. ♻️ This comment has been updated with latest results. |
There was a problem hiding this 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 introduces reporting tools for manager-developer agent communication and fixes various bugs while updating dependencies. The primary focus is enabling progress tracking and issue reporting between agents in a connector building workflow.
- Add reporting tools for agents to communicate progress and failures back to managers
- Fix parameter handling and improve formatting in validation testing reports
- Update Pydantic dependency to version 2.11.9 across projects
Reviewed Changes
Copilot reviewed 8 out of 10 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
pyproject.toml | Updates Pydantic dependency version constraint |
poe_tasks.toml | Modifies install and lock commands to handle multi-project setup |
connector_builder_mcp/validation_testing.py | Removes unused parameter, fixes variable naming, and improves report formatting |
connector_builder_agents/src/tools.py | Adds new reporting tools for tool failures and progress tracking |
connector_builder_agents/src/run.py | Integrates handback mechanism between developer and manager agents |
connector_builder_agents/src/guidance.py | Updates manager instructions to use new reporting tools |
connector_builder_agents/src/agents.py | Implements handoff mechanism from developer back to manager |
connector_builder_agents/pyproject.toml | Updates Pydantic dependency to match main project |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
Introduce tools for reporting progress and issues back to the manager.
Summary of Changes:
manifest.yaml
if we have rights to write it. (Fale gracefully otherwise.)@pedroslopez - FYI
Also, added a while back but working better now, CI pushes auto-kickoff a couple connector builds: