feat: Add securityScore and lastPasswordUpdate in AccountEntity

[tevincent]

No description provided.

[Copilot]

Pull request overview

This PR extends the account persistence model to store security-related profile metadata (security score + last password update) and adds a refresh pathway to re-sync that metadata from the backend, including a UI-triggered refresh from the account list.

Changes:

• Add securityScore and lastPasswordUpdate fields to AccountEntity and update persistence calls to populate them from UserProfile.preferences.security.
• Add a new AuthenticatorFacade.refreshUserProfileFor(...) API and wire it through AuthenticatorFacadeImpl.
• Add an account-list refresh hook to trigger user-profile syncing from the app UI.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
multiplatform-lib/src/commonMain/kotlin/models/migration/user/preferences/Preferences.kt	Makes Preferences.security non-null (now required) for profile parsing.
multiplatform-lib/src/commonMain/kotlin/internal/db/AccountsDao.kt	Makes getAccountsWith suspend and adds getAccount for one-shot reads.
multiplatform-lib/src/commonMain/kotlin/internal/db/AccountEntity.kt	Adds securityScore and lastPasswordUpdate columns to the entity.
multiplatform-lib/src/commonMain/kotlin/internal/AuthenticatorFacadeImpl.kt	Fetches/stores security fields from user profile; adds syncing helper and refresh entrypoint.
multiplatform-lib/src/commonMain/kotlin/DummyAuthenticatorFacade.kt	Adds stub implementation for the new facade method.
multiplatform-lib/src/commonMain/kotlin/AuthenticatorFacade.kt	Adds new abstract API refreshUserProfileFor(...).
multiplatform-lib/schemas/com.infomaniak.auth.lib.internal.db.AccountsDatabase/1.json	Updates exported Room schema to include new columns.
app/src/main/kotlin/com/infomaniak/auth/ui/screen/accountlist/AccountListViewModel.kt	Adds refreshUserProfiles() and nests AccountListUiState in the VM.
app/src/main/kotlin/com/infomaniak/auth/ui/screen/accountlist/AccountListScreen.kt	Wires pull-to-refresh to also refresh user profiles.

Comments suppressed due to low confidence (1)

multiplatform-lib/src/commonMain/kotlin/internal/AuthenticatorFacadeImpl.kt:346

• After syncAccountWithUserProfile(...) updates the account row with securityScore/lastPasswordUpdate, the subsequent dao.upsert(notConnectedAccount.copy(status = LoggedIn)) can overwrite those new columns with the (likely default) values from notConnectedAccount. Consider including the security fields in that final upsert, or avoid the second upsert if syncAccountWithUserProfile already persisted the needed updates.

                val userProfile = syncAccountWithUserProfile(apiToken.accessToken, userId)
                userProfile.apiToken = apiToken
                authenticatorBridge.persistUserProfile(userProfile)
            },
        )

        if (succeeded.not()) return false

        dao.upsert(notConnectedAccount.copy(status = AccountEntity.Status.LoggedIn))

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
24 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud