refactor(ui): merge identity screens into the identity module (#972)

* chore(ui): remove the superseded standalone identities screen

The Identity Hub covers everything the standalone identities list did, and
the nav has not offered a way to reach that list for some time. Four live
controls still routed to it, though, and `set_main_screen` persists the
choice — so finishing a wallet import and choosing "Create Identity" parked
the user on a screen with nothing highlighted in the nav, there and on every
later launch. Those controls now point at the hub, and the screen is gone.

Rerouted to `RootScreenIdentityHub`: "Create Platform Identity" after wallet
creation, "Create Identity" / "Load Existing Identity" on the import success
screen, "Register DPNS Name" on the add-identity success screen, and the
Register-DPNS breadcrumb. The seven `add_left_panel` arguments that named the
retired route highlighted nothing; they now name the hub, matching what
`keys_screen` already did.

Encoding 0 is retired and decodes to `None`, so a returning user's persisted
value falls through to the settings default. That default was DashPay, which
is itself absent from the nav — it becomes the hub, matching
`app::FALLBACK_ROOT_SCREEN`, under a regression test.

`OnboardingAction::ImportIdentity` is dropped: it is absent from
`ONBOARDING_CARDS` and constructed nowhere.

The `nav_button_specs` doc claimed the retired routes stayed reachable via
deep links and MCP tools. Neither exists: there is no URI handler anywhere,
and `src/mcp/` and `src/bin/` reference no UI screen type.

The deleted kittest file's three app-shell tests move to `startup.rs`; a
fourth duplicated `startup::test_app_startup`. Its two Keys-popup regressions
are already covered on the live path by `keys_screen.rs` and at model level
in `encrypted_key_storage.rs`.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

* refactor(ui): merge identity screens into the identity module

`src/ui/identities/` and `src/ui/identity/` differed by one letter and split
one domain across two module trees. Everything from the plural tree now sits
flat in `src/ui/identity/`, matching the one-directory-per-UI-domain layout
used by `ui/tokens/` and `ui/wallets/`; each screen keeps its own internal
multi-file grouping.

The plural `mod.rs` contributed no names the singular one already had, so its
constants, `removed_identities_banner`, `get_selected_wallet`, and test module
merge in byte-identical — only module declarations and the header doc changed.

Paths and imports only; no behavior change.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Lukasz Klimek <842586+lklimek@users.noreply.github.com>
Co-authored-by: Claude Opus 5 <noreply@anthropic.com>

Author: 3 people

src/backend_task/identity/load_identity.rs

@@ -17,7 +17,7 @@ use crate::model::qualified_identity::{
     DPNSNameInfo, IdentityStatus, IdentityType, QualifiedIdentity,
 };
 use crate::model::wallet::{Wallet, WalletSeedHash};
-use crate::ui::identities::add_new_identity_screen::MAX_IDENTITY_INDEX;
+use crate::ui::identity::add_new_identity_screen::MAX_IDENTITY_INDEX;
 use dash_sdk::Sdk;
 use dash_sdk::dashcore_rpc::dashcore::PrivateKey;
 use dash_sdk::dashcore_rpc::dashcore::key::Secp256k1;

src/context/mod.rs

@@ -1330,7 +1330,7 @@ impl AppContext {
     fn owning_wallet_hash(&self, id: Identifier) -> Option<WalletSeedHash> {
         let identities = self.load_local_qualified_identities().ok()?;
         let qi = identities.into_iter().find(|qi| qi.identity.id() == id)?;
-        let wallet = crate::ui::identities::get_selected_wallet(&qi, Some(self), None).ok()??;
+        let wallet = crate::ui::identity::get_selected_wallet(&qi, Some(self), None).ok()??;
         let hash = wallet.read().ok()?.seed_hash();
         Some(hash)
     }

src/ui/contracts_documents/document_action_screen.rs

@@ -20,7 +20,7 @@ use crate::ui::helpers::{
     TransactionType, add_contract_doc_type_chooser_with_filtering, add_key_chooser_with_doc_type,
     show_success_screen_with_info,
 };
-use crate::ui::identities::get_selected_wallet;
+use crate::ui::identity::get_selected_wallet;
 use crate::ui::theme::{ComponentStyles, DashColors};
 use base64::Engine;
 use base64::engine::general_purpose::STANDARD;

src/ui/contracts_documents/register_contract_screen.rs

@@ -15,7 +15,7 @@ use crate::ui::components::wallet_unlock_popup::{
 };
 use crate::ui::components::{BannerHandle, MessageBanner, OptionBannerExt, ResultBannerExt};
 use crate::ui::helpers::{TransactionType, add_key_chooser};
-use crate::ui::identities::get_selected_wallet;
+use crate::ui::identity::get_selected_wallet;
 use crate::ui::theme::{ComponentStyles, DashColors};
 use crate::ui::{BackendTaskSuccessResult, MessageType, ScreenLike};
 use dash_sdk::dpp::data_contract::accessors::v0::DataContractV0Setters;

src/ui/contracts_documents/update_contract_screen.rs

@@ -16,7 +16,7 @@ use crate::ui::components::wallet_unlock_popup::{
 };
 use crate::ui::components::{BannerHandle, MessageBanner, OptionBannerExt, ResultBannerExt};
 use crate::ui::helpers::{TransactionType, add_key_chooser};
-use crate::ui::identities::get_selected_wallet;
+use crate::ui::identity::get_selected_wallet;
 use crate::ui::theme::{ComponentStyles, DashColors};
 use crate::ui::{BackendTaskSuccessResult, MessageType, ScreenLike};
 use dash_sdk::dpp::data_contract::accessors::v0::{DataContractV0Getters, DataContractV0Setters};

src/ui/dashpay/add_contact_screen.rs

@@ -19,8 +19,8 @@ use crate::ui::components::wallet_unlock_popup::{
 };
 use crate::ui::dashpay::DashPaySubscreen;
 use crate::ui::helpers::{TransactionType, add_key_chooser};
-use crate::ui::identities::get_selected_wallet;
-use crate::ui::identities::keys::add_key_screen::AddKeyScreen;
+use crate::ui::identity::get_selected_wallet;
+use crate::ui::identity::keys::add_key_screen::AddKeyScreen;
 use crate::ui::theme::{DashColors, Typography};
 use crate::ui::{MessageType, RootScreenType, Screen, ScreenLike};
 use dash_sdk::dpp::identity::accessors::IdentityGettersV0;

src/ui/dashpay/contact_requests.rs

@@ -14,8 +14,8 @@ use crate::ui::components::wallet_unlock_popup::{
     WalletUnlockPopup, WalletUnlockResult, try_open_wallet_no_password, wallet_needs_unlock,
 };
 use crate::ui::components::{MessageBanner, ResultBannerExt};
-use crate::ui::identities::get_selected_wallet;
-use crate::ui::identities::keys::add_key_screen::AddKeyScreen;
+use crate::ui::identity::get_selected_wallet;
+use crate::ui::identity::keys::add_key_screen::AddKeyScreen;
 use crate::ui::theme::DashColors;
 use crate::ui::{MessageType, Screen, ScreenLike, ScreenType};
 use dash_sdk::dpp::document::DocumentV0Getters;

src/ui/dashpay/profile_screen.rs

@@ -17,7 +17,7 @@ use crate::ui::components::wallet_unlock_popup::{
 };
 use crate::ui::components::{MessageBanner, ResultBannerExt};
 use crate::ui::helpers::{ModalOpeningGuard, clicked_outside_window_after_open};
-use crate::ui::identities::get_selected_wallet;
+use crate::ui::identity::get_selected_wallet;
 use crate::ui::state::AvatarCache;
 use crate::ui::theme::{ComponentStyles, DashColors, ResponseExt};
 use dash_sdk::dpp::identity::accessors::IdentityGettersV0;

src/ui/dashpay/qr_code_generator.rs

@@ -15,8 +15,8 @@ use crate::ui::components::wallet_unlock_popup::{
 };
 use crate::ui::components::{MessageBanner, ResultBannerExt};
 use crate::ui::dashpay::dashpay_screen::DashPaySubscreen;
-use crate::ui::identities::funding_common::generate_qr_code_image;
-use crate::ui::identities::get_selected_wallet;
+use crate::ui::identity::funding_common::generate_qr_code_image;
+use crate::ui::identity::get_selected_wallet;
 use crate::ui::theme::DashColors;
 use crate::ui::{MessageType, RootScreenType, ScreenLike};
 use eframe::epaint::TextureHandle;

src/ui/dashpay/qr_scanner.rs

@@ -15,7 +15,7 @@ use crate::ui::components::wallet_unlock_popup::{
 };
 use crate::ui::components::{MessageBanner, ResultBannerExt};
 use crate::ui::dashpay::dashpay_screen::DashPaySubscreen;
-use crate::ui::identities::get_selected_wallet;
+use crate::ui::identity::get_selected_wallet;
 use crate::ui::{MessageType, RootScreenType, ScreenLike};
 use dash_sdk::dpp::identity::accessors::IdentityGettersV0;
 use dash_sdk::dpp::identity::{KeyType, Purpose, SecurityLevel};