Add .razorExtension (#8096)

Depends on the following PRs:

dotnet/razor#11510
dotnet/roslyn#77715

Author: Andrew Hall

.gitignore

@@ -13,6 +13,7 @@ out
 .razoromnisharp/
 .razortelemetry/
 .razorDevKit/
+.razorExtension/
 .vscode-test/
 msbuild/signing/signJs/*.log
 msbuild/signing/signVsix/*.log

.vscodeignore

@@ -7,6 +7,7 @@
 !.razor/**
 !.razorDevKit/**
 !.razoromnisharp/**
+!.razorExtension/**
 .rpt2_cache/**
 .config/**
 .devcontainer/**

package.json

@@ -37,9 +37,9 @@
     }
   },
   "defaults": {
-    "roslyn": "4.14.0-3.25164.3",
+    "roslyn": "4.14.0-3.25175.12",
     "omniSharp": "1.39.12",
-    "razor": "9.0.0-preview.25161.2",
+    "razor": "9.0.0-preview.25177.4",
     "razorOmnisharp": "7.0.0-preview.23363.1",
     "xamlTools": "17.14.35927.264"
   },

src/lsptoolshost/extensions/builtInComponents.ts

@@ -32,6 +32,11 @@ export const componentInfo: { [key: string]: ComponentInfo } = {
         optionName: 'razorDevKit',
         componentDllPaths: ['Microsoft.VisualStudio.DevKit.Razor.dll'],
     },
+    razorExtension: {
+        defaultFolderName: '.razorExtension',
+        optionName: 'razorExtension',
+        componentDllPaths: ['Microsoft.VisualStudioCode.RazorExtension.dll'],
+    },
 };
 
 export function getComponentPaths(componentName: string, options: LanguageServerOptions | undefined): string[] {

src/lsptoolshost/server/roslynLanguageServer.ts

@@ -619,6 +619,10 @@ export class RoslynLanguageServer {
             path.join(razorPath, 'Targets', 'Microsoft.NET.Sdk.Razor.DesignTime.targets')
         );
 
+        getComponentPaths('razorExtension', languageServerOptions).forEach((path) =>
+            additionalExtensionPaths.push(path)
+        );
+
         // Get the brokered service pipe name from C# Dev Kit (if installed).
         // We explicitly call this in the LSP server start action instead of awaiting it
         // in our activation because C# Dev Kit depends on C# activation completing.
@@ -966,6 +970,7 @@ export class RoslynLanguageServer {
                 additionalExtensionPaths.push(path)
             );
         }
+
         return args;
     }
 

src/razor/src/dynamicFile/dynamicFileInfoHandler.ts

@@ -8,13 +8,14 @@ import { UriConverter } from '../../../lsptoolshost/utils/uriConverter';
 import { RazorDocumentManager } from '../document/razorDocumentManager';
 import { RazorLogger } from '../razorLogger';
 import { ProvideDynamicFileParams } from './provideDynamicFileParams';
-import { ProvideDynamicFileResponse, DynamicFileUpdate } from './provideDynamicFileResponse';
+import { ProvideDynamicFileResponse } from './provideDynamicFileResponse';
 import { RemoveDynamicFileParams } from './removeDynamicFileParams';
 import { CSharpProjectedDocument } from '../csharp/csharpProjectedDocument';
 import { RazorDocumentChangeKind } from '../document/razorDocumentChangeKind';
 import { RazorDynamicFileChangedParams } from './dynamicFileUpdatedParams';
 import { TextDocumentIdentifier } from 'vscode-languageserver-protocol';
-import { ServerTextChange } from '../rpc/serverTextChange';
+import { razorTextChange } from './razorTextChange';
+import { razorTextSpan } from './razorTextSpan';
 
 // Handles Razor generated doc communication between the Roslyn workspace and Razor.
 // didChange behavior for Razor generated docs is handled in the RazorDocumentManager.
@@ -75,19 +76,11 @@ export class DynamicFileInfoHandler {
             if (request.fullText) {
                 // The server asked for a full replace so the newtext is the important
                 // thing here, the span doesn't matter.
-                const change: ServerTextChange = {
-                    newText: razorDocument.csharpDocument.getContent(),
-                    span: {
-                        start: 0,
-                        length: 0,
-                    },
-                };
-
-                const update = new DynamicFileUpdate([change]);
+                const change = new razorTextChange(razorDocument.csharpDocument.getContent(), new razorTextSpan(0, 0));
 
                 return new ProvideDynamicFileResponse(
                     request.razorDocument,
-                    [update],
+                    [change],
                     csharpDocument.checksum,
                     csharpDocument.checksumAlgorithm,
                     csharpDocument.encodingCodePage
@@ -99,22 +92,18 @@ export class DynamicFileInfoHandler {
             if (this.documentManager.isRazorDocumentOpenInCSharpWorkspace(vscodeUri)) {
                 // Open documents have didOpen/didChange to update the csharp buffer. Razor
                 // does not send edits and instead lets vscode handle them.
-                return new ProvideDynamicFileResponse(
-                    { uri: virtualCsharpUri },
-                    null,
-                    csharpDocument.checksum,
-                    csharpDocument.checksumAlgorithm,
-                    csharpDocument.encodingCodePage
-                );
+                return null;
             } else {
                 // Closed documents provide edits since the last time they were requested since
                 // there is no open buffer in vscode corresponding to the csharp content.
                 const response = csharpDocument.applyEdits();
-                const updates = response.edits?.map((e) => new DynamicFileUpdate(e.changes)) ?? null;
+                const updates = response.edits?.flatMap((e) =>
+                    e.changes.map((c) => new razorTextChange(c.newText, new razorTextSpan(c.span.start, c.span.length)))
+                );
 
                 return new ProvideDynamicFileResponse(
                     { uri: virtualCsharpUri },
-                    updates,
+                    updates ?? [],
                     response.originalChecksum,
                     response.originalChecksumAlgorithm,
                     response.originalEncodingCodePage

src/razor/src/dynamicFile/provideDynamicFileResponse.ts

@@ -5,12 +5,13 @@
 
 import { TextDocumentIdentifier } from 'vscode-languageserver-protocol';
 import { ServerTextChange } from '../rpc/serverTextChange';
+import { razorTextChange } from './razorTextChange';
 
 // matches https://github.com/dotnet/roslyn/blob/9e91ca6590450e66e0041ee3135bbf044ac0687a/src/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/HostWorkspace/RazorDynamicFileInfoProvider.cs#L28
 export class ProvideDynamicFileResponse {
     constructor(
         public readonly csharpDocument: TextDocumentIdentifier | null,
-        public readonly updates: DynamicFileUpdate[] | null,
+        public readonly edits: razorTextChange[],
         public readonly checksum: string,
         public readonly checksumAlgorithm: number,
         public readonly encodingCodePage: number | null

src/razor/src/dynamicFile/razorTextChange.ts

@@ -0,0 +1,10 @@
+/*---------------------------------------------------------------------------------------------
+ *  Copyright (c) Microsoft Corporation. All rights reserved.
+ *  Licensed under the MIT License. See License.txt in the project root for license information.
+ *--------------------------------------------------------------------------------------------*/
+
+import { razorTextSpan } from './razorTextSpan';
+
+export class razorTextChange {
+    constructor(public readonly newText: string | null, public readonly span: razorTextSpan) {}
+}

src/razor/src/dynamicFile/razorTextSpan.ts

@@ -0,0 +1,8 @@
+/*---------------------------------------------------------------------------------------------
+ *  Copyright (c) Microsoft Corporation. All rights reserved.
+ *  Licensed under the MIT License. See License.txt in the project root for license information.
+ *--------------------------------------------------------------------------------------------*/
+
+export class razorTextSpan {
+    constructor(public readonly start: number, public readonly length: number) {}
+}

tasks/offlinePackagingTasks.ts

@@ -27,6 +27,7 @@ import {
     xamlToolsDirectory,
     razorLanguageServerDirectory,
     razorDevKitDirectory,
+    razorExtensionDirectory,
 } from '../tasks/projectPaths';
 import { getPackageJSON } from '../tasks/packageJson';
 import { createPackageAsync, generateVsixManifest } from '../tasks/vsceTasks';
@@ -98,6 +99,12 @@ export const allNugetPackages: { [key: string]: NugetPackageInfo } = {
         getPackageContentPath: (_platformInfo) => 'content',
         vsixOutputPath: razorDevKitDirectory,
     },
+    razorExtension: {
+        getPackageName: (_platformInfo) => 'Microsoft.VisualStudioCode.RazorExtension',
+        packageJsonName: 'razor',
+        getPackageContentPath: (_platformInfo) => 'content',
+        vsixOutputPath: razorExtensionDirectory,
+    },
 };
 
 const vsixTasks: string[] = [];
@@ -185,6 +192,9 @@ gulp.task(
 
         // Also pull in the Razor DevKit dependencies nuget package.
         await acquireNugetPackage(allNugetPackages.razorDevKit, undefined, getPackageJSON(), true);
+
+        // Pull in the .razorExtension code that gets loaded in the roslyn language server
+        await acquireNugetPackage(allNugetPackages.razorExtension, undefined, getPackageJSON(), true);
     }, 'installDependencies')
 );