fix: openapi revamp: return error to LLM if args are invalid JSON (#843)

g-linville · tylerslaton · web-flow · commit c441cb4cf785 · 2024-09-06T15:18:56.000-04:00
Signed-off-by: Grant Linville &lt;grant@acorn.io&gt;
Co-authored-by: Tyler Slaton &lt;54378333+tylerslaton@users.noreply.github.com&gt;
diff --git a/pkg/openapi/run.go b/pkg/openapi/run.go
@@ -38,7 +38,8 @@ func Run(operationID, defaultHost, args string, t *openapi3.T, envs []string) (s
 	// Validate args against the schema.
 	validationResult, err := gojsonschema.Validate(gojsonschema.NewStringLoader(schemaJSON), gojsonschema.NewStringLoader(args))
 	if err != nil {
-		return "", false, err
+		// We don't return an error here because we want the LLM to be able to maintain control and try again.
+		return fmt.Sprintf("ERROR: failed to validate arguments. Make sure your arguments are valid JSON. %v", err), false, nil
 	}
 
 	if !validationResult.Valid() {

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,8 @@ func Run(operationID, defaultHost, args string, t *openapi3.T, envs []string) (s`
`38`	`38`	`// Validate args against the schema.`
`39`	`39`	`validationResult, err := gojsonschema.Validate(gojsonschema.NewStringLoader(schemaJSON), gojsonschema.NewStringLoader(args))`
`40`	`40`	`if err != nil {`
`41`		`- return "", false, err`
	`41`	`+ // We don't return an error here because we want the LLM to be able to maintain control and try again.`
	`42`	`+ return fmt.Sprintf("ERROR: failed to validate arguments. Make sure your arguments are valid JSON. %v", err), false, nil`
`42`	`43`	`}`
`43`	`44`
`44`	`45`	`if !validationResult.Valid() {`