@gitkraken/core-gitlens 0.7.1 → 0.8.0

package/dist/plus/integrations/reads/issueBatch.d.ts

@@ -1,41 +1,73 @@
 import { IssueShape } from "../../../git/models/issue.js";
 import { IntegrationIds } from "../constants.js";
-import { ProviderResult } from "../results.js";
+import { ProviderResult, ProviderWarning } from "../results.js";
+import { BatchSlot } from "../models/integration.js";
 import { ProviderReadContext } from "./context.js";
 //#region ../plus/integrations/src/reads/issueBatch.d.ts
 /**
- * The BATCH issue read: resolve N `(owner, repo, number)` coordinates in one request.
+ * The BATCH issue read: resolve N issues BY IDENTITY in one call — `(owner, repo, number)` coordinates on a git
+ * host, `(resourceId, ABC-123)` identifiers on an issue tracker.
  *
  * A sibling of the issue searches rather than a mode of one, because it answers a different question. A search
  * asks "what matches"; this asks "does this exact issue exist", which is what a caller correlating a branch name
  * to the issue it references is actually asking. Three consequences follow, and they are the reason this exists:
  *
- * - It resolves by EXACT NUMBER, so relevance never enters into it.
+ * - It resolves by EXACT IDENTIFIER, so relevance never enters into it.
  * - No result ceiling applies, so there is no partial window to reason about and no omission to report.
  * - An absent slot is a PROVEN ABSENCE, not "not found within a page budget". That is the property that matters
  *   most: a caller can CACHE a miss. Emulating this with a paged list cannot prove absence without walking the
  *   whole scope, so a miss stays unproven and the walk repeats on every pass, forever.
  *
- * Modeled on `countIssues` for its shape — caller-owned `key` echoed back, per-target isolation, chunked into as
- * few requests as possible — because both take a set of independent questions and answer them together.
+ * Modeled on `countIssues` for its shape — caller-owned `key` echoed back, per-target isolation, as few requests
+ * as the provider allows — because both take a set of independent questions and answer them together.
+ *
+ * ONE integration call per invocation, whatever the target count, as in `getPullRequestsBatch`: the integration
+ * fans its targets out (chunked for GitHub/GHE, one request per target with bounded concurrency on GitLab and
+ * Azure DevOps and on the trackers) and settles each independently, so failing targets spend at most one strike
+ * of the integration's failure budget.
+ */
+/** One issue to resolve, echoed back under the caller's own `key`. Which form a call takes depends on its provider. */
+export type IssueBatchTarget =
+/**
+ * A git host's issue, by repository coordinate. GitHub/GHE, GitLab (and self-managed) and Azure DevOps (and
+ * Server); Bitbucket and Bitbucket Data Center have no issues and refuse.
  */
-/** One issue to resolve, identified by coordinate and echoed back under the caller's own `key`. */
-export interface IssueBatchTarget {
-  /** Caller-owned identifier, echoed on the result so no positional matching is needed. Must be unique. */
+{
   key: string;
+  /** GitHub owner, GitLab namespace path (may be nested), or Azure DevOps organization (or collection). */
   owner: string;
+  /**
+   * Repository name. Required, but ignored on Azure DevOps, whose work items belong to the project rather
+   * than to a repository.
+   */
   repo: string;
+  /** Issue number, GitLab issue iid, or Azure DevOps work item id. */
   number: number;
-}
+  /** Azure DevOps project. Required there, ignored elsewhere. */
+  project?: string;
+} |
+/**
+ * An issue tracker's issue, by its own identifier (e.g. `ABC-123`) within a resource. Jira (Cloud and Data
+ * Center) and Linear. For Jira Data Center `resourceId` is the host — the instance's single resource, as
+ * `listOrgs` reports it — and `resourceUrl` is not needed, since the browser link is built from the
+ * connection's own base URL; Jira Cloud requires it.
+ */
+{
+  key: string;
+  resourceId: string;
+  resourceUrl?: string;
+  identifier: string;
+};
 /** The answer for one {@link IssueBatchTarget}. */
 export interface IssueBatchResult {
   key: string;
   /**
    * The resolved issue, or `undefined` when it PROVABLY does not exist (or is not visible to this connection).
    *
-   * Absent is an answer here, unlike every paged read on this facade: a target whose chunk FAILED is not
-   * returned at all and sets `fetchFailed`, so a caller can tell "proven absent" from "unknown" and cache the
-   * first without ever caching the second.
+   * Absent is an answer here, unlike every paged read on this facade: a target that FAILED — its own request
+   * outright, or just its own alias within an otherwise-answering GitHub chunk (e.g. an org enforcing SAML SSO
+   * the token isn't authorized for) — is not returned at all and sets `fetchFailed`, so a caller can tell
+   * "proven absent" from "unknown" and cache the first without ever caching the second.
    */
   issue?: IssueShape;
 }
@@ -49,5 +81,33 @@ export declare function getIssuesBatch(ctx: ProviderReadContext, options: {
    */
   domain?: string;
 }): Promise<ProviderResult<IssueBatchResult>>;
+/**
+ * Trims `owner`, `repo` and `project` once, shared by every coordinate-form batch read (this one, the pull request
+ * batch and the pull-requests-by-branch read): leading/trailing whitespace can't change which repository a
+ * coordinate names, so it's stripped and accepted rather than refused, matching #5821's scope-name precedent.
+ */
+export declare function trimCoordinateFields<T extends {
+  owner: string;
+  repo: string;
+  project?: string;
+}>(target: T): T;
+/**
+ * The answer when no integration resolves for a batch read. Targets were asked about, so this is never a silent
+ * empty success: a supplied connection or domain that no longer resolves gets its own warning, and the untargeted
+ * primary path (e.g. a self-managed provider with no configured host) a connection warning, as `getCurrentAccount`
+ * gives — either way `fetchFailed`, so no caller mistakes the dropped targets for an answer.
+ */
+export declare function unresolvedIntegration<T>(ctx: ProviderReadContext, providerId: IntegrationIds, connectionId: string | undefined, domain: string | undefined): ProviderResult<T>;
+/**
+ * Appends the warning for a batch target that could not be checked, shared by every batch read: for a refusal, the
+ * scope failure it was recorded as (see `IntegrationBase.settleBatchRefusals`), published as an account-wide read
+ * publishes one, scoped and with its cause; otherwise, the reason itself.
+ */
+export declare function appendBatchSlotWarning(warnings: ProviderWarning[], providerId: IntegrationIds, domain: string | undefined, connectionId: string | undefined, slot: Extract<BatchSlot<unknown>, {
+  status: 'rejected';
+}>): void;
+export declare function findDuplicateKey(targets: readonly {
+  key: string;
+}[]): string | undefined;
 //#endregion
 //# sourceMappingURL=issueBatch.d.ts.map

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

@@ -1,74 +1,137 @@
-import { chunk } from "../../../utils/array.js";
-import { mapBounded } from "../../../utils/promise.js";
-import { appendDedupedWarning } from "../results.js";
 import "../constants.js";
-import { isGitHostIntegration, isIssuesHostIntegrationId, warnOnMissingSessionForDomain } from "../utils/integration.utils.js";
-import { gitHostOnlySurfaceWarning, issuesUnsupportedWarning, otherWarning } from "./warnings.js";
+import { githubGraphQLInt32Max, isAzureProviderId, isGitHubProviderId } from "../providers/providerErrors.js";
+import { appendDedupedWarning, toProviderWarning } from "../results.js";
+import { mergeAssessmentInto } from "../collectionMetadata.js";
+import { areDomainsOnSameHost, hostFromDomain } from "../utils/domain.utils.js";
+import { isGitHostIntegration, isIssuesHostIntegrationId, isIssuesSelfManagedHostIntegrationId } from "../utils/integration.utils.js";
+import { gitHostOnlySurfaceWarning, issueTrackerOnlySurfaceWarning, issuesUnsupportedWarning, noConnectionWarning, otherWarning } from "./warnings.js";
 import { runCaptured } from "./drains.js";
+import { isIssuesIntegration } from "../models/issuesIntegration.js";
 //#region ../plus/integrations/src/reads/issueBatch.ts
-/**
-* How many coordinates go into one upstream request.
-*
-* Measured rather than inherited from `issueCountChunkSize`, since that one rests on a zero-node selection and
-* this carries the full issue projection per alias. Against the live API on all-resolving coordinates: 10 took
-* 875ms, 25 took 890ms — so up to 25 is effectively free — 50 took 1.3s, and 75 took 2.7s. No complexity refusal
-* appeared up to 100.
-*
-* 25 is where latency is still flat, with room before it degrades. It also puts any realistic correlation batch
-* in ONE request, which is the whole point of the read.
-*/
-const issueBatchChunkSize = 25;
-async function getIssuesBatch(ctx, options) {
-	const refused = (warning) => ({
+const surface = "Batch issue resolution";
+function refused(warning) {
+	return {
 		items: [],
 		warnings: [warning],
 		fetchFailed: true
-	});
-	if (isIssuesHostIntegrationId(options.providerId)) return refused(gitHostOnlySurfaceWarning(options.providerId, void 0, options.connectionId, "Batch issue resolution"));
+	};
+}
+async function getIssuesBatch(ctx, options) {
 	if (options.targets.length === 0) return {
 		items: [],
 		warnings: []
 	};
 	const duplicateKey = findDuplicateKey(options.targets);
 	if (duplicateKey != null) return refused(otherWarning(options.providerId, void 0, options.connectionId, `Duplicate issue batch target key '${duplicateKey}'; keys identify results, so each must be unique.`));
+	if (isIssuesHostIntegrationId(options.providerId)) return getIssuesBatchForTracker(ctx, options.providerId, options.targets, options.connectionId, options.domain);
+	const targets = options.targets;
+	if (!targets.every(isCoordinateTarget)) return refused(otherWarning(options.providerId, void 0, options.connectionId, `${surface} for '${options.providerId}' takes repository coordinates { key, owner, repo, number, project? }, not tracker identifiers.`));
+	const trimmedTargets = targets.map(trimCoordinateFields);
+	const invalid = findInvalidCoordinateTarget(options.providerId, trimmedTargets);
+	if (invalid != null) return refused(otherWarning(options.providerId, void 0, options.connectionId, invalid));
 	const integration = await ctx.getIntegrationForRead(options.providerId, options.connectionId, options.domain);
-	if (integration == null) {
-		const early = ctx.earlyReturnConnectionWarnings(options.providerId, options.connectionId, options.domain);
-		return {
-			items: [],
-			warnings: early.warnings,
-			fetchFailed: early.fetchFailed || void 0
-		};
-	}
-	if (!isGitHostIntegration(integration)) return refused(gitHostOnlySurfaceWarning(options.providerId, void 0, options.connectionId, "Batch issue resolution"));
+	if (integration == null) return unresolvedIntegration(ctx, options.providerId, options.connectionId, options.domain);
+	if (!isGitHostIntegration(integration)) return refused(gitHostOnlySurfaceWarning(options.providerId, void 0, options.connectionId, surface));
 	const domain = ctx.domainForRead(integration, options.providerId, options.connectionId, options.domain);
-	const warnOnMissingSession = warnOnMissingSessionForDomain(options.providerId, options.domain);
 	if (!integration.supportsIssues) return refused(issuesUnsupportedWarning(options.providerId, domain, options.connectionId));
 	const warnings = [];
 	let fetchFailed = false;
-	const batches = await mapBounded(chunk([...options.targets], issueBatchChunkSize), 6, (batch) => runCaptured(options.providerId, domain, options.connectionId, () => integration.getIssuesBatchResult(batch.map((t) => ({
+	const { value: slots, warning } = await runCaptured(options.providerId, domain, options.connectionId, () => integration.getIssuesBatchResult(trimmedTargets.map((t) => ({
 		owner: t.owner,
 		repo: t.repo,
-		number: t.number
-	})), void 0, options.connectionId), { warnOnMissingSession }).then((result) => ({
-		batch,
-		...result
-	})));
+		number: t.number,
+		project: t.project
+	})), void 0, options.connectionId), { warnOnMissingSession: true });
+	if (warning != null) appendDedupedWarning(warnings, warning);
+	if (slots == null) {
+		if (warning == null) appendDedupedWarning(warnings, otherWarning(options.providerId, domain, options.connectionId, `${surface} is not supported by '${options.providerId}'; resolve issues individually instead.`));
+		return {
+			items: [],
+			warnings,
+			fetchFailed: true
+		};
+	}
 	const items = [];
-	for (const { batch, value, warning } of batches) {
-		if (warning != null) appendDedupedWarning(warnings, warning);
-		if (value == null) {
-			if (warning == null) appendDedupedWarning(warnings, otherWarning(options.providerId, domain, options.connectionId, `Batch issue resolution is not supported by '${options.providerId}'; resolve issues individually instead.`));
+	for (let i = 0; i < trimmedTargets.length; i++) {
+		const slot = slots[i];
+		if (slot.status === "rejected") {
+			appendBatchSlotWarning(warnings, options.providerId, domain, options.connectionId, slot);
 			fetchFailed = true;
 			continue;
 		}
-		for (let i = 0; i < batch.length; i++) {
-			const issue = value[i];
-			items.push({
-				key: batch[i].key,
-				...issue != null ? { issue } : {}
-			});
+		const issue = slot.value;
+		items.push({
+			key: trimmedTargets[i].key,
+			...issue != null ? { issue } : {}
+		});
+	}
+	return {
+		items,
+		warnings,
+		fetchFailed: fetchFailed || void 0
+	};
+}
+/**
+* The tracker form. ONE integration call per invocation, whatever the target count: the integration makes one
+* single-issue request per target, with bounded concurrency, and settles each independently — see
+* `IssuesIntegration.getIssuesByResourceIdBatchResult`.
+*
+* `resourceId` is trusted and the read does no resource discovery. It is handed to the provider's by-resource-id
+* read as-is, never wrapped into a synthesized descriptor: Linear and Trello answer `undefined` for a descriptor
+* that fails `isIssueResourceDescriptor`, which would be published as a proven absence.
+*
+* `domain` selects the host of a self-managed tracker (Jira Data Center), as it does for
+* `listIssueTrackerIssuesPage`, and is ignored for the cloud trackers, which have a single canonical host. Unlike
+* the paged reads, this one never falls back to the primary connection for a self-managed tracker: it requires a
+* `domain` or a `connectionId` with a configured host and refuses otherwise. Two self-hosted instances routinely
+* issue the same project and issue keys, and `issue: undefined` is a proven absence a caller may cache — an answer
+* from whichever host happens to be primary would be cached under a key that names a different instance (#5872).
+*/
+async function getIssuesBatchForTracker(ctx, providerId, targets, connectionId, domain) {
+	if (!targets.every(isTrackerTarget)) return refused(otherWarning(providerId, void 0, connectionId, `${surface} for '${providerId}' takes tracker identifiers { key, resourceId, resourceUrl?, identifier }, not repository coordinates.`));
+	const trimmedTargets = targets.map(trimTrackerTarget);
+	const invalid = findInvalidTrackerTarget(providerId, trimmedTargets);
+	if (invalid != null) return refused(otherWarning(providerId, void 0, connectionId, invalid));
+	if (isIssuesSelfManagedHostIntegrationId(providerId) && (domain != null ? hostFromDomain(domain) == null : connectionId == null || !ctx.getConfigured(providerId).some((c) => c.id === connectionId && hostFromDomain(c.domain) != null))) return refused(otherWarning(providerId, void 0, connectionId, `${surface} requires a domain or a configured connection id for '${providerId}': every configured host can hold a different issue under the same key, so the read does not answer from the primary connection.`));
+	const integration = await ctx.getIntegrationForRead(providerId, connectionId, domain);
+	if (integration == null) return unresolvedIntegration(ctx, providerId, connectionId, domain);
+	if (!isIssuesIntegration(integration)) return refused(issueTrackerOnlySurfaceWarning(providerId, connectionId, surface));
+	const unsupported = () => otherWarning(providerId, void 0, connectionId, `${surface} is not supported by '${providerId}'; its single-issue read cannot prove an absence, so a miss would not be safe to cache.`);
+	if (!integration.supportsIssueLookupByResourceId) return refused(unsupported());
+	if (isIssuesSelfManagedHostIntegrationId(providerId)) {
+		const mismatched = trimmedTargets.find((t) => !areDomainsOnSameHost(t.resourceId, integration.domain));
+		if (mismatched != null) return refused(otherWarning(providerId, void 0, connectionId, `Issue batch target '${mismatched.key}' requires the resource id of '${providerId}' to name the host the read resolved to.`));
+	}
+	const resolvedDomain = ctx.domainForRead(integration, providerId, connectionId, domain);
+	const warnings = [];
+	let fetchFailed = false;
+	const { value: slots, warning } = await runCaptured(providerId, resolvedDomain, connectionId, () => integration.getIssuesByResourceIdBatchResult(trimmedTargets.map((t) => ({
+		resourceId: t.resourceId,
+		identifier: t.identifier,
+		resourceUrl: t.resourceUrl
+	})), connectionId), { warnOnMissingSession: true });
+	if (warning != null) appendDedupedWarning(warnings, warning);
+	if (slots == null) {
+		if (warning == null) appendDedupedWarning(warnings, unsupported());
+		return {
+			items: [],
+			warnings,
+			fetchFailed: true
+		};
+	}
+	const items = [];
+	for (let i = 0; i < trimmedTargets.length; i++) {
+		const slot = slots[i];
+		if (slot.status === "rejected") {
+			appendBatchSlotWarning(warnings, providerId, resolvedDomain, connectionId, slot);
+			fetchFailed = true;
+			continue;
 		}
+		const issue = slot.value;
+		items.push({
+			key: trimmedTargets[i].key,
+			...issue != null ? { issue } : {}
+		});
 	}
 	return {
 		items,
@@ -76,6 +139,88 @@ async function getIssuesBatch(ctx, options) {
 		fetchFailed: fetchFailed || void 0
 	};
 }
+function isCoordinateTarget(target) {
+	return "owner" in target;
+}
+function isTrackerTarget(target) {
+	return "resourceId" in target;
+}
+/**
+* Trims `owner`, `repo` and `project` once, shared by every coordinate-form batch read (this one, the pull request
+* batch and the pull-requests-by-branch read): leading/trailing whitespace can't change which repository a
+* coordinate names, so it's stripped and accepted rather than refused, matching #5821's scope-name precedent.
+*/
+function trimCoordinateFields(target) {
+	return {
+		...target,
+		owner: target.owner.trim(),
+		repo: target.repo.trim(),
+		project: target.project?.trim()
+	};
+}
+/** Trims `resourceId`, `resourceUrl` and `identifier` once; the validated value must be the one sent to the provider. */
+function trimTrackerTarget(target) {
+	return {
+		...target,
+		resourceId: target.resourceId.trim(),
+		resourceUrl: target.resourceUrl?.trim() || void 0,
+		identifier: target.identifier.trim()
+	};
+}
+function findInvalidCoordinateTarget(providerId, targets) {
+	const isAzure = isAzureProviderId(providerId);
+	const isGitHub = isGitHubProviderId(providerId);
+	for (const target of targets) {
+		if (!Number.isSafeInteger(target.number) || target.number <= 0) return `Issue batch target '${target.key}' has number ${target.number}; expected a positive integer.`;
+		if (isGitHub && target.number > 2147483647) return `Issue batch target '${target.key}' has number ${target.number}; GitHub numbers are 32-bit and cannot exceed ${githubGraphQLInt32Max}.`;
+		if (target.owner.length === 0) return `Issue batch target '${target.key}' requires a non-empty owner.`;
+		if (isAzure) {
+			if (!target.project) return `Issue batch target '${target.key}' requires a project for '${providerId}'.`;
+		} else if (target.repo.length === 0) return `Issue batch target '${target.key}' requires a non-empty repo.`;
+	}
+}
+function findInvalidTrackerTarget(providerId, targets) {
+	for (const target of targets) {
+		if (target.resourceId.length === 0) return `Issue batch target '${target.key}' requires a resource id.`;
+		if (target.identifier.length === 0) return `Issue batch target '${target.key}' requires an issue identifier.`;
+		if (providerId === "jira" && !target.resourceUrl) return `Issue batch target '${target.key}' requires the Jira resource URL so the result contains a browser link without resource discovery.`;
+	}
+}
+/**
+* The answer when no integration resolves for a batch read. Targets were asked about, so this is never a silent
+* empty success: a supplied connection or domain that no longer resolves gets its own warning, and the untargeted
+* primary path (e.g. a self-managed provider with no configured host) a connection warning, as `getCurrentAccount`
+* gives — either way `fetchFailed`, so no caller mistakes the dropped targets for an answer.
+*/
+function unresolvedIntegration(ctx, providerId, connectionId, domain) {
+	const early = ctx.earlyReturnConnectionWarnings(providerId, connectionId, domain);
+	if (early.warnings.length > 0) return {
+		items: [],
+		warnings: early.warnings,
+		fetchFailed: true
+	};
+	const resolvedDomain = ctx.resolveDomainForRead(providerId, connectionId, domain);
+	return {
+		items: [],
+		warnings: [noConnectionWarning(providerId, resolvedDomain, connectionId)],
+		fetchFailed: true
+	};
+}
+/**
+* Appends the warning for a batch target that could not be checked, shared by every batch read: for a refusal, the
+* scope failure it was recorded as (see `IntegrationBase.settleBatchRefusals`), published as an account-wide read
+* publishes one, scoped and with its cause; otherwise, the reason itself.
+*/
+function appendBatchSlotWarning(warnings, providerId, domain, connectionId, slot) {
+	if (slot.failure != null) {
+		mergeAssessmentInto(warnings, providerId, domain, connectionId, {
+			completeness: "partial",
+			failures: [slot.failure]
+		});
+		return;
+	}
+	appendDedupedWarning(warnings, toProviderWarning(providerId, domain, connectionId, slot.reason));
+}
 function findDuplicateKey(targets) {
 	const seen = /* @__PURE__ */ new Set();
 	for (const target of targets) {
@@ -84,6 +229,6 @@ function findDuplicateKey(targets) {
 	}
 }
 //#endregion
-export { getIssuesBatch };
+export { appendBatchSlotWarning, findDuplicateKey, getIssuesBatch, trimCoordinateFields, unresolvedIntegration };
 
 //# sourceMappingURL=issueBatch.js.map

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

@@ -1 +1 @@
-{"version":3,"file":"issueBatch.js","names":[],"sources":["../../../../../plus/integrations/src/reads/issueBatch.ts"],"sourcesContent":["import type { IssueShape } from '@gitlens/git/models/issue.js';\nimport { chunk } from '@gitlens/utils/array.js';\nimport { mapBounded } from '@gitlens/utils/promise.js';\nimport type { IntegrationIds } from '../constants.js';\nimport { providerFanOutConcurrency } from '../constants.js';\nimport type { ProviderResult, ProviderWarning } from '../results.js';\nimport { appendDedupedWarning } from '../results.js';\nimport {\n\tisGitHostIntegration,\n\tisIssuesHostIntegrationId,\n\twarnOnMissingSessionForDomain,\n} from '../utils/integration.utils.js';\nimport type { ProviderReadContext } from './context.js';\nimport { runCaptured } from './drains.js';\nimport { gitHostOnlySurfaceWarning, issuesUnsupportedWarning, otherWarning } from './warnings.js';\n\n/**\n * The BATCH issue read: resolve N `(owner, repo, number)` coordinates in one request.\n *\n * A sibling of the issue searches rather than a mode of one, because it answers a different question. A search\n * asks \"what matches\"; this asks \"does this exact issue exist\", which is what a caller correlating a branch name\n * to the issue it references is actually asking. Three consequences follow, and they are the reason this exists:\n *\n * - It resolves by EXACT NUMBER, so relevance never enters into it.\n * - No result ceiling applies, so there is no partial window to reason about and no omission to report.\n * - An absent slot is a PROVEN ABSENCE, not \"not found within a page budget\". That is the property that matters\n *   most: a caller can CACHE a miss. Emulating this with a paged list cannot prove absence without walking the\n *   whole scope, so a miss stays unproven and the walk repeats on every pass, forever.\n *\n * Modeled on `countIssues` for its shape — caller-owned `key` echoed back, per-target isolation, chunked into as\n * few requests as possible — because both take a set of independent questions and answer them together.\n */\n\n/** One issue to resolve, identified by coordinate and echoed back under the caller's own `key`. */\nexport interface IssueBatchTarget {\n\t/** Caller-owned identifier, echoed on the result so no positional matching is needed. Must be unique. */\n\tkey: string;\n\towner: string;\n\trepo: string;\n\tnumber: number;\n}\n\n/** The answer for one {@link IssueBatchTarget}. */\nexport interface IssueBatchResult {\n\tkey: string;\n\t/**\n\t * The resolved issue, or `undefined` when it PROVABLY does not exist (or is not visible to this connection).\n\t *\n\t * Absent is an answer here, unlike every paged read on this facade: a target whose chunk FAILED is not\n\t * returned at all and sets `fetchFailed`, so a caller can tell \"proven absent\" from \"unknown\" and cache the\n\t * first without ever caching the second.\n\t */\n\tissue?: IssueShape;\n}\n\n/**\n * How many coordinates go into one upstream request.\n *\n * Measured rather than inherited from `issueCountChunkSize`, since that one rests on a zero-node selection and\n * this carries the full issue projection per alias. Against the live API on all-resolving coordinates: 10 took\n * 875ms, 25 took 890ms — so up to 25 is effectively free — 50 took 1.3s, and 75 took 2.7s. No complexity refusal\n * appeared up to 100.\n *\n * 25 is where latency is still flat, with room before it degrades. It also puts any realistic correlation batch\n * in ONE request, which is the whole point of the read.\n */\nconst issueBatchChunkSize = 25;\n\nexport async function getIssuesBatch(\n\tctx: ProviderReadContext,\n\toptions: {\n\t\tproviderId: IntegrationIds;\n\t\ttargets: readonly IssueBatchTarget[];\n\t\tconnectionId?: string;\n\t\t/**\n\t\t * Explicit self-managed host domain. Used only when the requested connection has no configured domain;\n\t\t * it must come from the trusted authentication configuration, not repository or remote data.\n\t\t */\n\t\tdomain?: string;\n\t},\n): Promise<ProviderResult<IssueBatchResult>> {\n\tconst refused = (warning: ProviderWarning): ProviderResult<IssueBatchResult> => ({\n\t\titems: [],\n\t\twarnings: [warning],\n\t\tfetchFailed: true,\n\t});\n\n\tif (isIssuesHostIntegrationId(options.providerId)) {\n\t\treturn refused(\n\t\t\tgitHostOnlySurfaceWarning(options.providerId, undefined, options.connectionId, 'Batch issue resolution'),\n\t\t);\n\t}\n\n\t// Nothing was asked for, so nothing is missing: an empty success, not a refusal.\n\tif (options.targets.length === 0) return { items: [], warnings: [] };\n\n\tconst duplicateKey = findDuplicateKey(options.targets);\n\tif (duplicateKey != null) {\n\t\t// Refuses the whole call rather than deduping: `key` exists so the caller can match results without\n\t\t// positional bookkeeping, and two results under one key make that ambiguous for EVERY target, not just\n\t\t// the repeated one.\n\t\treturn refused(\n\t\t\totherWarning(\n\t\t\t\toptions.providerId,\n\t\t\t\tundefined,\n\t\t\t\toptions.connectionId,\n\t\t\t\t`Duplicate issue batch target key '${duplicateKey}'; keys identify results, so each must be unique.`,\n\t\t\t),\n\t\t);\n\t}\n\n\tconst integration = await ctx.getIntegrationForRead(options.providerId, options.connectionId, options.domain);\n\tif (integration == null) {\n\t\t// A supplied connection or domain that no longer resolves is a broken target, not an empty account.\n\t\tconst early = ctx.earlyReturnConnectionWarnings(options.providerId, options.connectionId, options.domain);\n\t\treturn { items: [], warnings: early.warnings, fetchFailed: early.fetchFailed || undefined };\n\t}\n\tif (!isGitHostIntegration(integration)) {\n\t\treturn refused(\n\t\t\tgitHostOnlySurfaceWarning(options.providerId, undefined, options.connectionId, 'Batch issue resolution'),\n\t\t);\n\t}\n\n\tconst domain = ctx.domainForRead(integration, options.providerId, options.connectionId, options.domain);\n\tconst warnOnMissingSession = warnOnMissingSessionForDomain(options.providerId, options.domain);\n\n\tif (!integration.supportsIssues) {\n\t\treturn refused(issuesUnsupportedWarning(options.providerId, domain, options.connectionId));\n\t}\n\n\tconst warnings: ProviderWarning[] = [];\n\tlet fetchFailed = false;\n\n\t// Chunks are independent requests over their own slice of targets — nothing in one reads what another\n\t// produced, and `runCaptured` never throws — so they run concurrently, bounded like every other fan-out here.\n\t// `mapBounded` returns in input order, so `items` stays in target order.\n\tconst batches = await mapBounded(\n\t\tchunk([...options.targets], issueBatchChunkSize),\n\t\tproviderFanOutConcurrency,\n\t\tbatch =>\n\t\t\trunCaptured(\n\t\t\t\toptions.providerId,\n\t\t\t\tdomain,\n\t\t\t\toptions.connectionId,\n\t\t\t\t() =>\n\t\t\t\t\tintegration.getIssuesBatchResult(\n\t\t\t\t\t\tbatch.map(t => ({ owner: t.owner, repo: t.repo, number: t.number })),\n\t\t\t\t\t\tundefined,\n\t\t\t\t\t\toptions.connectionId,\n\t\t\t\t\t),\n\t\t\t\t{ warnOnMissingSession: warnOnMissingSession },\n\t\t\t).then(result => ({ batch: batch, ...result })),\n\t);\n\n\tconst items: IssueBatchResult[] = [];\n\tfor (const { batch, value, warning } of batches) {\n\t\tif (warning != null) {\n\t\t\tappendDedupedWarning(warnings, warning);\n\t\t}\n\t\tif (value == null) {\n\t\t\t// A provider that doesn't implement the batch hook answers `undefined` with no error. Either way this\n\t\t\t// chunk contributes nothing while the chunks around it still do, so its targets are DROPPED rather\n\t\t\t// than reported as absent — the difference between \"unknown\" and \"proven absent\" is the read's whole\n\t\t\t// value, and a failure must never be cached as an answer.\n\t\t\tif (warning == null) {\n\t\t\t\tappendDedupedWarning(\n\t\t\t\t\twarnings,\n\t\t\t\t\totherWarning(\n\t\t\t\t\t\toptions.providerId,\n\t\t\t\t\t\tdomain,\n\t\t\t\t\t\toptions.connectionId,\n\t\t\t\t\t\t`Batch issue resolution is not supported by '${options.providerId}'; resolve issues individually instead.`,\n\t\t\t\t\t),\n\t\t\t\t);\n\t\t\t}\n\t\t\tfetchFailed = true;\n\t\t\tcontinue;\n\t\t}\n\n\t\tfor (let i = 0; i < batch.length; i++) {\n\t\t\tconst issue = value[i];\n\t\t\titems.push({ key: batch[i].key, ...(issue != null ? { issue: issue } : {}) });\n\t\t}\n\t}\n\n\treturn { items: items, warnings: warnings, fetchFailed: fetchFailed || undefined };\n}\n\nfunction findDuplicateKey(targets: readonly { key: string }[]): string | undefined {\n\tconst seen = new Set<string>();\n\tfor (const target of targets) {\n\t\tif (seen.has(target.key)) return target.key;\n\n\t\tseen.add(target.key);\n\t}\n\treturn undefined;\n}\n"],"mappings":";;;;;;;;;;;;;;;;;;;AAkEA,MAAM,sBAAsB;AAE5B,eAAsB,eACrB,KACA,SAU4C;CAC5C,MAAM,WAAW,aAAgE;EAChF,OAAO,CAAC;EACR,UAAU,CAAC,OAAO;EAClB,aAAa;CACd;CAEA,IAAI,0BAA0B,QAAQ,UAAU,GAC/C,OAAO,QACN,0BAA0B,QAAQ,YAAY,KAAA,GAAW,QAAQ,cAAc,wBAAwB,CACxG;CAID,IAAI,QAAQ,QAAQ,WAAW,GAAG,OAAO;EAAE,OAAO,CAAC;EAAG,UAAU,CAAC;CAAE;CAEnE,MAAM,eAAe,iBAAiB,QAAQ,OAAO;CACrD,IAAI,gBAAgB,MAInB,OAAO,QACN,aACC,QAAQ,YACR,KAAA,GACA,QAAQ,cACR,qCAAqC,aAAa,kDACnD,CACD;CAGD,MAAM,cAAc,MAAM,IAAI,sBAAsB,QAAQ,YAAY,QAAQ,cAAc,QAAQ,MAAM;CAC5G,IAAI,eAAe,MAAM;EAExB,MAAM,QAAQ,IAAI,8BAA8B,QAAQ,YAAY,QAAQ,cAAc,QAAQ,MAAM;EACxG,OAAO;GAAE,OAAO,CAAC;GAAG,UAAU,MAAM;GAAU,aAAa,MAAM,eAAe,KAAA;EAAU;CAC3F;CACA,IAAI,CAAC,qBAAqB,WAAW,GACpC,OAAO,QACN,0BAA0B,QAAQ,YAAY,KAAA,GAAW,QAAQ,cAAc,wBAAwB,CACxG;CAGD,MAAM,SAAS,IAAI,cAAc,aAAa,QAAQ,YAAY,QAAQ,cAAc,QAAQ,MAAM;CACtG,MAAM,uBAAuB,8BAA8B,QAAQ,YAAY,QAAQ,MAAM;CAE7F,IAAI,CAAC,YAAY,gBAChB,OAAO,QAAQ,yBAAyB,QAAQ,YAAY,QAAQ,QAAQ,YAAY,CAAC;CAG1F,MAAM,WAA8B,CAAC;CACrC,IAAI,cAAc;CAKlB,MAAM,UAAU,MAAM,WACrB,MAAM,CAAC,GAAG,QAAQ,OAAO,GAAG,mBAAmB,GAAA,IAE/C,UACC,YACC,QAAQ,YACR,QACA,QAAQ,oBAEP,YAAY,qBACX,MAAM,KAAI,OAAM;EAAE,OAAO,EAAE;EAAO,MAAM,EAAE;EAAM,QAAQ,EAAE;CAAO,EAAE,GACnE,KAAA,GACA,QAAQ,YACT,GACD,EAAwB,qBAAqB,CAC9C,CAAC,CAAC,MAAK,YAAW;EAAS;EAAO,GAAG;CAAO,EAAE,CAChD;CAEA,MAAM,QAA4B,CAAC;CACnC,KAAK,MAAM,EAAE,OAAO,OAAO,aAAa,SAAS;EAChD,IAAI,WAAW,MACd,qBAAqB,UAAU,OAAO;EAEvC,IAAI,SAAS,MAAM;GAKlB,IAAI,WAAW,MACd,qBACC,UACA,aACC,QAAQ,YACR,QACA,QAAQ,cACR,+CAA+C,QAAQ,WAAW,wCACnE,CACD;GAED,cAAc;GACd;EACD;EAEA,KAAK,IAAI,IAAI,GAAG,IAAI,MAAM,QAAQ,KAAK;GACtC,MAAM,QAAQ,MAAM;GACpB,MAAM,KAAK;IAAE,KAAK,MAAM,EAAE,CAAC;IAAK,GAAI,SAAS,OAAO,EAAS,MAAM,IAAI,CAAC;GAAG,CAAC;EAC7E;CACD;CAEA,OAAO;EAAS;EAAiB;EAAU,aAAa,eAAe,KAAA;CAAU;AAClF;AAEA,SAAS,iBAAiB,SAAyD;CAClF,MAAM,uBAAO,IAAI,IAAY;CAC7B,KAAK,MAAM,UAAU,SAAS;EAC7B,IAAI,KAAK,IAAI,OAAO,GAAG,GAAG,OAAO,OAAO;EAExC,KAAK,IAAI,OAAO,GAAG;CACpB;AAED"}
+{"version":3,"file":"issueBatch.js","names":[],"sources":["../../../../../plus/integrations/src/reads/issueBatch.ts"],"sourcesContent":["import type { IssueShape } from '@gitlens/git/models/issue.js';\nimport { mergeAssessmentInto } from '../collectionMetadata.js';\nimport type { IntegrationIds, IssuesHostIntegrationIds } from '../constants.js';\nimport { IssuesCloudHostIntegrationId } from '../constants.js';\nimport type { BatchSlot } from '../models/integration.js';\nimport { isIssuesIntegration } from '../models/issuesIntegration.js';\nimport { githubGraphQLInt32Max, isAzureProviderId, isGitHubProviderId } from '../providers/providerErrors.js';\nimport type { ProviderResult, ProviderWarning } from '../results.js';\nimport { appendDedupedWarning, toProviderWarning } from '../results.js';\nimport { areDomainsOnSameHost, hostFromDomain } from '../utils/domain.utils.js';\nimport {\n\tisGitHostIntegration,\n\tisIssuesHostIntegrationId,\n\tisIssuesSelfManagedHostIntegrationId,\n} from '../utils/integration.utils.js';\nimport type { ProviderReadContext } from './context.js';\nimport { runCaptured } from './drains.js';\nimport {\n\tgitHostOnlySurfaceWarning,\n\tissuesUnsupportedWarning,\n\tissueTrackerOnlySurfaceWarning,\n\tnoConnectionWarning,\n\totherWarning,\n} from './warnings.js';\n\n/**\n * The BATCH issue read: resolve N issues BY IDENTITY in one call — `(owner, repo, number)` coordinates on a git\n * host, `(resourceId, ABC-123)` identifiers on an issue tracker.\n *\n * A sibling of the issue searches rather than a mode of one, because it answers a different question. A search\n * asks \"what matches\"; this asks \"does this exact issue exist\", which is what a caller correlating a branch name\n * to the issue it references is actually asking. Three consequences follow, and they are the reason this exists:\n *\n * - It resolves by EXACT IDENTIFIER, so relevance never enters into it.\n * - No result ceiling applies, so there is no partial window to reason about and no omission to report.\n * - An absent slot is a PROVEN ABSENCE, not \"not found within a page budget\". That is the property that matters\n *   most: a caller can CACHE a miss. Emulating this with a paged list cannot prove absence without walking the\n *   whole scope, so a miss stays unproven and the walk repeats on every pass, forever.\n *\n * Modeled on `countIssues` for its shape — caller-owned `key` echoed back, per-target isolation, as few requests\n * as the provider allows — because both take a set of independent questions and answer them together.\n *\n * ONE integration call per invocation, whatever the target count, as in `getPullRequestsBatch`: the integration\n * fans its targets out (chunked for GitHub/GHE, one request per target with bounded concurrency on GitLab and\n * Azure DevOps and on the trackers) and settles each independently, so failing targets spend at most one strike\n * of the integration's failure budget.\n */\n\n/** One issue to resolve, echoed back under the caller's own `key`. Which form a call takes depends on its provider. */\nexport type IssueBatchTarget =\n\t/**\n\t * A git host's issue, by repository coordinate. GitHub/GHE, GitLab (and self-managed) and Azure DevOps (and\n\t * Server); Bitbucket and Bitbucket Data Center have no issues and refuse.\n\t */\n\t| {\n\t\t\tkey: string;\n\t\t\t/** GitHub owner, GitLab namespace path (may be nested), or Azure DevOps organization (or collection). */\n\t\t\towner: string;\n\t\t\t/**\n\t\t\t * Repository name. Required, but ignored on Azure DevOps, whose work items belong to the project rather\n\t\t\t * than to a repository.\n\t\t\t */\n\t\t\trepo: string;\n\t\t\t/** Issue number, GitLab issue iid, or Azure DevOps work item id. */\n\t\t\tnumber: number;\n\t\t\t/** Azure DevOps project. Required there, ignored elsewhere. */\n\t\t\tproject?: string;\n\t  }\n\t/**\n\t * An issue tracker's issue, by its own identifier (e.g. `ABC-123`) within a resource. Jira (Cloud and Data\n\t * Center) and Linear. For Jira Data Center `resourceId` is the host — the instance's single resource, as\n\t * `listOrgs` reports it — and `resourceUrl` is not needed, since the browser link is built from the\n\t * connection's own base URL; Jira Cloud requires it.\n\t */\n\t| { key: string; resourceId: string; resourceUrl?: string; identifier: string };\n\ntype CoordinateTarget = Extract<IssueBatchTarget, { owner: string }>;\ntype TrackerTarget = Extract<IssueBatchTarget, { resourceId: string }>;\n\n/** The answer for one {@link IssueBatchTarget}. */\nexport interface IssueBatchResult {\n\tkey: string;\n\t/**\n\t * The resolved issue, or `undefined` when it PROVABLY does not exist (or is not visible to this connection).\n\t *\n\t * Absent is an answer here, unlike every paged read on this facade: a target that FAILED — its own request\n\t * outright, or just its own alias within an otherwise-answering GitHub chunk (e.g. an org enforcing SAML SSO\n\t * the token isn't authorized for) — is not returned at all and sets `fetchFailed`, so a caller can tell\n\t * \"proven absent\" from \"unknown\" and cache the first without ever caching the second.\n\t */\n\tissue?: IssueShape;\n}\n\nconst surface = 'Batch issue resolution';\n\nfunction refused(warning: ProviderWarning): ProviderResult<IssueBatchResult> {\n\treturn { items: [], warnings: [warning], fetchFailed: true };\n}\n\nexport async function getIssuesBatch(\n\tctx: ProviderReadContext,\n\toptions: {\n\t\tproviderId: IntegrationIds;\n\t\ttargets: readonly IssueBatchTarget[];\n\t\tconnectionId?: string;\n\t\t/**\n\t\t * Explicit self-managed host domain. Used only when the requested connection has no configured domain;\n\t\t * it must come from the trusted authentication configuration, not repository or remote data.\n\t\t */\n\t\tdomain?: string;\n\t},\n): Promise<ProviderResult<IssueBatchResult>> {\n\t// Nothing was asked for, so nothing is missing: an empty success, not a refusal.\n\tif (options.targets.length === 0) return { items: [], warnings: [] };\n\n\tconst duplicateKey = findDuplicateKey(options.targets);\n\tif (duplicateKey != null) {\n\t\t// Refuses the whole call rather than deduping: `key` exists so the caller can match results without\n\t\t// positional bookkeeping, and two results under one key make that ambiguous for EVERY target, not just\n\t\t// the repeated one.\n\t\treturn refused(\n\t\t\totherWarning(\n\t\t\t\toptions.providerId,\n\t\t\t\tundefined,\n\t\t\t\toptions.connectionId,\n\t\t\t\t`Duplicate issue batch target key '${duplicateKey}'; keys identify results, so each must be unique.`,\n\t\t\t),\n\t\t);\n\t}\n\n\tif (isIssuesHostIntegrationId(options.providerId)) {\n\t\treturn getIssuesBatchForTracker(ctx, options.providerId, options.targets, options.connectionId, options.domain);\n\t}\n\n\tconst targets = options.targets;\n\tif (!targets.every(isCoordinateTarget)) {\n\t\treturn refused(\n\t\t\totherWarning(\n\t\t\t\toptions.providerId,\n\t\t\t\tundefined,\n\t\t\t\toptions.connectionId,\n\t\t\t\t`${surface} for '${options.providerId}' takes repository coordinates { key, owner, repo, number, project? }, not tracker identifiers.`,\n\t\t\t),\n\t\t);\n\t}\n\n\t// The validated value must be the one sent to the provider, so every string field is trimmed once here.\n\tconst trimmedTargets = targets.map(trimCoordinateFields);\n\n\t// Deterministic caller bugs, refused like a duplicate key rather than sent upstream, where a blank owner or repo\n\t// would come back as a missing repository — a proven absence the caller would cache.\n\tconst invalid = findInvalidCoordinateTarget(options.providerId, trimmedTargets);\n\tif (invalid != null) {\n\t\treturn refused(otherWarning(options.providerId, undefined, options.connectionId, invalid));\n\t}\n\n\tconst integration = await ctx.getIntegrationForRead(options.providerId, options.connectionId, options.domain);\n\tif (integration == null) {\n\t\treturn unresolvedIntegration(ctx, options.providerId, options.connectionId, options.domain);\n\t}\n\tif (!isGitHostIntegration(integration)) {\n\t\treturn refused(gitHostOnlySurfaceWarning(options.providerId, undefined, options.connectionId, surface));\n\t}\n\n\tconst domain = ctx.domainForRead(integration, options.providerId, options.connectionId, options.domain);\n\n\tif (!integration.supportsIssues) {\n\t\treturn refused(issuesUnsupportedWarning(options.providerId, domain, options.connectionId));\n\t}\n\n\tconst warnings: ProviderWarning[] = [];\n\tlet fetchFailed = false;\n\n\tconst { value: slots, warning } = await runCaptured(\n\t\toptions.providerId,\n\t\tdomain,\n\t\toptions.connectionId,\n\t\t() =>\n\t\t\tintegration.getIssuesBatchResult(\n\t\t\t\ttrimmedTargets.map(t => ({ owner: t.owner, repo: t.repo, number: t.number, project: t.project })),\n\t\t\t\tundefined,\n\t\t\t\toptions.connectionId,\n\t\t\t),\n\t\t// A missing session must surface as a connection warning, including on the primary path, which\n\t\t// otherwise reads it as \"not connected\" and says nothing — and this read would then call it unsupported.\n\t\t{ warnOnMissingSession: true },\n\t);\n\tif (warning != null) {\n\t\tappendDedupedWarning(warnings, warning);\n\t}\n\n\tif (slots == null) {\n\t\t// A provider that doesn't implement the batch hook answers `undefined` with no error. Either way its\n\t\t// targets are DROPPED rather than reported as absent — the difference between \"unknown\" and \"proven\n\t\t// absent\" is the read's whole value, and a failure must never be cached as an answer.\n\t\tif (warning == null) {\n\t\t\tappendDedupedWarning(\n\t\t\t\twarnings,\n\t\t\t\totherWarning(\n\t\t\t\t\toptions.providerId,\n\t\t\t\t\tdomain,\n\t\t\t\t\toptions.connectionId,\n\t\t\t\t\t`${surface} is not supported by '${options.providerId}'; resolve issues individually instead.`,\n\t\t\t\t),\n\t\t\t);\n\t\t}\n\t\treturn { items: [], warnings: warnings, fetchFailed: true };\n\t}\n\n\tconst items: IssueBatchResult[] = [];\n\tfor (let i = 0; i < trimmedTargets.length; i++) {\n\t\tconst slot = slots[i];\n\t\tif (slot.status === 'rejected') {\n\t\t\t// Dropped, never reported absent, like a whole-call failure: only THIS target failed.\n\t\t\tappendBatchSlotWarning(warnings, options.providerId, domain, options.connectionId, slot);\n\t\t\tfetchFailed = true;\n\t\t\tcontinue;\n\t\t}\n\n\t\tconst issue = slot.value;\n\t\titems.push({ key: trimmedTargets[i].key, ...(issue != null ? { issue: issue } : {}) });\n\t}\n\n\treturn { items: items, warnings: warnings, fetchFailed: fetchFailed || undefined };\n}\n\n/**\n * The tracker form. ONE integration call per invocation, whatever the target count: the integration makes one\n * single-issue request per target, with bounded concurrency, and settles each independently — see\n * `IssuesIntegration.getIssuesByResourceIdBatchResult`.\n *\n * `resourceId` is trusted and the read does no resource discovery. It is handed to the provider's by-resource-id\n * read as-is, never wrapped into a synthesized descriptor: Linear and Trello answer `undefined` for a descriptor\n * that fails `isIssueResourceDescriptor`, which would be published as a proven absence.\n *\n * `domain` selects the host of a self-managed tracker (Jira Data Center), as it does for\n * `listIssueTrackerIssuesPage`, and is ignored for the cloud trackers, which have a single canonical host. Unlike\n * the paged reads, this one never falls back to the primary connection for a self-managed tracker: it requires a\n * `domain` or a `connectionId` with a configured host and refuses otherwise. Two self-hosted instances routinely\n * issue the same project and issue keys, and `issue: undefined` is a proven absence a caller may cache — an answer\n * from whichever host happens to be primary would be cached under a key that names a different instance (#5872).\n */\nasync function getIssuesBatchForTracker(\n\tctx: ProviderReadContext,\n\tproviderId: IssuesHostIntegrationIds,\n\ttargets: readonly IssueBatchTarget[],\n\tconnectionId: string | undefined,\n\tdomain: string | undefined,\n): Promise<ProviderResult<IssueBatchResult>> {\n\tif (!targets.every(isTrackerTarget)) {\n\t\treturn refused(\n\t\t\totherWarning(\n\t\t\t\tproviderId,\n\t\t\t\tundefined,\n\t\t\t\tconnectionId,\n\t\t\t\t`${surface} for '${providerId}' takes tracker identifiers { key, resourceId, resourceUrl?, identifier }, not repository coordinates.`,\n\t\t\t),\n\t\t);\n\t}\n\n\t// The validated value must be the one sent to the provider, so every string field is trimmed once here.\n\tconst trimmedTargets = targets.map(trimTrackerTarget);\n\n\t// Deterministic caller bugs, refused like a duplicate key rather than sent upstream to fail one by one.\n\tconst invalid = findInvalidTrackerTarget(providerId, trimmedTargets);\n\tif (invalid != null) {\n\t\treturn refused(otherWarning(providerId, undefined, connectionId, invalid));\n\t}\n\n\t// A `domain` only selects a host when it parses to one, and a `connectionId` only when it names a configured\n\t// connection that has one; anything else would resolve the primary host instead, which is the fallback this\n\t// read refuses.\n\tif (\n\t\tisIssuesSelfManagedHostIntegrationId(providerId) &&\n\t\t(domain != null\n\t\t\t? hostFromDomain(domain) == null\n\t\t\t: connectionId == null ||\n\t\t\t\t!ctx.getConfigured(providerId).some(c => c.id === connectionId && hostFromDomain(c.domain) != null))\n\t) {\n\t\treturn refused(\n\t\t\totherWarning(\n\t\t\t\tproviderId,\n\t\t\t\tundefined,\n\t\t\t\tconnectionId,\n\t\t\t\t`${surface} requires a domain or a configured connection id for '${providerId}': every configured host can hold a different issue under the same key, so the read does not answer from the primary connection.`,\n\t\t\t),\n\t\t);\n\t}\n\n\tconst integration = await ctx.getIntegrationForRead(providerId, connectionId, domain);\n\tif (integration == null) return unresolvedIntegration(ctx, providerId, connectionId, domain);\n\tif (!isIssuesIntegration(integration)) {\n\t\treturn refused(issueTrackerOnlySurfaceWarning(providerId, connectionId, surface));\n\t}\n\n\tconst unsupported = (): ProviderWarning =>\n\t\totherWarning(\n\t\t\tproviderId,\n\t\t\tundefined,\n\t\t\tconnectionId,\n\t\t\t`${surface} is not supported by '${providerId}'; its single-issue read cannot prove an absence, so a miss would not be safe to cache.`,\n\t\t);\n\tif (!integration.supportsIssueLookupByResourceId) return refused(unsupported());\n\n\t// A self-managed tracker's single resource is its host, and a caller keys (and caches) each answer by\n\t// `resourceId` while the read is addressed to the host `domain`/`connectionId` resolved. When they disagree,\n\t// host B's answer — including a cacheable absence — would be stored under host A's key. Refused whole, like\n\t// the caller bugs above: a target naming another host means the call selected a host its caller did not mean.\n\tif (isIssuesSelfManagedHostIntegrationId(providerId)) {\n\t\tconst mismatched = trimmedTargets.find(t => !areDomainsOnSameHost(t.resourceId, integration.domain));\n\t\tif (mismatched != null) {\n\t\t\treturn refused(\n\t\t\t\totherWarning(\n\t\t\t\t\tproviderId,\n\t\t\t\t\tundefined,\n\t\t\t\t\tconnectionId,\n\t\t\t\t\t`Issue batch target '${mismatched.key}' requires the resource id of '${providerId}' to name the host the read resolved to.`,\n\t\t\t\t),\n\t\t\t);\n\t\t}\n\t}\n\n\tconst resolvedDomain = ctx.domainForRead(integration, providerId, connectionId, domain);\n\tconst warnings: ProviderWarning[] = [];\n\tlet fetchFailed = false;\n\n\tconst { value: slots, warning } = await runCaptured(\n\t\tproviderId,\n\t\tresolvedDomain,\n\t\tconnectionId,\n\t\t() =>\n\t\t\tintegration.getIssuesByResourceIdBatchResult(\n\t\t\t\ttrimmedTargets.map(t => ({\n\t\t\t\t\tresourceId: t.resourceId,\n\t\t\t\t\tidentifier: t.identifier,\n\t\t\t\t\tresourceUrl: t.resourceUrl,\n\t\t\t\t})),\n\t\t\t\tconnectionId,\n\t\t\t),\n\t\t// The read core answers `undefined` when it cannot resolve a session. That must surface as a connection\n\t\t// warning, including on the primary path, rather than as nothing — or worse, be read as absence.\n\t\t{ warnOnMissingSession: true },\n\t);\n\tif (warning != null) {\n\t\tappendDedupedWarning(warnings, warning);\n\t}\n\n\tif (slots == null) {\n\t\t// Dropped, never reported absent: \"unknown\" and \"proven absent\" must stay distinguishable.\n\t\tif (warning == null) {\n\t\t\tappendDedupedWarning(warnings, unsupported());\n\t\t}\n\t\treturn { items: [], warnings: warnings, fetchFailed: true };\n\t}\n\n\tconst items: IssueBatchResult[] = [];\n\tfor (let i = 0; i < trimmedTargets.length; i++) {\n\t\tconst slot = slots[i];\n\t\tif (slot.status === 'rejected') {\n\t\t\t// Dropped, never reported absent, like a whole-call failure.\n\t\t\tappendBatchSlotWarning(warnings, providerId, resolvedDomain, connectionId, slot);\n\t\t\tfetchFailed = true;\n\t\t\tcontinue;\n\t\t}\n\n\t\tconst issue = slot.value;\n\t\titems.push({ key: trimmedTargets[i].key, ...(issue != null ? { issue: issue } : {}) });\n\t}\n\n\treturn { items: items, warnings: warnings, fetchFailed: fetchFailed || undefined };\n}\n\nfunction isCoordinateTarget(target: IssueBatchTarget): target is CoordinateTarget {\n\treturn 'owner' in target;\n}\n\nfunction isTrackerTarget(target: IssueBatchTarget): target is TrackerTarget {\n\treturn 'resourceId' in target;\n}\n\n/**\n * Trims `owner`, `repo` and `project` once, shared by every coordinate-form batch read (this one, the pull request\n * batch and the pull-requests-by-branch read): leading/trailing whitespace can't change which repository a\n * coordinate names, so it's stripped and accepted rather than refused, matching #5821's scope-name precedent.\n */\nexport function trimCoordinateFields<T extends { owner: string; repo: string; project?: string }>(target: T): T {\n\treturn { ...target, owner: target.owner.trim(), repo: target.repo.trim(), project: target.project?.trim() };\n}\n\n/** Trims `resourceId`, `resourceUrl` and `identifier` once; the validated value must be the one sent to the provider. */\nfunction trimTrackerTarget(target: TrackerTarget): TrackerTarget {\n\treturn {\n\t\t...target,\n\t\tresourceId: target.resourceId.trim(),\n\t\tresourceUrl: target.resourceUrl?.trim() || undefined,\n\t\tidentifier: target.identifier.trim(),\n\t};\n}\n\nfunction findInvalidCoordinateTarget(\n\tproviderId: IntegrationIds,\n\ttargets: readonly CoordinateTarget[],\n): string | undefined {\n\tconst isAzure = isAzureProviderId(providerId);\n\tconst isGitHub = isGitHubProviderId(providerId);\n\tfor (const target of targets) {\n\t\tif (!Number.isSafeInteger(target.number) || target.number <= 0) {\n\t\t\treturn `Issue batch target '${target.key}' has number ${target.number}; expected a positive integer.`;\n\t\t}\n\t\tif (isGitHub && target.number > githubGraphQLInt32Max) {\n\t\t\treturn `Issue batch target '${target.key}' has number ${target.number}; GitHub numbers are 32-bit and cannot exceed ${githubGraphQLInt32Max}.`;\n\t\t}\n\t\tif (target.owner.length === 0) {\n\t\t\treturn `Issue batch target '${target.key}' requires a non-empty owner.`;\n\t\t}\n\t\tif (isAzure) {\n\t\t\tif (!target.project) {\n\t\t\t\treturn `Issue batch target '${target.key}' requires a project for '${providerId}'.`;\n\t\t\t}\n\t\t} else if (target.repo.length === 0) {\n\t\t\treturn `Issue batch target '${target.key}' requires a non-empty repo.`;\n\t\t}\n\t}\n\treturn undefined;\n}\n\nfunction findInvalidTrackerTarget(\n\tproviderId: IssuesHostIntegrationIds,\n\ttargets: readonly TrackerTarget[],\n): string | undefined {\n\tfor (const target of targets) {\n\t\tif (target.resourceId.length === 0) {\n\t\t\treturn `Issue batch target '${target.key}' requires a resource id.`;\n\t\t}\n\t\tif (target.identifier.length === 0) {\n\t\t\treturn `Issue batch target '${target.key}' requires an issue identifier.`;\n\t\t}\n\t\tif (providerId === IssuesCloudHostIntegrationId.Jira && !target.resourceUrl) {\n\t\t\treturn `Issue batch target '${target.key}' requires the Jira resource URL so the result contains a browser link without resource discovery.`;\n\t\t}\n\t}\n\treturn undefined;\n}\n\n/**\n * The answer when no integration resolves for a batch read. Targets were asked about, so this is never a silent\n * empty success: a supplied connection or domain that no longer resolves gets its own warning, and the untargeted\n * primary path (e.g. a self-managed provider with no configured host) a connection warning, as `getCurrentAccount`\n * gives — either way `fetchFailed`, so no caller mistakes the dropped targets for an answer.\n */\nexport function unresolvedIntegration<T>(\n\tctx: ProviderReadContext,\n\tproviderId: IntegrationIds,\n\tconnectionId: string | undefined,\n\tdomain: string | undefined,\n): ProviderResult<T> {\n\tconst early = ctx.earlyReturnConnectionWarnings(providerId, connectionId, domain);\n\tif (early.warnings.length > 0) return { items: [], warnings: early.warnings, fetchFailed: true };\n\n\tconst resolvedDomain = ctx.resolveDomainForRead(providerId, connectionId, domain);\n\treturn { items: [], warnings: [noConnectionWarning(providerId, resolvedDomain, connectionId)], fetchFailed: true };\n}\n\n/**\n * Appends the warning for a batch target that could not be checked, shared by every batch read: for a refusal, the\n * scope failure it was recorded as (see `IntegrationBase.settleBatchRefusals`), published as an account-wide read\n * publishes one, scoped and with its cause; otherwise, the reason itself.\n */\nexport function appendBatchSlotWarning(\n\twarnings: ProviderWarning[],\n\tproviderId: IntegrationIds,\n\tdomain: string | undefined,\n\tconnectionId: string | undefined,\n\tslot: Extract<BatchSlot<unknown>, { status: 'rejected' }>,\n): void {\n\tif (slot.failure != null) {\n\t\tmergeAssessmentInto(warnings, providerId, domain, connectionId, {\n\t\t\tcompleteness: 'partial',\n\t\t\tfailures: [slot.failure],\n\t\t});\n\t\treturn;\n\t}\n\n\tappendDedupedWarning(warnings, toProviderWarning(providerId, domain, connectionId, slot.reason));\n}\n\nexport function findDuplicateKey(targets: readonly { key: string }[]): string | undefined {\n\tconst seen = new Set<string>();\n\tfor (const target of targets) {\n\t\tif (seen.has(target.key)) return target.key;\n\n\t\tseen.add(target.key);\n\t}\n\treturn undefined;\n}\n"],"mappings":";;;;;;;;;;AA6FA,MAAM,UAAU;AAEhB,SAAS,QAAQ,SAA4D;CAC5E,OAAO;EAAE,OAAO,CAAC;EAAG,UAAU,CAAC,OAAO;EAAG,aAAa;CAAK;AAC5D;AAEA,eAAsB,eACrB,KACA,SAU4C;CAE5C,IAAI,QAAQ,QAAQ,WAAW,GAAG,OAAO;EAAE,OAAO,CAAC;EAAG,UAAU,CAAC;CAAE;CAEnE,MAAM,eAAe,iBAAiB,QAAQ,OAAO;CACrD,IAAI,gBAAgB,MAInB,OAAO,QACN,aACC,QAAQ,YACR,KAAA,GACA,QAAQ,cACR,qCAAqC,aAAa,kDACnD,CACD;CAGD,IAAI,0BAA0B,QAAQ,UAAU,GAC/C,OAAO,yBAAyB,KAAK,QAAQ,YAAY,QAAQ,SAAS,QAAQ,cAAc,QAAQ,MAAM;CAG/G,MAAM,UAAU,QAAQ;CACxB,IAAI,CAAC,QAAQ,MAAM,kBAAkB,GACpC,OAAO,QACN,aACC,QAAQ,YACR,KAAA,GACA,QAAQ,cACR,GAAG,QAAQ,QAAQ,QAAQ,WAAW,gGACvC,CACD;CAID,MAAM,iBAAiB,QAAQ,IAAI,oBAAoB;CAIvD,MAAM,UAAU,4BAA4B,QAAQ,YAAY,cAAc;CAC9E,IAAI,WAAW,MACd,OAAO,QAAQ,aAAa,QAAQ,YAAY,KAAA,GAAW,QAAQ,cAAc,OAAO,CAAC;CAG1F,MAAM,cAAc,MAAM,IAAI,sBAAsB,QAAQ,YAAY,QAAQ,cAAc,QAAQ,MAAM;CAC5G,IAAI,eAAe,MAClB,OAAO,sBAAsB,KAAK,QAAQ,YAAY,QAAQ,cAAc,QAAQ,MAAM;CAE3F,IAAI,CAAC,qBAAqB,WAAW,GACpC,OAAO,QAAQ,0BAA0B,QAAQ,YAAY,KAAA,GAAW,QAAQ,cAAc,OAAO,CAAC;CAGvG,MAAM,SAAS,IAAI,cAAc,aAAa,QAAQ,YAAY,QAAQ,cAAc,QAAQ,MAAM;CAEtG,IAAI,CAAC,YAAY,gBAChB,OAAO,QAAQ,yBAAyB,QAAQ,YAAY,QAAQ,QAAQ,YAAY,CAAC;CAG1F,MAAM,WAA8B,CAAC;CACrC,IAAI,cAAc;CAElB,MAAM,EAAE,OAAO,OAAO,YAAY,MAAM,YACvC,QAAQ,YACR,QACA,QAAQ,oBAEP,YAAY,qBACX,eAAe,KAAI,OAAM;EAAE,OAAO,EAAE;EAAO,MAAM,EAAE;EAAM,QAAQ,EAAE;EAAQ,SAAS,EAAE;CAAQ,EAAE,GAChG,KAAA,GACA,QAAQ,YACT,GAGD,EAAE,sBAAsB,KAAK,CAC9B;CACA,IAAI,WAAW,MACd,qBAAqB,UAAU,OAAO;CAGvC,IAAI,SAAS,MAAM;EAIlB,IAAI,WAAW,MACd,qBACC,UACA,aACC,QAAQ,YACR,QACA,QAAQ,cACR,GAAG,QAAQ,wBAAwB,QAAQ,WAAW,wCACvD,CACD;EAED,OAAO;GAAE,OAAO,CAAC;GAAa;GAAU,aAAa;EAAK;CAC3D;CAEA,MAAM,QAA4B,CAAC;CACnC,KAAK,IAAI,IAAI,GAAG,IAAI,eAAe,QAAQ,KAAK;EAC/C,MAAM,OAAO,MAAM;EACnB,IAAI,KAAK,WAAW,YAAY;GAE/B,uBAAuB,UAAU,QAAQ,YAAY,QAAQ,QAAQ,cAAc,IAAI;GACvF,cAAc;GACd;EACD;EAEA,MAAM,QAAQ,KAAK;EACnB,MAAM,KAAK;GAAE,KAAK,eAAe,EAAE,CAAC;GAAK,GAAI,SAAS,OAAO,EAAS,MAAM,IAAI,CAAC;EAAG,CAAC;CACtF;CAEA,OAAO;EAAS;EAAiB;EAAU,aAAa,eAAe,KAAA;CAAU;AAClF;;;;;;;;;;;;;;;;;AAkBA,eAAe,yBACd,KACA,YACA,SACA,cACA,QAC4C;CAC5C,IAAI,CAAC,QAAQ,MAAM,eAAe,GACjC,OAAO,QACN,aACC,YACA,KAAA,GACA,cACA,GAAG,QAAQ,QAAQ,WAAW,uGAC/B,CACD;CAID,MAAM,iBAAiB,QAAQ,IAAI,iBAAiB;CAGpD,MAAM,UAAU,yBAAyB,YAAY,cAAc;CACnE,IAAI,WAAW,MACd,OAAO,QAAQ,aAAa,YAAY,KAAA,GAAW,cAAc,OAAO,CAAC;CAM1E,IACC,qCAAqC,UAAU,MAC9C,UAAU,OACR,eAAe,MAAM,KAAK,OAC1B,gBAAgB,QACjB,CAAC,IAAI,cAAc,UAAU,CAAC,CAAC,MAAK,MAAK,EAAE,OAAO,gBAAgB,eAAe,EAAE,MAAM,KAAK,IAAI,IAEpG,OAAO,QACN,aACC,YACA,KAAA,GACA,cACA,GAAG,QAAQ,wDAAwD,WAAW,iIAC/E,CACD;CAGD,MAAM,cAAc,MAAM,IAAI,sBAAsB,YAAY,cAAc,MAAM;CACpF,IAAI,eAAe,MAAM,OAAO,sBAAsB,KAAK,YAAY,cAAc,MAAM;CAC3F,IAAI,CAAC,oBAAoB,WAAW,GACnC,OAAO,QAAQ,+BAA+B,YAAY,cAAc,OAAO,CAAC;CAGjF,MAAM,oBACL,aACC,YACA,KAAA,GACA,cACA,GAAG,QAAQ,wBAAwB,WAAW,wFAC/C;CACD,IAAI,CAAC,YAAY,iCAAiC,OAAO,QAAQ,YAAY,CAAC;CAM9E,IAAI,qCAAqC,UAAU,GAAG;EACrD,MAAM,aAAa,eAAe,MAAK,MAAK,CAAC,qBAAqB,EAAE,YAAY,YAAY,MAAM,CAAC;EACnG,IAAI,cAAc,MACjB,OAAO,QACN,aACC,YACA,KAAA,GACA,cACA,uBAAuB,WAAW,IAAI,iCAAiC,WAAW,yCACnF,CACD;CAEF;CAEA,MAAM,iBAAiB,IAAI,cAAc,aAAa,YAAY,cAAc,MAAM;CACtF,MAAM,WAA8B,CAAC;CACrC,IAAI,cAAc;CAElB,MAAM,EAAE,OAAO,OAAO,YAAY,MAAM,YACvC,YACA,gBACA,oBAEC,YAAY,iCACX,eAAe,KAAI,OAAM;EACxB,YAAY,EAAE;EACd,YAAY,EAAE;EACd,aAAa,EAAE;CAChB,EAAE,GACF,YACD,GAGD,EAAE,sBAAsB,KAAK,CAC9B;CACA,IAAI,WAAW,MACd,qBAAqB,UAAU,OAAO;CAGvC,IAAI,SAAS,MAAM;EAElB,IAAI,WAAW,MACd,qBAAqB,UAAU,YAAY,CAAC;EAE7C,OAAO;GAAE,OAAO,CAAC;GAAa;GAAU,aAAa;EAAK;CAC3D;CAEA,MAAM,QAA4B,CAAC;CACnC,KAAK,IAAI,IAAI,GAAG,IAAI,eAAe,QAAQ,KAAK;EAC/C,MAAM,OAAO,MAAM;EACnB,IAAI,KAAK,WAAW,YAAY;GAE/B,uBAAuB,UAAU,YAAY,gBAAgB,cAAc,IAAI;GAC/E,cAAc;GACd;EACD;EAEA,MAAM,QAAQ,KAAK;EACnB,MAAM,KAAK;GAAE,KAAK,eAAe,EAAE,CAAC;GAAK,GAAI,SAAS,OAAO,EAAS,MAAM,IAAI,CAAC;EAAG,CAAC;CACtF;CAEA,OAAO;EAAS;EAAiB;EAAU,aAAa,eAAe,KAAA;CAAU;AAClF;AAEA,SAAS,mBAAmB,QAAsD;CACjF,OAAO,WAAW;AACnB;AAEA,SAAS,gBAAgB,QAAmD;CAC3E,OAAO,gBAAgB;AACxB;;;;;;AAOA,SAAgB,qBAAkF,QAAc;CAC/G,OAAO;EAAE,GAAG;EAAQ,OAAO,OAAO,MAAM,KAAK;EAAG,MAAM,OAAO,KAAK,KAAK;EAAG,SAAS,OAAO,SAAS,KAAK;CAAE;AAC3G;;AAGA,SAAS,kBAAkB,QAAsC;CAChE,OAAO;EACN,GAAG;EACH,YAAY,OAAO,WAAW,KAAK;EACnC,aAAa,OAAO,aAAa,KAAK,KAAK,KAAA;EAC3C,YAAY,OAAO,WAAW,KAAK;CACpC;AACD;AAEA,SAAS,4BACR,YACA,SACqB;CACrB,MAAM,UAAU,kBAAkB,UAAU;CAC5C,MAAM,WAAW,mBAAmB,UAAU;CAC9C,KAAK,MAAM,UAAU,SAAS;EAC7B,IAAI,CAAC,OAAO,cAAc,OAAO,MAAM,KAAK,OAAO,UAAU,GAC5D,OAAO,uBAAuB,OAAO,IAAI,eAAe,OAAO,OAAO;EAEvE,IAAI,YAAY,OAAO,SAAA,YACtB,OAAO,uBAAuB,OAAO,IAAI,eAAe,OAAO,OAAO,gDAAgD,sBAAsB;EAE7I,IAAI,OAAO,MAAM,WAAW,GAC3B,OAAO,uBAAuB,OAAO,IAAI;EAE1C,IAAI,SACC;OAAA,CAAC,OAAO,SACX,OAAO,uBAAuB,OAAO,IAAI,4BAA4B,WAAW;EAAA,OAE3E,IAAI,OAAO,KAAK,WAAW,GACjC,OAAO,uBAAuB,OAAO,IAAI;CAE3C;AAED;AAEA,SAAS,yBACR,YACA,SACqB;CACrB,KAAK,MAAM,UAAU,SAAS;EAC7B,IAAI,OAAO,WAAW,WAAW,GAChC,OAAO,uBAAuB,OAAO,IAAI;EAE1C,IAAI,OAAO,WAAW,WAAW,GAChC,OAAO,uBAAuB,OAAO,IAAI;EAE1C,IAAI,eAAA,UAAoD,CAAC,OAAO,aAC/D,OAAO,uBAAuB,OAAO,IAAI;CAE3C;AAED;;;;;;;AAQA,SAAgB,sBACf,KACA,YACA,cACA,QACoB;CACpB,MAAM,QAAQ,IAAI,8BAA8B,YAAY,cAAc,MAAM;CAChF,IAAI,MAAM,SAAS,SAAS,GAAG,OAAO;EAAE,OAAO,CAAC;EAAG,UAAU,MAAM;EAAU,aAAa;CAAK;CAE/F,MAAM,iBAAiB,IAAI,qBAAqB,YAAY,cAAc,MAAM;CAChF,OAAO;EAAE,OAAO,CAAC;EAAG,UAAU,CAAC,oBAAoB,YAAY,gBAAgB,YAAY,CAAC;EAAG,aAAa;CAAK;AAClH;;;;;;AAOA,SAAgB,uBACf,UACA,YACA,QACA,cACA,MACO;CACP,IAAI,KAAK,WAAW,MAAM;EACzB,oBAAoB,UAAU,YAAY,QAAQ,cAAc;GAC/D,cAAc;GACd,UAAU,CAAC,KAAK,OAAO;EACxB,CAAC;EACD;CACD;CAEA,qBAAqB,UAAU,kBAAkB,YAAY,QAAQ,cAAc,KAAK,MAAM,CAAC;AAChG;AAEA,SAAgB,iBAAiB,SAAyD;CACzF,MAAM,uBAAO,IAAI,IAAY;CAC7B,KAAK,MAAM,UAAU,SAAS;EAC7B,IAAI,KAAK,IAAI,OAAO,GAAG,GAAG,OAAO,OAAO;EAExC,KAAK,IAAI,OAAO,GAAG;CACpB;AAED"}

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

@@ -1,7 +1,7 @@
 import { mapBounded } from "../../../utils/promise.js";
+import "../constants.js";
 import { appendDedupedWarning, reconcileOmissionsWithFailure } from "../results.js";
 import { assessCollectionMetadata, mergeAssessmentInto } from "../collectionMetadata.js";
-import "../constants.js";
 import "../providerFilters.js";
 import { mergeCollectionMetadata, parsePageCursor } from "../providers/utils/providerPaging.js";
 import { isIssuesHostIntegrationId } from "../utils/integration.utils.js";

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

@@ -1,6 +1,6 @@
+import "../constants.js";
 import { reconcileOmissionsWithFailure } from "../results.js";
 import { mergeAssessmentInto } from "../collectionMetadata.js";
-import "../constants.js";
 import { mergeCollectionMetadata } from "../providers/utils/providerPaging.js";
 import { isGitHostIntegration, isIssuesHostIntegrationId, warnOnMissingSessionForDomain } from "../utils/integration.utils.js";
 import { isRepoIdsInput, providersMetadata } from "../providers/models.js";

package/dist/plus/integrations/reads/pullRequestBatch.d.ts

@@ -0,0 +1,57 @@
+import { PullRequestShape } from "../../../git/models/pullRequest.js";
+import { IntegrationIds } from "../constants.js";
+import { ProviderResult } from "../results.js";
+import { ProviderReadContext } from "./context.js";
+//#region ../plus/integrations/src/reads/pullRequestBatch.d.ts
+/**
+ * The BATCH pull request read: resolve N pull requests, in any state, BY COORDINATE — the pull-request twin of
+ * `getIssuesBatch`, and for the same reason. "Does this exact pull request exist" is an identity question; a
+ * paged list can only answer it by walking the whole scope, so a miss stays unproven and the walk repeats forever.
+ * Here an absent slot is a PROVEN ABSENCE a caller can cache.
+ *
+ * Deliberately NOT routed through `Integration.getPullRequest`: that read answers `undefined` both when not
+ * connected and on any failure, and caches through `IntegrationCacheProvider.getPullRequest`, whose key carries no
+ * connection and whose by-id bucket never expires a miss.
+ *
+ * ONE integration call per invocation, whatever the target count: the manager makes a single
+ * `getPullRequestsBatchResult` call and the integration fans its targets out (chunked for GitHub/GHE, one request
+ * per target with bounded concurrency everywhere else) and settles each independently, so one bad target never
+ * spends more than its own slot — see `GitHostIntegration.getPullRequestsBatchResult`.
+ */
+/** One pull request to resolve, identified by coordinate and echoed back under the caller's own `key`. */
+export interface PullRequestBatchTarget {
+  /** Caller-owned identifier, echoed on the result so no positional matching is needed. Must be unique. */
+  key: string;
+  /**
+   * GitHub owner, GitLab namespace path (may be nested), Bitbucket workspace, Bitbucket Data Center project key, or
+   * Azure DevOps organization.
+   */
+  owner: string;
+  /** Repository name or slug. */
+  repo: string;
+  /** Pull request number, GitLab merge request iid, or Azure DevOps pull request id. */
+  number: number;
+  /** Azure DevOps project. Required there, ignored elsewhere. */
+  project?: string;
+}
+/** The answer for one {@link PullRequestBatchTarget}. */
+export interface PullRequestBatchResult {
+  key: string;
+  /**
+   * The resolved pull request, whatever its state, or `undefined` when it PROVABLY does not exist (or is not
+   * visible to this connection). A target whose read FAILED is not returned at all and sets `fetchFailed`.
+   */
+  pullRequest?: PullRequestShape;
+}
+export declare function getPullRequestsBatch(ctx: ProviderReadContext, options: {
+  providerId: IntegrationIds;
+  targets: readonly PullRequestBatchTarget[];
+  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<PullRequestBatchResult>>;
+//#endregion
+//# sourceMappingURL=pullRequestBatch.d.ts.map