Merge pull request modelcontextprotocol#28 from jkoelker/jk/technicals

fix(tools): fix get_advanced_price_history

Author: jkoelker

src/schwab_mcp/tools/_protocols.py

@@ -18,7 +18,7 @@ class PriceHistoryNamespace(Protocol):
 class PriceHistoryClient(Protocol):
     PriceHistory: PriceHistoryNamespace
 
-    def get_advanced_price_history(
+    def get_price_history(
         self, symbol: str, **kwargs: Any
     ) -> Awaitable[Any]: ...
 

src/schwab_mcp/tools/history.py

@@ -81,7 +81,7 @@ async def get_advanced_price_history(
         frequency = int(frequency)
 
     return await call(
-        client.get_advanced_price_history,
+        client.get_price_history,
         symbol,
         period_type=period_type_enum,
         period=period_enum,

tests/test_history.py

@@ -22,7 +22,7 @@ class DummyHistoryClient:
         FrequencyType=Enum("FrequencyType", "MINUTE DAILY WEEKLY MONTHLY"),
     )
 
-    async def get_advanced_price_history(self, *args, **kwargs):
+    async def get_price_history(self, *args, **kwargs):
         return None
 
     async def get_price_history_every_minute(self, *args, **kwargs):
@@ -74,7 +74,7 @@ async def fake_call(func, *args, **kwargs):
     )
 
     assert result == "ok"
-    assert captured["func"] == client.get_advanced_price_history
+    assert captured["func"] == client.get_price_history
 
     args = captured["args"]
     assert isinstance(args, tuple)