Refactor isOptional to be part of ComponentInfo interface

Co-authored-by: JoeRobich <611219+JoeRobich@users.noreply.github.com>

Author: Copilot

src/lsptoolshost/extensions/builtInComponents.ts

@@ -11,6 +11,7 @@ interface ComponentInfo {
     defaultFolderName: string;
     optionName: string;
     componentDllPaths: string[];
+    isOptional?: boolean;
 }
 
 export const componentInfo: { [key: string]: ComponentInfo } = {
@@ -26,11 +27,13 @@ export const componentInfo: { [key: string]: ComponentInfo } = {
             'Microsoft.VisualStudio.DesignTools.CodeAnalysis.dll',
             'Microsoft.VisualStudio.DesignTools.CodeAnalysis.Diagnostics.dll',
         ],
+        isOptional: true,
     },
     razorDevKit: {
         defaultFolderName: '.razorDevKit',
         optionName: 'razorDevKit',
         componentDllPaths: ['Microsoft.VisualStudio.DevKit.Razor.dll'],
+        isOptional: true,
     },
     razorExtension: {
         defaultFolderName: '.razorExtension',
@@ -41,20 +44,17 @@ export const componentInfo: { [key: string]: ComponentInfo } = {
         defaultFolderName: '.roslynCopilot',
         optionName: 'roslynCopilot',
         componentDllPaths: ['Microsoft.VisualStudio.Copilot.Roslyn.LanguageServer.dll'],
+        isOptional: true,
     },
 };
 
-export function getComponentPaths(
-    componentName: string,
-    options: LanguageServerOptions | undefined,
-    isOptional = false
-): string[] {
+export function getComponentPaths(componentName: string, options: LanguageServerOptions | undefined): string[] {
     const component = componentInfo[componentName];
     const baseFolder = getComponentFolderPath(component, options);
     const paths = component.componentDllPaths.map((dllPath) => path.join(baseFolder, dllPath));
     for (const dllPath of paths) {
         if (!fs.existsSync(dllPath)) {
-            if (isOptional) {
+            if (component.isOptional) {
                 // Component is optional and doesn't exist - return empty array
                 return [];
             }

src/lsptoolshost/server/roslynLanguageServer.ts

@@ -1026,7 +1026,7 @@ export class RoslynLanguageServer {
 
         // Also include the Xaml Dev Kit extensions, if enabled.
         if (languageServerOptions.enableXamlTools) {
-            getComponentPaths('xamlTools', languageServerOptions, true).forEach((path) =>
+            getComponentPaths('xamlTools', languageServerOptions).forEach((path) =>
                 additionalExtensionPaths.push(path)
             );
         }
@@ -1096,7 +1096,7 @@ export class RoslynLanguageServer {
             await exports.setupTelemetryEnvironmentAsync(env);
         }
 
-        getComponentPaths('roslynCopilot', languageServerOptions, true).forEach((extPath) => {
+        getComponentPaths('roslynCopilot', languageServerOptions).forEach((extPath) => {
             additionalExtensionPaths.push(extPath);
         });
     }

src/razor/src/extension.ts

@@ -104,7 +104,7 @@ export async function activate(
             await setupDevKitEnvironment(dotnetInfo.env, csharpDevkitExtension, logger);
 
             if (vscode.env.isTelemetryEnabled) {
-                const razorComponentPaths = getComponentPaths('razorDevKit', undefined, true);
+                const razorComponentPaths = getComponentPaths('razorDevKit', undefined);
                 if (razorComponentPaths.length !== 1) {
                     logger.logError('Failed to find Razor DevKit telemetry extension path.', undefined);
                 } else {