Skip to content

Commit 2460b18

Browse files
phernandezclaude
andcommitted
fix(skills): correct invalid picoschema enum YAML in memory skills
The Task schema in memory-tasks/SKILL.md and the enum examples in memory-schema/SKILL.md used a bare YAML flow sequence followed by a trailing description: status?(enum): [active, blocked, done, abandoned], current state A flow sequence ([...]) cannot be followed by ', current state' — python-frontmatter/PyYAML raises a ParserError. In schema_router.py, _schema_frontmatter_from_file catches that error and silently falls back to DB metadata, so the schema's enum constraints never load. Per the picoschema parser docstring, enum descriptions belong inside the parens: status?(enum, current state): [active, blocked, done, abandoned] Fixes 4 lines (1 in memory-tasks, 3 in memory-schema). The memory-literary-analysis schemas use the quoted-string form ("role(enum)": "[...], desc"), which is valid YAML handled by _parse_enum_string, so they were left unchanged. Verified: all yaml schema blocks now parse via parse_schema_note, and just package-check-skills passes (10 skills validated). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: phernandez <paul@basicmachines.co>
1 parent ec81f72 commit 2460b18

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

skills/memory-schema/SKILL.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,14 +47,14 @@ Use `(enum)` with a list of allowed values:
4747

4848
```yaml
4949
schema:
50-
status(enum): [active, blocked, done, abandoned], current state
50+
status(enum, current state): [active, blocked, done, abandoned]
5151
```
5252

5353
Optional enum:
5454

5555
```yaml
5656
schema:
57-
priority?(enum): [low, medium, high, critical], task priority
57+
priority?(enum, task priority): [low, medium, high, critical]
5858
```
5959

6060
### Arrays
@@ -100,7 +100,7 @@ schema:
100100
attendees?(array): Person, who attended
101101
decisions?(array): string, decisions made
102102
action_items?(array): string, follow-up tasks
103-
status?(enum): [scheduled, completed, cancelled], meeting state
103+
status?(enum, meeting state): [scheduled, completed, cancelled]
104104
settings:
105105
validation: warn
106106
---

skills/memory-tasks/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ entity: Task
2626
version: 1
2727
schema:
2828
description: string, what needs to be done
29-
status?(enum): [active, blocked, done, abandoned], current state
29+
status?(enum, current state): [active, blocked, done, abandoned]
3030
assigned_to?: string, who is working on this
3131
steps?(array): string, ordered steps to complete
3232
current_step?: integer, which step number we're on (1-indexed)

0 commit comments

Comments
 (0)