Skip to content

Prevent inputs param from being stripped from actions_run_trigger tool schema #2417

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
kerobbi merged 3 commits into from
May 11, 2026
+36 −5
Merged

Prevent inputs param from being stripped from actions_run_trigger tool schema #2417

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
039808a
add empty properties to inputs param
kerobbi May 1, 2026
b966807
add test cases for valid and invalid inputs
kerobbi May 1, 2026
e472532
Merge branch 'main' into kerobbi/fix-inputs-schema
kerobbi May 11, 2026
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
1 change: 1 addition & 0 deletions pkg/github/__toolsnaps__/actions_run_trigger.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
"properties": {
"inputs": {
"description": "Inputs the workflow accepts. Only used for 'run_workflow' method.",
"properties": {},
"type": "object"
},
"method": {
Expand Down
9 changes: 4 additions & 5 deletions pkg/github/actions.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -544,6 +544,7 @@ func ActionsRunTrigger(t translations.TranslationHelperFunc) inventory.ServerToo
"inputs": {
Type: "object",
Description: "Inputs the workflow accepts. Only used for 'run_workflow' method.",
Properties: map[string]*jsonschema.Schema{},
},
"run_id": {
Type: "number",
Expand Down Expand Up @@ -574,11 +575,9 @@ func ActionsRunTrigger(t translations.TranslationHelperFunc) inventory.ServerToo
runID, _ := OptionalIntParam(args, "run_id")

// Get optional inputs parameter
var inputs map[string]any
if requestInputs, ok := args["inputs"]; ok {
if inputsMap, ok := requestInputs.(map[string]any); ok {
inputs = inputsMap
}
inputs, err := OptionalParam[map[string]any](args, "inputs")
if err != nil {
return utils.NewToolResultError(err.Error()), nil, nil
}
Comment thread
kerobbi marked this conversation as resolved.

// Validate required parameters based on action type
Expand Down
Loading