The function params seem to be overcomplicated, causing confusion in models (e.g., llama 3.3 70B).
It might be better if the standard searches defaulted to ALL as their country, for example, instead of making the model decide whether the "country" parameter is the country of search origin or the country of search topic. Because if the country of the search topic isn't in your list of countries, it gets very confused, retries multiple times, thinks about all of possible country options it has, and so on.
Certainly bigger models could do better, but I don't think brave MCP is really being connected to SOTA models much; it's going to appeal for local, self-hosted models more often.
Suggest that the more advanced params could be under "advanced_search_*" function names instead.
The function params seem to be overcomplicated, causing confusion in models (e.g., llama 3.3 70B).
It might be better if the standard searches defaulted to ALL as their country, for example, instead of making the model decide whether the "country" parameter is the country of search origin or the country of search topic. Because if the country of the search topic isn't in your list of countries, it gets very confused, retries multiple times, thinks about all of possible country options it has, and so on.
Certainly bigger models could do better, but I don't think brave MCP is really being connected to SOTA models much; it's going to appeal for local, self-hosted models more often.
Suggest that the more advanced params could be under "advanced_search_*" function names instead.