Skip to content

Commit fd2b188

Browse files
committed
Revert "feat: add optional logfire instrumentation for cloud mode distributed tracing"
This reverts commit 1fa93ec.
1 parent 453cba9 commit fd2b188

3 files changed

Lines changed: 0 additions & 233 deletions

File tree

pyproject.toml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,6 @@ dependencies = [
3737
"aiofiles>=24.1.0", # Async file I/O
3838
]
3939

40-
[project.optional-dependencies]
41-
cloud = [
42-
"logfire>=4.10.0", # Distributed tracing for cloud mode
43-
]
4440

4541
[project.urls]
4642
Homepage = "https://github.com/basicmachines-co/basic-memory"

src/basic_memory/api/app.py

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -31,17 +31,6 @@ async def lifespan(app: FastAPI): # pragma: no cover
3131
app_config = ConfigManager().config
3232
logger.info("Starting Basic Memory API")
3333

34-
# Instrument httpx client for distributed tracing when in cloud mode
35-
if app_config.cloud_mode_enabled:
36-
try:
37-
import logfire # pyright: ignore[reportMissingImports]
38-
from basic_memory.mcp.async_client import client as httpx_client
39-
40-
logger.info("Cloud mode enabled - instrumenting httpx client for distributed tracing")
41-
logfire.instrument_httpx(client=httpx_client)
42-
except ImportError:
43-
logger.warning("logfire not available - skipping httpx instrumentation")
44-
4534
await initialize_app(app_config)
4635

4736
# Cache database connections in app state for performance

0 commit comments

Comments
 (0)