gptscript-ai/clio

`sdkserver.writeError` errors seen in TUI when chatting withe agents.

Closed this issue · 3 comments

sangee2004 commented

clio version v0.1.1

Steps to reproduce the problem

  1. Install clio
  2. Chat with K8s agent by asking it to list pods and then scale on of the existing deployments .
  3. Notice that the following error message appears in TUI
http: superfluous response.WriteHeader call from github.com/gptscript-ai/gptscript/pkg/sdkserver.writeError (run.go:141)

Note - This does not prevent any responses being presented to the user.

> scale nginx to 32024/07/23 10:56:59 http: superfluous response.WriteHeader call from github.com/gptscript-ai/gptscript/pkg/sdkserver.writeError (run.go:141)
> scale nginx to 3
> scale nginx to 3

    ┌───────────────────────────────────────────────────────────────────────────┐
    │ Call Arguments:                                                           │
    │                                                                           │
    │ exec {"command":"kubectl scale deployment nginx-deployment --replicas=3"} │
    └───────────────────────────────────────────────────────────────────────────┘
                                                                                 
  Running  kubectl scale deployment nginx-deployment --replicas=3                                                                                                                              

    ┌─────────────────────────────────────────┐
    │ deployment.apps/nginx-deployment scaled │
    └─────────────────────────────────────────┘
                                               
  The  nginx-deployment  has been successfully scaled to 3 replicas.                                                                                                                           
                                                                                                                                                                                               
  Would you like to verify the status of the pods or perform any other actions?                                                                                                                

@K8s>
thedadams commented

The issue is addressed here: gptscript-ai/gptscript#657

This will have to be brought into clio after it is merged.

thedadams commented

@sangee2004 The change has been merged into clio and is ready for testing.

sangee2004 commented

Tested with clio version v0.1.2

This issue is not seen anymore.