Skip to content

Adopt CallContext threading and spanPrefix#847

Open
lbwexler wants to merge 6 commits into
developfrom
spanItAll
Open

Adopt CallContext threading and spanPrefix#847
lbwexler wants to merge 6 commits into
developfrom
spanItAll

Conversation

@lbwexler
Copy link
Copy Markdown
Member

@lbwexler lbwexler commented May 4, 2026

Companion to xh/hoist-react#4360.

API Changes

  • AuthModel.completeAuthAsync now accepts and forwards CallContext from the framework's auth flow.
  • Adopted spanPrefix on Toolbox services and example panel models (Portfolio, Docs, GitHub, Contacts, News, Recalls, Weather, FileManager, Roadmap, ViewManagerTest) — span name strings trimmed to their local segment. Wire-format span names unchanged.
  • Threaded CallContext through service signatures where applicable.

@lbwexler lbwexler requested a review from amcclain May 4, 2026 13:47
lbwexler and others added 3 commits May 8, 2026 16:31
Tracks API tightening in hoist-react PR #4360:
- `spanPrefix` → `telemetryPrefix` on all model/service overrides.
- `Runner.track` → `withTrack` on builder chains.
- `Runner.fetchJson` → `runFetchJson` on builder chains.

Folded post-fetch work that mutates or caches results (ContactService,
DocService) into the `.run()` lambda so it's covered by the runner's span.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant