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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: s-skillkit
-Version: 0.1.0
+Version: 0.1.2
 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.0 → s_skillkit-0.1.2}/pyproject.toml

@@ -1,48 +1,48 @@
-[project]
-name = "s-skillkit"
-version = "0.1.0"
-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.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 = ["."]

{s_skillkit-0.1.0 → s_skillkit-0.1.2}/skillkit/deps_installer.py

@@ -1,185 +1,189 @@
-"""Установка runtime-зависимостей навыка (tooling).
-
-Навык типа ``tooling`` может декларировать ``runtime_dependencies`` —
-список ``{kind, spec}``, где ``kind ∈ {pip, npm, system}``:
-
-- ``pip``    → ставим через **uv** (``uv pip install``) → fallback ``pip``
-  (``python -m pip install``). Нет ни uv, ни pip → НЕ падаем, кладём в
-  ``skipped`` с готовой инструкцией (graceful degradation);
-- ``npm``    → ставим через ``npm install -g`` только если ``npm`` есть в PATH;
-  иначе ``skipped`` + инструкция (node/npm — не наша забота ставить);
-- ``system`` → НИКОГДА не ставим сами (apt/brew/choco/права) — только
-  инструкция в ``skipped``.
-
-Идемпотентность — на менеджере пакетов (повторный install уже стоящего = no-op).
-Функция НИКОГДА не бросает: ошибка любой зависимости → она в ``failed``/
-``skipped``, остальные продолжают. Возврат::
-
-    {"installed": [{kind, spec}, ...],
-     "skipped":   [{kind, spec, reason, instruction}, ...],
-     "failed":    [{kind, spec, reason}, ...]}
-
-Кроссплатформенность: выбор менеджера — через ``shutil.which`` (обёрнут в
-``_which`` для мокабельности) + ``python -m pip`` (текущий интерпретатор,
-работает на win/mac/linux). subprocess-примитив ``_run`` тоже подменяем в тестах.
-"""
-from __future__ import annotations
-
-import shutil
-import subprocess
-import sys
-from typing import Any
-
-# kinds, которые умеем обрабатывать (прочее → skipped с reason unknown-kind).
-_KNOWN_KINDS = frozenset({"pip", "npm", "system"})
-
-
-# --------------------------------------------------------------------------
-#  Подменяемые в тестах примитивы окружения
-# --------------------------------------------------------------------------
-def _which(name: str) -> str | None:
-    """Путь к исполняемому ``name`` в PATH или None (обёртка для моков)."""
-    return shutil.which(name)
-
-
-def _pip_available() -> bool:
-    """Доступен ли ``pip`` для текущего интерпретатора (``python -m pip``)."""
-    try:
-        proc = subprocess.run(
-            [sys.executable, "-m", "pip", "--version"],
-            capture_output=True,
-            check=False,
-        )
-        return proc.returncode == 0
-    except OSError:
-        return False
-
-
-def _run(cmd: list[str]) -> int:
-    """Запустить команду установки, вернуть returncode (не бросает на OSError)."""
-    try:
-        return subprocess.run(cmd, check=False).returncode
-    except OSError:
-        return 127
-
-
-# --------------------------------------------------------------------------
-#  Public API
-# --------------------------------------------------------------------------
-def install_runtime_dependencies(
-    deps: list[dict[str, Any]] | tuple[dict[str, Any], ...] | None,
-) -> dict[str, list[dict[str, Any]]]:
-    """Поставить runtime-зависимости навыка. Никогда не бросает.
-
-    ``deps`` — список ``{"kind": pip|npm|system, "spec": "..."}`` (как в манифесте
-    ``runtime_dependencies`` / ``aggregated_runtime_dependencies``). Кривые
-    элементы (не-dict, без kind/spec, неизвестный kind) пропускаются/идут в
-    skipped. Возвращает отчёт ``{installed, skipped, failed}``.
-    """
-    report: dict[str, list[dict[str, Any]]] = {
-        "installed": [],
-        "skipped": [],
-        "failed": [],
-    }
-    if not deps:
-        return report
-
-    for raw in deps:
-        if not isinstance(raw, dict):
-            continue  # мусор — игнор
-        kind = raw.get("kind")
-        spec = raw.get("spec")
-        if not kind or not spec:
-            continue  # неполная декларация — игнор
-        kind = str(kind)
-        spec = str(spec)
-        if kind not in _KNOWN_KINDS:
-            report["skipped"].append(
-                {
-                    "kind": kind,
-                    "spec": spec,
-                    "reason": f"неизвестный kind зависимости: {kind}",
-                    "instruction": f"установите {spec} вручную ({kind}).",
-                }
-            )
-            continue
-        _dispatch(kind, spec, report)
-    return report
-
-
-def _dispatch(
-    kind: str, spec: str, report: dict[str, list[dict[str, Any]]]
-) -> None:
-    """Маршрутизация одной зависимости по kind в нужную ветку установки."""
-    if kind == "pip":
-        _install_pip(spec, report)
-    elif kind == "npm":
-        _install_npm(spec, report)
-    else:  # system
-        report["skipped"].append(
-            {
-                "kind": "system",
-                "spec": spec,
-                "reason": "системная зависимость не ставится автоматически",
-                "instruction": (
-                    f"установите системный пакет «{spec}» вашим пакетным "
-                    "менеджером (apt/brew/choco/...)."
-                ),
-            }
-        )
-
-
-# --------------------------------------------------------------------------
-#  pip: uv pip install → python -m pip install
-# --------------------------------------------------------------------------
-def _install_pip(spec: str, report: dict[str, list[dict[str, Any]]]) -> None:
-    entry = {"kind": "pip", "spec": spec}
-    if _which("uv") is not None:
-        cmd = ["uv", "pip", "install", spec]
-    elif _pip_available():
-        cmd = [sys.executable, "-m", "pip", "install", spec]
-    else:
-        report["skipped"].append(
-            {
-                **entry,
-                "reason": "не найдены ни uv, ни pip",
-                "instruction": (
-                    f"установите uv (https://docs.astral.sh/uv/) или pip, затем "
-                    f"`pip install {spec}`."
-                ),
-            }
-        )
-        return
-    rc = _run(cmd)
-    if rc == 0:
-        report["installed"].append(entry)
-    else:
-        report["failed"].append(
-            {**entry, "reason": f"менеджер вернул код {rc}"}
-        )
-
-
-# --------------------------------------------------------------------------
-#  npm: npm install -g (только если npm есть)
-# --------------------------------------------------------------------------
-def _install_npm(spec: str, report: dict[str, list[dict[str, Any]]]) -> None:
-    entry = {"kind": "npm", "spec": spec}
-    if _which("npm") is None:
-        report["skipped"].append(
-            {
-                **entry,
-                "reason": "npm не найден в PATH",
-                "instruction": (
-                    f"установите Node.js/npm (https://nodejs.org), затем "
-                    f"`npm install -g {spec}`."
-                ),
-            }
-        )
-        return
-    rc = _run(["npm", "install", "-g", spec])
-    if rc == 0:
-        report["installed"].append(entry)
-    else:
-        report["failed"].append(
-            {**entry, "reason": f"npm вернул код {rc}"}
-        )
+"""Установка runtime-зависимостей навыка (tooling).
+
+Навык типа ``tooling`` может декларировать ``runtime_dependencies`` —
+список ``{kind, spec}``, где ``kind ∈ {pip, npm, system}``:
+
+- ``pip``    → ставим через **uv** (``uv pip install``) → fallback ``pip``
+  (``python -m pip install``). Нет ни uv, ни pip → НЕ падаем, кладём в
+  ``skipped`` с готовой инструкцией (graceful degradation);
+- ``npm``    → ставим через ``npm install -g`` только если ``npm`` есть в PATH;
+  иначе ``skipped`` + инструкция (node/npm — не наша забота ставить);
+- ``system`` → НИКОГДА не ставим сами (apt/brew/choco/права) — только
+  инструкция в ``skipped``.
+
+Идемпотентность — на менеджере пакетов (повторный install уже стоящего = no-op).
+Функция НИКОГДА не бросает: ошибка любой зависимости → она в ``failed``/
+``skipped``, остальные продолжают. Возврат::
+
+    {"installed": [{kind, spec}, ...],
+     "skipped":   [{kind, spec, reason, instruction}, ...],
+     "failed":    [{kind, spec, reason}, ...]}
+
+Кроссплатформенность: выбор менеджера — через ``shutil.which`` (обёрнут в
+``_which`` для мокабельности) + ``python -m pip`` (текущий интерпретатор,
+работает на win/mac/linux). subprocess-примитив ``_run`` тоже подменяем в тестах.
+"""
+from __future__ import annotations
+
+import shutil
+import subprocess
+import sys
+from typing import Any
+
+# kinds, которые умеем обрабатывать (прочее → skipped с reason unknown-kind).
+_KNOWN_KINDS = frozenset({"pip", "npm", "system"})
+
+
+# --------------------------------------------------------------------------
+#  Подменяемые в тестах примитивы окружения
+# --------------------------------------------------------------------------
+def _which(name: str) -> str | None:
+    """Путь к исполняемому ``name`` в PATH или None (обёртка для моков)."""
+    return shutil.which(name)
+
+
+def _pip_available() -> bool:
+    """Доступен ли ``pip`` для текущего интерпретатора (``python -m pip``)."""
+    try:
+        proc = subprocess.run(
+            [sys.executable, "-m", "pip", "--version"],
+            capture_output=True,
+            check=False,
+        )
+        return proc.returncode == 0
+    except OSError:
+        return False
+
+
+def _run(cmd: list[str]) -> int:
+    """Запустить команду установки, вернуть returncode (не бросает на OSError)."""
+    try:
+        return subprocess.run(cmd, check=False).returncode
+    except OSError:
+        return 127
+
+
+# --------------------------------------------------------------------------
+#  Public API
+# --------------------------------------------------------------------------
+def install_runtime_dependencies(
+    deps: list[dict[str, Any]] | tuple[dict[str, Any], ...] | None,
+) -> dict[str, list[dict[str, Any]]]:
+    """Поставить runtime-зависимости навыка. Никогда не бросает.
+
+    ``deps`` — список ``{"kind": pip|npm|system, "spec": "..."}`` (как в манифесте
+    ``runtime_dependencies`` / ``aggregated_runtime_dependencies``). Кривые
+    элементы (не-dict, без kind/spec, неизвестный kind) пропускаются/идут в
+    skipped. Возвращает отчёт ``{installed, skipped, failed}``.
+    """
+    report: dict[str, list[dict[str, Any]]] = {
+        "installed": [],
+        "skipped": [],
+        "failed": [],
+    }
+    if not deps:
+        return report
+
+    for raw in deps:
+        if not isinstance(raw, dict):
+            continue  # мусор — игнор
+        kind = raw.get("kind")
+        spec = raw.get("spec")
+        if not kind or not spec:
+            continue  # неполная декларация — игнор
+        kind = str(kind)
+        spec = str(spec)
+        if kind not in _KNOWN_KINDS:
+            report["skipped"].append(
+                {
+                    "kind": kind,
+                    "spec": spec,
+                    "reason": f"неизвестный kind зависимости: {kind}",
+                    "instruction": f"установите {spec} вручную ({kind}).",
+                }
+            )
+            continue
+        _dispatch(kind, spec, report)
+    return report
+
+
+def _dispatch(
+    kind: str, spec: str, report: dict[str, list[dict[str, Any]]]
+) -> None:
+    """Маршрутизация одной зависимости по kind в нужную ветку установки."""
+    if kind == "pip":
+        _install_pip(spec, report)
+    elif kind == "npm":
+        _install_npm(spec, report)
+    else:  # system
+        report["skipped"].append(
+            {
+                "kind": "system",
+                "spec": spec,
+                "reason": "системная зависимость не ставится автоматически",
+                "instruction": (
+                    f"установите системный пакет «{spec}» вашим пакетным "
+                    "менеджером (apt/brew/choco/...)."
+                ),
+            }
+        )
+
+
+# --------------------------------------------------------------------------
+#  pip: uv pip install → python -m pip install
+# --------------------------------------------------------------------------
+def _install_pip(spec: str, report: dict[str, list[dict[str, Any]]]) -> None:
+    entry = {"kind": "pip", "spec": spec}
+    # ``--upgrade``: при обновлении навыка повторный apply «pkg>=X» должен
+    # ПОДНЯТЬ уже стоящий пакет до новой версии (без флага pip/uv считают
+    # требование удовлетворённым и ничего не делают). Идемпотентность сохраняется
+    # — для уже-последней версии --upgrade это no-op.
+    if _which("uv") is not None:
+        cmd = ["uv", "pip", "install", "--upgrade", spec]
+    elif _pip_available():
+        cmd = [sys.executable, "-m", "pip", "install", "--upgrade", spec]
+    else:
+        report["skipped"].append(
+            {
+                **entry,
+                "reason": "не найдены ни uv, ни pip",
+                "instruction": (
+                    f"установите uv (https://docs.astral.sh/uv/) или pip, затем "
+                    f"`pip install {spec}`."
+                ),
+            }
+        )
+        return
+    rc = _run(cmd)
+    if rc == 0:
+        report["installed"].append(entry)
+    else:
+        report["failed"].append(
+            {**entry, "reason": f"менеджер вернул код {rc}"}
+        )
+
+
+# --------------------------------------------------------------------------
+#  npm: npm install -g (только если npm есть)
+# --------------------------------------------------------------------------
+def _install_npm(spec: str, report: dict[str, list[dict[str, Any]]]) -> None:
+    entry = {"kind": "npm", "spec": spec}
+    if _which("npm") is None:
+        report["skipped"].append(
+            {
+                **entry,
+                "reason": "npm не найден в PATH",
+                "instruction": (
+                    f"установите Node.js/npm (https://nodejs.org), затем "
+                    f"`npm install -g {spec}`."
+                ),
+            }
+        )
+        return
+    rc = _run(["npm", "install", "-g", spec])
+    if rc == 0:
+        report["installed"].append(entry)
+    else:
+        report["failed"].append(
+            {**entry, "reason": f"npm вернул код {rc}"}
+        )

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

@@ -747,6 +747,31 @@ class SkillStore:
                 is_update=True, scope=scope, skill_id=skill_id,
             )
 
+        # BUG-3 (defence-in-depth): версия с ПУСТЫМ files-инвентарём (например,
+        # git-synced бэкендом до фикса BUG-1) НЕ должна трактоваться как «все
+        # файлы удалены». sha-diff против пустого new.files дал бы removed=ВСЕ и
+        # затёр стор (воспроизведено: update 1.0.0→1.0.1 оставил только
+        # _skill_meta.json). При наличии repo_url делаем полный ре-clone новой
+        # версии (как git-url путь), сохраняя preserved-пути, вместо diff.
+        if not manifest.get("files"):
+            preserved = _preserved_for(manifest, self._target)
+            with self._clone_version(repo_url, version) 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)
+            write_meta(
+                slug_dir,
+                self._build_meta(
+                    slug, version, commit_sha, manifest, scope, project, skill_id,
+                    source="hub", repo_url=repo_url,
+                ),
+            )
+            return InstallResult(
+                slug=slug, version=version, target_dir=slug_dir,
+                is_update=True, scope=scope, skill_id=skill_id,
+                filter_result=filter_result,
+            )
+
         added, changed, removed = _manifest_diff(old_manifest, manifest)
         preserved = _preserved_for(manifest, self._target)