fix: hide reshare option when vault has MLDSA key

[johnnyluo]

Summary

• Hide the reshare option in vault advanced settings when the vault has an MLDSA key (publicKeyMLDSA44 != nil)
• MLDSA keys do not support reshare yet, so the option should not be visible for those vaults

Test Plan

• Open a vault without an MLDSA key → reshare option is visible (existing fast vault excluded)
• Generate an MLDSA key for a vault → reshare option is hidden
• SwiftLint passes (no new violations)

Co-Authored-By: Claude noreply@anthropic.com

Summary by CodeRabbit

• Bug Fixes
  • Updated vault reshare option visibility to properly account for additional vault configurations, improving the accuracy of when the reshare feature appears in vault settings.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7bbef432-d4f3-4231-8941-428dbea6fdb4

📥 Commits

Reviewing files that changed from the base of the PR and between 7183647 and 6f45198.

📒 Files selected for processing (1)

• VultisigApp/VultisigApp/Views/Vault/Settings/VaultAdvancedSettingsScreen.swift

---

Walkthrough

Modified the Reshare row visibility condition in VaultAdvancedSettingsScreen.swift to require both the vault is not fast and publicKeyMLDSA44 is nil, adding an additional check to hide the option when an MLDSA44 key already exists.

Changes

Cohort / File(s)	Summary
Reshare Visibility Logic VultisigApp/VultisigApp/Views/Vault/Settings/VaultAdvancedSettingsScreen.swift	Updated Reshare row visibility condition to check both vault.isFast and publicKeyMLDSA44 nil status, preventing the option from appearing when an MLDSA44 key is already present.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: hiding the reshare option when a vault has an MLDSA key, which is the primary modification in the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch hide-reshare-for-mldsa-vaults

---

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