Bill/AI-Trader
1
0
Fork 0
mirror of https://github.com/Xe138/AI-Trader.git synced 2026-04-01 17:17:24 -04:00
Code Issues Packages Projects Releases Wiki Activity

debug: enable stdout/stderr for MCP services to diagnose parameter injection

Browse Source 
MCP services were started with stdout/stderr redirected to DEVNULL, making
debug logs invisible. This prevented diagnosing why _current_position parameter
is not being received by buy() function.

Changed subprocess.Popen to redirect MCP service output to main process
stdout/stderr, allowing [DEBUG buy] logs to be visible in docker logs.

This will help identify whether:
1. _current_position is being sent by ContextInjector but not received
2. MCP HTTP transport filters underscore-prefixed parameters
3. Parameter serialization is failing

Related to negative cash bug where final position shows -$3,049.83 instead
of +$727.92 tracked by ContextInjector.
This commit is contained in:
Bill Ballou
2025-11-07 14:56:48 -05:00
parent bee6afe531
commit 0eb5fcc940
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
agent_tools/start_mcp_services.py
View File

@@ -78,10 +78,11 @@ class MCPServiceManager:
env['PYTHONPATH'] = str(Path.cwd()) env['PYTHONPATH'] = str(Path.cwd())
# Start service process (output goes to Docker logs) # Start service process (output goes to Docker logs)
# Enable stdout/stderr for debugging (previously sent to DEVNULL)
process = subprocess.Popen( process = subprocess.Popen(
[sys.executable, str(script_path)], [sys.executable, str(script_path)],
stdout=subprocess.DEVNULL, stdout=sys.stdout, # Redirect to main process stdout
stderr=subprocess.DEVNULL, stderr=sys.stderr, # Redirect to main process stderr
cwd=Path.cwd(), # Use current working directory (/app) cwd=Path.cwd(), # Use current working directory (/app)
env=env # Pass environment with PYTHONPATH env=env # Pass environment with PYTHONPATH
) )