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

use the current document to track range for persistence #5812

Merged
merged 1 commit into from
Oct 4, 2024

Conversation

hitesh-1997
Copy link
Contributor

Use the current document for tracking the insertText. Using the pre-calculated text was making the offline diffs inaccurate, as introduced in the PR.

Test plan

CI checks

@hitesh-1997 hitesh-1997 merged commit 21d11b0 into main Oct 4, 2024
19 checks passed
@hitesh-1997 hitesh-1997 deleted the hitesh/use-document-for-insert-range branch October 4, 2024 20:11
hitesh-1997 added a commit that referenced this pull request Oct 4, 2024
Use the current document for tracking the insertText. Using the
pre-calculated text was making the offline diffs inaccurate, as
introduced in the
[PR](https://github.com/sourcegraph/cody/pull/5767/files).

## Test plan
CI checks
hitesh-1997 added a commit that referenced this pull request Oct 4, 2024
hitesh-1997 added a commit that referenced this pull request Oct 5, 2024
Cherry-pick

- Autocomplete: re-enable the agent tests.
[pull/5784](#5784)
- Autocomplete: Using the current document instead of precalculated text
to get insertion text.
[pull/5812](#5812)

## Test plan
cherry-pick, updated changelog and version

---------

Co-authored-by: Valery Bugakov <skymk1@gmail.com>
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.

2 participants