Skip to content

feat(wallet): add typed Stellar signing pipeline (#758) - #854

Closed
edehvictor wants to merge 1 commit into
accesslayerorg:mainfrom
edehvictor:fix/signing-pipeline-758
Closed

feat(wallet): add typed Stellar signing pipeline (#758)#854
edehvictor wants to merge 1 commit into
accesslayerorg:mainfrom
edehvictor:fix/signing-pipeline-758

Conversation

@edehvictor

Copy link
Copy Markdown
Contributor

Closes #758

Changes

  • add a common Signer interface with production Freighter and session-reused Ledger WebHID implementations
  • verify wallet network, perform Ledger health checks, enforce the device transaction-size limit, and classify extension/APDU/WebHID failures into actionable typed errors
  • retry only Ledger timeouts using 2s, 4s, and 8s backoff while immediately surfacing locked-device, app-not-open, and user-rejection failures
  • add deterministic signature aggregation that rejects mismatched transaction payloads
  • expose a useSigner hook that detects available signers, respects the persisted wallet preference, and safely falls back when the preferred signer is unavailable
  • add an accessible three-step signing progress component with inline recovery guidance
  • add the official Freighter, Stellar SDK, and Ledger WebHID dependencies with a minimal lockfile update

Testing

  • vitest run src/lib/signing/__tests__ src/hooks/__tests__/useSigner.test.tsx — 20 tests passed
  • tsc -b --pretty false — passed
  • targeted ESLint for all changed TypeScript/TSX files — passed
  • git diff --check — passed

Notes

  • Ledger behavior is exercised through deterministic error/retry tests; CI does not require a physical WebHID device.

@netlify

netlify Bot commented Aug 30, 2026

Copy link
Copy Markdown

Deploy Preview for accesslayer ready!

Name Link
🔨 Latest commit 92b9c7c
🔍 Latest deploy log https://app.netlify.com/projects/accesslayer/deploys/6a93ff048bf4c30008a15c1a
😎 Deploy Preview https://deploy-preview-854--accesslayer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions

Copy link
Copy Markdown

👋 Thanks for your contribution! We don't accept pull requests directly to main. Please re-open your PR targeting the dev branch instead. See our contributing guide for details.

@github-actions github-actions Bot closed this Aug 30, 2026
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