Fix serena-go tool reference in smoke test workflows#368
Conversation
- Remove backticks from `serena-go` tool reference in markdown files - Recompile smoke-copilot and smoke-codex workflows - Now correctly instructs agent to use "serena-go tool" instead of "serena tool" - This prevents agent from trying to run go commands directly via bash Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
lpcox
left a comment
lpcox
left a comment
There was a problem hiding this comment.
always use githubnext/gh-aw/actions/setup@v0.37.0 for "Setup Scripts"
| actions | ||
| persist-credentials: false | ||
| - name: Setup Scripts | ||
| uses: githubnext/gh-aw/actions/setup@v0.37.0 |
There was a problem hiding this comment.
Restored to githubnext/gh-aw/actions/setup@v0.37.0 in commit 4dcfc49.
| actions | ||
| persist-credentials: false | ||
| - name: Setup Scripts | ||
| uses: githubnext/gh-aw/actions/setup@v0.37.0 |
There was a problem hiding this comment.
Restored to githubnext/gh-aw/actions/setup@v0.37.0 in commit 4dcfc49.
| actions | ||
| persist-credentials: false | ||
| - name: Setup Scripts | ||
| uses: githubnext/gh-aw/actions/setup@v0.37.0 |
There was a problem hiding this comment.
Restored to githubnext/gh-aw/actions/setup@v0.37.0 in commit 4dcfc49.
| actions | ||
| persist-credentials: false | ||
| - name: Setup Scripts | ||
| uses: githubnext/gh-aw/actions/setup@v0.37.0 |
There was a problem hiding this comment.
Restored to githubnext/gh-aw/actions/setup@v0.37.0 in commit 4dcfc49.
| persist-credentials: false | ||
| - name: Setup Scripts | ||
| uses: githubnext/gh-aw/actions/setup@v0.37.0 | ||
| uses: ./actions/setup |
There was a problem hiding this comment.
Restored to githubnext/gh-aw/actions/setup@v0.37.0 in commit 4dcfc49.
| persist-credentials: false | ||
| - name: Setup Scripts | ||
| uses: githubnext/gh-aw/actions/setup@v0.37.0 | ||
| uses: ./actions/setup |
There was a problem hiding this comment.
Restored to githubnext/gh-aw/actions/setup@v0.37.0 in commit 4dcfc49.
| actions | ||
| persist-credentials: false | ||
| - name: Setup Scripts | ||
| uses: githubnext/gh-aw/actions/setup@v0.37.0 |
There was a problem hiding this comment.
Restored to githubnext/gh-aw/actions/setup@v0.37.0 in commit 4dcfc49.
- Reverted changes to actions/setup references (keep githubnext/gh-aw/actions/setup@v0.37.0) - Reverted version bumps and other compiler changes - Kept only the essential fix: changing `serena` tool to serena-go tool Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Reverted all |
Fix smoke-copilot workflow serena-go tool reference
Problem Identified ✅
The smoke-copilot action was trying to run
go versiondirectly via bash, resulting in "bash: go: command not found" error. The agent was not using the Serena MCP server tool as intended.Root Cause Analysis ✅
.mdfiles used backticks aroundserena-gotool name:`serena-go``serena-go`to`serena`during compilation-gosuffix was stripped, likely due to tool name normalization against the MCP server nameserenatool instead ofserena-gotoolgocommands directly via bashFix Applied ✅
serena-goin smoke-copilot.mdserena-goin smoke-codex.mdFiles Changed
.github/workflows/smoke-copilot.md- Line 63: Removed backticks.github/workflows/smoke-codex.md- Line 61: Removed backticks.github/workflows/smoke-copilot.lock.yml- Line 728: Now shows "serena-go tool" (minimal change)Expected Result
The workflow will now correctly instruct the agent to use the
serena-goMCP tool for Go operations, preventing the "go: command not found" error.✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.