Polishing session view (#7075)

- Adds back button
- Adds open on web command
- Use codicons
- Fix overflow
- Add state restore

Author: mjbvz

package.json

@@ -51,7 +51,8 @@
 		"onFileSystem:newIssue",
 		"onFileSystem:pr",
 		"onFileSystem:githubpr",
-		"onFileSystem:review"
+		"onFileSystem:review",
+		"onWebviewPanel:pr.padawanSessionLogView"
 	],
 	"browser": "./dist/browser/extension",
 	"l10n": "./dist/browser/extension",
@@ -1654,6 +1655,11 @@
 				"command": "padawan.openSessionLog",
 				"title": "Open Padawan Session Log",
 				"category": "%command.pull.request.category%"
+			},
+			{
+				"command": "sessionLog.openOnWeb",
+				"title": "Open on Web",
+				"icon": "$(globe)"
 			}
 		],
 		"viewsWelcome": [
@@ -2367,6 +2373,10 @@
 				{
 					"command": "review.copyPrLink",
 					"when": "github:inReviewMode"
+				},
+				{
+					"command": "sessionLog.openOnWeb",
+					"when": "false"
 				}
 			],
 			"view/title": [
@@ -2833,6 +2843,11 @@
 					"command": "pr.addFileComment",
 					"group": "navigation",
 					"when": "(resourceScheme == pr) || (resourcePath in github:viewedFiles) || (resourcePath in github:unviewedFiles)"
+				},
+				{
+					"command": "sessionLog.openOnWeb",
+					"group": "navigation",
+					"when": "activeWebviewPanelId == 'pr.padawanSessionLogView'"
 				}
 			],
 			"editor/content": [
@@ -3918,6 +3933,7 @@
 		"prepare": "husky install"
 	},
 	"devDependencies": {
+		"@shikijs/monaco": "^3.7.0",
 		"@types/chai": "^4.1.4",
 		"@types/glob": "7.1.3",
 		"@types/lru-cache": "^5.1.0",
@@ -3953,18 +3969,21 @@
 		"jsdom": "19.0.0",
 		"jsdom-global": "3.0.2",
 		"json5": "2.2.2",
+		"markdown-it": "^14.1.0",
 		"merge-options": "3.0.4",
 		"minimist": "^1.2.6",
 		"mkdirp": "1.0.4",
 		"mocha": "^9.0.1",
 		"mocha-junit-reporter": "1.23.0",
 		"mocha-multi-reporters": "1.1.7",
+		"monaco-editor-webpack-plugin": "^7.1.0",
 		"os-browserify": "^0.3.0",
 		"p-all": "^1.0.0",
 		"path-browserify": "1.0.1",
 		"process": "^0.11.10",
 		"raw-loader": "4.0.2",
 		"react-testing-library": "7.0.1",
+		"shiki": "^3.7.0",
 		"sinon": "9.0.0",
 		"source-map-support": "0.5.19",
 		"stream-browserify": "^3.0.0",
@@ -3984,7 +4003,7 @@
 	"dependencies": {
 		"@octokit/rest": "20.1.2",
 		"@octokit/types": "13.8.0",
-		"@shikijs/monaco": "^3.7.0",
+		"@vscode/codicons": "^0.0.36",
 		"@vscode/extension-telemetry": "0.7.5",
 		"@vscode/prompt-tsx": "^0.3.0-alpha.12",
 		"apollo-boost": "^0.4.9",
@@ -3997,12 +4016,10 @@
 		"fast-deep-equal": "^3.1.3",
 		"jszip": "^3.10.1",
 		"lru-cache": "6.0.0",
-		"markdown-it": "^14.1.0",
 		"marked": "^4.0.10",
 		"monaco-editor": "^0.52.2",
 		"react": "^16.12.0",
 		"react-dom": "^16.12.0",
-		"shiki": "^3.7.0",
 		"ssh-config": "4.1.1",
 		"stream-http": "^3.2.0",
 		"tunnel": "0.0.6",
@@ -4012,4 +4029,4 @@
 		"vsls": "^0.3.967"
 	},
 	"license": "MIT"
-}
+}

src/common/timelineEvent.ts

@@ -151,7 +151,7 @@ export interface ReopenedEvent {
 	createdAt: string;
 }
 
-export interface SessionLinkInfo {
+export interface SessionPullInfo {
 	host: string;
 	owner: string;
 	repo: string;
@@ -163,7 +163,7 @@ export interface CopilotStartedEvent {
 	event: EventType.CopilotStarted;
 	createdAt: string;
 	onBehalfOf: IAccount;
-	sessionLink?: SessionLinkInfo;
+	sessionLink?: SessionPullInfo;
 }
 
 export interface CopilotFinishedEvent {
@@ -178,7 +178,7 @@ export interface CopilotFinishedErrorEvent {
 	event: EventType.CopilotFinishedError;
 	createdAt: string;
 	onBehalfOf: IAccount;
-	sessionLink: SessionLinkInfo;
+	sessionLink: SessionPullInfo;
 }
 
 export type TimelineEvent = CommitEvent | ReviewEvent | CommentEvent | NewCommitsSinceReviewEvent | MergedEvent | AssignEvent | UnassignEvent | HeadRefDeleteEvent | CrossReferencedEvent | ClosedEvent | ReopenedEvent | CopilotStartedEvent | CopilotFinishedEvent | CopilotFinishedErrorEvent;

src/extension.ts

@@ -249,7 +249,7 @@ async function init(
 
 	context.subscriptions.push(new GitLensIntegration());
 
-	const sessionLogViewManager = new SessionLogViewManager(credentialStore, context);
+	const sessionLogViewManager = new SessionLogViewManager(credentialStore, context, reposManager, telemetry);
 	context.subscriptions.push(sessionLogViewManager);
 
 	await vscode.commands.executeCommand('setContext', 'github:initialized', true);

src/github/pullRequestOverview.ts

@@ -12,7 +12,7 @@ import { disposeAll } from '../common/lifecycle';
 import Logger from '../common/logger';
 import { DEFAULT_MERGE_METHOD, PR_SETTINGS_NAMESPACE } from '../common/settingKeys';
 import { ITelemetry } from '../common/telemetry';
-import { ReviewEvent, SessionLinkInfo } from '../common/timelineEvent';
+import { ReviewEvent, SessionPullInfo } from '../common/timelineEvent';
 import { asPromise, formatError } from '../common/utils';
 import { IRequestMessage, PULL_REQUEST_OVERVIEW_VIEW_TYPE } from '../common/webview';
 import { SessionLogViewManager } from '../view/sessionLogView';
@@ -454,7 +454,7 @@ export class PullRequestOverviewPanel extends IssueOverviewPanel<PullRequestMode
 		}
 	}
 
-	private async openSessionLog(_message: IRequestMessage<{ link: SessionLinkInfo }>): Promise<void> {
+	private async openSessionLog(_message: IRequestMessage<{ link: SessionPullInfo }>): Promise<void> {
 		try {
 			SessionLogViewManager.instance?.openForPull(this._item);
 		} catch (e) {

src/github/utils.ts

@@ -1002,7 +1002,7 @@ export function parseSelectRestTimelineEvents(
 	events: OctokitCommon.ListEventsForTimelineResponse[]
 ): Common.TimelineEvent[] {
 	const parsedEvents: Common.TimelineEvent[] = [];
-	const sessionLink: Common.SessionLinkInfo = {
+	const sessionLink: Common.SessionPullInfo = {
 		host: issueModel.githubRepository.remote.gitProtocol.host,
 		owner: issueModel.githubRepository.remote.owner,
 		repo: issueModel.githubRepository.remote.repositoryName,