Only use MCP handler with non-empty string

Author: bandoti

common/common.cpp

@@ -1839,8 +1839,8 @@ std::string common_chat_apply_template(
         toolcall::handler::ptr handler,
         toolcall::sampling_updater * update_sparams)
 {
-    const auto & tmpl_selected =
-        handler != nullptr && tmpl.template_tool_use ? *tmpl.template_tool_use : *tmpl.template_default;
+    bool use_tool_template = (use_jinja && handler != nullptr) && tmpl.template_tool_use;
+    const auto & tmpl_selected = use_tool_template ? *tmpl.template_tool_use : *tmpl.template_default;
 
     if (use_jinja) {
         common_chat_inputs inputs;

common/toolcall/handler.cpp

@@ -21,7 +21,9 @@ std::shared_ptr<toolcall::handler> toolcall::create_handler(const toolcall::para
     bool has_uri = std::holds_alternative<std::string>(tools);
     if (has_uri) {
         auto tools_str = std::get<std::string>(tools);
-        result.reset(new toolcall::handler(std::make_unique<toolcall::mcp_impl>(tools_str, choice)));
+        if (! tools_str.empty()) {
+            result.reset(new toolcall::handler(std::make_unique<toolcall::mcp_impl>(tools_str, choice)));
+        }
 
     } else {
         auto tools_ptr = std::get<toolcall::json_ptr>(tools);