Skip to content

Fix Link wallet disconnect recovery - #158

Merged
fmhall merged 1 commit into
mainfrom
mason/fix-link-wallet-disconnect
Oct 2, 2026
Merged

fmhall merged 1 commit into
mainfrom
mason/fix-link-wallet-disconnect

Conversation

@fmhall

@fmhall fmhall commented Oct 2, 2026

Copy link
Copy Markdown
Member

Link disconnect requires a recent sign-in. Older valid sessions received a generic connection error, and the sign-in page redirected authenticated users home, preventing recovery.

This adds a phone reauthentication flow that returns to the wallet page, separates disconnect failures from connection failures, and shows recovery messages based on the current wallet state. The existing ownership, session freshness, and revoke-before-delete checks remain in place.

Validation:

  • pnpm check — lint, types, formatting, Knip, and 788 tests.
  • pnpm build — production build with local placeholder database and Kernel values.
  • Regression coverage exercises stale-session rejection, a newly issued phone session, failed revocation retaining the account, successful disconnect, and stale errors after account removal.

The integration uses the real Better Auth Link plugin and PGlite with mocked provider HTTP. Live Link revocation and phone OTP delivery were not exercised.

@fmhall
fmhall merged commit 2e0c289 into main Oct 2, 2026
2 checks passed

This branch was successfully deployed

1 active deployment
Preview — c1240892 Deployed Oct 2, 2026 by vercel[bot]
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