@microsoft/vscode-azext-azureauth 6.0.0-alpha.6 → 6.0.0-alpha.7

package/dist/cjs/src/providers/AzureSubscriptionProviderBase.js

@@ -51,7 +51,7 @@ const Limiter_1 = require("../utils/Limiter");
 const NotSignedInError_1 = require("../utils/NotSignedInError");
 const screen_1 = require("../utils/screen");
 const tryGetTokenExpiration_1 = require("../utils/tryGetTokenExpiration");
-const EventDebounce = 5 * 1000; // 5 seconds minimum between `onRefreshSuggested` events
+const EventDebounce = 2 * 1000; // 2 seconds minimum between `onRefreshSuggested` events
 const EventSilenceTime = 5 * 1000; // 5 seconds after sign-in to silence `onRefreshSuggested` events
 const TenantListConcurrency = 3; // We will try to list tenants for at most 3 accounts in parallel
 const SubscriptionListConcurrency = 5; // We will try to list subscriptions for at most 5 account+tenants in parallel
@@ -94,7 +94,11 @@ class AzureSubscriptionProviderBase {
         return this.refreshSuggestedEmitter.event(callback, thisArg, disposables);
     }
     fireRefreshSuggestedIfNeeded(evtArgs) {
-        if (this.suppressRefreshSuggestedEvents || Date.now() < this.lastRefreshSuggestedTime + EventDebounce) {
+        // subscriptionFilterChange is an explicit user action and must never be suppressed,
+        // otherwise re-selecting a subscription shortly after unselecting one gets swallowed
+        // by the debounce/silence window that the first refresh triggered.
+        if (evtArgs.reason !== 'subscriptionFilterChange' &&
+            (this.suppressRefreshSuggestedEvents || Date.now() < this.lastRefreshSuggestedTime + EventDebounce)) {
             // Suppress and/or debounce events to avoid flooding
             return false;
         }

package/dist/esm/src/contracts/AzureSubscriptionProvider.d.ts

@@ -10,7 +10,7 @@ export interface AzureSubscriptionProvider {
     /**
      * Fires when the list of available subscriptions may have changed, and a refresh is suggested.
      * The callback will be given a {@link RefreshSuggestedEvent} with more information.
-     * @note This will be fired at most every 5 seconds, to avoid flooding. It is also suppressed
+     * @note This will be fired at most every 2 seconds, to avoid flooding. It is also suppressed
      * during operations this provider is performing itself, such as during sign-in.
      */
     onRefreshSuggested: vscode.Event<RefreshSuggestedEvent>;

package/dist/esm/src/providers/AzureSubscriptionProviderBase.js

@@ -15,7 +15,7 @@ import { Limiter } from '../utils/Limiter';
 import { isNotSignedInError, NotSignedInError } from '../utils/NotSignedInError';
 import { screen } from '../utils/screen';
 import { tryGetTokenExpiration } from '../utils/tryGetTokenExpiration';
-const EventDebounce = 5 * 1000; // 5 seconds minimum between `onRefreshSuggested` events
+const EventDebounce = 2 * 1000; // 2 seconds minimum between `onRefreshSuggested` events
 const EventSilenceTime = 5 * 1000; // 5 seconds after sign-in to silence `onRefreshSuggested` events
 const TenantListConcurrency = 3; // We will try to list tenants for at most 3 accounts in parallel
 const SubscriptionListConcurrency = 5; // We will try to list subscriptions for at most 5 account+tenants in parallel
@@ -58,7 +58,11 @@ export class AzureSubscriptionProviderBase {
         return this.refreshSuggestedEmitter.event(callback, thisArg, disposables);
     }
     fireRefreshSuggestedIfNeeded(evtArgs) {
-        if (this.suppressRefreshSuggestedEvents || Date.now() < this.lastRefreshSuggestedTime + EventDebounce) {
+        // subscriptionFilterChange is an explicit user action and must never be suppressed,
+        // otherwise re-selecting a subscription shortly after unselecting one gets swallowed
+        // by the debounce/silence window that the first refresh triggered.
+        if (evtArgs.reason !== 'subscriptionFilterChange' &&
+            (this.suppressRefreshSuggestedEvents || Date.now() < this.lastRefreshSuggestedTime + EventDebounce)) {
             // Suppress and/or debounce events to avoid flooding
             return false;
         }

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "@microsoft/vscode-azext-azureauth",
     "author": "Microsoft Corporation",
-    "version": "6.0.0-alpha.6",
+    "version": "6.0.0-alpha.7",
     "description": "Azure authentication helpers for Visual Studio Code",
     "tags": [
         "azure",