Fetches issue data for linear links in commit tooltips

(#4543, #4579)

Author: sergeibbb

src/autolinks/autolinksProvider.ts

@@ -178,8 +178,11 @@ export class AutolinksProvider implements Disposable {
 	}
 
 	getAutolinkEnrichableId(autolink: Autolink): string {
+		// TODO: this should return linking key for all types of providers: such as TST-123 or #89 or PR 89 (or a pair: key+id).
+		// Each provider should form whatever ID they need in their specific getIssueOrPullRequest() method.
 		switch (autolink.provider?.id) {
 			case IssuesCloudHostIntegrationId.Jira:
+			case IssuesCloudHostIntegrationId.Linear:
 				return `${autolink.prefix}${autolink.id}`;
 			default:
 				return autolink.id;

src/plus/integrations/providers/linear.ts

@@ -10,7 +10,7 @@ import { Logger } from '../../../system/logger';
 import type { IntegrationAuthenticationProviderDescriptor } from '../authentication/integrationAuthenticationProvider';
 import type { ProviderAuthenticationSession } from '../authentication/models';
 import { IssuesIntegration } from '../models/issuesIntegration';
-import type { IssueFilter } from './models';
+import type { IssueFilter, ProviderIssue } from './models';
 import { fromProviderIssue, providersMetadata, toIssueShape } from './models';
 
 const metadata = providersMetadata[IssuesCloudHostIntegrationId.Linear];
@@ -210,19 +210,29 @@ export class LinearIntegration extends IssuesIntegration<IssuesCloudHostIntegrat
 		}
 		return issues;
 	}
-	protected override getProviderIssueOrPullRequest(
-		_session: ProviderAuthenticationSession,
-		_resource: ResourceDescriptor,
-		_id: string,
+	protected override async getProviderIssueOrPullRequest(
+		session: ProviderAuthenticationSession,
+		resource: ResourceDescriptor,
+		id: string,
 		_type: undefined | IssueOrPullRequestType,
 	): Promise<IssueOrPullRequest | undefined> {
-		throw new Error('Method not implemented.');
+		const issue = await this.getRawProviderIssue(session, resource, id);
+		return issue && toIssueShape(issue, this);
 	}
 	protected override async getProviderIssue(
 		session: ProviderAuthenticationSession,
 		resource: ResourceDescriptor,
 		id: string,
 	): Promise<Issue | undefined> {
+		const result = await this.getRawProviderIssue(session, resource, id);
+		return result && fromProviderIssue(result, this);
+	}
+
+	private async getRawProviderIssue(
+		session: ProviderAuthenticationSession,
+		resource: ResourceDescriptor,
+		id: string,
+	): Promise<ProviderIssue | undefined> {
 		const api = await this.getProvidersApi();
 		try {
 			if (!isIssueResourceDescriptor(resource)) {
@@ -243,7 +253,7 @@ export class LinearIntegration extends IssuesIntegration<IssuesCloudHostIntegrat
 
 			if (result == null) return undefined;
 
-			return fromProviderIssue(result, this);
+			return result;
 		} catch (ex) {
 			Logger.error(ex, 'getProviderIssue');
 			return undefined;