s-skillkit 0.4.0__tar.gz → 0.4.1__tar.gz

{s_skillkit-0.4.0 → s_skillkit-0.4.1}/CHANGELOG.md

@@ -6,6 +6,28 @@
 
 ## [Unreleased]
 
+## [0.4.1] — 2026-09-01
+
+### Fixed — `IAgentTarget` снова экспортируется из `skillkit.targets` (#2542)
+
+Выпуск 0.4.0 уронил `skillery login` у пользователя с чистой установкой:
+
+```
+from skillkit.targets import (
+ImportError: cannot import name 'IAgentTarget' from 'skillkit.targets'
+```
+
+Протокол переименовали `IAgentTarget` → `AgentTarget` (lib#667) и положили
+совместимое имя в `skillkit/targets/base.py`. Этого оказалось мало: чужой код
+зовёт имя **из пакета**, а не из модуля `base`. Теперь алиас реэкспортируется
+пакетом и объявлен в его `__all__`.
+
+Почему не поймали заранее: у разработчика в окружении стоял кит 0.3.7 (там ещё
+старое имя и нет нового), а `s-skillkit` тянется с верхней границей открытой —
+пользователь получил 0.4.0 и упал. Сторож `tests/test_targets_public_names.py`
+теперь проверяет ровно тот импорт, на котором падал клиент.
+
+
 ### Added — канон навыка переехал в кит: `skillkit.canon` (#162)
 
 - **Один источник правды вместо трёх.** Форма навыка (раскладка `src/<пакет>/` +

{s_skillkit-0.4.0 → s_skillkit-0.4.1}/PKG-INFO

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

{s_skillkit-0.4.0 → s_skillkit-0.4.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "s-skillkit"
-version = "0.4.0"
+version = "0.4.1"
 description = "Локальное ядро управления навыками: install из git/локальной папки, junction/symlink-линковка в scope агента, .skillignore-фильтр, build manifest, project-манифест. 0 завязок на сеть/auth — stdlib + tomli-w + pathspec + platformdirs."
 readme = "README.md"
 license = { text = "MIT" }

{s_skillkit-0.4.0 → s_skillkit-0.4.1}/skillkit/targets/__init__.py

@@ -8,6 +8,12 @@ from skillkit.targets.base import (
     BaseAgentTarget,
     HookConfigDescriptor,
     AgentTarget,
+    #: Совместимое имя (lib#667, skl#2542). Протокол переименован из
+    #: `IAgentTarget`, но чужой код продолжает звать старое имя — и зовёт его
+    #: ИЗ ПАКЕТА, а не из `base`. Алиас в `base` без этой строки никого не спас:
+    #: 01.09.2026 выпуск 0.4.0 уронил `skillery login` у живого пользователя
+    #: с `cannot import name 'IAgentTarget' from 'skillkit.targets'`.
+    IAgentTarget,
     McpConfigDescriptor,
 )
 from skillkit.targets.claude_code import ClaudeCodeTarget
@@ -53,6 +59,7 @@ __all__ = [
     "CodexTarget",
     "HookConfigDescriptor",
     "AgentTarget",
+    "IAgentTarget",
     "McpConfigDescriptor",
     "OpenCodeTarget",
     "agent_skill_dir",

s_skillkit-0.4.1/tests/test_targets_public_names.py

@@ -0,0 +1,83 @@
+"""Пакет `skillkit.targets` отдаёт оба имени протокола (skl#2542).
+
+Прецедент 01.09.2026: протокол переименовали `IAgentTarget` → `AgentTarget`
+(lib#667) и положили алиас в `skillkit/targets/base.py`. Этого оказалось мало —
+чужой код зовёт имя ИЗ ПАКЕТА:
+
+    from skillkit.targets import IAgentTarget
+
+Выпуск 0.4.0 уехал на PyPI и уронил `skillery login` у живого пользователя:
+`ImportError: cannot import name 'IAgentTarget' from 'skillkit.targets'`.
+
+Урок, ради которого написан файл: алиас обязан стоять там, ГДЕ ЕГО ИЩУТ. Тесты
+кита импортировали из `base` — и потому молчали, пока клиент падал.
+"""
+
+from __future__ import annotations
+
+
+def test_оба_имени_есть_в_пакете():
+    """Ровно тот импорт, на котором упал пользователь."""
+    from skillkit.targets import AgentTarget, IAgentTarget  # noqa: F401
+
+    assert IAgentTarget is AgentTarget, "алиас обязан быть ТЕМ ЖЕ объектом"
+
+
+def test_оба_имени_есть_в_модуле_base():
+    """Старый путь импорта тоже жив: им пользуется код, писавшийся до пакета."""
+    from skillkit.targets.base import AgentTarget, IAgentTarget
+
+    assert IAgentTarget is AgentTarget
+
+
+def test_имена_объявлены_в_all():
+    """`__all__` — обещание пакета. Имя, которого там нет, легко потерять при
+    следующей уборке экспорта: ровно так оно и потерялось."""
+    import skillkit.targets as пакет
+
+    assert "AgentTarget" in пакет.__all__
+    assert "IAgentTarget" in пакет.__all__
+
+
+def test_алиас_виден_снаружи_как_атрибут():
+    """`from ... import` и `getattr` — разные пути; проверяем оба, потому что
+    потребители пользуются и тем, и другим."""
+    import skillkit.targets as пакет
+
+    assert getattr(пакет, "IAgentTarget", None) is пакет.AgentTarget
+
+
+def test_алиас_остаётся_протоколом_а_не_заглушкой():
+    """Имя должно оставаться РАБОЧИМ протоколом, а не просто существовать.
+
+    Иначе импорт легко «починить» строкой-пустышкой и перенести беду на шаг
+    дальше от места правки — туда, где именем аннотируют или наследуются.
+
+    `isinstance` здесь НЕ проверяется намеренно: протокол объявлен без
+    `@runtime_checkable`, он для аннотаций, а не для проверок во время работы.
+    Первая версия этого теста звала isinstance и упала — ошибка была в ожидании,
+    а не в ките.
+    """
+    from typing import get_protocol_members, is_protocol
+
+    from skillkit.targets import IAgentTarget
+
+    assert is_protocol(IAgentTarget), "имя перестало быть протоколом"
+    члены = get_protocol_members(IAgentTarget)
+    assert "base_dir" in члены, f"протокол потерял ключевой метод, осталось: {sorted(члены)}"
+
+
+def test_настоящая_цель_подходит_под_протокол():
+    """Проверка формы БЕЗ isinstance: у живой цели есть всё, что обещает контракт.
+
+    Так ловится расхождение протокола и реализации, ради которого протокол и
+    заводился, — но без требования делать его runtime_checkable.
+    """
+    from typing import get_protocol_members
+
+    from skillkit.targets import IAgentTarget, get_target
+
+    цель = get_target("claude_code")
+    нет = [и for и in get_protocol_members(IAgentTarget) if not hasattr(цель, и)]
+
+    assert not нет, f"цель claude_code не выполняет контракт: не хватает {нет}"