feat: make lowercase keywords the standard convention in MDL output

Convert all Go source files, executor commands, visitor code, and tests
to produce and expect lowercase MDL keywords. The grammar already
supports case-insensitive parsing; this change establishes lowercase as
the canonical output convention so generated MDL reads more like a
modern DSL than SQL/BASIC.

- Visitor: store widget types as lowercase (ToLower instead of ToUpper)
- Executor: DESCRIBE/SHOW commands emit lowercase keywords throughout
- Widget registry: normalize MDLName and MDLContainer to lowercase on load
- Microflow formatter: lowercase aggregate functions and log levels
- All test assertions updated to expect lowercase

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: ako

mdl/backend/mpr/convert_test.go

@@ -121,7 +121,7 @@ func TestJsonStructureFieldPreservation(t *testing.T) {
 	js := &mpr.JsonStructure{
 		ContainerID:   model.ID("m1"),
 		Name:          "MyJson",
-		Documentation: "Test JSON structure",
+		Documentation: "Test json structure",
 		JsonSnippet:   `{"a":1}`,
 		Elements: []*mpr.JsonElement{
 			{

mdl/backend/mpr/datagrid_builder.go

@@ -1191,7 +1191,7 @@ func (b *MprBackend) buildFilterWidgetBSON(widgetID, filterName string, projectP
 	rawType, rawObject, _, _, err := widgets.GetTemplateFullBSON(widgetID, types.GenerateID, projectPath)
 	if err != nil || rawType == nil {
 		if err != nil {
-			log.Printf("WARNING: failed to load template for widget %s: %v; using minimal fallback", widgetID, err)
+			log.Printf("warning: failed to load template for widget %s: %v; using minimal fallback", widgetID, err)
 		}
 		return b.buildMinimalFilterWidgetBSON(widgetID, filterName)
 	}

mdl/backend/mpr/page_mutator.go

@@ -258,7 +258,7 @@ func (m *mprPageMutator) DropVariable(name string) error {
 
 func (m *mprPageMutator) SetLayout(newLayout string, paramMappings map[string]string) error {
 	if m.containerType == backend.ContainerSnippet {
-		return fmt.Errorf("SET Layout is not supported for snippets")
+		return fmt.Errorf("set Layout is not supported for snippets")
 	}
 
 	formCall := dGetDoc(m.rawData, "FormCall")

mdl/backend/mpr/widget_builder.go

@@ -365,7 +365,7 @@ func updatePropertyInArray(arr bson.A, propertyTypeID string, updateFn func(bson
 	}
 	if !matched {
 		// TODO(shared-types): propagate warning instead of logging — requires interface change.
-		log.Printf("WARNING: updatePropertyInArray: no match for TypePointer %s in %d properties", propertyTypeID, len(arr)-1)
+		log.Printf("warning: updatePropertyInArray: no match for TypePointer %s in %d properties", propertyTypeID, len(arr)-1)
 	}
 	return result
 }

mdl/backend/mpr/workflow_mutator.go

@@ -56,7 +56,7 @@ func (b *MprBackend) openWorkflowForMutation(unitID model.ID) (backend.WorkflowM
 
 func (m *mprWorkflowMutator) SetProperty(prop string, value string) error {
 	switch prop {
-	case "DISPLAY":
+	case "display":
 		wfName := dGetDoc(m.rawData, "WorkflowName")
 		if wfName == nil {
 			newName := bson.D{
@@ -71,7 +71,7 @@ func (m *mprWorkflowMutator) SetProperty(prop string, value string) error {
 		dSet(m.rawData, "Title", value)
 		return nil
 
-	case "DESCRIPTION":
+	case "description":
 		wfDesc := dGetDoc(m.rawData, "WorkflowDescription")
 		if wfDesc == nil {
 			newDesc := bson.D{
@@ -113,7 +113,7 @@ func (m *mprWorkflowMutator) SetPropertyWithEntity(prop string, value string, en
 		}
 		return nil
 
-	case "PARAMETER":
+	case "parameter":
 		if value == "" {
 			for i, elem := range m.rawData {
 				if elem.Key == "Parameter" {
@@ -159,7 +159,7 @@ func (m *mprWorkflowMutator) SetActivityProperty(activityRef string, atPos int,
 	}
 
 	switch prop {
-	case "PAGE":
+	case "page":
 		taskPage := dGetDoc(actDoc, "TaskPage")
 		if taskPage != nil {
 			dSet(taskPage, "Page", value)
@@ -173,7 +173,7 @@ func (m *mprWorkflowMutator) SetActivityProperty(activityRef string, atPos int,
 		}
 		return nil
 
-	case "DESCRIPTION":
+	case "description":
 		taskDesc := dGetDoc(actDoc, "TaskDescription")
 		if taskDesc != nil {
 			dSet(taskDesc, "Text", value)

mdl/backend/mpr/workflow_mutator_test.go

@@ -81,8 +81,8 @@ func TestWorkflowMutator_SetProperty_Display(t *testing.T) {
 	doc := makeWorkflowDoc()
 	m := newMutator(doc)
 
-	if err := m.SetProperty("DISPLAY", "New Title"); err != nil {
-		t.Fatalf("SetProperty DISPLAY failed: %v", err)
+	if err := m.SetProperty("display", "New Title"); err != nil {
+		t.Fatalf("SetProperty display failed: %v", err)
 	}
 
 	if got := dGetString(m.rawData, "Title"); got != "New Title" {
@@ -108,8 +108,8 @@ func TestWorkflowMutator_SetProperty_Display_NilSubDoc(t *testing.T) {
 	}
 	m := newMutator(doc)
 
-	if err := m.SetProperty("DISPLAY", "Created Title"); err != nil {
-		t.Fatalf("SetProperty DISPLAY with nil sub-doc failed: %v", err)
+	if err := m.SetProperty("display", "Created Title"); err != nil {
+		t.Fatalf("SetProperty display with nil sub-doc failed: %v", err)
 	}
 
 	if got := dGetString(m.rawData, "Title"); got != "Created Title" {
@@ -128,8 +128,8 @@ func TestWorkflowMutator_SetProperty_Description(t *testing.T) {
 	doc := makeWorkflowDoc()
 	m := newMutator(doc)
 
-	if err := m.SetProperty("DESCRIPTION", "Updated desc"); err != nil {
-		t.Fatalf("SetProperty DESCRIPTION failed: %v", err)
+	if err := m.SetProperty("description", "Updated desc"); err != nil {
+		t.Fatalf("SetProperty description failed: %v", err)
 	}
 
 	wfDesc := dGetDoc(m.rawData, "WorkflowDescription")
@@ -152,8 +152,8 @@ func TestWorkflowMutator_SetProperty_Description_NilSubDoc(t *testing.T) {
 	}
 	m := newMutator(doc)
 
-	if err := m.SetProperty("DESCRIPTION", "New desc"); err != nil {
-		t.Fatalf("SetProperty DESCRIPTION with nil sub-doc failed: %v", err)
+	if err := m.SetProperty("description", "New desc"); err != nil {
+		t.Fatalf("SetProperty description with nil sub-doc failed: %v", err)
 	}
 
 	wfDesc := dGetDoc(m.rawData, "WorkflowDescription")
@@ -1142,8 +1142,8 @@ func TestWorkflowMutator_SetActivityProperty_Page_New(t *testing.T) {
 	act = append(act, bson.E{Key: "TaskPage", Value: nil})
 	m := newMutator(makeWorkflowDoc(act))
 
-	if err := m.SetActivityProperty("Review", 0, "PAGE", "MyModule.TaskPage"); err != nil {
-		t.Fatalf("SetActivityProperty PAGE failed: %v", err)
+	if err := m.SetActivityProperty("Review", 0, "page", "MyModule.TaskPage"); err != nil {
+		t.Fatalf("SetActivityProperty page failed: %v", err)
 	}
 
 	actDoc, _ := m.findActivityByCaption("Review", 0)
@@ -1163,8 +1163,8 @@ func TestWorkflowMutator_SetActivityProperty_Page_MissingKey(t *testing.T) {
 	m := newMutator(makeWorkflowDoc(act))
 
 	// No error returned, but the set is silently lost
-	if err := m.SetActivityProperty("Review", 0, "PAGE", "MyModule.TaskPage"); err != nil {
-		t.Fatalf("SetActivityProperty PAGE failed: %v", err)
+	if err := m.SetActivityProperty("Review", 0, "page", "MyModule.TaskPage"); err != nil {
+		t.Fatalf("SetActivityProperty page failed: %v", err)
 	}
 
 	actDoc, _ := m.findActivityByCaption("Review", 0)
@@ -1184,8 +1184,8 @@ func TestWorkflowMutator_SetActivityProperty_Page_Existing(t *testing.T) {
 	}})
 	m := newMutator(makeWorkflowDoc(act))
 
-	if err := m.SetActivityProperty("Review", 0, "PAGE", "NewModule.NewPage"); err != nil {
-		t.Fatalf("SetActivityProperty PAGE update failed: %v", err)
+	if err := m.SetActivityProperty("Review", 0, "page", "NewModule.NewPage"); err != nil {
+		t.Fatalf("SetActivityProperty page update failed: %v", err)
 	}
 
 	actDoc, _ := m.findActivityByCaption("Review", 0)
@@ -1204,8 +1204,8 @@ func TestWorkflowMutator_SetActivityProperty_Description(t *testing.T) {
 	}})
 	m := newMutator(makeWorkflowDoc(act))
 
-	if err := m.SetActivityProperty("Review", 0, "DESCRIPTION", "new desc"); err != nil {
-		t.Fatalf("SetActivityProperty DESCRIPTION failed: %v", err)
+	if err := m.SetActivityProperty("Review", 0, "description", "new desc"); err != nil {
+		t.Fatalf("SetActivityProperty description failed: %v", err)
 	}
 
 	actDoc, _ := m.findActivityByCaption("Review", 0)
@@ -1299,8 +1299,8 @@ func TestWorkflowMutator_SetPropertyWithEntity_Parameter_New(t *testing.T) {
 	doc = append(doc, bson.E{Key: "Parameter", Value: nil})
 	m := newMutator(doc)
 
-	if err := m.SetPropertyWithEntity("PARAMETER", "WorkflowContext", "MyModule.Order"); err != nil {
-		t.Fatalf("SetPropertyWithEntity PARAMETER failed: %v", err)
+	if err := m.SetPropertyWithEntity("parameter", "WorkflowContext", "MyModule.Order"); err != nil {
+		t.Fatalf("SetPropertyWithEntity parameter failed: %v", err)
 	}
 
 	param := dGetDoc(m.rawData, "Parameter")
@@ -1322,8 +1322,8 @@ func TestWorkflowMutator_SetPropertyWithEntity_Parameter_Update(t *testing.T) {
 	}})
 	m := newMutator(doc)
 
-	if err := m.SetPropertyWithEntity("PARAMETER", "WorkflowContext", "NewModule.NewEntity"); err != nil {
-		t.Fatalf("SetPropertyWithEntity PARAMETER update failed: %v", err)
+	if err := m.SetPropertyWithEntity("parameter", "WorkflowContext", "NewModule.NewEntity"); err != nil {
+		t.Fatalf("SetPropertyWithEntity parameter update failed: %v", err)
 	}
 
 	param := dGetDoc(m.rawData, "Parameter")
@@ -1339,8 +1339,8 @@ func TestWorkflowMutator_SetPropertyWithEntity_Parameter_Clear(t *testing.T) {
 	}})
 	m := newMutator(doc)
 
-	if err := m.SetPropertyWithEntity("PARAMETER", "", ""); err != nil {
-		t.Fatalf("SetPropertyWithEntity PARAMETER clear failed: %v", err)
+	if err := m.SetPropertyWithEntity("parameter", "", ""); err != nil {
+		t.Fatalf("SetPropertyWithEntity parameter clear failed: %v", err)
 	}
 
 	if v := dGet(m.rawData, "Parameter"); v != nil {

mdl/executor/bugfix_integration_test.go

@@ -21,17 +21,17 @@ func TestDropCreateMicroflowReplacesContent(t *testing.T) {
 	name := testModule + ".MF_DropCreateTest"
 
 	// Create original microflow with a LOG statement
-	err := env.executeMDL(`CREATE MICROFLOW ` + name + ` ()
-BEGIN
-  LOG INFO 'original content';
-END;
+	err := env.executeMDL(`create microflow ` + name + ` ()
+begin
+  log info 'original content';
+end;
 /`)
 	if err != nil {
 		t.Fatalf("Failed to create original microflow: %v", err)
 	}
 
 	// Verify original content
-	output, err := env.describeMDL("DESCRIBE MICROFLOW " + name + ";")
+	output, err := env.describeMDL("describe microflow " + name + ";")
 	if err != nil {
 		t.Fatalf("Failed to describe original: %v", err)
 	}
@@ -40,30 +40,30 @@ END;
 	}
 
 	// DROP and recreate with different content
-	err = env.executeMDL("DROP MICROFLOW " + name + ";")
+	err = env.executeMDL("drop microflow " + name + ";")
 	if err != nil {
 		t.Fatalf("Failed to drop microflow: %v", err)
 	}
 
-	err = env.executeMDL(`CREATE MICROFLOW ` + name + ` ()
-BEGIN
-  LOG WARNING 'replacement content';
-END;
+	err = env.executeMDL(`create microflow ` + name + ` ()
+begin
+  log warning 'replacement content';
+end;
 /`)
 	if err != nil {
 		t.Fatalf("Failed to create replacement microflow: %v", err)
 	}
 
 	// DESCRIBE should show the NEW content
-	output, err = env.describeMDL("DESCRIBE MICROFLOW " + name + ";")
+	output, err = env.describeMDL("describe microflow " + name + ";")
 	if err != nil {
 		t.Fatalf("Failed to describe replacement: %v", err)
 	}
 	if !strings.Contains(output, "replacement content") {
-		t.Errorf("DROP+CREATE did not replace content. Got:\n%s", output)
+		t.Errorf("drop+create did not replace content. Got:\n%s", output)
 	}
 	if strings.Contains(output, "original content") {
-		t.Errorf("DROP+CREATE still shows original content. Got:\n%s", output)
+		t.Errorf("drop+create still shows original content. Got:\n%s", output)
 	}
 }
 
@@ -78,7 +78,7 @@ func TestDescribeEnumerationInSubfolder(t *testing.T) {
 	enumName := testModule + ".SubfolderTestStatus"
 
 	// Create an enumeration
-	err := env.executeMDL(`CREATE ENUMERATION ` + enumName + ` (
+	err := env.executeMDL(`create enumeration ` + enumName + ` (
 		Active 'Active',
 		Inactive 'Inactive'
 	);`)
@@ -87,18 +87,18 @@ func TestDescribeEnumerationInSubfolder(t *testing.T) {
 	}
 
 	// Move it to a subfolder
-	err = env.executeMDL(`MOVE ENUMERATION ` + enumName + ` TO FOLDER 'Enums';`)
+	err = env.executeMDL(`move enumeration ` + enumName + ` to folder 'Enums';`)
 	if err != nil {
 		t.Fatalf("Failed to move enumeration to folder: %v", err)
 	}
 
 	// DESCRIBE should still find it
-	output, err := env.describeMDL("DESCRIBE ENUMERATION " + enumName + ";")
+	output, err := env.describeMDL("describe enumeration " + enumName + ";")
 	if err != nil {
-		t.Errorf("DESCRIBE ENUMERATION failed for subfoldered enum: %v", err)
+		t.Errorf("describe enumeration failed for subfoldered enum: %v", err)
 		return
 	}
 	if !strings.Contains(output, "Active") || !strings.Contains(output, "Inactive") {
-		t.Errorf("DESCRIBE output missing enum values:\n%s", output)
+		t.Errorf("describe output missing enum values:\n%s", output)
 	}
 }