Revert "Remove old API in favor of this"

This reverts commit 9543501.

Author: Kartik Raj

src/client/interpreter/contracts.ts

@@ -3,7 +3,7 @@ import { CodeLensProvider, ConfigurationTarget, Disposable, Event, TextDocument,
 import { FileChangeType } from '../common/platform/fileSystemWatcher';
 import { Resource } from '../common/types';
 import { PythonEnvSource } from '../pythonEnvironments/base/info';
-import { ProgressNotificationEvent, PythonLocatorQuery } from '../pythonEnvironments/base/locator';
+import { PythonLocatorQuery } from '../pythonEnvironments/base/locator';
 import { CondaEnvironmentInfo } from '../pythonEnvironments/common/environmentManagers/conda';
 import { EnvironmentType, PythonEnvironment } from '../pythonEnvironments/info';
 
@@ -16,7 +16,7 @@ export type PythonEnvironmentsChangedEvent = {
 
 export const IComponentAdapter = Symbol('IComponentAdapter');
 export interface IComponentAdapter {
-    readonly onProgress: Event<ProgressNotificationEvent>;
+    readonly onRefreshStart: Event<void>;
     triggerRefresh(query?: PythonLocatorQuery & { clearCache?: boolean }, trigger?: 'auto' | 'ui'): Promise<void>;
     readonly refreshPromise: Promise<void> | undefined;
     readonly onChanged: Event<PythonEnvironmentsChangedEvent>;
@@ -63,6 +63,7 @@ export interface ICondaService {
 
 export const IInterpreterService = Symbol('IInterpreterService');
 export interface IInterpreterService {
+    readonly onRefreshStart: Event<void>;
     triggerRefresh(query?: PythonLocatorQuery & { clearCache?: boolean }, trigger?: 'auto' | 'ui'): Promise<void>;
     readonly refreshPromise: Promise<void> | undefined;
     readonly onDidChangeInterpreters: Event<PythonEnvironmentsChangedEvent>;

src/client/interpreter/display/progressDisplay.ts

@@ -12,7 +12,6 @@ import { IDisposableRegistry } from '../../common/types';
 import { createDeferred, Deferred } from '../../common/utils/async';
 import { Interpreters } from '../../common/utils/localize';
 import { traceDecoratorVerbose } from '../../logging';
-import { ProgressReportStage } from '../../pythonEnvironments/base/locator';
 import { IComponentAdapter } from '../contracts';
 
 // The parts of IComponentAdapter used here.
@@ -31,13 +30,11 @@ export class InterpreterLocatorProgressStatubarHandler implements IExtensionSing
     ) {}
 
     public async activate(): Promise<void> {
-        this.pyenvs.onProgress(
-            (event) => {
-                if (event.stage === ProgressReportStage.discoveryStarted) {
-                    this.showProgress();
-                    if (this.pyenvs.refreshPromise) {
-                        this.pyenvs.refreshPromise.then(() => this.hideProgress());
-                    }
+        this.pyenvs.onRefreshStart(
+            () => {
+                this.showProgress();
+                if (this.pyenvs.refreshPromise) {
+                    this.pyenvs.refreshPromise.then(() => this.hideProgress());
                 }
             },
             this,

src/client/interpreter/interpreterService.ts

@@ -40,6 +40,10 @@ export class InterpreterService implements Disposable, IInterpreterService {
         return this.pyenvs.hasInterpreters(filter);
     }
 
+    public get onRefreshStart(): Event<void> {
+        return this.pyenvs.onRefreshStart;
+    }
+
     public triggerRefresh(
         query?: PythonLocatorQuery & { clearCache?: boolean },
         trigger?: 'auto' | 'ui',

src/client/pythonEnvironments/api.ts

@@ -31,6 +31,10 @@ class PythonEnvironments implements IDiscoveryAPI {
         return this.locator.onProgress;
     }
 
+    public get onRefreshStart() {
+        return this.locator.onRefreshStart;
+    }
+
     public get refreshPromise() {
         return this.locator.refreshPromise;
     }

src/client/pythonEnvironments/base/locator.ts

@@ -188,9 +188,13 @@ export interface IResolvingLocator<I = PythonEnvInfo> extends IResolver, ILocato
 
 export interface IDiscoveryAPI {
     /**
-     * Fires with details of the current discovery progress, i.e when it starts, finishes or any other relevant stage.
+     * Fires when the known list of environments starts refreshing, i.e when discovery starts or restarts.
      */
     readonly onProgress: Event<ProgressNotificationEvent>;
+    /**
+     * Fires when the known list of environments starts refreshing, i.e when discovery starts or restarts.
+     */
+    readonly onRefreshStart: Event<void>;
     /**
      * Fires with details if the known list changes.
      */

src/client/pythonEnvironments/legacyIOC.ts

@@ -111,8 +111,8 @@ class ComponentAdapter implements IComponentAdapter {
         return this.api.refreshPromise;
     }
 
-    public get onProgress() {
-        return this.api.onProgress;
+    public get onRefreshStart(): vscode.Event<void> {
+        return this.api.onRefreshStart;
     }
 
     public get onChanged() {