Skip to content

fix: add missing name field to 7 scaffolded skill templates#30

Merged
sonupreetam merged 1 commit into
unbound-force:mainfrom
yvonnedevlinrh:fix/skill-template-names
Jun 30, 2026
Merged

fix: add missing name field to 7 scaffolded skill templates#30
sonupreetam merged 1 commit into
unbound-force:mainfrom
yvonnedevlinrh:fix/skill-template-names

Conversation

@yvonnedevlinrh

Copy link
Copy Markdown
Contributor

Summary

The 7 skill templates embedded in internal/agentkit/content/skills/ were missing the required name field in their YAML frontmatter. OpenCode's skill discovery requires both name and description fields to register a skill. Without name, skills scaffolded by replicator init were invisible to the skill tool and could not be loaded.

Changes

7 SKILL.md templates — added name: <directory-name> as the first frontmatter field:

Template Added field
always-on-guidance/SKILL.md name: always-on-guidance
forge-coordination/SKILL.md name: forge-coordination
forge-global/SKILL.md name: forge-global
learning-systems/SKILL.md name: learning-systems
replicator-cli/SKILL.md name: replicator-cli
system-design/SKILL.md name: system-design
testing-patterns/SKILL.md name: testing-patterns

Regression test — added TestSkillTemplates_HaveNameField in internal/agentkit/agentkit_test.go that walks the embedded filesystem and asserts every SKILL.md has a name field matching its directory name.

Context

Discovered via unbound-force/unbound-force#309. The live copies in unbound-force/unbound-force are being fixed separately — this PR fixes the replicator scaffold source so future replicator init runs produce discoverable skills.

Verification

  • make check passes (vet + all tests)
  • New test verifies all 7 templates and will catch future regressions

Closes #28


Signed-off-by: Yvonne Devlin ydevlin@redhat.com
Assisted-by: OpenCode (claude-opus-4-6)

The 7 skill templates embedded in internal/agentkit/content/skills/
were missing the required name field in their YAML frontmatter.
OpenCode's skill discovery requires both name and description fields
to register a skill. Without name, skills scaffolded by replicator
init were invisible to the skill tool.

Add name: <directory-name> as the first frontmatter field in each
SKILL.md template. Add TestSkillTemplates_HaveNameField regression
test that walks the embedded filesystem and asserts every SKILL.md
has a name field matching its directory name.

Closes unbound-force#28

Assisted-by: OpenCode (claude-opus-4-6)
Signed-off-by: Yvonne Devlin <ydevlin@redhat.com>
@yvonnedevlinrh yvonnedevlinrh force-pushed the fix/skill-template-names branch from 4a2d683 to 7742cf4 Compare June 30, 2026 15:24

@sonupreetam sonupreetam left a comment

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.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review 👀

Development

Successfully merging this pull request may close these issues.

fix: add missing name field to 7 scaffolded skill templates

2 participants