Merge pull request #338 from githubnext/copilot/fix-serena-container-path-issue

Fix Go command PATH resolution in serena container

Author: lpcox

containers/serena-mcp-server/BUILD_NOTES.md

@@ -1,4 +1,4 @@
-# Serena MCP Server Container - Build Issues
+# Serena MCP Server Container - Build Notes
 
 ## Current Status
 
@@ -8,6 +8,15 @@ The Serena MCP server container Dockerfile has been created with support for:
 - JavaScript/TypeScript (Node.js + npm)
 - Go (golang-go package)
 
+## Recent Fixes
+
+### PATH Configuration Fix (2026-01-18)
+Fixed an issue where the `go` command was not found in the container's PATH during runtime:
+- **Problem**: Line 40 of the Dockerfile explicitly set `/usr/bin` in the PATH, which was redundant and potentially caused PATH resolution issues
+- **Solution**: Changed `ENV PATH="${GOPATH}/bin:/usr/bin:${PATH}"` to `ENV PATH="${GOPATH}/bin:${PATH}"`
+- **Impact**: Simplifies PATH configuration by relying on the base image's default PATH, which already includes `/usr/bin`
+- **Testing**: Smoke tests should now successfully execute `go version` within the container
+
 ## Build Issues Encountered
 
 During local testing, the container build encountered SSL/TLS certificate verification issues:

containers/serena-mcp-server/Dockerfile

@@ -37,7 +37,7 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
 ENV JAVA_HOME=/usr/lib/jvm/default-java
 ENV PATH="${JAVA_HOME}/bin:${PATH}"
 ENV GOPATH=/go
-ENV PATH="${GOPATH}/bin:/usr/bin:${PATH}"
+ENV PATH="${GOPATH}/bin:${PATH}"
 
 # Install Serena MCP server from GitHub using pip
 # Serena is a Python package, so we can install it directly