s-skillkit 0.3.2__tar.gz → 0.3.3__tar.gz

{s_skillkit-0.3.2 → s_skillkit-0.3.3}/CHANGELOG.md

@@ -4,6 +4,42 @@
 [Keep a Changelog](https://keepachangelog.com/ru/1.1.0/); версионирование —
 [SemVer](https://semver.org/lang/ru/).
 
+## [Unreleased]
+
+### Fixed — окна консоли, `.pkgsrc` в зоне агента, потеря происхождения снапшота
+
+- **#1144 — прямые `subprocess` выкошены.** Демон стартует DETACHED (своей
+  консоли нет), поэтому КАЖДЫЙ консольный ребёнок (`uv`, `git`, `powershell`,
+  `cmd`) без `CREATE_NO_WINDOW` получал от Windows своё окно — пользователь
+  видел `uv.EXE` поверх браузера во время фоновой установки навыка.
+  `capture_output=True` окно НЕ подавляет (stdio и аллокация консоли в Win32 —
+  разные механизмы), и флаг стоял ровно в ОДНОМ из девяти мест. Все запуски
+  переведены на `librarykit.proc.run` (единая политика окон + обязательный
+  таймаут + захват вывода + маскировка секретов в логе). Появилась зависимость
+  `s-librarykit>=0.7.1`. Стережёт AST-гейт `tests/test_no_raw_subprocess.py`:
+  прямой `subprocess.run|Popen|call|check_output|check_call` в `skillkit/**`
+  роняет тесты.
+- **#1147 — `.pkgsrc` больше не утекает в папку агента.** При copy-fallback
+  (линковка стор→scope не удалась) копировался ВЕСЬ стор: `safe_copy_tree`
+  фильтровал только `.git`. Фильтр стал параметром (`skip_roots`), в зону агента
+  едет `AGENT_ZONE_SKIP_ROOTS` (`.git`, `.pkgsrc`). Копия стор→staging (перенос
+  версии) фильтр НЕ применяет — `.pkgsrc` обязана переезжать между версиями,
+  иначе теряется имя дистрибутива для `uv tool uninstall`. Инвариант: **стор —
+  источник истины; в зоне агента только артефакты навыка**. Плюс диагностика
+  первопричины: причина fallback'а логируется и различает «канон занят обычной
+  папкой» (`FileExistsError`) от «ОС/ФС не дала ссылку», а на Windows перед
+  сдачей в copy теперь пробуется `os.symlink` (Developer Mode), которого раньше
+  не пробовали вовсе.
+- **#1145 — снапшот-установка метится `source="hub"`.** Ядро нормализует
+  `SnapshotSource` → `LocalPathSource` (архив распакован во временную папку) ДО
+  записи меты, и происхождение терялось: 7 из 8 навыков с хаба получали
+  `source="local-path"` и МОЛЧА выпадали из автообновления CLI (оно берёт только
+  `source == "hub"`). Происхождение теперь едет явной меткой
+  (`SnapshotSource.origin` → `InstallRequest.source_label`) и перебивает вывод по
+  типу источника — и на установке, и на обновлении. Семантика реального
+  `install_from_path` не изменилась. Миграция уже установленных сторов — на
+  стороне CLI (признак: `source == "local-path"` при непустом `skill_id`).
+
 ## [0.3.1] — 2026-07-21
 
 Круг 3 независимой проверки нашёл два настоящих дефекта в правках 0.3.0 и один

{s_skillkit-0.3.2 → s_skillkit-0.3.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: s-skillkit
-Version: 0.3.2
+Version: 0.3.3
 Summary: Локальное ядро управления навыками: install из git/локальной папки, junction/symlink-линковка в scope агента, .skillignore-фильтр, build manifest, project-манифест. 0 завязок на сеть/auth — stdlib + tomli-w + pathspec + platformdirs.
 Author: Dmitry
 License: MIT
@@ -8,6 +8,7 @@ License-File: LICENSE
 Requires-Python: >=3.11
 Requires-Dist: pathspec>=0.12.0
 Requires-Dist: platformdirs>=4.0
+Requires-Dist: s-librarykit>=0.7.1
 Requires-Dist: tomli-w>=1.0
 Provides-Extra: dev
 Requires-Dist: pytest-asyncio>=0.24; extra == 'dev'
@@ -16,8 +17,10 @@ Description-Content-Type: text/markdown
 
 # s-skillkit
 
-Локальное ядро управления навыками (skills) — **SIBLING** `librarykit`
-(не зависит от него, от httpx, auth или сети). Только локальная ФС-механика.
+Локальное ядро управления навыками (skills): только локальная ФС-механика — ни
+сети, ни auth. Из `librarykit` берётся ровно один модуль — `librarykit.proc`
+(единый запуск подпроцессов: на Windows без всплывающих консольных окон и с
+обязательным таймаутом); `uv`, `git` и `npm` кит зовёт только через него.
 
 ## Что внутри
 

{s_skillkit-0.3.2 → s_skillkit-0.3.3}/README.md

@@ -1,7 +1,9 @@
 # s-skillkit
 
-Локальное ядро управления навыками (skills) — **SIBLING** `librarykit`
-(не зависит от него, от httpx, auth или сети). Только локальная ФС-механика.
+Локальное ядро управления навыками (skills): только локальная ФС-механика — ни
+сети, ни auth. Из `librarykit` берётся ровно один модуль — `librarykit.proc`
+(единый запуск подпроцессов: на Windows без всплывающих консольных окон и с
+обязательным таймаутом); `uv`, `git` и `npm` кит зовёт только через него.
 
 ## Что внутри
 

{s_skillkit-0.3.2 → s_skillkit-0.3.3}/pyproject.toml

@@ -1,21 +1,28 @@
 [project]
 name = "s-skillkit"
-version = "0.3.2"
+version = "0.3.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:
+# skillkit — локальная ФС-механика навыков (НЕ ходит в сеть, не знает auth).
+# Core deps:
 # tomli-w — запись TOML (project-манифест .skills-hub/skills.toml,
 #   _skill_meta.toml ридеры используют stdlib tomllib на чтение);
 # pathspec — gitignore-стиль .skillignore / files-allowlist фильтр;
-# platformdirs — нативные дефолтные пути (Paths.default) по ОС.
+# platformdirs — нативные дефолтные пути (Paths.default) по ОС;
+# s-librarykit — ТОЛЬКО ради ``librarykit.proc`` (#1144): единая политика запуска
+#   подпроцессов. Кит запускает uv/git/npm/powershell, а на Windows консольный
+#   ребёнок демона (у него нет своей консоли) без CREATE_NO_WINDOW получает
+#   ВИДИМОЕ окно поверх браузера; ``capture_output`` его не подавляет. Своей
+#   второй реализации «как правильно на Windows» в ките быть не должно —
+#   поэтому прежний статус «sibling, ни от чего не зависим» осознанно снят.
 dependencies = [
     "tomli-w>=1.0",
     "pathspec>=0.12.0",
     "platformdirs>=4.0",
+    "s-librarykit>=0.7.1",
 ]
 
 [project.optional-dependencies]

{s_skillkit-0.3.2 → s_skillkit-0.3.3}/skillkit/bootstrap.py

@@ -22,23 +22,36 @@ s-xray-gate). Кит НЕ зависит от него жёстко: это ``Ca
 запускает ``scripts/self_check.py`` (venv-python навыка → ``uv run`` → текущий
 интерпретатор), возвращает ``(ok, output)``.
 
-Мокабельность: subprocess идёт через ``_run_cmd`` (uv sync) и ``_run_capture``
+Мокабельность: запуск идёт через ``_run_cmd`` (uv sync) и ``_run_capture``
 (self_check); поиск uv — через ``_which``; ОС — константа ``IS_WINDOWS``.
+Оба примитива — обёртки над ``librarykit.proc.run`` (#1144): на Windows он
+всегда ставит ``CREATE_NO_WINDOW`` (bootstrap зовёт демон, у которого нет своей
+консоли — иначе окна поверх браузера) и требует явный таймаут.
 """
 from __future__ import annotations
 
+import logging
 import shutil
-import subprocess
 import sys
 from collections.abc import Callable
 from pathlib import Path
 from typing import Any
 
+from librarykit import proc
+from librarykit.proc import ProcResult
+
 from skillkit import deps_installer, uv_bootstrap
 from skillkit.deps_installer import _BINARY_KINDS
 
 IS_WINDOWS = sys.platform == "win32"
 
+_LOG = logging.getLogger(__name__)
+
+#: ``uv sync`` тянет всё дерево зависимостей навыка — это минуты на узком канале.
+_SYNC_TIMEOUT = 900.0
+#: self_check навыка — проверка окружения, а не сборка; висеть ей незачем.
+_SELF_CHECK_TIMEOUT = 300.0
+
 
 # --------------------------------------------------------------------------
 #  Подменяемые в тестах примитивы окружения
@@ -50,27 +63,28 @@ def _which(name: str) -> str | None:
 def _run_cmd(cmd: list[str], *, cwd: Path | None = None) -> int:
     """Запустить команду, вернуть returncode (не бросает на OSError)."""
     try:
-        return subprocess.run(
-            cmd, cwd=str(cwd) if cwd else None, check=False
-        ).returncode
+        return proc.run(cmd, cwd=cwd, timeout=_SYNC_TIMEOUT, log=_LOG).returncode
     except OSError:
         return 127
 
 
-def _run_capture(
-    cmd: list[str], *, cwd: Path | None = None
-) -> subprocess.CompletedProcess:
-    """Запустить команду с захватом вывода (не бросает на OSError)."""
+def _run_capture(cmd: list[str], *, cwd: Path | None = None) -> ProcResult:
+    """Запустить команду с захватом вывода (не бросает на OSError).
+
+    Возвращает ``ProcResult`` — по полям (``returncode``/``stdout``/``stderr``)
+    совместим с прежним ``CompletedProcess``, но дополнительно несёт ``timed_out``.
+    """
     try:
-        return subprocess.run(
-            cmd,
-            cwd=str(cwd) if cwd else None,
-            check=False,
-            capture_output=True,
-            text=True,
-        )
+        return proc.run(cmd, cwd=cwd, timeout=_SELF_CHECK_TIMEOUT, log=_LOG)
     except OSError as exc:
-        return subprocess.CompletedProcess(cmd, 127, "", str(exc))
+        return ProcResult(
+            cmd=tuple(str(c) for c in cmd),
+            returncode=127,
+            stdout="",
+            stderr=str(exc),
+            duration=0.0,
+            timed_out=False,
+        )
 
 
 # --------------------------------------------------------------------------
@@ -131,9 +145,10 @@ def run_self_check(skill_dir: Path | str) -> tuple[bool, str]:
     script = skill_dir / "scripts" / "self_check.py"
     if not script.exists():
         return (False, f"scripts/self_check.py не найден в {skill_dir}")
-    proc = _run_capture(_self_check_command(skill_dir, script), cwd=skill_dir)
-    output = (proc.stdout or "") + (proc.stderr or "")
-    return (proc.returncode == 0, output)
+    # Имя ``result``, а не ``proc``: модульный ``proc`` — это librarykit.proc.
+    result = _run_capture(_self_check_command(skill_dir, script), cwd=skill_dir)
+    output = (result.stdout or "") + (result.stderr or "")
+    return (result.returncode == 0, output)
 
 
 # Алиас для использования внутри bootstrap_skill (там параметр ``run_self_check``

{s_skillkit-0.3.2 → s_skillkit-0.3.3}/skillkit/deps_installer.py

@@ -27,19 +27,34 @@
 
 Кроссплатформенность: выбор менеджера — через ``shutil.which`` (обёрнут в
 ``_which`` для мокабельности) + ``python -m pip`` (текущий интерпретатор,
-работает на win/mac/linux). subprocess-примитив ``_run`` тоже подменяем в тестах.
+работает на win/mac/linux). Примитив запуска ``_run`` тоже подменяем в тестах.
+
+Запуск идёт ТОЛЬКО через ``librarykit.proc.run`` (#1144): на Windows он всегда
+ставит ``CREATE_NO_WINDOW`` (иначе фоновая установка навыка демоном рисует
+пользователю окна ``uv.EXE`` поверх браузера) и требует явный таймаут — висящий
+``uv`` в демоне некому прервать. Заодно снят прежний дефект: ``uv pip install``
+запускался без захвата вывода и сыпал прогресс-бары в stdout потребителя.
 """
 from __future__ import annotations
 
+import logging
 import shutil
-import subprocess
 import sys
 from pathlib import Path
 from typing import Any
 
+from librarykit import proc
+
 from skillkit import binary_installer
 from skillkit.pkg_manager import detect_project, missing_manager_report
 
+_LOG = logging.getLogger(__name__)
+
+#: Установка пакета — минуты (компиляция колёс, узкий канал), а не секунды.
+_INSTALL_TIMEOUT = 900.0
+#: Проба «есть ли pip» обязана отвечать быстро; вис здесь = вис установки.
+_PROBE_TIMEOUT = 60.0
+
 # kinds внешних бинарей (GitHub Releases) — делегируют binary_installer.
 _BINARY_KINDS = frozenset({"binary", "github-release"})
 
@@ -58,20 +73,24 @@ def _which(name: str) -> str | None:
 def _pip_available() -> bool:
     """Доступен ли ``pip`` для текущего интерпретатора (``python -m pip``)."""
     try:
-        proc = subprocess.run(
+        return proc.run(
             [sys.executable, "-m", "pip", "--version"],
-            capture_output=True,
-            check=False,
-        )
-        return proc.returncode == 0
+            timeout=_PROBE_TIMEOUT,
+            log=_LOG,
+        ).ok
     except OSError:
         return False
 
 
 def _run(cmd: list[str]) -> int:
-    """Запустить команду установки, вернуть returncode (не бросает на OSError)."""
+    """Запустить команду установки, вернуть returncode (не бросает на OSError).
+
+    Таймаут — не ошибка вызывающего, а такой же неуспех: ``ProcResult.ok`` уже
+    False, а ``returncode`` при убийстве по таймауту равен -1 (отличимо от кода
+    менеджера в отчёте ``failed``).
+    """
     try:
-        return subprocess.run(cmd, check=False).returncode
+        return proc.run(cmd, timeout=_INSTALL_TIMEOUT, log=_LOG).returncode
     except OSError:
         return 127
 

{s_skillkit-0.3.2 → s_skillkit-0.3.3}/skillkit/installer/__init__.py

@@ -27,7 +27,12 @@ skillkit-инъекция: store_dir передаётся явно (Paths.store_
 from __future__ import annotations
 
 import contextlib
-import subprocess  # noqa: F401  # re-export seam: тесты патчат installer.subprocess.run
+import logging
+
+# re-export seam: тесты патчат installer.subprocess.run. Сам кит подпроцессы
+# НЕ запускает напрямую (#1144) — их гоняет ``librarykit.proc.run``, который
+# внутри зовёт этот же stdlib-``subprocess``, поэтому патч продолжает работать.
+import subprocess  # noqa: F401
 import tarfile
 import tempfile
 from dataclasses import replace
@@ -52,6 +57,7 @@ from skillkit.installer.differ import _file_index, _manifest_diff  # noqa: F401
 from skillkit.installer.fetchers import ContentFetcher, GitContentFetcher
 from skillkit.installer.fsops import (
     _COPY_SKIP_ROOT,  # noqa: F401  # re-export seam
+    AGENT_ZONE_SKIP_ROOTS,
     PathTraversalError,
     _assert_within,
     _force_rmtree,  # noqa: F401  # re-export seam (звать ЗДЕСЬ нельзя — только PathGuard)
@@ -132,6 +138,9 @@ __all__ = [
 ]
 
 
+_LOG = logging.getLogger(__name__)
+
+
 # Брендо-нейтральные дефолты (само-имя кита); потребитель переопределяет через
 # ``set_managed_by`` / ``set_clone_prefix`` (обычно из ``config.configure`` —
 # выводятся из ``app_name``).
@@ -282,9 +291,13 @@ class SkillStore:
         # папку и ставим её как локальный источник (как прежний
         # ``install_from_snapshot`` → ``install_from_path``).
         if isinstance(request.source, SnapshotSource):
+            # #1145: происхождение архива НЕ теряется при нормализации —
+            # оно едет дальше явной меткой и перебивает вывод по типу источника
+            # (иначе мета получит "local-path" и навык выпадет из автоапдейта).
+            origin = request.source_label or request.source.origin
             with self._extract_snapshot(request.source.archive_path) as root:
                 return self._install(
-                    replace(request, source=LocalPathSource(root))
+                    replace(request, source=LocalPathSource(root), source_label=origin)
                 )
 
         slug = request.slug
@@ -531,7 +544,9 @@ class SkillStore:
             self._build_meta(
                 request.slug, version, request.commit_sha, request.manifest, scope,
                 request.project, self._skill_id_str(request),
-                source=source_label, repo_url=repo_url_meta,
+                # Явная метка (снапшот хаба) сильнее вывода по типу источника.
+                source=request.source_label or source_label,
+                repo_url=repo_url_meta,
                 skill_path=_source_skill_path(src),
             ),
         )
@@ -560,9 +575,24 @@ class SkillStore:
         try:
             kind = linker.create_link(link, store_dir)
             return True, kind
-        except OSError:
+        except OSError as exc:
             # Нет прав / ФС не поддерживает ссылки → копируем стор в scope.
-            safe_copy_tree(store_dir, link)
+            # #1147: причину fallback'а раньше никто не видел — а лечатся эти
+            # случаи по-разному. FileExistsError = канон занят ОБЫЧНОЙ папкой
+            # (её должен был освободить гейт, значит проблема выше по стеку);
+            # прочий OSError = среда (нет прав на symlink, ФС без ссылок).
+            reason = (
+                "канон занят обычной папкой"
+                if isinstance(exc, FileExistsError)
+                else "ОС/ФС не дала создать ссылку"
+            )
+            _LOG.warning(
+                "линковка %s → %s не удалась (%s: %s); копирую стор в scope",
+                link, store_dir, reason, exc,
+            )
+            # В зону агента едут ТОЛЬКО артефакты навыка: служебные каталоги
+            # (``.pkgsrc``) остаются в сторе — источнике истины.
+            safe_copy_tree(store_dir, link, skip_roots=AGENT_ZONE_SKIP_ROOTS)
             return False, "copy"
 
     def link_existing(
@@ -747,7 +777,11 @@ class SkillStore:
             slug_dir,
             self._build_meta(
                 request.slug, version, request.commit_sha, request.manifest, scope,
-                request.project, skill_id, source=src_label, repo_url=meta_repo,
+                request.project, skill_id,
+                # #1145: и на ветке ОБНОВЛЕНИЯ метка снапшота обязана уцелеть —
+                # иначе навык выпадал из автоапдейта после первого же update.
+                source=request.source_label or src_label,
+                repo_url=meta_repo,
                 skill_path=_source_skill_path(request.source),
             ),
         )

{s_skillkit-0.3.2 → s_skillkit-0.3.3}/skillkit/installer/fetchers.py

@@ -2,28 +2,38 @@
 
 ``ContentFetcher`` — протокол «достань репозиторий навыка на диск как контекст-
 менеджер, отдай Path на корень клона, по выходу — прибери за собой». Дефолт —
-``GitContentFetcher`` (прежнее поведение: ``git clone --depth 1`` через
-``subprocess``). Инъекция позволяет подменить получение источника (офлайн-
-зеркало, кеш, фейк в тестах) НЕ трогая оркестратор ``SkillStore``.
-
-Важно про seam ``subprocess``: ``GitContentFetcher`` зовёт ``subprocess.run`` из
-общего модульного объекта, поэтому существующий в тестах
-``monkeypatch.setattr(installer.subprocess, "run", ...)`` продолжает
-перехватывать клон (патчится атрибут того же модуля).
+``GitContentFetcher`` (прежнее поведение: ``git clone --depth 1``). Инъекция
+позволяет подменить получение источника (офлайн-зеркало, кеш, фейк в тестах)
+НЕ трогая оркестратор ``SkillStore``.
+
+Запуск git идёт через ``librarykit.proc.run`` (#1144) — единая политика окон и
+таймаутов кита. Здесь исторически стоял ЕДИНСТВЕННЫЙ во всём ките
+``CREATE_NO_WINDOW``; теперь флаг не дублируется в вызове, а приходит из ``proc``.
+
+Важно про seam: ``proc.run`` внутри зовёт ``subprocess.run``, поэтому
+существующий в тестах ``monkeypatch.setattr(installer.subprocess, "run", ...)``
+(патч атрибута общего stdlib-модуля) продолжает перехватывать клон.
 """
 from __future__ import annotations
 
 import contextlib
-import subprocess
-import sys
+import logging
 import tempfile
 from collections.abc import Iterator
 from pathlib import Path
 from typing import Protocol, runtime_checkable
 
+from librarykit import proc
+
 from skillkit.errors import CloneError
 from skillkit.installer.fsops import _force_rmtree
 
+_LOG = logging.getLogger(__name__)
+
+#: ``git clone --depth 1`` навыка: обычно секунды, но узкий канал/большой репо —
+#: минуты. Без таймаута зависший git держал бы фоновую установку вечно.
+_CLONE_TIMEOUT = 900.0
+
 
 @runtime_checkable
 class ContentFetcher(Protocol):
@@ -78,24 +88,24 @@ class GitContentFetcher:
         if not use_default_branch:
             cmd += ["--branch", ref]
         cmd += [url, str(clone_dir)]
-        # #1028: без окна на Windows. Когда git clone спавнит ДЕМОН (без
-        # консоли), Windows отдаёт git собственное окно git-bash → мигающая
-        # вкладка. CREATE_NO_WINDOW подавляет её. Вне Windows флага нет.
-        _run_kw: dict = {"check": True, "capture_output": True, "text": True}
-        if sys.platform == "win32":
-            _run_kw["creationflags"] = 0x08000000  # CREATE_NO_WINDOW
         try:
-            try:
-                subprocess.run(cmd, **_run_kw)
-            except subprocess.CalledProcessError as e:
-                stderr = e.stderr or e.stdout or ""
+            # #1028/#1144: без окна на Windows. Когда git clone спавнит ДЕМОН
+            # (без консоли), Windows отдаёт git собственное окно — мигающая
+            # вкладка поверх браузера. CREATE_NO_WINDOW ставит сам ``proc.run``.
+            # ``log=`` маскирует креды в URL, но stderr самого git может нести
+            # токен дословно — эту редакцию делаем ниже сами.
+            result = proc.run(cmd, timeout=_CLONE_TIMEOUT, log=_LOG)
+            if not result.ok:
+                stderr = result.stderr or result.stdout or ""
+                if result.timed_out:
+                    stderr = f"таймаут {_CLONE_TIMEOUT:.0f}s. {stderr}"
                 # Маскируем секрет из URL в сообщении об ошибке: если резолвер
-                # добавил токен в URL, не утечь его в текст RuntimeError.
+                # добавил токен в URL, не утечь его в текст CloneError.
                 if url != repo_url and "@" in url:
                     secret = url.split("//", 1)[-1].split("@", 1)[0]
                     if secret:
                         stderr = stderr.replace(secret, "***")
-                raise CloneError(f"git clone failed: {stderr}") from e
+                raise CloneError(f"git clone failed: {stderr}")
             yield clone_dir
         finally:
             _force_rmtree(tmp)

{s_skillkit-0.3.2 → s_skillkit-0.3.3}/skillkit/installer/fsops.py

@@ -22,6 +22,16 @@ class PathTraversalError(SkillkitError):
 # Внутри source-репо НЕ копируем в slug_dir (state репозитория, не skill).
 _COPY_SKIP_ROOT = frozenset({".git"})
 
+#: Что НЕ имеет права попасть в зону агента (``~/.claude/skills/<slug>/``) при
+#: copy-fallback (#1147). Инвариант: **стор — источник истины; в зоне агента
+#: только артефакты навыка**. ``.pkgsrc`` — служебный исходник пакета навыка,
+#: он нужен ``uv tool uninstall`` при снятии и живёт в СТОРЕ; скопированный
+#: рядом со ``SKILL.md`` он выглядел «лишней папкой, дублирующей снапшот».
+#: ВАЖНО: этот набор применяется ТОЛЬКО к копии в зону агента. Копия
+#: стор→staging (перенос версии) обязана нести ``.pkgsrc`` дальше, иначе имя
+#: дистрибутива для uninstall теряется навсегда.
+AGENT_ZONE_SKIP_ROOTS = frozenset({".git", ".pkgsrc"})
+
 
 def _on_rm_error(func, path, exc_info):
     """Чинит read-only-файлы в Windows (.git/objects/...) перед удалением."""
@@ -111,9 +121,16 @@ def _assert_within(base: Path, candidate: Path) -> Path:
     return Path(cand_abs)
 
 
-def safe_copy_tree(src: Path, dst: Path) -> None:
+def safe_copy_tree(
+    src: Path, dst: Path, *, skip_roots: frozenset[str] | set[str] | None = None
+) -> None:
     """Копирует содержимое src → dst, отвергая path-traversal (ТЗ §10).
 
+    ``skip_roots`` — какие каталоги ВЕРХНЕГО уровня не копировать; по умолчанию
+    ``_COPY_SKIP_ROOT`` (только ``.git``). Копирование в зону агента передаёт
+    сюда ``AGENT_ZONE_SKIP_ROOTS`` (плюс ``.pkgsrc``, см. #1147) — фильтр стал
+    параметром именно потому, что он РАЗНЫЙ для стора и для зоны агента.
+
     Правила безопасности:
     - `.git/` source-репо пропускается (это state репо, не содержимое skill).
     - Ссылки НЕ следуются вслепую; если цель ссылки резолвится наружу dst →
@@ -129,6 +146,7 @@ def safe_copy_tree(src: Path, dst: Path) -> None:
     """
     from skillkit import linker
 
+    skips = frozenset(skip_roots) if skip_roots is not None else _COPY_SKIP_ROOT
     src = Path(src)
     dst = Path(dst)
     dst.mkdir(parents=True, exist_ok=True)
@@ -140,11 +158,12 @@ def safe_copy_tree(src: Path, dst: Path) -> None:
     for root, dirnames, filenames in os.walk(src, followlinks=False):
         root_path = Path(root)
         rel_root = root_path.relative_to(src)
-        # Пропускаем .git/ и не спускаемся внутрь.
-        if rel_root.parts and rel_root.parts[0] in _COPY_SKIP_ROOT:
+        # Пропускаем корневые skip-каталоги (.git, при копии в зону агента —
+        # ещё и .pkgsrc) и не спускаемся внутрь.
+        if rel_root.parts and rel_root.parts[0] in skips:
             dirnames[:] = []
             continue
-        dirnames[:] = [d for d in dirnames if d not in _COPY_SKIP_ROOT]
+        dirnames[:] = [d for d in dirnames if d not in skips]
 
         linked_dirs: list[str] = []
         for name in list(dirnames):
@@ -160,7 +179,9 @@ def safe_copy_tree(src: Path, dst: Path) -> None:
                 # он НЕ отсекает (followlinks=False ловит только symlink) и
                 # спустился бы внутрь, дублируя/обходя guard.
                 target.mkdir(parents=True, exist_ok=True)
-                safe_copy_tree(child, target)
+                # Тот же набор исключений и внутри материализуемой ссылки —
+                # иначе фильтр «в зону агента» обходился бы через junction.
+                safe_copy_tree(child, target, skip_roots=skips)
                 linked_dirs.append(name)
             else:
                 target.mkdir(parents=True, exist_ok=True)

{s_skillkit-0.3.2 → s_skillkit-0.3.3}/skillkit/installer/models.py

@@ -89,9 +89,20 @@ class LocalPathSource:
 
 @dataclass(frozen=True)
 class SnapshotSource:
-    """Snapshot-архив навыка (``tar.gz``): распаковывается и ставится как local."""
+    """Snapshot-архив навыка (``tar.gz``): распаковывается и ставится как local.
+
+    ``origin`` (#1145) — ЧЕМ этот архив является по происхождению; попадает в
+    ``_skill_meta.json`` как ``source``. Дефолт ``"hub"``: снапшот — это раздача
+    хаба (``GET /skills/{slug}/versions/{semver}/snapshot``), а не локальная
+    папка пользователя. Поле нужно потому, что ядро нормализует ``SnapshotSource``
+    → ``LocalPathSource`` (архив распакован во временную папку) ДО записи меты, и
+    без него происхождение терялось: мета получала ``"local-path"``, а авто-
+    обновление CLI берёт только ``source == "hub"`` — установленные снапшотом
+    навыки молча выпадали из автоапдейта.
+    """
 
     archive_path: Path
+    origin: Source = "hub"
 
 
 @dataclass(frozen=True)
@@ -140,6 +151,11 @@ class InstallRequest:
     project: Path | None = None
     force: bool = False
     skill_id: str | int | None = None
+    #: Явная метка происхождения для ``_skill_meta.json`` (#1145). None ⇒ метка
+    #: выводится из типа ``source`` (как раньше). Задаётся, когда вывод по типу
+    #: соврал бы: снапшот хаба ставится ЧЕРЕЗ ``LocalPathSource`` (распакованная
+    #: временная папка), но по происхождению это "hub".
+    source_label: Source | None = None
 
 
 @dataclass

{s_skillkit-0.3.2 → s_skillkit-0.3.3}/skillkit/linker.py

@@ -13,14 +13,22 @@
 """
 from __future__ import annotations
 
+import logging
 import os
 import stat
-import subprocess
 import sys
 from pathlib import Path
 
+from librarykit import proc
+
 IS_WINDOWS = sys.platform == "win32"
 
+_LOG = logging.getLogger(__name__)
+
+#: ``mklink /J`` — одна операция ФС; секунды. Запуск через ``librarykit.proc.run``
+#: (#1144): ``cmd.exe`` из демона без консоли иначе мигает окном пользователю.
+_MKLINK_TIMEOUT = 30.0
+
 # Тип результата create_link: "junction" | "symlink".
 LinkKind = str
 
@@ -67,12 +75,21 @@ def _create_junction(link: Path, target: Path) -> None:
         return
     except (ImportError, AttributeError, OSError):
         pass
-    subprocess.run(
+    result = proc.run(
         ["cmd", "/c", "mklink", "/J", str(link), str(target)],
-        check=True,
-        capture_output=True,
-        text=True,
+        timeout=_MKLINK_TIMEOUT,
+        log=_LOG,
     )
+    if not result.ok:
+        # Наружу отдаём именно OSError: контракт create_link — «ОС не дала
+        # создать ссылку» ловится вызывающим как OSError (copy-fallback).
+        # Прежний ``check=True`` бросал CalledProcessError, которая мимо этого
+        # except проскакивала наверх необработанной.
+        detail = (result.stderr or result.stdout or "").strip()
+        raise OSError(
+            f"mklink /J не создал junction (код {result.returncode})"
+            + (f": {detail}" if detail else "")
+        )
 
 
 def create_link(link_path: Path, target_dir: Path) -> LinkKind:
@@ -99,8 +116,18 @@ def create_link(link_path: Path, target_dir: Path) -> LinkKind:
 
     link_path.parent.mkdir(parents=True, exist_ok=True)
     if IS_WINDOWS:
-        _create_junction(link_path, target_dir)
-        return "junction"
+        try:
+            _create_junction(link_path, target_dir)
+            return "junction"
+        except OSError as exc:
+            # #1147: раньше сдавались сразу в copy — а копия стора в зону агента
+            # и есть источник «лишних папок» там. Symlink на Windows работает
+            # при включённом Developer Mode / привилегии SeCreateSymbolicLink,
+            # то есть покрывает часть случаев, где junction не вышел. Не помог и
+            # он → OSError наверх, вызывающий делает copy-fallback осознанно.
+            _LOG.debug("junction не создан (%s) — пробую symlink: %s", exc, link_path)
+            os.symlink(target_dir, link_path, target_is_directory=True)
+            return "symlink"
     os.symlink(target_dir, link_path, target_is_directory=True)
     return "symlink"