Refactor CommandRegistry and add command registration utility for cha…

…t participants

src/common/chat-participants/ChatParticipantUtils.ts

@@ -4,7 +4,14 @@
  */
 
 import * as vscode from 'vscode';
+import { Command, CommandRegistry } from './CommandRegistry';
 
 export function createChatParticipant(participantId: string, handler: vscode.ChatRequestHandler): vscode.ChatParticipant {
     return vscode.chat.createChatParticipant(participantId, handler);
 }
+
+export function registerCommands(commandRegistry: CommandRegistry, commands: { [key: string]: Command }) {
+    for (const commandName in commands) {
+        commandRegistry.register(commandName, commands[commandName]);
+    }
+}

src/common/chat-participants/CommandRegistry.ts

@@ -4,15 +4,14 @@
  */
 
 import * as vscode from "vscode";
-import { CreateSiteCommand } from "./powerpages/commands/create-site/CreateSiteCommand";
 
 export interface Command {
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
     execute(request: any, stream: vscode.ChatResponseStream): Promise<any>;
 }
 
 export class CommandRegistry {
-    private commands: { [key: string]: Command } = {'create-site': new CreateSiteCommand()};
+    private commands: { [key: string]: Command } = {};
 
     register(commandName: string, command: Command) {
         this.commands[commandName] = command;

src/common/chat-participants/powerpages/PowerPagesChatParticipant.ts

@@ -4,15 +4,15 @@
  */
 
 import * as vscode from 'vscode';
-import { createChatParticipant } from '../ChatParticipantUtils';
+import { createChatParticipant, registerCommands } from '../ChatParticipantUtils';
 import { IComponentInfo, IPowerPagesChatResult } from './PowerPagesChatParticipantTypes';
 import { ITelemetry } from "../../OneDSLoggerTelemetry/telemetry/ITelemetry";
 import TelemetryReporter from '@vscode/extension-telemetry';
 import { sendApiRequest } from '../../copilot/IntelligenceApiService';
 import { PacWrapper } from '../../../client/pac/PacWrapper';
 import { intelligenceAPIAuthentication } from '../../services/AuthenticationProvider';
 import { ActiveOrgOutput } from '../../../client/pac/PacTypes';
-import { AUTHENTICATION_FAILED_MSG, COPILOT_NOT_AVAILABLE_MSG, COPILOT_NOT_RELEASED_MSG, DISCLAIMER_MESSAGE, INVALID_RESPONSE, NO_PROMPT_MESSAGE, PAC_AUTH_INPUT, PAC_AUTH_NOT_FOUND, POWERPAGES_CHAT_PARTICIPANT_ID, RESPONSE_AWAITED_MSG, RESPONSE_SCENARIOS, SKIP_CODES, STATER_PROMPTS, WELCOME_MESSAGE, WELCOME_PROMPT } from './PowerPagesChatParticipantConstants';
+import { AUTHENTICATION_FAILED_MSG, COPILOT_NOT_AVAILABLE_MSG, COPILOT_NOT_RELEASED_MSG, DISCLAIMER_MESSAGE, INVALID_RESPONSE, NO_PROMPT_MESSAGE, PAC_AUTH_INPUT, PAC_AUTH_NOT_FOUND, POWERPAGES_CHAT_PARTICIPANT_ID, POWERPAGES_COMMANDS, RESPONSE_AWAITED_MSG, RESPONSE_SCENARIOS, SKIP_CODES, STATER_PROMPTS, WELCOME_MESSAGE, WELCOME_PROMPT } from './PowerPagesChatParticipantConstants';
 import { ORG_DETAILS_KEY, handleOrgChangeSuccess, initializeOrgDetails } from '../../utilities/OrgHandlerUtils';
 import { createAndReferenceLocation, getComponentInfo, getEndpoint, provideChatParticipantFollowups, handleChatParticipantFeedback, createErrorResult, createSuccessResult, removeChatVariables } from './PowerPagesChatParticipantUtils';
 import { checkCopilotAvailability, fetchRelatedFiles, getActiveEditorContent } from '../../utilities/Utils';
@@ -25,10 +25,6 @@ import { oneDSLoggerWrapper } from '../../OneDSLoggerTelemetry/oneDSLoggerWrappe
 import { CommandRegistry } from '../CommandRegistry';
 import { VSCODE_EXTENSION_GITHUB_POWER_PAGES_AGENT_INVOKED, VSCODE_EXTENSION_GITHUB_POWER_PAGES_AGENT_ORG_DETAILS_NOT_FOUND, VSCODE_EXTENSION_GITHUB_POWER_PAGES_AGENT_ORG_DETAILS, VSCODE_EXTENSION_GITHUB_POWER_PAGES_AGENT_NOT_AVAILABLE_ECS, VSCODE_EXTENSION_GITHUB_POWER_PAGES_AGENT_SUCCESSFUL_PROMPT, VSCODE_EXTENSION_GITHUB_POWER_PAGES_AGENT_WELCOME_PROMPT, VSCODE_EXTENSION_GITHUB_POWER_PAGES_AGENT_NO_PROMPT, VSCODE_EXTENSION_GITHUB_POWER_PAGES_AGENT_LOCATION_REFERENCED, VSCODE_EXTENSION_GITHUB_POWER_PAGES_AGENT_WEBPAGE_RELATED_FILES, VSCODE_EXTENSION_GITHUB_POWER_PAGES_AGENT_SCENARIO, VSCODE_EXTENSION_GITHUB_POWER_PAGES_AGENT_ERROR, VSCODE_EXTENSION_GITHUB_POWER_PAGES_AGENT_COMMAND_TRIGGERED } from './PowerPagesChatParticipantTelemetryConstants';
 
-// Initialize Command Registry and Register Commands
-
-//Register Commands
-
 export class PowerPagesChatParticipant {
     private static instance: PowerPagesChatParticipant | null = null;
     private chatParticipant: vscode.ChatParticipant;
@@ -100,6 +96,8 @@ export class PowerPagesChatParticipant {
 
             const commandRegistry = new CommandRegistry();
 
+            registerCommands(commandRegistry, POWERPAGES_COMMANDS);
+
             if (!this.isOrgDetailsInitialized) {
                 stream.progress(PAC_AUTH_INPUT);
                 await this.initializeOrgDetails();

src/common/chat-participants/powerpages/PowerPagesChatParticipantConstants.ts

@@ -5,6 +5,7 @@
 
 import * as vscode from 'vscode';
 import { ADX_ENTITYFORM, ADX_ENTITYLIST } from '../../copilot/constants';
+import { CreateSiteCommand } from './commands/create-site/CreateSiteCommand';
 
 // Constants
 export const POWERPAGES_CHAT_PARTICIPANT_ID = 'powerpages';
@@ -58,3 +59,6 @@ export const NL2PAGE_GENERATING_WEBPAGES = vscode.l10n.t("Generating webpages...
 export const NL2PAGE_RESPONSE_FAILED = 'Failed to get page content from NL2Page service';
 export const NL2SITE_GENERATING_SITE = vscode.l10n.t("Generating a new Power Pages site...");
 export const FAILED_TO_CREATE_SITE = vscode.l10n.t('Failed to create a new Power Pages site. Please try again.');
+export const POWERPAGES_COMMANDS = {
+    'create-site': new CreateSiteCommand()
+}