This adopts the `NativeBrokerPlugin` provided by `@azure/msal-node-extensions` to provide the ability to use auth state from the OS, and show native auth dialogs instead of going to the browser.
This has several pieces:
* The adoption of the broker in the microsoft-authentication extension:
* Adding `NativeBrokerPlugin` to our PCAs
* Using the proposed handle API to pass the native window handle down to MSAL calls (btw, this API will change in a follow up PR)
* Adopting an AccountAccess layer to handle:
* giving the user control of which accounts VS Code uses
* an eventing layer so that auth state can be updated across multiple windows
* Getting the extension to build properly and only build what it really needs. This required several package.json/webpack hacks:
* Use a fake keytar since we don't use the feature in `@azure/msal-node-extensions` that uses keytar
* Use a fake dpapi layer since we don't use the feature in `@azure/msal-node-extensions` that uses it
* Ensure the msal runtime `.node` and `.dll` files are included in the bundle
* Get the VS Code build to allow a native node module in an extension: by having a list of native extensions that will be built in the "ci" part of the build - in other words when VS Code is building on the target platform
There are a couple of followups:
* Refactor the `handle` API to handle (heh) Auxiliary Windows https://github.com/microsoft/vscode/issues/233106
* Separate the call to `acquireTokenSilent` and `acquireTokenInteractive` and all the usage of this native node module into a separate process or maybe in Core... we'll see. Something to experiment with after we have something working. NEEDS FOLLOW UP ISSUE
Fixes https://github.com/microsoft/vscode/issues/229431
* Simplify distro
Also a ton of drive-by fixing around builds:
- simplified many oneliners
- fixed missing custom npm registry call setups
- remove unnecessary and duplicate work during builds
- many many fixes
* Build VS Code using TS 4.4
* Remove usages of deprecated `ClientRectList`
* Add any casts for missing `caretRangeFromPoint`
* Add temporary any casts for `zoom` css propery
This non-standard css property no longer exists in lib.dom.d.ts
* MouseWheelEvent -> WheelEvent
* Pick up new TS nightly
Co-authored-by: Alexandru Dima <alexdima@microsoft.com>
* ci: [mac] Unify tasks between different flavors
* take 3
* remove zip
* unzip in place
* logging
* log
* rm zip
* all the things
* remove thing to trigger another build
* publish server for x64 only and only start on non-arm
* delete commented out code
* remove unused sh script
* address main changes
* address feedback
* add a comment
* try slimming down yarn
* only compile createAsset
* install typescript
* move build to correct location
* change to 64
* add useful comments based on joao
Co-authored-by: deepak1556 <hop2deep@gmail.com>
Tyler James Leonhardt
Robo
Benjamin Pasero
João Moreno
Matt Bierner
Johannes
Alex Dima