fix(skills): prefer frontmatter description over body content in slash command menu#673
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
all-hands-bot
left a comment
There was a problem hiding this comment.
🟢 Good taste — Clean fix that correctly prioritizes explicit metadata over derived content.
Key insight: Swapping the priority order so description (curated frontmatter) takes precedence over content (parsed body) aligns with semantic correctness. Skills with explicit descriptions now display properly instead of showing truncated first-line content.
Risk Assessment: 🟢 LOW
- Single file, 3-line logic swap
- Correct semantic priority (metadata before parsing)
- Visual proof in screenshots
- No breaking changes, security concerns, or eval risk
Was this automated review useful? React with 👍 or 👎 to this review to help us measure review quality.
Workflow run: https://github.com/OpenHands/agent-canvas/actions/runs/26175810678
📸 Snapshot Test ReportWarning Snapshot comparison step crashed (timeout, OOM, or runner error) — diff results below may be incomplete or absent. ✅ All snapshots match the main branch baselines.
✅ Unchanged snapshots (73)
Generated by the Snapshot Tests workflow. This comment was created by an AI agent (OpenHands) on behalf of the repo maintainers. |
Summary
Swap the two branches in SlashCommandMenuItem so a skill's description field is used before falling back to parsing its content. Previously content took priority, so SKILL.md skills showed a truncated first body line instead of their curated frontmatter description. Items without a description still fall back to content parsing.
Snapshots
Before
After
Type
🐳 Docker images for this PR
• GHCR package: https://github.com/OpenHands/agent-canvas/pkgs/container/agent-canvas
ghcr.io/openhands/agent-canvasghcr.io/openhands/agent-server:1.23.1-pythonopenhands-automation==1.0.0a5fc3769f5c563e2f5e02f643f2c06073769fad845Pull (multi-arch manifest)
# Multi-arch manifest — Docker automatically pulls the correct architecture docker pull ghcr.io/openhands/agent-canvas:sha-fc3769fRun
All tags pushed for this build
About Multi-Architecture Support
sha-fc3769f) is a multi-arch manifest supporting both amd64 and arm64sha-fc3769f-amd64) are also available if needed