mirror of
https://github.com/usestrix/strix.git
synced 2026-08-24 03:42:37 +02:00
0xallam
c996f5efd5
- Strip PLAYBOOK / AUDIT / Phase-N / C-numbered references from module docstrings across 16 files; rename ``_PHASE1_PARALLEL_DEFAULT`` → ``_PARALLEL_TOOL_CALLS_DEFAULT``. - Delete unused exception classes: ``SandboxInitializationError``, ``ImplementedInClientSideOnlyError``. - Delete the no-op ``on_handoff`` hook (we don't use SDK handoffs). - Delete the unreachable backward-compat tab-delimited fallback in ``_parse_git_diff_output``. - Delete orphaned ``strix/tools/load_skill/`` (dir contained only a pycache) and stale pycache files. - Rewrite ``strix/skills/__init__.py``: 168 → 56 LoC. Drop seven helper functions (``get_available_skills``, ``get_all_skill_names``, ``validate_skill_names``, ``parse_skill_list``, ``validate_requested_skills``, ``generate_skills_description``, ``_get_all_categories``) — none had external callers; only ``load_skills`` is used. - Drop the stale ``strix/agents/sdk_factory.py`` per-file ruff ignore (file no longer exists).
57 lines
1.9 KiB
Python
57 lines
1.9 KiB
Python
import logging
|
|
import re
|
|
|
|
from strix.utils.resource_paths import get_strix_resource_path
|
|
|
|
|
|
logger = logging.getLogger(__name__)
|
|
|
|
_FRONTMATTER_PATTERN = re.compile(r"^---\s*\n.*?\n---\s*\n", re.DOTALL)
|
|
|
|
|
|
def load_skills(skill_names: list[str]) -> dict[str, str]:
|
|
"""Load skill markdown bodies (frontmatter stripped) by name.
|
|
|
|
Skill files live at ``strix/skills/<category>/<name>.md``. Names
|
|
can be ``"name"`` (any category), ``"category/name"``, or a bare
|
|
file at the skills root. Missing skills are logged and skipped.
|
|
"""
|
|
skills_dir = get_strix_resource_path("skills")
|
|
if not skills_dir.exists():
|
|
return {}
|
|
|
|
by_category: dict[str, str] = {}
|
|
for category_dir in skills_dir.iterdir():
|
|
if not category_dir.is_dir() or category_dir.name.startswith("__"):
|
|
continue
|
|
for file_path in category_dir.glob("*.md"):
|
|
by_category[file_path.stem] = f"{category_dir.name}/{file_path.stem}.md"
|
|
|
|
skill_content: dict[str, str] = {}
|
|
for skill_name in skill_names:
|
|
rel_path: str | None
|
|
if "/" in skill_name:
|
|
rel_path = f"{skill_name}.md"
|
|
elif skill_name in by_category:
|
|
rel_path = by_category[skill_name]
|
|
elif (skills_dir / f"{skill_name}.md").exists():
|
|
rel_path = f"{skill_name}.md"
|
|
else:
|
|
rel_path = None
|
|
|
|
if rel_path is None or not (skills_dir / rel_path).exists():
|
|
logger.warning("Skill not found: %s", skill_name)
|
|
continue
|
|
|
|
try:
|
|
content = (skills_dir / rel_path).read_text(encoding="utf-8")
|
|
except (OSError, ValueError) as e:
|
|
logger.warning("Failed to load skill %s: %s", skill_name, e)
|
|
continue
|
|
|
|
var_name = skill_name.split("/")[-1]
|
|
skill_content[var_name] = _FRONTMATTER_PATTERN.sub("", content).lstrip()
|
|
logger.info("Loaded skill: %s -> %s", skill_name, var_name)
|
|
|
|
return skill_content
|