Skip to content

Add LoadSpec spanning API#4353

Merged
lbwexler merged 10 commits into
developfrom
spanLoad
Apr 29, 2026
Merged

Add LoadSpec spanning API#4353
lbwexler merged 10 commits into
developfrom
spanLoad

Conversation

@lbwexler

@lbwexler lbwexler commented Apr 28, 2026

Copy link
Copy Markdown
Member

Summary

  • New LoadSpecConfig.span to let callers explicitly seed the parent trace context for a load.
  • FetchService falls back to opts.loadSpec?.span when opts.span is not provided, so fetches inherit the active load's span automatically.
  • Docs updated: CHANGELOG, svc/README.md (TraceService section), v85 upgrade notes.

+ better handling of self-referential span
+ fix some linting issues
# Conflicts:
#	CHANGELOG.md
#	svc/FetchService.ts
#	svc/README.md
#	svc/TraceService.ts
@lbwexler lbwexler merged commit 30de834 into develop Apr 29, 2026
3 checks passed
@lbwexler lbwexler deleted the spanLoad branch April 29, 2026 12:49
lbwexler added a commit to xh/toolbox that referenced this pull request Apr 29, 2026
## Summary

Adopts the new hoist-react `loadSpec` opt-in (xh/hoist-react#4353) in
three example models.

Pairs with hoist-react PR xh/hoist-react#4353.
haynesjm42 pushed a commit that referenced this pull request Jun 4, 2026
* Enhance FetchService to auto create spans

* + Bugfix for path tag
+ better handling of self-referential span
+ fix some linting issues

* new LoadSpec spanning API

* Tweaks from ATM code review

* Tweaks from ATM code review

* Remove support for automatically generated spans

* Remove support for automatically generated spans

* Remove support for automatically generated spans
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