fix: wire test_mcp_tool into MCP manager startup injection
The startup loop in main.py assigns _mcp_manager only to tools that
have the attribute, but test_mcp_tool never declared it and mcp_status
used _manager instead of _mcp_manager — so both received None forever.

Changes:
- test_mcp_tool: add __init__(mcp_manager) with _mcp_manager, fallback
  to module-level import if startup wiring skipped
- mcp_status: rename _manager → _mcp_manager, same fallback
- registry.py: register create_mcp_server and test_mcp_tool builtins
- main.py: include test_mcp_tool in startup wiring loop
- client.py: add 5s timeout to disconnect cleanup

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent df38579 commit ad5e068beb1e7ae33c7f7b6def15ccdd690c71d9
@Eugene Sukhodolskiy Eugene Sukhodolskiy authored on 15 May
Showing 4 changed files
View
navi/core/registry.py
View
navi/main.py
View
navi/mcp/client.py
View
navi/tools/mcp_status.py