Revert "Include branch protection on overview (#3411)"

This reverts commit bb9520b.

Work around for #3430 until a real fix is made after the 0.40.0 release.

Author: alexr00

src/github/credentials.ts

@@ -284,7 +284,7 @@ export class CredentialStore implements vscode.Disposable {
 			request: { agent, fetch: fetchCore },
 			userAgent: 'GitHub VSCode Pull Requests',
 			// `shadow-cat-preview` is required for Draft PR API access -- https://developer.github.com/v3/previews/#draft-pull-requests
-			previews: ['shadow-cat-preview', 'merge-info-preview'],
+			previews: ['shadow-cat-preview'],
 			auth: `${token || ''}`,
 			baseUrl: baseUrl,
 		});
@@ -321,7 +321,7 @@ const link = (url: string, token: string) =>
 		headers: {
 			...headers,
 			authorization: token ? `Bearer ${token}` : '',
-			Accept: 'application/vnd.github.shadow-cat-preview+json, application/vnd.github.antiope-preview+json, application/vnd.github.merge-info-preview+json',
+			Accept: 'application/vnd.github.shadow-cat-preview+json, application/vnd.github.antiope-preview+json',
 		},
 	})).concat(
 		createHttpLink({

src/github/githubRepository.ts

@@ -701,6 +701,7 @@ export class GitHubRepository implements vscode.Disposable {
 				},
 			});
 			Logger.debug(`Fetch pull request ${id} - done`, GitHubRepository.ID);
+
 			return this.createOrUpdatePullRequestModel(parseGraphQLPullRequest(data, this));
 		} catch (e) {
 			Logger.appendLine(`GithubRepository> Unable to fetch PR: ${e}`);
@@ -723,7 +724,7 @@ export class GitHubRepository implements vscode.Disposable {
 			});
 			Logger.debug(`Fetch issue ${id} - done`, GitHubRepository.ID);
 
-			return new IssueModel(this, remote, parseGraphQLIssue(data.repository.pullRequest, this));
+			return new IssueModel(this, remote, parseGraphQLPullRequest(data, this));
 		} catch (e) {
 			Logger.appendLine(`GithubRepository> Unable to fetch PR: ${e}`);
 			return;

src/github/graphql.ts

@@ -417,7 +417,6 @@ export interface PullRequest {
 	};
 	merged: boolean;
 	mergeable: 'MERGEABLE' | 'CONFLICTING' | 'UNKNOWN';
-	mergeStateStatus: 'BEHIND' | 'BLOCKED' | 'CLEAN' | 'DIRTY' | 'HAS_HOOKS' | 'UNKNOWN' | 'UNSTABLE';
 	isDraft?: boolean;
 	suggestedReviewers: SuggestedReviewerResponse[];
 	milestone?: {
@@ -442,16 +441,6 @@ export interface PullRequestResponse {
 	rateLimit: RateLimit;
 }
 
-export interface PullRequestMergabilityResponse {
-	repository: {
-		pullRequest: {
-			mergeable: 'MERGEABLE' | 'CONFLICTING' | 'UNKNOWN';
-			mergeStateStatus: 'BEHIND' | 'BLOCKED' | 'CLEAN' | 'DIRTY' | 'HAS_HOOKS' | 'UNKNOWN' | 'UNSTABLE';
-		};
-	};
-	rateLimit: RateLimit;
-}
-
 export interface IssuesSearchResponse {
 	search: {
 		issueCount: number;

src/github/interface.ts

@@ -24,7 +24,6 @@ export enum GithubItemStateEnum {
 export enum PullRequestMergeability {
 	Mergeable,
 	NotMergeable,
-	Conflict,
 	Unknown,
 }
 

src/github/pullRequestModel.ts

@@ -34,7 +34,7 @@ import {
 	PendingReviewIdResponse,
 	PullRequestCommentsResponse,
 	PullRequestFilesResponse,
-	PullRequestMergabilityResponse,
+	PullRequestResponse,
 	ReactionGroup,
 	ResolveReviewThreadResponse,
 	StartReviewResponse,
@@ -959,7 +959,7 @@ export class PullRequestModel extends IssueModel<PullRequest> implements IPullRe
 	 * Get the status checks of the pull request, those for the last commit.
 	 */
 	async getStatusChecks(): Promise<PullRequestChecks> {
-		const { query, remote, schema, octokit } = await this.githubRepository.ensure();
+		const { query, remote, schema } = await this.githubRepository.ensure();
 		let result;
 		try {
 			result = await query<GetChecksResponse>({
@@ -991,7 +991,7 @@ export class PullRequestModel extends IssueModel<PullRequest> implements IPullRe
 			};
 		}
 
-		const checks: PullRequestChecks = {
+		return {
 			state: statusCheckRollup.state.toLowerCase(),
 			statuses: statusCheckRollup.contexts.nodes.map(context => {
 				if (isCheckRun(context)) {
@@ -1017,25 +1017,6 @@ export class PullRequestModel extends IssueModel<PullRequest> implements IPullRe
 				}
 			}),
 		};
-
-		// Fun info: The checks don't include whether a review is required.
-		// Also, unless you're an admin on the repo, you can't just do octokit.repos.getBranchProtection
-		if (this.item.mergeable === PullRequestMergeability.NotMergeable) {
-			const branch = await octokit.repos.getBranch({ branch: this.base.ref, owner: remote.owner, repo: remote.repositoryName });
-			if (branch.data.protected && branch.data.protection.required_status_checks.enforcement_level !== 'off') {
-				// We need to add the "review required" check manually.
-				checks.statuses.unshift({
-					id: 'unknown',
-					context: 'Branch Protection',
-					description: 'Requirements have not been met.',
-					state: 'failure',
-					target_url: this.html_url
-				});
-				checks.state = 'failure';
-			}
-		}
-
-		return checks;
 	}
 
 	static async openDiffFromComment(
@@ -1205,7 +1186,7 @@ export class PullRequestModel extends IssueModel<PullRequest> implements IPullRe
 			Logger.debug(`Fetch pull request mergeability ${this.number} - enter`, PullRequestModel.ID);
 			const { query, remote, schema } = await this.githubRepository.ensure();
 
-			const { data } = await query<PullRequestMergabilityResponse>({
+			const { data } = await query<PullRequestResponse>({
 				query: schema.PullRequestMergeability,
 				variables: {
 					owner: remote.owner,
@@ -1214,7 +1195,7 @@ export class PullRequestModel extends IssueModel<PullRequest> implements IPullRe
 				},
 			});
 			Logger.debug(`Fetch pull request mergeability ${this.number} - done`, PullRequestModel.ID);
-			return parseMergeability(data.repository.pullRequest.mergeable, data.repository.pullRequest.mergeStateStatus);
+			return parseMergeability(data.repository.pullRequest.mergeable);
 		} catch (e) {
 			Logger.appendLine(`PullRequestModel> Unable to fetch PR Mergeability: ${e}`);
 			return PullRequestMergeability.Unknown;

src/github/queries.gql

@@ -334,7 +334,6 @@ query PullRequest($owner: String!, $name: String!, $number: Int!) {
 			}
 			merged
 			mergeable
-			mergeStateStatus
 			id
 			databaseId
 			isDraft
@@ -503,7 +502,6 @@ query PullRequestMergeability($owner: String!, $name: String!, $number: Int!) {
 	repository(owner: $owner, name: $name) {
 		pullRequest(number: $number) {
 			mergeable
-			mergeStateStatus
 		}
 	}
 	rateLimit {

src/github/utils.ts

@@ -479,24 +479,15 @@ export function parseMilestone(
 	};
 }
 
-export function parseMergeability(mergeability: 'UNKNOWN' | 'MERGEABLE' | 'CONFLICTING',
-	mergeStateStatus: 'BEHIND' | 'BLOCKED' | 'CLEAN' | 'DIRTY' | 'HAS_HOOKS' | 'UNKNOWN' | 'UNSTABLE'): PullRequestMergeability {
-	let parsed: PullRequestMergeability;
+export function parseMergeability(mergeability: 'UNKNOWN' | 'MERGEABLE' | 'CONFLICTING'): PullRequestMergeability {
 	switch (mergeability) {
 		case 'UNKNOWN':
-			parsed = PullRequestMergeability.Unknown;
-			break;
+			return PullRequestMergeability.Unknown;
 		case 'MERGEABLE':
-			parsed = PullRequestMergeability.Mergeable;
-			break;
+			return PullRequestMergeability.Mergeable;
 		case 'CONFLICTING':
-			parsed = PullRequestMergeability.Conflict;
-			break;
+			return PullRequestMergeability.NotMergeable;
 	}
-	if ((parsed !== PullRequestMergeability.Conflict) && (mergeStateStatus === 'BLOCKED')) {
-		parsed = PullRequestMergeability.NotMergeable;
-	}
-	return parsed;
 }
 
 export function parseGraphQLPullRequest(
@@ -522,7 +513,7 @@ export function parseGraphQLPullRequest(
 		base: parseRef(graphQLPullRequest.baseRef?.name ?? graphQLPullRequest.baseRefName, graphQLPullRequest.baseRefOid, graphQLPullRequest.baseRepository),
 		user: parseAuthor(graphQLPullRequest.author, githubRepository),
 		merged: graphQLPullRequest.merged,
-		mergeable: parseMergeability(graphQLPullRequest.mergeable, graphQLPullRequest.mergeStateStatus),
+		mergeable: parseMergeability(graphQLPullRequest.mergeable),
 		labels: graphQLPullRequest.labels.nodes,
 		isDraft: graphQLPullRequest.isDraft,
 		suggestedReviewers: parseSuggestedReviewers(graphQLPullRequest.suggestedReviewers),
@@ -596,7 +587,7 @@ export function parseGraphQLIssuesRequest(
 		base: parseRef(graphQLPullRequest.baseRef?.name ?? graphQLPullRequest.baseRefName, graphQLPullRequest.baseRefOid, graphQLPullRequest.baseRepository),
 		user: parseAuthor(graphQLPullRequest.author, githubRepository),
 		merged: graphQLPullRequest.merged,
-		mergeable: parseMergeability(graphQLPullRequest.mergeable, pullRequest.mergeStateStatus),
+		mergeable: parseMergeability(graphQLPullRequest.mergeable),
 		labels: graphQLPullRequest.labels.nodes,
 		isDraft: graphQLPullRequest.isDraft,
 		suggestedReviewers: parseSuggestedReviewers(graphQLPullRequest.suggestedReviewers),

src/test/builders/graphql/pullRequestBuilder.ts

@@ -67,7 +67,6 @@ export const PullRequestBuilder = createBuilderClass<PullRequestResponse>()({
 			}),
 			merged: { default: false },
 			mergeable: { default: 'MERGEABLE' },
-			mergeStateStatus: { default: 'CLEAN' },
 			isDraft: { default: false },
 			suggestedReviewers: { default: [] },
 		}),

webviews/components/merge.tsx

@@ -145,17 +145,15 @@ export const MergeStatus = ({ mergeable, isSimple }: { mergeable: PullRequestMer
 				? null
 				: mergeable === PullRequestMergeability.Mergeable
 					? checkIcon
-					: (mergeable === PullRequestMergeability.NotMergeable || mergeable === PullRequestMergeability.Conflict)
+					: mergeable === PullRequestMergeability.NotMergeable
 						? deleteIcon
 						: pendingIcon}
 			<div>
-				{(mergeable) === PullRequestMergeability.Mergeable
+				{mergeable === PullRequestMergeability.Mergeable
 					? 'This branch has no conflicts with the base branch.'
-					: mergeable === PullRequestMergeability.Conflict
+					: mergeable === PullRequestMergeability.NotMergeable
 						? 'This branch has conflicts that must be resolved.'
-						: mergeable === PullRequestMergeability.NotMergeable
-							? 'Branch protection policy must be fulfilled before merging.'
-							: 'Checking if this branch can be merged...'}
+						: 'Checking if this branch can be merged...'}
 			</div>
 		</div>
 	);

webviews/editorWebview/index.css

@@ -235,16 +235,6 @@ body .comment-container .review-comment-header {
 	width: 16px;
 }
 
-#status-checks .avatar-link svg {
-	width: 24px;
-	margin-right: 0px;
-    vertical-align: middle;
-}
-
-.status-check .avatar-link .avatar-icon {
-	margin-right: 0px;
-}
-
 #status-checks .merge-select-container {
 	display: flex;
 	align-items: center;