Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhancements and fixes to autocomplete behaviour #4509

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

bdavj
Copy link

@bdavj bdavj commented Mar 6, 2025

Description

Autocomplete behaviour enhancements and fixes to remedy issues with repeat requests and timeouts (#4413).

  • Re-surfacing control within the user configuration GUI for debounce delay
  • Adding control within user configuration for autocomplete model timeout
  • Fixing issues around debounce delay whereby

Screenshots

Screenshot 2025-03-06 at 11 18 39

Testing Instructions

Configuration Verification

  • Open user configuration GUI
  • Verify presence of new debounce delay and timeout controls
  • Confirm settings are persisted after modification

Functionality Testing

  • Set various debounce delays and observe completion timing
  • Test with longer model timeouts for complex completions
  • Verify multi-line completions display correctly

Validation Steps

  • Monitor .continue/prompt.log for completion responses
  • Confirm completions only trigger after configured debounce delay
  • Verify all lines of multi-line completions appear in IDE

Edge Cases

  • Test with minimal and maximum timeout values
  • Verify behavior with rapid consecutive typing
  • Check completion behavior across different file types

Related Issues

…lay and model timeout in configuration GUI, fix to debounce delay handling
Copy link

netlify bot commented Mar 6, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit 24a97ba
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/67dd8b5e2c6fe70007a55eba

Copy link
Collaborator

@tomasz-stefaniak tomasz-stefaniak left a comment

Choose a reason for hiding this comment

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

Thank you for contributing @bdavj! We've had this on our to-do list for a very long time, so I appreciate you taking care of this!

@bdavj bdavj requested a review from a team as a code owner March 21, 2025 15:52
@bdavj bdavj requested review from tomasz-stefaniak and removed request for a team March 21, 2025 15:52
@bdavj
Copy link
Author

bdavj commented Mar 21, 2025

Thanks @tomasz-stefaniak - seeing (what looks like?) an environmental test failure. Is this something external to the PR? Is there an updated dependency / configuration i need to merge back in? Thanks

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.

MultiLine Autocompletion not working even though model is providing multi line responses
2 participants