rm-hull / zaup2 / 19000962293

2%

push

github

refactor: Use standard `use-local-storage` hook (#930)

* Refactor: Use standard `use-local-storage` hook

This commit replaces the custom `useLocalStorage` implementation
with the `@rm-hull/use-local-storage` package.

This migration allows for better management of serialization logic
directly within the hook implementation.

Key changes:

*   The `useGeneralSettings` hook now returns an object
    `{ settings, updateSettings }` instead of an array.
*   The `useOtpParameters` hook utilizes the library's `Serializer`
    interface to handle password-based encryption and decryption
    of stored OTP data.
*   Components consuming these hooks have been updated to use
    object destructuring instead of array destructuring.

* Bump up the jam

* Smorgasborg(?) of changes

* fix: Throw error on failed OTP decryption

Ensures that if `CryptoJS.AES.decrypt` fails (e.g., due to a bad
password), we throw an explicit error message rather than relying
on subsequent JSON parsing failure.

* feat: Add loading state and global toaster

Exposes the `isLoading` status from `useLocalStorage` via
`useOtpParameters` to allow consumer components to handle
initial data hydration states properly.

Also integrates the global `Toaster` component for displaying
notifications across the application.

* refactor: Extract serializer classes

Extract `Encrypter` logic into `CryptoJsSerializer` to clean up the
`useOtpParameters` hook.

This commit also introduces the `WebCryptoSerializer` utility which
provides functions to decrypt data originally encrypted by CryptoJS
using the browser's native `window.crypto` methods (AES-CBC +
EVP_BytesToKey derivation).

This prepares the codebase for potentially migrating away from
`crypto-js`.

* imports

* feat: Handle OTP loading and decryption errors

Introduces robust error handling for data retrieval and decryption:

*   Displays an error toast notification if the OTP data fails to load
    in the UI (`Group.tsx`).
*   Catches decrypti... (continued)

28 of 961 branches covered (2.91%)

Branch coverage included in aggregate %.

98 of 4198 relevant lines covered (2.33%)

0.12 hits per line