Adds Explain WIP to inspect view

Author: d13

src/webviews/apps/commitDetails/components/gl-commit-details.ts

@@ -3,8 +3,10 @@ import { customElement, property, state } from 'lit/decorators.js';
 import { unsafeHTML } from 'lit/directives/unsafe-html.js';
 import { when } from 'lit/directives/when.js';
 import type { Autolink } from '../../../../autolinks/models/autolinks';
+import type { ExplainWipCommandArgs } from '../../../../commands/explainWip';
 import type { IssueOrPullRequest } from '../../../../git/models/issueOrPullRequest';
 import type { PullRequestShape } from '../../../../git/models/pullRequest';
+import { createCommandLink } from '../../../../system/commands';
 import type { Serialized } from '../../../../system/serialize';
 import type { State } from '../../../commitDetails/protocol';
 import { messageHeadlineSplitterToken } from '../../../commitDetails/protocol';
@@ -120,6 +122,37 @@ export class GlCommitDetails extends GlDetailsBase {
 		`;
 	}
 
+	private renderExplainChanges() {
+		if (this.state?.orgSettings.ai === false || this.state?.preferences.aiEnabled === false) return undefined;
+
+		if (this.isUncommitted) {
+			return html`
+				<gl-action-chip
+					label="Explain Working Changes"
+					icon="sparkle"
+					data-action="explain-commit"
+					aria-busy="${this.explainBusy ? 'true' : nothing}"
+					?disabled="${this.explainBusy ? true : nothing}"
+					@click=${this.onExplainChanges}
+					@keydown=${this.onExplainChanges}
+					><span>explain</span></gl-action-chip
+				>
+			`;
+		}
+		return html`
+			<gl-action-chip
+				label="Explain this ${this.isStash ? 'Stash' : 'Commit'}"
+				icon="sparkle"
+				data-action="explain-commit"
+				aria-busy="${this.explainBusy ? 'true' : nothing}"
+				?disabled="${this.explainBusy ? true : nothing}"
+				@click=${this.onExplainChanges}
+				@keydown=${this.onExplainChanges}
+				><span>explain</span></gl-action-chip
+			>
+		`;
+	}
+
 	private renderCommitMessage() {
 		const details = this.state?.commit;
 		if (details == null) return undefined;
@@ -140,21 +173,7 @@ export class GlCommitDetails extends GlDetailsBase {
 							></gl-commit-author>
 						`,
 					)}
-					${when(
-						this.state?.orgSettings.ai !== false && this.state?.preferences.aiEnabled !== false,
-						() => html`
-							<gl-action-chip
-								label="Explain this ${this.isStash ? 'Stash' : 'Commit'}"
-								icon="sparkle"
-								data-action="explain-commit"
-								aria-busy="${this.explainBusy ? 'true' : nothing}"
-								?disabled="${this.explainBusy ? true : nothing}"
-								@click=${this.onExplainChanges}
-								@keydown=${this.onExplainChanges}
-								><span>explain</span></gl-action-chip
-							>
-						`,
-					)}
+					${this.renderExplainChanges()}
 				</div>
 				<div>
 					<div class="message-block">
@@ -284,7 +303,7 @@ export class GlCommitDetails extends GlDetailsBase {
 
 	private renderAutoLinksChips() {
 		const autolinkState = this.autolinkState;
-		if (autolinkState == null) return html`<span></span>`;
+		if (autolinkState == null) return this.renderLearnAboutAutolinks();
 
 		const { autolinks, issues, prs, size } = autolinkState;
 

src/webviews/commitDetails/commitDetailsWebview.ts

@@ -8,6 +8,7 @@ import type { CopyMessageToClipboardCommandArgs } from '../../commands/copyMessa
 import type { CopyShaToClipboardCommandArgs } from '../../commands/copyShaToClipboard';
 import type { ExplainCommitCommandArgs } from '../../commands/explainCommit';
 import type { ExplainStashCommandArgs } from '../../commands/explainStash';
+import type { ExplainWipCommandArgs } from '../../commands/explainWip';
 import type { OpenPullRequestOnRemoteCommandArgs } from '../../commands/openPullRequestOnRemote';
 import type { ContextKeys } from '../../constants.context';
 import type { InspectTelemetryContext, Sources } from '../../constants.telemetry';
@@ -1121,15 +1122,26 @@ export class CommitDetailsWebviewProvider
 	private async explainRequest<T extends typeof ExplainRequest>(requestType: T, msg: IpcCallMessageType<T>) {
 		let params: DidExplainParams;
 		try {
-			const isStashCommit = isStash(this._context.commit);
-			await executeCommand<ExplainCommitCommandArgs | ExplainStashCommandArgs>(
-				isStashCommit ? 'gitlens.ai.explainStash' : 'gitlens.ai.explainCommit',
-				{
-					repoPath: this._context.commit!.repoPath,
-					rev: this._context.commit!.sha,
-					source: { source: 'inspect', type: isStashCommit ? 'stash' : 'commit' },
-				},
-			);
+			// check for uncommitted changes
+			if (
+				this._context.commit != null &&
+				(this._context.commit.isUncommitted || this._context.commit.isUncommittedStaged)
+			) {
+				await executeCommand<ExplainWipCommandArgs>('gitlens.ai.explainWip', {
+					repoPath: this._context.commit.repoPath,
+					source: { source: 'inspect', type: 'wip' },
+				});
+			} else {
+				const isStashCommit = isStash(this._context.commit);
+				await executeCommand<ExplainCommitCommandArgs | ExplainStashCommandArgs>(
+					isStashCommit ? 'gitlens.ai.explainStash' : 'gitlens.ai.explainCommit',
+					{
+						repoPath: this._context.commit!.repoPath,
+						rev: this._context.commit!.sha,
+						source: { source: 'inspect', type: isStashCommit ? 'stash' : 'commit' },
+					},
+				);
+			}
 
 			params = { result: { summary: '', body: '' } };
 		} catch (ex) {