Skip to content

Replace local wallet with @autonomys/auto-wallet SDK packages#1

Merged
jim-counter merged 3 commits intomainfrom
claude/nifty-blackwell
Feb 11, 2026
Merged

Replace local wallet with @autonomys/auto-wallet SDK packages#1
jim-counter merged 3 commits intomainfrom
claude/nifty-blackwell

Conversation

@jim-counter
Copy link
Copy Markdown
Member

@jim-counter jim-counter commented Feb 11, 2026

Summary

  • Removes the inline wallet connection implementation (~870 lines across 8 files: components, store, types, hooks, constants)
  • Replaces with @autonomys/auto-wallet and @autonomys/auto-wallet-react packages at ^1.6.9 from npm
  • Bumps @autonomys/auto-consensus and @autonomys/auto-utils from 1.5.14 to ^1.6.9
  • Removes @talismn/connect-wallets dependency (no longer needed)

What changed

The wallet connection logic that was originally built in this app has been extracted into reusable SDK packages (auto-sdk PR #511). This PR swaps the local code for the published packages, which include additional improvements from code review:

  • Connection sequence tracking to prevent race conditions
  • Dark mode theming support (CSS variable-based)
  • Graceful handling of missing wallet extensions
  • Clean state reset on reconnection failure

Test plan

  • yarn type-check passes
  • yarn build succeeds
  • Local dev server runs and wallet connection works
  • Verify wallet connect/disconnect flow with Subwallet and Talisman
  • Verify beneficiary verification form still works end-to-end

🤖 Generated with Claude Code

Remove the inline wallet connection implementation (components, store,
types, hooks, constants) and replace with the published
@autonomys/auto-wallet and @autonomys/auto-wallet-react packages at
v1.6.9 from npm. Also bumps auto-consensus and auto-utils from 1.5.14
to ^1.6.9.

This removes ~870 lines of local code in favour of the shared SDK
packages, which include all bugfixes and review feedback from PR #511.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
jim-counter and others added 2 commits February 11, 2026 16:03
The deploy job requires the github-pages environment which is only
available on pushes to main. Gate the deploy step with an if condition
so PRs still get the build check without a deploy failure.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace leftover 'substrate-wallet-demo-preferences' name from when the
project was a demo scaffold.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jim-counter jim-counter merged commit c538123 into main Feb 11, 2026
2 checks passed
@jim-counter jim-counter deleted the claude/nifty-blackwell branch February 11, 2026 16:09
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