Skip to content

Release 108: Mobile work in progress - #641

Draft
alichherawalla wants to merge 11 commits into
mainfrom
release/v108
Draft

Release 108: Mobile work in progress#641
alichherawalla wants to merge 11 commits into
mainfrom
release/v108

Conversation

@alichherawalla

Copy link
Copy Markdown
Collaborator

WIP. Do not merge yet.

Summary:

  • Move passphrase and lock state to one security owner.
  • Bind chat reply state and model adapters to the application-root lifetime.
  • Show shared chat refusal and interrupted-reply state.
  • Trust canonical model-transfer file roles.
  • Add rendered restart, companion-control, and sharing-stop journey coverage.

Verification:

  • The normal Mobile pre-push suite failed: 5 suites and 9 tests failed, with an additional teardown error.
  • The hook was bypassed only to publish this WIP branch. This PR is not verified or merge-ready.

Moves the phone sharing package to 192693de, where pressing Stop mid-transfer
ends the share and the stopped entry no longer shows a live percentage.
Turning the lock on, changing it, turning it off, unlocking and opening the app
all went through their own code, so the lock could disagree with the stored
passphrase. Every one of those now sends its intent to the one shared owner.

The second lock store and the old lock service are gone. The keychain and the
lock setting are each behind a small port that decides nothing, and the tests
reach the lock through that one owner.
@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sonarqubecloud

sonarqubecloud Bot commented Sep 8, 2026

Copy link
Copy Markdown

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.

1 participant