Skip to content

Add Tool Name property for List Code Scanning Alerts tool #272

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.

Sign up for GitHub

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

Merged
merged 8 commits into from
Apr 15, 2025
Merged

Add Tool Name property for List Code Scanning Alerts tool #272

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
8b13ebb
add tool name resource for list code scanning alerts tool
Dylan-Rinker Apr 14, 2025
4ce42b2
add actual changes
Dylan-Rinker Apr 14, 2025
6adb13c
remove main
Dylan-Rinker Apr 14, 2025
6278199
Rename variable `tool_name` to `toolName`
Dylan-Rinker Apr 14, 2025
6e69a1b
Update tool_name from 'CodeQL' to 'codeql'
Dylan-Rinker Apr 14, 2025
bf74d6c
Merge branch 'main' into addToolName
Dylan-Rinker Apr 15, 2025
5640b34
Merge branch 'main' into addToolName
Dylan-Rinker Apr 15, 2025
b92f972
Merge branch 'main' into addToolName
Dylan-Rinker Apr 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Rename variable tool_name to toolName
  • Loading branch information
@Dylan-Rinker
Dylan-Rinker authored Apr 14, 2025
commit 6278199d8e68f1ea553c91ca50d7f7688d62f30f
4 changes: 2 additions & 2 deletions pkg/github/code_scanning.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ func ListCodeScanningAlerts(getClient GetClientFn, t translations.TranslationHel
if err != nil {
return mcp.NewToolResultError(err.Error()), nil
}
tool_name, err := OptionalParam[string](request, "tool_name")
toolName, err := OptionalParam[string](request, "tool_name")
if err != nil {
return mcp.NewToolResultError(err.Error()), nil
}
Expand All @@ -126,7 +126,7 @@ func ListCodeScanningAlerts(getClient GetClientFn, t translations.TranslationHel
if err != nil {
return nil, fmt.Errorf("failed to get GitHub client: %w", err)
}
alerts, resp, err := client.CodeScanning.ListAlertsForRepo(ctx, owner, repo, &github.AlertListOptions{Ref: ref, State: state, Severity: severity, ToolName: tool_name})
alerts, resp, err := client.CodeScanning.ListAlertsForRepo(ctx, owner, repo, &github.AlertListOptions{Ref: ref, State: state, Severity: severity, ToolName: toolName})
if err != nil {
return nil, fmt.Errorf("failed to list alerts: %w", err)
}
Expand Down