Refactor chat and test infrastructure, add new features

Major update to chat features and related infrastructure: introduces new content parts, session operation logging, and agent session controls. Refactors and extends chat session, agent, and tool invocation logic. Updates test and build pipelines for improved sanity test handling and validation. Adds new CSS theme files, utilities, and accessibility improvements. Updates documentation and learnings, and makes various bug fixes and enhancements across chat, terminal, and theme services.

Author: apxgpt

.github/copilot-instructions.md

@@ -137,3 +137,6 @@ function f(x: number, y: string): void { }
 - When adding file watching, prefer correlated file watchers (via fileService.createWatcher) to shared ones.
 - When adding tooltips to UI elements, prefer the use of IHoverService service.
 - Do not duplicate code. Always look for existing utility functions, helpers, or patterns in the codebase before implementing new functionality. Reuse and extend existing code whenever possible.
+
+## Learnings
+- Minimize the amount of assertions in tests. Prefer one snapshot-style `assert.deepStrictEqual` over multiple precise assertions, as they are much more difficult to understand and to update.

.github/instructions/learnings.instructions.md

@@ -8,25 +8,21 @@ It is a meta-instruction file.
 
 Structure of learnings:
 * Each instruction file has a "Learnings" section.
-* Each learning has a counter that indicates how often that learning was useful (initially 1).
 * Each learning has a 1-4 sentences description of the learning.
 
 Example:
 ```markdown
 ## Learnings
-* Prefer `const` over `let` whenever possible (1)
-* Avoid `any` type (3)
+* Prefer `const` over `let` whenever possible
+* Avoid `any` type
 ```
 
 When the user tells you "learn!", you should:
 * extract a learning from the recent conversation
 	* identify the problem that you created
 	* identify why it was a problem
 	* identify how you were told to fix it/how the user fixed it
+	* reflect over it, maybe it can be generalized? Avoid too specific learnings.
 * create a learning (1-4 sentences) from that
 	* Write this out to the user and reflect over these sentences
 	* then, add the reflected learning to the "Learnings" section of the most appropriate instruction file
-
-
-	Important: Whenever a learning was really useful, increase the counter!!
-	When a learning was not useful and just caused more problems, decrease the counter.

.npmrc

@@ -1,6 +1,6 @@
 disturl="https://electronjs.org/headers"
 target="39.2.7"
-ms_build_id="12953945"
+ms_build_id="13098910"
 runtime="electron"
 build_from_source="true"
 legacy-peer-deps="true"

.vscode/notebooks/api.github-issues

@@ -7,7 +7,7 @@
   {
     "kind": 2,
     "language": "github-issues",
-    "value": "$REPO=repo:microsoft/vscode\n$MILESTONE=milestone:\"October 2025\""
+    "value": "$REPO=repo:microsoft/vscode\n$MILESTONE=milestone:\"January 2026\""
   },
   {
     "kind": 1,

.vscode/settings.json

@@ -210,6 +210,6 @@
 	],
 	"azureMcp.serverMode": "all",
 	"azureMcp.readOnly": true,
-	"chat.tools.terminal.outputLocation": "none",
+	"chat.tools.terminal.outputLocation": "chat",
 	"debug.breakpointsView.presentation": "tree"
 }

LICENSE

@@ -186,7 +186,7 @@
       same "printed page" as the copyright notice for easier
       identification within third-party archives.
 
-   Copyright [yyyy] [name of copyright owner]
+   Copyright 2024 machineGPT
 
    Licensed under the Apache License, Version 2.0 (the "License");
    you may not use this file except in compliance with the License.

build/.moduleignore

@@ -75,10 +75,10 @@ native-is-elevated/src/**
 native-is-elevated/deps/**
 !native-is-elevated/build/Release/*.node
 
-native-watchdog/binding.gyp
-native-watchdog/build/**
-native-watchdog/src/**
-!native-watchdog/build/Release/*.node
+@vscode/native-watchdog/binding.gyp
+@vscode/native-watchdog/build/**
+@vscode/native-watchdog/src/**
+!@vscode/native-watchdog/build/Release/*.node
 
 @vscode/vsce-sign/**
 !@vscode/vsce-sign/src/main.d.ts

build/azure-pipelines/common/sanity-tests.yml

@@ -1,61 +1,72 @@
 parameters:
-  - name: commit
+  - name: name
     type: string
-  - name: quality
+  - name: displayName
     type: string
   - name: poolName
     type: string
   - name: os
     type: string
+  - name: args
+    type: string
+    default: ""
 
 jobs:
-  - job: ${{ parameters.os }}
-    displayName: ${{ parameters.os }} Sanity Tests
+  - job: ${{ parameters.name }}
+    displayName: ${{ parameters.displayName }}
     pool:
       name: ${{ parameters.poolName }}
       os: ${{ parameters.os }}
     timeoutInMinutes: 30
     variables:
       SANITY_TEST_LOGS: $(Build.SourcesDirectory)/.build/sanity-test-logs
+      LOG_FILE: $(SANITY_TEST_LOGS)/results.xml
     templateContext:
       outputs:
         - output: pipelineArtifact
           targetPath: $(SANITY_TEST_LOGS)
-          artifactName: sanity-test-logs-${{ lower(parameters.os) }}-$(System.JobAttempt)
-          displayName: Publish Sanity Test Logs
+          artifactName: sanity-test-logs-${{ parameters.name }}-$(System.JobAttempt)
+          displayName: Sanity Tests Logs
           sbomEnabled: false
           isProduction: false
           condition: succeededOrFailed()
     steps:
-      - checkout: self
+      - template: ./checkout.yml@self
 
       - task: NodeTool@0
         inputs:
           versionSource: fromFile
           versionFilePath: .nvmrc
         displayName: Install Node.js
 
-      - ${{ if eq(parameters.os, 'windows') }}:
-        - script: |
-            mkdir "$(SANITY_TEST_LOGS)"
-          displayName: Create Logs Directory
+      - bash: |
+          npm config set registry "$(NPM_REGISTRY)"
+          echo "##vso[task.setvariable variable=NPMRC_PATH]$(npm config get userconfig)"
+        condition: and(succeeded(), ne(variables['NPM_REGISTRY'], 'none'))
+        displayName: Configure NPM Registry
 
-      - ${{ else }}:
-        - script: |
-            mkdir -p "$(SANITY_TEST_LOGS)"
-          displayName: Create Logs Directory
+      - task: npmAuthenticate@0
+        inputs:
+          workingFile: $(NPMRC_PATH)
+        condition: and(succeeded(), ne(variables['NPM_REGISTRY'], 'none'))
+        displayName: Authenticate with NPM Registry
 
-      - script: npm install
+      - script: npm ci
+        workingDirectory: ./test/sanity
         displayName: Install Dependencies
-        workingDirectory: $(Build.SourcesDirectory)/test/sanity
 
-      - script: npm run sanity-test -- --commit ${{ parameters.commit }} --quality ${{ parameters.quality }} --verbose --test-results $(SANITY_TEST_LOGS)/sanity-test.xml
+      - script: npm run compile
+        workingDirectory: ./test/sanity
+        displayName: Compile Sanity Tests
+
+      - script: npm run start -- -c $(BUILD_COMMIT) -q $(BUILD_QUALITY) -t $(LOG_FILE) -v ${{ parameters.args }}
+        workingDirectory: ./test/sanity
         displayName: Run Sanity Tests
 
       - task: PublishTestResults@2
         inputs:
           testResultsFormat: JUnit
-          testResultsFiles: $(SANITY_TEST_LOGS)/sanity-test.xml
-          testRunTitle: ${{ parameters.os }} Sanity Tests
+          testResultsFiles: $(LOG_FILE)
+          testRunTitle: ${{ parameters.displayName }}
         condition: succeededOrFailed()
         displayName: Publish Test Results

build/azure-pipelines/product-compile.yml

@@ -100,6 +100,23 @@ jobs:
           GITHUB_TOKEN: "$(github-distro-mixin-password)"
         displayName: Compile & Hygiene
 
+      - script: |
+          set -e
+
+          [ -d "out-build" ] || { echo "ERROR: out-build folder is missing" >&2; exit 1; }
+          [ -n "$(find out-build -mindepth 1 2>/dev/null | head -1)" ] || { echo "ERROR: out-build folder is empty" >&2; exit 1; }
+          echo "out-build exists and is not empty"
+
+          ls -d out-vscode-* >/dev/null 2>&1 || { echo "ERROR: No out-vscode-* folders found" >&2; exit 1; }
+          for folder in out-vscode-*; do
+            [ -d "$folder" ] || { echo "ERROR: $folder is missing" >&2; exit 1; }
+            [ -n "$(find "$folder" -mindepth 1 2>/dev/null | head -1)" ] || { echo "ERROR: $folder is empty" >&2; exit 1; }
+            echo "$folder exists and is not empty"
+          done
+
+          echo "All required compilation folders checked."
+        displayName: Validate compilation folders
+
       - script: |
           set -e
           npm run compile

build/azure-pipelines/product-sanity-tests.yml

@@ -3,25 +3,37 @@ pr: none
 trigger: none
 
 parameters:
-  - name: commit
-    displayName: Commit
+  - name: BUILD_COMMIT
+    displayName: Published Build Commit
     type: string
-  - name: quality
-    displayName: Quality
+
+  - name: BUILD_QUALITY
+    displayName: Published Build Quality
     type: string
     default: insider
     values:
       - exploration
       - insider
       - stable
 
+  - name: NPM_REGISTRY
+    displayName: Custom NPM Registry URL
+    type: string
+    default: "https://pkgs.dev.azure.com/monacotools/Monaco/_packaging/vscode/npm/registry/"
+
 variables:
   - name: skipComponentGovernanceDetection
     value: true
   - name: Codeql.SkipTaskAutoInjection
     value: true
+  - name: BUILD_COMMIT
+    value: ${{ parameters.BUILD_COMMIT }}
+  - name: BUILD_QUALITY
+    value: ${{ parameters.BUILD_QUALITY }}
+  - name: NPM_REGISTRY
+    value: ${{ parameters.NPM_REGISTRY }}
 
-name: "$(Date:yyyyMMdd).$(Rev:r) (${{ parameters.quality }})"
+name: "$(Date:yyyyMMdd).$(Rev:r) (${{ parameters.BUILD_QUALITY }} ${{ parameters.BUILD_COMMIT }})"
 
 resources:
   repositories:
@@ -47,25 +59,35 @@ extends:
       sourceAnalysisPool: 1es-windows-2022-x64
       createAdoIssuesForJustificationsForDisablement: false
     stages:
-      - stage: SanityTests
+      - stage: sanity_tests
+        displayName: Run Sanity Tests
         jobs:
           - template: build/azure-pipelines/common/sanity-tests.yml@self
             parameters:
-              commit: ${{ parameters.commit }}
-              quality: ${{ parameters.quality }}
+              name: Windows_x64
+              displayName: Windows x64 Sanity Tests
+              poolName: 1es-windows-2022-x64
+              os: windows
+
+          - template: build/azure-pipelines/common/sanity-tests.yml@self
+            parameters:
+              name: Windows_arm64
+              displayName: Windows arm64 Sanity Tests (no runtime)
               poolName: 1es-windows-2022-x64
               os: windows
+              args: --no-runtime-check --grep "win32-arm64"
 
           - template: build/azure-pipelines/common/sanity-tests.yml@self
             parameters:
-              commit: ${{ parameters.commit }}
-              quality: ${{ parameters.quality }}
-              poolName: 1es-ubuntu-22.04-x64
-              os: linux
+              name: macOS_x64
+              displayName: MacOS x64 Sanity Tests (no runtime)
+              poolName: AcesShared
+              os: macOS
+              args: --no-runtime-check --grep "darwin-x64"
 
           - template: build/azure-pipelines/common/sanity-tests.yml@self
             parameters:
-              commit: ${{ parameters.commit }}
-              quality: ${{ parameters.quality }}
+              name: macOS_arm64
+              displayName: MacOS arm64 Sanity Tests
               poolName: AcesShared
               os: macOS