fix newlines

Signed-off-by: Mmadu Manasseh <manasseh.mmadu@zapier.com>

Author: MeNsaaH

localdev/kubechecks/values.yaml

@@ -31,10 +31,7 @@ configMap:
 deployment:
   annotations:
     reloader.stakater.com/auto: "true" 
-  resources:
-    limits:
-      memory: 4Gi
-
+  
   image:
     pullPolicy: IfNotPresent
     name: "kubechecks"

pkg/msg/message.go

@@ -12,7 +12,6 @@ import (
 	"golang.org/x/exp/constraints"
 	"golang.org/x/exp/slices"
 
-	"github.com/rs/zerolog/log"
 	"github.com/zapier/kubechecks/pkg"
 )
 
@@ -266,7 +265,7 @@ func (m *Message) BuildComment(
 		} else {
 			appHeader += fmt.Sprintf("## ArgoCD Application Checks: `%s` %s\n", appName, m.vcs.ToEmoji(appState))
 		}
-		appHeader += "</summary>\n\n"
+		appHeader += "\n</summary>\n\n"
 
 		// Only split if adding the app header would exceed the chunk limit
 		if contentLength+len(appHeader) > maxContentLength {
@@ -299,7 +298,7 @@ func (m *Message) BuildComment(
 					summary = fmt.Sprintf("%s %s %s", check.Summary, check.State.BareString(), m.vcs.ToEmoji(check.State))
 				}
 			}
-			summaryHeader := fmt.Sprintf("<details>\n<summary>%s</summary>\n", summary)
+			summaryHeader := fmt.Sprintf("<details>\n\n<summary>%s</summary>\n\n", summary)
 
 			// Only split if adding the summary would exceed the chunk limit (not if it equals)
 			if contentLength+len(summaryHeader) > maxContentLength {
@@ -372,9 +371,6 @@ func (m *Message) BuildComment(
 		comments = append(comments, sb.String())
 	}
 
-	for _, comment := range comments {
-		log.Debug().Msgf("Comment length: %d", len(comment))
-	}
 	return comments
 }
 

pkg/msg/message_test.go

@@ -41,10 +41,13 @@ func TestBuildComment(t *testing.T) {
 <summary>
 
 ## ArgoCD Application Checks: ` + "`myapp`" + ` :test:
+
 </summary>
 
 <details>
+
 <summary>this failed bigly Error :test:</summary>
+
 should add some important details here
 </details>
 </details>
@@ -1324,7 +1327,7 @@ func TestBuildComment_ContentLengthLimits(t *testing.T) {
 
 	// Test case 5: Edge case with very small maxCommentLength
 	// This tests the minimum viable comment size
-	smallMaxCommentLength := len(continuedHeader) + 50 // Just enough for continuedHeader + minimal content
+	smallMaxCommentLength := len(continuedHeader) + 100 // Increased buffer to account for message structure overhead
 	appResults5 := map[string]*AppResults{
 		"testapp": {
 			results: []Result{