iatoolkit 2.50.1__tar.gz → 2.50.2__tar.gz

{iatoolkit-2.50.1/src/iatoolkit.egg-info → iatoolkit-2.50.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: iatoolkit
-Version: 2.50.1
+Version: 2.50.2
 Summary: IAToolkit
 Author: Fernando Libedinsky
 License-Expression: MIT

{iatoolkit-2.50.1 → iatoolkit-2.50.2}/src/iatoolkit/__init__.py

@@ -3,7 +3,7 @@
 #
 # IAToolkit is open source software.
 
-__version__ = "2.50.1"
+__version__ = "2.50.2"
 
 # Expose main classes and functions at the top level of the package
 

{iatoolkit-2.50.1 → iatoolkit-2.50.2}/src/iatoolkit/core.py

@@ -464,7 +464,9 @@ class IAToolkit:
         binder.bind(ParsingProviderResolver, to=ParsingProviderResolver)
         binder.bind(DoclingParsingProvider, to=DoclingParsingProvider)
         binder.bind(BasicParsingProvider, to=BasicParsingProvider)
-        binder.bind(PromptService, to=PromptService)
+        # this class can be setup before by iatoolkit enterprise
+        if not is_bound(self._injector, PromptService):
+            binder.bind(PromptService, to=PromptService)
         binder.bind(ExcelService, to=ExcelService)
         binder.bind(PdfService, to=PdfService)
         binder.bind(MailService, to=MailService)
@@ -480,7 +482,19 @@ class IAToolkit:
         binder.bind(BrandingService, to=BrandingService)
         binder.bind(I18nService, to=I18nService)
         binder.bind(LanguageService, to=LanguageService)
-        binder.bind(ConfigurationService, to=ConfigurationService)
+        # this class can be setup before by iatoolkit enterprise.
+        #
+        # Binding it unconditionally overwrote that override for the rest of this
+        # method and, crucially, for `_instantiate_company_instances()` a few
+        # steps later: anything built there — `TaskExecutorRegistry` is a
+        # singleton, so its whole graph froze — kept the community
+        # ConfigurationService for the life of the process, while every later
+        # `injector.get()` got the enterprise one. Prompts answered from the web
+        # and failed from the worker: the task executor's copy knew nothing of
+        # the model catalogue, so an OpenRouter model was routed by guessing at
+        # its name.
+        if not is_bound(self._injector, ConfigurationService):
+            binder.bind(ConfigurationService, to=ConfigurationService)
         binder.bind(TelemetryService, to=TelemetryService)
         binder.bind(EmbeddingService, to=EmbeddingService)
         binder.bind(HistoryManagerService, to=HistoryManagerService)

{iatoolkit-2.50.1 → iatoolkit-2.50.2}/src/iatoolkit/services/configuration_service.py

@@ -505,6 +505,16 @@ class ConfigurationService:
         else:
             if not config.get("llm", {}).get("model"):
                 add_error("llm", "Missing required key: 'model'")
+            elif not isinstance(config.get("llm", {}).get("model"), str):
+                # 'model' names the one model that answers when nothing else
+                # picks one. A list here reads as valid YAML and travels all the
+                # way to the provider SDK, where it surfaces as
+                # "'list' object has no attribute 'lower'" — a config mistake
+                # wearing a runtime error's clothes.
+                add_error(
+                    "llm.model",
+                    "Must be a single model name. To offer several, list them under 'available_models'.",
+                )
             if not config.get("llm", {}).get("provider_api_keys"):
                 add_error("llm", "Missing required key: 'provider_api_keys'")
             allowed_reasoning_efforts = {"minimal", "low", "medium", "high", "xhigh"}

{iatoolkit-2.50.1 → iatoolkit-2.50.2}/src/iatoolkit/services/query_service.py

@@ -209,11 +209,54 @@ class QueryService:
             if prompt_model:
                 effective_model = prompt_model
         if not effective_model:
-            llm_config = self.configuration_service.get_configuration(company_short_name, 'llm')
-            if llm_config and llm_config.get('model'):
-                effective_model = llm_config['model']
+            # Asked of the same resolution the model picker uses, not of the raw
+            # `llm.model` key: on a company backed by the model catalogue the
+            # default travels with its entitlements, so a YAML default the
+            # company can no longer use does not keep being sent.
+            default_model, _ = self.configuration_service.get_llm_configuration(company_short_name)
+            effective_model = self._coerce_model_name(default_model, company_short_name)
         return effective_model
 
+    @staticmethod
+    def _coerce_model_name(raw_model, company_short_name: str) -> Optional[str]:
+        """The configured default as a model name, or None.
+
+        A malformed `llm.model` — a list is the one seen in the wild — used to
+        travel untouched into the provider SDK and fail there as
+        "'list' object has no attribute 'lower'", which reads like a bug in the
+        toolkit rather than a line to fix in the company's configuration. The
+        first usable entry is taken so the company keeps working, and the log
+        says what to correct.
+        """
+        if isinstance(raw_model, str):
+            return raw_model.strip() or None
+        if raw_model is None:
+            return None
+        if isinstance(raw_model, (list, tuple)):
+            for item in raw_model:
+                candidate = str(item or "").strip()
+                if candidate:
+                    logging.warning(
+                        "Company '%s' declares llm.model as a %s; using '%s'. "
+                        "It must be a single model name — list alternatives under llm.available_models.",
+                        company_short_name,
+                        type(raw_model).__name__,
+                        candidate,
+                    )
+                    return candidate
+            return None
+
+        candidate = str(raw_model).strip()
+        if not candidate:
+            return None
+        logging.warning(
+            "Company '%s' declares llm.model as a %s; using '%s'. It must be a single model name.",
+            company_short_name,
+            type(raw_model).__name__,
+            candidate,
+        )
+        return candidate
+
     def _get_history_type(self, company_short_name: str, model: str) -> str:
         provider = self._get_provider(company_short_name, model)
         if provider in ("openai", "xai"):

{iatoolkit-2.50.1 → iatoolkit-2.50.2/src/iatoolkit.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: iatoolkit
-Version: 2.50.1
+Version: 2.50.2
 Summary: IAToolkit
 Author: Fernando Libedinsky
 License-Expression: MIT