Skip to content

fix: use responses API - #83

Merged
barredterra merged 3 commits into
version-16from
responses-api
Jul 30, 2026
Merged

fix: use responses API#83
barredterra merged 3 commits into
version-16from
responses-api

Conversation

@barredterra

@barredterra barredterra commented Jul 30, 2026

Copy link
Copy Markdown
Member

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.

@barredterra
barredterra changed the base branch from develop to version-16 July 30, 2026 20:32
@alyf-de alyf-de deleted a comment from greptile-apps Bot Jul 30, 2026
@barredterra barredterra changed the title responses api fix: use responses API Jul 30, 2026
@barredterra

Copy link
Copy Markdown
Member Author

@greptileai

@barredterra

Copy link
Copy Markdown
Member Author

@Mergifyio backport version-15 develop

@mergify

mergify Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

backport version-15 develop

✅ Backports have been created

Details

Cherry-pick of 05bdd55 has failed:

On branch mergify/bp/version-15/pr-83
Your branch is up to date with 'origin/version-15'.

You are currently cherry-picking commit 05bdd55.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   README.md
	modified:   ask_alyf/ask_alyf/agent.py
	modified:   ask_alyf/ask_alyf/field_agent.py
	modified:   ask_alyf/ask_alyf/test_code_tools.py
	modified:   pyproject.toml

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   ask_alyf/ask_alyf/test_field_agent.py

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

Comment thread ask_alyf/ask_alyf/agent.py
@greptile-apps

greptile-apps Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Confidence Score: 4/5

The PR does not appear safe to merge while existing OpenAI Compatible configurations can still be routed to an unsupported Responses endpoint.

The model builder unconditionally enables Responses API routing for both OpenAI and OpenAI Compatible providers, so configured servers that only implement Chat Completions still fail on invocation; documenting that limitation does not preserve the existing provider contract.

Files Needing Attention: ask_alyf/ask_alyf/agent.py, ask_alyf/ask_alyf/field_agent.py

Sequence Diagram

sequenceDiagram
    participant User
    participant Agent as Ask ALYF Agent
    participant Model as ChatOpenAI
    participant Provider as OpenAI-compatible Provider
    User->>Agent: Submit chat or field request
    Agent->>Model: "Build with use_responses_api=true"
    Model->>Provider: POST /v1/responses
    Provider-->>Model: Responses payload
    Model-->>Agent: AIMessage
    Agent->>Agent: Extract AIMessage.text
    Agent-->>User: Final answer text
Loading

Reviews (2): Last reviewed commit: "docs: require OpenAI Responses API" | Re-trigger Greptile

@barredterra
barredterra merged commit 05bdd55 into version-16 Jul 30, 2026
5 checks passed
@barredterra
barredterra deleted the responses-api branch July 30, 2026 20:54
@barredterra

Copy link
Copy Markdown
Member Author

@Mergifyio backport version-15 develop

@mergify

mergify Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

backport version-15 develop

☑️ Command backport version-15 develop ignored because it is already running from a previous command.

barredterra added a commit that referenced this pull request Jul 30, 2026
Co-authored-by: Raffael Meyer <14891507+barredterra@users.noreply.github.com>
barredterra added a commit that referenced this pull request Jul 30, 2026
Co-authored-by: Raffael Meyer <14891507+barredterra@users.noreply.github.com>
@barredterra barredterra mentioned this pull request Jul 31, 2026
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