Skip to content

feat: add lazy per-agent Skillware runtime plumbing#58

Merged
rosspeili merged 2 commits into
ARPAHLS:mainfrom
rosspeili:feat/issue-55-skill-runtime
Jun 17, 2026
Merged

feat: add lazy per-agent Skillware runtime plumbing#58
rosspeili merged 2 commits into
ARPAHLS:mainfrom
rosspeili:feat/issue-55-skill-runtime

Conversation

@rosspeili

@rosspeili rosspeili commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Introduce optional per-agent skills with lazy Skillware loading, execution guardrails, and a two-pass tool-call synthesis flow while preserving existing no-skill behavior. Add focused runtime tests and YAML examples for persona-level skill assignments.

Closes #55

Introduce optional per-agent skills with lazy Skillware loading, execution guardrails, and a two-pass tool-call synthesis flow while preserving existing no-skill behavior. Add focused runtime tests and YAML examples for persona-level skill assignments.
Patch test monkeypatching to target the imported rooms.agent module object directly so CI import resolution stays deterministic.
@rosspeili

Copy link
Copy Markdown
Contributor Author

Pushed a small follow-up fix for CI: 97185c3.

Root cause was monkeypatch path resolution in tests/test_skills_runtime.py under GitHub Actions. Updated the tests to patch litellm.completion on the imported rooms.agent module object directly (agent_module.litellm) for deterministic behavior.

Re-ran in venv:

  • pytest tests/test_skills_runtime.py -q
  • pytest tests/ -q ✅ (41 passed)

@rosspeili rosspeili merged commit 7c96ab9 into ARPAHLS:main Jun 17, 2026
1 check passed
@rosspeili rosspeili deleted the feat/issue-55-skill-runtime branch June 17, 2026 09:42
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.

feat(skill-runtime): add per-agent Skillware integration with lazy loading and guardrails

1 participant