Skip to content

fix(walletconnect): repair screen scan permissions and vault links#145

Open
BitHighlander wants to merge 1 commit intodevelopfrom
fix/walletconnect-permissions-and-links
Open

fix(walletconnect): repair screen scan permissions and vault links#145
BitHighlander wants to merge 1 commit intodevelopfrom
fix/walletconnect-permissions-and-links

Conversation

@BitHighlander
Copy link
Copy Markdown
Collaborator

Summary

  • stop treating CoreGraphics Screen Recording preflight as a hard gate for WalletConnect screen QR scan
  • let screencapture attempt the capture after requesting TCC access, then only surface the permission error when the capture actually fails like a permission problem
  • locate the real enclosing .app bundle before re-registering macOS URL schemes, so Vault can override old KeepKey Desktop reliably
  • rewrite proxied WalletConnect launch links from keepkey:// to keepkey-vault:// and relabel legacy "Launch Desktop" copy

Verification

  • git diff --check -- projects/keepkey-vault/src/bun/index.ts projects/keepkey-vault/src/bun/rest-api.ts
  • cd projects/keepkey-vault && npx tsc --noEmit --skipLibCheck still hits the existing missing minimatch type-definition issue
  • filtered preflight-style check reports 0 non-minimatch TypeScript errors

@BitHighlander BitHighlander requested a review from pastaghost as a code owner May 1, 2026 02:32
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