Skip to content

Conversation

@dalton-turner
Copy link
Collaborator

@dalton-turner dalton-turner commented Dec 17, 2025

Overview

Added support for skills generators in the agent rule system.

Added a new skills_generator() method to the AgentRuleGenerator trait that returns an optional SkillsGeneratorTrait implementation. This method allows agents to provide a skills generator for creating user-defined skill symlinks. The default implementation returns None for now.

Why make this change?

This change enables support for user-defined skills in compatible agents. By adding this trait method, we establish a standardized way for agents to expose skills functionality, making it easier to implement consistent skills support across different agent types while maintaining backward compatibility with agents that don't support skills.

  • Test Binary

@dalton-turner dalton-turner force-pushed the dalton/add-skills-generator-trait-and-skills-reader-module branch from 2e01e55 to 0af5341 Compare December 17, 2025 18:32
@dalton-turner dalton-turner force-pushed the dalton/add-skills-generator-to-agent-rule-generator branch 2 times, most recently from 471f9eb to ef5d64a Compare December 17, 2025 18:54
@dalton-turner dalton-turner force-pushed the dalton/add-skills-generator-trait-and-skills-reader-module branch from 0af5341 to 686ace1 Compare December 17, 2025 18:54
Base automatically changed from dalton/add-skills-generator-trait-and-skills-reader-module to main December 17, 2025 19:08
@dalton-turner dalton-turner force-pushed the dalton/add-skills-generator-to-agent-rule-generator branch from ef5d64a to 83c55f7 Compare December 17, 2025 19:16
@dalton-turner dalton-turner merged commit 59b4586 into main Dec 17, 2025
7 checks passed
Copy link
Collaborator Author

Merge activity

@dalton-turner dalton-turner deleted the dalton/add-skills-generator-to-agent-rule-generator branch December 17, 2025 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants