allod skill: list agent skills, or print one in full #256
No reviewers
Labels
No labels
blocked
bug
decision
duplicate
enhancement
help wanted
invalid
landed?
question
ready-to-merge
stale
wontfix
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
allod/tools!256
Loading…
Reference in a new issue
No description provided.
Delete branch "agent/skills-list"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
allod skillmakes skill discovery a command instead of an incantation an agent has to re-derive: bareallod skilllists every available skill (frontmatter name plus a brief description, truncated near 100 characters), andallod skill <name>prints that skill's SKILL.md in full, so an agent never needs to know where the skills directory lives.What changes for a running machine: a new read-only
skillnamespace in theallodCLI. Skills come fromallod/memory'sskills/directory by default, resolved through the repository registry and joined withWORK_DIR;--from <registry-id|dir>(repeatable, comma-separated) searches elsewhere. Nothing else changes: no existing command or output is touched, and the namespace carries no build tag.If it is wrong: a wrong listing or a wrong file printed is visible by running the command; a straight revert of this PR removes the namespace entirely, and nothing persists outside the process.
Refs allod/memory#73 and allod/memory#74 (the pointer PR in
allod/memorylands after this one).Risk
R1. Read-only command, no state mutated, no secrets, no cross-repo ordering on this side; worst credible failure is wrong or empty output, immediately visible by running the command. Rollback is a straight revert.
Validation
go test ./...green in all four build-tag combinations ('',-tags site,-tags secret,-tags site,secret), plusgofmt -landgo vetclean.cmd/allod/skill_test.gocover the sorted list, brief-description truncation, whole-file printing for one and two names, the unknown-name refusal that lists what is available, block-scalar folding, the name fallback, the no-SKILL.md skip warning, the no-skills refusal,--fromcomma and repeat accumulation, registry-id resolution throughINVENTORY/WORK_DIR, theallod/memorydefault and its refusal when the registry lacks the repo, unknown targets, and option handling.allod skilllisted all sixallod/memoryskills with truncated descriptions;allod skill pick-next-issueprinted that SKILL.md verbatim;allod skill nothererefused with exit 1, naming the available skills.Owner redirect: drop the
listsubcommand and make the namespace singular and self-sufficient. Bareallod skilllists the available skills (name plus a brief description);allod skill <name>prints that skill's SKILL.md in full, so an agent never needs to know where the skills directory lives.--from <registry-id|dir>(repeatable, comma-separated) stays for looking somewhere other thanallod/memory.allod skills list: one command that prints every skill's triggerto allod skill: list agent skills, or print one in full