Skip to content

fix: extend AI assistant provider timeout - #42096

Merged
tomjose92 merged 1 commit into
releasefrom
fix/ai-assistant-provider-timeout
Aug 6, 2026
Merged

fix: extend AI assistant provider timeout#42096
tomjose92 merged 1 commit into
releasefrom
fix/ai-assistant-provider-timeout

Conversation

@tomjose92

@tomjose92 tomjose92 commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • raise the remote AI provider response timeout from 60 seconds to 180 seconds
  • use one shared timeout for Claude, OpenAI, custom provider URLs, and Azure OpenAI
  • allow larger AI Assistant generations to finish instead of failing at the one-minute mark

The Local LLM client keeps its existing 120-second timeout.

Testing

  • server Spotless/pre-commit checks passed across all modules
  • CI server tests

Linear

https://linear.app/appsmith/issue/APP-15735/replace-internal-appsmith-app-with-native-co-pilot-component

Automation

/ok-to-test tags="@tag.Widget, @tag.IDE"

🔍 Cypress test results

Tip

🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/31090811946
Commit: cbbb640
Cypress dashboard.
Tags: @tag.Widget, @tag.IDE
Spec:


Thu, 06 Aug 2026 11:03:38 UTC

Summary by CodeRabbit

  • Bug Fixes
    • Increased the AI provider response timeout to 180 seconds.
    • Improved reliability for Claude, OpenAI, custom endpoint, and Azure OpenAI requests that require additional processing time.

@tomjose92
tomjose92 requested a review from a team as a code owner August 6, 2026 09:50
@tomjose92 tomjose92 added ok-to-test Required label for CI Bug Something isn't working labels Aug 6, 2026
@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

The AI assistant service adds a shared 180-second provider response timeout. Claude, OpenAI, custom-base-URL, and Azure OpenAI clients use this timeout instead of 60 seconds.

Changes

AI provider timeout

Layer / File(s) Summary
Shared timeout wiring
app/server/appsmith-server/src/main/java/com/appsmith/server/services/ce/AIAssistantServiceCEImpl.java
Defines the shared 180-second timeout and applies it to Claude, OpenAI, custom-base-URL, and Azure OpenAI clients.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers: subrata71

Poem

Longer waits now guide the call,
Claude and OpenAI heed them all.
Azure joins the patient queue,
One shared timeout makes it true.
Slow responses get room to bloom.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly and concisely describes the main change: extending the AI assistant provider timeout.
Description check ✅ Passed The description explains the change, scope, testing, and related Linear issue, but it omits the required Communication section.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/ai-assistant-provider-timeout

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@tomjose92
tomjose92 merged commit a6e465a into release Aug 6, 2026
50 of 53 checks passed
@tomjose92
tomjose92 deleted the fix/ai-assistant-provider-timeout branch August 6, 2026 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working ok-to-test Required label for CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants