s-skillkit 0.1.2__tar.gz → 0.1.3__tar.gz

{s_skillkit-0.1.2 → s_skillkit-0.1.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: s-skillkit
-Version: 0.1.2
+Version: 0.1.3
 Summary: Локальное ядро управления навыками: install из git/локальной папки, junction/symlink-линковка в scope агента, .skillignore-фильтр, build manifest, project-манифест. 0 завязок на сеть/auth — stdlib + tomli-w + pathspec + platformdirs.
 Author: Dmitry
 License: MIT

{s_skillkit-0.1.2 → s_skillkit-0.1.3}/pyproject.toml

@@ -1,48 +1,48 @@
-[project]
-name = "s-skillkit"
-version = "0.1.2"
-description = "Локальное ядро управления навыками: install из git/локальной папки, junction/symlink-линковка в scope агента, .skillignore-фильтр, build manifest, project-манифест. 0 завязок на сеть/auth — stdlib + tomli-w + pathspec + platformdirs."
-readme = "README.md"
-license = { text = "MIT" }
-requires-python = ">=3.11"
-authors = [{ name = "Dmitry" }]
-# skillkit — SIBLING librarykit (НЕ зависит от него, httpx, auth, сети).
-# Только локальная ФС-механика навыков. Core deps:
-# tomli-w — запись TOML (project-манифест .skills-hub/skills.toml,
-#   _skill_meta.toml ридеры используют stdlib tomllib на чтение);
-# pathspec — gitignore-стиль .skillignore / files-allowlist фильтр;
-# platformdirs — нативные дефолтные пути (Paths.default) по ОС.
-dependencies = [
-    "tomli-w>=1.0",
-    "pathspec>=0.12.0",
-    "platformdirs>=4.0",
-]
-
-[project.optional-dependencies]
-dev = ["pytest>=8", "pytest-asyncio>=0.24"]
-
-[build-system]
-requires = ["hatchling"]
-build-backend = "hatchling.build"
-
-[tool.hatch.build.targets.wheel]
-packages = ["skillkit"]
-
-[tool.ruff]
-line-length = 100
-target-version = "py311"
-src = ["."]
-
-[tool.ruff.lint]
-select = ["E", "F", "I", "B", "UP", "N", "SIM", "RUF"]
-# RUF001/RUF002/RUF003 — кириллица в строках/докстрингах/комментах намеренная
-#   (русскоязычный код), ruff принимает её за «двусмысленную» латиницу.
-# RUF022 — __all__ сгруппирован по модулям с комментами осознанно.
-# RUF059 — распаковка кортежа в тестах с неиспользуемой частью (фикстура
-#   возвращает (store, target, store_dir); читаемее, чем индексирование).
-ignore = ["RUF001", "RUF002", "RUF003", "RUF022", "RUF059"]
-
-[tool.pytest.ini_options]
-asyncio_mode = "auto"
-testpaths = ["tests"]
-pythonpath = ["."]
+[project]
+name = "s-skillkit"
+version = "0.1.3"
+description = "Локальное ядро управления навыками: install из git/локальной папки, junction/symlink-линковка в scope агента, .skillignore-фильтр, build manifest, project-манифест. 0 завязок на сеть/auth — stdlib + tomli-w + pathspec + platformdirs."
+readme = "README.md"
+license = { text = "MIT" }
+requires-python = ">=3.11"
+authors = [{ name = "Dmitry" }]
+# skillkit — SIBLING librarykit (НЕ зависит от него, httpx, auth, сети).
+# Только локальная ФС-механика навыков. Core deps:
+# tomli-w — запись TOML (project-манифест .skills-hub/skills.toml,
+#   _skill_meta.toml ридеры используют stdlib tomllib на чтение);
+# pathspec — gitignore-стиль .skillignore / files-allowlist фильтр;
+# platformdirs — нативные дефолтные пути (Paths.default) по ОС.
+dependencies = [
+    "tomli-w>=1.0",
+    "pathspec>=0.12.0",
+    "platformdirs>=4.0",
+]
+
+[project.optional-dependencies]
+dev = ["pytest>=8", "pytest-asyncio>=0.24"]
+
+[build-system]
+requires = ["hatchling"]
+build-backend = "hatchling.build"
+
+[tool.hatch.build.targets.wheel]
+packages = ["skillkit"]
+
+[tool.ruff]
+line-length = 100
+target-version = "py311"
+src = ["."]
+
+[tool.ruff.lint]
+select = ["E", "F", "I", "B", "UP", "N", "SIM", "RUF"]
+# RUF001/RUF002/RUF003 — кириллица в строках/докстрингах/комментах намеренная
+#   (русскоязычный код), ruff принимает её за «двусмысленную» латиницу.
+# RUF022 — __all__ сгруппирован по модулям с комментами осознанно.
+# RUF059 — распаковка кортежа в тестах с неиспользуемой частью (фикстура
+#   возвращает (store, target, store_dir); читаемее, чем индексирование).
+ignore = ["RUF001", "RUF002", "RUF003", "RUF022", "RUF059"]
+
+[tool.pytest.ini_options]
+asyncio_mode = "auto"
+testpaths = ["tests"]
+pythonpath = ["."]

{s_skillkit-0.1.2 → s_skillkit-0.1.3}/skillkit/installer.py

@@ -409,6 +409,7 @@ class SkillStore:
         skill_id: str | int | None = None,
         local_src: Path | None = None,
         git_ref: str | None = None,
+        skill_path: str | None = None,
     ) -> InstallResult:
         """Материализует навык в стор и линкует в scope агента.
 
@@ -449,6 +450,7 @@ class SkillStore:
             slug=slug, dir_name=dir_name, version=version, commit_sha=commit_sha,
             repo_url=repo_url, manifest=manifest, scope=scope, project=project,
             skill_id=skill_id_str, local_src=local_src, git_ref=git_ref,
+            skill_path=skill_path,
         )
         # git-url источник мог уточнить версию из frontmatter клона (фикс l2).
         effective_version = mat.get("version") or version
@@ -518,10 +520,12 @@ class SkillStore:
         skill_id: str | int | None = None,
         local_src: Path | None = None,
         git_ref: str | None = None,
+        skill_path: str | None = None,
     ) -> dict[str, Any]:
         """Материализует контент навыка в стор БЕЗ линковки в scope.
 
-        Возвращает {is_update, filter_result, update_diff, version}.
+        ``skill_path`` (#268) — подпапка навыка в репо (``skills/<name>/``);
+        None ⇒ корень репо. Возвращает {is_update, filter_result, update_diff, version}.
         """
         scope = "project" if project is not None else "global"
         dir_name = skill_dir_name(slug, skill_id)
@@ -530,12 +534,13 @@ class SkillStore:
             slug=slug, dir_name=dir_name, version=version, commit_sha=commit_sha,
             repo_url=repo_url, manifest=manifest, scope=scope, project=project,
             skill_id=skill_id_str, local_src=local_src, git_ref=git_ref,
+            skill_path=skill_path,
         )
 
     def _materialize_store(
         self, *, slug, dir_name, version, commit_sha, repo_url, manifest,
         scope, project, skill_id, local_src: Path | None = None,
-        git_ref: str | None = None,
+        git_ref: str | None = None, skill_path: str | None = None,
     ) -> dict[str, Any]:
         """Кладёт контент навыка в store_dir. Возвращает {is_update, filter_result, update_diff}."""
         store_dir = self._store_path(dir_name)
@@ -550,6 +555,7 @@ class SkillStore:
                 slug=slug, version=version, commit_sha=commit_sha, repo_url=repo_url,
                 manifest=manifest, scope=scope, project=project, slug_dir=store_dir,
                 skill_id=skill_id, local_src=local_src, git_ref=git_ref,
+                skill_path=skill_path,
             )
             return {"is_update": True, "filter_result": up.filter_result,
                     "update_diff": up.update_diff, "version": up.version}
@@ -558,7 +564,9 @@ class SkillStore:
         filter_result: dict[str, int] | None = None
         source = "hub"
         if repo_url:
-            with self._clone_version(repo_url, version, ref=git_ref) as cloned:
+            with self._clone_version(
+                repo_url, version, ref=git_ref, skill_path=skill_path
+            ) as cloned:
                 safe_copy_tree(cloned, store_dir)
             filter_result = apply_skill_filter(store_dir)
             if git_ref is not None:
@@ -684,6 +692,7 @@ class SkillStore:
         skill_id: str | None = None,
         local_src: Path | None = None,
         git_ref: str | None = None,
+        skill_path: str | None = None,
     ) -> InstallResult:
         """Докачивает diff между установленной и новой версией.
 
@@ -714,7 +723,9 @@ class SkillStore:
                 src_label = "local-path"
                 meta_repo = None
             else:
-                with self._clone_version(repo_url, version, ref=git_ref) as cloned:
+                with self._clone_version(
+                    repo_url, version, ref=git_ref, skill_path=skill_path
+                ) as cloned:
                     safe_copy_tree(cloned, slug_dir)
                 src_label = "git-url"
                 meta_repo = repo_url
@@ -755,7 +766,7 @@ class SkillStore:
         # версии (как git-url путь), сохраняя preserved-пути, вместо diff.
         if not manifest.get("files"):
             preserved = _preserved_for(manifest, self._target)
-            with self._clone_version(repo_url, version) as cloned:
+            with self._clone_version(repo_url, version, skill_path=skill_path) as cloned:
                 safe_copy_tree(cloned, slug_dir)
             version = _version_from_skill_md(slug_dir, version)
             filter_result = apply_skill_filter(slug_dir, extra_preserved=preserved)
@@ -776,7 +787,7 @@ class SkillStore:
         preserved = _preserved_for(manifest, self._target)
 
         filter_result: dict[str, int] | None = None
-        with self._clone_version(repo_url, version) as cloned:
+        with self._clone_version(repo_url, version, skill_path=skill_path) as cloned:
             # 4. Копируем added + changed.
             for rel in [*added, *changed]:
                 src_file = cloned / rel
@@ -913,7 +924,10 @@ class SkillStore:
     #  helpers
     # ------------------------------------------------------------------
     @contextlib.contextmanager
-    def _clone_version(self, repo_url: str, version: str, *, ref: str | None = None):
+    def _clone_version(
+        self, repo_url: str, version: str, *, ref: str | None = None,
+        skill_path: str | None = None,
+    ):
         """git clone во временную папку (yield Path).
 
         Поведение по ``ref``:
@@ -921,7 +935,11 @@ class SkillStore:
         - ``""`` (пусто) → git-url без явного ref: клон дефолтной ветки репо
           (``--branch`` опускается — ``--branch HEAD`` git не принимает);
         - иначе → дословный ref (ветка/тег/sha).
-        Папка удаляется при выходе из контекста.
+
+        ``skill_path`` (subdir-навык, #268): если задан — yield-им ПОДПАПКУ
+        ``<clone>/<skill_path>`` (с traversal-guard), а не корень репо. Так навык
+        может лежать подкаталогом CLI-репо (``skills/<name>/``), а материализуется
+        только он. Папка удаляется при выходе из контекста.
         """
         use_default_branch = ref == ""
         ref = ref if ref is not None else f"v{version}"
@@ -949,7 +967,15 @@ class SkillStore:
                     if secret:
                         stderr = stderr.replace(secret, "***")
                 raise RuntimeError(f"git clone failed: {stderr}") from e
-            yield clone_dir
+            # subdir-навык (#268): материализуем подпапку, не корень репо.
+            target = clone_dir
+            if skill_path:
+                target = _assert_within(clone_dir, clone_dir / skill_path)
+                if not target.is_dir():
+                    raise RuntimeError(
+                        f"skill_path '{skill_path}' не найден в репо {repo_url}"
+                    )
+            yield target
         finally:
             _force_rmtree(tmp)
 

s_skillkit-0.1.3/tests/test_w268_subdir_skill.py

@@ -0,0 +1,111 @@
+"""#268 — subdir-навык: материализуется ПОДПАПКА репо (skills/<name>/), не корень.
+
+Позволяет навыку жить подкаталогом CLI-репо: install клонирует репо, но в стор
+кладёт только skill_path-подпапку (SKILL.md + agents + references), а сам CLI
+ставится через runtime_dependency.
+"""
+from __future__ import annotations
+
+import hashlib
+from pathlib import Path
+from typing import Any
+
+from skillkit import ClaudeCodeTarget, SkillStore
+from skillkit import installer as installer_mod
+
+
+def _sha(text: str) -> str:
+    return hashlib.sha256(text.encode("utf-8")).hexdigest()
+
+
+def _manifest(files: dict[str, str]) -> dict[str, Any]:
+    return {
+        "version": "1.0.0", "description": "x",
+        "files": [
+            {"path": rel, "sha256": _sha(c), "size": len(c)} for rel, c in files.items()
+        ],
+        "preserved_paths": ["_local/"],
+    }
+
+
+def _make_fake_clone(layout: dict[str, str]):
+    """Фейковый git clone: пишет ВЕСЬ layout репо (корень + подпапки) в target."""
+    def fake_run(cmd: list[str], *a: Any, **k: Any):
+        assert cmd[0] == "git" and cmd[1] == "clone"
+        target = Path(cmd[-1])
+        target.mkdir(parents=True, exist_ok=True)
+        for rel, content in layout.items():
+            p = target / rel
+            p.parent.mkdir(parents=True, exist_ok=True)
+            p.write_text(content, encoding="utf-8")
+
+        class _R:
+            returncode = 0
+            stdout = ""
+            stderr = ""
+
+        return _R()
+
+    return fake_run
+
+
+# Репо CLI-проекта: код в корне + навык в skills/atlas/
+_REPO = {
+    "pyproject.toml": "[project]\nname='atlas'\n",
+    "src/atlas/cli.py": "# CLI",
+    "skills/atlas/SKILL.md": "---\nname: atlas\nversion: 1.0.0\n---\n# atlas skill",
+    "skills/atlas/agents/x.md": "AGENT",
+    "skills/atlas/references/r.md": "REF",
+}
+_SKILL_FILES = {
+    "SKILL.md": _REPO["skills/atlas/SKILL.md"],
+    "agents/x.md": "AGENT",
+    "references/r.md": "REF",
+}
+
+
+def test_subdir_materialize_takes_only_skill_path(tmp_path: Path, monkeypatch) -> None:
+    monkeypatch.setattr(installer_mod.subprocess, "run", _make_fake_clone(_REPO))
+    store = SkillStore(ClaudeCodeTarget(root=tmp_path / ".claude"), tmp_path / "store")
+    store.materialize(
+        slug="atlas", version="1.0.0", commit_sha="a",
+        repo_url="https://github.com/x/atlas-cli.git",
+        manifest=_manifest(_SKILL_FILES),
+        skill_path="skills/atlas",
+    )
+    store_dir = tmp_path / "store" / "atlas"
+    # В сторе — содержимое ПОДПАПКИ навыка:
+    assert (store_dir / "SKILL.md").exists()
+    assert (store_dir / "agents" / "x.md").read_text(encoding="utf-8") == "AGENT"
+    assert (store_dir / "references" / "r.md").exists()
+    # НЕ корневой код репо:
+    assert not (store_dir / "pyproject.toml").exists()
+    assert not (store_dir / "src").exists()
+    assert not (store_dir / "skills").exists()
+
+
+def test_subdir_missing_path_errors(tmp_path: Path, monkeypatch) -> None:
+    monkeypatch.setattr(installer_mod.subprocess, "run", _make_fake_clone(_REPO))
+    store = SkillStore(ClaudeCodeTarget(root=tmp_path / ".claude"), tmp_path / "store")
+    import pytest
+    with pytest.raises(RuntimeError, match="skill_path"):
+        store.materialize(
+            slug="atlas", version="1.0.0", commit_sha="a",
+            repo_url="https://github.com/x/atlas-cli.git",
+            manifest=_manifest(_SKILL_FILES),
+            skill_path="skills/nonexistent",
+        )
+
+
+def test_no_skill_path_uses_repo_root(tmp_path: Path, monkeypatch) -> None:
+    """Без skill_path (None) поведение прежнее — материализуется корень репо."""
+    root_layout = {"SKILL.md": "---\nname: x\nversion: 1.0.0\n---\n# x", "a.md": "A"}
+    monkeypatch.setattr(installer_mod.subprocess, "run", _make_fake_clone(root_layout))
+    store = SkillStore(ClaudeCodeTarget(root=tmp_path / ".claude"), tmp_path / "store")
+    store.materialize(
+        slug="x", version="1.0.0", commit_sha="a",
+        repo_url="https://github.com/x/x.git", manifest=_manifest(root_layout),
+    )
+    store_dir = tmp_path / "store" / "x"
+    assert (store_dir / "SKILL.md").exists()
+    assert (store_dir / "a.md").exists()

{s_skillkit-0.1.2 → s_skillkit-0.1.3}/uv.lock

@@ -96,7 +96,7 @@ wheels = [
 
 [[package]]
 name = "s-skillkit"
-version = "0.1.1"
+version = "0.1.2"
 source = { editable = "." }
 dependencies = [
     { name = "pathspec" },