Skip to content

fix(swift-sdk): wait for SDK rebuild before wallet activation#3669

Draft
thepastaclaw wants to merge 2 commits into
dashpay:claude/swift-xcuitest-wallet-persistencefrom
thepastaclaw:fix-3560-wallet-manager-sdk-switch
Draft

fix(swift-sdk): wait for SDK rebuild before wallet activation#3669
thepastaclaw wants to merge 2 commits into
dashpay:claude/swift-xcuitest-wallet-persistencefrom
thepastaclaw:fix-3560-wallet-manager-sdk-switch

Conversation

@thepastaclaw
Copy link
Copy Markdown
Collaborator

Summary

  • Defer wallet-manager activation during Swift Example App network switches
    until AppState finishes rebuilding the SDK.
  • Remember the pending network separately from currentNetwork, so activation
    does not cache a manager with the previous network's SDK.
  • Keep failed deferred activations retryable and avoid touching same-network SDK
    rebuilds that the current per-network cache cannot safely reconfigure.

Validation

  • git diff --check
  • Code review gate: Recommendation: ship
  • Swift Example App simulator build: blocked by existing local/generated FFI
    surface mismatch. The failure is outside this patch: local generated bindings
    report dash_sdk_private_key_to_wif and
    dash_sdk_validate_private_key_for_public_key expecting Bool, plus missing
    generated platform-wallet FFI symbols such as Handle.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 19, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 9ed0b11c-c004-49c2-9cde-4885b46d2cfc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@thepastaclaw
Copy link
Copy Markdown
Collaborator Author

thepastaclaw commented May 19, 2026

Review Gate

Commit: 7db756ec

  • Debounce: 6m ago (need 30m)

  • CI checks: builds passed, 0/0 tests passed

  • CodeRabbit review: comment found

  • Off-peak hours: off-peak (03:42 AM PT Tuesday)

  • Run review now (check to override)

@thepastaclaw thepastaclaw force-pushed the fix-3560-wallet-manager-sdk-switch branch from 7db756e to ef94761 Compare May 19, 2026 11:48
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