feat(mcp-servers): Allow configurable directories for filesystem server

[yikunliu-aws]

Summary

Makes the forwarded filesystem MCP server's allowed directories configurable instead of hardcoded.

Changes

mcp_servers/filesystem_server.py:

• Reads allowed directories from sys.argv[1:] at startup
• Falls back to C:\Users\Public\Documents if no args provided (backward-compatible)
• list_directory() defaults to the first allowed dir instead of a hardcoded path
• Multiple directories supported — each passed as a separate CLI arg

scripts/setup_mcp_redirection.sh:

• New flag: --allowed-dirs "C:\Users\Public\Documents,D:\Data" (comma-separated)
• Builds the manifest args array dynamically from the configured dirs
• Default unchanged: C:\Users\Public\Documents

How it works

The MCP server manifest on the image controls what args are passed to the server at launch:

{"mcpServers": {"filesystem": {
  "command": "C:\\Program Files\\Python312\\python.exe",
  "args": ["C:\\McpServers\\filesystem_server.py", "C:\\Users\\Public\\Documents", "D:\\Data"]
}}}

Users who build their own images can also edit C:\ProgramData\NICE\dcv\mcp_server_redirection_config.json directly — no setup script required.

Testing

• python -m py_compile passes
• bash -n on setup script passes
• ./scripts/ci_local.sh passes
• Not tested against a live fleet (image rebuild required for the manifest change to take effect)

[morgnza]

Update to forward slashes for a cleaner view - "C:/McpServers/filesystem_server.py"

[yikunliu-aws]

Addressed in 1d13b34 — switched all manifest paths to forward slashes (Python handles them natively on Windows). Also converts any backslashes in user-supplied --allowed-dirs to forward slashes for consistency. Much more readable now.