Simplify changes for now

Author: olaservo

pkg/github/instructions.go

@@ -19,28 +19,14 @@ func GenerateInstructions(enabledToolsets []string) string {
 	}
 
 	// Cross-toolset conditional instructions
-	if contains(enabledToolsets, "pull_requests") && contains(enabledToolsets, "context") {
-		instructions = append(instructions, "For team workflows: Use 'get_teams' and 'get_team_members' before assigning PR reviewers.")
-	}
-	
-	if hasAnySecurityToolset(enabledToolsets) {
-		instructions = append(instructions, "Security alert priority: secret_scanning → dependabot → code_scanning alerts.")
-	}
-	
 	if contains(enabledToolsets, "issues") && contains(enabledToolsets, "pull_requests") {
 		instructions = append(instructions, "Link issues to PRs using 'closes #123' or 'fixes #123' in PR descriptions.")
 	}
 
-	if contains(enabledToolsets, "repos") && contains(enabledToolsets, "actions") {
-		instructions = append(instructions, "For repository operations: Check workflow status before major changes using 'list_workflow_runs'.")
-	}
-	
-	baseInstruction := "The GitHub MCP Server provides GitHub API tools."
+	// Base instruction with context management
+	baseInstruction := "The GitHub MCP Server provides GitHub API tools. GitHub API responses can overflow context windows. Strategy: 1) Always prefer 'search_*' tools over 'list_*' tools when possible - search tools return filtered results, 2) Process large datasets in batches rather than all at once, 3) For summarization tasks, fetch minimal data first, then drill down into specifics, 4) When analyzing multiple items (issues, PRs, etc), process in groups of 5-10 to manage context."
 
-	// Add context management instruction for all configurations
-	contextInstruction := " GitHub API responses can overflow context windows. Strategy: 1) Always prefer 'search_*' tools over 'list_*' tools when possible - search tools return filtered results, 2) Process large datasets in batches rather than all at once, 3) For summarization tasks, fetch minimal data first, then drill down into specifics, 4) When analyzing multiple items (issues, PRs, etc), process in groups of 5-10 to manage context."
-	
-	allInstructions := []string{baseInstruction, contextInstruction}
+	allInstructions := []string{baseInstruction}
 	allInstructions = append(allInstructions, instructions...)
 
 	return strings.Join(allInstructions, " ")
@@ -68,16 +54,6 @@ func getToolsetInstructions(toolset string) string {
 	}
 }
 
-// hasAnySecurityToolset checks if any security-related toolsets are enabled
-func hasAnySecurityToolset(toolsets []string) bool {
-	securityToolsets := []string{"code_security", "secret_protection", "dependabot"}
-	for _, security := range securityToolsets {
-		if contains(toolsets, security) {
-			return true
-		}
-	}
-	return false
-}
 
 // contains checks if a slice contains a specific string
 func contains(slice []string, item string) bool {

pkg/github/instructions_test.go

@@ -47,23 +47,12 @@ func TestGenerateInstructions(t *testing.T) {
 				"rerun_failed_jobs",
 			},
 		},
-		{
-			name:            "security toolsets",
-			enabledToolsets: []string{"code_security", "secret_protection", "dependabot"},
-			expectedContains: []string{
-				"Security alert priority",
-				"secret_scanning",
-				"dependabot",
-				"code_scanning",
-			},
-		},
 		{
 			name:            "cross-toolset instructions",
 			enabledToolsets: []string{"context", "pull_requests"},
 			expectedContains: []string{
 				"get_me",
-				"get_teams",
-				"get_team_members",
+				"create_pending_pull_request_review",
 			},
 		},
 		{
@@ -141,45 +130,3 @@ func TestGetToolsetInstructions(t *testing.T) {
 	}
 }
 
-func TestHasAnySecurityToolset(t *testing.T) {
-	tests := []struct {
-		name     string
-		toolsets []string
-		expected bool
-	}{
-		{
-			name:     "no security toolsets",
-			toolsets: []string{"repos", "issues"},
-			expected: false,
-		},
-		{
-			name:     "has code_security",
-			toolsets: []string{"repos", "code_security"},
-			expected: true,
-		},
-		{
-			name:     "has secret_protection",
-			toolsets: []string{"secret_protection", "issues"},
-			expected: true,
-		},
-		{
-			name:     "has dependabot",
-			toolsets: []string{"dependabot"},
-			expected: true,
-		},
-		{
-			name:     "has all security toolsets",
-			toolsets: []string{"code_security", "secret_protection", "dependabot"},
-			expected: true,
-		},
-	}
-
-	for _, tt := range tests {
-		t.Run(tt.name, func(t *testing.T) {
-			result := hasAnySecurityToolset(tt.toolsets)
-			if result != tt.expected {
-				t.Errorf("Expected %v for toolsets %v, but got %v", tt.expected, tt.toolsets, result)
-			}
-		})
-	}
-}