[Bug]: Register for LLM is not registering the function in the agent's function map

[marklysze]

Describe the bug

When trying a swarm with a ConversableAgent-based agent that had Tools associated with it that were registered for llm, the swarm failed because it was setting up the internal tool executor to execute all tools associated with all agents. The reason is because the tools were in the agent's tools but not their _function_map.

I noticed that when we run _register_for_llm we do not call register_function that puts these functions in an agent's function map.

_register_for_execution does call it.

Steps to reproduce

No response

Model Used

No response

Expected Behavior

No response

Screenshots and logs

No response

Additional Information

No response

[marklysze]

Closing this as though a minor fix was applied, the issue raised expects that functions are registered in the function_map if they are just for LLM recommendation, and this is not the case.