fix: Fix the parameter issue of the str.format(...) method (#5119)

* fix: Fix the parameter issue of the `str.format(...)` method

Author: wangliang181230

apps/application/chat_pipeline/step/chat_step/impl/base_chat_step.py

@@ -183,7 +183,7 @@ def execute(self, message_list: List[BaseMessage],
                 mcp_output_enable=True,
                 **kwargs):
         chat_model = get_model_instance_by_model_workspace_id(model_id, workspace_id,
-                                                              **model_params_setting) if model_id is not None else None
+                                                              **(model_params_setting or {})) if model_id is not None else None
         if stream:
             return self.execute_stream(message_list, chat_id, problem_text, post_response_handler, chat_model,
                                        paragraph_list,

apps/application/flow/common.py

@@ -264,7 +264,7 @@ def is_valid_model_params(self):
                 node.properties.get('node_data', {})['model_params_setting'] = model_params_setting
             if node.properties.get('status', 200) != 200:
                 raise ValidationError(
-                    ErrorDetail(_("Node {node} is unavailable").format(node.properties.get("stepName"))))
+                    ErrorDetail(_("Node {node} is unavailable").format(node=node.properties.get("stepName"))))
         node_list = [node for node in self.nodes if (node.type == 'function-lib-node')]
         for node in node_list:
             function_lib_id = node.properties.get('node_data', {}).get('function_lib_id')

apps/application/flow/step_node/data_source_web_node/impl/base_data_source_web_node.py

@@ -81,7 +81,7 @@ def execute(self, **kwargs) -> NodeResult:
                 return NodeResult({'document_list': document_list, 'source_url': source_url, 'selector': selector},
                                   self.workflow_manage.params.get('knowledge_base') or {})
             maxkb_logger.error(_('data source web node:{node_id} error{error}{traceback}').format(
-                knowledge_id=node_id, error=str(e), traceback=traceback.format_exc()))
+                node_id=node_id, error=str(e), traceback=traceback.format_exc()))
 
     def get_details(self, index: int, **kwargs):
         return {

apps/application/flow/step_node/tool_lib_node/impl/base_tool_lib_node.py

@@ -75,7 +75,7 @@ def valid_reference_value(_type, value, name):
         else:
             raise Exception(_(
                 'Field: {name} Type: {_type} Value: {value} Unsupported types'
-            ).format(name=name, _type=_type))
+            ).format(name=name, _type=_type, value=value))
     except:
         return value
     if not isinstance(value, instance_type):

apps/application/flow/step_node/tool_node/impl/base_tool_node.py

@@ -50,7 +50,7 @@ def valid_reference_value(_type, value, name):
         else:
             raise Exception(_(
                 'Field: {name} Type: {_type} Value: {value} Unsupported types'
-            ).format(name=name, _type=_type))
+            ).format(name=name, _type=_type, value=value))
     except:
         return value
     if not isinstance(value, instance_type):

apps/knowledge/task/embedding.py

@@ -116,9 +116,9 @@ def embedding_by_knowledge(knowledge_id, model_id):
                 pass
     except Exception as e:
         maxkb_logger.error(
-            _('Vectorized knowledge: {knowledge_id} error {error} {traceback}'.format(knowledge_id=knowledge_id,
+            _('Vectorized knowledge: {knowledge_id} error {error} {traceback}').format(knowledge_id=knowledge_id,
                                                                                       error=str(e),
-                                                                                      traceback=traceback.format_exc())))
+                                                                                      traceback=traceback.format_exc()))
     finally:
         maxkb_logger.info(_('End--->Vectorized knowledge: {knowledge_id}').format(knowledge_id=knowledge_id))
 

apps/trigger/handler/impl/trigger/event_trigger.py

@@ -42,7 +42,7 @@ def valid_parameter_type(value, _type, desc):
         else:
             raise Exception(_(
                 'Field: {name} Type: {_type} Value: {value} Unsupported types'
-            ).format(name=desc, _type=_type))
+            ).format(name=desc, _type=_type, value=value))
     except:
         return value
     if not isinstance(value, instance_type):