feat(skiff): replace opencode target with agents standard dir
- targets: cursor, claude, codex, agents (~/.agents/skills, .agents/skills) - agents dir covers OMP native discovery; drop omp/opencode targets - project-level agents/cursor/codex share .agents/skills via idempotent symlinks - migrate local state: remove ~/.config/opencode/skills links, reinstall into ~/.agents/skills
This commit is contained in:
@@ -0,0 +1,108 @@
|
||||
from __future__ import annotations
|
||||
|
||||
import os
|
||||
import subprocess
|
||||
import sys
|
||||
import tempfile
|
||||
import unittest
|
||||
from pathlib import Path
|
||||
|
||||
|
||||
REPO_ROOT = Path(__file__).resolve().parents[1]
|
||||
|
||||
|
||||
def write_skill(root: Path, name: str) -> None:
|
||||
skill = root / "skills" / name
|
||||
skill.mkdir(parents=True)
|
||||
skill.joinpath("SKILL.md").write_text(
|
||||
f"---\nname: {name}\ndescription: >-\n"
|
||||
f" 测试 {name} skill。\n---\n\n# {name}\n",
|
||||
encoding="utf-8",
|
||||
)
|
||||
|
||||
|
||||
class AgentsTargetTests(unittest.TestCase):
|
||||
def setUp(self) -> None:
|
||||
self.temp_dir = tempfile.TemporaryDirectory()
|
||||
self.home = Path(self.temp_dir.name)
|
||||
self.skills_home = self.home / ".skills"
|
||||
self.skills_home.mkdir(parents=True)
|
||||
(self.skills_home / "catalog.yaml").write_text("", encoding="utf-8")
|
||||
write_skill(self.skills_home, "demo-skill")
|
||||
|
||||
def tearDown(self) -> None:
|
||||
self.temp_dir.cleanup()
|
||||
|
||||
def run_skiff(self, *args: str) -> subprocess.CompletedProcess[str]:
|
||||
env = os.environ.copy()
|
||||
env["HOME"] = str(self.home)
|
||||
env["PYTHONPATH"] = str(REPO_ROOT)
|
||||
return subprocess.run(
|
||||
[sys.executable, "-m", "skiff", *args],
|
||||
cwd=REPO_ROOT,
|
||||
env=env,
|
||||
text=True,
|
||||
capture_output=True,
|
||||
check=False,
|
||||
)
|
||||
|
||||
def test_global_add_creates_symlink_in_agents_skills(self) -> None:
|
||||
result = self.run_skiff("add", "builtin/demo-skill", "-g", "-a", "agents")
|
||||
|
||||
self.assertEqual(result.returncode, 0, result.stderr)
|
||||
link = self.home / ".agents" / "skills" / "demo-skill"
|
||||
self.assertTrue(link.is_symlink())
|
||||
self.assertEqual(
|
||||
link.resolve(), (self.skills_home / "skills" / "demo-skill").resolve()
|
||||
)
|
||||
|
||||
removed = self.run_skiff("rm", "builtin/demo-skill", "-g", "-a", "agents")
|
||||
self.assertEqual(removed.returncode, 0, removed.stderr)
|
||||
self.assertFalse(link.exists())
|
||||
|
||||
def test_project_add_targets_agents_and_codex_share_single_link(self) -> None:
|
||||
project = self.home / "app"
|
||||
project.mkdir()
|
||||
|
||||
added = self.run_skiff(
|
||||
"add",
|
||||
"builtin/demo-skill",
|
||||
"--project",
|
||||
str(project),
|
||||
"-a",
|
||||
"agents",
|
||||
"-a",
|
||||
"codex",
|
||||
)
|
||||
|
||||
self.assertEqual(added.returncode, 0, added.stderr)
|
||||
# agents 与 codex 项目级共用 .agents/skills,软链幂等只写一份
|
||||
links = list((project / ".agents" / "skills").glob("demo-skill"))
|
||||
self.assertEqual(len(links), 1)
|
||||
self.assertTrue(links[0].is_symlink())
|
||||
manifest = (project / ".skills.yaml").read_text(encoding="utf-8")
|
||||
self.assertIn("demo-skill", manifest)
|
||||
|
||||
removed = self.run_skiff(
|
||||
"rm", "builtin/demo-skill", "--project", str(project), "-y"
|
||||
)
|
||||
self.assertEqual(removed.returncode, 0, removed.stderr)
|
||||
self.assertFalse((project / ".agents" / "skills" / "demo-skill").exists())
|
||||
|
||||
def test_status_lists_installed_skill_for_agents_target(self) -> None:
|
||||
installed = self.run_skiff("add", "builtin/demo-skill", "-g", "-a", "agents")
|
||||
status = self.run_skiff("status", "-a", "agents")
|
||||
|
||||
self.assertEqual(installed.returncode, 0, installed.stderr)
|
||||
self.assertEqual(status.returncode, 0, status.stderr)
|
||||
self.assertIn("demo-skill", status.stdout)
|
||||
|
||||
def test_opencode_target_is_no_longer_supported(self) -> None:
|
||||
result = self.run_skiff("add", "builtin/demo-skill", "-g", "-a", "opencode")
|
||||
|
||||
self.assertNotEqual(result.returncode, 0)
|
||||
self.assertIn("未知 agent", result.stderr)
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
unittest.main()
|
||||
@@ -83,13 +83,13 @@ class CreateWorkflowTests(unittest.TestCase):
|
||||
".cursor/skills/skiff",
|
||||
".claude/skills/skiff",
|
||||
".codex/skills/skiff",
|
||||
".agents/skills/skiff",
|
||||
):
|
||||
link = self.home / relative
|
||||
self.assertTrue(link.is_symlink(), relative)
|
||||
self.assertEqual(link.resolve(), expected)
|
||||
self.assertIn("cursor", first.stdout)
|
||||
self.assertIn("claude", first.stdout)
|
||||
self.assertIn("codex", first.stdout)
|
||||
for agent in ("cursor", "claude", "codex", "agents"):
|
||||
self.assertIn(agent, first.stdout)
|
||||
|
||||
def test_update_pulls_skills_home(self) -> None:
|
||||
with (
|
||||
|
||||
@@ -32,6 +32,7 @@ class InstallScriptTests(unittest.TestCase):
|
||||
".cursor/skills/skiff",
|
||||
".claude/skills/skiff",
|
||||
".codex/skills/skiff",
|
||||
".agents/skills/skiff",
|
||||
):
|
||||
link = home / relative
|
||||
self.assertTrue(link.is_symlink(), relative)
|
||||
|
||||
@@ -78,7 +78,7 @@ class SelectorTests(unittest.TestCase):
|
||||
"ack",
|
||||
"builtin",
|
||||
installed=False,
|
||||
readonly_status="全局: cursor,claude,codex,opencode",
|
||||
readonly_status="全局: cursor,claude,codex,agents",
|
||||
)
|
||||
]
|
||||
screen = Mock()
|
||||
@@ -101,7 +101,7 @@ class SelectorTests(unittest.TestCase):
|
||||
if call.args[0] == 3
|
||||
)
|
||||
self.assertIn("[ ] ack", title)
|
||||
self.assertIn("全局: cursor,claude,codex,opencode", title)
|
||||
self.assertIn("全局: cursor,claude,codex,agents", title)
|
||||
self.assertNotIn("初始化", title)
|
||||
|
||||
def test_selector_renders_description_on_indented_second_line(self) -> None:
|
||||
@@ -370,7 +370,7 @@ class SelectCommandTests(unittest.TestCase):
|
||||
other.mkdir()
|
||||
global_dirs = {
|
||||
target: root / target / "skills"
|
||||
for target in ("cursor", "claude", "codex", "opencode")
|
||||
for target in ("cursor", "claude", "codex", "agents")
|
||||
}
|
||||
for directory in global_dirs.values():
|
||||
directory.mkdir(parents=True)
|
||||
@@ -385,7 +385,7 @@ class SelectCommandTests(unittest.TestCase):
|
||||
note = cli._global_installation_note(
|
||||
"ack",
|
||||
expected,
|
||||
["cursor", "claude", "codex", "opencode"],
|
||||
["cursor", "claude", "codex", "agents"],
|
||||
)
|
||||
|
||||
self.assertEqual(note, "全局: cursor;全局同名冲突: claude")
|
||||
|
||||
Reference in New Issue
Block a user