feat: output raw HTML in get_activity_map for native markdown block rendering

2026-05-30 18:34:42 +02:00
parent ab971b8307
commit ef3b8305ab
3 changed files with 14 additions and 19 deletions
@@ -1,3 +1,3 @@
 """Strava MCP Server"""

-__version__ = "0.2.3"
+__version__ = "0.2.4"
@@ -844,13 +844,11 @@ def register(mcp: FastMCP, strava: StravaClient) -> None:
  </script>
 </body>
 </html>"""
-            escaped_html = html_content.strip().replace('"', "&quot;")
-            iframe_content = f'<iframe srcdoc="{escaped_html}" style="width: 100%; height: 600px; border: none; border-radius: 12px;"></iframe>'
            return [
-                TextContent(type="text", text=iframe_content),
+                TextContent(type="text", text=html_content.strip()),
                TextContent(
                    type="text",
-                    text="[System Context: The interactive map has been generated successfully. You must output the exact <iframe> HTML block from the tool result verbatim in your chat response so that Open WebUI can render it inline. Do not modify the iframe, and do not explain that you cannot render it; just output the tag verbatim.]",
+                    text="[System: Render the above content directly as an HTML block so the UI can iframe it.]",
                ),
            ]
        except Exception as e:
@@ -48,23 +48,20 @@ async def test_get_activity_map_success():

    assert len(result) == 2
    assert result[0].type == "text"
-    iframe_text = result[0].text
-    assert iframe_text.startswith('<iframe srcdoc="')
-    assert iframe_text.endswith('"></iframe>')
+    html_text = result[0].text

    assert result[1].type == "text"
-    assert "System Context" in result[1].text
+    assert "Render the above content" in result[1].text

-    decoded_html = iframe_text.replace("&quot;", '"')
-    assert "<!DOCTYPE html>" in decoded_html
-    assert "unpkg.com/leaflet" in decoded_html
-    assert "latlngData" in decoded_html
-    assert "[47.1, 9.1]" in decoded_html
-    assert 'sportEmoji = "🚴"' in decoded_html
-    assert "L.control.layers" in decoded_html
-    assert "satellite" in decoded_html
-    assert "🟢" in decoded_html
-    assert "🏁" in decoded_html
+    assert "<!DOCTYPE html>" in html_text
+    assert "unpkg.com/leaflet" in html_text
+    assert "latlngData" in html_text
+    assert "[47.1, 9.1]" in html_text
+    assert 'sportEmoji = "🚴"' in html_text
+    assert "L.control.layers" in html_text
+    assert "satellite" in html_text
+    assert "🟢" in html_text
+    assert "🏁" in html_text


@pytest.mark.asyncio