Improve issue conversion from Bitbucket's model to GitLens model

(#4047, #4119)

Author: sergeibbb

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

@@ -1,4 +1,5 @@
-import { RepositoryAccessLevel } from '../../../../git/models/issue';
+import type { IssueRepository } from '../../../../git/models/issue';
+import { Issue, RepositoryAccessLevel } from '../../../../git/models/issue';
 import type { IssueOrPullRequestState } from '../../../../git/models/issueOrPullRequest';
 import type { PullRequestMember, PullRequestReviewer } from '../../../../git/models/pullRequest';
 import { PullRequest, PullRequestReviewDecision, PullRequestReviewState } from '../../../../git/models/pullRequest';
@@ -191,6 +192,12 @@ export interface BitbucketIssue {
 	created_on: string;
 	updated_on: string;
 	repository: BitbucketRepository;
+	votes?: number;
+	content: {
+		raw: string;
+		markup: string;
+		html: string;
+	};
 	links: {
 		self: BitbucketLink;
 		html: BitbucketLink;
@@ -235,6 +242,10 @@ export function isClosedBitbucketPullRequestState(state: BitbucketPullRequestSta
 	return bitbucketPullRequestStateToState(state) !== 'opened';
 }
 
+export function isClosedBitbucketIssueState(state: BitbucketIssueState): boolean {
+	return bitbucketIssueStateToState(state) !== 'opened';
+}
+
 export function fromBitbucketUser(user: BitbucketUser): PullRequestMember {
 	return {
 		avatarUrl: user.links.avatar.href,
@@ -290,6 +301,46 @@ function getBitbucketReviewDecision(pr: BitbucketPullRequest): PullRequestReview
 	return PullRequestReviewDecision.ReviewRequired; // nobody has reviewed yet.
 }
 
+function fromBitbucketRepository(repo: BitbucketRepository): IssueRepository {
+	return {
+		owner: repo.full_name.split('/')[0],
+		repo: repo.name,
+		id: repo.uuid,
+		// TODO: Remove this assumption once actual access level is available
+		accessLevel: RepositoryAccessLevel.Write,
+	};
+}
+
+export function fromBitbucketIssue(issue: BitbucketIssue, provider: Provider): Issue {
+	return new Issue(
+		provider,
+		issue.id.toString(),
+		issue.id.toString(),
+		issue.title,
+		issue.links.html.href,
+		new Date(issue.created_on),
+		new Date(issue.updated_on),
+		isClosedBitbucketIssueState(issue.state),
+		bitbucketIssueStateToState(issue.state),
+		fromBitbucketUser(issue.reporter),
+		issue.assignee ? [fromBitbucketUser(issue.assignee)] : [],
+		fromBitbucketRepository(issue.repository),
+		undefined, // closedDate
+		undefined, // labels
+		undefined, // commentsCount
+		issue.votes, // thumbsUpCount
+		issue.content.html, // body
+		!issue.repository?.project
+			? undefined
+			: {
+					id: issue.repository.project.uuid,
+					name: issue.repository.project.name,
+					resourceId: issue.repository.project.uuid,
+					resourceName: issue.repository.project.name,
+			  },
+	);
+}
+
 export function fromBitbucketPullRequest(pr: BitbucketPullRequest, provider: Provider): PullRequest {
 	return new PullRequest(
 		provider,
@@ -298,13 +349,7 @@ export function fromBitbucketPullRequest(pr: BitbucketPullRequest, provider: Pro
 		pr.id.toString(),
 		pr.title,
 		pr.links.html.href,
-		{
-			owner: pr.destination.repository.full_name.split('/')[0],
-			repo: pr.destination.repository.name,
-			id: pr.destination.repository.uuid,
-			// TODO: Remove this assumption once actual access level is available
-			accessLevel: RepositoryAccessLevel.Write,
-		},
+		fromBitbucketRepository(pr.destination.repository),
 		bitbucketPullRequestStateToState(pr.state),
 		new Date(pr.created_on),
 		new Date(pr.updated_on),