Flight Log / getsentry/sentry-python #4476
fix(langchain): Make `span_map` an instance variable — a Cursor (Anysphere) flight by Daniel Szoke
Daniel Szoke (@szokeasaurusrex) opened this Cursor (Anysphere) pull request in getsentry/sentry-python on Jun 16, 2025; merged clean in 8.8 d, merged by @szokeasaurusrex: +13 −4, 2 files, 2 reviews.
Flown by
- Lines
- +13 −4
- Files
- 2
- Commits
- 1
- Reviews
- 2
- To merge
- 8.8 d
- Task
- fix
Line and file counts are summed per commit from the AIDev dataset; GitHub's net diff can be smaller.
Cursor's fix in getsentry/sentry-python makes span_map an instance variable, so separate SentryLangchainCallback instances no longer share one object. It also removes the max_span_map_size class variable. The change spans two files in one commit, with 13 additions and 4 deletions. It merges clean, with zero change requests, after 212.31 hours.
From the pull request description
`span_map` should be an instance variable; otherwise, separate instances of the `SentryLangchainCallback` share the same `span_map` object, which is clearly not intended here. Also, remove the `max_span_map_size` class variable, it is always set on the instance, and so not needed. Ref #4443
View the pull request on GitHub →
This is your flight? Claim it — sign in with GitHub. Ownership is proven by the sign-in: the account that opened the PR is the only one that can claim it.
Flight timeline
- —reviewed
- —reviewed
- —committedCursor Agent
- Jun 25, 2025mergedszokeasaurusrex
- Jun 25, 2025closedszokeasaurusrex
Read next
Cite this flight: AgentRiders Flight Log, getsentry/sentry-python#4476, record updated Nov 4, 2026, https://agentriders.com/logs/getsentry/sentry-python/4476/
Data from public GitHub activity; this page is not affiliated with GitHub or the repository.