Skip to content

fix: remediate end-to-end app reliability issues - #23

Draft
AnganSamadder wants to merge 16 commits into
mainfrom
fix/e2e-remediation
Draft

fix: remediate end-to-end app reliability issues#23
AnganSamadder wants to merge 16 commits into
mainfrom
fix/e2e-remediation

Conversation

@AnganSamadder

Copy link
Copy Markdown
Owner

Summary

  • harden account deletion so retries are safe, shared history is preserved, and realtime/auth teardown cannot resurrect stale state
  • validate link-request provenance and keep friend discovery, requests, and synchronization private and retry-safe
  • persist expense notes across local and Convex flows
  • fix add-expense alert presentation, settlement-share totals, navigation resets, destructive debug-data confirmation, friend-name editing, and the inactive receipt-camera control
  • add and repair focused backend/iOS regression coverage for each behavior

Why

End-to-end testing exposed several independent reliability problems around identity state, account switching, destructive cleanup, and transient UI state. The main root causes were client-trusted link targets, non-idempotent account teardown, stale realtime data crossing session boundaries, total-vs-current-user settlement calculations, and UI state surviving account changes.

Impact

These changes reduce cross-account data leakage, make destructive and retryable flows deterministic, preserve shared financial history during account deletion, and eliminate the tested UI glitches without changing the intended product model.

Validation

  • FAIL_ON_WARNINGS=1 ./scripts/test-ci-locally.sh
    • backend: 63/63
    • iOS: 2,581/2,581
    • web smoke: passed
    • overall coverage: 48.97% (CI threshold: 48%)
    • warnings: 0
  • SANITIZER=thread FAIL_ON_WARNINGS=1 ./scripts/test-ci-locally.sh
    • iOS: 2,581/2,581
    • Thread Sanitizer findings: 0
  • SANITIZER=address FAIL_ON_WARNINGS=1 ./scripts/test-ci-locally.sh
    • iOS: 2,581/2,581
    • Address Sanitizer findings: 0

Review

Independent backend/security, iOS/lifecycle, and PR-architecture reviews were completed. All actionable findings were resolved before publication.

@vercel

vercel Bot commented Jul 31, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
payback Ready Ready Preview Jul 31, 2026 3:04am

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