fix(agent): unwrap pre-wrapped tool schemas from context engines and memory providers

[srojk34]

Summary

• Context engines and memory providers can return schemas already in {"type":"function","function":{...}} (OpenAI tool) form. Each consumer site blindly wrapped them again, producing a tool with no top-level name.
• An un-named tool entry is silently un-routable inside Hermes and triggers HTTP 400: tools[N].function: missing field name on strict providers such as DeepSeek — failing every request in the session, not just the offending tool.
• Added a nameless-schema guard in the context-engine loop so any schema still lacking a name after unwrapping is skipped rather than appended as dead weight.

Changes

File	Change
agent/agent_init.py	Unwrap pre-wrapped schemas before registration; skip nameless entries
agent/memory_manager.py	Same unwrap in _add_schemas_to_agent, _tool_to_provider index, and get_all_tool_schemas()
tests/agent/test_memory_provider.py	5 new tests covering both fix paths; updated TestContextEngineToolsetGate helper to mirror the fixed production code

Test plan

• uv run --frozen python -m pytest tests/agent/test_memory_provider.py -x -q → 96 passed
• Verify that a context engine returning already-wrapped schemas no longer causes HTTP 400 on DeepSeek / other strict providers
• Confirm that valid unwrapped schemas continue to register and route normally

Fixes #47707

[teknium1]

Automated hermes-sweeper review: this fix is already implemented on current main via merged PR #52140.

Evidence:

• agent/memory_manager.py:49 defines normalize_tool_schema(), which unwraps already-wrapped OpenAI tool entries and rejects schemas without a resolvable string name.
• agent/agent_init.py:1714 uses that normalization in the context-engine schema loop and skips malformed entries instead of appending a nameless tool.
• agent/memory_manager.py:412 and agent/memory_manager.py:703 apply the same normalization to memory-provider routing and advertised schemas.
• tests/agent/test_memory_provider.py:1492 covers the wrapped-schema, nameless-schema, double-wrapped, and real injection paths.
• The implementation landed in merge commit cedbb4cfa275bd594e1f99cb5be554cd1fc1d668 from fix(agent): validate context/memory tool schemas before wrapping (#47707) #52140, which also closes the same underlying Context-engine/memory-provider tool schemas are wrapped without validating a top-level name — one malformed schema 400s the entire request #47707 issue.

Thanks for the report and fix work here; the behavior is now covered on main.