wehub-resource-sync
82 lines
2.3 KiB
Python
82 lines
2.3 KiB
Python
"""skill_create / skill_edit frontmatter must round-trip through the loader."""
|
|
|
|
from __future__ import annotations
|
|
|
|
import pytest
|
|
|
|
from opensquilla.skills.loader import SkillLoader
|
|
from opensquilla.tools.builtin import skill_tools
|
|
from opensquilla.tools.registry import get_default_registry
|
|
|
|
|
|
@pytest.fixture()
|
|
def loader(tmp_path):
|
|
workspace = tmp_path / "workspace-skills"
|
|
workspace.mkdir()
|
|
ldr = SkillLoader(
|
|
workspace_dir=workspace,
|
|
snapshot_path=tmp_path / "cache" / "skills_snapshot.json",
|
|
)
|
|
saved = skill_tools._loader
|
|
skill_tools.create_skill_tools(ldr)
|
|
yield ldr
|
|
skill_tools._loader = saved
|
|
|
|
|
|
def _handler(name):
|
|
registered = get_default_registry().get(name)
|
|
assert registered is not None
|
|
return registered.handler
|
|
|
|
|
|
@pytest.mark.parametrize(
|
|
"description",
|
|
[
|
|
"Helps with: budgets and planning",
|
|
"[DRAFT] budget helper",
|
|
"use #tags carefully",
|
|
],
|
|
)
|
|
async def test_skill_create_description_round_trips(loader, description):
|
|
await _handler("skill_create")(
|
|
name="budget-helper",
|
|
description=description,
|
|
content="Do budget things.",
|
|
)
|
|
|
|
spec = loader.get_by_name("budget-helper")
|
|
assert spec is not None
|
|
assert spec.description == description
|
|
|
|
|
|
async def test_skill_create_trigger_with_colon_stays_a_string(loader):
|
|
await _handler("skill_create")(
|
|
name="deploy-helper",
|
|
description="deploy helper",
|
|
content="Deploy things.",
|
|
triggers=["deploy: production"],
|
|
)
|
|
|
|
spec = loader.get_by_name("deploy-helper")
|
|
assert spec is not None
|
|
assert spec.triggers == ["deploy: production"]
|
|
loader.find_by_trigger("please deploy: production now")
|
|
|
|
|
|
async def test_skill_edit_preserves_description_needing_quotes(loader):
|
|
skill_dir = loader.workspace_dir / "notes-helper"
|
|
skill_dir.mkdir()
|
|
(skill_dir / "SKILL.md").write_text(
|
|
'---\nname: notes-helper\ndescription: "Notes: capture and file them"\n---\n\nOld body.\n',
|
|
encoding="utf-8",
|
|
)
|
|
loader.invalidate_cache()
|
|
assert loader.get_by_name("notes-helper") is not None
|
|
|
|
await _handler("skill_edit")(name="notes-helper", content="New body.")
|
|
|
|
spec = loader.get_by_name("notes-helper")
|
|
assert spec is not None
|
|
assert spec.description == "Notes: capture and file them"
|
|
assert spec.content == "New body."
|