upgrade to 'v2.87.31'

Author: muhammadsammy

.gitignore

@@ -15,6 +15,7 @@ out
 .razorDevKit/
 .razorExtension/
 .vscode-test/
+.roslynCopilot/
 msbuild/signing/signJs/*.log
 msbuild/signing/signVsix/*.log
 dist/

.vscode/launch.json

@@ -92,6 +92,33 @@
             "preLaunchTask": "packageDev",
             "internalConsoleOptions": "openOnSessionStart"
         },
+        {
+            "name": "[Razor Cohost] Run Current File Integration Test",
+            "type": "extensionHost",
+            "request": "launch",
+            "runtimeExecutable": "${execPath}",
+            "args": [
+                // Launch VSCode using a specific profile to ensure that user settings are not used.
+                // This profile must be imported into vscode before running this launch configuration.
+                // The profile can be found under /test/csharp-test-profile.
+                "--profile",
+                "csharp-test-profile",
+                "${workspaceRoot}/test/razor/razorIntegrationTests/testAssets/RazorApp/.vscode/RazorApp.code-workspace",
+                "--extensionDevelopmentPath=${workspaceRoot}",
+                "--extensionTestsPath=${workspaceRoot}/out/test/razor/razorIntegrationTests",
+                "--log",
+                "ms-dotnettools.csharp:trace"
+            ],
+            "env": {
+                "CODE_EXTENSIONS_PATH": "${workspaceRoot}",
+                "TEST_FILE_FILTER": "${file}"
+            },
+            "sourceMaps": true,
+            "outFiles": ["${workspaceRoot}/dist/*.js", "${workspaceRoot}/out/test/**/*.js"],
+            "resolveSourceMapLocations": ["${workspaceFolder}/**", "!**/node_modules/**"],
+            "preLaunchTask": "packageDev",
+            "internalConsoleOptions": "openOnSessionStart"
+        },
         {
             "name": "[O#] Run Current File Integration Tests",
             "type": "extensionHost",
@@ -197,6 +224,15 @@
             "program": "${workspaceFolder}/node_modules/gulp/bin/gulp.js",
             "args": ["test:integration:razor"],
             "cwd": "${workspaceFolder}"
+        },
+        {
+            "type": "node",
+            "request": "launch",
+            "name": "Razor Cohost integration tests",
+            "preLaunchTask": "build",
+            "program": "${workspaceFolder}/node_modules/gulp/bin/gulp.js",
+            "args": ["test:integration:razor:cohost"],
+            "cwd": "${workspaceFolder}"
         }
     ],
     "inputs": [

CHANGELOG.md

@@ -3,6 +3,87 @@
 - Diagnostics related feature requests and improvements [#5951](https://github.com/dotnet/vscode-csharp/issues/5951)
 - Debug from .csproj and .sln [#5876](https://github.com/dotnet/vscode-csharp/issues/5876)
 
+# 2.87.x
+* Fix acquisition of components with proxies (PR: [#8462](https://github.com/dotnet/vscode-csharp/pull/8462))
+* Bump Roslyn to 5.0.0-2.25404.10 (PR: [#8490](https://github.com/dotnet/vscode-csharp/pull/8490))
+  * Fix file based programs getting stuck in the host workspace (PR: [#79730](https://github.com/dotnet/roslyn/pull/79730))
+  * Fix exception in additional file diagnostics (PR: [#79501](https://github.com/dotnet/roslyn/pull/79501))
+  * Simplify stale project handling in hot reload(PR: [#79386](https://github.com/dotnet/roslyn/pull/79386))
+  * Fix static extension method not showing up on enum type(PR: [#79454](https://github.com/dotnet/roslyn/pull/79454))
+  * Update 'use auto property' to respect user options around `this.` for property access.(PR: [#79405](https://github.com/dotnet/roslyn/pull/79405))
+  * Fix make-readonly with spans.(PR: [#79393](https://github.com/dotnet/roslyn/pull/79393))
+  * Change how trailing trivia is preserved in use-auto-prop(PR: [#79425](https://github.com/dotnet/roslyn/pull/79425))
+  * Special case a few well-known immutable types in use-auto-prop(PR: [#79422](https://github.com/dotnet/roslyn/pull/79422))
+  * Preserve comments when switching to auto props(PR: [#79420](https://github.com/dotnet/roslyn/pull/79420))
+  * Support interpolated string handlers in extension blocks(PR: [#78425](https://github.com/dotnet/roslyn/pull/78425))
+  * Offer 'implement interface' when inside a type's body(PR: [#79382](https://github.com/dotnet/roslyn/pull/79382))
+  * EnC support for project level changes(PR: [#79239](https://github.com/dotnet/roslyn/pull/79239))
+  * Ensure generated types come after top level statements(PR: [#79378](https://github.com/dotnet/roslyn/pull/79378))
+  * Allow user to still create a new field/prop when offering to initialize an existing prop(PR: [#79376](https://github.com/dotnet/roslyn/pull/79376))
+  * Fix crash in replace property with methods(PR: [#79358](https://github.com/dotnet/roslyn/pull/79358))
+  * Fix issue offering to remove nullable cast in a ternary expression(PR: [#79355](https://github.com/dotnet/roslyn/pull/79355))
+  * Fix 'use var' with spans(PR: [#79348](https://github.com/dotnet/roslyn/pull/79348))
+  * Fix not offering to remove unnecessary nullable pragmas(PR: [#79356](https://github.com/dotnet/roslyn/pull/79356))
+  * Fix crash in use-null-prop fixer(PR: [#79340](https://github.com/dotnet/roslyn/pull/79340))
+* Update debugger to v2.86.0(PR: [#8424](https://github.com/dotnet/vscode-csharp/pull/8424))
+* Bump Razor to 10.0.0-preview.25368.1 (PR: [#8430](https://github.com/dotnet/vscode-csharp/pull/8430))
+  * Fixing override completion in VSCode when LSP is enabled (PR: [#12039](https://github.com/dotnet/razor/pull/12039))
+  * [Cohost] Fix rename and unskip test (PR: [#11952](https://github.com/dotnet/razor/pull/11952))
+  * Ensure unique file paths for non-file Uris (PR: [#12037](https://github.com/dotnet/razor/pull/12037))
+  * Fix completion of hyphenated items (PR: [#12035](https://github.com/dotnet/razor/pull/12035))
+  * Allow completion items to add using directives (PR: [#12034](https://github.com/dotnet/razor/pull/12034))
+
+# 2.86.x
+* Bump Roslyn to 5.0.0-1.25361.2 (PR: [#8416](https://github.com/dotnet/vscode-csharp/pull/8416))
+  * Fix issue with diagnostics when renaming a file (case change only) (PR: [#79298](https://github.com/dotnet/roslyn/pull/79298))
+  * Allow rename to (optionally) process source generated documents(PR: [#78984](https://github.com/dotnet/roslyn/pull/78984))
+  * Initial support for adding obsolete attributes to primary constructors(PR: [#79271](https://github.com/dotnet/roslyn/pull/79271))
+  * Allow the Razor extension to report telemetry (and initialize)(PR: [#79254](https://github.com/dotnet/roslyn/pull/79254))
+  * Use same location for 'use implicit object' as we do for 'use collection expression'.(PR: [#79265](https://github.com/dotnet/roslyn/pull/79265))
+  * Use the miscellaneous files project name for rich misc projects(PR: [#79267](https://github.com/dotnet/roslyn/pull/79267))
+  * Razor: Don't double register for document sync(PR: [#79241](https://github.com/dotnet/roslyn/pull/79241))
+  * Allow LanguageServerProjectSystems to load into more than one workspace(PR: [#78975](https://github.com/dotnet/roslyn/pull/78975))
+  * Improve diagnostic for ambiguous predefined type(PR: [#79196](https://github.com/dotnet/roslyn/pull/79196))
+  * Fix renaming constructor via alias(PR: [#79175](https://github.com/dotnet/roslyn/pull/79175))
+  * Speed up performance of remove-unused-imports fix all(PR: [#79173](https://github.com/dotnet/roslyn/pull/79173))
+  * Improve performance in metadata-to-project-reference conversion(PR: [#79163](https://github.com/dotnet/roslyn/pull/79163))
+  * Fix issue where we offered to use a collection expression when teh user didn't have a viable Add method(PR: [#79157](https://github.com/dotnet/roslyn/pull/79157))
+  * Don't load the razor redirector unless it could actually be a razor assembly(PR: [#79154](https://github.com/dotnet/roslyn/pull/79154))
+  * Tweak clarity rules for parentheses where ?? is involved(PR: [#79159](https://github.com/dotnet/roslyn/pull/79159))
+  * Avoid hitting object pool when not needed(PR: [#79158](https://github.com/dotnet/roslyn/pull/79158))
+  * Merge 'extension operators' into 'main'(PR: [#79151](https://github.com/dotnet/roslyn/pull/79151))
+  * Extensions: interceptors(PR: [#79010](https://github.com/dotnet/roslyn/pull/79010))
+  * Small but trivial allocation improvement in FileWatchedReferenceFactory.StopWatchingReference(PR: [#79140](https://github.com/dotnet/roslyn/pull/79140))
+  * Slight allocation reduction in CommandLineParser.FlattenArgs (PR: [#79139](https://github.com/dotnet/roslyn/pull/79139))
+* Fix promptForDevKitDebugConfigurations without CDK installed (PR: [#8390](https://github.com/dotnet/vscode-csharp/pull/8390))
+* Remove Razor ForceRuntimeCodeGeneration flag (PR: [#8398](https://github.com/dotnet/vscode-csharp/pull/8398))
+* Bump Razor to 10.0.0-preview.25329.6 (PR: [#8413](https://github.com/dotnet/vscode-csharp/pull/8413))
+  * Use snippet InsertText in directive attributes to insert equals and quotes (PR: [#12010](https://github.com/dotnet/razor/pull/12010))
+  * Fix edit mapping for Html documents (PR: [#12027](https://github.com/dotnet/razor/pull/12027))
+  * Don't show HTML snippets in style, script, or html comment blocks (PR: [#12021](https://github.com/dotnet/razor/pull/12021))
+  * Allow our VS Code extension to report telemetry (through Roslyn) (PR: [#12015](https://github.com/dotnet/razor/pull/12015))
+  * Hard code ForceRuntimeCodeGeneration feature flag in all cases (PR: [#12007](https://github.com/dotnet/razor/pull/12007))
+  * Report cohost status in initialize telemetry (PR: [#12001](https://github.com/dotnet/razor/pull/12001))
+  * Implement "Extract to Foo.razor.css" code action (PR: [#11989](https://github.com/dotnet/razor/pull/11989))
+  * Filter all html diagnostics out if they're reported in a comment (PR: [#12002](https://github.com/dotnet/razor/pull/12002))
+  * Analyzer redirector (PR: [#11972](https://github.com/dotnet/razor/pull/11972))
+  * Partial fix for https://github.com/dotnet/vscode-csharp/issues/7678 (PR: [#11681](https://github.com/dotnet/razor/pull/11681))
+  * Simplify tag to self-closing code action (PR: [#11802](https://github.com/dotnet/razor/pull/11802))
+  * Bind directive attribute `event` parameter HTML event completions (PR: [#11804](https://github.com/dotnet/razor/pull/11804))
+  * Allow Razor completion items on deletion (PR: [#11991](https://github.com/dotnet/razor/pull/11991))
+
+
+# 2.85.x
+* Bump Roslyn to 5.0.0-1.25326.2 (PR: [#8386](https://github.com/dotnet/vscode-csharp/pull/8386))
+  * Fix nullable oblivious quick info (PR: [#79102](https://github.com/dotnet/roslyn/pull/79102))
+  * Clear provider on disposal (PR: [#79128](https://github.com/dotnet/roslyn/pull/79128))
+  * Improve performance of fix all that only changes syntax (PR: [#79108](https://github.com/dotnet/roslyn/pull/79108))
+  * Convert to regular constructor for partial types (PR: [#79092](https://github.com/dotnet/roslyn/pull/79092))
+  * Fix roslyn test modifiers in semantic tokens (PR: [#79070](https://github.com/dotnet/roslyn/pull/79070))
+  * Fix restore for 'dotnet run app.cs' in IDE (PR: [#78990](https://github.com/dotnet/roslyn/pull/78990))
+* Bump Razor to 10.0.0-preview.25329.6 (PR: [#8392](https://github.com/dotnet/vscode-csharp/pull/8392))
+  * Fix formatting issue with code block opening braces (PR: [#11969](https://github.com/dotnet/razor/pull/11969))
+
 # 2.84.x
 * Bump Roslyn to 5.0.0-1.25319.11 (PR: [#8380](https://github.com/dotnet/vscode-csharp/pull/8380))
   * Preserve trailing directives when converting a Program.Main program to top level statements. (#79062) (PR: [#79062](https://github.com/dotnet/roslyn/pull/79062))

CONTRIBUTING.md

@@ -36,12 +36,15 @@ Follow these steps to build, run, and test the repository:
 
 #### Building
 
+If you have the ability to run powershell, you can invoke "init.ps1" from the root of the repo. If not, the following steps will get build going for you as well:
+
 1. Run `npm install -g vsts-npm-auth`, then run `vsts-npm-auth -config .npmrc` - This command will configure your credentials for the next command.
    a.  If you have already authenticated before, but the token expired, you may need to run `vsts-npm-auth -config .npmrc -f` instead.
 2. Run `npm i` - This command installs the project dependencies.
 3. Run `npm i -g gulp` - This command installs Gulp globally.
 4. Run `gulp installDependencies` - This command downloads the various dependencies as specified by the version in the [package.json](package.json) file.
-5. Run `code .` - This command opens the project in Visual Studio Code.
+
+You can now run `code .` - This command opens the project in Visual Studio Code.
 
 #### Running
 
@@ -168,7 +171,7 @@ To package this extension, we need to create VSIX Packages. The VSIX packages ca
 
 ## Updating the `Roslyn` Language Server Version
 
-In order to pull in new packages from upstreams into the msft_consumption feed we use for restoring, you will need to be a member of the 'CSharp VS Code Extension contributors' group in the [Azure Devops instance](https://dev.azure.com/azure-public/vside/_settings/teams).  
+In order to pull in new packages from upstreams into the msft_consumption feed we use for restoring, you will need to be a member of the 'CSharp VS Code Extension contributors' group in the [Azure Devops instance](https://dev.azure.com/azure-public/vside/_settings/teams).
 
 To update the version of the roslyn server used by the extension do the following:
 1.  Find the the Roslyn signed build you want from [here](https://dnceng.visualstudio.com/internal/_build?definitionId=327&_a=summary).  Typically the latest successful build of main is fine.
@@ -195,5 +198,5 @@ The marketplace release is managed by an internal AzDo pipeline.  On the pipelin
 1.  The branch will **always** be main, no matter if release a build from prerelease or release.
 2.  Uncheck the "test" option.
 3.  In "Resources", choose "dotnet-vscode-csharp [officialBuildCI]", then check only the build that should be released, and then confirm with "Use selected run".  Based on the selected build, it will automatically determine if it is prerelease or release. ![release pipeline image](./docs/release_pipeline.png)
-4.  The pipeline parameters should then look something like the following image.  Hit "Run". ![release pipeline parameters image](./docs/release_pipeline_params.png) 
+4.  The pipeline parameters should then look something like the following image.  Hit "Run". ![release pipeline parameters image](./docs/release_pipeline_params.png)
 5.  After a bit, the pipeline will request approval from an authorized approver before it actually uploads to the marketplace.  Hit approve and it will continue.

SUPPORT.md

@@ -68,7 +68,7 @@ For issues with Razor, the Razor Log output window can contain useful informatio
 4. **Reset the Log Level**:
    - Once the logs are collected, reset the log level to `Info`.
 
-#### Project Loading Problems
+### Project Loading Problems
 
 Missing language features are often caused by a failure to load the project(s) or solution. To diagnose and resolve these issues, follow these steps:
 
@@ -86,7 +86,7 @@ Missing language features are often caused by a failure to load the project(s) o
    - If you are using C# Dev Kit, check the Solution Explorer to ensure the project is displayed with the expected references.
    - If the references or structure are not as expected, include the contents of the `Projects` output window in your issue report.
 
-#### Colorization problems
+### Colorization problems
 If you encounter issues with document classification (e.g., incorrect syntax highlighting or colorization), please provide the following information to help us diagnose the problem:
 
 1. **Theme in Use**:
@@ -101,7 +101,7 @@ If you encounter issues with document classification (e.g., incorrect syntax hig
 
      ![Inspect Tokens and Scopes Output](./docs/inspect_tokens.png)
 
-#### Diagnostics problems
+### Diagnostics problems
 
 For issues with diagnostics, please provide values of the background analysis scope options, `dotnet.backgroundAnalysis.analyzerDiagnosticsScope` and `dotnet.backgroundAnalysis.compilerDiagnosticsScope`
 ![background analysis settings](./docs/background_analysis.png)
@@ -116,6 +116,21 @@ If the language server crashes, general logs are often helpful for diagnosing th
 > [!WARNING]
 > The dump can contain detailed information on the project - generally we will provide an email so that it can be shared privately
 
+#### Recording a language server trace
+
+When investigating performance issues, we may request a performance trace of the language server to diagnose what is causing the problem.  These are typically taken via [dotnet-trace](https://learn.microsoft.com/en-us/dotnet/core/diagnostics/dotnet-trace) (a cross platform tool to collect performance traces of .NET processes)
+
+The C# extension has a built in command, `csharp.recordLanguageServerTrace` to help with trace collection.  This command will install `dotnet-trace` as a global tool and invoke it against the language server.
+
+1.  Invoke the record language server trace command
+![alt text](docs/recordTraceCommand.png)
+2.  Select the folder to save the trace.
+3.  Accept the default trace arguments, or change them if requested
+![alt text](docs/recordTraceArgs.png)
+4.  A new terminal window will open to run the trace collection.  While the trace is running, reproduce the peformance issue.  When done, hit <Enter> or <Ctrl+C> in the trace window to stop the trace
+![alt text](docs/recordTraceTerminal.png)
+5.  Share the trace.  Note that the trace may contain PII, so generally we will provide an email or other confidential way to share the trace with us.
+
 ## Microsoft Support Policy
 
 Support for this project is limited to the resources listed above.

azure-pipelines.yml

@@ -17,6 +17,7 @@ pr:
   paths:
     exclude:
     - '*.md'
+    - 'version.json'
     - .github/*
     - azure-pipelines/release.yml
     - CHANGELOG.

azure-pipelines/test-matrix.yml

@@ -28,6 +28,9 @@ jobs:
       RazorTests:
         npmCommand: test:integration:razor
         isIntegration: true
+      RazorCohostTests:
+        npmCommand: test:integration:razor:cohost
+        isIntegration: true
   pool: ${{ parameters.pool }}
   ${{ if parameters.containerName }}:
     container: ${{ parameters.containerName }}