Skip to content

Automatic core spans from reactor events (CL-5171) - #305

Merged
TheGreatAxios merged 2 commits into
mainfrom
stack/w2a-cl-5171-auto-spans
Aug 4, 2026
Merged

Automatic core spans from reactor events (CL-5171)#305
TheGreatAxios merged 2 commits into
mainfrom
stack/w2a-cl-5171-auto-spans

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Stack

PerfTrace stack 2/8 — base: stack/w1a-cl-5160-perftrace-core (#303)

Linear

https://linear.app/abklabs/issue/CL-5171/automatic-core-spans-from-reactor-events

Summary

Wire automatic PerfTrace spans from reactor events (turn, inference, tools, etc.) so every session produces a span tree without manual instrumentation.

Test evidence

  • Reactor span tests green in worktree
  • Review PASS

Merge note

Merge after #303 (CL-5160).

@linear-code

linear-code Bot commented Aug 4, 2026

Copy link
Copy Markdown

CL-5171

@TheGreatAxios
TheGreatAxios force-pushed the stack/w2a-cl-5171-auto-spans branch from 1eb6a74 to ddb9c72 Compare August 4, 2026 16:21
@TheGreatAxios
TheGreatAxios force-pushed the stack/w1a-cl-5160-perftrace-core branch from f76e8cb to 8575c45 Compare August 4, 2026 18:10
@TheGreatAxios
TheGreatAxios force-pushed the stack/w2a-cl-5171-auto-spans branch from ddb9c72 to c3442c6 Compare August 4, 2026 18:10
Maps reactor events to nested perf spans: turn contains inference,
which contains inference.ttft and inference.stream when stream deltas
arrive; tool spans nest under turn. Wired always-on in createRunSink
alongside the existing turn collector so coarse durationMs is unchanged.
Interrupt mid-inference or mid-tool must not nest the next call under a
stale turn or leave orphan tool spans. Route all turn-close paths through
one closeTurn helper (closeOpenTools + end turn) and cover abandon
restart paths in tests.
@TheGreatAxios
TheGreatAxios force-pushed the stack/w2a-cl-5171-auto-spans branch from c3442c6 to 073d6de Compare August 4, 2026 22:42
@TheGreatAxios
TheGreatAxios changed the base branch from stack/w1a-cl-5160-perftrace-core to main August 4, 2026 22:42
@TheGreatAxios
TheGreatAxios merged commit 02c9207 into main Aug 4, 2026
1 check passed
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