Chat cleanup: remove participant and convert some tools to skills

Author: alexr00

package.json

@@ -418,6 +418,7 @@
 	"welcome.github.activePullRequest.contents": "Loading...",
 	"languageModelTools.github-pull-request_issue_fetch.displayName": "Get a GitHub Issue or pull request",
 	"languageModelTools.github-pull-request_issue_summarize.displayName": "Summarize a GitHub Issue or pull request",
+	"languageModelTools.github-pull-request_labels_fetch.displayName": "Fetch labels from a GitHub repository",
 	"languageModelTools.github-pull-request_notification_fetch.displayName": "Get a GitHub Notification",
 	"languageModelTools.github-pull-request_notification_summarize.displayName": "Summarize a GitHub Notification",
 	"languageModelTools.github-pull-request_suggest-fix.displayName": "Suggest a Fix for a GitHub Issue",

package.nls.json

@@ -26,6 +26,7 @@ export namespace contexts {
 export namespace commands {
 	export const OPEN_CHAT = 'workbench.action.chat.open';
 	export const NEW_CHAT = 'workbench.action.chat.newChat';
+	export const SHOW_CHAT = 'workbench.panel.chat';
 	export const CHAT_SETUP_ACTION_ID = 'workbench.action.chat.triggerSetup';
 
 	export const QUICK_CHAT_OPEN = 'workbench.action.quickchat.toggle';

src/common/executeCommands.ts

@@ -33,7 +33,6 @@ import { GitLensIntegration } from './integrations/gitlens/gitlensImpl';
 import { IssueFeatureRegistrar } from './issues/issueFeatureRegistrar';
 import { StateManager } from './issues/stateManager';
 import { IssueContextProvider } from './lm/issueContextProvider';
-import { ChatParticipant, ChatParticipantState } from './lm/participants';
 import { PullRequestContextProvider } from './lm/pullRequestContextProvider';
 import { registerTools } from './lm/tools/tools';
 import { migrate } from './migrations';
@@ -300,17 +299,11 @@ async function init(
 }
 
 function initChat(context: vscode.ExtensionContext, credentialStore: CredentialStore, reposManager: RepositoriesManager) {
-	const createParticipant = () => {
-		const chatParticipantState = new ChatParticipantState();
-		context.subscriptions.push(new ChatParticipant(context, chatParticipantState));
-		registerTools(context, credentialStore, reposManager, chatParticipantState);
-	};
-
 	const chatEnabled = () => vscode.workspace.getConfiguration(PR_SETTINGS_NAMESPACE).get<boolean>(EXPERIMENTAL_CHAT, false);
 	if (chatEnabled()) {
-		createParticipant();
+		registerTools(context, credentialStore, reposManager);
 	} else {
-		initBasedOnSettingChange(PR_SETTINGS_NAMESPACE, EXPERIMENTAL_CHAT, chatEnabled, createParticipant, context.subscriptions);
+		initBasedOnSettingChange(PR_SETTINGS_NAMESPACE, EXPERIMENTAL_CHAT, chatEnabled, () => registerTools(context, credentialStore, reposManager), context.subscriptions);
 	}
 }
 

src/extension.ts

@@ -565,11 +565,13 @@ export class IssueFeatureRegistrar extends Disposable {
 			*/
 				this.telemetry.sendTelemetryEvent('issue.chatSummarizeIssue');
 				if (issue instanceof IssueModel) {
-					commands.executeCommand(commands.NEW_CHAT, { inputValue: vscode.l10n.t('@githubpr Summarize issue {0}/{1}#{2}', issue.remote.owner, issue.remote.repositoryName, issue.number) });
+					commands.executeCommand(commands.NEW_CHAT, { inputValue: vscode.l10n.t('Summarize issue {0}/{1}#{2}', issue.remote.owner, issue.remote.repositoryName, issue.number) });
+					commands.executeCommand(commands.SHOW_CHAT);
 				} else {
 					const pullRequestModel = issue.pullRequestModel;
 					const remote = pullRequestModel.githubRepository.remote;
-					commands.executeCommand(commands.NEW_CHAT, { inputValue: vscode.l10n.t('@githubpr Summarize pull request {0}/{1}#{2}', remote.owner, remote.repositoryName, pullRequestModel.number) });
+					commands.executeCommand(commands.NEW_CHAT, { inputValue: vscode.l10n.t('Summarize pull request {0}/{1}#{2}', remote.owner, remote.repositoryName, pullRequestModel.number) });
+					commands.executeCommand(commands.SHOW_CHAT);
 				}
 			}),
 		);
@@ -582,7 +584,8 @@ export class IssueFeatureRegistrar extends Disposable {
 				"issue.chatSuggestFix" : {}
 			*/
 				this.telemetry.sendTelemetryEvent('issue.chatSuggestFix');
-				commands.executeCommand(commands.NEW_CHAT, { inputValue: vscode.l10n.t('@githubpr Find a fix for issue {0}/{1}#{2}', issue.remote.owner, issue.remote.repositoryName, issue.number) });
+				commands.executeCommand(commands.NEW_CHAT, { inputValue: vscode.l10n.t('Find a fix for issue {0}/{1}#{2}', issue.remote.owner, issue.remote.repositoryName, issue.number) });
+				commands.executeCommand(commands.SHOW_CHAT);
 			}),
 		);
 		this._register(vscode.commands.registerCommand('issues.configureIssuesViewlet', async () => {