Make added, removed, changed, optional in auth session change event, f…

…ixes #117469
microsoft · Feb 24, 2021 · 7f481d2 · 7f481d2
1 parent 2852916
commit 7f481d2
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/src/vs/vscode.d.ts b/src/vs/vscode.d.ts
@@ -12349,17 +12349,17 @@ declare module 'vscode' {
  /**
  * The [AuthenticationSession](#AuthenticationSession)s of the [AuthenticationProvider](#AuthentiationProvider) that have been added.
  */
- readonly added: ReadonlyArray<AuthenticationSession>;
+ readonly added?: ReadonlyArray<AuthenticationSession>;
 
  /**
  * The [AuthenticationSession](#AuthenticationSession)s of the [AuthenticationProvider](#AuthentiationProvider) that have been removed.
  */
- readonly removed: ReadonlyArray<AuthenticationSession>;
+ readonly removed?: ReadonlyArray<AuthenticationSession>;
 
  /**
  * The [AuthenticationSession](#AuthenticationSession)s of the [AuthenticationProvider](#AuthentiationProvider) that have been changed.
  */
- readonly changed: ReadonlyArray<AuthenticationSession>;
+ readonly changed?: ReadonlyArray<AuthenticationSession>;
  }
 
  /**

diff --git a/src/vs/workbench/api/common/extHostAuthentication.ts b/src/vs/workbench/api/common/extHostAuthentication.ts
@@ -111,7 +111,11 @@ export class ExtHostAuthentication implements ExtHostAuthenticationShape {
  }
 
  const listener = provider.onDidChangeSessions(e => {
- this._proxy.$sendDidChangeSessions(id, e);
+ this._proxy.$sendDidChangeSessions(id, {
+ added: e.added ?? [],
+ changed: e.changed ?? [],
+ removed: e.changed ?? []
+ });
  });
 
  this._proxy.$registerAuthenticationProvider(id, label, options?.supportsMultipleAccounts ?? false);