perf: pre-warm name lookup maps to eliminate O(n²) BSON parsing in catalog source

Each describeMicroflow() call was re-parsing ALL domain models, microflows,
and pages from BSON to build name lookup maps. With 925 microflows × 8
workers × 3 list calls, that's ~2,775 full BSON parsing passes.

Pre-warm entityNames, microflowNames, and pageNames maps once in
PreWarmCache() and share across all parallel describe goroutines.
Reduces source generation from O(n²) to O(n).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: ako

mdl/executor/cmd_catalog.go

@@ -13,6 +13,7 @@ import (
 
 	"github.com/mendixlabs/mxcli/mdl/ast"
 	"github.com/mendixlabs/mxcli/mdl/catalog"
+	"github.com/mendixlabs/mxcli/model"
 )
 
 // execShowCatalogTables handles SHOW CATALOG TABLES.
@@ -788,10 +789,39 @@ func (e *Executor) captureDescribeParallel(objectType string, qualifiedName stri
 	return buf.String(), nil
 }
 
-// PreWarmCache ensures the hierarchy cache is populated before parallel operations.
+// PreWarmCache ensures all caches are populated before parallel operations.
 // Must be called from the main goroutine before using captureDescribeParallel.
+// This avoids O(n²) re-parsing in describe functions by building name lookup
+// maps once and sharing them across all goroutines.
 func (e *Executor) PreWarmCache() {
-	e.getHierarchy()
+	h, _ := e.getHierarchy()
+	if h == nil || e.cache == nil {
+		return
+	}
+
+	// Build entity name lookup
+	e.cache.entityNames = make(map[model.ID]string)
+	dms, _ := e.reader.ListDomainModels()
+	for _, dm := range dms {
+		modName := h.GetModuleName(dm.ContainerID)
+		for _, ent := range dm.Entities {
+			e.cache.entityNames[ent.ID] = modName + "." + ent.Name
+		}
+	}
+
+	// Build microflow name lookup
+	e.cache.microflowNames = make(map[model.ID]string)
+	mfs, _ := e.reader.ListMicroflows()
+	for _, mf := range mfs {
+		e.cache.microflowNames[mf.ID] = h.GetQualifiedName(mf.ContainerID, mf.Name)
+	}
+
+	// Build page name lookup
+	e.cache.pageNames = make(map[model.ID]string)
+	pgs, _ := e.reader.ListPages()
+	for _, pg := range pgs {
+		e.cache.pageNames[pg.ID] = h.GetQualifiedName(pg.ContainerID, pg.Name)
+	}
 }
 
 // execSearch handles SEARCH 'query' command.

mdl/executor/cmd_microflows_show.go

@@ -267,35 +267,21 @@ func (e *Executor) describeMicroflow(name ast.QualifiedName) error {
 		return fmt.Errorf("failed to build hierarchy: %w", err)
 	}
 
-	// Build entity name lookup for resolving entity IDs
-	entityNames := make(map[model.ID]string)
-	domainModels, _ := e.reader.ListDomainModels()
-	for _, dm := range domainModels {
-		modName := h.GetModuleName(dm.ContainerID)
-		for _, entity := range dm.Entities {
-			entityNames[entity.ID] = modName + "." + entity.Name
-		}
-	}
-
-	// Build microflow name lookup for resolving microflow IDs
-	microflowNames := make(map[model.ID]string)
-
-	// Build page name lookup for resolving page IDs
-	pageNames := make(map[model.ID]string)
-	allPages, _ := e.reader.ListPages()
-	for _, p := range allPages {
-		pageNames[p.ID] = h.GetQualifiedName(p.ContainerID, p.Name)
-	}
+	// Use pre-warmed cache if available (from PreWarmCache), otherwise build on demand
+	entityNames := e.getEntityNames(h)
+	microflowNames := e.getMicroflowNames(h)
 
 	// Find the microflow
 	allMicroflows, err := e.reader.ListMicroflows()
 	if err != nil {
 		return fmt.Errorf("failed to list microflows: %w", err)
 	}
 
-	// Build microflow name lookup
-	for _, mf := range allMicroflows {
-		microflowNames[mf.ID] = h.GetQualifiedName(mf.ContainerID, mf.Name)
+	// Supplement microflow name lookup if not pre-warmed
+	if len(microflowNames) == 0 {
+		for _, mf := range allMicroflows {
+			microflowNames[mf.ID] = h.GetQualifiedName(mf.ContainerID, mf.Name)
+		}
 	}
 
 	var targetMf *microflows.Microflow

mdl/executor/executor.go

@@ -34,6 +34,11 @@ type executorCache struct {
 
 	// Track domain models modified during this session for finalization
 	modifiedDomainModels map[model.ID]string // domain model unit ID -> module name
+
+	// Pre-warmed name lookup maps for parallel describe (goroutine-safe after init)
+	entityNames    map[model.ID]string // entity ID -> "Module.EntityName"
+	microflowNames map[model.ID]string // microflow ID -> "Module.MicroflowName"
+	pageNames      map[model.ID]string // page ID -> "Module.PageName"
 }
 
 // createdMicroflowInfo tracks a microflow created during this session.
@@ -61,6 +66,48 @@ type createdSnippetInfo struct {
 	ContainerID model.ID
 }
 
+// getEntityNames returns the entity name lookup map, using the pre-warmed cache if available.
+func (e *Executor) getEntityNames(h *ContainerHierarchy) map[model.ID]string {
+	if e.cache != nil && len(e.cache.entityNames) > 0 {
+		return e.cache.entityNames
+	}
+	entityNames := make(map[model.ID]string)
+	dms, _ := e.reader.ListDomainModels()
+	for _, dm := range dms {
+		modName := h.GetModuleName(dm.ContainerID)
+		for _, ent := range dm.Entities {
+			entityNames[ent.ID] = modName + "." + ent.Name
+		}
+	}
+	return entityNames
+}
+
+// getMicroflowNames returns the microflow name lookup map, using the pre-warmed cache if available.
+func (e *Executor) getMicroflowNames(h *ContainerHierarchy) map[model.ID]string {
+	if e.cache != nil && len(e.cache.microflowNames) > 0 {
+		return e.cache.microflowNames
+	}
+	microflowNames := make(map[model.ID]string)
+	mfs, _ := e.reader.ListMicroflows()
+	for _, mf := range mfs {
+		microflowNames[mf.ID] = h.GetQualifiedName(mf.ContainerID, mf.Name)
+	}
+	return microflowNames
+}
+
+// getPageNames returns the page name lookup map, using the pre-warmed cache if available.
+func (e *Executor) getPageNames(h *ContainerHierarchy) map[model.ID]string {
+	if e.cache != nil && len(e.cache.pageNames) > 0 {
+		return e.cache.pageNames
+	}
+	pageNames := make(map[model.ID]string)
+	pgs, _ := e.reader.ListPages()
+	for _, pg := range pgs {
+		pageNames[pg.ID] = h.GetQualifiedName(pg.ContainerID, pg.Name)
+	}
+	return pageNames
+}
+
 const (
 	// maxOutputLines is the per-statement line limit. Statements that produce more
 	// lines than this are aborted to prevent runaway output from infinite loops.