unique-toolkit 2026.40.0.dev3__tar.gz → 2026.40.0.dev5__tar.gz

{unique_toolkit-2026.40.0.dev3 → unique_toolkit-2026.40.0.dev5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: unique-toolkit
-Version: 2026.40.0.dev3
+Version: 2026.40.0.dev5
 Summary: 
 Author: Cedric Klinkert, Martin Fadler, Sadique Sheik, Fabian Schläpfer, Pascal Hauri, Ejiroghene Oki-Peter, Rami Azouz, Ricardo Visini, Sanzio Monti
 Author-email: Cedric Klinkert <cedric.klinkert@unique.ch>, Martin Fadler <martin.fadler@unique.ch>, Sadique Sheik <sadique@unique.ch>, Fabian Schläpfer <fabian@unique.ch>, Pascal Hauri <pascal@unique.ch>, Ejiroghene Oki-Peter <oki-peter.ejiroghene@unique.ai>, Rami Azouz <rami.ext@unique.ch>, Ricardo Visini <ricardo.visini@unique.ch>, Sanzio Monti <sanzio.monti@unique.ai>

{unique_toolkit-2026.40.0.dev3 → unique_toolkit-2026.40.0.dev5}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "unique_toolkit"
-version = "2026.40.0.dev3"
+version = "2026.40.0.dev5"
 description = ""
 readme = "README.md"
 requires-python = ">=3.12"

{unique_toolkit-2026.40.0.dev3 → unique_toolkit-2026.40.0.dev5}/unique_toolkit/agentic/feature_flags/feature_flags.py

@@ -23,10 +23,6 @@ class FeatureFlagNames(StrEnum):
         "FEATURE_FLAG_ENABLE_MCP_METADATA_FALLBACK_UN_19145"
     )
     enable_html_rendering_un_15131 = "FEATURE_FLAG_ENABLE_HTML_RENDERING_UN_15131"
-    enable_code_execution_fence_un_17972 = (
-        "FEATURE_FLAG_ENABLE_CODE_EXECUTION_FENCE_UN_17972"
-    )
-    enable_html_with_fence_un_17927 = "FEATURE_FLAG_ENABLE_HTML_WITH_FENCE_UN_17927"
     enable_web_search_argument_screening_un_18741 = (
         "FEATURE_FLAG_ENABLE_WEB_SEARCH_ARGUMENT_SCREENING_UN_18741"
     )
@@ -110,12 +106,24 @@ class FeatureFlags(BaseSettings):
 
     enable_code_execution_fence_un_17972: FeatureFlag = Field(
         default=FeatureFlag(False),
-        description="Emit codeExecution fences in message.text for code interpreter outputs (UN-17972). When disabled, inline file refs are kept as-is. Can be 'true' or comma-separated company IDs.",
+        description=(
+            "Retired: replaced by the `enable_code_execution_fence` config field "
+            "on ShowExecutedCodePostprocessorConfig (default true). "
+            "No code reads this field anymore. Kept only so removing it doesn't "
+            "trip the config-checker's breaking-change gate on this settings "
+            "model; delete once that checker allows field removal."
+        ),
     )
 
     enable_html_with_fence_un_17927: FeatureFlag = Field(
         default=FeatureFlag(False),
-        description="Render HTML code interpreter files as htmlWithSource fences instead of HtmlRendering blocks (UN-17927). Requires enable_code_execution_fence_un_17972 to also be enabled. Can be 'true' or comma-separated company IDs.",
+        description=(
+            "Retired: replaced by the `enable_html_with_fence` config field "
+            "on ShowExecutedCodePostprocessorConfig (default true). "
+            "No code reads this field anymore. Kept only so removing it doesn't "
+            "trip the config-checker's breaking-change gate on this settings "
+            "model; delete once that checker allows field removal."
+        ),
     )
 
     enable_web_search_argument_screening_un_18741: FeatureFlag = Field(

{unique_toolkit-2026.40.0.dev3 → unique_toolkit-2026.40.0.dev5}/unique_toolkit/agentic/tools/openai_builtin/code_interpreter/postprocessors/code_display.py

@@ -6,15 +6,10 @@ from typing import override
 from pydantic import BaseModel, Field
 from pydantic.json_schema import SkipJsonSchema
 
-from unique_toolkit.agentic.feature_flags import FeatureFlagNames
 from unique_toolkit.agentic.postprocessor.postprocessor_manager import (
     ResponsesApiPostprocessor,
 )
 from unique_toolkit.agentic.tools.config import get_configuration_dict
-from unique_toolkit.experimental.resources.feature_flags import (
-    COMPANY_ID_PLACEHOLDER,
-    is_flag_enabled,
-)
 from unique_toolkit.language_model.schemas import ResponsesLanguageModelStreamResponse
 
 _TEMPLATE = """
@@ -39,6 +34,22 @@ class ShowExecutedCodePostprocessorConfig(BaseModel):
         default=True,
         description="Show the executed source code to the user",
     )
+    enable_code_execution_fence: bool = Field(
+        default=True,
+        description=(
+            "Show generated files as interactive cards with a built-in code view "
+            "and hide the legacy collapsible code block below."
+        ),
+    )
+    enable_html_with_fence: bool = Field(
+        default=True,
+        description=(
+            "Show generated HTML files as interactive cards with a built-in code "
+            "view instead of a plain rendered HTML block. Has no effect on its "
+            "own: 'enable_code_execution_fence' must be on as well. While that "
+            "switch is off, HTML files stay a plain rendered HTML block."
+        ),
+    )
     remove_from_history: SkipJsonSchema[bool] = (
         Field(  # At the moment, it's not possible to keep executed code in the history
             default=True,
@@ -55,20 +66,15 @@ class ShowExecutedCodePostprocessor(ResponsesApiPostprocessor):
     def __init__(
         self,
         config: ShowExecutedCodePostprocessorConfig,
-        company_id: str | None = None,
     ):
         super().__init__(self.__class__.__name__)
         self._config = config
-        self._company_id = company_id
-        # Resolved in run() (before apply_postprocessing_to_response) since flag evaluation is async.
-        self._is_enabled = False
+        # When fences are enabled, the fence itself shows the executed code,
+        # so this legacy <details> display must stay off.
+        self._is_enabled = config.enable and not config.enable_code_execution_fence
 
     @override
     async def run(self, loop_response: ResponsesLanguageModelStreamResponse) -> None:
-        self._is_enabled = self._config.enable and not await is_flag_enabled(
-            FeatureFlagNames.enable_code_execution_fence_un_17972,
-            company_id=self._company_id or COMPANY_ID_PLACEHOLDER,
-        )
         if self._is_enabled:
             await asyncio.sleep(self._config.sleep_time_before_display)
 

{unique_toolkit-2026.40.0.dev3 → unique_toolkit-2026.40.0.dev5}/unique_toolkit/agentic/tools/openai_builtin/code_interpreter/postprocessors/generated_files.py

@@ -21,7 +21,6 @@ from tenacity import (
 
 from unique_toolkit import ChatService
 from unique_toolkit._common.execution import failsafe_async
-from unique_toolkit.agentic.feature_flags import FeatureFlagNames
 from unique_toolkit.agentic.postprocessor.postprocessor_manager import (
     ResponsesApiPostprocessor,
 )
@@ -40,10 +39,6 @@ from unique_toolkit.agentic.tools.openai_builtin.code_interpreter.schemas import
 )
 from unique_toolkit.content.schemas import ContentReference
 from unique_toolkit.content.service import ContentService
-from unique_toolkit.experimental.resources.feature_flags import (
-    COMPANY_ID_PLACEHOLDER,
-    is_flag_enabled,
-)
 from unique_toolkit.language_model.schemas import ResponsesLanguageModelStreamResponse
 from unique_toolkit.services.knowledge_base import KnowledgeBaseService
 from unique_toolkit.short_term_memory.service import ShortTermMemoryService
@@ -376,6 +371,8 @@ class DisplayCodeInterpreterFilesPostProcessor(
         company_id: str | None = None,
         user_id: str | None = None,
         chat_id: str | None = None,
+        code_execution_fence_enabled: bool = True,
+        html_with_fence_enabled: bool = True,
     ) -> None:
         super().__init__(self.__class__.__name__)
 
@@ -383,7 +380,6 @@ class DisplayCodeInterpreterFilesPostProcessor(
         self._chat_service = chat_service
         self._client = client
         self._config = config
-        self._company_id = company_id
 
         if self._chat_service is None:
             raise ValueError("ChatService is required if uploadToChat is True")
@@ -404,9 +400,11 @@ class DisplayCodeInterpreterFilesPostProcessor(
         self._file_size_map: dict[str, int] = {}
         self._container_files: list[CodeInterpreterContainerFile] = []
 
-        # Resolved in run() (before apply_postprocessing_to_response) since flag evaluation is async.
-        self._fence_ff_on = False
-        self._html_fence_ff_on = False
+        # Both switches are plain config values now (were FFs UN-17972 / UN-17927).
+        # They live on ShowExecutedCodePostprocessorConfig ("Code display" on the
+        # config page) and are passed in by the orchestrator.
+        self._fence_enabled = code_execution_fence_enabled
+        self._html_fence_enabled = html_with_fence_enabled
 
     def _build_retry(self) -> AsyncRetrying:
         """Build a tenacity retry policy from the current config.
@@ -511,17 +509,6 @@ class DisplayCodeInterpreterFilesPostProcessor(
         run_t0 = time.monotonic()
         self._log.info("run() started — fetching and uploading code interpreter files")
 
-        self._fence_ff_on = await is_flag_enabled(
-            FeatureFlagNames.enable_code_execution_fence_un_17972,
-            company_id=self._company_id or COMPANY_ID_PLACEHOLDER,
-        )
-        # htmlWithSource requires BOTH the fence FF and the HTML-fence FF on; default
-        # (FF off) keeps HtmlRendering, so existing deployments are unaffected.
-        self._html_fence_ff_on = await is_flag_enabled(
-            FeatureFlagNames.enable_html_with_fence_un_17927,
-            company_id=self._company_id or COMPANY_ID_PLACEHOLDER,
-        )
-
         container_files = await self._resolve_container_files(loop_response)
         self._container_files = container_files
         self._log.info(
@@ -668,9 +655,9 @@ class DisplayCodeInterpreterFilesPostProcessor(
     ) -> bool:
         apply_t0 = time.monotonic()
         self._log.info(
-            "apply_postprocessing started — %d file(s) in content_map, fence_ff=%s",
+            "apply_postprocessing started — %d file(s) in content_map, fence_enabled=%s",
             len(self._content_map),
-            self._fence_ff_on,
+            self._fence_enabled,
         )
 
         if loop_response.message.references is None:
@@ -711,9 +698,9 @@ class DisplayCodeInterpreterFilesPostProcessor(
                 )
                 changed |= replaced
 
-            # HTML rendered as HtmlRendering block unless the dedicated html-fence FF
-            # is on (enable_html_with_fence_un_17927). Default keeps HtmlRendering.
-            elif is_html and not self._html_fence_ff_on:
+            # HTML uses HtmlRendering unless BOTH switches are on (htmlWithSource
+            # needs the code-execution fence too).
+            elif is_html and not (self._html_fence_enabled and self._fence_enabled):
                 loop_response.message.text, replaced = _replace_container_html_citation(
                     text=loop_response.message.text or "",
                     filename=filename,
@@ -721,15 +708,15 @@ class DisplayCodeInterpreterFilesPostProcessor(
                 )
                 changed |= replaced
 
-            # Non-HTML files, or HTML when self._html_fence_ff_on → inline content link for
-            # subsequent fence injection (htmlWithSource / imgWithSource / fileWithSource)
+            # Non-HTML files, or HTML when both fence switches are on → inline
+            # content link for subsequent fence injection
             else:
                 loop_response.message.text, replaced = _replace_container_file_citation(
                     text=loop_response.message.text or "",
                     filename=filename,
                     content_id=content_id,
                     ref_number=ref_number,
-                    use_content_link=self._fence_ff_on,
+                    use_content_link=self._fence_enabled,
                 )
                 changed |= replaced
 
@@ -746,7 +733,7 @@ class DisplayCodeInterpreterFilesPostProcessor(
 
             # HtmlRendering and htmlWithSource both embed contentId directly — no ContentReference needed
             is_html_rendered = is_html
-            if replaced and not (is_image or is_html_rendered or self._fence_ff_on):
+            if replaced and not (is_image or is_html_rendered or self._fence_enabled):
                 loop_response.message.references.append(
                     ContentReference(
                         sequence_number=ref_number,
@@ -765,12 +752,12 @@ class DisplayCodeInterpreterFilesPostProcessor(
             error_files,
         )
 
-        if self._fence_ff_on:
+        if self._fence_enabled:
             code_blocks = _build_code_blocks(
                 loop_response,
                 self._content_map,
                 self._container_files,
-                include_html=self._html_fence_ff_on,
+                include_html=self._html_fence_enabled,
             )
             self._log.info(
                 "Fence injection — %d code block(s), files: %s",
@@ -778,7 +765,7 @@ class DisplayCodeInterpreterFilesPostProcessor(
                 [f.filename for b in code_blocks for f in b.files],
             )
             _warn_unmatched_code_blocks(
-                self._content_map, code_blocks, include_html=self._html_fence_ff_on
+                self._content_map, code_blocks, include_html=self._html_fence_enabled
             )
             text_before = loop_response.message.text
             loop_response.message.text = _inject_code_execution_fences(
@@ -1503,7 +1490,7 @@ def _warn_unmatched_code_blocks(
 ) -> None:
     """Warn for files that were uploaded but could not be matched to any code block.
 
-    When the fence feature flag is on, every uploaded file should map to a code block
+    When fence rendering is enabled, every uploaded file should map to a code block
     via its /mnt/data/<filename> path so it can receive a fence.  If a file is not
     matched (e.g. the LLM used a variable for the output path rather than a literal
     string) it falls back to a plain unique://content/ link with no code context.
@@ -1634,10 +1621,10 @@ def _replace_container_file_citation(
 ) -> tuple[str, bool]:
     """Replace a sandbox file link with either an inline content link or a superscript ref.
 
-    When the fence feature flag is on (use_content_link=True), the sandbox link is
-    replaced with [filename](unique://content/{id}) so the subsequent fence injection
-    step can locate and wrap it. When the flag is off (use_content_link=False), the
-    original pre-fence behaviour is restored: the link is replaced with <sup>N</sup>
+    When fence rendering is enabled in the config (use_content_link=True), the sandbox
+    link is replaced with [filename](unique://content/{id}) so the subsequent fence
+    injection step can locate and wrap it. When it is disabled (use_content_link=False),
+    the original pre-fence behaviour is restored: the link is replaced with <sup>N</sup>
     and the file remains accessible via the references panel.
     """
     file_markdown = _sandbox_link_pattern(filename)

{unique_toolkit-2026.40.0.dev3 → unique_toolkit-2026.40.0.dev5}/unique_toolkit/agentic/tools/openai_builtin/code_interpreter/tool/config.py

@@ -96,9 +96,9 @@ DEFAULT_TOOL_DESCRIPTION_FOR_SYSTEM_PROMPT = (
     _TOOL_DESCRIPTION_FOR_SYSTEM_PROMPT_BASE + _TOOL_DESCRIPTION_FILES_SUFFIX
 ).strip()
 
-# Used when the code-execution fence feature flag (UN-17972) is enabled.
-# The frontend derives the artifact title from the filename itself, so the
-# LLM no longer needs to produce a markdown heading before the sandbox link.
+# Default system prompt. The frontend derives the artifact title from the
+# filename itself, so the LLM does not need to produce a markdown heading
+# before the sandbox link.
 #
 # UN-19711 / Horizon follow-up: The HTML rules in the fence prompt reduce iframe height
 # collapse when chat auto-measures the embedded document. Remaining gaps belong in the