Skip to content

feat: expose message signing over browser bridge - #139

Merged
0xpantera merged 1 commit into
mainfrom
feature/issue-137-bridge-message-signing
Jun 24, 2026
Merged

feat: expose message signing over browser bridge#139
0xpantera merged 1 commit into
mainfrom
feature/issue-137-bridge-message-signing

Conversation

@0xpantera

Copy link
Copy Markdown
Collaborator

Summary

  • expose personal_sign and eth_signTypedData_v4 through the localhost EIP-1193 browser bridge
  • keep production signing routed through signerd approval/signing, with dev-mock deterministic signatures for QA only
  • explicitly refuse raw eth_sign as not clear-signable
  • add browser extension coverage and WalletBeat signature QA lane

Dependency note

  • Adds alloy-dyn-abi with the eip712 feature to deckard-browser-bridge so the bridge can parse and hash dynamic EIP-712 typed data instead of hand-rolling typed-data encoding.

Test plan

  • cargo fmt --all --check
  • just check
  • cargo test --workspace
  • pnpm run qa:extension
  • pnpm run qa:walletbeat
  • pnpm run qa:walletbeat:signatures

Closes #137

@0xpantera
0xpantera force-pushed the feature/issue-137-bridge-message-signing branch from e10d6b9 to b35f4ec Compare June 24, 2026 11:07
@0xpantera 0xpantera self-assigned this Jun 24, 2026
@0xpantera
0xpantera merged commit 8396fc6 into main Jun 24, 2026
5 checks passed
@0xpantera
0xpantera deleted the feature/issue-137-bridge-message-signing branch June 24, 2026 11:16
hellno added a commit that referenced this pull request Jul 10, 2026
…-bridge doc (#220)

- ADR 0006: accept the shipped 127.0.0.1:8765 loopback bridge as the alpha
  transport (temporary/experimental); native messaging stays the post-audit
  target; priority is the two ends (core + extension). Resolves the #202 decision.
- browser-bridge.md: correct two stale claims — the bridge forwards the full
  EIP-1193 set (personal_sign / eth_signTypedData_v4 / eth_sendTransaction /
  EIP-5792), not 'only 3 methods', and those are no longer 'not implemented'
  (#139/#141/#146/#148, attributed via #198).
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.

Browser bridge: expose reviewed message signing over EIP-1193

1 participant