fix: revert 3 CI-breaking regressions from PR #35, fix DateTime roundtrip

Reverted:
- DataTypes$LongType → DataTypes$IntegerType (4 files): Mendix type
  cache doesn't have LongType; Long must serialize as IntegerType
- XPath token quoting restored: '[%Token%]' not [%Token%] (CE0161)
- ComboBox setAssociationRef AttributeRef removed: 2-part association
  path causes ArgumentNullException in Mendix AttributeRef.AttributeId

Fixed:
- DateTime roundtrip: absent LocalizeDate defaults to DateTime, not Date
- Business events writer sets LocalizeDate for Date/DateTime distinction
- cmd_diff_local.go: same absent-LocalizeDate fix in both code paths

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

Author: ako

mdl/executor/bugfix_test.go

@@ -258,13 +258,13 @@ func TestExpressionToXPath_TokenQuoting(t *testing.T) {
 			name:     "Token_CurrentDateTime",
 			expr:     &ast.TokenExpr{Token: "CurrentDateTime"},
 			wantExpr: "[%CurrentDateTime%]",
-			wantXP:   "[%CurrentDateTime%]",
+			wantXP:   "'[%CurrentDateTime%]'",
 		},
 		{
 			name:     "Token_CurrentUser",
 			expr:     &ast.TokenExpr{Token: "CurrentUser"},
 			wantExpr: "[%CurrentUser%]",
-			wantXP:   "[%CurrentUser%]",
+			wantXP:   "'[%CurrentUser%]'",
 		},
 		{
 			name: "BinaryExpr_with_token",
@@ -274,7 +274,7 @@ func TestExpressionToXPath_TokenQuoting(t *testing.T) {
 				Right:    &ast.TokenExpr{Token: "CurrentDateTime"},
 			},
 			wantExpr: "DueDate < [%CurrentDateTime%]",
-			wantXP:   "DueDate < [%CurrentDateTime%]",
+			wantXP:   "DueDate < '[%CurrentDateTime%]'",
 		},
 		{
 			name:     "Variable_unchanged",
@@ -358,7 +358,7 @@ func TestExpressionToXPath_XPathPathExpr(t *testing.T) {
 				Operator: "=",
 				Right:    &ast.TokenExpr{Token: "CurrentUser"},
 			},
-			wantXP: "System.owner = [%CurrentUser%]",
+			wantXP: "System.owner = '[%CurrentUser%]'",
 		},
 		{
 			name: "not_with_path",

mdl/executor/cmd_dbconnection.go

@@ -326,7 +326,7 @@ func astDataTypeToDBType(dt ast.DataType) string {
 	case ast.TypeInteger:
 		return "DataTypes$IntegerType"
 	case ast.TypeLong:
-		return "DataTypes$LongType"
+		return "DataTypes$IntegerType" // Long maps to IntegerType in DataTypes
 	case ast.TypeDecimal:
 		return "DataTypes$DecimalType"
 	case ast.TypeBoolean:

mdl/executor/cmd_diff_local.go

@@ -374,7 +374,8 @@ func (e *Executor) attributeBsonToMDL(raw map[string]any) string {
 		case strings.Contains(attrType, "BooleanAttributeType"):
 			typeStr = "Boolean"
 		case strings.Contains(attrType, "DateTimeAttributeType"):
-			if localize, ok := raw["LocalizeDate"].(bool); ok && localize {
+			localize, ok := raw["LocalizeDate"].(bool)
+			if !ok || localize {
 				typeStr = "DateTime"
 			} else {
 				typeStr = "Date"
@@ -408,7 +409,8 @@ func (e *Executor) attributeBsonToMDL(raw map[string]any) string {
 			case strings.Contains(typeType, "BooleanAttributeType"):
 				typeStr = "Boolean"
 			case strings.Contains(typeType, "DateTimeAttributeType"):
-				if localize, ok := typeObj["LocalizeDate"].(bool); ok && localize {
+				localize, ok := typeObj["LocalizeDate"].(bool)
+				if !ok || localize {
 					typeStr = "DateTime"
 				} else {
 					typeStr = "Date"

mdl/executor/cmd_microflows_helpers.go

@@ -140,8 +140,8 @@ func expressionToString(expr ast.Expression) string {
 }
 
 // expressionToXPath converts an AST Expression to an XPath constraint string.
-// Mendix tokens like [%CurrentDateTime%] are written unquoted — they are
-// special XPath placeholders, not string literals.
+// Unlike expressionToString (for Mendix expressions), XPath requires Mendix
+// tokens like [%CurrentDateTime%] to be quoted: '[%CurrentDateTime%]'.
 func expressionToXPath(expr ast.Expression) string {
 	if expr == nil {
 		return ""
@@ -152,7 +152,7 @@ func expressionToXPath(expr ast.Expression) string {
 
 	switch e := expr.(type) {
 	case *ast.TokenExpr:
-		return "[%" + e.Token + "%]"
+		return "'[%" + e.Token + "%]'"
 	case *ast.BinaryExpr:
 		left := expressionToXPath(e.Left)
 		right := expressionToXPath(e.Right)

mdl/executor/cmd_pages_builder_input.go

@@ -172,24 +172,14 @@ func setDataSource(val bson.D, ds pages.DataSource) bson.D {
 	return result
 }
 
-// setAssociationRef sets both the AttributeRef and EntityRef fields in a WidgetValue
-// for an association binding on a pluggable widget.
-// AttributeRef stores the association qualified name (e.g., "Module.Order_Customer").
-// EntityRef uses DomainModels$IndirectEntityRef with a Steps array containing
+// setAssociationRef sets the EntityRef field in a WidgetValue for an association binding
+// on a pluggable widget. Uses DomainModels$IndirectEntityRef with a Steps array containing
 // a DomainModels$EntityRefStep that specifies the association and destination entity.
-// MxBuild requires both: association path in AttributeRef (CE8812) and
-// target entity in EntityRef (CE0642).
+// MxBuild requires the EntityRef to resolve the association target (CE0642).
 func setAssociationRef(val bson.D, assocPath string, entityName string) bson.D {
 	result := make(bson.D, 0, len(val))
 	for _, elem := range val {
-		if elem.Key == "AttributeRef" && assocPath != "" {
-			result = append(result, bson.E{Key: "AttributeRef", Value: bson.D{
-				{Key: "$ID", Value: mpr.IDToBsonBinary(mpr.GenerateID())},
-				{Key: "$Type", Value: "DomainModels$AttributeRef"},
-				{Key: "Attribute", Value: assocPath},
-				{Key: "EntityRef", Value: nil},
-			}})
-		} else if elem.Key == "EntityRef" && entityName != "" {
+		if elem.Key == "EntityRef" && entityName != "" {
 			result = append(result, bson.E{Key: "EntityRef", Value: bson.D{
 				{Key: "$ID", Value: mpr.IDToBsonBinary(mpr.GenerateID())},
 				{Key: "$Type", Value: "DomainModels$IndirectEntityRef"},

sdk/mpr/parser_domainmodel.go

@@ -299,13 +299,14 @@ func parseAttributeType(raw map[string]any) domainmodel.AttributeType {
 		t.ID = typeID
 		return t
 	case "DomainModels$DateTimeAttributeType":
-		localize, _ := raw["LocalizeDate"].(bool)
-		if localize {
+		localize, ok := raw["LocalizeDate"].(bool)
+		if !ok || localize {
+			// Default to DateTime when LocalizeDate is absent or true
 			t := &domainmodel.DateTimeAttributeType{LocalizeDate: true}
 			t.ID = typeID
 			return t
 		}
-		// LocalizeDate=false means date-only type
+		// LocalizeDate explicitly false means date-only type
 		dt := &domainmodel.DateAttributeType{}
 		dt.ID = typeID
 		return dt

sdk/mpr/writer_businessevents.go

@@ -154,10 +154,17 @@ func serializeBusinessEventAttribute(attr *model.BusinessEventAttribute) bson.M
 	}
 
 	// Convert attribute type to BSON format: "Long" → {"$Type": "DomainModels$LongAttributeType", "$ID": ...}
-	attrDoc["AttributeType"] = bson.M{
+	attrTypeDoc := bson.M{
 		"$Type": attributeTypeToBsonType(attr.AttributeType),
 		"$ID":   idToBsonBinary(generateUUID()),
 	}
+	// Date and DateTime both use DateTimeAttributeType; distinguish via LocalizeDate
+	if attr.AttributeType == "DateTime" {
+		attrTypeDoc["LocalizeDate"] = true
+	} else if attr.AttributeType == "Date" {
+		attrTypeDoc["LocalizeDate"] = false
+	}
+	attrDoc["AttributeType"] = attrTypeDoc
 
 	return attrDoc
 }

sdk/mpr/writer_enumeration.go

@@ -161,9 +161,11 @@ func serializeConstantDataType(dt model.ConstantDataType) bson.D {
 			{Key: "$Type", Value: "DataTypes$IntegerType"},
 		}
 	case "Long":
+		// Mendix uses IntegerType for both Integer and Long in BSON storage.
+		// DataTypes$LongType does not exist in the metamodel type cache.
 		return bson.D{
 			{Key: "$ID", Value: typeID},
-			{Key: "$Type", Value: "DataTypes$LongType"},
+			{Key: "$Type", Value: "DataTypes$IntegerType"},
 		}
 	case "Decimal":
 		return bson.D{

sdk/mpr/writer_microflow.go

@@ -254,9 +254,10 @@ func serializeMicroflowDataType(dt microflows.DataType) bson.D {
 			{Key: "$Type", Value: "DataTypes$IntegerType"},
 		}
 	case *microflows.LongType:
+		// Mendix uses IntegerType for 64-bit integers (Long in Java)
 		return bson.D{
 			{Key: "$ID", Value: idToBsonBinary(generateUUID())},
-			{Key: "$Type", Value: "DataTypes$LongType"},
+			{Key: "$Type", Value: "DataTypes$IntegerType"},
 		}
 	case *microflows.DecimalType:
 		return bson.D{

sdk/mpr/writer_rest.go

@@ -269,7 +269,7 @@ func serializeRestDataType(typeName string) bson.M {
 	case "Integer":
 		bsonType = "DataTypes$IntegerType"
 	case "Long":
-		bsonType = "DataTypes$LongType"
+		bsonType = "DataTypes$IntegerType" // Long maps to IntegerType in DataTypes
 	case "Decimal":
 		bsonType = "DataTypes$DecimalType"
 	case "Boolean":