Skip to content

Commit 106f6ea

Browse files
authored
config(amazonq): temporarily disable cursorUpdateManager.ts (#7576)
## Problem Will revisit this feature later once we have other pieces working first ## Solution --- - Treat all work as PUBLIC. Private `feature/x` branches will not be squash-merged at release time. - Your code changes must meet the guidelines in [CONTRIBUTING.md](https://github.com/aws/aws-toolkit-vscode/blob/master/CONTRIBUTING.md#guidelines). - License: I confirm that my contribution is made under the terms of the Apache 2.0 license.
1 parent a4ab56c commit 106f6ea

File tree

1 file changed

+0
-21
lines changed

1 file changed

+0
-21
lines changed

packages/amazonq/src/lsp/client.ts

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ import { SessionManager } from '../app/inline/sessionManager'
5050
import { LineTracker } from '../app/inline/stateTracker/lineTracker'
5151
import { InlineTutorialAnnotation } from '../app/inline/tutorials/inlineTutorialAnnotation'
5252
import { InlineChatTutorialAnnotation } from '../app/inline/tutorials/inlineChatTutorialAnnotation'
53-
import { CursorUpdateManager } from '../app/inline/cursorUpdateManager'
5453

5554
const localize = nls.loadMessageBundle()
5655
const logger = getLogger('amazonqLsp.lspClient')
@@ -247,42 +246,22 @@ async function onLanguageServerReady(
247246
await activate(client, encryptionKey, resourcePaths.ui)
248247
}
249248

250-
// Get the provider from the manager
251-
const inlineCompletionProvider = inlineManager.getInlineCompletionProvider()
252-
253249
const refreshInterval = auth.startTokenRefreshInterval(10 * oneSecond)
254250

255-
// Create and start the CursorUpdateManager with the provider from InlineManager
256-
const cursorUpdateManager = new CursorUpdateManager(client, inlineCompletionProvider)
257-
258251
// We manually push the cached values the first time since event handlers, which should push, may not have been setup yet.
259252
// Execution order is weird and should be fixed in the flare implementation.
260253
// TODO: Revisit if we need this if we setup the event handlers properly
261254
if (AuthUtil.instance.isConnectionValid()) {
262255
await sendProfileToLsp(client)
263256

264-
// Start the CursorUpdateManager
265-
await cursorUpdateManager.start()
266-
267257
await pushConfigUpdate(client, {
268258
type: 'customization',
269259
customization: getSelectedCustomization(),
270260
})
271261
}
272262

273-
// Track cursor position changes
274-
const cursorUpdateDisposable = vscode.window.onDidChangeTextEditorSelection(
275-
(e: vscode.TextEditorSelectionChangeEvent) => {
276-
if (e.textEditor === vscode.window.activeTextEditor) {
277-
cursorUpdateManager.updatePosition(e.selections[0].active, e.textEditor.document.uri.toString())
278-
}
279-
}
280-
)
281-
282263
toDispose.push(
283264
inlineManager,
284-
cursorUpdateManager, // Add CursorUpdateManager to disposables
285-
cursorUpdateDisposable,
286265
Commands.register({ id: 'aws.amazonq.invokeInlineCompletion', autoconnect: true }, async () => {
287266
await vscode.commands.executeCommand('editor.action.inlineSuggest.trigger')
288267
}),

0 commit comments

Comments
 (0)