Skip to content

feat: add multi-wallet support and Lighthouse CI (#343, #344)#1

Merged
devJaja merged 1 commit into
mainfrom
feat/multi-wallet-and-lighthouse-ci
Jun 1, 2026
Merged

feat: add multi-wallet support and Lighthouse CI (#343, #344)#1
devJaja merged 1 commit into
mainfrom
feat/multi-wallet-and-lighthouse-ci

Conversation

@devJaja
Copy link
Copy Markdown
Owner

@devJaja devJaja commented Apr 29, 2026

Summary

Closes OlaGreat#343 — Multi-wallet support
Closes OlaGreat#344 — Lighthouse CI for frontend performance


OlaGreat#343 — Multiple wallet connections

New file: frontend/src/lib/wallet-adapters.ts

  • WalletAdapter interface with a consistent connect(): Promise<string> API
  • Adapters for Freighter, Albedo (window.albedo), and Lobstr (window.lobstr)
  • getAvailableWallets() detects installed wallets at runtime; graceful fallback when none found

Modified: frontend/src/components/wallet-connect.tsx

  • Dynamic wallet selector — renders a button per detected wallet
  • Shows install prompt when no wallets are available
  • Persists chosen wallet ID + address in localStorage for session restore

Modified: frontend/src/lib/stellar.ts

  • Re-exports wallet utilities so callers can import from a single module

OlaGreat#344 — Lighthouse CI

New file: lighthouserc.js

  • Thresholds: performance ≥ 0.7, LCP ≤ 4s, FCP ≤ 3s, CLS ≤ 0.25, TBT warn at 600ms
  • Uploads results to temporary public storage

Modified: .github/workflows/frontend.yml

  • New lighthouse job runs on PRs only
  • Fails CI on breached error-level assertions
  • Uses LHCI_GITHUB_APP_TOKEN secret for PR comments

Also fixed

  • Pre-existing React hooks-rules violations in profile-card.tsx that were already breaking the build

…eat#344)

- Add wallet-adapters.ts with Freighter, Albedo, Lobstr adapter pattern
- Rewrite wallet-connect.tsx to detect and display available wallets
- Re-export wallet utilities from stellar.ts
- Add lighthouserc.js with LCP/CLS/FID thresholds
- Add Lighthouse CI job to frontend.yml (PR-only)
- Fix pre-existing conditional hook violations in profile-card.tsx
@devJaja devJaja merged commit 9b7de55 into main Jun 1, 2026
2 of 3 checks passed
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.

Add automated Lighthouse CI for frontend performance Add support for multiple wallet connections

1 participant