- 
                Notifications
    You must be signed in to change notification settings 
- Fork 3.7k
feat: instant edit for find/replace tools #8473
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
Conversation
| ✅ Review Complete Code Review Summary | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 issue found across 10 files
Prompt for AI agents (all 1 issues)
Understand the root cause of the following 1 issues and fix them.
<file name="extensions/intellij/src/main/kotlin/com/github/continuedev/continueintellijextension/editor/DiffStreamHandler.kt">
<violation number="1" location="extensions/intellij/src/main/kotlin/com/github/continuedev/continueintellijextension/editor/DiffStreamHandler.kt:213">
Calling getLineStartOffset(currentLine) when currentLine equals document.lineCount (e.g., appending lines at EOF) throws IndexOutOfBoundsException; guard for this case before computing the offset.</violation>
</file>
React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.
        
          
                ...src/main/kotlin/com/github/continuedev/continueintellijextension/editor/DiffStreamHandler.kt
              
                Outdated
          
            Show resolved
            Hide resolved
        
      Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
| ApplicationManager.getApplication().invokeLater { | ||
| WriteCommandAction.runWriteCommandAction(project) { | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How confident are you that this is the right approach here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3/10
seems to work
| 🎉 This PR is included in version 1.31.0 🎉 The release is available on: Your semantic-release bot 📦🚀 | 
| 🎉 This PR is included in version 1.4.0 🎉 The release is available on: Your semantic-release bot 📦🚀 | 
| 🎉 This PR is included in version 1.28.0 🎉 The release is available on: Your semantic-release bot 📦🚀 | 
Description
We've been using diff streaming approaches built for when model streams an apply response in. Added utils for instant apply. See the before an after here:
instant-apply.mp4
And after for jetbrains
https://github.com/user-attachments/assets/21ea1ac1-9190-4356-bf91-16dcd0658db6
Summary by cubic
Instant apply for find/replace edits in VS Code and JetBrains. Diffs are computed once and applied immediately, replacing streaming for these tools.
New Features
Refactors