unique_orchestrator 2026.20.0.dev7__tar.gz → 2026.20.0.dev8__tar.gz

{unique_orchestrator-2026.20.0.dev7 → unique_orchestrator-2026.20.0.dev8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: unique-orchestrator
-Version: 2026.20.0.dev7
+Version: 2026.20.0.dev8
 Summary: 
 Author: Andreas Hauri
 Author-email: Andreas Hauri <andreas.hauri@unique.ai>
@@ -8,14 +8,14 @@ License: Proprietary
 Requires-Dist: pydantic>=2.8.2,<3
 Requires-Dist: typing-extensions>=4.9.0,<5
 Requires-Dist: jinja2>=3.1.0,<4
-Requires-Dist: unique-toolkit>=2026.20.0.dev11
+Requires-Dist: unique-toolkit>=2026.20.0.dev12
 Requires-Dist: unique-stock-ticker>=2026.20.0.dev6
 Requires-Dist: unique-follow-up-questions>=2026.20.0.dev6
 Requires-Dist: unique-internal-search>=2026.20.0.dev6
 Requires-Dist: unique-deep-research>=2026.20.0.dev6
 Requires-Dist: unique-web-search>=2026.20.0.dev7
 Requires-Dist: unique-swot>=2026.20.0.dev7
-Requires-Dist: unique-skill-tool>=2026.20.0.dev6
+Requires-Dist: unique-skill-tool>=2026.20.0.dev7
 Requires-Dist: openai>=1.109.1,<3
 Requires-Dist: python-frontmatter>=1.1.0,<2
 Requires-Python: >=3.12, <4

{unique_orchestrator-2026.20.0.dev7 → unique_orchestrator-2026.20.0.dev8}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "unique_orchestrator"
-version = "2026.20.0.dev7"
+version = "2026.20.0.dev8"
 description = ""
 readme = "README.md"
 license = { text = "Proprietary" }
@@ -12,14 +12,14 @@ dependencies = [
     "pydantic>=2.8.2,<3",
     "typing-extensions>=4.9.0,<5",
     "jinja2>=3.1.0,<4",
-    "unique-toolkit>=2026.20.0.dev11",
+    "unique-toolkit>=2026.20.0.dev12",
     "unique-stock-ticker>=2026.20.0.dev6",
     "unique-follow-up-questions>=2026.20.0.dev6",
     "unique-internal-search>=2026.20.0.dev6",
     "unique-deep-research>=2026.20.0.dev6",
     "unique-web-search>=2026.20.0.dev7",
     "unique-swot>=2026.20.0.dev7",
-    "unique-skill-tool>=2026.20.0.dev6",
+    "unique-skill-tool>=2026.20.0.dev7",
     "openai>=1.109.1,<3",
     "python-frontmatter>=1.1.0,<2",
 ]

{unique_orchestrator-2026.20.0.dev7 → unique_orchestrator-2026.20.0.dev8}/unique_orchestrator/_builders/skill_setup.py

@@ -40,13 +40,15 @@ from __future__ import annotations
 
 import asyncio
 from logging import Logger
-from typing import TYPE_CHECKING, Any
+from typing import TYPE_CHECKING, Any, cast
 
 import frontmatter
 from pydantic import ValidationError
+from unique_skill_tool.config import SkillToolConfig
 from unique_skill_tool.schemas import SkillDefinition
 from unique_skill_tool.service import SkillTool
 from unique_skill_tool.utils import extract_prefix_skills
+from unique_toolkit.agentic.tools.config import ToolBuildConfig
 from unique_toolkit.agentic.tools.schemas import ToolCallResponse
 from unique_toolkit.content.schemas import Content, ContentInfo
 from unique_toolkit.content.smart_rules import (
@@ -70,6 +72,16 @@ if TYPE_CHECKING:
     from unique_orchestrator.config import UniqueAIConfig
 
 
+def _find_skill_tool_build_config(
+    tools: list[ToolBuildConfig],
+) -> ToolBuildConfig | None:
+    """Return the SkillTool entry from ``space.tools`` if present."""
+    for tool in tools:
+        if tool.name == SkillTool.name:
+            return tool
+    return None
+
+
 _SUBTREE_PAGE_SIZE = 100
 _MAX_SUBTREE_ITEMS = 10_000
 
@@ -294,21 +306,39 @@ async def configure_skill_tool(
     content_service: ContentService,
     tool_manager: ToolManager | ResponsesApiToolManager,
 ) -> None:
-    """Register the SkillTool if enabled in the config.
-
-    Lists all ``.md`` files in the configured ``scope_ids`` (and every
-    sub-folder reachable from them), downloads each one concurrently,
-    and registers the SkillTool with the parsed skills.
+    """Populate the SkillTool's skill registry when it is enabled in ``space.tools``.
+
+    The SkillTool is integrated like every other tool: it lives as a
+    ``ToolBuildConfig`` entry in ``config.space.tools`` and is built by
+    :class:`ToolFactory` when the :class:`ToolManager` is constructed.
+    This function only handles the runtime concern of loading the skill
+    definitions from the knowledge base and injecting them into the
+    already-constructed tool instance.
+
+    The tool is treated as "active" only when both:
+
+    * an entry with name ``SkillTool.name`` is present in
+      ``config.space.tools`` and its
+      :attr:`~ToolBuildConfig.is_enabled` flag is ``True``, **and**
+    * the resulting skill registry contains at least one skill.
+
+    When either is not the case the SkillTool is excluded from the
+    manager so it is never advertised to the model. Its system prompt
+    claims HIGHEST PRIORITY and would otherwise waste iterations or
+    provoke hallucinated skill names when no skills exist.
     """
-    skill_config = config.agent.experimental.skill_tool_config
-    if not skill_config.enabled:
+    skill_tool_build_config = _find_skill_tool_build_config(config.space.tools)
+    if skill_tool_build_config is None or not skill_tool_build_config.is_enabled:
         return
 
+    skill_config = cast(SkillToolConfig, skill_tool_build_config.configuration)
+
     if not skill_config.scope_ids:
         logger.warning(
             "SkillTool is enabled but no scope_ids are configured — "
             "no skills will be loaded."
         )
+        tool_manager.exclude_tool(SkillTool.name)
         return
 
     knowledge_base_service = KnowledgeBaseService(
@@ -324,18 +354,20 @@ async def configure_skill_tool(
     )
 
     if not skill_registry:
-        logger.info(
-            "SkillTool has an empty skill registry — tool will not be registered."
-        )
+        logger.info("SkillTool has an empty skill registry — tool will be excluded.")
+        tool_manager.exclude_tool(SkillTool.name)
         return
 
-    tool_manager.add_tool(
-        SkillTool(
-            event=event,
-            skill_registry=skill_registry,
-            config=skill_config,
+    skill_tool = tool_manager.get_tool_by_name(SkillTool.name)
+    if not isinstance(skill_tool, SkillTool):
+        logger.warning(
+            "SkillTool is configured in space.tools but the manager did "
+            "not produce a SkillTool instance — skills will not be "
+            "available."
         )
-    )
+        return
+
+    skill_tool.skill_registry = skill_registry
 
 
 async def preload_invoked_skills(

{unique_orchestrator-2026.20.0.dev7 → unique_orchestrator-2026.20.0.dev8}/unique_orchestrator/config.py

@@ -9,9 +9,6 @@ from unique_deep_research.service import DeepResearchTool
 from unique_follow_up_questions.config import FollowUpQuestionsConfig
 from unique_internal_search.config import InternalSearchConfig
 from unique_internal_search.service import InternalSearchTool
-from unique_skill_tool.config import (
-    SkillToolConfig,
-)
 from unique_stock_ticker.config import StockTickerConfig
 from unique_swot import SwotAnalysisTool, SwotAnalysisToolConfig
 from unique_toolkit._common.validators import (
@@ -351,8 +348,6 @@ class ExperimentalConfig(BaseToolConfig):
 
     open_file_tool_config: OpenFileToolConfig = OpenFileToolConfig()
 
-    skill_tool_config: SkillToolConfig = SkillToolConfig()
-
     use_responses_api: bool = Field(
         default=False,
         description="If set, the main agent will use the Responses API from OpenAI",

{unique_orchestrator-2026.20.0.dev7 → unique_orchestrator-2026.20.0.dev8}/unique_orchestrator/unique_ai.py

@@ -214,15 +214,14 @@ class UniqueAI:
         self._execution_times = []
         run_start = time.perf_counter()
 
-        if self._config.agent.experimental.skill_tool_config.enabled:
-            stripped_text = await preload_invoked_skills(
-                event=self._event,
-                tool_manager=self._tool_manager,
-                history_manager=self._history_manager,
-                logger=self._logger,
-            )
-            if stripped_text is not None:
-                self._event.payload.user_message.text = stripped_text
+        stripped_text = await preload_invoked_skills(
+            event=self._event,
+            tool_manager=self._tool_manager,
+            history_manager=self._history_manager,
+            logger=self._logger,
+        )
+        if stripped_text is not None:
+            self._event.payload.user_message.text = stripped_text
 
         sub = self._chat_service.cancellation.on_cancellation.subscribe(
             self._on_cancellation