pydantic/logfire

logfire run suggests adding `opentelemetry-instrumentation-` packages instead of dependency groups

Open

#1,297 opened on Aug 5, 2025

View on GitHub
 (1 comment) (0 reactions) (0 assignees)Python (244 forks)github user discovery
Feature Requestgood first issue

Repository metrics

Stars
 (4,282 stars)
PR merge metrics
 (Avg merge 7d 8h) (35 merged PRs in 30d)

Description

Instead of this:


╭──────────────────────────────────────────────────────────────── Logfire Summary ────────────────────────────────────────────────────────────────╮
│                                                                                                                                                 │
│  Your instrumentation checklist:                                                                                                                │
│                                                                                                                                                 │
│  ☐ requests (need to install opentelemetry-instrumentation-requests)                                                                            │
│  ☐ sqlite3 (need to install opentelemetry-instrumentation-sqlite3)                                                                              │
│  ☐ urllib (need to install opentelemetry-instrumentation-urllib)                                                                                │
│                                                                                                                                                 │
│                                                                                                                                                 │
│  To install all recommended packages at once, run:                                                                                              │
│                                                                                                                                                 │
│  uv add opentelemetry-instrumentation-requests opentelemetry-instrumentation-sqlite3 opentelemetry-instrumentation-urllib                       │
│                                                                                                                                                 │
│  ─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────  │
│                                                                                                                                                 │
│  To hide this summary box, use: logfire run --no-summary.                                                                                       │
│                                                                                                                                                 │
╰─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯

I'd like to see uv add logfire[requests,sqlite3,urllib] for those packages we have dependency groups for. I think we should have dependency groups for all instrumentation packages we recommend.

If we detect we were run using uvx logfire run (can we?), it would be good to suggest re-running with uvx --from 'logfire[requests,sqlite3,urllib]' logfire run.

If we detect we're run with uv run --with logfire logfire run (can we?), we should suggest uv run --with 'logfire[requests,sqlite3,urllib]' logfire run.

If we can't detect exactly how we're run, as is likely, it may be worth listing that they should install the logfire[requests,sqlite3,urllib] package and then suggest all three methods.

Contributor guide