cleanups

sourcegraph · Jan 6, 2025 · 0f22a0e · 0f22a0e
1 parent bee12bc
commit 0f22a0e
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/vscode/src/autoedits/create-autoedits-provider.ts b/vscode/src/autoedits/create-autoedits-provider.ts
@@ -5,31 +5,41 @@ import {
     type AuthenticatedAuthStatus,
     type ChatClient,
     NEVER,
+    type PickResolvedConfiguration,
     type UnauthenticatedAuthStatus,
     createDisposables,
     promiseFactoryToObservable,
+    skipPendingOperation,
 } from '@sourcegraph/cody-shared'
 import { AutoeditsProvider } from './autoedits-provider'
 import { autoeditsOutputChannelLogger } from './output-channel-logger'
 
 interface AutoeditsItemProviderArgs {
+    config: PickResolvedConfiguration<{ configuration: true }>
     authStatus: UnauthenticatedAuthStatus | Pick<AuthenticatedAuthStatus, 'authenticated' | 'endpoint'>
     chatClient: ChatClient
 }
 
 export function createAutoEditsProvider({
+    config: { configuration },
     authStatus,
     chatClient,
 }: AutoeditsItemProviderArgs): Observable<void> {
+    if (!configuration.experimentalAutoeditsEnabled) {
+        return NEVER
+    }
+
     if (!authStatus.authenticated) {
         if (!authStatus.pendingValidation) {
             autoeditsOutputChannelLogger.logDebug('createProvider', 'You are not signed in.')
         }
         return NEVER
     }
+
     return promiseFactoryToObservable(async () => {
         return await getAutoeditsProviderDocumentFilters()
     }).pipe(
+        skipPendingOperation(),
         createDisposables(documentFilters => {
             const provider = new AutoeditsProvider(chatClient)
             return [

diff --git a/vscode/src/main.ts b/vscode/src/main.ts
@@ -718,11 +718,11 @@ function registerAutoEdits(chatClient: ChatClient, disposables: vscode.Disposabl
                         if (!shouldEnableExperimentalAutoedits(config, autoeditEnabled, authStatus)) {
                             return NEVER
                         }
-                        const res = createAutoEditsProvider({
+                        return createAutoEditsProvider({
+                            config,
                             authStatus,
                             chatClient,
                         })
-                        return res
                     }),
                     catchError(error => {
                         logError('registerAutoedits', 'Error', error)