Skip to content

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

Merged
barredterra merged 1 commit into
developfrom
mergify/bp/develop/pr-83
Jul 30, 2026
Merged

fix: use responses API (backport #83)#85
barredterra merged 1 commit into
developfrom
mergify/bp/develop/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)
@mergify mergify Bot mentioned this pull request Jul 30, 2026
@greptile-apps

greptile-apps Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Confidence Score: 5/5

The PR appears safe to merge, with the intentional Responses API compatibility requirement clearly documented.

The dependency upgrade accompanies the new model option, both model-call paths use the same output extraction contract, and the changed tests cover Responses API activation and text-only rendering without revealing reasoning metadata.

Sequence Diagram

sequenceDiagram
    participant User
    participant Agent as Ask ALYF Agent
    participant LC as LangChain ChatOpenAI
    participant API as Responses API
    User->>Agent: Send chat or field-writing request
    Agent->>LC: Invoke configured model
    LC->>API: POST /v1/responses
    API-->>LC: Reasoning and text content blocks
    LC-->>Agent: AIMessage
    Agent->>Agent: Read AIMessage.text
    Agent-->>User: Final answer text only
Loading

Reviews (1): Last reviewed commit: "fix: use responses API (#83)" | Re-trigger Greptile

@barredterra
barredterra merged commit 8dd98a7 into develop Jul 30, 2026
5 checks passed
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