Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add initial support for multi-tool workflows#685

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
mattdholloway merged 13 commits intomainfrommulti-tool-workflows
Jul 30, 2025

Conversation

@mattdholloway
Copy link
Contributor

@mattdhollowaymattdholloway commentedJul 16, 2025
edited
Loading

@mattdhollowaymattdholloway self-assigned thisJul 16, 2025
@mattdhollowaymattdholloway marked this pull request as ready for reviewJuly 25, 2025 12:02
CopilotAI review requested due to automatic review settingsJuly 25, 2025 12:02
@mattdhollowaymattdholloway requested a review froma team as acode ownerJuly 25, 2025 12:02
Copy link
Contributor

CopilotAI 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 adds support for multi-tool workflows by introducing four new workflow prompts that provide guided interactions for common GitHub development tasks. The workflows combine multiple existing GitHub tools into structured, conversational flows for more comprehensive automation.

  • Adds four new workflow prompts for PR review, notification triage, issue investigation, and issue-to-fix workflows
  • Integrates workflow prompts into existing toolsets (issues, pull_requests, notifications)
  • Provides guided multi-step processes that leverage existing GitHub API tools

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

FileDescription
pkg/github/workflow_prompts.goNew file containing four workflow prompt implementations with guided conversation flows
pkg/github/tools.goIntegration of new workflow prompts into existing toolsets for issues, pull requests, and notifications
Comments suppressed due to low confidence (4)

pkg/github/workflow_prompts.go:13

  • The return parameter name 'tool' should be 'prompt' since it returns an mcp.Prompt, not a tool. This naming is inconsistent and misleading.
func PullRequestReviewWorkflowPrompt(t translations.TranslationHelperFunc) (tool mcp.Prompt, handler server.PromptHandlerFunc) {

pkg/github/workflow_prompts.go:53

  • The return parameter name 'tool' should be 'prompt' since it returns an mcp.Prompt, not a tool. This naming is inconsistent and misleading.
func NotificationTriageWorkflowPrompt(t translations.TranslationHelperFunc) (tool mcp.Prompt, handler server.PromptHandlerFunc) {

pkg/github/workflow_prompts.go:92

  • The return parameter name 'tool' should be 'prompt' since it returns an mcp.Prompt, not a tool. This naming is inconsistent and misleading.
func IssueInvestigationWorkflowPrompt(t translations.TranslationHelperFunc) (tool mcp.Prompt, handler server.PromptHandlerFunc) {

pkg/github/workflow_prompts.go:140

  • The return parameter name 'tool' should be 'prompt' since it returns an mcp.Prompt, not a tool. This naming is inconsistent and misleading.
func IssueToFixWorkflowPrompt(t translations.TranslationHelperFunc) (tool mcp.Prompt, handler server.PromptHandlerFunc) {

@mattdhollowaymattdholloway changed the titleAdd support for multi-tool workflowsAdd initial support for multi-tool workflowsJul 30, 2025
@mattdhollowaymattdholloway merged commit771d7b4 intomainJul 30, 2025
16 checks passed
@mattdhollowaymattdholloway deleted the multi-tool-workflows branchJuly 30, 2025 10:40
nickytonline pushed a commit to nickytonline/github-mcp-http that referenced this pull requestOct 4, 2025
* initial workflows* fixing tabs* remove unused SecurityAlertWorkflowPrompt and RepositorySetupWorkflowPrompt* add workflow prompt for creating issue and assigning to copilot* Update pkg/github/workflow_prompts.goCo-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>* remove notif triage* remove code inv workflow tool* rm another---------Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
SamMorrowDrums added a commit that referenced this pull requestDec 17, 2025
Add CachedInventory to build tool/resource/prompt definitions once atstartup rather than per-request. This is particularly useful for theremote server pattern where a new server instance is created per request.Key features:- InitInventoryCache(t) initializes the cache once at startup- CachedInventoryBuilder() returns a builder with pre-cached definitions- Per-request configuration (read-only, toolsets, feature flags) still works- Thread-safe via sync.Once- Backward compatible: NewInventory(t) still works without cachingThis addresses the performance concern raised in go-sdk PR#685 at ahigher level by caching the entire []ServerTool slice rather thanindividual schemas.Related:modelcontextprotocol/go-sdk#685
SamMorrowDrums added a commit that referenced this pull requestDec 17, 2025
Add CachedInventory to build tool/resource/prompt definitions once atstartup rather than per-request. This is particularly useful for theremote server pattern where a new server instance is created per request.Key features:- InitInventoryCache(t) initializes the cache once at startup- InitInventoryCacheWithExtras(t, tools, resources, prompts) allows  injecting additional items (e.g., remote-only Copilot tools)- CachedInventoryBuilder() returns a builder with pre-cached definitions- Per-request configuration (read-only, toolsets, feature flags) still works- Thread-safe via sync.Once- Backward compatible: NewInventory(t) still works without cachingThis addresses the performance concern raised in go-sdk PR#685 at ahigher level by caching the entire []ServerTool slice rather thanindividual schemas.Related:modelcontextprotocol/go-sdk#685
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

Copilot code reviewCopilotCopilot left review comments

@almaleksiaalmaleksiaalmaleksia approved these changes

Assignees

@mattdhollowaymattdholloway

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mattdholloway@almaleksia

[8]ページ先頭

©2009-2025 Movatter.jp