Added auto generated display names for params and commands

go.mod

@@ -35,6 +35,7 @@ require (
 	go.abhg.dev/goldmark/frontmatter v0.2.0
 	golang.org/x/exp v0.0.0-20240222234643-814bf88cf225
 	golang.org/x/sync v0.6.0
+	golang.org/x/text v0.14.0
 )
 
 require (
@@ -71,7 +72,6 @@ require (
 	go.opencensus.io v0.23.0 // indirect
 	golang.org/x/net v0.24.0 // indirect
 	golang.org/x/oauth2 v0.1.0 // indirect
-	golang.org/x/text v0.14.0 // indirect
 	golang.org/x/xerrors v0.0.0-20220907171357-04be3eba64a2 // indirect
 	google.golang.org/api v0.100.0 // indirect
 	google.golang.org/appengine v1.6.7 // indirect

internal/runner/slackcmd.go

@@ -84,7 +84,7 @@ func SlackCommandRequest(flow *markdown.Flow, hopsMsg *nats.HopsMsg, matchError
 		hopsMsg.Data["trigger_id"].(string),
 		slack.ModalViewRequest{
 			Type:  slack.VTModal,
-			Title: slack.NewTextBlockObject("plain_text", flow.ID, false, false),
+			Title: slack.NewTextBlockObject("plain_text", flow.DisplayName(), false, false),
 			Blocks: slack.Blocks{
 				BlockSet: blocks,
 				// slack.NewTextBlockObject() // We want the rendered slack markdown here
@@ -125,21 +125,20 @@ func CommandToSlackBlocks(command markdown.Command) ([]slack.Block, error) {
 
 	for _, p := range command {
 		name, param := p.Param()
+		displayName := p.DisplayName()
 
 		switch param.Type {
 		case "text", "string":
-			blocks = append(blocks, ParamToTextInputBlock(name, param))
+			blocks = append(blocks, ParamToTextInputBlock(name, displayName, param))
 		case "bool":
-			blocks = append(blocks, ParamToBooleanInputBlock(name, param))
+			blocks = append(blocks, ParamToBooleanInputBlock(name, displayName, param))
 		case "number":
-			blocks = append(blocks, ParamToNumberInputBlock(name, param))
+			blocks = append(blocks, ParamToNumberInputBlock(name, displayName, param))
 		default:
 			return nil, fmt.Errorf("unable to parse param '%s' - unknown type '%s'", name, param.Type)
 		}
 	}
 
-	// Add the submit button etc? - Need to ensure we're handling no params
-
 	return blocks, nil
 }
 
@@ -152,17 +151,17 @@ func ParamBlockLabel(name string) *slack.TextBlockObject {
 	}
 }
 
-func ParamInputBlock(name string, param markdown.Param, elem slack.BlockElement) slack.Block {
+func ParamInputBlock(name, displayName string, param markdown.Param, elem slack.BlockElement) slack.Block {
 	return slack.InputBlock{
 		BlockID:  name,
 		Type:     slack.MBTInput,
-		Label:    ParamBlockLabel(name),
+		Label:    ParamBlockLabel(displayName),
 		Element:  elem,
 		Optional: !param.Required,
 	}
 }
 
-func ParamToNumberInputBlock(name string, param markdown.Param) slack.Block {
+func ParamToNumberInputBlock(name, displayName string, param markdown.Param) slack.Block {
 	elem := slack.NumberInputBlockElement{
 		Type:             slack.METNumber,
 		ActionID:         name,
@@ -179,10 +178,10 @@ func ParamToNumberInputBlock(name string, param markdown.Param) slack.Block {
 		}
 	}
 
-	return ParamInputBlock(name, param, elem)
+	return ParamInputBlock(name, displayName, param, elem)
 }
 
-func ParamToBooleanInputBlock(name string, param markdown.Param) slack.Block {
+func ParamToBooleanInputBlock(name, displayName string, param markdown.Param) slack.Block {
 	trueOption := &slack.OptionBlockObject{
 		Value: "true",
 		Text: &slack.TextBlockObject{
@@ -217,10 +216,10 @@ func ParamToBooleanInputBlock(name string, param markdown.Param) slack.Block {
 		elem.InitialOption = falseOption
 	}
 
-	return ParamInputBlock(name, param, elem)
+	return ParamInputBlock(name, displayName, param, elem)
 }
 
-func ParamToTextInputBlock(name string, param markdown.Param) slack.Block {
+func ParamToTextInputBlock(name, displayName string, param markdown.Param) slack.Block {
 	elem := slack.PlainTextInputBlockElement{
 		Type:        slack.METPlainTextInput,
 		Placeholder: nil,
@@ -233,7 +232,7 @@ func ParamToTextInputBlock(name string, param markdown.Param) slack.Block {
 		elem.InitialValue = defaultVal
 	}
 
-	return ParamInputBlock(name, param, elem)
+	return ParamInputBlock(name, displayName, param, elem)
 }
 
 func parseViewSubmissionCommand(payload map[string]any) (map[string]any, error) {

markdown/flow.go

@@ -14,6 +14,8 @@ import (
 	"github.com/yuin/goldmark/ast"
 	"github.com/zclconf/go-cty/cty/gocty"
 	"go.abhg.dev/goldmark/frontmatter"
+	"golang.org/x/text/cases"
+	"golang.org/x/text/language"
 )
 
 type (
@@ -239,6 +241,14 @@ func (f *Flow) ActionName() string {
 	return strings.ReplaceAll(f.ID, ".", "-")
 }
 
+func (f *Flow) DisplayName() string {
+	if strings.ToLower(f.fileName) == "index" {
+		return titleCase(f.dirName)
+	}
+
+	return titleCase(f.ID)
+}
+
 func (f *Flow) IfValue(evalCtx *hcl.EvalContext) (bool, error) {
 	if f.If == "" {
 		return true, nil
@@ -258,6 +268,14 @@ func (f *Flow) IfValue(evalCtx *hcl.EvalContext) (bool, error) {
 	return matches, nil
 }
 
+func (pi *ParamItem) DisplayName() string {
+	for name := range *pi {
+		return titleCase(name)
+	}
+
+	return ""
+}
+
 func (pi *ParamItem) Param() (string, Param) {
 	for name, p := range *pi {
 		if p.Type == "" {
@@ -269,3 +287,11 @@ func (pi *ParamItem) Param() (string, Param) {
 
 	return "", Param{}
 }
+
+var titleCaseReplacer = strings.NewReplacer("_", " ", ".", " ")
+
+func titleCase(label string) string {
+	caser := cases.Title(language.BritishEnglish)
+	label = titleCaseReplacer.Replace(label)
+	return caser.String(label)
+}