Skip to content

Commit 257e14d

Browse files
committed
Merge main
2 parents 3298c41 + 9255dc5 commit 257e14d

Some content is hidden

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

52 files changed

+4279
-554
lines changed

.changeset/curvy-cows-cry.md

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
---
2+
"roo-cline": patch
3+
---
4+
5+
fix: ensure consistent line endings in git fallback strategy
6+
7+
Fixed a cross-platform issue where tests would fail on GitHub Windows runners but pass on local Windows machines due to line ending differences. The fix ensures consistent line ending handling by:
8+
9+
1. Normalizing CRLF to LF when reading files in the git fallback strategy
10+
2. Disabling Git's automatic line ending conversion using core.autocrlf setting
11+
3. Maintaining consistent line ending usage throughout the text operations

.github/workflows/code-qa.yml

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,10 @@ jobs:
6262
run: npm run knip
6363

6464
test-extension:
65-
runs-on: ubuntu-latest
65+
runs-on: ${{ matrix.os }}
66+
strategy:
67+
matrix:
68+
os: [ubuntu-latest, windows-latest]
6669
steps:
6770
- name: Checkout code
6871
uses: actions/checkout@v4
@@ -73,11 +76,16 @@ jobs:
7376
cache: 'npm'
7477
- name: Install dependencies
7578
run: npm run install:all
79+
- name: Compile (to build and copy WASM files)
80+
run: npm run compile
7681
- name: Run unit tests
7782
run: npx jest --silent
7883

7984
test-webview:
80-
runs-on: ubuntu-latest
85+
runs-on: ${{ matrix.os }}
86+
strategy:
87+
matrix:
88+
os: [ubuntu-latest, windows-latest]
8189
steps:
8290
- name: Checkout code
8391
uses: actions/checkout@v4

README.md

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

jest.config.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,12 @@ module.exports = {
1919
],
2020
},
2121
testMatch: ["**/__tests__/**/*.test.ts"],
22+
// Platform-specific test configuration
23+
testPathIgnorePatterns: [
24+
// Skip platform-specific tests based on environment
25+
...(process.platform === "win32" ? [".*\\.bash\\.test\\.ts$"] : [".*\\.cmd\\.test\\.ts$"]),
26+
// PowerShell tests are conditionally skipped in the test files themselves using the setupFilesAfterEnv
27+
],
2228
moduleNameMapper: {
2329
"^vscode$": "<rootDir>/src/__mocks__/vscode.js",
2430
"@modelcontextprotocol/sdk$": "<rootDir>/src/__mocks__/@modelcontextprotocol/sdk/index.js",
@@ -39,4 +45,5 @@ module.exports = {
3945
modulePathIgnorePatterns: [".vscode-test"],
4046
reporters: [["jest-simple-dot-reporter", {}]],
4147
setupFiles: ["<rootDir>/src/__mocks__/jest.setup.ts"],
48+
setupFilesAfterEnv: ["<rootDir>/src/integrations/terminal/__tests__/setupTerminalTests.ts"],
4249
}

locales/ca/README.md

Lines changed: 20 additions & 22 deletions
Large diffs are not rendered by default.

locales/de/README.md

Lines changed: 20 additions & 22 deletions
Large diffs are not rendered by default.

locales/es/README.md

Lines changed: 20 additions & 22 deletions
Large diffs are not rendered by default.

locales/fr/README.md

Lines changed: 20 additions & 22 deletions
Large diffs are not rendered by default.

locales/hi/README.md

Lines changed: 20 additions & 22 deletions
Large diffs are not rendered by default.

locales/it/README.md

Lines changed: 20 additions & 22 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)