fix-ci

[josefkarasek]

No description provided.

[Copilot]

Pull request overview

This PR adjusts the scripts/common.py utilities to avoid importing PyYAML at module import time, which helps prevent failures in environments (e.g., CI steps) that import these helpers but don’t need YAML functionality.

Changes:

• Removed the top-level import yaml from scripts/common.py.
• Added deferred import yaml statements inside load_yaml() and parse_skill_frontmatter() so YAML is only required when those functions are called.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

Suppressed comments (2)

scripts/common.py:39

• Same as load_yaml(): when PyYAML isn’t installed, the import error here is hard to interpret for users running the script. Wrapping it with a short, actionable message would make failures clearer.

def parse_skill_frontmatter(skill_md_path: Path) -> dict[str, Any]:
    import yaml

scripts/common.py:30

• If PyYAML isn’t installed, this will raise a generic ModuleNotFoundError: No module named 'yaml' without context. Consider catching it and raising a clearer message indicating PyYAML is an optional dependency and how to install it.

This issue also appears on line 37 of the same file.

def load_yaml(path: Path) -> Any:
    import yaml

    with path.open(encoding="utf-8") as handle:
        return yaml.safe_load(handle)