@gitkraken/core-gitlens 0.4.0 → 0.4.100

package/dist/plus/integrations/providers/models.js

@@ -1,27 +1,80 @@
-import { GitBuildStatusState, GitPullRequestMergeableState, GitPullRequestReviewState, GitPullRequestState, } from '@gitkraken/provider-apis';
-import { EntityIdentifierUtils } from '@gitkraken/provider-apis/entity-identifiers';
-import { GitProviderUtils } from '@gitkraken/provider-apis/provider-utils';
+import entityIdentifiersModule from '@gitkraken/provider-apis/entity-identifiers';
+import providerUtilsModule from '@gitkraken/provider-apis/provider-utils';
 import { Issue, RepositoryAccessLevel } from '../../../git/models/issue.js';
 import { PullRequest, PullRequestMergeableState, PullRequestReviewDecision, PullRequestReviewState, PullRequestStatusCheckRollupState, } from '../../../git/models/pullRequest.js';
 import { gitSuffixRegex } from '../../../git/utils/remote.utils.js';
 import { equalsIgnoreCase } from '../../../utils/string.js';
 import { GitCloudHostIntegrationId, GitSelfManagedHostIntegrationId, IssuesCloudHostIntegrationId, } from '../constants.js';
+import { IssueFilter, PullRequestFilter } from '../providerFilters.js';
+export { IssueFilter, PullRequestFilter } from '../providerFilters.js';
 import { getEntityIdentifierInput } from './utils.js';
+const { EntityIdentifierUtils } = entityIdentifiersModule;
+const { GitProviderUtils } = providerUtilsModule;
+// Local runtime copies of the `@gitkraken/provider-apis` string enums. Duplicated (not imported as
+// values) because the SDK ships as CJS whose named enum exports can't be statically imported as ESM
+// values from the bundled `dist`. Exported so the enum-parity test can deep-equal them against the
+// real SDK enums and turn any upstream drift into a test failure instead of a silent mismatch.
+export const GitBuildStatusState = {
+    ActionRequired: 'ACTION_REQUIRED',
+    Cancelled: 'CANCELLED',
+    Error: 'ERROR',
+    Failed: 'FAILED',
+    Pending: 'PENDING',
+    Running: 'RUNNING',
+    Skipped: 'SKIPPED',
+    Success: 'SUCCESS',
+    Warning: 'WARNING',
+    OptionalActionRequired: 'OPTIONAL_ACTION_REQUIRED',
+};
+export const GitIssueState = {
+    Open: 'OPEN',
+    Closed: 'CLOSED',
+};
+export const GitPullRequestState = {
+    Open: 'OPEN',
+    Closed: 'CLOSED',
+    Merged: 'MERGED',
+};
+export const GitPullRequestReviewState = {
+    Approved: 'APPROVED',
+    ChangesRequested: 'CHANGES_REQUESTED',
+    Commented: 'COMMENTED',
+    ReviewRequested: 'REVIEW_REQUESTED',
+};
+export const GitPullRequestMergeableState = {
+    Behind: 'BEHIND',
+    Blocked: 'BLOCKED',
+    Conflicts: 'CONFLICTS',
+    FailingChecks: 'FAILING_CHECKS',
+    Mergeable: 'MERGEABLE',
+    Unknown: 'UNKNOWN',
+    UnknownAndBlocked: 'UNKNOWN_AND_BLOCKED',
+    Unstable: 'UNSTABLE',
+};
+/**
+ * Returns an account-wide PR identity. Provider PR ids are commonly scoped to a repository or organization,
+ * so an id alone is never safe for deduplication across a multi-repository fan-out. Repository identity wins
+ * when available so the same row still deduplicates when a later facet adds or removes its URL.
+ */
+export function getProviderPullRequestIdentity(pr) {
+    const repository = pr.repository;
+    const urlIdentity = pr.url != null && pr.url.trim() !== '' ? `url:${pr.url}` : undefined;
+    if (repository == null)
+        return urlIdentity;
+    const owner = typeof repository.owner === 'string' ? repository.owner : repository.owner?.login;
+    const repositoryId = repository.id != null ? String(repository.id).trim() : '';
+    if (repositoryId !== '')
+        return `repository:${repositoryId}:pull-request:${pr.id}`;
+    const name = repository.name?.trim() ?? '';
+    const namespace = (repository.namespace ?? owner)?.trim() ?? '';
+    const project = repository.project?.trim() ?? '';
+    if (name !== '' && (namespace !== '' || project !== '')) {
+        return `repository:${namespace}/${project}/${name}:pull-request:${pr.id}`;
+    }
+    return urlIdentity;
+}
 export const ProviderPullRequestReviewState = GitPullRequestReviewState;
 export const ProviderBuildStatusState = GitBuildStatusState;
-export var PullRequestFilter;
-(function (PullRequestFilter) {
-    PullRequestFilter["Author"] = "author";
-    PullRequestFilter["Assignee"] = "assignee";
-    PullRequestFilter["ReviewRequested"] = "review-requested";
-    PullRequestFilter["Mention"] = "mention";
-})(PullRequestFilter || (PullRequestFilter = {}));
-export var IssueFilter;
-(function (IssueFilter) {
-    IssueFilter["Author"] = "author";
-    IssueFilter["Assignee"] = "assignee";
-    IssueFilter["Mention"] = "mention";
-})(IssueFilter || (IssueFilter = {}));
 export var PagingMode;
 (function (PagingMode) {
     PagingMode["Project"] = "project";
@@ -44,8 +97,17 @@ export const providersMetadata = {
             PullRequestFilter.ReviewRequested,
             PullRequestFilter.Mention,
         ],
+        supportedAccountWidePullRequestFilters: [
+            PullRequestFilter.Author,
+            PullRequestFilter.Assignee,
+            PullRequestFilter.ReviewRequested,
+            PullRequestFilter.Mention,
+        ],
         // Use 'username' property on account for issue filters
         supportedIssueFilters: [IssueFilter.Author, IssueFilter.Assignee, IssueFilter.Mention],
+        // The account-wide read is three independent searches (`author:@me`, `assignee:@me`, `mentions:@me`) behind
+        // one composite cursor, so any subset of them is expressible.
+        supportedAccountWideIssueFilters: [IssueFilter.Author, IssueFilter.Assignee, IssueFilter.Mention],
         scopes: ['repo', 'read:user', 'user:email'],
     },
     [GitSelfManagedHostIntegrationId.CloudGitHubEnterprise]: {
@@ -63,8 +125,17 @@ export const providersMetadata = {
             PullRequestFilter.ReviewRequested,
             PullRequestFilter.Mention,
         ],
+        supportedAccountWidePullRequestFilters: [
+            PullRequestFilter.Author,
+            PullRequestFilter.Assignee,
+            PullRequestFilter.ReviewRequested,
+            PullRequestFilter.Mention,
+        ],
         // Use 'username' property on account for issue filters
         supportedIssueFilters: [IssueFilter.Author, IssueFilter.Assignee, IssueFilter.Mention],
+        // The account-wide read is three independent searches (`author:@me`, `assignee:@me`, `mentions:@me`) behind
+        // one composite cursor, so any subset of them is expressible.
+        supportedAccountWideIssueFilters: [IssueFilter.Author, IssueFilter.Assignee, IssueFilter.Mention],
         scopes: ['repo', 'read:user', 'user:email'],
     },
     [GitCloudHostIntegrationId.GitLab]: {
@@ -81,8 +152,17 @@ export const providersMetadata = {
             PullRequestFilter.Assignee,
             PullRequestFilter.ReviewRequested,
         ],
+        supportedAccountWidePullRequestFilters: [
+            PullRequestFilter.Author,
+            PullRequestFilter.Assignee,
+            PullRequestFilter.ReviewRequested,
+        ],
         // Use 'username' property on account for issue filters
         supportedIssueFilters: [IssueFilter.Author, IssueFilter.Assignee],
+        // The account-wide read drains one `scope=assigned_to_me` pass and one `scope=all` + `authorUsername` pass,
+        // unioned, so either axis is expressible on its own. Mention is absent because GitLab's REST issue read has
+        // no first-class mention filter to narrow with.
+        supportedAccountWideIssueFilters: [IssueFilter.Assignee, IssueFilter.Author],
         scopes: ['api', 'read_user', 'read_repository'],
     },
     [GitSelfManagedHostIntegrationId.CloudGitLabSelfHosted]: {
@@ -99,8 +179,17 @@ export const providersMetadata = {
             PullRequestFilter.Assignee,
             PullRequestFilter.ReviewRequested,
         ],
+        supportedAccountWidePullRequestFilters: [
+            PullRequestFilter.Author,
+            PullRequestFilter.Assignee,
+            PullRequestFilter.ReviewRequested,
+        ],
         // Use 'username' property on account for issue filters
         supportedIssueFilters: [IssueFilter.Author, IssueFilter.Assignee],
+        // The account-wide read drains one `scope=assigned_to_me` pass and one `scope=all` + `authorUsername` pass,
+        // unioned, so either axis is expressible on its own. Mention is absent because GitLab's REST issue read has
+        // no first-class mention filter to narrow with.
+        supportedAccountWideIssueFilters: [IssueFilter.Assignee, IssueFilter.Author],
         scopes: ['api', 'read_user', 'read_repository'],
     },
     [GitCloudHostIntegrationId.Bitbucket]: {
@@ -110,8 +199,9 @@ export const providersMetadata = {
         type: 'git',
         iconKey: GitCloudHostIntegrationId.Bitbucket,
         pullRequestsPagingMode: PagingMode.Repo,
-        // Use 'id' property on account for PR filters
-        supportedPullRequestFilters: [PullRequestFilter.Author],
+        // Use 'id' property on account for PR filters (reviewer filter keyed by account id / reviewerId)
+        supportedPullRequestFilters: [PullRequestFilter.Author, PullRequestFilter.ReviewRequested],
+        supportedAccountWidePullRequestFilters: [PullRequestFilter.Author, PullRequestFilter.ReviewRequested],
         scopes: ['account:read', 'repository:read', 'pullrequest:read', 'issue:read'],
     },
     [GitSelfManagedHostIntegrationId.BitbucketServer]: {
@@ -121,6 +211,7 @@ export const providersMetadata = {
         type: 'git',
         iconKey: GitSelfManagedHostIntegrationId.BitbucketServer,
         supportedPullRequestFilters: [PullRequestFilter.Author, PullRequestFilter.ReviewRequested],
+        supportedAccountWidePullRequestFilters: [PullRequestFilter.Author, PullRequestFilter.ReviewRequested],
         scopes: ['Project (Read)', 'Repository (Write)'],
     },
     [GitCloudHostIntegrationId.AzureDevOps]: {
@@ -131,10 +222,22 @@ export const providersMetadata = {
         iconKey: GitCloudHostIntegrationId.AzureDevOps,
         issuesPagingMode: PagingMode.Project,
         pullRequestsPagingMode: PagingMode.Repo,
-        // Use 'id' property on account for PR filters
-        supportedPullRequestFilters: [PullRequestFilter.Author, PullRequestFilter.Assignee],
+        // Use 'id' property on account for PR filters (reviewer filter keyed by account id / reviewerId)
+        supportedPullRequestFilters: [
+            PullRequestFilter.Author,
+            PullRequestFilter.Assignee,
+            PullRequestFilter.ReviewRequested,
+        ],
+        supportedAccountWidePullRequestFilters: [
+            PullRequestFilter.Author,
+            PullRequestFilter.Assignee,
+            PullRequestFilter.ReviewRequested,
+        ],
         // Use 'name' property on account for issue filters
         supportedIssueFilters: [IssueFilter.Author, IssueFilter.Assignee, IssueFilter.Mention],
+        // The account-wide read drains one (project × assignee) and one (project × author) query per project, so
+        // either axis is expressible on its own. There is no mention query to narrow to.
+        supportedAccountWideIssueFilters: [IssueFilter.Author, IssueFilter.Assignee],
         scopes: ['vso.code', 'vso.identity', 'vso.project', 'vso.profile', 'vso.work'],
     },
     [GitSelfManagedHostIntegrationId.AzureDevOpsServer]: {
@@ -145,10 +248,22 @@ export const providersMetadata = {
         iconKey: GitCloudHostIntegrationId.AzureDevOps,
         issuesPagingMode: PagingMode.Project,
         pullRequestsPagingMode: PagingMode.Repo,
-        // Use 'id' property on account for PR filters
-        supportedPullRequestFilters: [PullRequestFilter.Author, PullRequestFilter.Assignee],
+        // Use 'id' property on account for PR filters (reviewer filter keyed by account id / reviewerId)
+        supportedPullRequestFilters: [
+            PullRequestFilter.Author,
+            PullRequestFilter.Assignee,
+            PullRequestFilter.ReviewRequested,
+        ],
+        supportedAccountWidePullRequestFilters: [
+            PullRequestFilter.Author,
+            PullRequestFilter.Assignee,
+            PullRequestFilter.ReviewRequested,
+        ],
         // Use 'name' property on account for issue filters
         supportedIssueFilters: [IssueFilter.Author, IssueFilter.Assignee, IssueFilter.Mention],
+        // The account-wide read drains one (project × assignee) and one (project × author) query per project, so
+        // either axis is expressible on its own. There is no mention query to narrow to.
+        supportedAccountWideIssueFilters: [IssueFilter.Author, IssueFilter.Assignee],
         scopes: ['vso.code', 'vso.identity', 'vso.project', 'vso.profile', 'vso.work'],
     },
     [IssuesCloudHostIntegrationId.Jira]: {
@@ -202,6 +317,8 @@ export const providersMetadata = {
         type: 'issues',
         iconKey: IssuesCloudHostIntegrationId.Linear,
         scopes: [],
+        // Linear scopes "my issues" client-side by the viewer's assignee id; author/mention aren't supported.
+        supportedIssueFilters: [IssueFilter.Assignee],
     },
     [IssuesCloudHostIntegrationId.Trello]: {
         domain: 'trello.com',
@@ -210,6 +327,8 @@ export const providersMetadata = {
         type: 'issues',
         iconKey: IssuesCloudHostIntegrationId.Trello,
         scopes: [],
+        // Trello cards are filtered by the assignee (member) only; author/mention have no Trello equivalent.
+        supportedIssueFilters: [IssueFilter.Assignee],
     },
 };
 export function getReasonsForUserIssue(issue, userLogin) {
@@ -231,14 +350,28 @@ export function getReasonsForUserIssue(issue, userLogin) {
     }
     return reasons;
 }
+function toIssueIdentifier(value) {
+    return String(value);
+}
 export function toIssueShape(issue, provider) {
     // TODO: Add some protections/baselines rather than killing the transformation here
-    if (issue.updatedDate == null || issue.author == null || issue.url == null)
+    // `author` is intentionally not required: some providers have no per-item creator (e.g. Trello cards,
+    // which the SDK maps with `author: null`), and dropping every such item would discard the whole board.
+    // Fall back to an empty author instead so these issues still surface.
+    if (issue.updatedDate == null || issue.url == null)
         return undefined;
     return {
         type: 'issue',
         provider: provider,
-        id: issue.number,
+        // `id` is the provider's display number/key (GitHub number, Jira/Linear key, Trello idShort), NOT a
+        // globally-unique id: it's rendered to users as `#{id}`, used to build branch names, and passed back
+        // to the provider as the `getIssue`/cache lookup key, all of which expect the number. `nodeId` is the
+        // provider-native stable id, but its uniqueness scope is provider-specific (Azure work-item ids are
+        // organization-scoped), so cross-scope consumers must include provider/domain/container identity too.
+        // provider-apis declares this as a string, but some providers (notably
+        // Azure's broadened work-item path) return a number at runtime. Keep the
+        // public IssueShape contract truthful for consumers that use string APIs.
+        id: toIssueIdentifier(issue.number),
         nodeId: issue.graphQLId ?? issue.id,
         title: issue.title,
         url: issue.url,
@@ -248,33 +381,56 @@ export function toIssueShape(issue, provider) {
         closed: issue.closedDate != null,
         state: issue.closedDate != null ? 'closed' : 'opened',
         author: {
-            id: issue.author.id ?? '',
-            name: issue.author.name ?? '',
-            avatarUrl: issue.author.avatarUrl ?? undefined,
-            url: issue.author.url ?? undefined,
+            id: issue.author?.id ?? '',
+            // An absent name stays absent, matching {@link fromProviderAccount}; see `IssueMember.name`.
+            name: issue.author?.name ?? undefined,
+            avatarUrl: issue.author?.avatarUrl ?? undefined,
+            url: issue.author?.url ?? undefined,
         },
         assignees: issue.assignees?.map(assignee => ({
             id: assignee.id ?? '',
-            name: assignee.name ?? '',
+            name: assignee.name ?? undefined,
             avatarUrl: assignee.avatarUrl ?? undefined,
             url: assignee.url ?? undefined,
         })) ?? [],
-        project: {
-            id: issue.project?.id ?? '',
-            name: issue.project?.name ?? '',
-            resourceId: issue.project?.resourceId ?? '',
-            resourceName: issue.project?.namespace ?? '',
-        },
+        project: issue.project?.id && issue.project.resourceId && issue.project.namespace
+            ? {
+                id: issue.project.id,
+                name: issue.project.name,
+                resourceId: issue.project.resourceId,
+                resourceName: issue.project.namespace,
+            }
+            : undefined,
         repository: issue.repository?.owner?.login != null
             ? {
                 owner: issue.repository.owner.login,
                 repo: issue.repository.name,
+                id: issue.repository.id,
             }
             : undefined,
         labels: issue.labels.map(label => ({ color: label.color ?? undefined, name: label.name })),
         commentsCount: issue.commentCount ?? undefined,
         thumbsUpCount: issue.upvoteCount ?? undefined,
         body: issue.description ?? undefined,
+        issueType: issue.type ?? undefined,
+    };
+}
+/**
+ * Maps a raw provider-apis {@link ProviderRepository} to the GitLens-owned {@link ProviderRepositoryShape}
+ * the ProviderBackend `listRepos` facade surfaces, so consumers don't depend on the SDK repo type. The SDK's
+ * nullable fields (`webUrl`/`httpsUrl`/`sshUrl`/`defaultBranch`) collapse to `undefined`, matching the
+ * `?? undefined` convention in {@link toIssueShape}/{@link toAccount}.
+ */
+export function toProviderRepositoryShape(repo) {
+    return {
+        id: repo.id,
+        namespace: repo.namespace,
+        name: repo.name,
+        project: repo.project ?? undefined,
+        url: repo.webUrl ?? undefined,
+        cloneUrlHttps: repo.httpsUrl ?? undefined,
+        cloneUrlSsh: repo.sshUrl ?? undefined,
+        defaultBranch: repo.defaultBranch?.name ?? undefined,
     };
 }
 export function issueFilterToReason(filter) {
@@ -411,16 +567,77 @@ export function toProviderPullRequestState(state) {
 export function fromProviderPullRequestState(state) {
     return state === GitPullRequestState.Open ? 'opened' : state === GitPullRequestState.Closed ? 'closed' : 'merged';
 }
+function toProviderPullRequestStatesCore(state) {
+    switch (state) {
+        case 'open':
+        case 'opened':
+            return [GitPullRequestState.Open];
+        case 'closed':
+            return [GitPullRequestState.Closed];
+        case 'merged':
+            return [GitPullRequestState.Merged];
+        case 'all':
+            return [GitPullRequestState.Open, GitPullRequestState.Closed, GitPullRequestState.Merged];
+    }
+}
+/** Maps PR include/state filters to the SDK's `states` input. `undefined`/omitted preserves the open-only default. */
+export function toProviderPullRequestStates(state) {
+    if (state == null)
+        return undefined;
+    const states = (Array.isArray(state) ? state : [state]).flatMap(s => toProviderPullRequestStatesCore(s));
+    return states.length > 0 ? [...new Set(states)] : undefined;
+}
+/** Maps an issue state filter to the SDK's `states` input. `undefined`/omitted preserves the open-only default. */
+export function toProviderIssueStates(state) {
+    switch (state) {
+        case 'open':
+            return [GitIssueState.Open];
+        case 'closed':
+            return [GitIssueState.Closed];
+        case 'all':
+            return [GitIssueState.Open, GitIssueState.Closed];
+        default:
+            return undefined;
+    }
+}
+/**
+ * Resolves a normalized {@link ProviderScope} to the provider-appropriate read inputs, dispatching on the
+ * provider's {@link PagingMode} (the authoritative per-provider scoping selector). Project-mode providers
+ * (Azure issues, Jira) produce `projectInputs` from `org` + `project`; repo/repos-mode providers produce a
+ * repo `GetRepoInput[]` from `scope.repos` (carrying `project` where relevant, e.g. Azure). This is the
+ * single public entry point consumers can use instead of hand-building the three underlying representations.
+ */
+export function resolveProviderScope(scope, pagingMode) {
+    if (pagingMode === PagingMode.Project) {
+        if (scope.org != null && scope.project != null) {
+            return { projectInputs: [{ namespace: scope.org, project: scope.project }] };
+        }
+        return {};
+    }
+    const reposInput = scope.repos?.map(r => ({ namespace: r.owner, name: r.name, project: scope.project }));
+    return { reposInput: reposInput };
+}
+export function providerPullRequestMatchesSearch(pr, search) {
+    const term = search.trim().toLowerCase();
+    if (term.length === 0)
+        return true;
+    return pr.title.toLowerCase().includes(term) || (pr.description?.toLowerCase().includes(term) ?? false);
+}
+function toProviderRemoteInfo(ref) {
+    // Match the SDK convention: only populate remoteInfo when both clone URLs are known.
+    return ref?.cloneHttps && ref?.cloneSsh ? { cloneUrlHTTPS: ref.cloneHttps, cloneUrlSSH: ref.cloneSsh } : null;
+}
 export function toProviderPullRequest(pr) {
     const prReviews = [...(pr.reviewRequests ?? []), ...(pr.latestReviews ?? [])];
     return {
         id: pr.id,
         graphQLId: pr.nodeId,
-        number: Number.parseInt(pr.id, 10),
+        number: pr.number ?? Number.parseInt(pr.id, 10),
         title: pr.title,
-        description: null,
+        description: pr.body ?? null,
         url: pr.url,
         state: toProviderPullRequestState(pr.state),
+        isCrossRepository: pr.refs?.isCrossRepository ?? false,
         isDraft: pr.isDraft ?? false,
         createdDate: pr.createdDate,
         updatedDate: pr.updatedDate,
@@ -450,12 +667,12 @@ export function toProviderPullRequest(pr) {
         reviewDecision: toProviderReviewDecision(pr.reviewDecision, prReviews),
         repository: pr.repository != null
             ? {
-                id: pr.repository.repo,
+                id: pr.repository.id ?? '',
                 name: pr.repository.repo,
                 owner: {
                     login: pr.repository.owner,
                 },
-                remoteInfo: null, // TODO: Add the urls to our model
+                remoteInfo: toProviderRemoteInfo(pr.refs?.base),
             }
             : {
                 id: '',
@@ -472,7 +689,8 @@ export function toProviderPullRequest(pr) {
                 owner: {
                     login: pr.refs.head.owner,
                 },
-                remoteInfo: null,
+                remoteInfo: toProviderRemoteInfo(pr.refs.head),
+                isFork: pr.refs.head.isFork,
             }
             : null,
         headCommit: pr.statusCheckRollupState != null
@@ -506,41 +724,58 @@ export function toProviderPullRequest(pr) {
     };
 }
 export function fromProviderPullRequest(pr, provider, options) {
+    const repository = pr.repository;
+    const repositoryName = repository?.name ?? '';
+    const repositoryOwner = repository?.owner?.login ?? '';
+    const repositoryRemoteInfo = repository?.remoteInfo;
+    const headRepository = pr.headRepository;
+    const headRepositoryRemoteInfo = headRepository?.remoteInfo;
     return new PullRequest(provider, fromProviderAccount(pr.author), pr.id, pr.graphQLId || pr.id, pr.title, pr.url ?? '', {
-        owner: pr.repository.owner.login,
-        repo: pr.repository.name,
+        owner: repositoryOwner,
+        repo: repositoryName,
         // This has to be here until we can take this information from ProviderPullRequest:
         accessLevel: RepositoryAccessLevel.Write,
-        id: pr.repository.id,
+        id: repository?.id ?? '',
     }, fromProviderPullRequestState(pr.state), pr.createdDate, pr.updatedDate, pr.closedDate ?? undefined, pr.mergedDate ?? undefined, pr.mergeableState ? fromProviderPullRequestMergeableState[pr.mergeableState] : undefined, pr.permissions?.canMerge || pr.permissions?.canMergeAndBypassProtections ? true : undefined, {
         base: {
             branch: pr.baseRef?.name ?? '',
             sha: pr.baseRef?.oid ?? '',
-            repo: pr.repository.name,
-            owner: pr.repository.owner.login,
+            repo: repositoryName,
+            owner: repositoryOwner,
             exists: pr.baseRef != null,
-            url: pr.repository.remoteInfo?.cloneUrlHTTPS
-                ? pr.repository.remoteInfo.cloneUrlHTTPS.replace(gitSuffixRegex, '')
+            url: repositoryRemoteInfo?.cloneUrlHTTPS
+                ? repositoryRemoteInfo.cloneUrlHTTPS.replace(gitSuffixRegex, '')
                 : '',
+            cloneHttps: repositoryRemoteInfo?.cloneUrlHTTPS || undefined,
+            cloneSsh: repositoryRemoteInfo?.cloneUrlSSH || undefined,
         },
         head: {
             branch: pr.headRef?.name ?? '',
             sha: pr.headRef?.oid ?? '',
-            repo: pr.headRepository?.name ?? '',
-            owner: pr.headRepository?.owner.login ?? '',
+            repo: headRepository?.name ?? '',
+            owner: headRepository?.owner?.login ?? '',
             exists: pr.headRef != null,
-            url: pr.headRepository?.remoteInfo?.cloneUrlHTTPS
-                ? pr.headRepository.remoteInfo.cloneUrlHTTPS.replace(gitSuffixRegex, '')
+            url: headRepositoryRemoteInfo?.cloneUrlHTTPS
+                ? headRepositoryRemoteInfo.cloneUrlHTTPS.replace(gitSuffixRegex, '')
                 : '',
+            cloneHttps: headRepositoryRemoteInfo?.cloneUrlHTTPS || undefined,
+            cloneSsh: headRepositoryRemoteInfo?.cloneUrlSSH || undefined,
+            isFork: headRepository?.isFork,
         },
-        isCrossRepository: pr.headRepository?.id !== pr.repository.id,
+        isCrossRepository: pr.isCrossRepository,
     }, pr.isDraft, pr.additions ?? undefined, pr.deletions ?? undefined, pr.commentCount ?? undefined, pr.upvoteCount ?? undefined, pr.reviewDecision ? fromPullRequestReviewDecision[pr.reviewDecision] : undefined, toReviewRequests(pr.reviews), toCompletedReviews(pr.reviews), pr.assignees?.map(fromProviderAccount) ?? undefined, pr.headCommit?.buildStatuses?.[0]?.state
         ? fromProviderBuildStatusState[pr.headCommit.buildStatuses[0].state]
-        : undefined, options?.project, pr.version);
+        : undefined, options?.project, pr.version, pr.description ?? undefined, pr.number, options?.currentAccountId != null ? pr.author?.id === options.currentAccountId : undefined);
 }
 export function fromProviderIssue(issue, integration, options) {
-    return new Issue(integration, issue.id, issue.graphQLId, issue.title, issue.url ?? '', issue.createdDate, issue.updatedDate ?? issue.closedDate ?? issue.createdDate, issue.closedDate != null, issue.closedDate != null ? 'closed' : 'opened', fromProviderAccount(issue.author), issue.assignees?.map(fromProviderAccount) ?? undefined, undefined, // TODO: issue repo
-    issue.closedDate ?? undefined, undefined, issue.commentCount ?? undefined, issue.upvoteCount ?? undefined, issue.description ?? undefined, options?.project != null
+    const identifier = toIssueIdentifier(issue.number);
+    return new Issue(integration, identifier, issue.graphQLId ?? issue.id, issue.title, issue.url ?? '', issue.createdDate, issue.updatedDate ?? issue.closedDate ?? issue.createdDate, issue.closedDate != null, issue.closedDate != null ? 'closed' : 'opened', fromProviderAccount(issue.author), issue.assignees?.map(fromProviderAccount) ?? undefined, issue.repository != null
+        ? {
+            owner: issue.repository.owner.login ?? '',
+            repo: issue.repository.name,
+            id: issue.repository.id,
+        }
+        : undefined, issue.closedDate ?? undefined, issue.labels?.map(label => ({ name: label.name, color: label.color ?? undefined })), issue.commentCount ?? undefined, issue.upvoteCount ?? undefined, issue.description ?? undefined, options?.project != null
         ? {
             id: options.project.id,
             name: options.project.name,
@@ -554,7 +789,7 @@ export function fromProviderIssue(issue, integration, options) {
                 resourceId: issue.project.resourceId,
                 resourceName: issue.project.namespace,
             }
-            : undefined, issue.number);
+            : undefined, identifier, issue.type ?? undefined);
 }
 export function toProviderPullRequestWithUniqueId(pr) {
     return {
@@ -576,9 +811,19 @@ export function toProviderAccount(account) {
 export function fromProviderAccount(account) {
     return {
         id: account?.id ?? '',
-        name: account?.name ?? 'unknown',
+        // An absent name stays absent. `'unknown'` was a display fallback invented in the provider layer, and a
+        // consumer couldn't tell it apart from a member genuinely named that — so it couldn't be undone where a
+        // name-shaped placeholder is wrong (rendering an avatar-only chip, or building an AI prompt, where
+        // `Assignees: unknown` reads as a real assignee). It also disagreed with {@link toIssueShape}, which
+        // collapsed the same absent name to `''` — same facade method, two fallbacks. Both now emit `undefined`.
+        name: account?.name ?? undefined,
         avatarUrl: account?.avatarUrl ?? undefined,
-        url: account?.url ?? '',
+        // `url` is optional, so an absent one must be `undefined`, not `''`. `''` passes a `!= null` presence
+        // check and renders as a link to nowhere, and it disagreed with {@link toIssueShape} — which already
+        // collapses to `undefined` — even though BOTH mappers feed `listIssuesPage` (the repo-scoped path goes
+        // through `toIssueShape`, Azure's account-wide path through {@link fromProviderIssue}). Same facade
+        // method, two shapes. Matches {@link toProviderRepositoryShape}, which collapses every absent optional.
+        url: account?.url ?? undefined,
     };
 }
 export const getActionablePullRequests = GitProviderUtils.getActionablePullRequests;
@@ -588,7 +833,10 @@ export function isGitHubDotCom(domain) {
 export function isGitLabDotCom(domain) {
     return equalsIgnoreCase(domain, 'gitlab.com');
 }
-const azureCloudDomainRegex = /^dev\.azure\.com$|\bvisualstudio\.com$/i;
+// Anchor the `visualstudio.com` alternative on a label boundary (`^` or `.`) rather than `\b`: `\b` treats a
+// hyphen as a word boundary, so `\bvisualstudio\.com$` matched attacker-controlled hosts like
+// `evil-visualstudio.com` and classified them as Azure cloud, changing their auth/URL handling.
+const azureCloudDomainRegex = /^dev\.azure\.com$|(?:^|\.)visualstudio\.com$/i;
 export function isAzureCloudDomain(domain) {
     return domain != null && azureCloudDomainRegex.test(domain);
 }