fix: slack labels in section limit (#1344)

* fix: slack labels in section limit

* chore: sort labels

* feat: accept the entire resoruce in `slackSectionLabels`

notification/template.go

@@ -3,8 +3,11 @@ package notification
 import (
 	"encoding/json"
 	"fmt"
+	"slices"
+	"strings"
 
 	"github.com/flanksource/duty/models"
+	"github.com/samber/lo"
 )
 
 var templateFuncs = map[string]any{
@@ -32,31 +35,79 @@ var templateFuncs = map[string]any{
 			return ":white_circle:"
 		}
 	},
-	"slackSectionLabels": func(labels map[string]any) string {
-		if len(labels) == 0 {
+	"slackSectionLabels": func(resource map[string]any) string {
+		if resource == nil {
 			return ""
 		}
 
-		var fields []any
-		for k, v := range labels {
-			fields = append(fields, map[string]any{
-				"type":     "mrkdwn",
-				"text":     fmt.Sprintf("*%s*: %s", k, v),
-				"verbatim": true,
-			})
+		configTags := map[string]any{}
+		var tagFields []map[string]any
+		if tagsRaw, ok := resource["tags"]; ok {
+			if tags, ok := tagsRaw.(map[string]any); ok {
+				configTags = tags
+				for k, v := range tags {
+					tagFields = append(tagFields, map[string]any{
+						"type":     "mrkdwn",
+						"text":     fmt.Sprintf("*%s*: %s", k, v),
+						"verbatim": true,
+					})
+				}
+			}
 		}
 
-		var m = map[string]any{
-			"type":   "section",
-			"fields": fields,
-			"text": map[string]any{
-				"type": "mrkdwn",
-				"text": "*Labels*",
-			},
+		var labelFields []map[string]any
+		if labelsRaw, ok := resource["labels"]; ok {
+			if labels, ok := labelsRaw.(map[string]any); ok {
+				for k, v := range labels {
+					if _, ok := configTags[k]; ok {
+						continue // Already pulled from tags
+					}
+
+					labelFields = append(labelFields, map[string]any{
+						"type":     "mrkdwn",
+						"text":     fmt.Sprintf("*%s*: %s", k, v),
+						"verbatim": true,
+					})
+				}
+			}
+		}
+
+		if len(tagFields) == 0 && len(labelFields) == 0 {
+			return ""
+		}
+
+		slices.SortFunc(tagFields, func(a, b map[string]any) int {
+			return strings.Compare(a["text"].(string), b["text"].(string))
+		})
+
+		slices.SortFunc(labelFields, func(a, b map[string]any) int {
+			return strings.Compare(a["text"].(string), b["text"].(string))
+		})
+
+		fields := append(tagFields, labelFields...)
+
+		var outputs []string
+		const maxFieldsPerSection = 10
+		for i, chunk := range lo.Chunk(fields, maxFieldsPerSection) {
+			var m = map[string]any{
+				"type":   "section",
+				"fields": chunk,
+			}
+
+			if i == 0 {
+				m["text"] = map[string]any{
+					"type": "mrkdwn",
+					"text": "*Labels*",
+				}
+			}
+
+			out, err := json.Marshal(m)
+			if err == nil {
+				outputs = append(outputs, string(out))
+			}
 		}
 
-		out, _ := json.Marshal(m)
-		return string(out)
+		return strings.Join(outputs, ",")
 	},
 	"slackSectionTextFieldPlain": func(text string) string {
 		return fmt.Sprintf(`{

notification/templates/check.failed

@@ -14,7 +14,7 @@
 				{{end}}
 			]
 		},
-		{{ if .check.labels}}{{slackSectionLabels .check.labels}},{{end}}
+		{{ if .check.labels}}{{slackSectionLabels .check}},{{end}}
 		{{ slackURLAction "View Health Check" .permalink}}
 	]
 }

notification/templates/check.passed

@@ -14,7 +14,7 @@
 				{{end}}
 			]
 		},
-    {{ if .check.labels}}{{slackSectionLabels .check.labels}},{{end}}
+    {{ if .check.labels}}{{slackSectionLabels .check}},{{end}}
     {{ slackURLAction "View Health Check" .permalink}}
   ]
 }

notification/templates/component.health

@@ -14,7 +14,7 @@
 				{{end}}
 			]
 		},
-		{{if .component.labels}}{{slackSectionLabels .component.labels}},{{end}}
+		{{if .component.labels}}{{slackSectionLabels .component}},{{end}}
 		{{slackURLAction "View Component" .permalink}}
 	]
 }

notification/templates/config.db.update

@@ -14,7 +14,7 @@
 				{{end}}
 			]
 		},
-		{{if .config.labels}}{{slackSectionLabels .config.labels}},{{end}}
+		{{if .config.labels}}{{slackSectionLabels .config}},{{end}}
 		{{slackURLAction "View Catalog" .permalink}}
 	]
 }

notification/templates/config.health

@@ -14,7 +14,7 @@
 				{{end}}
 			]
 		},
-		{{if .config.labels}}{{slackSectionLabels .config.labels}},{{end}}
+		{{if .config.labels}}{{slackSectionLabels .config}},{{end}}
 		{{slackURLAction "View Catalog" .permalink}}
 	]
 }