Remove 'mcp' render mode from user-facing options

The mcp render mode exposed too much internal detail (service names, full
metadata) which confused smaller models into trying to call services directly.

Changes:
- Remove 'mcp' from renderMode descriptions in tool definitions
- Map renderMode='mcp' to 'compact' if explicitly requested
- Update AGENTS.md to remove mcp mode documentation
- Internal rendering still uses mcp mode, just not exposed to users

Users should use:
- compact (default): actionable summary
- aria: accessibility tree for field discovery
- text/html: rendered output

@@ -601,7 +601,7 @@
             <parameter name="path" required="true"/>
             <parameter name="path" required="true"/>
             <parameter name="parameters" type="Map"><description>Parameters to pass to screen</description></parameter>
             <parameter name="parameters" type="Map"><description>Parameters to pass to screen</description></parameter>
             <parameter name="action"><description>Action being processed: if not null, use real screen rendering instead of test mock</description></parameter>
             <parameter name="action"><description>Action being processed: if not null, use real screen rendering instead of test mock</description></parameter>
-            <parameter name="renderMode" default="compact"><description>Render mode: compact (default, actionable summary), aria (accessibility tree), mcp (full metadata), text, html, xml, vuet, qvt</description></parameter>
+            <parameter name="renderMode" default="compact"><description>Render mode: compact (default, actionable summary), aria (accessibility tree), text, html, xml, vuet, qvt</description></parameter>
             <parameter name="sessionId"><description>Session ID for user context restoration</description></parameter>
             <parameter name="sessionId"><description>Session ID for user context restoration</description></parameter>
         </in-parameters>
         </in-parameters>
         <out-parameters>
         <out-parameters>
@@ -1412,8 +1412,10 @@ def wikiInstructions = getWikiInstructions(inputScreenPath)
             }
             }
 
 
             // Regular screen rendering with current user context - use our custom ScreenTestImpl
             // Regular screen rendering with current user context - use our custom ScreenTestImpl
-            // For compact/ARIA modes, we still render with MCP to get semantic data, then convert
-            def actualScreenRenderMode = (renderMode == "aria" || renderMode == "compact" || renderMode == null) ? "mcp" : renderMode
+            // For compact/ARIA modes, we still render internally with MCP to get semantic data, then convert
+            // Note: "mcp" is not exposed to users - if requested, treat as "compact"
+            def userRenderMode = (renderMode == "mcp") ? "compact" : renderMode
+            def actualScreenRenderMode = (userRenderMode == "aria" || userRenderMode == "compact" || userRenderMode == null) ? "mcp" : userRenderMode
             def screenTest = new org.moqui.mcp.CustomScreenTestImpl(ec.ecfi)
             def screenTest = new org.moqui.mcp.CustomScreenTestImpl(ec.ecfi)
                 .rootScreen(rootScreen)
                 .rootScreen(rootScreen)
                 .renderMode(actualScreenRenderMode)
                 .renderMode(actualScreenRenderMode)
@@ -2165,7 +2167,7 @@ def wikiInstructions = getWikiInstructions(inputScreenPath)
         <in-parameters>
         <in-parameters>
             <parameter name="path" required="false"><description>Screen path to browse (e.g. 'PopCommerce'). Leave empty for root apps.</description></parameter>
             <parameter name="path" required="false"><description>Screen path to browse (e.g. 'PopCommerce'). Leave empty for root apps.</description></parameter>
             <parameter name="action"><description>Action to process before rendering: null (browse), 'submit' (form), 'create', 'update', or transition name</description></parameter>
             <parameter name="action"><description>Action to process before rendering: null (browse), 'submit' (form), 'create', 'update', or transition name</description></parameter>
-            <parameter name="renderMode" default="compact"><description>Render mode: compact (default, actionable summary), aria (accessibility tree), mcp (full metadata), text, html, xml, vuet, qvt</description></parameter>
+            <parameter name="renderMode" default="compact"><description>Render mode: compact (default, actionable summary), aria (accessibility tree), text, html, xml, vuet, qvt</description></parameter>
             <parameter name="parameters" type="Map"><description>Parameters to pass to screen during rendering or action</description></parameter>
             <parameter name="parameters" type="Map"><description>Parameters to pass to screen during rendering or action</description></parameter>
             <parameter name="sessionId"/>
             <parameter name="sessionId"/>
         </in-parameters>
         </in-parameters>
@@ -2782,13 +2784,13 @@ def wikiInstructions = getWikiInstructions(inputScreenPath)
                     [
                     [
                         name: "moqui_browse_screens",
                         name: "moqui_browse_screens",
                         title: "Browse Screens",
                         title: "Browse Screens",
-                        description: "Browse Moqui screen hierarchy, process actions, and render screen content. Input 'path' (empty for root). Default renderMode is 'mcp'.",
+                        description: "Browse Moqui screen hierarchy, process actions, and render screen content. Input 'path' (empty for root). Default renderMode is 'compact'.",
                         inputSchema: [
                         inputSchema: [
                             type: "object",
                             type: "object",
                             properties: [
                             properties: [
                                 "path": [type: "string", description: "Path to browse (e.g. 'PopCommerce')"],
                                 "path": [type: "string", description: "Path to browse (e.g. 'PopCommerce')"],
                                 "action": [type: "string", description: "Action to process before rendering: null (browse), 'submit' (form), 'create', 'update', or transition name"],
                                 "action": [type: "string", description: "Action to process before rendering: null (browse), 'submit' (form), 'create', 'update', or transition name"],
-                                "renderMode": [type: "string", description: "Render mode: compact (default, actionable summary), aria (accessibility tree), mcp (full metadata), text, html, xml, vuet, qvt"],
+                                "renderMode": [type: "string", description: "Render mode: compact (default, actionable summary), aria (accessibility tree), text, html, xml, vuet, qvt"],
                                 "parameters": [type: "object", description: "Parameters to pass to screen during rendering or action"]
                                 "parameters": [type: "object", description: "Parameters to pass to screen during rendering or action"]
                             ]
                             ]
                         ]
                         ]