fix: add RENAME WORKFLOW support (#466)

Add WORKFLOW to the renameTarget grammar rule, regenerate the ANTLR4
parser, and wire the "workflow" case through the executor dispatcher
and document-exists check. RenameDocumentByName already handles
workflows generically so no SDK changes are needed.

Also add TestRename_Workflow_Success / _NotFound mock tests, update
the pre-existing TestRename_UnsupportedType (which was using "workflow"
as its unsupported-type stub), and extend 25-rename-examples.mdl with
a create/rename/drop workflow example.

Closes #466

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

Author: ako

mdl-examples/doctype-tests/25-rename-examples.mdl

@@ -39,7 +39,15 @@ rename java action RenameTest.OldHelper to NewHelper;
 -- Dry run: preview what references would change without modifying anything
 rename java action RenameTest.NewHelper to AnotherName dry run;
 
+-- Rename workflow
+create workflow RenameTest.OldProcess
+begin
+end workflow;
+
+rename workflow RenameTest.OldProcess to NewProcess;
+
 -- Clean up
+drop workflow RenameTest.NewProcess;
 drop java action RenameTest.NewHelper;
 drop association RenameTest.Order_Client;
 drop entity RenameTest.client;

mdl/ast/ast_rename.go

@@ -7,7 +7,7 @@ package ast
 //	RENAME ENTITY Module.OldName TO NewName [DRY RUN];
 //	RENAME MODULE OldName TO NewName [DRY RUN];
 //	RENAME JAVA ACTION Module.OldName TO NewName [DRY RUN];
-//	... and microflow, nanoflow, page, enumeration, association, constant
+//	... and microflow, nanoflow, page, enumeration, association, constant, workflow
 type RenameStmt struct {
 	ObjectType string        // lowercase type key, e.g. "entity", "module", "javaaction"
 	Name       QualifiedName // Current name (Module.Entity or Module for modules)

mdl/executor/cmd_rename.go

@@ -33,6 +33,8 @@ func execRename(ctx *ExecContext, s *ast.RenameStmt) error {
 		return execRenameAssociation(ctx, s)
 	case "constant":
 		return execRenameDocument(ctx, s, "constant")
+	case "workflow":
+		return execRenameDocument(ctx, s, "workflow")
 	case "javaaction":
 		return execRenameJavaAction(ctx, s)
 	case "module":
@@ -200,6 +202,15 @@ func execRenameDocument(ctx *ExecContext, s *ast.RenameStmt, docType string) err
 				break
 			}
 		}
+	case "workflow":
+		wfs, _ := ctx.Backend.ListWorkflows()
+		for _, wf := range wfs {
+			modID := h.FindModuleID(wf.ContainerID)
+			if h.GetModuleName(modID) == s.Name.Module && wf.Name == s.Name.Name {
+				found = true
+				break
+			}
+		}
 	}
 
 	if !found {

mdl/executor/cmd_rename_mock_test.go

@@ -13,6 +13,7 @@ import (
 	"github.com/mendixlabs/mxcli/sdk/domainmodel"
 	"github.com/mendixlabs/mxcli/sdk/microflows"
 	"github.com/mendixlabs/mxcli/sdk/pages"
+	"github.com/mendixlabs/mxcli/sdk/workflows"
 )
 
 // ---------------------------------------------------------------------------
@@ -39,7 +40,7 @@ func TestRename_UnsupportedType(t *testing.T) {
 	mb := &mock.MockBackend{IsConnectedFunc: func() bool { return true }}
 	ctx, _ := newMockCtx(t, withBackend(mb))
 	err := execRename(ctx, &ast.RenameStmt{
-		ObjectType: "workflow",
+		ObjectType: "snippet",
 		Name:       ast.QualifiedName{Module: "M", Name: "N"},
 		NewName:    "X",
 	})
@@ -382,3 +383,63 @@ func TestRename_JavaAction_NotFound(t *testing.T) {
 	assertError(t, err)
 	assertContainsStr(t, err.Error(), "not found")
 }
+
+// ---------------------------------------------------------------------------
+// Rename workflow — happy path
+// ---------------------------------------------------------------------------
+
+func TestRename_Workflow_Success(t *testing.T) {
+	mod := mkModule("BPModule")
+	wf := mkWorkflow(mod.ID, "OldProcess")
+	renameCalled := false
+	mb := &mock.MockBackend{
+		IsConnectedFunc:   func() bool { return true },
+		ListModulesFunc:   func() ([]*model.Module, error) { return []*model.Module{mod}, nil },
+		ListFoldersFunc:   func() ([]*types.FolderInfo, error) { return nil, nil },
+		ListWorkflowsFunc: func() ([]*workflows.Workflow, error) { return []*workflows.Workflow{wf}, nil },
+		RenameReferencesFunc: func(old, new string, dryRun bool) ([]types.RenameHit, error) {
+			return nil, nil
+		},
+		RenameDocumentByNameFunc: func(module, old, newName string) error {
+			renameCalled = true
+			return nil
+		},
+	}
+	h := mkHierarchy(mod)
+	withContainer(h, wf.ContainerID, mod.ID)
+	ctx, buf := newMockCtx(t, withBackend(mb), withHierarchy(h))
+	assertNoError(t, execRename(ctx, &ast.RenameStmt{
+		ObjectType: "workflow",
+		Name:       ast.QualifiedName{Module: "BPModule", Name: "OldProcess"},
+		NewName:    "NewProcess",
+	}))
+	if !renameCalled {
+		t.Error("Expected RenameDocumentByName to be called")
+	}
+	assertContainsStr(t, buf.String(), "Renamed workflow")
+	assertContainsStr(t, buf.String(), "BPModule.OldProcess")
+	assertContainsStr(t, buf.String(), "BPModule.NewProcess")
+}
+
+// ---------------------------------------------------------------------------
+// Rename workflow — not found
+// ---------------------------------------------------------------------------
+
+func TestRename_Workflow_NotFound(t *testing.T) {
+	mod := mkModule("BPModule")
+	mb := &mock.MockBackend{
+		IsConnectedFunc:   func() bool { return true },
+		ListModulesFunc:   func() ([]*model.Module, error) { return []*model.Module{mod}, nil },
+		ListFoldersFunc:   func() ([]*types.FolderInfo, error) { return nil, nil },
+		ListWorkflowsFunc: func() ([]*workflows.Workflow, error) { return nil, nil },
+	}
+	h := mkHierarchy(mod)
+	ctx, _ := newMockCtx(t, withBackend(mb), withHierarchy(h))
+	err := execRename(ctx, &ast.RenameStmt{
+		ObjectType: "workflow",
+		Name:       ast.QualifiedName{Module: "BPModule", Name: "Missing"},
+		NewName:    "New",
+	})
+	assertError(t, err)
+	assertContainsStr(t, err.Error(), "not found")
+}

mdl/grammar/MDLParser.g4

@@ -306,7 +306,7 @@ renameStatement
     ;
 
 renameTarget
-    : ENTITY | MICROFLOW | NANOFLOW | PAGE | ENUMERATION | ASSOCIATION | CONSTANT | JAVA ACTION
+    : ENTITY | MICROFLOW | NANOFLOW | PAGE | ENUMERATION | ASSOCIATION | CONSTANT | JAVA ACTION | WORKFLOW
     ;
 
 /**