-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Update documentation for command-style microagents with user input support #8990
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
base: main
Are you sure you want to change the base?
Conversation
…d variable prompts
Co-authored-by: Engel Nyst <[email protected]>
Co-authored-by: Engel Nyst <[email protected]>
Co-authored-by: Engel Nyst <[email protected]>
Looks like there are a few issues preventing this PR from being merged!
If you'd like me to help, just leave a comment, like
Feel free to include any additional details that might help me get this PR into a better state. You can manage your notification settings |
@OpenHands please fix the failing actions on PR #8990 |
I'm on it! xingyaoww can track my progress at all-hands.dev |
- Add missing newlines at end of microagent files (fixed by pre-commit) - Add pytest-asyncio dependency to fix async test execution - All non-Docker tests now pass
Summary: Fixed Failing Actions on PR #8990I have successfully identified and fixed the failing GitHub Actions on PR #8990. Here's what I accomplished: Issues Identified and Fixed:
Changes Made:
Verification:
The changes are minimal and targeted - only fixing the specific issues that were causing the CI failures. Based on my local testing, the new workflow runs should now pass successfully. Link to PR: #8990 |
- Update test assertions to expect filename as microagent name instead of 'default' - Remove debug print statements from microagent.py - Revert pytest-asyncio dependency addition as requested - All tests now pass with the new filename-based naming behavior
- Change test filenames from 'test.md' to match expected microagent names - Use 'default.md' for tests expecting 'default' name - Use 'custom_name.md' for test expecting 'custom_name' name - Use 'test_agent.md' for test expecting 'test_agent' name - This properly tests the filename-based naming behavior
This PR updates the microagent documentation to include task microagents and MCP tools support. It also clarifies field requirements in the microagent frontmatter.
Changes:
name
field from all microagent files (using filename instead)type
field is optional and inferred from other fields:inputs
exist ->TASK
triggers
exist ->KNOWLEDGE
REPO
version
,agent
, andauthor
fields are kept for backward compatibility but not usedRelated to PR #8139 which adds task microagent support.
To run this PR locally, use the following command: