Skip to content

Rename cmd/mcp to cmd/workflow-mcp-server for correct go install binary naming#189

Merged
intel352 merged 2 commits intomainfrom
copilot/fix-binary-naming-mcp
Feb 27, 2026
Merged

Rename cmd/mcp to cmd/workflow-mcp-server for correct go install binary naming#189
intel352 merged 2 commits intomainfrom
copilot/fix-binary-naming-mcp

Conversation

Copy link
Contributor

Copilot AI commented Feb 27, 2026

go install derives the binary name from the last path component of the module path. cmd/mcp produces a binary named mcp — too generic and conflicts with other MCP tools.

  • Renamed cmd/mcp/cmd/workflow-mcp-server/ so go install produces workflow-mcp-server
  • Updated Makefile build-mcp target path
  • Updated docs/mcp.md build commands, architecture references, and added go install instructions
go install github.com/GoCodeAlone/workflow/cmd/workflow-mcp-server@latest

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…ry naming

Co-authored-by: intel352 <77607+intel352@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix binary naming for mcp server installation Rename cmd/mcp to cmd/workflow-mcp-server for correct go install binary naming Feb 27, 2026
@intel352 intel352 marked this pull request as ready for review February 27, 2026 05:33
Copilot AI review requested due to automatic review settings February 27, 2026 05:33
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR renames the MCP server directory from cmd/mcp to cmd/workflow-mcp-server to ensure that go install produces a properly namespaced binary name. When using go install, the binary name is derived from the last path component, so cmd/mcp would create a generic mcp binary that could conflict with other MCP tools. The new name workflow-mcp-server is more descriptive and avoids namespace collisions.

Changes:

  • Renamed directory from cmd/mcp/ to cmd/workflow-mcp-server/
  • Updated all documentation references and build commands to use the new path
  • Added go install instructions with the correct module path

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
cmd/workflow-mcp-server/main.go Updated package comment to reflect new command name
Makefile Updated build-mcp target to point to new directory path
docs/mcp.md Updated build commands, added go install instructions, and updated architecture diagram with new path

@intel352 intel352 merged commit e927d42 into main Feb 27, 2026
18 checks passed
@intel352 intel352 deleted the copilot/fix-binary-naming-mcp branch February 27, 2026 05:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants