Skip to content

feat(delegate): skill inheritance for subagents#3631

Open
Mibayy wants to merge 1 commit into
NousResearch:mainfrom
Mibayy:feat/delegate-skill-injection
Open

feat(delegate): skill inheritance for subagents#3631
Mibayy wants to merge 1 commit into
NousResearch:mainfrom
Mibayy:feat/delegate-skill-injection

Conversation

@Mibayy

@Mibayy Mibayy commented Mar 28, 2026

Copy link
Copy Markdown
Contributor

Enables passing skills=["skill-name"] in the task dict to inject skill content into subagent system prompts.

What this does

  • Subagents can now load skills — closes a real gap where parent knowledge doesn't transfer
  • Skill content is injected into the child system prompt under a "Loaded skills" section
  • Gracefully skips missing skills (no crash)

Implementation

  • _load_skill_content(name) searches skills/ and ~/.hermes/skills/ for SKILL.md
  • _build_child_system_prompt() accepts optional skills list
  • delegate_task() passes t.get("skills") through to child agent construction

Tests

5 tests covering:

  • Skill content lookup (found/not found)
  • Prompt injection
  • Missing skills silently skipped
  • No crash when skills omitted

All existing delegate tests still pass.


This PR addresses one of the three items requested in #3387 review. Two more PRs coming:

  1. ✅ Skill injection (this PR)
  2. MCP workspace guard
  3. Configurable max_depth

@alt-glitch alt-glitch added type/feature New feature or request P2 Medium — degraded but workaround exists tool/delegate Subagent delegation tool/skills Skills system (list, view, manage) labels May 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P2 Medium — degraded but workaround exists tool/delegate Subagent delegation tool/skills Skills system (list, view, manage) type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants