[automated] Merge branch 'main' => 'prerelease'

.vscode/launch.json

@@ -97,33 +97,6 @@
             "preLaunchTask": "packageDev",
             "internalConsoleOptions": "openOnSessionStart"
         },
-        {
-            "name": "[Razor] 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/devkit_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": "[Razor Cohost] Run Current File Integration Test",
             "type": "extensionHost",
@@ -248,15 +221,6 @@
             "args": ["generateOptionsSchema"],
             "cwd": "${workspaceFolder}"
         },
-        {
-            "type": "node",
-            "request": "launch",
-            "name": "Razor integration tests",
-            "preLaunchTask": "build",
-            "program": "${workspaceFolder}/node_modules/gulp/bin/gulp.js",
-            "args": ["test:integration:razor"],
-            "cwd": "${workspaceFolder}"
-        },
         {
             "type": "node",
             "request": "launch",

CHANGELOG.md

@@ -3,6 +3,46 @@
 - 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.103.x
+* include roslyn event source configs in profiling command (PR: [#8808](https://github.com/dotnet/vscode-csharp/pull/8808))
+* Revise SUPPORT.md for clarity on sharing information (PR: [#8807](https://github.com/dotnet/vscode-csharp/pull/8807))
+* Fix indentation after pressing Enter in Razor files (PR: [#8809](https://github.com/dotnet/vscode-csharp/pull/8809))
+* Remove non-cohosting editor code (PR: [#8810](https://github.com/dotnet/vscode-csharp/pull/8810))
+* Bump Razor to 10.0.0-preview.25577.1 (PR: [#8820](https://github.com/dotnet/vscode-csharp/pull/8820))
+  * Explicitly handle switch expressions in parser (PR: [#12500](https://github.com/dotnet/razor/pull/12500))
+  * Bring back Go to Def, FAR and hover for components via Roslyn (PR: [#12501](https://github.com/dotnet/razor/pull/12501))
+  * Don't add newlines at the start of implicit expressions in on type formatting (PR: [#12489](https://github.com/dotnet/razor/pull/12489))
+  * Add the RazorCSharp keywords to completion (PR: [#12522](https://github.com/dotnet/razor/pull/12522))
+  * Remove non-cohost bits from C# extension (PR: [#12518](https://github.com/dotnet/razor/pull/12518))
+* Update Roslyn to 5.3.0-2.25571.4 (PR: [#8813](https://github.com/dotnet/vscode-csharp/pull/8813))
+  * Fix typing of 'else' causing unwanted indentation of following statement (PR: [#81393](https://github.com/dotnet/roslyn/pull/81393))
+  * Show semantic errors for loose files with top-level statements (PR: [#81326](https://github.com/dotnet/roslyn/pull/81326))
+  * Simplify and fix fading options. (PR: [#81346](https://github.com/dotnet/roslyn/pull/81346))
+  * Colorization for array, pointer, and function pointer type aliases followup (PR: [#81363](https://github.com/dotnet/roslyn/pull/81363))
+  * Fix: Support compound assignments in "Use null propagation" analyzer (PR: [#81328](https://github.com/dotnet/roslyn/pull/81328))
+  * Fix IDE0032: Code fixer adds setter for static properties with instance constructor writes (PR: [#81327](https://github.com/dotnet/roslyn/pull/81327))
+  * Add 'add extension completion' support for modern extensions. (PR: [#81239](https://github.com/dotnet/roslyn/pull/81239))
+* Add second opt-out flag for file-based programs (PR: [#8823](https://github.com/dotnet/vscode-csharp/pull/8823))
+* Update Roslyn to 5.3.0-2.25603.1 (PR: [#8823](https://github.com/dotnet/vscode-csharp/pull/8823))
+  * Improve error message for misplaced variable designator in property patterns (PR: [#81287](https://github.com/dotnet/roslyn/pull/81287))
+  * Add EnableFileBasedProgramsWhenAmbiguous flag (PR: [#81513](https://github.com/dotnet/roslyn/pull/81513))
+  * Include extension properties in object initializer completion (PR: [#81498](https://github.com/dotnet/roslyn/pull/81498))
+  * Fix IDE0048 to trigger consistently for `is null` and `is not null` in coalesce expressions (PR: [#81489](https://github.com/dotnet/roslyn/pull/81489))
+  * Do not offer to add a `using namespace` on an alias qualified name. (PR: [#81487](https://github.com/dotnet/roslyn/pull/81487))
+  * Add code refactoring to offer usings for fully qualified names (PR: [#81484](https://github.com/dotnet/roslyn/pull/81484))
+  * Fix RemoveUnusedMembers handling of default parameter reads (PR: [#80810](https://github.com/dotnet/roslyn/pull/80810))
+  * Fix exception in InlineMethodRefactoringProvider for recursive calls (PR: [#81469](https://github.com/dotnet/roslyn/pull/81469))
+  * Fix issue where we weren't offering snippets inline after an expression (PR: [#81466](https://github.com/dotnet/roslyn/pull/81466))
+  * Follow user naming preferences when generating type snippets (PR: [#81465](https://github.com/dotnet/roslyn/pull/81465))
+  * Fix 'use var' with forr/for snippets (PR: [#81464](https://github.com/dotnet/roslyn/pull/81464))
+  * Map edits in LSP rename, like in VS (PR: [#81450](https://github.com/dotnet/roslyn/pull/81450))
+  * Remove more unused bits of Razor EA (PR: [#81452](https://github.com/dotnet/roslyn/pull/81452))
+  * Fix for-snippet not simplifying (PR: [#81461](https://github.com/dotnet/roslyn/pull/81461))
+  * Fix duplicate parameter names in introduce parameter refactoring (PR: [#81404](https://github.com/dotnet/roslyn/pull/81404))
+  * Fade out portion of linq expression that can be simplified (PR: [#81419](https://github.com/dotnet/roslyn/pull/81419))
+  * Improve LSP conformance in MS.CA.LanguageServer (PR: [#81174](https://github.com/dotnet/roslyn/pull/81174))
+  * Improve 'else' indentation in a couple more cases (PR: [#81411](https://github.com/dotnet/roslyn/pull/81411))
+
 # 2.102.x
 * Update Roslyn to 5.3.0-2.25568.9 (PR: [#8799](https://github.com/dotnet/vscode-csharp/pull/8799))
   * Handle automatic restores on directly on the server (PR: [#81233](https://github.com/dotnet/roslyn/pull/81233))

SUPPORT.md

@@ -115,7 +115,7 @@ If the language server crashes, general logs are often helpful for diagnosing th
 - When the server crashes, a dump in the specified folder will be created.
 
 > [!WARNING]
-> The dump can contain detailed information on the project - generally we will provide an email so that it can be shared privately
+> The dump will contain detailed information about the workspace.  See [Sharing information privately](#sharing-information-privately)
 
 ### Recording a language server trace
 
@@ -130,15 +130,19 @@ The C# extension has a built in command, `csharp.recordLanguageServerTrace` to h
 ![alt text](docs/images/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/images/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.
+5.  Share the trace.
+
+> [!WARNING]
+> The trace will contain detailed information about the workspace.  See [Sharing information privately](#sharing-information-privately)
 
 ### Sharing information privately
-Detailed logs, dumps, traces, and other information can sometimes contain private information that you do not wish to share publicly.  Instead, you can utilize the Developer Community page to share these privately to Microsoft.
+Detailed logs, dumps, traces, and other information can sometimes contain private information that you do not wish to share publicly on GitHub (for example file paths and file contents).  Instead, you can utilize the Developer Community page to share these privately to Microsoft.
 
 1.  Go to https://developercommunity.visualstudio.com/dotnet/report
-2.  Fill in the issue title, reference the GitHub issue in the description, and upload the attachments
+2.  Fill in the issue title, reference the GitHub issue in the description, and upload the attachments.  Note that there is a 2 GB limit on attached files.  Dumps can often be larger than that, so we recommend compressing them to a `.zip` before uploading.
+
 ![developer community feedback page](docs/images/developer_community_feedback.png)
-3.  Once created, a comment on the GitHub issue a link to the new Developer Community ticket.
+4.  Once created, a comment on the GitHub issue a link to the new Developer Community ticket.
 
 
 

azure-pipelines.yml

@@ -56,9 +56,9 @@ schedules:
 
 variables:
 - name: testVSCodeVersion
-  ${{ if eq( variables['Build.Reason'], 'Schedule' ) }}: 
+  ${{ if eq( variables['Build.Reason'], 'Schedule' ) }}:
     value: insiders
-  ${{ else }}: 
+  ${{ else }}:
     value: stable
 
 stages:
@@ -84,7 +84,7 @@ stages:
       installAdditionalLinuxDependencies: true
       pool:
         name: NetCore-Public
-        demands: ImageOverride -equals 1es-ubuntu-2004-open
+        demands: ImageOverride -equals build.ubuntu.2204.amd64.open
       containerName: mcr.microsoft.com/dotnet/sdk:8.0-noble
 
 - stage:
@@ -102,7 +102,7 @@ stages:
       installAdditionalLinuxDependencies: true
       pool:
         name: NetCore-Public
-        demands: ImageOverride -equals 1es-ubuntu-2004-open
+        demands: ImageOverride -equals build.ubuntu.2204.amd64.open
       containerName: mcr.microsoft.com/dotnet/sdk:9.0-noble
 
 - stage:
@@ -118,7 +118,7 @@ stages:
       installAdditionalLinuxDependencies: true
       pool:
         name: NetCore-Public
-        demands: ImageOverride -equals 1es-ubuntu-2004-open
+        demands: ImageOverride -equals build.ubuntu.2204.amd64.open
       containerName: mcr.microsoft.com/dotnet/sdk:10.0.100-rc.2-noble
 
 - stage: Test_Windows_Stage
@@ -159,7 +159,7 @@ stages:
     strategy:
       matrix:
         linux:
-          demandsName: 1es-ubuntu-2004-open
+          demandsName: build.ubuntu.2204.amd64.open
         windows:
           demandsName: 1es-windows-2022-open
     pool:

azure-pipelines/test-omnisharp.yml

@@ -15,6 +15,12 @@ steps:
   parameters:
     installDotNet: ${{ parameters.installDotNet }}
 
+# The build.ubuntu.2204.amd64.open image is missing packages which are required for running integration tests.
+- script: sudo apt-get update -y && sudo apt-get install -y libglib2.0-0 libnss3 libatk-bridge2.0-dev libdrm2 libgtk-3-0 libgbm-dev xvfb
+  displayName: 'Install additional Linux dependencies'
+  target: host
+  condition: eq(variables['Agent.OS'], 'Linux')
+
 - template: test-prereqs.yml
 
 - script: npm run omnisharptest