unique_toolkit 1.64.2__tar.gz → 1.64.4__tar.gz

{unique_toolkit-1.64.2 → unique_toolkit-1.64.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: unique-toolkit
-Version: 1.64.2
+Version: 1.64.4
 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-1.64.2 → unique_toolkit-1.64.4}/pyproject.toml

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

{unique_toolkit-1.64.2 → unique_toolkit-1.64.4}/unique_toolkit/_common/config_checker/cli.py

@@ -163,9 +163,7 @@ def check(
         report = validator.validate_all(
             artifacts_dir, config_entries, fail_on_missing=fail_on_missing
         )
-        configs_with_changes = [
-            r for r in report.results if r.valid and r.default_changes
-        ]
+        configs_with_changes = [r for r in report.results if r.default_changes]
         total_default_changes = sum(
             len(r.default_changes or []) for r in configs_with_changes
         )
@@ -277,9 +275,7 @@ def _generate_markdown_report(
 
     # Default changes (if requested)
     if report_defaults:
-        configs_with_changes = [
-            r for r in report.results if r.valid and r.default_changes
-        ]
+        configs_with_changes = [r for r in report.results if r.default_changes]
         if configs_with_changes:
             lines.append("### 📊 Default Value Changes (non-breaking)\n")
             for result in configs_with_changes:

{unique_toolkit-1.64.2 → unique_toolkit-1.64.4}/unique_toolkit/_common/config_checker/validator.py

@@ -147,12 +147,24 @@ class ConfigValidator:
 
             errors.extend(new_errors)
 
+            # Still compute default changes so they are reported even when validation fails
+            default_changes = None
+            try:
+                default_instance = new_model.model_construct()
+                default_changes = self.differ.compare_defaults(
+                    old_json, default_instance
+                )
+            except Exception as e:
+                logger.debug(
+                    f"Could not compute default changes for '{config_name}': {e}"
+                )
+
             return ConfigValidationResult(
                 config_name=config_name,
                 valid=False,
                 errors=errors,
                 warnings=warnings if warnings else None,
-                default_changes=None,
+                default_changes=default_changes if default_changes else None,
             )
 
     def validate_all(

{unique_toolkit-1.64.2 → unique_toolkit-1.64.4}/unique_toolkit/agentic/tools/openai_builtin/code_interpreter/postprocessors/generated_files.py

@@ -196,11 +196,11 @@ class DisplayCodeInterpreterFilesPostProcessor(
                 )
                 changed |= replaced
 
-            # HTML rendered as legacy HtmlRendering block — only when fence FF is off
-            elif (
-                is_html
-                and not fence_ff_on
-                and feature_flags.enable_html_rendering_un_15131.is_enabled(
+            # HTML rendered as HtmlRendering block — when fence FF is on, or when
+            # the legacy HTML-rendering FF is on (fence FF off path).
+            elif is_html and (
+                fence_ff_on
+                or feature_flags.enable_html_rendering_un_15131.is_enabled(
                     self._company_id
                 )
             ):
@@ -211,7 +211,7 @@ class DisplayCodeInterpreterFilesPostProcessor(
                 )
                 changed |= replaced
 
-            # Files and HTML (fence FF on: HTML → htmlWithSource; others → fileWithSource)
+            # Non-HTML files
             else:
                 loop_response.message.text, replaced = _replace_container_file_citation(
                     text=loop_response.message.text,
@@ -222,21 +222,14 @@ class DisplayCodeInterpreterFilesPostProcessor(
                 )
                 changed |= replaced
 
-            is_html_rendered = (
-                is_html
-                and not fence_ff_on
-                and feature_flags.enable_html_rendering_un_15131.is_enabled(
+            # HtmlRendering blocks carry contentId directly — no separate reference needed
+            is_html_rendered = is_html and (
+                fence_ff_on
+                or feature_flags.enable_html_rendering_un_15131.is_enabled(
                     self._company_id
                 )
             )
-            # htmlWithSource fences carry contentId directly — no separate reference needed
-            is_html_fenced = is_html and fence_ff_on
-            if (
-                replaced
-                and not is_image
-                and not is_html_rendered
-                and not is_html_fenced
-            ):
+            if replaced and not is_image and not is_html_rendered:
                 loop_response.message.references.append(
                     ContentReference(
                         sequence_number=ref_number,
@@ -520,6 +513,9 @@ def _build_file_fence(file: CodeInterpreterFile, code: str, fence_id: int) -> st
     content_id, a title derived from the filename, the type, and the
     escaped source code.
 
+    HTML files are handled separately via HtmlRendering blocks and never
+    reach this function.
+
     Format (4 backticks so inner code backticks never close the fence):
       ````imgWithSource(id='{n}', contentId='{id}', title="{title}", code="{escaped_code}")````
       ````fileWithSource(id='{n}', contentId='{id}', title="{title}", type="{type}", code="{escaped_code}")````
@@ -529,8 +525,6 @@ def _build_file_fence(file: CodeInterpreterFile, code: str, fence_id: int) -> st
     outer = "````"
     if file.type == "image":
         tag = f"imgWithSource(id='{fence_id}', contentId='{file.content_id}', title=\"{title}\", code=\"{escaped}\")"
-    elif file.type == "html":
-        tag = f"htmlWithSource(id='{fence_id}', contentId='{file.content_id}', title=\"{title}\", code=\"{escaped}\")"
     else:
         ftype = _file_frontend_type(file.filename)
         tag = f'fileWithSource(id=\'{fence_id}\', contentId=\'{file.content_id}\', title="{title}", type="{ftype}", code="{escaped}")'
@@ -554,7 +548,7 @@ def _inline_ref_pattern(file: CodeInterpreterFile) -> re.Pattern[str]:
 
 
 _FENCE_BLOCK_START = re.compile(
-    r"^[^\n`]+?(````(?:imgWithSource|fileWithSource|htmlWithSource)\()",
+    r"^[^\n`]+?(````(?:imgWithSource|fileWithSource)\()",
     re.MULTILINE,
 )
 
@@ -564,9 +558,9 @@ _FENCE_BLOCK_START = re.compile(
 #   - cross-line: 1 or 2 newlines optionally surrounded by horizontal whitespace
 #     (list-item linebreak, or blank-line paragraph gap)
 _CONSECUTIVE_FENCES_RE = re.compile(
-    r"(````(?:imgWithSource|fileWithSource|htmlWithSource)\([^\n]*\)````)"
+    r"(````(?:imgWithSource|fileWithSource)\([^\n]*\)````)"
     r"(?:[^\n`]*|[ \t]*\n{1,2}[ \t]*)"
-    r"(?=````(?:imgWithSource|fileWithSource|htmlWithSource)\()"
+    r"(?=````(?:imgWithSource|fileWithSource)\()"
 )
 
 
@@ -816,6 +810,9 @@ def _warn_unmatched_code_blocks(
     for filename, content_id in content_map.items():
         if content_id is None:
             continue
+        # HTML files are rendered via HtmlRendering blocks, not fence injection
+        if _get_file_type(filename) == "html":
+            continue
         if filename not in fenced_filenames:
             logger.warning(
                 "File '%s' (content_id=%s) could not be matched to any code block "
@@ -895,13 +892,9 @@ def _replace_container_html_citation(
         return text, False
 
     logger.info("Inserting HTML rendering block for '%s'", filename)
-    html_rendering_block = f"""```HtmlRendering
-100%
-500px
-
-unique://content/{content_id}
-
-```"""
+    html_rendering_block = (
+        f"```HtmlRendering\n800px\n600px\n\n\nunique://content/{content_id}\n\n```"
+    )
     return re.sub(
         html_markdown,
         html_rendering_block,