Skip to content

chore: Update Core for fake Block Store support - #324

Merged
tevincent merged 2 commits into
mainfrom
fake-block-store
Oct 6, 2026
Merged

tevincent merged 2 commits into
mainfrom
fake-block-store

Conversation

@LouisCAD

@LouisCAD LouisCAD commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@LouisCAD
LouisCAD requested review from tevincent and a balanced review from Copilot October 6, 2026 11:43
@tevincent
tevincent enabled auto-merge October 6, 2026 11:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The integration is coherent and the only finding is a non-blocking stale comment.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
What changed in this PR

Adds fake Block Store support for debuggable builds while preserving Play Services behavior in production.

Changes:

  • Updates the Core submodule with the Block Store abstraction.
  • Migrates passkey backup to Core’s Block Store API.
  • Passes backup output for fake-store persistence.
File Description
Core Advances Core to fake Block Store support.
BlockStoreBackup.kt Uses Core’s Block Store abstraction.
AuthenticatorFullBackupAgent.kt Supplies backup output to Core.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread app/src/standard/kotlin/com/infomaniak/auth/backup/BlockStoreBackup.kt Outdated
@sonarqubecloud

sonarqubecloud Bot commented Oct 6, 2026

Copy link
Copy Markdown

@tevincent
tevincent merged commit 5baa1e6 into main Oct 6, 2026
12 checks passed
@tevincent
tevincent deleted the fake-block-store branch October 6, 2026 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants