Add MCP_GATEWAY_LOG_DIR environment variable support for log directory configuration

[Copilot]

The MCP_GATEWAY_LOG_DIR environment variable was documented and used for validation in containerized mode but not actually respected by the CLI when determining the log directory.

Changes

• Modified internal/cmd/root.go:

  • Added getDefaultLogDir() helper that checks MCP_GATEWAY_LOG_DIR environment variable
  • Updated --log-dir flag to use environment variable as default when set
  • Changed default log directory from /tmp/gh-aw/sandbox/mcp to /tmp/gh-aw/mcp-logs

• Updated all documentation and scripts:

  • README.md: Updated default paths and examples
  • AGENTS.md: Updated reference to default log directory
  • Updated internal/config/env_validation.go fallback value
  • Updated run_containerized.sh script defaults

• Added tests:

  • Unit test for getDefaultLogDir() behavior
  • Integration test verifying end-to-end functionality with environment variable

Priority Order

Log directory is now determined by:

1. Explicit --log-dir flag (highest priority)
2. MCP_GATEWAY_LOG_DIR environment variable
3. Hardcoded default /tmp/gh-aw/mcp-logs

Usage

# Without environment variable
./awmg --help  # Shows default: /tmp/gh-aw/mcp-logs

# With environment variable
export MCP_GATEWAY_LOG_DIR=/custom/log/path
./awmg --help  # Shows default: /custom/log/path

# Explicit flag always wins
./awmg --log-dir /explicit/path  # Uses /explicit/path regardless of env var

Original prompt

Add support for specifying the log dir (MCP_GATEWAY_LOG_DIR) in dockerized mode as well (override default)

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[pelikhan]

@copilot default logs folder should be /tmp/gh-aw/mcp-logs/

[Copilot]

@copilot default logs folder should be /tmp/gh-aw/mcp-logs/

Updated the default log directory to /tmp/gh-aw/mcp-logs/ in commit 94b1091. This changes:

• The hardcoded default constant in internal/cmd/root.go
• All documentation (README.md, AGENTS.md)
• The validation and containerized scripts
• All tests continue to pass