fix(GB-63): Add min_pi_count parameter to pi_consensus_top tool

Default changed from 3 to 2 to match reality (we have max 2 PIs per symbol).
Allows querying consensus positions held by 2+ PIs instead of requiring 3+.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
Christian Gick
2026-01-20 18:47:07 +02:00
parent 5ce18bb435
commit 72bc14ed2c

View File

@@ -563,6 +563,7 @@ async def list_tools() -> list[Tool]:
"type": "object",
"properties": {
"limit": {"type": "number", "default": 10, "description": "Max results (1-100)"},
"min_pi_count": {"type": "number", "default": 2, "description": "Minimum PIs required (default: 2)"},
},
"required": [],
},
@@ -823,7 +824,8 @@ async def call_tool(name: str, arguments: dict[str, Any]) -> list[TextContent]:
result = await api_get(f"/api/pi-intelligence/consensus/{symbol}")
case "pi_consensus_top":
limit = arguments.get("limit", 10)
result = await api_get(f"/api/pi-intelligence/consensus/top?limit={limit}")
min_pi_count = arguments.get("min_pi_count", 2)
result = await api_get(f"/api/pi-intelligence/consensus/top?limit={limit}&min_pi_count={min_pi_count}")
case "pi_scan_trigger":
usernames = arguments.get("usernames")
endpoint = "/api/pi-intelligence/scan/trigger"