-
Notifications
You must be signed in to change notification settings - Fork 6
docs: Add documentation of skills to the CLI section #156
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
This commit addresses user confusion about where to place skills when using the OpenHands CLI and how to verify they are working correctly. Changes made: - Added comprehensive 'Using Skills with CLI' section to cli-mode.mdx explaining: - Both skill locations: ~/.openhands/skills/ (user-level) and .openhands/skills/ (repo-level) - How to create general and keyword-triggered skills - How to verify skills are working - Note about microagents vs skills naming - Updated skills overview matrix to clarify CLI supports both locations - Enhanced CLI platform-specific differences to show both skill locations - Added extensive troubleshooting section to keyword.mdx covering: - Common file location issues - Frontmatter formatting problems - How to verify skills are enabled - Trigger keyword best practices - Common mistakes (microagents folder, YAML formatting, etc.) - Step-by-step testing procedure - Updated cli-settings.mdx to add Skills Configuration section with: - Default skill locations - Environment variables for controlling skills - Link to troubleshooting guide These changes help users understand that: 1. Skills work in CLI mode with file-based configuration 2. Skills can be placed in two locations (user-level and repo-level) 3. The folder should be named 'skills' not 'microagents' 4. Skills are enabled by default 5. How to troubleshoot when skills don't trigger as expected Fixes documentation gap reported by users who couldn't get skills working in CLI mode.
- Replace verbose skills documentation with concise summary - Add links to existing comprehensive skills documentation - Remove redundant examples already covered in skills overview - Maintains essential information about user and repo skill locations This addresses feedback to make CLI docs more concise and avoid duplication of content that already exists in /overview/skills. Co-authored-by: openhands <[email protected]>
|
|
I'm on it! neubig can track my progress at all-hands.dev |
- Remove Skills Support section from cli-mode.mdx (per user request) - Make skills documentation in cli-settings.mdx self-contained with: - Clear explanation of what skills are (markdown files, always-active or keyword-triggered) - Skill locations (user-level and repo-level) - Skill types with links to detailed docs - Environment variables for configuration - Comprehensive links to all related documentation - Simplify troubleshooting in keyword.mdx to first 3 bullets: - Kept critical information from other bullets (folder name mistake, case sensitivity) - Folded common mistakes into frontmatter section - Consolidated testing guidance into bullet 3 - Update all cross-references in overview/skills.mdx to point to cli-settings.mdx instead of removed cli-mode.mdx section All documentation now consistently points to cli-settings.mdx#skills-configuration for CLI skills information. Co-authored-by: openhands <[email protected]>
|
@OpenHands reflect xingyaoww's review comments. |
|
I'm on it! neubig can track my progress at all-hands.dev |
- Remove Skills Configuration section from cli-settings.mdx (CLI/SDK don't support those env vars) - Remove Troubleshooting section from keyword.mdx Co-authored-by: openhands <[email protected]>
|
I've addressed xingyaoww's review comments on PR #156. The changes have been pushed to the branch. Summary of ChangesBased on xingyaoww's review comments:
Both files have been reverted to their original state from main, removing the additions that were flagged in the review. Commit: The PR has been updated: #156 |
Summary
This PR adds documentation about skills to the CLI section, mostly relying on pointing to the existing documentation in other places.