removed intermediate function for OpenF1 Gradio tools. Now the display name for the MCP tools should be correct in the MCP host

app.py

@@ -120,61 +120,42 @@ def openf1_tools_tab():
         with gr.Accordion("get_api_endpoints()", open=False):
             btn = gr.Button("Get all endpoints")
             output = gr.JSON()
-            def _get_api_endpoints():
-                return openf1_tools.get_api_endpoints()
-            btn.click(_get_api_endpoints, outputs=output)
+            btn.click(openf1_tools.get_api_endpoints, outputs=output)
         with gr.Accordion("get_api_endpoint(endpoint)", open=False):
             endpoint_in = gr.Textbox(label="Endpoint", placeholder="e.g. sessions")
             btn = gr.Button("Get endpoint info")
             output = gr.JSON()
-            def _get_api_endpoint(endpoint):
-                return openf1_tools.get_api_endpoint(endpoint)
-            btn.click(_get_api_endpoint, inputs=endpoint_in, outputs=output)
+            btn.click(openf1_tools.get_api_endpoint, inputs=endpoint_in, outputs=output)
         with gr.Accordion("get_endpoint_info(endpoint)", open=False):
             endpoint_in = gr.Textbox(label="Endpoint", placeholder="e.g. sessions")
             btn = gr.Button("Get endpoint details")
             output = gr.JSON()
-            def _get_endpoint_info(endpoint):
-                return openf1_tools.get_endpoint_info(endpoint)
-            btn.click(_get_endpoint_info, inputs=endpoint_in, outputs=output)
+            btn.click(openf1_tools.get_endpoint_info, inputs=endpoint_in, outputs=output)
         with gr.Accordion("get_filter_info(filter_name)", open=False):
             filter_in = gr.Textbox(label="Filter name", placeholder="e.g. driver_number")
             btn = gr.Button("Get filter info")
             output = gr.JSON()
-            def _get_filter_info(filter_name):
-                return openf1_tools.get_filter_info(filter_name)
-            btn.click(_get_filter_info, inputs=filter_in, outputs=output)
+            btn.click(openf1_tools.get_filter_info, inputs=filter_in, outputs=output)
         with gr.Accordion("get_filter_string(filter_name, filter_value, operator)", open=False):
             filter_name = gr.Textbox(label="Filter name", placeholder="e.g. driver_number")
             filter_value = gr.Textbox(label="Filter value", placeholder="e.g. 16")
             operator = gr.Dropdown(label="Operator", choices=["=", ">", "<", ">=", "<="], value="=")
             btn = gr.Button("Get filter string")
             output = gr.Textbox(label="Filter string", info="Example: driver_number=16&")
-            def _get_filter_string(filter_name, filter_value, operator):
-                return openf1_tools.get_filter_string(filter_name, filter_value, operator)
-            btn.click(_get_filter_string, inputs=[filter_name, filter_value, operator], outputs=output)
+            btn.click(openf1_tools.get_filter_string, inputs=[filter_name, filter_value, operator], outputs=output)
         with gr.Accordion("apply_filters(api_string, *filters)", open=False):
             api_string = gr.Textbox(label="Base API string", placeholder="e.g. https://api.openf1.org/v1/sessions?")
             filters = gr.Textbox(label="Filters (comma-separated)", placeholder="e.g. driver_number=16&,session_key=123&")
             btn = gr.Button("Apply filters")
             output = gr.Textbox(label="Full API string")
-            def _apply_filters(api_string, filters):
-                # Expect filters as comma-separated
-                filter_list = [f.strip() for f in filters.split(",") if f.strip()]
-                return openf1_tools.apply_filters(api_string, *filter_list)
-            btn.click(_apply_filters, inputs=[api_string, filters], outputs=output)
+            btn.click(openf1_tools.apply_filters, inputs=[api_string, filters], outputs=output)
         with gr.Accordion("send_request(api_string)", open=False):
             with gr.Accordion("Example API requests (copy & paste into text box below)", open=False):
                 gr.Markdown(MARKDOWN_OPENF1_EXAMPLES)
             api_string = gr.Textbox(label="Full API string", placeholder="e.g. https://api.openf1.org/v1/sessions?driver_number=16")
             btn = gr.Button("Send API request")
             output = gr.JSON()
-            def _send_request(api_string):
-                try:
-                    return openf1_tools.send_request(api_string)
-                except Exception as e:
-                    return {"error": str(e)}
-            btn.click(_send_request, inputs=api_string, outputs=output)
+            btn.click(openf1_tools.send_request, inputs=api_string, outputs=output)
     return openf1_tools_tab
 
 # OpenF1 tabs

todo.txt

@@ -1,4 +1,6 @@
 - For driver championship standings, make the Driver dropdown depend on the selected season. As it is implemented now one could not use it for older seasons!
     * Solution is to have a static json file that maps drivers for each season/year
 - Same applies for constructor championship standings but instead Constructor dropdown
-    * Similar solution to above
+    * Similar solution to above
+- Add driver comparison
+- simplify the mcp_client.py to single call.

utils/constants.py

@@ -106,7 +106,8 @@ Each of these endpoints have different **_filters_** that can be used to filter
 
 The implemented functions make it possible to:
 - Get all available endpoints - `get_api_endpoints()`
-- Get information about a specific endpoint - `get_api_endpoint(endpoint)`
+- Get api string for a specific endpoint - `get_api_endpoint(endpoint)`
+- Get details about a specific endpoint - `get_endpoint_info(endpoint)`
 - Get information about a specific filter - `get_filter_info(filter_name)`
 - Get a filter string for a specific filter - `get_filter_string(filter_name, filter_value, operator)`
 - Apply filters to an API string - `apply_filters(api_string, *filters)`