Skip to content

fix: use responses API (backport #83) - #84

Merged
barredterra merged 3 commits into
version-15from
mergify/bp/version-15/pr-83
Jul 30, 2026
Merged

fix: use responses API (backport #83)#84
barredterra merged 3 commits into
version-15from
mergify/bp/version-15/pr-83

Conversation

@mergify

@mergify mergify Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Use the OpenAI Responses API for model calls and upgrade langchain-openai, avoiding model-specific handling for reasoning models with tools. Extract output through AIMessage.text so structured reasoning metadata never appears in chat responses.


This is an automatic backport of pull request #83 done by Mergify.

(cherry picked from commit 05bdd55)

# Conflicts:
#	ask_alyf/ask_alyf/test_field_agent.py
@mergify

This comment was marked as resolved.

@mergify mergify Bot added the conflicts label Jul 30, 2026
@mergify mergify Bot mentioned this pull request Jul 30, 2026
Comment thread ask_alyf/ask_alyf/test_field_agent.py Outdated
@greptile-apps

greptile-apps Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains; the previously reported conflict markers have been removed from the current field-agent test module.

Sequence Diagram

sequenceDiagram
  participant Client
  participant AskALYF
  participant ChatOpenAI
  participant ResponsesAPI as OpenAI Responses API
  Client->>AskALYF: Send chat message
  AskALYF->>ChatOpenAI: Invoke agent with message history
  ChatOpenAI->>ResponsesAPI: POST /v1/responses
  ResponsesAPI-->>ChatOpenAI: AIMessage with response blocks
  ChatOpenAI-->>AskALYF: Agent result messages
  AskALYF->>AskALYF: Extract AIMessage.text
  AskALYF-->>Client: Final answer text
Loading

Reviews (3): Last reviewed commit: "style: remove blank line" | Re-trigger Greptile

Removed unused import of UnitTestCase from frappe.tests.
@barredterra
barredterra merged commit ed18f2a into version-15 Jul 30, 2026
5 checks passed
@barredterra
barredterra deleted the mergify/bp/version-15/pr-83 branch July 30, 2026 21:24
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