Reformat and other fixes

Author: khode-mx

CLAUDE.md

@@ -4,9 +4,9 @@ This file provides guidance for Claude Code when working with this repository.
 
 ## Canonical Test Project Path
 
-**ALWAYS use:** `/Users/Dennis.Kho/Sandbox/KhodeClaudeLab-main/KhodeClaudeLab.mpr`
+**ALWAYS use:** `/Users/Dennis.Kho/Sandbox/ClaudeKhodeLab-main/ClaudeKhodeLab.mpr`
 
-A disposable test copy exists at `/tmp/KhodeClaudeLab-test/` — **never use it for `mxcli exec` or write operations**. Before running any `mxcli -p <path> exec` command, verify the path starts with `/Users/Dennis.Kho/Sandbox/KhodeClaudeLab-main/`. If the path points to `/tmp/` or any other copy, stop and correct it first.
+A disposable test copy exists at `/tmp/KhodeClaudeLab-test/` — **never use it for `mxcli exec` or write operations**. Before running any `mxcli -p <path> exec` command, verify the path starts with `/Users/Dennis.Kho/Sandbox/ClaudeKhodeLab-main/`. If the path points to `/tmp/` or any other copy, stop and correct it first.
 
 ## Project Overview
 

cmd/mxcli/bson/registry.go

@@ -16,14 +16,14 @@ var TypeRegistry = map[string]reflect.Type{
 	"Workflows$Annotation":                        reflect.TypeOf(metamodel.WorkflowsAnnotation{}),
 	"Workflows$BezierCurve":                       reflect.TypeOf(metamodel.WorkflowsBezierCurve{}),
 	"Workflows$BooleanCase":                       reflect.TypeOf(metamodel.WorkflowsBooleanCase{}),
-	"Workflows$BooleanConditionOutcome":            reflect.TypeOf(metamodel.WorkflowsBooleanConditionOutcome{}),
+	"Workflows$BooleanConditionOutcome":           reflect.TypeOf(metamodel.WorkflowsBooleanConditionOutcome{}),
 	"Workflows$CallMicroflowTask":                 reflect.TypeOf(metamodel.WorkflowsCallMicroflowTask{}),
 	"Workflows$CallWorkflowActivity":              reflect.TypeOf(metamodel.WorkflowsCallWorkflowActivity{}),
 	"Workflows$ConsensusCompletionCriteria":       reflect.TypeOf(metamodel.WorkflowsConsensusCompletionCriteria{}),
 	"Workflows$EndOfBoundaryEventPathActivity":    reflect.TypeOf(metamodel.WorkflowsEndOfBoundaryEventPathActivity{}),
 	"Workflows$EndOfParallelSplitPathActivity":    reflect.TypeOf(metamodel.WorkflowsEndOfParallelSplitPathActivity{}),
 	"Workflows$EndWorkflowActivity":               reflect.TypeOf(metamodel.WorkflowsEndWorkflowActivity{}),
-	"Workflows$EnumerationValueConditionOutcome":   reflect.TypeOf(metamodel.WorkflowsEnumerationValueConditionOutcome{}),
+	"Workflows$EnumerationValueConditionOutcome":  reflect.TypeOf(metamodel.WorkflowsEnumerationValueConditionOutcome{}),
 	"Workflows$ExclusiveSplitActivity":            reflect.TypeOf(metamodel.WorkflowsExclusiveSplitActivity{}),
 	"Workflows$FloatingAnnotation":                reflect.TypeOf(metamodel.WorkflowsFloatingAnnotation{}),
 	"Workflows$Flow":                              reflect.TypeOf(metamodel.WorkflowsFlow{}),

cmd/mxcli/cmd_lint.go

@@ -141,6 +141,9 @@ Examples:
 		lint.AddRule(rules.NewWeakPasswordPolicyRule())
 		lint.AddRule(rules.NewDemoUsersActiveRule())
 
+		// MPR008 - requires BSON inspection
+		lint.AddRule(rules.NewOverlappingActivitiesRule())
+
 		// Convention rules (CONV011-CONV014) - require BSON inspection
 		lint.AddRule(rules.NewNoCommitInLoopRule())
 		lint.AddRule(rules.NewExclusiveSplitCaptionRule())

cmd/mxcli/cmd_report.go

@@ -101,6 +101,9 @@ Examples:
 		lint.AddRule(rules.NewWeakPasswordPolicyRule())
 		lint.AddRule(rules.NewDemoUsersActiveRule())
 
+		// MPR008 - requires BSON inspection
+		lint.AddRule(rules.NewOverlappingActivitiesRule())
+
 		// Convention rules (CONV011-CONV014)
 		lint.AddRule(rules.NewNoCommitInLoopRule())
 		lint.AddRule(rules.NewExclusiveSplitCaptionRule())

cmd/mxcli/cmd_widget.go

@@ -177,7 +177,7 @@ func generateDefJSON(mpkDef *mpk.WidgetDefinition, mdlName string) *executor.Wid
 				mapping.Value = prop.DefaultValue
 			}
 			mappings = append(mappings, mapping)
-		// Skip action, expression, textTemplate, object, icon, image, file — too complex for auto-mapping
+			// Skip action, expression, textTemplate, object, icon, image, file — too complex for auto-mapping
 		}
 	}
 

cmd/mxcli/lsp_diagnostics.go

@@ -294,7 +294,7 @@ func runSemanticValidation(text string) []protocol.Diagnostic {
 				},
 				Severity: violationToLSPSeverity(v.Severity),
 				Source:   "mdl-check",
-				Code:    v.RuleID,
+				Code:     v.RuleID,
 				Message:  msg,
 			})
 		}

cmd/mxcli/tui/agent_listener.go

@@ -172,4 +172,3 @@ func (al *AgentListener) handleSyncAction(req AgentRequest) (AgentResponse, bool
 	}
 	return AgentResponse{}, false
 }
-

cmd/mxcli/tui/agent_protocol.go

@@ -9,7 +9,7 @@ import (
 // AgentRequest is a JSON command from an external agent (e.g. Claude).
 type AgentRequest struct {
 	ID     int    `json:"id"`
-	Action string `json:"action"`          // "exec", "check", "state", "navigate", "delete", "create_module", "format", "describe", "list"
+	Action string `json:"action"`           // "exec", "check", "state", "navigate", "delete", "create_module", "format", "describe", "list"
 	MDL    string `json:"mdl,omitempty"`    // for "exec", "format"
 	Target string `json:"target,omitempty"` // for "navigate", "delete", "describe", "list" (e.g. "entity:Module.Entity")
 	Name   string `json:"name,omitempty"`   // for "create_module"
@@ -83,19 +83,19 @@ func buildAgentDescribeCmd(target string) (string, error) {
 
 // listKeywords maps lowercase node type names to their MDL SHOW keyword(s).
 var listKeywords = map[string]string{
-	"entities":        "ENTITIES",
-	"associations":    "ASSOCIATIONS",
-	"enumerations":    "ENUMERATIONS",
-	"constants":       "CONSTANTS",
-	"microflows":      "MICROFLOWS",
-	"nanoflows":       "NANOFLOWS",
-	"pages":           "PAGES",
-	"snippets":        "SNIPPETS",
-	"layouts":         "LAYOUTS",
-	"workflows":       "WORKFLOWS",
-	"modules":         "MODULES",
+	"entities":         "ENTITIES",
+	"associations":     "ASSOCIATIONS",
+	"enumerations":     "ENUMERATIONS",
+	"constants":        "CONSTANTS",
+	"microflows":       "MICROFLOWS",
+	"nanoflows":        "NANOFLOWS",
+	"pages":            "PAGES",
+	"snippets":         "SNIPPETS",
+	"layouts":          "LAYOUTS",
+	"workflows":        "WORKFLOWS",
+	"modules":          "MODULES",
 	"imagecollections": "IMAGE COLLECTIONS",
-	"javaactions":     "JAVA ACTIONS",
+	"javaactions":      "JAVA ACTIONS",
 }
 
 // buildListCmd returns the MDL SHOW command for a target like "entities" or "entities:Module".

cmd/mxcli/tui/app.go

@@ -52,18 +52,18 @@ type App struct {
 	statusBar     StatusBar
 	previewEngine *PreviewEngine
 
-	watcher       *Watcher
-	checkErrors   []CheckError // nil = no check run yet, empty = pass
-	checkRunning  bool
+	watcher      *Watcher
+	checkErrors  []CheckError // nil = no check run yet, empty = pass
+	checkRunning bool
 
 	pendingSession *TUISession // session to restore after tree loads
 
 	agentListener    *AgentListener
-	agentAutoProceed bool           // skip human confirmation for agent ops (set before tea.NewProgram)
-	agentPending     *agentPendingOp // non-nil when waiting for user confirmation
+	agentAutoProceed bool                 // skip human confirmation for agent ops (set before tea.NewProgram)
+	agentPending     *agentPendingOp      // non-nil when waiting for user confirmation
 	agentCheckCh     chan<- AgentResponse // non-nil when agent check is in-flight
-	agentCheckReqID  int               // request ID for pending agent check
-	agentExecCtx     *agentExecContext   // non-nil when agent-initiated exec/delete/create is in progress
+	agentCheckReqID  int                  // request ID for pending agent check
+	agentExecCtx     *agentExecContext    // non-nil when agent-initiated exec/delete/create is in progress
 }
 
 // agentPendingOp tracks an in-flight agent operation awaiting user confirmation.

cmd/mxcli/tui/browserview.go

@@ -19,7 +19,6 @@ type BrowserView struct {
 	mxcliPath     string
 	projectPath   string
 	previewEngine *PreviewEngine
-
 }
 
 // NewBrowserView creates a BrowserView wrapping the Miller view from the given tab.