Skip to content

feat: add interception tracing #63

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
pawbana merged 18 commits into from
Dec 5, 2025
Merged

feat: add interception tracing #63

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
5f8e36a
feat: add interception tracing
pawbana Nov 19, 2025
f608619
EndSpanErr span nil check + typo fix
pawbana Nov 25, 2025
5fe4137
fix context used by async recorder
pawbana Nov 26, 2025
597f3c0
ServerProxyManager init tracing
pawbana Nov 26, 2025
d5b0614
review 1: streaming upstream fix, tool attrs, request path
pawbana Nov 27, 2025
a863d67
fmt fix
pawbana Nov 27, 2025
5e81f3c
re-add tool_name attr to tool.Call + toolCount attr in fetchTools trace
pawbana Nov 28, 2025
1fd9095
test fix
pawbana Nov 28, 2025
7760d31
add tool call trace test, fixed openAI tool call attrs?
pawbana Nov 28, 2025
1ac08c1
rename aibridge -> tracing
pawbana Dec 3, 2025
454cd8b
traceNewStreaming rename
pawbana Dec 3, 2025
dc0ad44
fix openAI tool arguments consistency
pawbana Dec 3, 2025
3b3d540
fix bedrock streaming tests
pawbana Dec 4, 2025
feded7a
make argument order consistent
pawbana Dec 4, 2025
59934d9
Merge branch 'main' into pb/add-otel
pawbana Dec 4, 2025
6f1bc8c
review 2
pawbana Dec 4, 2025
2971b6d
Merge branch 'main' into pb/add-otel
pawbana Dec 5, 2025
ea859c0
go.mod comment
pawbana Dec 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions bridge_integration_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1214,6 +1214,7 @@ func TestErrorHandling(t *testing.T) {
resp, err := http.DefaultClient.Do(req)
t.Cleanup(func() { _ = resp.Body.Close() })
require.NoError(t, err)
bridgeSrv.Close()
Copy link
Contributor Author

@pawbana pawbana Nov 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should fix: https://github.com/coder/aibridge/actions/runs/19761723059/job/56625007286


tc.responseHandlerFn(resp)
recorderClient.verifyAllInterceptionsEnded(t)
Expand Down