Python SDK中环境变量处理问题的分析与解决

问题背景

在modelcontextprotocol/python-sdk项目中，当用户通过StdioServerParameters配置环境变量时，系统原有的默认环境变量会被完全覆盖，导致一些关键环境变量（如PATH）丢失。这个问题在使用npx启动MCP服务器时尤为明显，因为npx依赖PATH环境变量来查找可执行文件。

问题分析

在stdio.py文件中，环境变量的处理逻辑存在一个潜在问题。当用户提供了自定义环境变量时，代码会直接使用这些变量而忽略默认环境变量。这种处理方式会导致：

1. 系统PATH环境变量丢失，使得shell无法找到常用命令
2. 其他重要的默认环境变量被覆盖
3. 在特定环境下（如uv隔离环境）执行npx命令失败

技术细节

原代码中的环境变量处理逻辑如下：

process = await anyio.open_process(
    [server.command, *server.args],
    env=server.env if server.env is not None else get_default_environment(),
    stderr=sys.stderr,
)

这种三元表达式处理方式会导致当用户提供env参数时，完全忽略get_default_environment()返回的默认环境变量。

解决方案

更合理的处理方式应该是合并默认环境变量和用户自定义环境变量，其中用户自定义变量可以覆盖默认值。修改后的代码如下：

process = await anyio.open_process(
    [server.command, *server.args],
    env={
        **get_default_environment(),
        **(server.env or {})
    },
    stderr=sys.stderr,
)

这种解决方案具有以下优点：

1. 保留了所有默认环境变量
2. 允许用户覆盖特定环境变量
3. 保持了向后兼容性
4. 解决了npx等命令找不到的问题

影响范围

该问题主要影响以下场景：

1. 在隔离环境中运行MCP服务器
2. 使用npx等依赖PATH环境变量的命令
3. 需要同时使用默认环境变量和自定义环境变量的情况

最佳实践

对于开发者使用该SDK时，建议：

1. 明确了解环境变量的继承和覆盖机制
2. 只覆盖必要的环境变量，而不是提供完整的环境变量集合
3. 在隔离环境中特别注意PATH等关键环境变量的设置

该问题已在1.5.0版本中修复，开发者可以升级到最新版本来获得修复。