@gitkraken/core-gitlens 0.5.112 → 0.5.114

package/src/plus/integrations/integrationService.ts

@@ -87,6 +87,8 @@ import { countIssues, countPullRequests } from './reads/counts.js';
 import type { SupportedFilters } from './reads/filters.js';
 import { getSupportedFilters } from './reads/filters.js';
 import { listOrgs, listProjects, listRepos } from './reads/hierarchy.js';
+import type { IssueBatchResult, IssueBatchTarget } from './reads/issueBatch.js';
+import { getIssuesBatch } from './reads/issueBatch.js';
 import { listIssuesPage } from './reads/issues.js';
 import { listIssueTrackerIssuesPage } from './reads/issueTracker.js';
 import { listPullRequestsPage } from './reads/pullRequests.js';
@@ -1053,6 +1055,8 @@ export class IntegrationService implements Disposable, RepositoryResolutionConte
 		 * from the trusted authentication configuration, not repository or remote data.
 		 */
 		domain?: string;
+		/** See {@link IntegrationManager.listPullRequestsPage}. */
+		summary?: boolean;
 	}): Promise<ProviderPagedResult<PullRequestShape>> {
 		return listPullRequestsPage(this, options);
 	}
@@ -1069,6 +1073,8 @@ export class IntegrationService implements Disposable, RepositoryResolutionConte
 		forceSync?: boolean;
 		connectionId?: string;
 		domain?: string;
+		/** See {@link IntegrationManager.searchPullRequestsPage}. */
+		summary?: boolean;
 	}): Promise<ProviderPagedResult<PullRequestShape>> {
 		return searchPullRequestsPage(this, options);
 	}
@@ -1152,6 +1158,19 @@ export class IntegrationService implements Disposable, RepositoryResolutionConte
 		return countIssues(this, options);
 	}
 
+	async getIssuesBatch(options: {
+		providerId: IntegrationIds;
+		targets: readonly IssueBatchTarget[];
+		connectionId?: string;
+		/**
+		 * Explicit self-managed host domain. Used only when the requested connection has no configured domain;
+		 * it must come from the trusted authentication configuration, not repository or remote data.
+		 */
+		domain?: string;
+	}): Promise<ProviderResult<IssueBatchResult>> {
+		return getIssuesBatch(this, options);
+	}
+
 	/**
 	 * How many pull requests match each scope, without fetching any — the PR twin of {@link countIssues}. See
 	 * {@link IntegrationManager.countPullRequests} for the cost model, the per-scope isolation rules, and why

package/src/plus/integrations/manager.ts

@@ -16,6 +16,7 @@ import type {
 	PullRequestCountScope,
 } from './reads/counts.js';
 import type { SupportedFilters } from './reads/filters.js';
+import type { IssueBatchResult, IssueBatchTarget } from './reads/issueBatch.js';
 import type {
 	ConnectionStateChangeEvent,
 	ProviderBroadenResult,
@@ -364,6 +365,15 @@ export interface IntegrationManager {
 		 * it must come from the trusted authentication configuration, not repository or remote data.
 		 */
 		domain?: string;
+		/**
+		 * Requests the lightweight row shape: everything except the per-row build-status rollup
+		 * (`statusCheckRollupState`) and `commitCount`, which share one provider selection and are therefore
+		 * dropped together. Set it on list/aggregate surfaces that read neither.
+		 *
+		 * The account-wide branch already requests this shape. A provider without a lightweight projection
+		 * ignores the option, so it is a hint rather than a guarantee about which fields are present.
+		 */
+		summary?: boolean;
 	}): Promise<ProviderPagedResult<PullRequestShape>>;
 	/**
 	 * Pull requests matching structured criteria over a repository/organization or current-user relationship
@@ -407,6 +417,14 @@ export interface IntegrationManager {
 		connectionId?: string;
 		/** Self-managed host domain fallback; see {@link ProviderSweepTarget.domain}. */
 		domain?: string;
+		/**
+		 * Requests the lightweight row shape: identity, body, author, repository, branch refs and stack info,
+		 * without review, check or diff statistics. It also raises the default page size.
+		 *
+		 * A provider without a lightweight projection ignores it and returns its usual shape, so this is a hint
+		 * rather than a contract about which fields are present.
+		 */
+		summary?: boolean;
 	}): Promise<ProviderPagedResult<PullRequestShape>>;
 	listIssuesPage(options: {
 		providerId: IntegrationIds;
@@ -546,6 +564,31 @@ export interface IntegrationManager {
 		/** Self-managed host domain fallback; see {@link ProviderSweepTarget.domain}. */
 		domain?: string;
 	}): Promise<ProviderResult<IssueCountResult>>;
+	/**
+	 * Resolves several issues BY COORDINATE — `(owner, repo, number)` — in one request.
+	 *
+	 * The read for "which issue does this branch name reference", which is an IDENTITY question rather than a
+	 * search. Emulating it by paging a scoped list and matching the identifier cannot prove absence without
+	 * walking the whole scope, so a miss stays unproven, uncacheable, and repeats its whole budget on every pass.
+	 * This answers it in one request per chunk, and a miss is final.
+	 *
+	 * Results are echoed under the caller's own `key`, so no positional matching is needed. Per-target isolation
+	 * is the rule: a chunk that fails upstream warns and drops only its own targets (with `fetchFailed` set) while
+	 * every other target still answers.
+	 *
+	 * `issue: undefined` means PROVEN ABSENT — the issue does not exist, or is not visible to this connection —
+	 * and is safe to cache. A target whose chunk failed is NOT returned at all, so the two are distinguishable;
+	 * caching a failure as an absence is exactly the bug this distinction prevents. GitHub/GHE only: a provider
+	 * that cannot batch refuses outright rather than degrading into N requests behind the caller's back.
+	 */
+	getIssuesBatch(options: {
+		providerId: IntegrationIds;
+		/** Each `key` must be unique — a duplicate refuses the whole call, since keys identify results. */
+		targets: readonly IssueBatchTarget[];
+		connectionId?: string;
+		/** Self-managed host domain fallback; see {@link ProviderSweepTarget.domain}. */
+		domain?: string;
+	}): Promise<ProviderResult<IssueBatchResult>>;
 	/**
 	 * How many pull requests match each scope, fetching none of them — the PR twin of {@link countIssues}, behind a
 	 * "this will fetch ~N pull requests" preview and a live count next to an unapplied filter. Same cost model,

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

@@ -1,4 +1,4 @@
-import type { CollectionMetadata, CollectionScopeFailure } from '@gitkraken/provider-apis';
+import type { CollectionMetadata, CollectionScopeFailure, GitHubPullRequestFieldMap } from '@gitkraken/provider-apis';
 import type { Account, UnidentifiedAuthor } from '../../../git/models/author.js';
 import type { DefaultBranch } from '../../../git/models/defaultBranch.js';
 import type { IssueSearchCriteria, IssueShape } from '../../../git/models/issue.js';
@@ -117,6 +117,18 @@ export type SearchMyPullRequestsOptions = {
 	includeReviewRequested?: boolean;
 };
 
+type MyPullRequestsForReposOptions = {
+	filters?: PullRequestFilter[];
+	cursor?: string;
+	customUrl?: string;
+	page?: number;
+	pageSize?: number;
+	/** PR states to include; when omitted the provider returns its default (open only). */
+	state?: PullRequestStateFilter | PullRequestStateFilter[];
+	/** Requests the lightweight row shape without per-row build status or commit count. */
+	summary?: boolean;
+};
+
 /**
  * Rejects a read the provider can't serve as asked: logs the reason and returns it as the `{ error }` half of an
  * {@link IntegrationResult}, timed from `start`.
@@ -132,6 +144,15 @@ function unsupportedRead<T>(message: string, start: number, logContext?: string)
 	return { error: new Error(message), duration: performance.now() - start };
 }
 
+/**
+ * Both fields gate the same GitHub `commits` subtree, so either truthy key retains the build-status rollup.
+ * An absent map retains the full row; a summary read explicitly drops both optional fields.
+ */
+const summaryPullRequestFields: Required<GitHubPullRequestFieldMap> = {
+	headCommit: false,
+	commitCount: false,
+};
+
 export abstract class GitHostIntegration<
 	ID extends IntegrationIds = IntegrationIds,
 	T extends ResourceDescriptor = ResourceDescriptor,
@@ -1084,15 +1105,7 @@ export abstract class GitHostIntegration<
 
 	async getMyPullRequestsForRepos(
 		reposOrRepoIds: ProviderReposInput,
-		options?: {
-			filters?: PullRequestFilter[];
-			cursor?: string;
-			customUrl?: string;
-			page?: number;
-			pageSize?: number;
-			/** PR states to include; when omitted the provider returns its default (open only). */
-			state?: PullRequestStateFilter | PullRequestStateFilter[];
-		},
+		options?: MyPullRequestsForReposOptions,
 		connectionId?: string,
 	): Promise<ProviderApiPagedResult<ProviderPullRequest> | undefined> {
 		return (await this.getMyPullRequestsForReposResult(reposOrRepoIds, options, connectionId))?.value;
@@ -1106,14 +1119,7 @@ export abstract class GitHostIntegration<
 	 */
 	async getMyPullRequestsForReposResult(
 		reposOrRepoIds: ProviderReposInput,
-		options?: {
-			filters?: PullRequestFilter[];
-			cursor?: string;
-			customUrl?: string;
-			page?: number;
-			pageSize?: number;
-			state?: PullRequestStateFilter | PullRequestStateFilter[];
-		},
+		options?: MyPullRequestsForReposOptions,
 		connectionId?: string,
 	): Promise<IntegrationResult<ProviderApiPagedResult<ProviderPullRequest> | undefined>> {
 		const scope = getScopedLogger();
@@ -1364,6 +1370,7 @@ export abstract class GitHostIntegration<
 				states: states,
 				// Azure DevOps only populates clone URLs on request (extra call); no-op elsewhere.
 				includeRemoteInfo: isAzureDevOpsProvider(providerId) ? true : undefined,
+				fields: options?.summary ? summaryPullRequestFields : undefined,
 			});
 			return { value: result, duration: performance.now() - start };
 		} catch (ex) {
@@ -1575,6 +1582,8 @@ export abstract class GitHostIntegration<
 			criteria?: PullRequestSearchCriteria;
 			cursor?: string;
 			pageSize?: number;
+			/** See {@link IntegrationManager.searchPullRequestsPage}. */
+			summary?: boolean;
 		},
 		cancellation?: AbortSignal,
 		connectionId?: string,
@@ -1603,6 +1612,8 @@ export abstract class GitHostIntegration<
 			criteria?: PullRequestSearchCriteria;
 			cursor?: string;
 			pageSize?: number;
+			/** See the caller's option of the same name on {@link searchPullRequestsPage}. */
+			summary?: boolean;
 		},
 		cancellation?: AbortSignal,
 	): Promise<ProviderPullRequestSearchPage | undefined>;
@@ -1778,6 +1789,54 @@ export abstract class GitHostIntegration<
 		cancellation?: AbortSignal,
 	): Promise<(number | undefined)[] | undefined>;
 
+	/**
+	 * Result-returning wrapper for the BATCH issue read: resolves several `(owner, repo, number)` coordinates in
+	 * one request. Recovers thrown errors into `{ error }` like the reads around it.
+	 *
+	 * Distinct from every search on this class, and deliberately so. A search answers "what matches"; this answers
+	 * "does this exact issue exist", which is the question a caller correlating a branch name to an issue is
+	 * actually asking. It has no result ceiling, no ordering and no cursor, and — the property that matters most —
+	 * an absent slot is a PROVEN ABSENCE rather than "not found within a page budget", so a caller can cache a
+	 * miss instead of re-walking for it forever.
+	 *
+	 * Results come back POSITIONALLY — one per input coordinate, in order — for the same reason
+	 * {@link countIssuesResult} does: a caller's key must never reach the provider query. `undefined` in a slot
+	 * means the issue does not exist or is not visible to this token, never that the read failed.
+	 */
+	async getIssuesBatchResult(
+		coordinates: readonly { owner: string; repo: string; number: number }[],
+		cancellation?: AbortSignal,
+		connectionId?: string,
+	): Promise<IntegrationResult<(IssueShape | undefined)[] | undefined>> {
+		const scope = getScopedLogger();
+		// `connectionId` targets a specific account (multi-account); omitted reads the primary.
+		const session = await this.resolveReadSession(connectionId, scope);
+		if (session == null) return undefined;
+
+		const start = performance.now();
+		try {
+			const issues = await this.getProviderIssuesBatch?.(session, coordinates, cancellation);
+			this.resetRequestExceptionCount('getIssuesBatch');
+			return { value: issues, duration: performance.now() - start };
+		} catch (ex) {
+			this.handleProviderException('getIssuesBatch', ex, { scope: scope, connectionId: connectionId });
+			return { error: toError(ex), duration: performance.now() - start };
+		}
+	}
+
+	/**
+	 * OPTIONAL: only a provider that can resolve SEVERAL issues by coordinate in one request implements this.
+	 * GitHub aliases its point read into one document; the SDK exposes only a singular `getIssue` for every other
+	 * provider, and there is no plural variant to build on. A provider that can't answer doesn't implement it and
+	 * the facade refuses the read, so a caller keeps its own per-issue path rather than being handed a batch that
+	 * silently degraded into N requests.
+	 */
+	protected getProviderIssuesBatch?(
+		session: ProviderAuthenticationSession,
+		coordinates: readonly { owner: string; repo: string; number: number }[],
+		cancellation?: AbortSignal,
+	): Promise<(IssueShape | undefined)[] | undefined>;
+
 	/** The PR twin of {@link countIssuesResult}: counts each scope's pull requests, transferring none. */
 	async countPullRequestsResult(
 		scopes: readonly { repos?: ProviderRepoInput[]; org?: string; criteria?: PullRequestSearchCriteria }[],

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

@@ -65,6 +65,7 @@ type SyncReqUsecase = Exclude<
 	| 'getDefaultBranch'
 	| 'getIssue'
 	| 'getIssueOrPullRequest'
+	| 'getIssuesBatch'
 	| 'getIssuesForProject'
 	| 'getIssuesForRepos'
 	| 'getMyPullRequestsForUser'

package/src/plus/integrations/providers/github.ts

@@ -612,6 +612,7 @@ abstract class GitHubIntegrationBase<ID extends GitHubIntegrationIds> extends Gi
 			criteria?: PullRequestSearchCriteria;
 			cursor?: string;
 			pageSize?: number;
+			summary?: boolean;
 		},
 		cancellation?: AbortSignal,
 	): Promise<ProviderPullRequestSearchPage | undefined> {
@@ -625,6 +626,7 @@ abstract class GitHubIntegrationBase<ID extends GitHubIntegrationIds> extends Gi
 				baseUrl: this.apiBaseUrl,
 				cursor: options.cursor,
 				pageSize: options.pageSize,
+				summary: options.summary,
 			},
 			cancellation,
 		);
@@ -687,6 +689,24 @@ abstract class GitHubIntegrationBase<ID extends GitHubIntegrationIds> extends Gi
 		);
 	}
 
+	/**
+	 * Resolves several issues by `(owner, repo, number)` in ONE request, by aliasing the point read rather than a
+	 * search — see {@link GitHubApi.getIssuesBatch} for why that distinction is the whole design.
+	 */
+	protected override async getProviderIssuesBatch(
+		session: ProviderAuthenticationSession,
+		coordinates: readonly { owner: string; repo: string; number: number }[],
+		cancellation?: AbortSignal,
+	): Promise<(IssueShape | undefined)[] | undefined> {
+		return (await this.authenticationService.apis.github)?.getIssuesBatch(
+			this,
+			toTokenWithInfo(this.id, session),
+			coordinates,
+			{ baseUrl: this.apiBaseUrl, includeBody: true },
+			cancellation,
+		);
+	}
+
 	/**
 	 * Counts several pull-request scopes in ONE request. Like {@link countProviderIssues}, GitHub's `search`
 	 * reports `issueCount` on a zero-node selection, so a count preview costs no pull-request transfer.

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

@@ -16,6 +16,7 @@ import type {
 	GitBuildStatus,
 	GitBuildStatusState as GitBuildStatusStateType,
 	GitHub,
+	GitHubPullRequestFieldMap,
 	GitIssueState as GitIssueStateType,
 	GitLab,
 	GitLabGroup,
@@ -329,6 +330,11 @@ export interface GetPullRequestsOptions {
 	// Opt in to repository remote metadata (clone URLs) when the PR payload lacks it. Only Azure DevOps
 	// acts on this today (extra API call); it is a no-op for the other providers.
 	includeRemoteInfo?: boolean;
+	// Field selection for the row shape. GitHub-shaped because it is the only provider whose PR read gates
+	// anything on it; the others ignore it. Declared here rather than passed through a spread so a typo or a
+	// wrong-shaped map is a compile error instead of a silently ignored key. An ABSENT map means "select
+	// everything", so only summary reads pass one.
+	fields?: GitHubPullRequestFieldMap;
 }
 
 export interface GetPullRequestsForUserOptions {