remove unused SecurityAlertWorkflowPrompt and RepositorySetupWorkflowPrompt

mattdholloway · mattdholloway · commit 75bfae4a5fdb · 2025-07-22T10:08:23.000+01:00
diff --git a/pkg/github/tools.go b/pkg/github/tools.go
@@ -46,9 +46,7 @@ func DefaultToolsetGroup(readOnly bool, getClient GetClientFn, getGQLClient GetG
 			toolsets.NewServerResourceTemplate(GetRepositoryResourceCommitContent(getClient, getRawClient, t)),
 			toolsets.NewServerResourceTemplate(GetRepositoryResourceTagContent(getClient, getRawClient, t)),
 			toolsets.NewServerResourceTemplate(GetRepositoryResourcePrContent(getClient, getRawClient, t)),
-		).AddPrompts(
-		toolsets.NewServerPrompt(RepositorySetupWorkflowPrompt(t)),
-	)
+		)
 	issues := toolsets.NewToolset("issues", "GitHub Issues related tools").
 		AddReadTools(
 			toolsets.NewServerTool(GetIssue(getClient, t)),
@@ -108,9 +106,7 @@ func DefaultToolsetGroup(readOnly bool, getClient GetClientFn, getGQLClient GetG
 		AddReadTools(
 			toolsets.NewServerTool(GetCodeScanningAlert(getClient, t)),
 			toolsets.NewServerTool(ListCodeScanningAlerts(getClient, t)),
-		).AddPrompts(
-		toolsets.NewServerPrompt(SecurityAlertWorkflowPrompt(t)),
-	)
+		)
 	secretProtection := toolsets.NewToolset("secret_protection", "Secret protection related tools, such as GitHub Secret Scanning").
 		AddReadTools(
 			toolsets.NewServerTool(GetSecretScanningAlert(getClient, t)),
@@ -120,9 +116,7 @@ func DefaultToolsetGroup(readOnly bool, getClient GetClientFn, getGQLClient GetG
 		AddReadTools(
 			toolsets.NewServerTool(GetDependabotAlert(getClient, t)),
 			toolsets.NewServerTool(ListDependabotAlerts(getClient, t)),
-		).AddPrompts(
-		toolsets.NewServerPrompt(SecurityAlertWorkflowPrompt(t)),
-	)
+		)
 
 	notifications := toolsets.NewToolset("notifications", "GitHub Notifications related tools").
 		AddReadTools(
diff --git a/pkg/github/workflow_prompts.go b/pkg/github/workflow_prompts.go
@@ -135,102 +135,3 @@ func IssueInvestigationWorkflowPrompt(t translations.TranslationHelperFunc) (too
 			}, nil
 		}
 }
-
-// SecurityAlertWorkflowPrompt provides guided workflow for managing security alerts
-func SecurityAlertWorkflowPrompt(t translations.TranslationHelperFunc) (tool mcp.Prompt, handler server.PromptHandlerFunc) {
-	return mcp.NewPrompt("SecurityAlertWorkflow",
-			mcp.WithPromptDescription(t("PROMPT_SECURITY_ALERT_WORKFLOW_DESCRIPTION", "Convert security alerts into trackable issues and assign to appropriate resources")),
-			mcp.WithArgument("owner", mcp.ArgumentDescription("Repository owner"), mcp.RequiredArgument()),
-			mcp.WithArgument("repo", mcp.ArgumentDescription("Repository name"), mcp.RequiredArgument()),
-			mcp.WithArgument("alertType", mcp.ArgumentDescription("Type of alerts to process: dependabot, code_scanning, or secret_scanning")),
-		), func(_ context.Context, request mcp.GetPromptRequest) (*mcp.GetPromptResult, error) {
-			owner := request.Params.Arguments["owner"]
-			repo := request.Params.Arguments["repo"]
-			alertType := "dependabot"
-			if t, exists := request.Params.Arguments["alertType"]; exists {
-				alertType = fmt.Sprintf("%v", t)
-			}
-
-			messages := []mcp.PromptMessage{
-				{
-					Role:    "system",
-					Content: mcp.NewTextContent("You are a security management assistant helping to process security alerts and convert them into actionable work items. You should examine alerts, create tracking issues for important ones, and assign appropriate work to Copilot where suitable."),
-				},
-				{
-					Role:    "user",
-					Content: mcp.NewTextContent(fmt.Sprintf("I need to process %s security alerts for %s/%s. Please help me convert critical alerts into trackable issues and assign work appropriately.", alertType, owner, repo)),
-				},
-				{
-					Role:    "assistant",
-					Content: mcp.NewTextContent(fmt.Sprintf("I'll help you process %s alerts for %s/%s systematically. Let me examine the alerts, identify critical ones, and create appropriate tracking issues.", alertType, owner, repo)),
-				},
-				{
-					Role:    "user",
-					Content: mcp.NewTextContent("Great! Please prioritize by severity and create issues for critical/high priority alerts. For straightforward dependency updates, consider assigning to Copilot. For complex security issues, create detailed issues for human review."),
-				},
-				{
-					Role:    "assistant",
-					Content: mcp.NewTextContent("Perfect approach! I'll:\n\n1. List and examine the security alerts\n2. Prioritize by severity (critical/high first)\n3. Create detailed tracking issues\n4. Assign simple dependency updates to Copilot\n5. Flag complex security issues for human review\n\nLet me start by examining the alerts."),
-				},
-			}
-			return &mcp.GetPromptResult{
-				Messages: messages,
-			}, nil
-		}
-}
-
-// RepositorySetupWorkflowPrompt provides guided workflow for setting up new repositories
-func RepositorySetupWorkflowPrompt(t translations.TranslationHelperFunc) (tool mcp.Prompt, handler server.PromptHandlerFunc) {
-	return mcp.NewPrompt("RepositorySetupWorkflow",
-			mcp.WithPromptDescription(t("PROMPT_REPOSITORY_SETUP_WORKFLOW_DESCRIPTION", "Guide through setting up a new repository with initial content and structure")),
-			mcp.WithArgument("repoName", mcp.ArgumentDescription("Name for the new repository"), mcp.RequiredArgument()),
-			mcp.WithArgument("description", mcp.ArgumentDescription("Repository description")),
-			mcp.WithArgument("private", mcp.ArgumentDescription("Whether repository should be private (true/false)")),
-		), func(_ context.Context, request mcp.GetPromptRequest) (*mcp.GetPromptResult, error) {
-			repoName := request.Params.Arguments["repoName"]
-			description := ""
-			if d, exists := request.Params.Arguments["description"]; exists {
-				description = fmt.Sprintf("%v", d)
-			}
-			private := "false"
-			if p, exists := request.Params.Arguments["private"]; exists {
-				private = fmt.Sprintf("%v", p)
-			}
-
-			messages := []mcp.PromptMessage{
-				{
-					Role:    "system",
-					Content: mcp.NewTextContent("You are a repository setup assistant helping to create and configure new GitHub repositories with best practices. You should guide through creating the repository, setting up initial structure, and establishing good development workflows."),
-				},
-				{
-					Role: "user",
-					Content: mcp.NewTextContent(fmt.Sprintf("I want to create a new repository called '%s'%s%s. Please help me set it up with proper initial structure.", repoName, func() string {
-						if description != "" {
-							return fmt.Sprintf(" with description '%s'", description)
-						}
-						return ""
-					}(), func() string {
-						if private == "true" {
-							return " (private repository)"
-						}
-						return ""
-					}())),
-				},
-				{
-					Role:    "assistant",
-					Content: mcp.NewTextContent(fmt.Sprintf("I'll help you create and set up the '%s' repository with a proper initial structure. Let me guide you through the process step by step.", repoName)),
-				},
-				{
-					Role:    "user",
-					Content: mcp.NewTextContent("Perfect! Please:\n1. Create the repository\n2. Set up a development branch\n3. Add essential files (README, .gitignore, etc.)\n4. Create an initial pull request to establish the workflow\n\nLet me know what type of project this is so you can suggest appropriate templates."),
-				},
-				{
-					Role:    "assistant",
-					Content: mcp.NewTextContent("Excellent plan! I'll create a well-structured repository with:\n\n Repository creation\n Development branch setup\n Essential files (README, .gitignore, etc.)\n Initial PR workflow\n\nWhat type of project is this? (e.g., JavaScript/Node.js, Python, Go, documentation, etc.) This will help me suggest the right templates and structure."),
-				},
-			}
-			return &mcp.GetPromptResult{
-				Messages: messages,
-			}, nil
-		}
-}
