Empty array

Signed-off-by: David Gageot <david.gageot@docker.com>

Author: dgageot

pkg/server/server.go

@@ -98,7 +98,7 @@ func (s *Server) Serve(ctx context.Context, ln net.Listener) error {
 // API handlers
 
 func (s *Server) getAgents(c echo.Context) error {
-	var agents []api.Agent
+	agents := []api.Agent{}
 	for k, agentSource := range s.agentSources {
 		slog.Debug("API source", "source", agentSource.Name())
 

pkg/server/server_test.go

@@ -21,7 +21,6 @@ import (
 )
 
 func TestServer_ListAgents(t *testing.T) {
-	// t.Parallel()
 	t.Setenv("OPENAI_API_KEY", "dummy")
 	t.Setenv("ANTHROPIC_API_KEY", "dummy")
 
@@ -48,6 +47,14 @@ func TestServer_ListAgents(t *testing.T) {
 	assert.False(t, agents[2].Multi)
 }
 
+func TestServer_EmptyList(t *testing.T) {
+	ctx := t.Context()
+	lnPath := startServer(t, ctx, prepareAgentsDir(t))
+
+	buf := httpGET(t, ctx, lnPath, "/api/agents")
+	assert.Equal(t, "[]\n", string(buf)) // We don't want null, but an empty array
+}
+
 func TestServer_ListSessions(t *testing.T) {
 	t.Parallel()