Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions skills/catalog.json
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,10 @@
"metadata": {
"emoji": "📝",
"vellum": {
"display-name": "Document Writer"
"display-name": "Document Writer",
"includes": [
"document"
]
}
},
"compatibility": "Designed for Vellum personal assistants"
Expand Down Expand Up @@ -463,7 +466,7 @@
},
{
"id": "watch-together",
"name": "Watch Together",
"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": {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚩 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)

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Expand Down
Loading