Skip to content

Commit 6779ea2

Browse files
authored
Add Go runtime to serena-mcp-server container (#703)
Container had `gopls` (Go LSP) but was missing the Go runtime itself. Serena needs both for Go code analysis and compilation. ## Changes - **Dockerfile**: Added `golang-go` to apt-get install, preserving existing cleanup steps - **BUILD_NOTES.md**: Documented the addition Result: Go 1.24.4 runtime now available alongside gopls v0.17.1. Container size 912MB with proper apt cache cleanup (16K remaining). <!-- START COPILOT CODING AGENT TIPS --> --- 💬 We'd love your input! Share your thoughts on Copilot coding agent in our [2 minute survey](https://gh.io/copilot-coding-agent-survey).
2 parents 125f22a + 57d9a28 commit 6779ea2

2 files changed

Lines changed: 10 additions & 0 deletions

File tree

containers/serena-mcp-server/BUILD_NOTES.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,14 @@ The Serena MCP server container Dockerfile has been created with support for:
1010

1111
## Recent Fixes
1212

13+
### Go Runtime Re-added (2026-02-05)
14+
Re-added Go runtime to the container to support Go code analysis:
15+
- **Problem**: The Dockerfile only installed `gopls` (Go LSP) but not the Go runtime itself
16+
- **Solution**: Added `golang-go` package back to apt-get install with proper cleanup
17+
- **Impact**: Container now includes both Go runtime and gopls for complete Go support
18+
- **Testing**: The `go version` command now works, and Go code can be compiled/executed
19+
- **Image Size**: Cleanup steps ensure minimal impact on image size
20+
1321
### PATH Configuration Fix (2026-01-18)
1422
Fixed an issue where the `go` command was not found in the container's PATH during runtime:
1523
- **Problem**: Line 40 of the Dockerfile explicitly set `/usr/bin` in the PATH, which was redundant and potentially caused PATH resolution issues

containers/serena-mcp-server/Dockerfile

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,13 @@ WORKDIR /app
2424
# - git: required for some Serena operations
2525
# - nodejs/npm: for TypeScript language server
2626
# - ca-certificates: for HTTPS
27+
# - golang-go: Go runtime for Go code analysis
2728
RUN apt-get update && apt-get install -y --no-install-recommends \
2829
git \
2930
nodejs \
3031
npm \
3132
ca-certificates \
33+
golang-go \
3234
&& rm -rf /var/lib/apt/lists/* \
3335
&& apt-get clean
3436

0 commit comments

Comments
 (0)