Skip to content

feat: add retriever, trace and session - #267

Open
namrataghadi-galileo wants to merge 1 commit into
mainfrom
feature/SAO-18190-support-trace-session-retriever-spans
Open

namrataghadi-galileo wants to merge 1 commit into
mainfrom
feature/SAO-18190-support-trace-session-retriever-spans

Conversation

@namrataghadi-galileo

@namrataghadi-galileo namrataghadi-galileo commented Oct 8, 2026 •

Copy link
Copy Markdown
Collaborator

Agent Control events can apply to LLM calls, tool calls, retriever calls, traces, or sessions. The SDK previously accepted only llm_call and tool_call, so the other supported values could fail enum conversion before a control span was emitted. This change adds retriever_call, trace_call, and session_call while preserving the existing values, extends the control span model and OpenAPI contract, and preserves applies_to and trace/session/parent correlation through OTEL conversion.

Tests

  • Added coverage for all five supported applies_to values in the bridge and OTEL conversion paths.
  • Focused tests: poetry run pytest tests/test_agent_control.py tests/test_agent_control_bridge.py tests/test_attribute_mapping.py tests/test_span_converter.py -q (133 passed).

@github-actions

github-actions Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@namrataghadi-galileo

Copy link
Copy Markdown
Collaborator Author

I have read the CLA Document and I hereby sign the CLA

@namrataghadi-galileo

Copy link
Copy Markdown
Collaborator Author

recheck

@namrataghadi-galileo

Copy link
Copy Markdown
Collaborator Author

/recheck

@namrataghadi-galileo
namrataghadi-galileo enabled auto-merge (squash) October 9, 2026 17:08
@namrataghadi-galileo

namrataghadi-galileo commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator Author

I have read the CLA Document and I hereby accept the CLA

This branch has not been deployed

No deployments
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.

3 participants