Skip to content

fix(mobile): blank Recents only on the profile tab - #5532

Merged
iscekic merged 2 commits into
mainfrom
fix/privacy-cover-profile-only
Aug 27, 2026
Merged

fix(mobile): blank Recents only on the profile tab#5532
iscekic merged 2 commits into
mainfrom
fix/privacy-cover-profile-only

Conversation

@iscekic

@iscekic iscekic commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

What

The privacy cover blanked the OS app-switcher snapshot on login, agent-chat, pr-review, kilo-pass, device-sessions, share-gate and the whole profile tab. That is most of the app, so Recents looked permanently blank.

Cover the profile tab only. It is the one surface that shows the credit balance and account identity.

Changes

  • isPrivacyCoverRoute() is now a single token match on (3_profile).
  • The home and KiloClaw exclusions are dead branches under the new rule, so they are removed.
  • screenLayout={privacyScreenLayout} is removed from the two navigators that no longer have a covered route: (app) and pr-review/[owner]/[repo]/[number]. The profile-tab navigators keep it.
  • Android FLAG_SECURE now follows the same narrowed rule; iOS never called it.

Verification

pnpm format && pnpm typecheck && pnpm lint && pnpm check:unused && pnpm test pass in apps/mobile.

The privacy cover blanked the app switcher snapshot on login, agent-chat,
pr-review, kilo-pass, device-sessions and share-gate. Those surfaces hold
nothing worth a blank Recents card, and the cover fired often enough that
the app looked permanently blank in the switcher.

Cover the profile tab only. It is the one surface that shows the credit
balance and account identity. The home and KiloClaw exclusions become
dead branches under the new rule, so they go with it, along with the
`screenLayout` wiring on the two navigators that no longer have a covered
route.
@iscekic iscekic self-assigned this Aug 27, 2026
@kilo-code-bot

kilo-code-bot Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (6 files)
  • apps/mobile/src/lib/privacy-cover.ts
  • apps/mobile/src/lib/privacy-cover.test.ts
  • apps/mobile/src/components/privacy-cover-overlay.tsx
  • apps/mobile/src/components/privacy-cover-overlay.mounted.test.tsx
  • apps/mobile/src/app/(app)/_layout.tsx
  • apps/mobile/src/app/(app)/pr-review/[owner]/[repo]/[number]/_layout.tsx

Reviewed by grok-4.6 · Input: 164.4K · Output: 7.8K · Cached: 158K

Review guidance: REVIEW.md from base branch main

@iscekic iscekic added the human-ready The PR is ready for human review. label Aug 27, 2026
@iscekic
iscekic enabled auto-merge (squash) August 27, 2026 12:39
@iscekic
iscekic merged commit cfab6dc into main Aug 27, 2026
17 checks passed
@iscekic
iscekic deleted the fix/privacy-cover-profile-only branch August 27, 2026 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

human-ready The PR is ready for human review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants