Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit e1b951c

Browse files
committedJan 15, 2025·
Update packages with a VS Code engine bump to 1.96
With new string casts to quiet new warnings.
1 parent 0c5f2f8 commit e1b951c

File tree

8 files changed

+260
-281
lines changed

8 files changed

+260
-281
lines changed
 

‎docs/development.md

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,8 @@ dictates which version of Node.js the extension is eventually run with. This
3636
lets us finally update our `@types/node` development dependency to match, our
3737
developer machines if necessary, and the CI and OneBranch pipeline tasks.
3838

39-
[`vscodeVersion`]: https://github.com/microsoft/azuredatastudio/blob/4970733324ef8254b7c22a5dc55af7f8a1dea93f/product.json#L50
40-
[`electron`]: https://github.com/microsoft/vscode/blob/384ff7382de624fb94dbaf6da11977bba1ecd427/package.json#L159
41-
[Electron]: https://www.electronjs.org/blog/electron-30-0
39+
[`electron`]: https://github.com/microsoft/vscode/blob/138f619c86f1199955d53b4166bef66ef252935c/package.json#L156
40+
[Electron]: https://releases.electronjs.org/release/v32.2.6
4241
[Node.js]: https://nodejs.org/en/download/package-manager
4342

4443
### Building the Code
@@ -65,7 +64,7 @@ To debug the extension use one of the provided `Launch Extension` debug configur
6564

6665
All three templates use pre-launch tasks to build the code, and support automatic restart of the extension host on changes to the Extension source code. [Hot Reload](https://devblogs.microsoft.com/dotnet/introducing-net-hot-reload/) is also enabled for PowerShell Editor Services.
6766

68-
> [!WARNING]
67+
> [!WARNING]
6968
> There is a current limitation that, if you restart the extension/extension host or it is restarted due to a extension code change, the editor services attachment will be disconnected due to the PSES terminal being terminated, and you will either need to restart the debug session completely, or do a manual build of PSES and run the `Attach to Editor Services` debug launch manually.
7069
7170
Try the `powershell.developer.editorServicesWaitForDebugger` setting to ensure that you are fully attached before the extension startup process continues.

‎package-lock.json

Lines changed: 234 additions & 263 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎package.json

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
"publisher": "ms-vscode",
77
"description": "Develop PowerShell modules, commands and scripts in Visual Studio Code!",
88
"engines": {
9-
"vscode": "^1.94.0"
9+
"vscode": "^1.96.0"
1010
},
1111
"author": "Microsoft Corporation",
1212
"license": "SEE LICENSE IN LICENSE.txt",
@@ -75,30 +75,30 @@
7575
"optionalDependencies": {
7676
"@types/mocha": "^10.0.10",
7777
"@types/mock-fs": "^4.13.4",
78-
"@types/node": "^20.17.9",
78+
"@types/node": "^20.18.1",
7979
"@types/node-fetch": "^2.6.12",
8080
"@types/rewire": "^2.5.30",
8181
"@types/semver": "^7.5.8",
8282
"@types/sinon": "^17.0.3",
8383
"@types/ungap__structured-clone": "^1.2.0",
8484
"@types/uuid": "^9.0.8",
85-
"@types/vscode": "~1.94.0",
86-
"@typescript-eslint/eslint-plugin": "^8.17.0",
87-
"@typescript-eslint/parser": "^8.17.0",
88-
"@ungap/structured-clone": "^1.2.0",
85+
"@types/vscode": "~1.96.0",
86+
"@typescript-eslint/eslint-plugin": "^8.20.0",
87+
"@typescript-eslint/parser": "^8.20.0",
88+
"@ungap/structured-clone": "^1.2.1",
8989
"@vscode/debugprotocol": "^1.68.0",
9090
"@vscode/test-electron": "^2.4.1",
9191
"eslint": "^8.57.0",
9292
"eslint-plugin-header": "^3.1.1",
93-
"glob": "^11.0.0",
93+
"glob": "^11.0.1",
9494
"mocha": "^10.8.2",
9595
"mocha-explorer-launcher-scripts": "^0.4.0",
9696
"mocha-multi-reporters": "^1.5.1",
9797
"mock-fs": "^5.4.1",
9898
"rewire": "^7.0.0",
9999
"sinon": "^18.0.1",
100100
"source-map-support": "^0.5.21",
101-
"typescript": "^5.7.2"
101+
"typescript": "^5.7.3"
102102
},
103103
"extensionDependencies": [
104104
"vscode.powershell"

‎src/features/GenerateBugReport.ts

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,16 @@ export class GenerateBugReportFeature implements vscode.Disposable {
1717
"labels=Issue-Bug",
1818
"template=bug-report.yml",
1919
"powershell-version=" + this.getRuntimeInfo(),
20-
"vscode-version=" + vscode.version + "\n" + process.arch,
21-
"extension-version=" + sessionManager.Publisher + "." + sessionManager.HostName + "@" + sessionManager.HostVersion,
20+
"vscode-version=" +
21+
vscode.version +
22+
"\n" +
23+
(process.arch as string),
24+
"extension-version=" +
25+
sessionManager.Publisher +
26+
"." +
27+
sessionManager.HostName +
28+
"@" +
29+
sessionManager.HostVersion,
2230
];
2331
const url = vscode.Uri.parse(issuesUrl + encodeURIComponent(params.join("&")));
2432
await vscode.env.openExternal(url);

‎src/features/OpenInISE.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ export class OpenInISEFeature implements vscode.Disposable {
1616

1717
const document = editor.document;
1818
const uri = document.uri;
19-
let ISEPath = process.env.windir ?? "C:\\Windows";
19+
let ISEPath: string = process.env.windir ?? "C:\\Windows";
2020

2121
if (process.env.PROCESSOR_ARCHITEW6432 !== undefined) {
2222
ISEPath += "\\Sysnative";

‎test/core/platform.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,10 +79,10 @@ let additionalPowerShellExes: Record<string, string>;
7979
let successAdditionalTestCases: ITestPlatformSuccessCase[];
8080

8181
if (process.platform === "win32") {
82-
const msixAppDir = path.join(process.env.LOCALAPPDATA!, "Microsoft", "WindowsApps");
82+
const msixAppDir = path.join(process.env.LOCALAPPDATA, "Microsoft", "WindowsApps");
8383
const pwshMsixPath = path.join(msixAppDir, "Microsoft.PowerShell_8wekyb3d8bbwe", "pwsh.exe");
8484
const pwshPreviewMsixPath = path.join(msixAppDir, "Microsoft.PowerShellPreview_8wekyb3d8bbwe", "pwsh.exe");
85-
const pwshDailyDir = path.join(process.env.LOCALAPPDATA!, "Microsoft", "powershell-daily");
85+
const pwshDailyDir = path.join(process.env.LOCALAPPDATA, "Microsoft", "powershell-daily");
8686
const pwshDailyPath = path.join(pwshDailyDir, "pwsh.exe");
8787

8888
successTestCases = [

‎test/runTestsInner.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ function runTestsInner(testsRoot: string): Promise<void> {
1919
console.log(`\n\n=====\nTest Runner START\n${testsRoot}\n=====`);
2020

2121
/** Passed from RunTests */
22-
const rootDir = process.env.__TEST_EXTENSION_DEVELOPMENT_PATH;
22+
const rootDir: string = process.env.__TEST_EXTENSION_DEVELOPMENT_PATH;
2323
if (!rootDir) {
2424
throw new Error("Missing environment variable __TEST_EXTENSIONDEVELOPMENTPATH, this is probably a bug in runTests.ts");
2525
}

‎tsconfig.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
"ES2022",
99
"DOM"
1010
],
11+
"types": [ "node" ],
1112
"sourceMap": true,
1213
"rootDir": ".",
1314
"strict": true,

0 commit comments

Comments
 (0)
Please sign in to comment.