chore(ci): align API and tests with repository conventions

Author: paulbalaji

pkg/api/ci.go

@@ -30,16 +30,31 @@ func ciRequest[T any](ctx context.Context, token, orgID, path string, payload in
 	}
 
 	req.Header.Add("Content-Type", "application/json")
-	req.Header.Add("Authorization", fmt.Sprintf("Bearer %s", token))
-	req.Header.Add("x-depot-org", orgID)
+	if token != "" {
+		req.Header.Add("Authorization", fmt.Sprintf("Bearer %s", token))
+	}
+	if orgID != "" {
+		req.Header.Add("x-depot-org", orgID)
+	}
 	req.Header.Add("User-Agent", Agent())
+	req.Header.Add("Depot-User-Agent", Agent())
 
 	resp, err := client.Do(req)
 	if err != nil {
 		return nil, err
 	}
 	defer resp.Body.Close()
 
+	infoMessage := resp.Header.Get("X-Depot-Info-Message")
+	if infoMessage != "" {
+		fmt.Println(infoStyle.Render(infoMessage))
+	}
+
+	warnMessage := resp.Header.Get("X-Depot-Warn-Message")
+	if warnMessage != "" {
+		fmt.Println(warnStyle.Render(warnMessage))
+	}
+
 	body, err := io.ReadAll(resp.Body)
 	if err != nil {
 		return nil, err

pkg/ci/migrate/copy_test.go

@@ -3,6 +3,7 @@ package migrate
 import (
 	"os"
 	"path/filepath"
+	"strings"
 	"testing"
 )
 
@@ -124,7 +125,7 @@ func TestCopyMissingGitHub(t *testing.T) {
 		}
 	}
 	// Check that ".github" is mentioned in error
-	if errMsg := err.Error(); !contains(errMsg, ".github") {
+	if errMsg := err.Error(); !strings.Contains(errMsg, ".github") {
 		t.Errorf("error message should mention '.github', got: %s", errMsg)
 	}
 }
@@ -247,7 +248,7 @@ func TestCopySkipSymlinks(t *testing.T) {
 	// Verify warning mentions symlink
 	hasSymlinkWarning := false
 	for _, w := range result.Warnings {
-		if contains(w, "symlink") {
+		if strings.Contains(w, "symlink") {
 			hasSymlinkWarning = true
 			break
 		}
@@ -323,7 +324,7 @@ func TestCopyMissingSubDir(t *testing.T) {
 
 	hasActionsWarning := false
 	for _, w := range result.Warnings {
-		if contains(w, "actions") {
+		if strings.Contains(w, "actions") {
 			hasActionsWarning = true
 			break
 		}
@@ -337,13 +338,3 @@ func TestCopyMissingSubDir(t *testing.T) {
 		t.Errorf("workflows/ci.yml not found: %v", err)
 	}
 }
-
-// Helper function to check if string contains substring
-func contains(s, substr string) bool {
-	for i := 0; i <= len(s)-len(substr); i++ {
-		if s[i:i+len(substr)] == substr {
-			return true
-		}
-	}
-	return false
-}