Speed up opening session log tab (#7165)

Author: mjbvz

src/common/timelineEvent.ts

@@ -154,10 +154,11 @@ export interface ReopenedEvent {
 }
 
 export interface SessionPullInfo {
+	id: number;
 	host: string;
 	owner: string;
 	repo: string;
-	pullId: number;
+	pullNumber: number;
 }
 
 export interface SessionLinkInfo extends SessionPullInfo {

src/github/copilotRemoteAgent.ts

@@ -14,7 +14,7 @@ import { GitHubRemote } from '../common/remote';
 import { CODING_AGENT, CODING_AGENT_AUTO_COMMIT_AND_PUSH, CODING_AGENT_ENABLED } from '../common/settingKeys';
 import { toOpenPullRequestWebviewUri } from '../common/uri';
 import { OctokitCommon } from './common';
-import { CopilotApi, RemoteAgentJobPayload } from './copilotApi';
+import { CopilotApi, RemoteAgentJobPayload, SessionInfo } from './copilotApi';
 import { CopilotPRWatcher, CopilotStateModel } from './copilotPrWatcher';
 import { CredentialStore } from './credentials';
 import { FolderRepositoryManager } from './folderRepositoryManager';
@@ -27,8 +27,8 @@ type RemoteAgentErrorResult = { error: string; state: 'error' };
 type RemoteAgentResult = RemoteAgentSuccessResult | RemoteAgentErrorResult;
 
 export interface IAPISessionLogs {
-	sessionId: string;
-	logs: string;
+	readonly info: SessionInfo;
+	readonly logs: string;
 }
 
 export interface ICopilotRemoteAgentCommandArgs {
@@ -443,7 +443,7 @@ export class CopilotRemoteAgentManager extends Disposable {
 		}
 
 		const logs = await capi.getLogsFromSession(session.id);
-		return { sessionId: session.id, logs };
+		return { info: session, logs };
 	}
 
 	async getSessionUrlFromPullRequest(pullRequest: PullRequestModel): Promise<string | undefined> {
@@ -459,16 +459,6 @@ export class CopilotRemoteAgentManager extends Disposable {
 		return sessions.html_url;
 	}
 
-	async getSessionLogsFromSessionId(sessionId: string): Promise<IAPISessionLogs> {
-		const capi = await this.copilotApi;
-		if (!capi) {
-			return { sessionId: '', logs: '' };
-		}
-
-		const logs = await capi.getLogsFromSession(sessionId);
-		return { sessionId, logs };
-	}
-
 	private getLatestRun<T extends { last_updated_at?: string; updated_at?: string }>(runs: T[]): T {
 		return runs
 			.slice()

src/github/utils.ts

@@ -4,8 +4,8 @@
  *--------------------------------------------------------------------------------------------*/
 'use strict';
 
-import * as crypto from 'crypto';
 import * as OctokitTypes from '@octokit/types';
+import * as crypto from 'crypto';
 import * as vscode from 'vscode';
 import { Repository } from '../api/api';
 import { GitApiImpl } from '../api/api1';
@@ -1003,11 +1003,12 @@ export function parseSelectRestTimelineEvents(
 ): Common.TimelineEvent[] {
 	const parsedEvents: Common.TimelineEvent[] = [];
 
-	const prSessionLink = {
+	const prSessionLink: Common.SessionPullInfo = {
+		id: issueModel.id,
 		host: issueModel.githubRepository.remote.gitProtocol.host,
 		owner: issueModel.githubRepository.remote.owner,
 		repo: issueModel.githubRepository.remote.repositoryName,
-		pullId: issueModel.number,
+		pullNumber: issueModel.number,
 	};
 
 	let sessionIndex = 0;