feat: add comprehensive mock-based handler tests (189 tests, 33 files)

Phase 1-8: 85 read/write handler tests with mock backend infrastructure
Phase 9: 104 additional tests covering error paths, not-connected guards, and JSON format output

Test categories:
- Read-path handlers (SHOW/DESCRIBE) across 20+ domains
- Write-path handlers (CREATE/DROP) for 10 operations
- Backend error propagation for all handler groups
- Not-connected guard verification for 29 guarded handlers
- JSON format output validation for 26 show/list handlers

Author: retran

internal/marketplace/types.go

@@ -18,7 +18,7 @@ var BaseURL = "https://marketplace-api.mendix.com"
 type Content struct {
 	ContentID       int        `json:"contentId"`
 	Publisher       string     `json:"publisher"`
-	Type            string     `json:"type"`            // "Module", "Widget", "Theme", "Starter App", ...
+	Type            string     `json:"type"` // "Module", "Widget", "Theme", "Starter App", ...
 	Categories      []Category `json:"categories"`
 	SupportCategory string     `json:"supportCategory"` // "Platform", "Community", "Deprecated", ...
 	LicenseURL      string     `json:"licenseUrl,omitempty"`

mdl/ast/ast_agenteditor.go

@@ -16,16 +16,16 @@ package ast
 //	  [, DeepLinkURL: '...']
 //	);
 type CreateModelStmt struct {
-	Name             QualifiedName
-	Documentation    string
-	Provider         string         // "MxCloudGenAI" by default
-	Key              *QualifiedName // qualified name of the String constant holding the Portal key
-	DisplayName      string         // optional Portal-populated metadata
-	KeyName          string         // optional Portal-populated metadata
-	KeyID            string         // optional Portal-populated metadata
-	Environment      string         // optional Portal-populated metadata
-	ResourceName     string         // optional Portal-populated metadata
-	DeepLinkURL      string         // optional Portal-populated metadata
+	Name          QualifiedName
+	Documentation string
+	Provider      string         // "MxCloudGenAI" by default
+	Key           *QualifiedName // qualified name of the String constant holding the Portal key
+	DisplayName   string         // optional Portal-populated metadata
+	KeyName       string         // optional Portal-populated metadata
+	KeyID         string         // optional Portal-populated metadata
+	Environment   string         // optional Portal-populated metadata
+	ResourceName  string         // optional Portal-populated metadata
+	DeepLinkURL   string         // optional Portal-populated metadata
 }
 
 func (s *CreateModelStmt) isStatement() {}
@@ -93,21 +93,21 @@ func (s *DropKnowledgeBaseStmt) isStatement() {}
 
 // CreateAgentStmt represents CREATE AGENT Module.Name (...) [{ body }].
 type CreateAgentStmt struct {
-	Name         QualifiedName
+	Name          QualifiedName
 	Documentation string
-	UsageType    string // "Task" or "Conversational"
-	Description  string
-	Model        *QualifiedName // reference to a Model document
-	Entity       *QualifiedName // reference to a domain entity
-	MaxTokens    *int
-	ToolChoice   string
-	Temperature  *float64
-	TopP         *float64
-	SystemPrompt string
-	UserPrompt   string
-	Variables    []AgentVarDef
-	Tools        []AgentToolDef
-	KBTools      []AgentKBToolDef
+	UsageType     string // "Task" or "Conversational"
+	Description   string
+	Model         *QualifiedName // reference to a Model document
+	Entity        *QualifiedName // reference to a domain entity
+	MaxTokens     *int
+	ToolChoice    string
+	Temperature   *float64
+	TopP          *float64
+	SystemPrompt  string
+	UserPrompt    string
+	Variables     []AgentVarDef
+	Tools         []AgentToolDef
+	KBTools       []AgentKBToolDef
 }
 
 func (s *CreateAgentStmt) isStatement() {}
@@ -136,10 +136,10 @@ type AgentToolDef struct {
 
 // AgentKBToolDef represents a KNOWLEDGE BASE block in CREATE AGENT body.
 type AgentKBToolDef struct {
-	Name                 string // per-agent identifier
-	Source               *QualifiedName
-	Collection           string
-	MaxResults           int
-	Description          string
-	Enabled              bool
+	Name        string // per-agent identifier
+	Source      *QualifiedName
+	Collection  string
+	MaxResults  int
+	Description string
+	Enabled     bool
 }

mdl/backend/mock/backend.go

@@ -260,8 +260,16 @@ type MockBackend struct {
 	FindAllCustomWidgetTypesFunc func(widgetID string) ([]*mpr.RawCustomWidgetType, error)
 
 	// AgentEditorBackend
-	ListAgentEditorModelsFunc              func() ([]*agenteditor.Model, error)
-	ListAgentEditorKnowledgeBasesFunc      func() ([]*agenteditor.KnowledgeBase, error)
-	ListAgentEditorConsumedMCPServicesFunc func() ([]*agenteditor.ConsumedMCPService, error)
-	ListAgentEditorAgentsFunc              func() ([]*agenteditor.Agent, error)
+	ListAgentEditorModelsFunc               func() ([]*agenteditor.Model, error)
+	ListAgentEditorKnowledgeBasesFunc       func() ([]*agenteditor.KnowledgeBase, error)
+	ListAgentEditorConsumedMCPServicesFunc  func() ([]*agenteditor.ConsumedMCPService, error)
+	ListAgentEditorAgentsFunc               func() ([]*agenteditor.Agent, error)
+	CreateAgentEditorModelFunc              func(m *agenteditor.Model) error
+	DeleteAgentEditorModelFunc              func(id string) error
+	CreateAgentEditorKnowledgeBaseFunc      func(kb *agenteditor.KnowledgeBase) error
+	DeleteAgentEditorKnowledgeBaseFunc      func(id string) error
+	CreateAgentEditorConsumedMCPServiceFunc func(svc *agenteditor.ConsumedMCPService) error
+	DeleteAgentEditorConsumedMCPServiceFunc func(id string) error
+	CreateAgentEditorAgentFunc              func(a *agenteditor.Agent) error
+	DeleteAgentEditorAgentFunc              func(id string) error
 }

mdl/backend/mock/mock_infrastructure.go

@@ -188,13 +188,58 @@ func (m *MockBackend) ListAgentEditorAgents() ([]*agenteditor.Agent, error) {
 	return nil, nil
 }
 
-func (m *MockBackend) CreateAgentEditorModel(_ *agenteditor.Model) error                  { return nil }
-func (m *MockBackend) DeleteAgentEditorModel(_ string) error                              { return nil }
-func (m *MockBackend) CreateAgentEditorKnowledgeBase(_ *agenteditor.KnowledgeBase) error  { return nil }
-func (m *MockBackend) DeleteAgentEditorKnowledgeBase(_ string) error                      { return nil }
-func (m *MockBackend) CreateAgentEditorConsumedMCPService(_ *agenteditor.ConsumedMCPService) error {
+func (m *MockBackend) CreateAgentEditorModel(model *agenteditor.Model) error {
+	if m.CreateAgentEditorModelFunc != nil {
+		return m.CreateAgentEditorModelFunc(model)
+	}
+	return nil
+}
+
+func (m *MockBackend) DeleteAgentEditorModel(id string) error {
+	if m.DeleteAgentEditorModelFunc != nil {
+		return m.DeleteAgentEditorModelFunc(id)
+	}
+	return nil
+}
+
+func (m *MockBackend) CreateAgentEditorKnowledgeBase(kb *agenteditor.KnowledgeBase) error {
+	if m.CreateAgentEditorKnowledgeBaseFunc != nil {
+		return m.CreateAgentEditorKnowledgeBaseFunc(kb)
+	}
+	return nil
+}
+
+func (m *MockBackend) DeleteAgentEditorKnowledgeBase(id string) error {
+	if m.DeleteAgentEditorKnowledgeBaseFunc != nil {
+		return m.DeleteAgentEditorKnowledgeBaseFunc(id)
+	}
+	return nil
+}
+
+func (m *MockBackend) CreateAgentEditorConsumedMCPService(svc *agenteditor.ConsumedMCPService) error {
+	if m.CreateAgentEditorConsumedMCPServiceFunc != nil {
+		return m.CreateAgentEditorConsumedMCPServiceFunc(svc)
+	}
+	return nil
+}
+
+func (m *MockBackend) DeleteAgentEditorConsumedMCPService(id string) error {
+	if m.DeleteAgentEditorConsumedMCPServiceFunc != nil {
+		return m.DeleteAgentEditorConsumedMCPServiceFunc(id)
+	}
+	return nil
+}
+
+func (m *MockBackend) CreateAgentEditorAgent(a *agenteditor.Agent) error {
+	if m.CreateAgentEditorAgentFunc != nil {
+		return m.CreateAgentEditorAgentFunc(a)
+	}
+	return nil
+}
+
+func (m *MockBackend) DeleteAgentEditorAgent(id string) error {
+	if m.DeleteAgentEditorAgentFunc != nil {
+		return m.DeleteAgentEditorAgentFunc(id)
+	}
 	return nil
 }
-func (m *MockBackend) DeleteAgentEditorConsumedMCPService(_ string) error { return nil }
-func (m *MockBackend) CreateAgentEditorAgent(_ *agenteditor.Agent) error  { return nil }
-func (m *MockBackend) DeleteAgentEditorAgent(_ string) error              { return nil }