fix: enable LSP diagnostics for editable mendix-mdl:// documents

Remove the skip guards that suppressed parse diagnostics and semantic
validation for mendix-mdl:// URIs. These were added when the documents
were read-only but now that they are editable, users need error
squiggles as they type.

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

Author: ako

cmd/mxcli/lsp_diagnostics.go

@@ -60,19 +60,6 @@ func parseMDLDiagnostics(text string) []protocol.Diagnostic {
 
 // publishDiagnostics parses the document and sends diagnostics to the client.
 func (s *mdlServer) publishDiagnostics(ctx context.Context, docURI uri.URI, text string) {
-	// Skip diagnostics for virtual documents (mendix-mdl:// scheme).
-	// These are read-only DESCRIBE output (e.g. workflow pseudo-MDL) that
-	// may not conform to the MDL grammar.
-	uriStr := string(docURI)
-	if strings.HasPrefix(uriStr, "mendix-mdl://") {
-		// Clear any stale diagnostics
-		s.client.PublishDiagnostics(ctx, &protocol.PublishDiagnosticsParams{
-			URI:         protocol.DocumentURI(docURI),
-			Diagnostics: []protocol.Diagnostic{},
-		})
-		return
-	}
-
 	diags := parseMDLDiagnostics(text)
 	// If no parse errors, run semantic validation inline
 	if len(diags) == 0 {
@@ -138,11 +125,6 @@ func (s *mdlServer) DidClose(ctx context.Context, params *protocol.DidCloseTextD
 func (s *mdlServer) DidSave(ctx context.Context, params *protocol.DidSaveTextDocumentParams) error {
 	docURI := uri.URI(params.TextDocument.URI)
 
-	// Skip semantic checks for virtual documents (read-only DESCRIBE output)
-	if strings.HasPrefix(string(docURI), "mendix-mdl://") {
-		return nil
-	}
-
 	s.mu.Lock()
 	text := s.docs[docURI]
 	s.mu.Unlock()