Skip to content

Commit 0e6722e

Browse files
committed
feat: include fields in prompt only when there are fields
This essentially allows tools to "display" info to a user, even when using SDKs. Signed-off-by: Donnie Adams <[email protected]>
1 parent c8cf310 commit 0e6722e

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

pkg/prompt/prompt.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,9 +61,13 @@ func SysPrompt(ctx context.Context, envs []string, input string, _ chan<- string
6161

6262
for _, env := range envs {
6363
if url, ok := strings.CutPrefix(env, types.PromptURLEnvVar+"="); ok {
64+
var fields []string
65+
if params.Fields != "" {
66+
fields = strings.Split(params.Fields, ",")
67+
}
6468
httpPrompt := types.Prompt{
6569
Message: params.Message,
66-
Fields: strings.Split(params.Fields, ","),
70+
Fields: fields,
6771
Sensitive: params.Sensitive == "true",
6872
}
6973
return sysPromptHTTP(ctx, envs, url, httpPrompt)
@@ -76,7 +80,7 @@ func SysPrompt(ctx context.Context, envs []string, input string, _ chan<- string
7680
func sysPrompt(ctx context.Context, req types.Prompt) (_ string, err error) {
7781
defer context2.GetPauseFuncFromCtx(ctx)()()
7882

79-
if req.Message != "" && len(req.Fields) == 1 && strings.TrimSpace(req.Fields[0]) == "" {
83+
if req.Message != "" && len(req.Fields) == 0 {
8084
var errs []error
8185
_, err := fmt.Fprintln(os.Stderr, req.Message)
8286
errs = append(errs, err)

0 commit comments

Comments
 (0)