fix: quote ProtocolVersion in DESCRIBE CONSUMED MCP SERVICE output (issue #435)

ProtocolVersion was emitted unquoted (e.g. ProtocolVersion: 2024-11-05),
which the parser reads as integer arithmetic (2024 minus 11 minus 5) and
rejects. The value is a version string, not a number, and must be quoted.

Changed the format string from `%s` to `'%s'` (matching how Version and
Documentation are already handled). The quoted form already round-trips
correctly via the STRING_LITERAL alternative in modelProperty.

Updated TestDescribeAgentEditorConsumedMCPService_Mock to assert the
quoted form and added a parse roundtrip check.

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

Author: ako

mdl/executor/cmd_agenteditor_mcpservices.go

@@ -83,7 +83,7 @@ func describeAgentEditorConsumedMCPService(ctx *ExecContext, name ast.QualifiedN
 
 	var lines []string
 	if c.ProtocolVersion != "" {
-		lines = append(lines, fmt.Sprintf("  ProtocolVersion: %s", c.ProtocolVersion))
+		lines = append(lines, fmt.Sprintf("  ProtocolVersion: '%s'", escapeSQLString(c.ProtocolVersion)))
 	}
 	if c.Version != "" {
 		lines = append(lines, fmt.Sprintf("  Version: '%s'", escapeSQLString(c.Version)))

mdl/executor/cmd_agenteditor_mock_test.go

@@ -7,6 +7,7 @@ import (
 
 	"github.com/mendixlabs/mxcli/mdl/ast"
 	"github.com/mendixlabs/mxcli/mdl/backend/mock"
+	"github.com/mendixlabs/mxcli/mdl/visitor"
 	"github.com/mendixlabs/mxcli/model"
 	"github.com/mendixlabs/mxcli/sdk/agenteditor"
 )
@@ -524,7 +525,14 @@ func TestDescribeAgentEditorConsumedMCPService_Mock(t *testing.T) {
 
 	out := buf.String()
 	assertContainsStr(t, out, "create consumed mcp service")
-	assertContainsStr(t, out, "ProtocolVersion")
+	// ProtocolVersion with a date value must be quoted so the output is re-parseable (issue #435)
+	assertContainsStr(t, out, "ProtocolVersion: '2025-03-26'")
+
+	// Roundtrip: DESCRIBE output must parse without errors
+	_, parseErrs := visitor.Build(out)
+	if len(parseErrs) > 0 {
+		t.Errorf("describe output is not valid MDL: %v\nOutput:\n%s", parseErrs[0], out)
+	}
 }
 
 // ---------------------------------------------------------------------------