hermes - 💡(How to fix) Fix fix: pre_tool_call hook missing session_id in three call sites

Bug description

The get_pre_tool_call_block_message() function is called without session_id in three places, causing pre_tool_call hooks that rely on session-based platform detection to fail silently.

Affected code paths

1. agent/agent_runtime_helpers.py:1561 — invoke_tool()
2. agent/tool_executor.py:129 — execute_tool_calls_concurrent()
3. agent/tool_executor.py:504 — execute_tool_calls_sequential()

Root cause

These call sites pass function_name, function_args, task_id but omit session_id. The model_tools.handle_function_call() path already passes session_id correctly — these three were missed.

Without session_id, _get_session_source("") returns None, so any hook that checks session source to distinguish platform sessions will silently skip the check and never block.

Reproduction

1. Write a plugin with a pre_tool_call hook that blocks based on session_id source
2. Trigger a tool call from a non-CLI platform (wecom, telegram, etc.)
3. The hook never fires because session_id is empty

Fix

 block_message = get_pre_tool_call_block_message(
     function_name, function_args, task_id=effective_task_id or "",
+    session_id=getattr(agent, "session_id", "") or "",
 )