feat: remove getDeviceList

packages/core/src/LocalUser/Manager.ts

@@ -37,7 +37,6 @@ import type { UserData, DelegationToken } from './UserData';
 import type { Client, PullOptions } from '../Network/Client';
 import { OidcNonceManager } from '../OidcNonce/Manager';
 import { Status } from '../Session/status';
-import type { Device } from '../Users/types';
 import { makeSessionCertificate } from './SessionCertificate';
 
 export type PrivateProvisionalKeys = {
@@ -279,12 +278,6 @@ export class LocalUserManager extends EventEmitter {
     }
   };
 
-  listDevices = async (): Promise<Array<Device>> => {
-    await this.updateLocalUser({ isLight: false });
-    const devices = this._localUser.devices;
-    return devices.filter(d => !d.isGhostDevice);
-  };
-
   getSessionToken = async (verification: VerificationWithToken): Promise<string> => {
     await this.updateLocalUser({ isLight: true });
 

packages/core/src/Session/Session.ts

@@ -159,7 +159,6 @@ export class Session extends EventEmitter {
   createNewDevice = this._promiseChain(this._forward(this._getLocalUserManager, 'createNewDevice'), this._setReady, this._stopIfNotRetryable);
 
   getVerificationKey = this._forward(this._getLocalUserManager, 'getVerificationKey');
-  listDevices = this._forward(this._getLocalUserManager, 'listDevices');
   deviceId = () => this._localUserManager.localUser.deviceId;
 
   setVerificationMethod = this._forward(this._getLocalUserManager, 'setVerificationMethod');

packages/core/src/Tanker.ts

@@ -62,7 +62,6 @@ export type TankerCoreOptions = {
 
 export type TankerOptions = Partial<Omit<TankerCoreOptions, 'dataStore'> & { dataStore: Partial<DataStoreOptions>; }>;
 
-export type Device = { id: string; };
 export type ProvisionalVerification = EmailVerification | PhoneNumberVerification;
 
 export function optionsWithDefaults(options: TankerOptions, defaults: TankerCoreOptions): TankerCoreOptions {
@@ -439,17 +438,6 @@ export class Tanker extends EventEmitter {
     }
   }
 
-  async getDeviceList(): Promise<Array<Device>> {
-    console.warn('The "getDeviceList" method is deprecated, it will be removed in the future');
-
-    assertStatus(this.status, statuses.READY, 'get the device list');
-
-    const devices = await this.session.listDevices();
-    return devices.map(d => ({
-      id: utils.toBase64(d.deviceId),
-    }));
-  }
-
   async share(resourceIds: Array<b64string>, options: SharingOptions): Promise<void> {
     assertStatus(this.status, statuses.READY, 'share');
 

packages/core/src/index.ts

@@ -52,7 +52,7 @@ export type {
   LegacyEmailVerificationMethod,
 } from './LocalUser/types';
 export type { AttachResult } from './ProvisionalIdentity/types';
-export type { TankerOptions, Device, ProvisionalVerification } from './Tanker';
+export type { TankerOptions, ProvisionalVerification } from './Tanker';
 export type { EncryptionSession } from './DataProtection/EncryptionSession';
 export type { UploadStream } from './CloudStorage/UploadStream';
 export type { DownloadStream } from './CloudStorage/DownloadStream';

packages/functional-tests/src/index.ts

@@ -11,7 +11,6 @@ import { generateEncryptionTests } from './encryption';
 import { generateEnrollTests } from './enroll';
 import { generateFakeAuthenticationTests } from './fake-authentication';
 import { generateNetworkTests } from './network';
-import { generateGetDeviceListTests } from './getDeviceList';
 import { generateGroupsTests } from './groups';
 import { generateSessionTests } from './session';
 import { generateUploadTests } from './upload';
@@ -64,7 +63,6 @@ export function generateFunctionalTests(
     });
 
     generateSessionTests(args);
-    generateGetDeviceListTests(args);
     generateVerificationTests(args);
     generateEncryptionTests(args);
     generateEncryptionSessionTests(args);

packages/functional-tests/src/session.ts

@@ -179,11 +179,6 @@ export const generateSessionTests = (args: TestArgs) => {
         await expect(bobLaptop.status).to.equal(IDENTITY_VERIFICATION_NEEDED);
 
         await bobLaptop.verifyIdentity({ passphrase: 'passphrase' });
-
-        // Check two devices have been created
-        const devices = await bobLaptop.getDeviceList();
-        expect(devices).to.have.lengthOf(2);
-        expect(devices).to.deep.include.members([{ id: bobLaptop.deviceId }]);
       });
     });
 
@@ -215,11 +210,6 @@ export const generateSessionTests = (args: TestArgs) => {
         await expect(bobLaptop.status).to.equal(IDENTITY_VERIFICATION_NEEDED);
 
         await bobLaptop.verifyIdentity({ passphrase: 'passphrase' });
-
-        // Check two devices have been created
-        const devices = await bobLaptop.getDeviceList();
-        expect(devices).to.have.lengthOf(3);
-        expect(devices).to.deep.include.members([{ id: bobLaptop.deviceId }]);
       });
     });
   });