Fix ai debug panic and wrong end point printed for byoc (#877)

* Fix AI debugger panic

* Fix endpoint parameter order

---------

Co-authored-by: Edward J <[email protected]>

Author: edwardrf

src/pkg/cli/client/byoc/aws/byoc.go

@@ -823,7 +823,7 @@ func (b *ByocAws) update(ctx context.Context, projectName, delegateDomain string
 		for _, port := range service.Ports {
 			hasIngress = hasIngress || port.Mode == compose.Mode_INGRESS
 			hasHost = hasHost || port.Mode == compose.Mode_HOST
-			si.Endpoints = append(si.Endpoints, b.getEndpoint(projectName, delegateDomain, fqn, &port))
+			si.Endpoints = append(si.Endpoints, b.getEndpoint(fqn, projectName, delegateDomain, &port))
 			mode := defangv1.Mode_INGRESS
 			if port.Mode == compose.Mode_HOST {
 				mode = defangv1.Mode_HOST

src/pkg/cli/debug.go

@@ -72,6 +72,7 @@ func Debug(ctx context.Context, l client.Loader, c client.FabricClient, p client
 		Etag:     etag,
 		Files:    files,
 		Services: failedServices,
+		Project:  project.Name,
 	}
 	err := p.Query(ctx, &req)
 	if err != nil {

src/pkg/cli/debug_test.go

@@ -2,13 +2,15 @@ package cli
 
 import (
 	"context"
+	"errors"
 	"testing"
 
+	"github.com/DefangLabs/defang/src/pkg/cli/client"
 	"github.com/DefangLabs/defang/src/pkg/cli/compose"
 	defangv1 "github.com/DefangLabs/defang/src/protos/io/defang/v1"
 )
 
-func TestDebug(t *testing.T) {
+func TestFindMathingProjectFiles(t *testing.T) {
 	project, err := compose.NewLoader(compose.WithPath("../../tests/debugproj/compose.yaml")).LoadProject(context.Background())
 	if err != nil {
 		t.Fatal(err)
@@ -33,3 +35,47 @@ func TestDebug(t *testing.T) {
 		}
 	}
 }
+
+type MustHaveProjectNameQueryProvider struct {
+	client.Provider
+}
+
+func (m MustHaveProjectNameQueryProvider) Query(ctx context.Context, req *defangv1.DebugRequest) error {
+	if req.Project == "" {
+		return errors.New("project name is missing")
+	}
+	return nil
+}
+
+type MockDebugFabricClient struct {
+	client.FabricClient
+}
+
+func (m MockDebugFabricClient) Debug(ctx context.Context, req *defangv1.DebugRequest) (*defangv1.DebugResponse, error) {
+	return &defangv1.DebugResponse{}, nil
+}
+
+func TestQueryHasProject(t *testing.T) {
+	project, err := compose.NewLoader(compose.WithPath("../../tests/debugproj/compose.yaml")).LoadProject(context.Background())
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	provider := MustHaveProjectNameQueryProvider{}
+	fabricClient := MockDebugFabricClient{}
+
+	if err := Debug(context.Background(), compose.NewLoader(), fabricClient, provider, "etag", project, []string{"service"}); err != nil {
+		t.Errorf("expected no error, got %v", err)
+	}
+
+	project.Name = ""
+
+	if err := Debug(context.Background(), compose.NewLoader(), fabricClient, provider, "etag", project, []string{"service"}); err == nil {
+		t.Error("expected error, got nil")
+	} else {
+		if err.Error() != "project name is missing" {
+			t.Errorf("expected error %q, got %q", "project name is missing", err.Error())
+		}
+	}
+
+}