Copilot PR feedback

Author: alexr00

src/github/githubRepository.ts

@@ -981,7 +981,7 @@ export class GitHubRepository extends Disposable {
 				job_id: checkRunDatabaseId,
 			});
 			Logger.debug(`Fetch check run logs via Actions API - done`, this.id);
-			return result.data as unknown as string;
+			return result.data as string;
 		} catch {
 			// Not a GitHub Actions job - fall through to Checks API
 		}

src/view/checkRunLogContentProvider.ts

@@ -6,6 +6,7 @@
 import * as vscode from 'vscode';
 import Logger from '../common/logger';
 import { Schemes } from '../common/uri';
+import { formatError } from '../common/utils';
 import { RepositoriesManager } from '../github/repositoriesManager';
 
 interface CheckRunLogParams {
@@ -51,8 +52,8 @@ export class CheckRunLogContentProvider implements vscode.TextDocumentContentPro
 				try {
 					return await repo.getCheckRunLogs(params.checkRunDatabaseId);
 				} catch (e) {
-					Logger.error(`Failed to fetch check run logs: ${e}`, 'CheckRunLog');
-					return `Failed to fetch check run logs: ${e}`;
+					Logger.error(`Failed to fetch check run logs: ${formatError(e)}`, 'CheckRunLog');
+					return `Failed to fetch check run logs. See logs for details.`;
 				}
 			}
 		}

webviews/common/context.tsx

@@ -363,7 +363,7 @@ export class PRContext {
 
 	public viewCheckLogs = (status: PullRequestCheckStatus) => this.postMessage({ command: 'pr.view-check-logs', args: { status } });
 
-		public openCommitChanges = async (commitSha: string) => {
+	public openCommitChanges = async (commitSha: string) => {
 		this.updatePR({ loadingCommit: commitSha });
 		try {
 			const args: OpenCommitChangesArgs = { commitSha };

webviews/editorWebview/index.css

@@ -239,7 +239,8 @@ body .comment-container .review-comment-header {
 	display: flex;
 	align-items: center;
 	gap: 4px;
-	}
+}
+
 .view-check-logs-placeholder {
 	width: 20px;
 }