Continued work to implement deep subscreen access

@@ -1186,27 +1186,17 @@ try {
                                         // Try to construct a reasonable path based on CURRENT screen being processed (not always the original parent)
                                         // Try to construct a reasonable path based on CURRENT screen being processed (not always the original parent)
                                         def currentScreenPath = screenPath  // This is the current screen whose subscreens we're processing
                                         def currentScreenPath = screenPath  // This is the current screen whose subscreens we're processing
                                         
                                         
-                                        if (currentScreenPath.contains("Catalog/Product.xml")) {
-                                            // We're processing Product.xml's subscreens
-                                            actualSubScreenPath = "component://SimpleScreens/screen/SimpleScreens/Catalog/Product/${subscreenName}.xml"
-                                        } else if (currentScreenPath.contains("Catalog.xml")) {
-                                            // We're processing Catalog.xml's subscreens
-                                            actualSubScreenPath = "component://SimpleScreens/screen/SimpleScreens/Catalog/${subscreenName}.xml"
-                                        } else if (currentScreenPath.contains("PopCommerceAdmin")) {
-                                            actualSubScreenPath = currentScreenPath.replace(".xml", "/${subscreenName}.xml")
-                                        } else {
-                                            // Generic fallback: construct based on current screen path
-                                            def lastSlash = currentScreenPath.lastIndexOf('/')
-                                            if (lastSlash > 0) {
-                                                def basePath = currentScreenPath.substring(0, lastSlash + 1)
-                                                actualSubScreenPath = basePath + subscreenName + ".xml"
-                                            }
+                                        // Generic fallback: construct based on current screen path
+                                        def lastSlash = currentScreenPath.lastIndexOf('/')
+                                        if (lastSlash > 0) {
+                                            def basePath = currentScreenPath.substring(0, lastSlash + 1)
+                                            actualSubScreenPath = basePath + subscreenName + ".xml"
                                         }
                                         }
                                         ec.logger.info("MCP Screen Discovery: Constructed fallback subscreen location for ${subScreenEntry.key}: ${actualSubScreenPath}")
                                         ec.logger.info("MCP Screen Discovery: Constructed fallback subscreen location for ${subScreenEntry.key}: ${actualSubScreenPath}")
                                     }
                                     }
                                     
                                     
                                     if (actualSubScreenPath && !processedScreens.contains(actualSubScreenPath)) {
                                     if (actualSubScreenPath && !processedScreens.contains(actualSubScreenPath)) {
-                                        ec.logger.info("MCP Screen Discovery: Processing subscreen path: ${actualSubScreenPath}")
+                                        ec.logger.info("MCP Screen Discovery: Processing subscreen path: ${actualSubScreenPath} ${screenPath}")
                                         processScreenWithSubscreens(actualSubScreenPath, screenPath, processedScreens, toolsAccumulator, toolName, level + 1)
                                         processScreenWithSubscreens(actualSubScreenPath, screenPath, processedScreens, toolsAccumulator, toolName, level + 1)
                                     } else if (!actualSubScreenPath) {
                                     } else if (!actualSubScreenPath) {
                                         ec.logger.info("MCP Screen Discovery: Subscreen entry ${subScreenEntry.key} has no location, skipping")
                                         ec.logger.info("MCP Screen Discovery: Subscreen entry ${subScreenEntry.key} has no location, skipping")
@@ -1615,7 +1605,7 @@ def startTime = System.currentTimeMillis()
             // Regular screen rendering with timeout for subscreen
             // Regular screen rendering with timeout for subscreen
                 
                 
             try {
             try {
-                ec.logger.info("TESTRENDER ${subscreenPathParts} ${renderParams}")
+                ec.logger.info("TESTRENDER ${subscreenName.replaceAll('_','/')} ${renderParams}")
                 // For subscreens, the path should be relative to the parent screen that's already set as root
                 // For subscreens, the path should be relative to the parent screen that's already set as root
                 // Since we're using the parent screen as root, we only need the subscreen name part
                 // Since we're using the parent screen as root, we only need the subscreen name part
                 def testRender = screenTest.render(subscreenName.replaceAll('_','/'), renderParams, "POST")
                 def testRender = screenTest.render(subscreenName.replaceAll('_','/'), renderParams, "POST")

@@ -273,7 +273,7 @@ class CustomScreenTestImpl implements McpScreenTest {
                 // rootScreenDef is the parent screen, screenPath is the subscreen path
                 // rootScreenDef is the parent screen, screenPath is the subscreen path
                 screenPathList = new ArrayList<>()
                 screenPathList = new ArrayList<>()
                 // Add root screen path (already a full component:// path)
                 // Add root screen path (already a full component:// path)
-                screenPathList.add(csti.rootScreenDef.location)
+                //screenPathList.add(csti.rootScreenDef.location)
                 // Add subscreen path segments
                 // Add subscreen path segments
                 String[] pathSegments = stri.screenPath.split('/')
                 String[] pathSegments = stri.screenPath.split('/')
                 for (String segment in pathSegments) {
                 for (String segment in pathSegments) {
@@ -316,7 +316,7 @@ class CustomScreenTestImpl implements McpScreenTest {
 
 
             // do the render
             // do the render
             try {
             try {
-                logger.info("Starting render for ${stri.screenPath} with root ${csti.rootScreenLocation}")
+                logger.info("Starting render for ${screenPathList} with root ${csti.rootScreenLocation}")
                 screenRender.render(wfs.getRequest(), wfs.getResponse())
                 screenRender.render(wfs.getRequest(), wfs.getResponse())
                 // get the response text from the WebFacadeStub
                 // get the response text from the WebFacadeStub
                 stri.outputString = wfs.getResponseText()
                 stri.outputString = wfs.getResponseText()

@@ -67,6 +67,8 @@
     <screen-facade boundary-comments="false">
     <screen-facade boundary-comments="false">
         <screen-text-output type="html" mime-type="text/html" 
         <screen-text-output type="html" mime-type="text/html" 
                 macro-template-location="template/screen-macro/ScreenHtmlMacros.ftl"/>
                 macro-template-location="template/screen-macro/ScreenHtmlMacros.ftl"/>
+        <screen-text-output type="markdown" mime-type="text/markdown"
+                macro-template-location="template/screen-macro/DefaultScreenMacros.markdown.ftl"/>
     </screen-facade>
     </screen-facade>
 
 
     <!-- Test entity facade with in-memory database -->
     <!-- Test entity facade with in-memory database -->