[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

jupyter_server/extension/manager.py

@@ -5,17 +5,15 @@
 import importlib
 from itertools import starmap
 
+import jsonschema
 from tornado.gen import multi
 from traitlets import Any, Bool, Dict, HasTraits, Instance, List, Unicode, default, observe
 from traitlets import validate as validate_trait
 from traitlets.config import LoggingConfigurable
-import jsonschema
-
 
 from .config import ExtensionConfigManager
 from .utils import ExtensionMetadataError, ExtensionModuleNotFound, get_loader, get_metadata
 
-
 # probably this should go in it's own file? Not sure where though
 MCP_TOOL_SCHEMA = {
     "type": "object",
@@ -27,9 +25,9 @@
             "properties": {
                 "type": {"type": "string", "enum": ["object"]},
                 "properties": {"type": "object"},
-                "required": {"type": "array", "items": {"type": "string"}}
+                "required": {"type": "array", "items": {"type": "string"}},
             },
-            "required": ["type", "properties"]
+            "required": ["type", "properties"],
         },
         "annotations": {
             "type": "object",
@@ -38,15 +36,16 @@
                 "readOnlyHint": {"type": "boolean"},
                 "destructiveHint": {"type": "boolean"},
                 "idempotentHint": {"type": "boolean"},
-                "openWorldHint": {"type": "boolean"}
+                "openWorldHint": {"type": "boolean"},
             },
-            "additionalProperties": True
-        }
+            "additionalProperties": True,
+        },
     },
     "required": ["name", "inputSchema"],
-    "additionalProperties": False
+    "additionalProperties": False,
 }
 
+
 class ExtensionPoint(HasTraits):
     """A simple API for connecting to a Jupyter Server extension
     point defined by metadata and importable from a Python package.
@@ -129,9 +128,9 @@ def name(self):
     def module(self):
         """The imported module (using importlib.import_module)"""
         return self._module
-    
+
     @property
-    def tools(self): 
+    def tools(self):
         """Structured tools exposed by this extension point, if any."""
         loc = self.app or self.module
         if not loc:
@@ -149,7 +148,7 @@ def tools(self):
                 jsonschema.validate(instance=tool["metadata"], schema=MCP_TOOL_SCHEMA)
                 tools[name] = tool
         except Exception as e:
-            # not sure if this should fail quietly, raise an error, or log it? 
+            # not sure if this should fail quietly, raise an error, or log it?
             print(f"[tool-discovery] Failed to load tools from {self.module_name}: {e}")
         return tools
 
@@ -167,7 +166,6 @@ def _get_linker(self):
             )
         return linker
 
-
     def _get_loader(self):
         """Get a loader."""
         loc = self.app
@@ -500,7 +498,6 @@ def load_all_extensions(self):
         for name in self.sorted_extensions:
             self.load_extension(name)
 
-    
     def get_tools(self) -> Dict[str, Any]:
         """Aggregate tools from all extensions that expose them."""
         all_tools = {}

jupyter_server/serverapp.py

@@ -2540,8 +2540,8 @@ def load_server_extensions(self) -> None:
         """
         self.extension_manager.load_all_extensions()
 
-    # is this how I would expose it? and Is this a good name? 
-    def get_tools(self): 
+    # is this how I would expose it? and Is this a good name?
+    def get_tools(self):
         return self.extension_manager.get_tools()
 
     def init_mime_overrides(self) -> None:

jupyter_server/services/contents/handlers.py

@@ -423,12 +423,13 @@ async def post(self, path=""):
         self.set_status(201)
         self.finish()
 
-# Somehow this doesn't feel like the right service for this to go in? 
+
+# Somehow this doesn't feel like the right service for this to go in?
 class ListToolInfoHandler(APIHandler):
     @web.authenticated
     async def get(self):
         tools = self.serverapp.extension_manager.discover_tools()
-        self.finish({"discovered_tools": tools}) 
+        self.finish({"discovered_tools": tools})
 
 
 # -----------------------------------------------------------------------------
@@ -449,5 +450,4 @@ async def get(self):
     (r"/api/contents%s" % path_regex, ContentsHandler),
     (r"/api/notebooks/?(.*)", NotebooksRedirectHandler),
     (r"/api/tools", ListToolInfoHandler),
-
 ]