Skip to content

Commit d78107b

Browse files
committed
fix(test): pass explicit args=[] to StdioTransport for stdio fixture
The fastmcp docs show `args` as optional but the installed StdioTransport.__init__ has it as a required positional. CI failed with TypeError on the first stdio test run; explicit empty list fixes it. (The unit-test failure on the same CI run — test_kill_signal_diagnostics PyMemoryView buffer flake — is unrelated to this PR; that test file was not touched and other PRs' runs on the same runner image pass it fine. Flagging here as a known unrelated flake; will not address in this PR.)
1 parent bd52122 commit d78107b

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

tests/src/e2e/conftest.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1325,7 +1325,10 @@ async def stdio_mcp_client(
13251325
"HA_MAX_RETRIES": "1",
13261326
}
13271327

1328-
transport = StdioTransport(command="ha-mcp", env=env)
1328+
# `args` is a required positional in this fastmcp version (the docs
1329+
# show it as optional, but the installed StdioTransport.__init__
1330+
# disagrees — pass an explicit empty list).
1331+
transport = StdioTransport(command="ha-mcp", args=[], env=env)
13291332
client = Client(transport)
13301333
async with client:
13311334
logger.debug("🔗 FastMCP client connected (stdio subprocess transport)")

0 commit comments

Comments
 (0)