fix: address Lukasz review comments from PR #563

[thepastaclaw]

Summary

Addresses all review comments from @lklimek on PR #563.

Changes

1. [MEDIUM] Replace guard-then-unwrap with match/let-else (all 8 token screens)
Replaced fragile .is_none() guard + later .unwrap() with idiomatic match destructuring in: mint, claim, destroy, freeze, pause, resume, transfer, unfreeze. Follows the pattern already used in token_creator.rs:1487-1496.

2. [LOW] Move identity removal after successful DB deletion
shift_remove() now only runs after delete_local_qualified_identity() succeeds. On failure, shows a user-facing error message instead of silently diverging UI/DB state.

3. [MEDIUM] Add backend security level validation
Added validate_identity_key_security_levels() in backend_task/identity/mod.rs that enforces MEDIUM for ENCRYPTION/DECRYPTION and CRITICAL for TRANSFER keys. Defense-in-depth beyond UI-only enforcement.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (2)

• master
• v1.0-dev

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.

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
• Post copyable unit tests in a comment

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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