-
-
Notifications
You must be signed in to change notification settings - Fork 69.3k
CI: skills-python tests fail due to missing PyYAML dependency #24342
Description
Problem
The skills-python CI test job is failing during test collection with:
ModuleNotFoundError: No module named 'yaml'
Root Cause
The test file skills/skill-creator/scripts/test_quick_validate.py imports from quick_validate.py:
from quick_validate import validate_skillAnd quick_validate.py contains:
import yamlHowever, PyYAML is not installed in the GitHub Actions CI environment. The test fails during collection (before any tests even run) because the module import fails.
Evidence from CI
skills/skill-creator/scripts/test_quick_validate.py:10: in <module>
from quick_validate import validate_skill
skills/skill-creator/scripts/quick_validate.py:11: in <module>
import yaml
E ModuleNotFoundError: No module named 'yaml'
Suggested Fixes
Choose one of the following approaches:
Option 1: Add PyYAML to requirements
Add pyyaml to the appropriate requirements file (e.g., skills/skill-creator/requirements.txt or a test-specific requirements file).
Option 2: Update CI workflow
Install PyYAML in the skills-python CI job:
- name: Install dependencies
run: |
pip install pyyaml
# ... other dependenciesOption 3: Mock the import (if testing without real yaml parsing)
If test_quick_validate.py doesn't need actual yaml parsing, stub the yaml import in the test setup.
Impact
This is blocking the skills-python CI check on all PRs. While it doesn't affect the core TypeScript/Node.js functionality, it prevents validating Python skill validation scripts.
Related
This issue was discovered while working on:
- fix(telegram): disable autoSelectFamily by default on Node 22+ #24259 (Telegram network config)
- fix: prevent thinking block corruption during context compaction #24261 (Thinking blocks corruption fix)
Both PRs pass all TypeScript/agent tests but are blocked by this pre-existing Python dependency issue.