@gitkraken/core-gitlens 0.5.106 → 0.5.108

package/dist/plus/integrations/reads/sweeps.js

@@ -6,65 +6,129 @@ import { isGitHostIntegration, isIssuesHostIntegrationId } from '../utils/integr
 import { drainPullRequests, getCurrentAccountId, resolvePullRequestSweepTargets } from './drains.js';
 import { resolveAccountWidePullRequestFilters, resolvePullRequestFilters } from './filters.js';
 import { gitHostOnlySurfaceWarning, noConnectionWarning, unsupportedAccountWidePullRequestFiltersWarning, unsupportedFiltersWarning, } from './warnings.js';
+/**
+ * Drain ONE sweep target. Extracted from the fan-out so the fan-out callback has a single exit: every
+ * per-target observation (see `onTargetSettled`) is then reported in one place instead of at each of this
+ * function's several early returns, where a missed branch would silently drop a provider's attribution.
+ *
+ * `undefined` means the target resolved to no reachable connection and is deliberately not attributed in the
+ * aggregate result.
+ */
+async function sweepTarget(ctx, options, target, attributeUnavailableProviders) {
+    const { providerId: id, connectionId, domain: requestedDomain } = target;
+    const repos = options?.repos ?? [];
+    const maxPages = options?.maxPages ?? 100;
+    /** A target that never reached a drain: the provider itself failed, so its slice is empty and attributed. */
+    const rejectedTarget = (warnings) => ({
+        items: [],
+        warnings: warnings,
+        fetchFailed: true,
+        truncated: false,
+        providerId: id,
+        failedProvider: true,
+    });
+    if (isIssuesHostIntegrationId(id)) {
+        return rejectedTarget([gitHostOnlySurfaceWarning(id, requestedDomain, connectionId, 'pull request sweeps')]);
+    }
+    const integration = await ctx.getIntegrationForRead(id, connectionId, requestedDomain);
+    if (integration == null) {
+        // A requested connection that can't be resolved is a broken connection — surface it as a
+        // warning + fetchFailed rather than dropping the provider's slice silently.
+        const early = ctx.earlyReturnConnectionWarnings(id, connectionId, requestedDomain);
+        if (early.warnings.length === 0 && !attributeUnavailableProviders)
+            return undefined;
+        return rejectedTarget(early.warnings.length !== 0 ? early.warnings : [noConnectionWarning(id, requestedDomain, connectionId)]);
+    }
+    if (!isGitHostIntegration(integration)) {
+        return rejectedTarget([gitHostOnlySurfaceWarning(id, requestedDomain, connectionId, 'pull request sweeps')]);
+    }
+    await ctx.forceRefreshIfRequested(integration, options?.forceSync, connectionId);
+    const domain = ctx.domainForRead(integration, id, connectionId, requestedDomain);
+    const accountWide = repos.length === 0;
+    const requestedFilters = target.filters ?? options?.filters;
+    const resolved = accountWide
+        ? resolveAccountWidePullRequestFilters(id, requestedFilters)
+        : resolvePullRequestFilters(id, requestedFilters);
+    if (resolved.unsupported) {
+        return rejectedTarget([
+            accountWide
+                ? unsupportedAccountWidePullRequestFiltersWarning(id, domain, connectionId, requestedFilters ?? [])
+                : unsupportedFiltersWarning(id, domain, connectionId),
+        ]);
+    }
+    const drain = await drainPullRequests(integration, id, domain, repos, options?.states, resolved.filters, accountWide ? (options?.includeReviewRequested ?? false) : false, connectionId, maxPages, attributeUnavailableProviders);
+    const currentAccountId = drain.items.some(pr => pr.author != null)
+        ? await getCurrentAccountId(integration, connectionId)
+        : undefined;
+    // Normalize the raw provider-apis PRs to the GitLens-owned shape here, where the per-provider
+    // `integration` (the mapper's provider reference) is in scope; the aggregation below only sees drains.
+    return {
+        ...drain,
+        items: drain.items.map(pr => fromProviderPullRequest(pr, integration, { currentAccountId: currentAccountId })),
+        providerId: id,
+    };
+}
+/**
+ * How a target ended, as the consumer buckets it. `failedProvider` outranks `fetchFailed` because a target
+ * whose provider failed produced no slice to be partial about, and `undefined` is the target that resolved to
+ * no reachable connection at all.
+ */
+function sliceOutcome(slice) {
+    if (slice == null)
+        return 'skipped';
+    if (slice.failedProvider)
+        return 'failed-provider';
+    if (slice.fetchFailed)
+        return 'fetch-failed';
+    return 'ok';
+}
+/**
+ * Per-target reporting for a sweep's `onTargetSettled`, opened once per sweep: this stamps the fan-out start,
+ * the returned function stamps one target's start, and the function IT returns is the only thing that can
+ * report that target. The timestamps therefore never cross a boundary, so there is no pair of interchangeable
+ * numbers for a call site to transpose, and the whole reporting concern is one value the fan-out either has or
+ * does not. That is also what keeps the option free when it is omitted: with no observer there is nothing to
+ * open, so no clock is read and no reporting object exists — do not "simplify" that into an unconditional
+ * open, because a host whose perf gate is off (the default) is entitled to pay nothing for it.
+ *
+ * The try/catch is what makes the observer observation-only: called from the fan-out's success path, a throwing
+ * callback would otherwise propagate out of the `mapBounded` task and reject the entire sweep — corrupting the
+ * read, not just the metric. Swallowed silently; the consumer owns its own aggregation.
+ *
+ * The domain is resolved here rather than carried out of {@link sweepTarget}, so every target reports it by the
+ * same rule no matter how far it got. `resolveDomainForRead` needs no integration instance, which is what makes
+ * that possible: a target rejected by the first guard resolves the same host a fully drained one does.
+ */
+function startSweepReporting(ctx, observe) {
+    const fanOutStartedAt = performance.now();
+    return function beginTarget(target) {
+        const startedAt = performance.now();
+        return function reportSettled(slice) {
+            try {
+                observe({
+                    providerId: target.providerId,
+                    domain: ctx.resolveDomainForRead(target.providerId, target.connectionId, target.domain),
+                    connectionId: target.connectionId,
+                    count: slice?.items.length ?? 0,
+                    durationMs: performance.now() - startedAt,
+                    queueWaitMs: startedAt - fanOutStartedAt,
+                    outcome: sliceOutcome(slice),
+                    truncated: slice?.truncated ?? false,
+                });
+            }
+            catch { }
+        };
+    };
+}
 export async function sweepPullRequests(ctx, options) {
     const { targets, attributeUnavailableProviders } = resolvePullRequestSweepTargets(options);
-    const maxPages = options?.maxPages ?? 100;
-    const repos = options?.repos ?? [];
+    const observe = options?.onTargetSettled;
+    const beginTarget = observe != null ? startSweepReporting(ctx, observe) : undefined;
     const results = await mapBounded(targets, providerFanOutConcurrency, async (target) => {
-        const { providerId: id, connectionId, domain: requestedDomain } = target;
-        /** A target that never reached a drain: the provider itself failed, so its slice is empty and attributed. */
-        const rejectedTarget = (warnings) => ({
-            items: [],
-            warnings: warnings,
-            fetchFailed: true,
-            truncated: false,
-            providerId: id,
-            failedProvider: true,
-        });
-        if (isIssuesHostIntegrationId(id)) {
-            return rejectedTarget([
-                gitHostOnlySurfaceWarning(id, requestedDomain, connectionId, 'pull request sweeps'),
-            ]);
-        }
-        const integration = await ctx.getIntegrationForRead(id, connectionId, requestedDomain);
-        if (integration == null) {
-            // A requested connection that can't be resolved is a broken connection — surface it as a
-            // warning + fetchFailed rather than dropping the provider's slice silently.
-            const early = ctx.earlyReturnConnectionWarnings(id, connectionId, requestedDomain);
-            if (early.warnings.length === 0 && !attributeUnavailableProviders)
-                return undefined;
-            return rejectedTarget(early.warnings.length !== 0 ? early.warnings : [noConnectionWarning(id, requestedDomain, connectionId)]);
-        }
-        if (!isGitHostIntegration(integration)) {
-            return rejectedTarget([
-                gitHostOnlySurfaceWarning(id, requestedDomain, connectionId, 'pull request sweeps'),
-            ]);
-        }
-        await ctx.forceRefreshIfRequested(integration, options?.forceSync, connectionId);
-        const domain = ctx.domainForRead(integration, id, connectionId, requestedDomain);
-        const accountWide = repos.length === 0;
-        const requestedFilters = target.filters ?? options?.filters;
-        const resolved = accountWide
-            ? resolveAccountWidePullRequestFilters(id, requestedFilters)
-            : resolvePullRequestFilters(id, requestedFilters);
-        if (resolved.unsupported) {
-            return rejectedTarget([
-                accountWide
-                    ? unsupportedAccountWidePullRequestFiltersWarning(id, domain, connectionId, requestedFilters ?? [])
-                    : unsupportedFiltersWarning(id, domain, connectionId),
-            ]);
-        }
-        const drain = await drainPullRequests(integration, id, domain, repos, options?.states, resolved.filters, accountWide ? (options?.includeReviewRequested ?? false) : false, connectionId, maxPages, attributeUnavailableProviders);
-        const currentAccountId = drain.items.some(pr => pr.author != null)
-            ? await getCurrentAccountId(integration, connectionId)
-            : undefined;
-        // Normalize the raw provider-apis PRs to the GitLens-owned shape here, where the per-provider
-        // `integration` (the mapper's provider reference) is in scope; the aggregation below only sees drains.
-        return {
-            ...drain,
-            items: drain.items.map(pr => fromProviderPullRequest(pr, integration, { currentAccountId: currentAccountId })),
-            providerId: id,
-        };
+        const reportSettled = beginTarget?.(target);
+        const slice = await sweepTarget(ctx, options, target, attributeUnavailableProviders);
+        reportSettled?.(slice);
+        return slice;
     });
     const items = [];
     const warnings = [];

package/dist/plus/integrations/reads/sweeps.js.map

@@ -1 +1 @@
-{"version":3,"file":"sweeps.js","sourceRoot":"","sources":["../../../../src/plus/integrations/reads/sweeps.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,UAAU,EAAE,MAAM,2BAA2B,CAAC;AAEvD,OAAO,EAAE,yBAAyB,EAAE,MAAM,iBAAiB,CAAC;AAE5D,OAAO,EAAE,uBAAuB,EAAE,MAAM,wBAAwB,CAAC;AAEjE,OAAO,EAAE,oBAAoB,EAAE,MAAM,eAAe,CAAC;AACrD,OAAO,EAAE,oBAAoB,EAAE,yBAAyB,EAAE,MAAM,+BAA+B,CAAC;AAEhG,OAAO,EAAE,iBAAiB,EAAE,mBAAmB,EAAE,8BAA8B,EAAE,MAAM,aAAa,CAAC;AACrG,OAAO,EAAE,oCAAoC,EAAE,yBAAyB,EAAE,MAAM,cAAc,CAAC;AAC/F,OAAO,EACN,yBAAyB,EACzB,mBAAmB,EACnB,+CAA+C,EAC/C,yBAAyB,GACzB,MAAM,eAAe,CAAC;AAsBvB,MAAM,CAAC,KAAK,UAAU,iBAAiB,CACtC,GAAwB,EACxB,OAAiC;IAEjC,MAAM,EAAE,OAAO,EAAE,6BAA6B,EAAE,GAAG,8BAA8B,CAAC,OAAO,CAAC,CAAC;IAC3F,MAAM,QAAQ,GAAG,OAAO,EAAE,QAAQ,IAAI,GAAG,CAAC;IAC1C,MAAM,KAAK,GAAG,OAAO,EAAE,KAAK,IAAI,EAAE,CAAC;IAEnC,MAAM,OAAO,GAAG,MAAM,UAAU,CAAC,OAAO,EAAE,yBAAyB,EAAE,KAAK,EAAC,MAAM,EAAC,EAAE;QACnF,MAAM,EAAE,UAAU,EAAE,EAAE,EAAE,YAAY,EAAE,MAAM,EAAE,eAAe,EAAE,GAAG,MAAM,CAAC;QACzE,6GAA6G;QAC7G,MAAM,cAAc,GAAG,CAAC,QAA2B,EAAc,EAAE,CAAC,CAAC;YACpE,KAAK,EAAE,EAAE;YACT,QAAQ,EAAE,QAAQ;YAClB,WAAW,EAAE,IAAI;YACjB,SAAS,EAAE,KAAK;YAChB,UAAU,EAAE,EAAE;YACd,cAAc,EAAE,IAAI;SACpB,CAAC,CAAC;QAEH,IAAI,yBAAyB,CAAC,EAAE,CAAC,EAAE,CAAC;YACnC,OAAO,cAAc,CAAC;gBACrB,yBAAyB,CAAC,EAAE,EAAE,eAAe,EAAE,YAAY,EAAE,qBAAqB,CAAC;aACnF,CAAC,CAAC;QACJ,CAAC;QAED,MAAM,WAAW,GAAG,MAAM,GAAG,CAAC,qBAAqB,CAAC,EAAE,EAAE,YAAY,EAAE,eAAe,CAAC,CAAC;QACvF,IAAI,WAAW,IAAI,IAAI,EAAE,CAAC;YACzB,yFAAyF;YACzF,4EAA4E;YAC5E,MAAM,KAAK,GAAG,GAAG,CAAC,6BAA6B,CAAC,EAAE,EAAE,YAAY,EAAE,eAAe,CAAC,CAAC;YACnF,IAAI,KAAK,CAAC,QAAQ,CAAC,MAAM,KAAK,CAAC,IAAI,CAAC,6BAA6B;gBAAE,OAAO,SAAS,CAAC;YAEpF,OAAO,cAAc,CACpB,KAAK,CAAC,QAAQ,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,mBAAmB,CAAC,EAAE,EAAE,eAAe,EAAE,YAAY,CAAC,CAAC,CACvG,CAAC;QACH,CAAC;QACD,IAAI,CAAC,oBAAoB,CAAC,WAAW,CAAC,EAAE,CAAC;YACxC,OAAO,cAAc,CAAC;gBACrB,yBAAyB,CAAC,EAAE,EAAE,eAAe,EAAE,YAAY,EAAE,qBAAqB,CAAC;aACnF,CAAC,CAAC;QACJ,CAAC;QAED,MAAM,GAAG,CAAC,uBAAuB,CAAC,WAAW,EAAE,OAAO,EAAE,SAAS,EAAE,YAAY,CAAC,CAAC;QAEjF,MAAM,MAAM,GAAG,GAAG,CAAC,aAAa,CAAC,WAAW,EAAE,EAAE,EAAE,YAAY,EAAE,eAAe,CAAC,CAAC;QACjF,MAAM,WAAW,GAAG,KAAK,CAAC,MAAM,KAAK,CAAC,CAAC;QACvC,MAAM,gBAAgB,GAAG,MAAM,CAAC,OAAO,IAAI,OAAO,EAAE,OAAO,CAAC;QAC5D,MAAM,QAAQ,GAAG,WAAW;YAC3B,CAAC,CAAC,oCAAoC,CAAC,EAAE,EAAE,gBAAgB,CAAC;YAC5D,CAAC,CAAC,yBAAyB,CAAC,EAAE,EAAE,gBAAgB,CAAC,CAAC;QACnD,IAAI,QAAQ,CAAC,WAAW,EAAE,CAAC;YAC1B,OAAO,cAAc,CAAC;gBACrB,WAAW;oBACV,CAAC,CAAC,+CAA+C,CAAC,EAAE,EAAE,MAAM,EAAE,YAAY,EAAE,gBAAgB,IAAI,EAAE,CAAC;oBACnG,CAAC,CAAC,yBAAyB,CAAC,EAAE,EAAE,MAAM,EAAE,YAAY,CAAC;aACtD,CAAC,CAAC;QACJ,CAAC;QAED,MAAM,KAAK,GAAG,MAAM,iBAAiB,CACpC,WAAW,EACX,EAAE,EACF,MAAM,EACN,KAAK,EACL,OAAO,EAAE,MAAM,EACf,QAAQ,CAAC,OAAO,EAChB,WAAW,CAAC,CAAC,CAAC,CAAC,OAAO,EAAE,sBAAsB,IAAI,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,EAChE,YAAY,EACZ,QAAQ,EACR,6BAA6B,CAC7B,CAAC;QACF,MAAM,gBAAgB,GAAG,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,CAAC,MAAM,IAAI,IAAI,CAAC;YACjE,CAAC,CAAC,MAAM,mBAAmB,CAAC,WAAW,EAAE,YAAY,CAAC;YACtD,CAAC,CAAC,SAAS,CAAC;QACb,8FAA8F;QAC9F,uGAAuG;QACvG,OAAO;YACN,GAAG,KAAK;YACR,KAAK,EAAE,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,CAC3B,uBAAuB,CAAC,EAAE,EAAE,WAAW,EAAE,EAAE,gBAAgB,EAAE,gBAAgB,EAAE,CAAC,CAChF;YACD,UAAU,EAAE,EAAE;SACd,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,MAAM,KAAK,GAAuB,EAAE,CAAC;IACrC,MAAM,QAAQ,GAAsB,EAAE,CAAC;IACvC,MAAM,iBAAiB,GAAG,IAAI,GAAG,EAAkB,CAAC;IACpD,MAAM,qBAAqB,GAAG,IAAI,GAAG,EAAkB,CAAC;IACxD,IAAI,WAAW,GAAG,KAAK,CAAC;IACxB,IAAI,SAAS,GAAG,KAAK,CAAC;IACtB,KAAK,MAAM,KAAK,IAAI,OAAO,EAAE,CAAC;QAC7B,IAAI,KAAK,IAAI,IAAI,EAAE,CAAC;YACnB,SAAS;QACV,CAAC;QAED,KAAK,CAAC,IAAI,CAAC,GAAG,KAAK,CAAC,KAAK,CAAC,CAAC;QAC3B,KAAK,MAAM,CAAC,IAAI,KAAK,CAAC,QAAQ,EAAE,CAAC;YAChC,oBAAoB,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC;QACnC,CAAC;QACD,IAAI,KAAK,CAAC,WAAW,EAAE,CAAC;YACvB,WAAW,GAAG,IAAI,CAAC;QACpB,CAAC;QACD,IAAI,KAAK,CAAC,cAAc,EAAE,CAAC;YAC1B,iBAAiB,CAAC,GAAG,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC;QACzC,CAAC;aAAM,IAAI,KAAK,CAAC,WAAW,IAAI,KAAK,CAAC,SAAS,EAAE,CAAC;YACjD,qBAAqB,CAAC,GAAG,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC;QAC7C,CAAC;QACD,IAAI,KAAK,CAAC,SAAS,EAAE,CAAC;YACrB,SAAS,GAAG,IAAI,CAAC;QAClB,CAAC;IACF,CAAC;IAED,OAAO;QACN,KAAK,EAAE,KAAK;QACZ,QAAQ,EAAE,QAAQ;QAClB,gGAAgG;QAChG,8FAA8F;QAC9F,mFAAmF;QACnF,IAAI,EAAE;YACL,WAAW,EAAE,CAAC;YACd,YAAY,EAAE,KAAK,CAAC,MAAM;YAC1B,QAAQ,EAAE,CAAC,SAAS,IAAI,CAAC,WAAW;YACpC,SAAS,EAAE,SAAS,IAAI,SAAS;SACjC;QACD,qGAAqG;QACrG,gGAAgG;QAChG,qGAAqG;QACrG,0EAA0E;QAC1E,OAAO,EAAE,KAAK;QACd,WAAW,EAAE,WAAW,IAAI,SAAS;QACrC,iBAAiB,EAAE,CAAC,GAAG,iBAAiB,CAAC;QACzC,qBAAqB,EAAE,CAAC,GAAG,qBAAqB,CAAC;KACjD,CAAC;AACH,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,uBAAuB,CACtC,GAAwB,EACxB,OAAuC;IAEvC,OAAO,iBAAiB,CAAC,GAAG,EAAE,EAAE,GAAG,OAAO,EAAE,MAAM,EAAE,CAAC,QAAQ,EAAE,QAAQ,CAAC,EAAE,CAAC,CAAC;AAC7E,CAAC"}
+{"version":3,"file":"sweeps.js","sourceRoot":"","sources":["../../../../src/plus/integrations/reads/sweeps.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,UAAU,EAAE,MAAM,2BAA2B,CAAC;AAEvD,OAAO,EAAE,yBAAyB,EAAE,MAAM,iBAAiB,CAAC;AAO5D,OAAO,EAAE,uBAAuB,EAAE,MAAM,wBAAwB,CAAC;AAEjE,OAAO,EAAE,oBAAoB,EAAE,MAAM,eAAe,CAAC;AACrD,OAAO,EAAE,oBAAoB,EAAE,yBAAyB,EAAE,MAAM,+BAA+B,CAAC;AAEhG,OAAO,EAAE,iBAAiB,EAAE,mBAAmB,EAAE,8BAA8B,EAAE,MAAM,aAAa,CAAC;AACrG,OAAO,EAAE,oCAAoC,EAAE,yBAAyB,EAAE,MAAM,cAAc,CAAC;AAC/F,OAAO,EACN,yBAAyB,EACzB,mBAAmB,EACnB,+CAA+C,EAC/C,yBAAyB,GACzB,MAAM,eAAe,CAAC;AAsBvB;;;;;;;GAOG;AACH,KAAK,UAAU,WAAW,CACzB,GAAwB,EACxB,OAA4C,EAC5C,MAA2B,EAC3B,6BAAsC;IAEtC,MAAM,EAAE,UAAU,EAAE,EAAE,EAAE,YAAY,EAAE,MAAM,EAAE,eAAe,EAAE,GAAG,MAAM,CAAC;IACzE,MAAM,KAAK,GAAG,OAAO,EAAE,KAAK,IAAI,EAAE,CAAC;IACnC,MAAM,QAAQ,GAAG,OAAO,EAAE,QAAQ,IAAI,GAAG,CAAC;IAC1C,6GAA6G;IAC7G,MAAM,cAAc,GAAG,CAAC,QAA2B,EAAc,EAAE,CAAC,CAAC;QACpE,KAAK,EAAE,EAAE;QACT,QAAQ,EAAE,QAAQ;QAClB,WAAW,EAAE,IAAI;QACjB,SAAS,EAAE,KAAK;QAChB,UAAU,EAAE,EAAE;QACd,cAAc,EAAE,IAAI;KACpB,CAAC,CAAC;IAEH,IAAI,yBAAyB,CAAC,EAAE,CAAC,EAAE,CAAC;QACnC,OAAO,cAAc,CAAC,CAAC,yBAAyB,CAAC,EAAE,EAAE,eAAe,EAAE,YAAY,EAAE,qBAAqB,CAAC,CAAC,CAAC,CAAC;IAC9G,CAAC;IAED,MAAM,WAAW,GAAG,MAAM,GAAG,CAAC,qBAAqB,CAAC,EAAE,EAAE,YAAY,EAAE,eAAe,CAAC,CAAC;IACvF,IAAI,WAAW,IAAI,IAAI,EAAE,CAAC;QACzB,yFAAyF;QACzF,4EAA4E;QAC5E,MAAM,KAAK,GAAG,GAAG,CAAC,6BAA6B,CAAC,EAAE,EAAE,YAAY,EAAE,eAAe,CAAC,CAAC;QACnF,IAAI,KAAK,CAAC,QAAQ,CAAC,MAAM,KAAK,CAAC,IAAI,CAAC,6BAA6B;YAAE,OAAO,SAAS,CAAC;QAEpF,OAAO,cAAc,CACpB,KAAK,CAAC,QAAQ,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,mBAAmB,CAAC,EAAE,EAAE,eAAe,EAAE,YAAY,CAAC,CAAC,CACvG,CAAC;IACH,CAAC;IACD,IAAI,CAAC,oBAAoB,CAAC,WAAW,CAAC,EAAE,CAAC;QACxC,OAAO,cAAc,CAAC,CAAC,yBAAyB,CAAC,EAAE,EAAE,eAAe,EAAE,YAAY,EAAE,qBAAqB,CAAC,CAAC,CAAC,CAAC;IAC9G,CAAC;IAED,MAAM,GAAG,CAAC,uBAAuB,CAAC,WAAW,EAAE,OAAO,EAAE,SAAS,EAAE,YAAY,CAAC,CAAC;IAEjF,MAAM,MAAM,GAAG,GAAG,CAAC,aAAa,CAAC,WAAW,EAAE,EAAE,EAAE,YAAY,EAAE,eAAe,CAAC,CAAC;IACjF,MAAM,WAAW,GAAG,KAAK,CAAC,MAAM,KAAK,CAAC,CAAC;IACvC,MAAM,gBAAgB,GAAG,MAAM,CAAC,OAAO,IAAI,OAAO,EAAE,OAAO,CAAC;IAC5D,MAAM,QAAQ,GAAG,WAAW;QAC3B,CAAC,CAAC,oCAAoC,CAAC,EAAE,EAAE,gBAAgB,CAAC;QAC5D,CAAC,CAAC,yBAAyB,CAAC,EAAE,EAAE,gBAAgB,CAAC,CAAC;IACnD,IAAI,QAAQ,CAAC,WAAW,EAAE,CAAC;QAC1B,OAAO,cAAc,CAAC;YACrB,WAAW;gBACV,CAAC,CAAC,+CAA+C,CAAC,EAAE,EAAE,MAAM,EAAE,YAAY,EAAE,gBAAgB,IAAI,EAAE,CAAC;gBACnG,CAAC,CAAC,yBAAyB,CAAC,EAAE,EAAE,MAAM,EAAE,YAAY,CAAC;SACtD,CAAC,CAAC;IACJ,CAAC;IAED,MAAM,KAAK,GAAG,MAAM,iBAAiB,CACpC,WAAW,EACX,EAAE,EACF,MAAM,EACN,KAAK,EACL,OAAO,EAAE,MAAM,EACf,QAAQ,CAAC,OAAO,EAChB,WAAW,CAAC,CAAC,CAAC,CAAC,OAAO,EAAE,sBAAsB,IAAI,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,EAChE,YAAY,EACZ,QAAQ,EACR,6BAA6B,CAC7B,CAAC;IACF,MAAM,gBAAgB,GAAG,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,CAAC,MAAM,IAAI,IAAI,CAAC;QACjE,CAAC,CAAC,MAAM,mBAAmB,CAAC,WAAW,EAAE,YAAY,CAAC;QACtD,CAAC,CAAC,SAAS,CAAC;IACb,8FAA8F;IAC9F,uGAAuG;IACvG,OAAO;QACN,GAAG,KAAK;QACR,KAAK,EAAE,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,CAAC,uBAAuB,CAAC,EAAE,EAAE,WAAW,EAAE,EAAE,gBAAgB,EAAE,gBAAgB,EAAE,CAAC,CAAC;QAC9G,UAAU,EAAE,EAAE;KACd,CAAC;AACH,CAAC;AAED;;;;GAIG;AACH,SAAS,YAAY,CAAC,KAA6B;IAClD,IAAI,KAAK,IAAI,IAAI;QAAE,OAAO,SAAS,CAAC;IACpC,IAAI,KAAK,CAAC,cAAc;QAAE,OAAO,iBAAiB,CAAC;IACnD,IAAI,KAAK,CAAC,WAAW;QAAE,OAAO,cAAc,CAAC;IAC7C,OAAO,IAAI,CAAC;AACb,CAAC;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,SAAS,mBAAmB,CAAC,GAAwB,EAAE,OAAkD;IACxG,MAAM,eAAe,GAAG,WAAW,CAAC,GAAG,EAAE,CAAC;IAE1C,OAAO,SAAS,WAAW,CAAC,MAA2B;QACtD,MAAM,SAAS,GAAG,WAAW,CAAC,GAAG,EAAE,CAAC;QAEpC,OAAO,SAAS,aAAa,CAAC,KAA6B;YAC1D,IAAI,CAAC;gBACJ,OAAO,CAAC;oBACP,UAAU,EAAE,MAAM,CAAC,UAAU;oBAC7B,MAAM,EAAE,GAAG,CAAC,oBAAoB,CAAC,MAAM,CAAC,UAAU,EAAE,MAAM,CAAC,YAAY,EAAE,MAAM,CAAC,MAAM,CAAC;oBACvF,YAAY,EAAE,MAAM,CAAC,YAAY;oBACjC,KAAK,EAAE,KAAK,EAAE,KAAK,CAAC,MAAM,IAAI,CAAC;oBAC/B,UAAU,EAAE,WAAW,CAAC,GAAG,EAAE,GAAG,SAAS;oBACzC,WAAW,EAAE,SAAS,GAAG,eAAe;oBACxC,OAAO,EAAE,YAAY,CAAC,KAAK,CAAC;oBAC5B,SAAS,EAAE,KAAK,EAAE,SAAS,IAAI,KAAK;iBACpC,CAAC,CAAC;YACJ,CAAC;YAAC,MAAM,CAAC,CAAA,CAAC;QACX,CAAC,CAAC;IACH,CAAC,CAAC;AACH,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,iBAAiB,CACtC,GAAwB,EACxB,OAAiC;IAEjC,MAAM,EAAE,OAAO,EAAE,6BAA6B,EAAE,GAAG,8BAA8B,CAAC,OAAO,CAAC,CAAC;IAE3F,MAAM,OAAO,GAAG,OAAO,EAAE,eAAe,CAAC;IACzC,MAAM,WAAW,GAAG,OAAO,IAAI,IAAI,CAAC,CAAC,CAAC,mBAAmB,CAAC,GAAG,EAAE,OAAO,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;IAEpF,MAAM,OAAO,GAAG,MAAM,UAAU,CAAC,OAAO,EAAE,yBAAyB,EAAE,KAAK,EAAC,MAAM,EAAC,EAAE;QACnF,MAAM,aAAa,GAAG,WAAW,EAAE,CAAC,MAAM,CAAC,CAAC;QAC5C,MAAM,KAAK,GAAG,MAAM,WAAW,CAAC,GAAG,EAAE,OAAO,EAAE,MAAM,EAAE,6BAA6B,CAAC,CAAC;QACrF,aAAa,EAAE,CAAC,KAAK,CAAC,CAAC;QACvB,OAAO,KAAK,CAAC;IACd,CAAC,CAAC,CAAC;IAEH,MAAM,KAAK,GAAuB,EAAE,CAAC;IACrC,MAAM,QAAQ,GAAsB,EAAE,CAAC;IACvC,MAAM,iBAAiB,GAAG,IAAI,GAAG,EAAkB,CAAC;IACpD,MAAM,qBAAqB,GAAG,IAAI,GAAG,EAAkB,CAAC;IACxD,IAAI,WAAW,GAAG,KAAK,CAAC;IACxB,IAAI,SAAS,GAAG,KAAK,CAAC;IACtB,KAAK,MAAM,KAAK,IAAI,OAAO,EAAE,CAAC;QAC7B,IAAI,KAAK,IAAI,IAAI,EAAE,CAAC;YACnB,SAAS;QACV,CAAC;QAED,KAAK,CAAC,IAAI,CAAC,GAAG,KAAK,CAAC,KAAK,CAAC,CAAC;QAC3B,KAAK,MAAM,CAAC,IAAI,KAAK,CAAC,QAAQ,EAAE,CAAC;YAChC,oBAAoB,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC;QACnC,CAAC;QACD,IAAI,KAAK,CAAC,WAAW,EAAE,CAAC;YACvB,WAAW,GAAG,IAAI,CAAC;QACpB,CAAC;QACD,IAAI,KAAK,CAAC,cAAc,EAAE,CAAC;YAC1B,iBAAiB,CAAC,GAAG,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC;QACzC,CAAC;aAAM,IAAI,KAAK,CAAC,WAAW,IAAI,KAAK,CAAC,SAAS,EAAE,CAAC;YACjD,qBAAqB,CAAC,GAAG,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC;QAC7C,CAAC;QACD,IAAI,KAAK,CAAC,SAAS,EAAE,CAAC;YACrB,SAAS,GAAG,IAAI,CAAC;QAClB,CAAC;IACF,CAAC;IAED,OAAO;QACN,KAAK,EAAE,KAAK;QACZ,QAAQ,EAAE,QAAQ;QAClB,gGAAgG;QAChG,8FAA8F;QAC9F,mFAAmF;QACnF,IAAI,EAAE;YACL,WAAW,EAAE,CAAC;YACd,YAAY,EAAE,KAAK,CAAC,MAAM;YAC1B,QAAQ,EAAE,CAAC,SAAS,IAAI,CAAC,WAAW;YACpC,SAAS,EAAE,SAAS,IAAI,SAAS;SACjC;QACD,qGAAqG;QACrG,gGAAgG;QAChG,qGAAqG;QACrG,0EAA0E;QAC1E,OAAO,EAAE,KAAK;QACd,WAAW,EAAE,WAAW,IAAI,SAAS;QACrC,iBAAiB,EAAE,CAAC,GAAG,iBAAiB,CAAC;QACzC,qBAAqB,EAAE,CAAC,GAAG,qBAAqB,CAAC;KACjD,CAAC;AACH,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,uBAAuB,CACtC,GAAwB,EACxB,OAAuC;IAEvC,OAAO,iBAAiB,CAAC,GAAG,EAAE,EAAE,GAAG,OAAO,EAAE,MAAM,EAAE,CAAC,QAAQ,EAAE,QAAQ,CAAC,EAAE,CAAC,CAAC;AAC7E,CAAC"}

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@gitkraken/core-gitlens",
   "description": "GitLens core — shared Git / AI / GitHub primitives for internal GitKraken consumption",
-  "version": "0.5.106",
+  "version": "0.5.108",
   "license": "SEE LICENSE IN LICENSE",
   "author": {
     "name": "GitKraken",

package/src/plus/git-github/api/github.ts

@@ -3510,6 +3510,15 @@ export class GitHubApi {
 		return `https://avatars.githubusercontent.com/u/e?email=${encodeURIComponent(email)}&s=${avatarSize}`;
 	}
 
+	/**
+	 * One page of the current user's pull requests, filtered by state and optionally by an explicit
+	 * relationship qualifier. Backs the PR sweeps, which drain it page by page.
+	 *
+	 * Ordering is part of the contract, not an option: always `sort:updated` (most recently updated first),
+	 * matching {@link searchPullRequestsPage}. A caller that stops before `hasMore` clears — every sweep with a
+	 * page budget — therefore retains a well-defined recency window instead of an arbitrary slice of GitHub's
+	 * relevance ranking.
+	 */
 	@trace({ args: (provider, token) => ({ provider: provider.name, token: `<token:${token.microHash}>` }) })
 	async searchMyPullRequestsPage(
 		provider: Provider,
@@ -3608,12 +3617,20 @@ export class GitHubApi {
 				options?.includeDefaultInvolvement === false
 					? 'is:pr archived:false'
 					: 'is:pr involves:@me archived:false';
+			// Ordering is part of the contract, not an option — same as `searchPullRequestsPage`. Without it
+			// GitHub answers in `best-match` (relevance) order, so any result set the caller stops short of
+			// draining is an arbitrary sample rather than "the N most recent": which rows land inside a page
+			// budget can then shift with GitHub's ranking even when nothing changed upstream. Consumers that
+			// cap the walk depend on this to make their window deterministic and time-bounded.
 			const rsp = await this.graphql<SearchResult>(
 				provider,
 				token,
 				query,
 				{
-					search: [stateQualifier, relationshipQualifier, search].filter(Boolean).join(' ').trim(),
+					search: [stateQualifier, relationshipQualifier, search, 'sort:updated']
+						.filter(Boolean)
+						.join(' ')
+						.trim(),
 					cursor: options?.cursor,
 					baseUrl: options?.baseUrl,
 					avatarSize: options?.avatarSize,

package/src/plus/integrations/index.ts

@@ -113,6 +113,7 @@ export type {
 	ProviderRepositoriesInput,
 	ProviderRepositoryInput,
 	ProviderSweepTarget,
+	ProviderSweepTargetEvent,
 	PullRequestSweepOptions,
 } from './manager.js';
 

package/src/plus/integrations/integrationService.ts

@@ -1574,12 +1574,31 @@ export class IntegrationService implements Disposable, RepositoryResolutionConte
 			}
 		}
 
+		// Concurrent, not serial: each `syncCloudConnection` is a cloud round trip (a forced sync deletes the
+		// stored session and refetches it), so a serial loop pays the SUM of every provider's latency on a
+		// path that gates EVERY provider read. `reconcileCloudConnections` below is already `Promise.all`.
+		//
+		// Safe because every shared mutable path is already serialized, which is why this can be a plain
+		// fan-out rather than needing per-provider grouping:
+		// - `ensureProvider` is `@gate()`d on `providerId`, so the two integrations a multi-host
+		//   self-managed id yields (one per domain, see `getSupportedCloudIntegrations`) share one
+		//   in-flight construction instead of racing to `providers.set`.
+		// - `ensureSession` is `@gate()`d per integration instance.
+		// - `addOrUpdateConfigured`/`removeConfigured` mutate `configured` in a synchronous critical
+		//   section (no `await` between read and `set`), and `storeConfigured` has its own write queue.
+		// - Secrets and the `connected:` workspace flags are keyed by integration id + domain.
+		const integrations: Integration[] = [];
 		for await (const integration of this.getSupportedCloudIntegrations(domainsById)) {
-			await integration.syncCloudConnection(
-				this.getCloudConnectionState(integration, connectedIntegrations, domainsById),
-				forceConnect,
-			);
+			integrations.push(integration);
 		}
+		await Promise.all(
+			integrations.map(integration =>
+				integration.syncCloudConnection(
+					this.getCloudConnectionState(integration, connectedIntegrations, domainsById),
+					forceConnect,
+				),
+			),
+		);
 
 		// Persist every account when the backend advertises per-connection identity (multi-account). This
 		// is a strict no-op for backends that return a single, id-less connection per provider.

package/src/plus/integrations/manager.ts

@@ -59,6 +59,63 @@ export interface ProviderSweepTarget {
 	filters?: PullRequestFilter[];
 }
 
+/**
+ * One target's contribution to a pull-request sweep, reported as it settles.
+ *
+ * A sweep makes a single call and distributes its targets internally, so its aggregate result carries no
+ * per-target timing at all and only derivable per-target counts. This is the boundary that exposes them, for a
+ * host attributing a slow or failing sweep to the provider responsible.
+ *
+ * `outcome` is reported as one value rather than the underlying booleans because a consumer buckets by it, and
+ * because "the whole target is unusable" and "the target returned a slice with a gap" are different facts:
+ * a `failed-provider` target contributes nothing, a `fetch-failed` one contributes `count` rows that are
+ * incomplete. `skipped` is a target that resolved to no reachable connection and is deliberately not
+ * attributed in the aggregate result — reported anyway so a consumer counting targets never loses one.
+ *
+ * A target that produced no slice at all reports `count: 0` and `truncated: false` — always for `skipped`, and
+ * for `failed-provider` as `drainPullRequests` reports it today. Read those fields rather than deriving them
+ * from `outcome`: they are the target's own values, not constants the outcome guarantees.
+ */
+export interface ProviderSweepTargetEvent {
+	providerId: IntegrationIds;
+	/**
+	 * The self-managed host this target selects, resolved from its configured connection, its explicit domain,
+	 * or the provider's primary configured host — the same rule the read itself uses, and the same one whether
+	 * the target drained fully or was rejected by the first guard.
+	 *
+	 * Always `undefined` for a cloud provider: it has a single host, so there is nothing to disambiguate. Group
+	 * by `providerId` and treat this as a label, not part of the key.
+	 */
+	domain: string | undefined;
+	connectionId: string | undefined;
+	/**
+	 * Rows this target contributed to the aggregate result. The sweep concatenates target slices without a
+	 * cross-target pass — duplicates are collapsed per target, across its own pages — so these sum to exactly
+	 * `items.length`, and a sum that disagrees means a target went unreported.
+	 */
+	count: number;
+	/**
+	 * Wall time from this target's worker picking it up to its slice being ready.
+	 *
+	 * Targets in the same sweep run concurrently, so these intervals OVERLAP and are not additive: summing them
+	 * across a sweep exceeds the sweep's own duration. Compare them against each other, not against a total.
+	 */
+	durationMs: number;
+	/**
+	 * Wall time between the fan-out starting and this target's worker picking the target up.
+	 *
+	 * Structurally 0 only while the target count fits the fan-out's concurrency limit, which a selection of a
+	 * few providers does and the default sweep does NOT: with no `targets`/`providerIds` it opens one target per
+	 * supported git host, more than the limit, so the last ones genuinely wait. A non-zero value there is the
+	 * normal case, not an anomaly — it is the cost of the bound, and only worth acting on if it rivals
+	 * `durationMs`.
+	 */
+	queueWaitMs: number;
+	outcome: 'ok' | 'failed-provider' | 'fetch-failed' | 'skipped';
+	/** Whether this target left pages unread. */
+	truncated: boolean;
+}
+
 type ProviderSweepSelection =
 	| {
 			targets: readonly ProviderSweepTarget[];
@@ -105,6 +162,25 @@ type PullRequestSweepCommonOptions = {
 	includeReviewRequested?: boolean;
 	forceSync?: boolean;
 	maxPages?: number;
+	/**
+	 * Fired once per target as it settles, for host-side per-provider attribution.
+	 *
+	 * Observation only: it cannot influence the sweep, and a SYNCHRONOUS throw is swallowed rather than allowed
+	 * to turn a successful target into a failed one. Return nothing and do no async work: the `void` return type
+	 * admits an `async` callback, but nothing awaits it, so a rejection from one escapes that guarantee as an
+	 * unhandled rejection. It is also invoked synchronously in the middle of the fan-out — do not re-enter the
+	 * manager from it.
+	 *
+	 * Omitting it costs nothing at all, not even a clock read, so a host that only measures behind a gate can
+	 * leave the gate off without paying for the option.
+	 *
+	 * It reports how a target SETTLED, not every way one can end, and **delivery does not stop when the sweep
+	 * fails**: if a target's read throws instead of reporting failure through its slice, the sweep rejects with
+	 * that error, but its sibling targets are already in flight and are not cancelled, so their events still
+	 * arrive — after the returned promise has rejected. Key the accumulator to the call rather than closing it
+	 * on rejection, or a late event lands in whatever bucket is current by then.
+	 */
+	onTargetSettled?: (event: ProviderSweepTargetEvent) => void;
 };
 
 export type PullRequestSweepOptions = PullRequestSweepCommonOptions & ProviderSweepSelection;

package/src/plus/integrations/models/integration.ts

@@ -269,7 +269,13 @@ export abstract class IntegrationBase<
 			// id, so an unscoped clear would sign the user out of unrelated hosts. deleteAllSessions derives an
 			// undefined domain for cloud providers, so they still clear every account as intended.
 			const authProvider = await this.authenticationService.get(this.authProvider.id);
-			void authProvider.deleteAllSessions(this.authProviderDescriptor);
+			// Awaited, not fire-and-forget: a caller that awaits `disconnect()` has to be able to rely on the
+			// secrets and descriptors actually being gone when it resumes. Left floating, the only thing that
+			// ever made this land in time was incidental scheduling slack — `syncCloudIntegrations` used to
+			// await each provider in turn, so a later iteration's suspension let the previous provider's delete
+			// finish. Syncing providers concurrently removes that slack and the clear was observably still
+			// pending when the sync returned.
+			await authProvider.deleteAllSessions(this.authProviderDescriptor);
 		}
 
 		this.resetRequestExceptionCount('all');