-
Notifications
You must be signed in to change notification settings - Fork 28
Add update tool to MCP server #4110
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
pkg/cli/mcp_server.go
Outdated
| Engine string `json:"engine,omitempty" jsonschema:"Override AI engine (claude, codex, copilot, custom)"` | ||
| PR bool `json:"pr,omitempty" jsonschema:"Create a pull request with the workflow changes"` | ||
| Dir string `json:"dir,omitempty" jsonschema:"Relative directory containing workflows (default: .github/workflows)"` | ||
| NoStopAfter bool `json:"no_stop_after,omitempty" jsonschema:"Remove any stop-after field from the updated workflow"` | ||
| StopAfter string `json:"stop_after,omitempty" jsonschema:"Override stop-after value in the updated workflow (e.g., '+48h', '2025-12-31 23:59:59')"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot remove support for those flags
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed support for engine, pr, dir, no_stop_after, and stop_after flags. The update tool now only supports workflows, major, and force parameters. (4a014fd)
…te tool Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
Agentic Changeset Generator triggered by this pull request. |
There was a problem hiding this 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 adds an "update" tool to the MCP server that exposes the gh aw update command through the Model Context Protocol. The tool enables AI agents to update workflows from their source repositories and check for gh-aw extension updates.
Key Changes:
- Added
updatetool to MCP server with essential flags (workflows, major, force) - Updated MCP server help text to document the new tool
- Added comprehensive test coverage for the new tool's schema and functionality
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
pkg/cli/mcp_server.go |
Implements the update tool with handler that wraps the CLI command, following existing tool patterns (status, compile, logs, audit, mcp-inspect, add) |
pkg/cli/mcp_server_test.go |
Updates test expectations to include 7 tools (was 6) and adds schema validation test for the update tool |
.changeset/patch-add-update-tool-mcp.md |
Documents the change for release notes as a patch-level update |
.github/workflows/docs-noob-tester.lock.yml |
Pins Playwright MCP version from @latest to @1.56.1 (unrelated workflow recompilation) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Exposes the
gh aw updatecommand through the MCP protocol with essential update flags.Changes
pkg/cli/mcp_server.go: Addedupdatetool with handler that wraps the CLI commandworkflows,major,forcepkg/cli/mcp_server_test.go: Updated test expectationsTestMCPServer_ListToolsto expect 7 tools (was 6)TestMCPServer_UpdateToolSchemato validate tool schema and descriptionMCP Tool Arguments
{ "name": "update", "arguments": { "workflows": ["workflow-id"], "major": true, "force": true } }Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.