fix: backend environment variable trumps default (#165)

Author: mathislucka

src/deepset_mcp/main.py

@@ -96,13 +96,13 @@ def main(
         ),
     ] = TransportEnum.STDIO,
     object_store_backend: Annotated[
-        str,
+        str | None,
         typer.Option(
             "--object-store-backend",
             help="Object store backend type: 'memory' or 'redis'. "
-            "Can also be set via DEEPSET_OBJECT_STORE_BACKEND environment variable.",
+            "Can also be set via DEEPSET_OBJECT_STORE_BACKEND environment variable. Default is 'memory'.",
         ),
-    ] = "memory",
+    ] = None,
     redis_url: Annotated[
         str | None,
         typer.Option(
@@ -153,7 +153,7 @@ def main(
     docs_share_url = docs_share_url or os.getenv("DEEPSET_DOCS_SHARE_URL", DEEPSET_DOCS_DEFAULT_SHARE_URL)
 
     # ObjectStore configuration
-    backend = object_store_backend or os.getenv("DEEPSET_OBJECT_STORE_BACKEND", "memory")
+    backend = str(object_store_backend or os.getenv("DEEPSET_OBJECT_STORE_BACKEND", "memory"))
     redis_url = redis_url or os.getenv("DEEPSET_REDIS_URL")
     ttl = int(os.getenv("DEEPSET_OBJECT_STORE_TTL", str(object_store_ttl)))
 

src/deepset_mcp/tools/tokonomics/explorer.py

@@ -288,7 +288,8 @@ def _make_header(self, obj_id: str, path: str, obj: Any) -> str:
         if hasattr(obj, "__module__") and obj.__module__ not in ("builtins", "__main__"):
             type_name = f"{obj.__module__}.{type_name}"
 
-        location = f"@{obj_id}" + (f".{path}" if path else "")
+        location = "@" if not obj_id.startswith("@") else ""
+        location += obj_id + (f".{path}" if path else "")
 
         # Add size info for sized objects
         size_info = ""