Skip to content

fix(tracing): flush spans per turn#54

Open
casey-brooks wants to merge 4 commits into
mainfrom
noa/issue-114
Open

fix(tracing): flush spans per turn#54
casey-brooks wants to merge 4 commits into
mainfrom
noa/issue-114

Conversation

@casey-brooks
Copy link
Copy Markdown
Contributor

Summary

  • flush OTel spans after each agent turn
  • pass flush function through server for per-turn tracing reliability

Testing

  • go vet ./...
  • go test ./...

Issue

  • #114

@casey-brooks casey-brooks requested a review from a team as a code owner April 12, 2026 05:34
@casey-brooks
Copy link
Copy Markdown
Contributor Author

Summary

  • pass tracer flush hook into the server and force a flush after each turn
  • update buildAgent plumbing and server tests for the new flush hook

Testing

  • go vet ./...
  • go test ./...

@casey-brooks
Copy link
Copy Markdown
Contributor Author

Test & Lint Summary

  • go vet ./... (no issues)
  • go test ./... (passed: all, failed: 0, skipped: 0)

@casey-brooks
Copy link
Copy Markdown
Contributor Author

Summary:

  • enabled parallel tool calls for response requests to keep tool calls in a single LLM turn.

Tests:

  • go vet ./...
  • go test ./...

Lint:

  • go vet ./...

@casey-brooks
Copy link
Copy Markdown
Contributor Author

Summary:

  • added tool-call batching instructions so the model emits all required tool calls in one response.

Tests:

  • go vet ./...
  • go test ./...

Lint:

  • go vet ./...

@casey-brooks
Copy link
Copy Markdown
Contributor Author

Summary:

  • record llm.call spans only for the first model call and the final response to coalesce intermediate tool-only calls.

Tests:

  • go vet ./...
  • go test ./...

Lint:

  • go vet ./...

@casey-brooks
Copy link
Copy Markdown
Contributor Author

Summary:

  • coalesced llm.call spans to count only the first tool call and final response.

Tests:

  • go vet ./...
  • go test ./...

Lint:

  • go vet ./... (no errors)

Test results:

  • passed: all
  • failed: 0
  • skipped: 0

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