Skip to content

Commit 57d9a28

Browse files
Copilotlpcox
andcommitted
Add Go runtime to serena-mcp-server container with cleanup
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
1 parent db064e6 commit 57d9a28

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)