-
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]>
| - **CLI**: Place skills in `~/.openhands/skills/` (user-level) or `.openhands/skills/` (repository-level) | ||
| - **GUI**: Place skills in `.openhands/skills/` in your repository root | ||
| - Ensure the file has a `.md` extension (e.g., `my-skill.md`, not `my-skill.txt`) | ||
| - Common mistake: Using folder name `microagents` instead of `skills` |
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.
this shouldn't be a mistake? it should be backward compatible
| - **CLI**: Place skills in `~/.openhands/skills/` (user-level) or `.openhands/skills/` (repository-level) | ||
| - **GUI**: Place skills in `.openhands/skills/` in your repository root | ||
| - Ensure the file has a `.md` extension (e.g., `my-skill.md`, not `my-skill.txt`) | ||
| - Common mistake: Using folder name `microagents` instead of `skills` |
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.
| - Common mistake: Using folder name `microagents` instead of `skills` |
| ### Skills-related Environment Variables | ||
|
|
||
| - `AGENT_ENABLE_PROMPT_EXTENSIONS` - Enable/disable skills (default: `true`) | ||
| - `AGENT_DISABLED_MICROAGENTS` - List of specific skills to disable |
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.
I don't think CLI/SDK support these env vars?
Summary
This PR adds documentation about skills to the CLI section, mostly relying on pointing to the existing documentation in other places.