Skip to content

Conversation

QianKuang8
Copy link
Contributor

@QianKuang8 QianKuang8 commented Sep 21, 2025

Description

There is a misalignment in the timeout handling logic for code autocompletion between the intellij and core.
The intellij utilizes modelTimeout as a strict, overall timeout for the entire process.
The core, however, sets the autocomplete timeout to 2.5 * modelTimeout, with the intention of displaying partial results after the initial modelTimeout is reached.

Here propose to increase the intellij's timeout to ensure it would not break before core

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-general-review or @continue-detailed-review

Checklist

  • [] I've read the contributing guide
  • [] The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Screen recording or screenshot

[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]

Tests

[ What tests were added or updated to ensure the changes work as expected? ]


Summary by cubic

Increase IntelliJ autocomplete timeout to 3× modelTimeout to match core behavior and prevent premature cancellation. This ensures partial results can return instead of failing early when responses take longer.

@QianKuang8 QianKuang8 requested a review from a team as a code owner September 21, 2025 09:29
@QianKuang8 QianKuang8 requested review from sestinj and removed request for a team September 21, 2025 09:29
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Sep 21, 2025
Copy link
Contributor

@sestinj sestinj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, thanks for the PR!

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Sep 22, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 22, 2025
@sestinj sestinj merged commit 295427c into continuedev:main Sep 22, 2025
58 of 59 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Sep 22, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Sep 22, 2025
@sestinj
Copy link
Contributor

sestinj commented Sep 22, 2025

🎉 This PR is included in version 1.18.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm This PR has been approved by a maintainer released size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants