Wrap wpc.getAllProcesses in a Promise

Replace use of util.promisify with an explicit Promise wrapper around wpc.getAllProcesses, removing the promisify import and adapting the call site to resolve with the process list. Update unit tests to match the changed callback shape (stubs now call callback(processList) and throw without an error-first parameter). This keeps Windows process enumeration working while aligning with the windows-process-tree callback behavior.

Author: ZA139

src/extension/debugger/attachQuickPick/provider.ts

@@ -3,7 +3,6 @@
 
 'use strict';
 
-import { promisify } from 'util';
 import { l10n } from 'vscode';
 import * as wpc from '@vscode/windows-process-tree';
 import { getOSType, OSType } from '../../common/platform';
@@ -64,8 +63,9 @@ export class AttachProcessProvider implements IAttachProcessProvider {
 
         if (osType === OSType.Windows) {
             try {
-                const getAllProcesses = promisify(wpc.getAllProcesses);
-                const processList = await getAllProcesses(wpc.ProcessDataFlag.CommandLine);
+                const processList = await new Promise<wpc.IProcessInfo[]>((resolve) => {
+                    wpc.getAllProcesses((processes) => resolve(processes), wpc.ProcessDataFlag.CommandLine);
+                });
                 return processList.map((p) => ({
                     label: p.name,
                     description: String(p.pid),

src/test/unittest/attachQuickPick/provider.unit.test.ts

@@ -169,7 +169,7 @@ suite('Attach to process - process provider', () => {
             },
         ];
         getOSTypeStub.returns(platform.OSType.Windows);
-        getAllProcessesStub.callsFake((_flag: any,callback: Function) => callback(null,processList));
+        getAllProcessesStub.callsFake((callback: Function) => callback(processList));
 
         const attachItems = await provider._getInternalProcessEntries();
         sinon.assert.notCalled(plainExecStub);
@@ -330,7 +330,7 @@ suite('Attach to process - process provider', () => {
                 },
             ];
 
-            getAllProcessesStub.callsFake((_flag: any,callback: Function) => callback(null,processList));
+            getAllProcessesStub.callsFake((callback: Function) => callback(processList));
 
             const output = await provider.getAttachItems();
 
@@ -416,7 +416,7 @@ suite('Attach to process - process provider', () => {
                 },
             ];
 
-            getAllProcessesStub.callsFake((_flag: any,callback: Function) => callback(null,processList));
+            getAllProcessesStub.callsFake((callback: Function) => callback(processList));
 
             const output = await provider.getAttachItems();
 
@@ -430,7 +430,7 @@ suite('Attach to process - process provider', () => {
         });
 
         test('Should fall back to wmic when getAllProcesses fails', async () => {
-            getAllProcessesStub.callsFake((_flag: any,_callback: Function) => {
+            getAllProcessesStub.callsFake((_callback: Function) => {
                 throw new Error('windows-process-tree unavailable');
             });
             const windowsOutput = `CommandLine=\r
@@ -484,7 +484,7 @@ ProcessId=5912\r
         });
 
         test('Items returned by getAttachItems via wmic fallback should be sorted alphabetically', async () => {
-            getAllProcessesStub.callsFake((_flag: any,_callback: Function) => {
+            getAllProcessesStub.callsFake((_callback: Function) => {
                 throw new Error('windows-process-tree unavailable');
             });
             const windowsOutput = `CommandLine=\r
@@ -538,7 +538,7 @@ ProcessId=5728\r
         });
 
         test('Python processes should be at the top of the list returned by getAttachItems via wmic fallback', async () => {
-            getAllProcessesStub.callsFake((_flag: any,_callback: Function) => {
+            getAllProcessesStub.callsFake((_callback: Function) => {
                 throw new Error('windows-process-tree unavailable');
             });
             const windowsOutput = `CommandLine=\r