Skip to content

Conversation

ulugbekna
Copy link
Contributor

not well tested due to 404 from the model

@ulugbekna ulugbekna self-assigned this Oct 6, 2025
@ulugbekna ulugbekna marked this pull request as ready for review October 6, 2025 15:39
@Copilot Copilot AI review requested due to automatic review settings October 6, 2025 15:39
@ulugbekna ulugbekna enabled auto-merge October 6, 2025 15:40
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for next cursor line prediction to the NES (Next Edit Suggestions) feature. When no suggestions are found during inline editing, the system now attempts to predict where the developer is most likely to make their next edit by analyzing the current context and prompting a specialized model.

  • Introduces next cursor position prediction functionality with configurable model endpoints
  • Refactors code organization by extracting a CurrentDocument class and utility functions
  • Updates prompt construction to support line numbers for better cursor prediction accuracy

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/platform/inlineEdits/common/statelessNextEditProvider.ts Added new cancellation reason for language context await phase
src/platform/configuration/common/configurationService.ts Added configuration keys for cursor prediction feature (enabled flag, model name, URL, API key)
src/extension/xtab/node/xtabProvider.ts Main implementation of cursor prediction logic, code refactoring, and integration with existing edit flow
src/extension/xtab/common/xtabCurrentDocument.ts New utility class to encapsulate document state and cursor position information
src/extension/xtab/common/promptCrafting.ts Refactored tag creation and extracted clipping functionality for reuse

@vs-code-engineering vs-code-engineering bot added this to the October 2025 milestone Oct 6, 2025
connor4312
connor4312 previously approved these changes Oct 6, 2025
roblourens
roblourens previously approved these changes Oct 6, 2025
@ulugbekna ulugbekna added this pull request to the merge queue Oct 6, 2025
@ulugbekna ulugbekna removed this pull request from the merge queue due to a manual request Oct 6, 2025
@ulugbekna ulugbekna force-pushed the ulugbekna/nes-next-cursor-prediction branch from 5fa0b05 to 18c8547 Compare October 6, 2025 20:32
@ulugbekna ulugbekna enabled auto-merge October 6, 2025 20:34
@ulugbekna ulugbekna added this pull request to the merge queue Oct 7, 2025
Merged via the queue into main with commit 44e806b Oct 7, 2025
16 checks passed
@ulugbekna ulugbekna deleted the ulugbekna/nes-next-cursor-prediction branch October 7, 2025 09:09
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.

4 participants