Skip to content

Add dokimos-openai integration module#149

Merged
fkapsahili merged 3 commits into
masterfrom
feature/dokimos-openai
Jul 19, 2026
Merged

Add dokimos-openai integration module#149
fkapsahili merged 3 commits into
masterfrom
feature/dokimos-openai

Conversation

@fkapsahili

@fkapsahili fkapsahili commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

This PR adds the OpenAiAgentTraces bridge that lives right now in dokimos-examples to a new module, so when using the raw OpenAI Java SDK can easily evaluate agent traces by adding one dependency.

OpenAiSupport mirrors LangChain4jSupport. openai-java is provided-scope / bring your own version.

@github-actions

Copy link
Copy Markdown
Contributor

✅ Eval gate passed

metric value
pass rate 100% → 100% (0 pts)
significant false
regressed cases 0
improved cases 0
added / removed 0 / 0
pairing positional

Promote the OpenAiAgentTraces example bridge to a published dokimos-openai
module. OpenAiSupport provides asJudge plus toAgentTrace/toToolCalls/
toToolDefinitions over the official openai-java SDK, mirroring
dokimos-langchain4j, so raw-OpenAI JVM teams can evaluate agent traces by
adding one dependency. Updates the example, agent-evaluation docs, and the
evaluate-openai plugin skill to use it.
The longer walkthrough still taught the hand-rolled bridge; capture tool
calls with OpenAiSupport.toToolCall so it matches the compact tab and the
runnable example.
@fkapsahili
fkapsahili force-pushed the feature/dokimos-openai branch from 1da4ee0 to 4ba4196 Compare July 11, 2026 08:30
Drop the no-arg asJudge(client) overload and the DEFAULT_JUDGE_MODEL
constant so the adapter does not bake a default judge model into the
public API; callers pass the ChatModel they want.
@fkapsahili
fkapsahili merged commit f02f7cb into master Jul 19, 2026
21 checks passed
@fkapsahili
fkapsahili deleted the feature/dokimos-openai branch July 19, 2026 09:18
fkapsahili added a commit that referenced this pull request Jul 19, 2026
The `dokimos-openai` module merged in #149 but, unlike every other
adapter, had no dedicated `integrations/` page and no row in the
installation table, so it was undiscoverable from the docs nav.

Adds `integrations/openai.md` (setup, `asJudge`, capturing tool calls
into an `AgentTrace`, converting tool definitions, and evaluating the
trace) mirroring the sibling integration pages, and adds the
`dokimos-openai` row to the installation table. Docs build clean.
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