Skip to content

chore: clarify PyYAML dependency + add CI#24

Closed
elliot245 wants to merge 1 commit into
mainfrom
chore/polish-yaml-ci
Closed

chore: clarify PyYAML dependency + add CI#24
elliot245 wants to merge 1 commit into
mainfrom
chore/polish-yaml-ci

Conversation

@elliot245

Copy link
Copy Markdown
Contributor

Polish pass for agent-manager-skill to reduce installation footguns and make failures more actionable.

Changes:

  • Clarify/declare the real Python dependency: pyyaml is required to parse YAML frontmatter (agents + injected skills).
  • Improve error UX: missing PyYAML now fails with a clear one-line install hint instead of a raw ImportError.
  • doctor now checks for pyyaml and reports a fix command.
  • Add GitHub Actions CI (compile + unittest across Python 3.10/3.12).

This is intentionally small and low-risk: no runtime behavior change when deps are present.

@elliot245 elliot245 closed this Feb 8, 2026
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.

1 participant