Skip to content

Commit 12f279f

Browse files
sandy081Copilot
sandy081
and
Copilot
committed
Merge origin/main into session database migration
Retain central catalog ownership and bounded synchronization. Adapt the new peer workspace-transition test to central chat creation and verify listing does not read transition history. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2 parents 82e64bd + 1b921e3 commit 12f279f

282 files changed

Lines changed: 13715 additions & 1858 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎.eslint-allowed-bracket-notation-files‎

Lines changed: 534 additions & 0 deletions
Large diffs are not rendered by default.

‎.eslint-plugin-local/code-no-bracket-notation-for-identifiers.ts‎

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,6 @@ export default new class NoBracketNotationForIdentifiers implements eslint.Rule.
3939
* Check if a string is a valid JavaScript identifier
4040
*/
4141
function isValidIdentifier(str: string): boolean {
42-
if (str.includes('\\')) {
43-
return false;
44-
}
4542
const scanner = ts.createScanner(ts.ScriptTarget.Latest, false, ts.LanguageVariant.Standard, str);
4643
const token = scanner.scan();
4744
const isIdentifierName = token === ts.SyntaxKind.Identifier
@@ -63,6 +60,11 @@ export default new class NoBracketNotationForIdentifiers implements eslint.Rule.
6360
return;
6461
}
6562

63+
// Preserve escaped property names instead of replacing their authored representation.
64+
if (memberExpr.property.raw.includes('\\')) {
65+
return;
66+
}
67+
6668
const propertyName = memberExpr.property.value;
6769

6870
// If it's a valid identifier, report it

‎.github/CODEOWNERS‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,3 +19,7 @@ src/vs/workbench/services/extensions/common/extensionPoints.json @TylerLeonhardt
1919
# review is required to make sure TypeScript is not a better choice.
2020
.eslint-allowed-javascript-files @alexr00 @alexdima @sbatten @TylerLeonhardt
2121

22+
# Allowlist for the `local/code-no-bracket-notation-for-identifiers` lint rule.
23+
# Adding entries here permits bracket notation for identifier properties;
24+
# review is required to make sure dot notation cannot be used instead.
25+
.eslint-allowed-bracket-notation-files @alexdima @dmitrivMS

‎.github/codeql/codeql-config.yml‎

Lines changed: 105 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,115 @@
1-
# Apply this file to CodeQL default setup with the repository property:
2-
# github-codeql-config-file: ./.github/codeql/codeql-config.yml
1+
# Applied by .github/workflows/codeql.yml.
32
paths-ignore:
4-
# Keep directory exclusions scoped: product code imports helpers from some
5-
# extension src/test directories.
6-
- 'test/**'
7-
- 'src/**/test/**'
8-
- 'build/**/test/**'
3+
# Development, build, packaging, and CI tooling, not shipped product code.
4+
- '.agents/**'
5+
- '.eslint-plugin-local/**'
6+
- '.github/**'
7+
- '.vscode-test.js'
8+
- '.vscode/**'
9+
- 'build/**'
10+
- 'cli/build.rs'
11+
- 'eslint.config.js'
12+
- 'extensions/**/.esbuild.*'
13+
- 'extensions/*/build/**'
14+
- 'extensions/*/esbuild*.mts'
15+
- 'extensions/*/scripts/**'
16+
- 'extensions/copilot/.eslintplugin/**'
17+
- 'extensions/copilot/.mocha-multi-reporters.js'
18+
- 'extensions/copilot/.mocharc.js'
19+
- 'extensions/copilot/.vscode-test.mjs'
20+
- 'extensions/copilot/.vscode/**'
21+
- 'extensions/copilot/chat-lib/vitest.config.ts'
22+
- 'extensions/copilot/script/**'
23+
- 'extensions/copilot/vite.config.ts'
24+
- 'extensions/esbuild*.mts'
25+
- 'extensions/postinstall.mjs'
26+
- 'extensions/search-result/syntaxes/generateTMLanguage.js'
27+
- 'gulpfile.mjs'
28+
- 'scripts/**'
29+
30+
# Generated build output and dependencies; analyze the product sources instead.
31+
- '**/node_modules/**'
32+
- '**/out/**'
33+
- '.build/**'
34+
- 'cli/target/**'
35+
- 'coverage/**'
36+
- 'extensions/**/dist/**'
37+
- 'out*/**'
38+
39+
# Test directories, fixtures, and helpers.
940
- 'cli/tests/**'
10-
- '.eslint-plugin-local/tests/**'
41+
- 'extensions/*/server/**/test/**'
42+
- 'extensions/*/test-workspace/**'
1143
- 'extensions/*/test/**'
1244
- 'extensions/*/tests/**'
45+
- 'extensions/configuration-editing/src/test/**'
46+
- 'extensions/copilot/src/extension/**/test/**'
47+
- 'extensions/copilot/src/extension/agents/node/adapters/openaiAdapterForSTests.ts'
48+
- 'extensions/copilot/src/extension/typescriptContext/serverPlugin/fixtures/**'
49+
- 'extensions/copilot/src/lib/**/test/**'
50+
- 'extensions/copilot/src/platform/**/test/**'
51+
- 'extensions/copilot/src/platform/commands/common/mockRunCommandExecutionService.ts'
52+
- 'extensions/copilot/src/platform/notebook/common/mockAlternativeContentService.ts'
53+
- 'extensions/copilot/src/platform/tasks/common/testTasksService.ts'
54+
- 'extensions/copilot/src/sanity-test-extension.ts'
55+
- 'extensions/copilot/src/shared-fetch-utils/**/test/**'
56+
- 'extensions/copilot/src/test-extension.ts'
57+
# Standalone chat-lib rewrites runtime VS Code API imports to util/common/test/shims.
58+
# These are required runtime dependencies, not tests; exclude only their test-only siblings.
59+
- 'extensions/copilot/src/util/common/test/annotatedSrc.ts'
60+
- 'extensions/copilot/src/util/common/test/mockChatResponseStream.ts'
61+
- 'extensions/copilot/src/util/common/test/simpleMock.ts'
62+
- 'extensions/copilot/src/util/common/test/testUtils.ts'
63+
- 'extensions/copilot/src/util/node/test/**'
64+
- 'extensions/copilot/src/util/test/**'
65+
- 'extensions/emmet/src/test/**'
66+
- 'extensions/git-base/src/test/**'
67+
- 'extensions/git/src/test/**'
68+
- 'extensions/github-authentication/src/test/**'
69+
- 'extensions/github/src/test/**'
70+
- 'extensions/ipynb/src/test/**'
71+
- 'extensions/markdown-language-features/src/test/**'
72+
- 'extensions/microsoft-authentication/src/**/test/**'
73+
- 'extensions/notebook-renderers/src/test/**'
74+
- 'extensions/npm/src/test/**'
75+
- 'extensions/terminal-suggest/fixtures/**'
76+
- 'extensions/terminal-suggest/src/**/test/**'
77+
- 'extensions/typescript-language-features/src/test-all.ts'
78+
- 'extensions/typescript-language-features/src/test/**'
79+
- 'extensions/vscode-api-tests/**'
80+
- 'extensions/vscode-colorize-perf-tests/**'
81+
- 'extensions/vscode-colorize-tests/**'
82+
- 'extensions/vscode-test-resolver/**'
83+
- 'src/**/test/**'
84+
- 'src/vs/workbench/contrib/terminal/browser/terminalTestHelpers.ts'
85+
- 'test/**'
86+
1387
# Keep exact suffixes: themes.test.contribution.ts ships in the product.
88+
- '**/*.fixture.cjs'
89+
- '**/*.fixture.cts'
90+
- '**/*.fixture.js'
91+
- '**/*.fixture.jsx'
92+
- '**/*.fixture.mjs'
93+
- '**/*.fixture.mts'
94+
- '**/*.fixture.ts'
95+
- '**/*.fixture.tsx'
96+
- '**/*.sanity-test.ts'
97+
- '**/*.sanity-test.tsx'
98+
- '**/*.spec.cjs'
99+
- '**/*.spec.cts'
100+
- '**/*.spec.js'
101+
- '**/*.spec.jsx'
102+
- '**/*.spec.mjs'
103+
- '**/*.spec.mts'
104+
- '**/*.spec.ts'
105+
- '**/*.spec.tsx'
106+
- '**/*.stest.ts'
107+
- '**/*.stest.tsx'
108+
- '**/*.test.cjs'
109+
- '**/*.test.cts'
14110
- '**/*.test.js'
15111
- '**/*.test.jsx'
16112
- '**/*.test.mjs'
17-
- '**/*.test.cjs'
113+
- '**/*.test.mts'
18114
- '**/*.test.ts'
19115
- '**/*.test.tsx'
20-
- '**/*.test.mts'
21-
- '**/*.test.cts'

‎.github/skills/policy-and-managed-settings/github-managed-settings.md‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,12 @@ tampered with. The merged bag is then projected onto the declared schema (see be
6363
Client-side merging still happens *within* a channel's value (e.g. `enabledPlugins`,
6464
`extraKnownMarketplaces`).
6565

66+
**Sandbox routing exception:** `sandbox.enabled` follows the runtime's `force-on-wins`
67+
contract: `true` from any managed channel wins over `false` from another channel.
68+
The shared resolver supplies this result to harness selection, permissions UI, and
69+
Policy Diagnostics; runtime enforcement remains authoritative. Other keys retain
70+
their existing delivery-channel precedence.
71+
6672
## Schema source of truth
6773

6874
When the developer has `copilot-agent-runtime` checked out side-by-side, reference

‎.github/workflows/chat-lib-package.yml‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@ name: chat-lib tests
22

33
on:
44
pull_request:
5+
branches:
6+
- main
7+
- 'release/*'
58
workflow_dispatch:
69

710
permissions:

‎.github/workflows/codeql.yml‎

Lines changed: 51 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,51 @@
1+
name: CodeQL
2+
3+
on:
4+
push:
5+
branches:
6+
- main
7+
- 'release/**/*'
8+
pull_request:
9+
branches:
10+
- main
11+
- 'release/**/*'
12+
schedule:
13+
- cron: '17 19 * * 6'
14+
15+
permissions: {}
16+
17+
jobs:
18+
analyze:
19+
name: Analyze (${{ matrix.language }})
20+
runs-on: ubuntu-latest
21+
permissions:
22+
actions: read
23+
contents: read
24+
packages: read
25+
security-events: write
26+
strategy:
27+
fail-fast: false
28+
matrix:
29+
# Other detected languages occur only in test fixtures or build tooling.
30+
include:
31+
- language: javascript-typescript
32+
build-mode: none
33+
- language: rust
34+
build-mode: none
35+
36+
steps:
37+
- name: Checkout repository
38+
uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
39+
40+
- name: Initialize CodeQL
41+
uses: github/codeql-action/init@cdf488f595d80d6e07e03d4674febd5ab45fa938 # v4.37.9
42+
with:
43+
languages: ${{ matrix.language }}
44+
build-mode: ${{ matrix.build-mode }}
45+
dependency-caching: true
46+
config-file: ./.github/codeql/codeql-config.yml
47+
48+
- name: Perform CodeQL Analysis
49+
uses: github/codeql-action/analyze@cdf488f595d80d6e07e03d4674febd5ab45fa938 # v4.37.9
50+
with:
51+
category: /language:${{ matrix.language }}

‎.github/workflows/copilot-setup-steps.yml‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,9 @@ on:
88
paths:
99
- .github/workflows/copilot-setup-steps.yml
1010
pull_request:
11+
branches:
12+
- main
13+
- 'release/*'
1114
paths:
1215
- .github/workflows/copilot-setup-steps.yml
1316

‎.github/workflows/monaco-editor.yml‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,9 @@ jobs:
6868
- name: Run Monaco Editor Checks
6969
run: npm run monaco-compile-check
7070

71+
- name: Check generated Monaco declarations
72+
run: npm run gulp monacodts-check
73+
7174
- name: Editor Distro & ESM
7275
run: npm run gulp editor-distro
7376

‎.github/workflows/telemetry.yml‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,11 @@
11
name: 'Telemetry'
2-
on: pull_request
2+
3+
on:
4+
pull_request:
5+
branches:
6+
- main
7+
- 'release/*'
8+
39
permissions: {}
410
jobs:
511
check-metadata:

0 commit comments

Comments
 (0)