chore: sync catalog.json with document-writer includes metadata#24402
Conversation
Co-Authored-By: Claude <noreply@anthropic.com>
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
| "name": "watch-together", | ||
| "description": "Watch TV shows and movies with the user in real time by processing screen captures into frames and audio analysis.", | ||
| "metadata": { | ||
| "vellum": { |
There was a problem hiding this comment.
🚩 watch-together emoji is nested under metadata.vellum unlike other skills
The watch-together entry has emoji nested under metadata.vellum.emoji (skills/catalog.json:472-473), while nearly all other skills place it at metadata.emoji (e.g., agentmail at line 10). This is because the watch-together/SKILL.md:4-6 frontmatter nests emoji under vellum: rather than at the top level of metadata:. The catalog generator copies metadata as-is from frontmatter, so this is a pre-existing inconsistency in the SKILL.md, not introduced by this PR. It may affect emoji resolution if the consumer only checks metadata.emoji and not metadata.vellum.emoji.
(Refers to lines 472-473)
Was this helpful? React with 👍 or 👎 to provide feedback.
#24402) Co-authored-by: Claude <noreply@anthropic.com>
1 participant
Summary
skills/catalog.jsonto pick up theincludes: ["document"]field added todocument-writer/SKILL.mdin fix: remove arbitrary 500-word threshold from document-writer skill #24388watch-togethername field (was incorrectly capitalized)Original prompt
can you address the feedback in this PR comment? #24388
🤖 Generated with Claude Code