Skip to content

Add displayLocation property and inlineEditSupport capability for tab-tab-tab workflow #492

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

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

Conversation

floralph
Copy link
Contributor

@floralph floralph commented May 4, 2025

Problem

The Next Edit Predictions (NEP) feature requires support for the tab-tab-tab workflow, which allows users to navigate through a series of related edit suggestions. However, the current implementation lacks the necessary properties and capability flags to support this workflow.

Solution

• Added the displayLocation property to InlineCompletionItemWithReferences with a concise comment explaining its purpose for tab-tab-tab workflow navigation
• Updated the comment for isInlineEdit to be more concise and focused on its functionality
• Added the inlineEditSupport capability to the awsClientCapabilities structure under textDocument.inlineCompletionWithReferences

These changes enable:

  1. Clients to declare support for inline edit features during initialization
  2. Servers to detect this capability and conditionally include the new properties
  3. Servers to decide whether to run the NEP code path based on client support

The implementation follows the LSP specification pattern for capability negotiation while maintaining backward compatibility with existing clients.

Testing

• Verified the changes compile without errors
• Confirmed the capability structure follows LSP conventions
• Validated the property definitions match the requirements from the NativeUI-POC implementation

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@floralph floralph requested a review from a team as a code owner May 4, 2025 06:56
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.

1 participant