fix: slack template functions in notification playbook action

Author: adityathebe

.gitignore

@@ -21,6 +21,6 @@ junit-report.xml
 nohup.out
 .envrc
 .creds
-
+msg.tpl
+.windsurfrules
 test-reports
-msg.tpl

fixtures/playbooks/postgres-backup.yaml

@@ -0,0 +1,29 @@
+---
+apiVersion: mission-control.flanksource.com/v1
+kind: Playbook
+metadata:
+  name: backup-postgres
+spec:
+  actions:
+    - name: backup-postgres
+      exec:
+        env:
+          - name: CONN_STRING
+            valueFrom:
+              secretKeyRef:
+                name: flanksource_postgres
+                key: DB_URL
+        script: pg_dump --dbname "$CONN_STRING" -F c -b -v -f /mnt/backup/postgres.dump
+    - name: notify
+      if: 'failure()'
+      notification:
+        title: "Postgres backup failed"
+        message: |
+          {
+            "blocks": [
+              {{slackSectionTextMD (printf `:rotating_light: *Postgres backup failed*`)}},
+              {{slackSectionTextMD (printf "*Error:* %s" getLastAction.error)}},
+              {{slackURLAction "Report" "https://console.flanksource.com/mission-control/playbooks/postgres-backup"}}
+            ]
+          }
+        connection: connection://mc/flanksource-slack

playbook/runner/template.go

@@ -14,6 +14,7 @@ import (
 	"github.com/samber/oops"
 
 	v1 "github.com/flanksource/incident-commander/api/v1"
+	"github.com/flanksource/incident-commander/notification"
 	"github.com/flanksource/incident-commander/playbook/actions"
 )
 
@@ -204,16 +205,17 @@ func TemplateEnv(ctx context.Context, env actions.TemplateEnv, template string)
 
 // TemplateAction all the go templates in the action
 func TemplateAction(ctx context.Context, actionSpec *v1.PlaybookAction, env actions.TemplateEnv) error {
-	templater := ctx.NewStructTemplater(env.AsMap(ctx), "template", getGomplateFuncs(ctx, env))
+	templateFuncs := collections.MergeMap(getGomplateFuncs(ctx, env), notification.TemplateFuncs)
+	templater := ctx.NewStructTemplater(env.AsMap(ctx), "template", templateFuncs)
 	if err := templater.Walk(&actionSpec); err != nil {
-		return err
+		return ctx.Oops().Wrapf(err, "failed to template action")
 	}
 
 	// TODO: make this work with template.Walk()
 	if actionSpec.Exec != nil && actionSpec.Exec.Connections.FromConfigItem != nil {
-		if v, err := ctx.NewStructTemplater(env.AsMap(ctx), "", getGomplateFuncs(ctx, env)).
+		if v, err := ctx.NewStructTemplater(env.AsMap(ctx), "", templateFuncs).
 			Template(*actionSpec.Exec.Connections.FromConfigItem); err != nil {
-			return err
+			return ctx.Oops().Wrapf(err, "failed to template exec action connection from config item")
 		} else {
 			actionSpec.Exec.Connections.FromConfigItem = &v
 		}