@metamask-previews/seedless-onboarding-controller 7.1.0-preview-8edbf7c9d → 7.1.0-preview-c97d5b99

package/CHANGELOG.md

@@ -9,6 +9,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Added
 
+- Add new `SeedlessOnboardingError` class for generic controller errors with support for `cause` and `details` properties ([#7660](https://github.com/MetaMask/core/pull/7660))
+  - Enables proper error chaining by wrapping underlying errors with additional context
+  - Includes `toJSON()` method for serialization in logging/transmission
+
+### Added
+
 - **BREAKING** The `encryptor` constructor param requires `encryptWithKey` method. ([#7800](https://github.com/MetaMask/core/pull/7800))
   - The method is to encrypt the vault with cached encryption key while the wallet is unlocked.
 - Added new public method, `getAccessToken`. ([#7800](https://github.com/MetaMask/core/pull/7800))
@@ -21,6 +27,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   - Exclude All the tokens values from the state log explicitly.
 - Bump `@metamask/keyring-controller` from `^25.0.0` to `^25.1.0` ([#7713](https://github.com/MetaMask/core/pull/7713))
 - Upgrade `@metamask/utils` from `^11.8.1` to `^11.9.0` ([#7511](https://github.com/MetaMask/core/pull/7511))
+- Refactor controller methods to throw `SeedlessOnboardingError` with original error as `cause` for better error tracing ([#7660](https://github.com/MetaMask/core/pull/7660))
+  - Affected methods: `authenticate`, `changePassword`, `#persistLocalEncryptionKey`, `#fetchAndParseSecretMetadata`, `refreshAuthTokens`
 
 ### Fixed
 

package/dist/SeedlessOnboardingController.cjs

@@ -326,7 +326,9 @@ class SeedlessOnboardingController extends base_controller_1.BaseController {
             }
             catch (error) {
                 log('Error authenticating user', error);
-                throw new Error(constants_1.SeedlessOnboardingControllerErrorMessage.AuthenticationError);
+                throw new errors_1.SeedlessOnboardingError(constants_1.SeedlessOnboardingControllerErrorMessage.AuthenticationError, {
+                    cause: error,
+                });
             }
         };
         return params.skipLock
@@ -502,7 +504,9 @@ class SeedlessOnboardingController extends base_controller_1.BaseController {
             }
             catch (error) {
                 log('Error changing password', error);
-                throw new Error(constants_1.SeedlessOnboardingControllerErrorMessage.FailedToChangePassword);
+                throw new errors_1.SeedlessOnboardingError(constants_1.SeedlessOnboardingControllerErrorMessage.FailedToChangePassword, {
+                    cause: error,
+                });
             }
         });
     }
@@ -691,7 +695,9 @@ class SeedlessOnboardingController extends base_controller_1.BaseController {
                 })
                     .catch((error) => {
                     log('Error fetching auth pub key', error);
-                    throw new Error(constants_1.SeedlessOnboardingControllerErrorMessage.FailedToFetchAuthPubKey);
+                    throw new errors_1.SeedlessOnboardingError(constants_1.SeedlessOnboardingControllerErrorMessage.FailedToFetchAuthPubKey, {
+                        cause: error,
+                    });
                 });
                 globalAuthPubKey = authPubKey;
             }
@@ -772,7 +778,9 @@ class SeedlessOnboardingController extends base_controller_1.BaseController {
             refreshToken,
         }).catch((error) => {
             log('Error refreshing JWT tokens', error);
-            throw new Error(constants_1.SeedlessOnboardingControllerErrorMessage.FailedToRefreshJWTTokens);
+            throw new errors_1.SeedlessOnboardingError(constants_1.SeedlessOnboardingControllerErrorMessage.FailedToRefreshJWTTokens, {
+                cause: error,
+            });
         });
         try {
             const { idTokens, accessToken, metadataAccessToken } = res;
@@ -804,7 +812,9 @@ class SeedlessOnboardingController extends base_controller_1.BaseController {
         }
         catch (error) {
             log('Error refreshing node auth tokens', error);
-            throw new Error(constants_1.SeedlessOnboardingControllerErrorMessage.AuthenticationError);
+            throw new errors_1.SeedlessOnboardingError(constants_1.SeedlessOnboardingControllerErrorMessage.AuthenticationError, {
+                cause: error,
+            });
         }
     }
     /**
@@ -1046,7 +1056,9 @@ async function _SeedlessOnboardingController_persistOprfKey(oprfKey, authPubKey)
             throw error;
         }
         log('Error persisting local encryption key', error);
-        throw new Error(constants_1.SeedlessOnboardingControllerErrorMessage.FailedToPersistOprfKey);
+        throw new errors_1.SeedlessOnboardingError(constants_1.SeedlessOnboardingControllerErrorMessage.FailedToPersistOprfKey, {
+            cause: error,
+        });
     }
 }, _SeedlessOnboardingController_persistAuthPubKey = function _SeedlessOnboardingController_persistAuthPubKey(params) {
     this.update((state) => {
@@ -1141,7 +1153,9 @@ async function _SeedlessOnboardingController_recoverEncKey(password) {
         if (__classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_isAuthTokenError).call(this, error)) {
             throw error;
         }
-        throw new Error(constants_1.SeedlessOnboardingControllerErrorMessage.FailedToFetchSecretMetadata);
+        throw new errors_1.SeedlessOnboardingError(constants_1.SeedlessOnboardingControllerErrorMessage.FailedToFetchSecretMetadata, {
+            cause: error,
+        });
     }
     // user must have at least one secret data
     if (secretData?.length > 0) {
@@ -1590,7 +1604,9 @@ async function _SeedlessOnboardingController_assertPasswordInSync(options) {
     })
         .catch((error) => {
         log('Error fetching auth pub key', error);
-        throw new Error(constants_1.SeedlessOnboardingControllerErrorMessage.FailedToFetchAuthPubKey);
+        throw new errors_1.SeedlessOnboardingError(constants_1.SeedlessOnboardingControllerErrorMessage.FailedToFetchAuthPubKey, {
+            cause: error,
+        });
     });
     const isPasswordOutdated = await this.checkIsPasswordOutdated({
         ...options,