Skip to content

feat: vss ldk debug ui#779

Open
ovitrif wants to merge 8 commits intomasterfrom
feat/ldk-vss-settings
Open

feat: vss ldk debug ui#779
ovitrif wants to merge 8 commits intomasterfrom
feat/ldk-vss-settings

Conversation

@ovitrif
Copy link
Collaborator

@ovitrif ovitrif commented Feb 12, 2026

This PR:

  1. Adds VSS LDK key management to the LDK Debug screen with list, delete, and export/share actions
  2. Organizes the VSS section into App and LDK tabs for clearer separation
  3. Refactors calculateTotalFee to use the Result API instead of try-catch

Description

Extends the LDK Debug screen with a tabbed VSS section separating App-level and LDK-level keys. The LDK tab supports listing all tagged keys, deleting individual keys by namespace, and exporting key values as shareable files via FileProvider. Also adds an exports cache path for file sharing and makes icon sizes configurable in SettingsTextButtonRow.

Preview

vss debug

QA Notes

1. VSS tabs

  • Open Dev Settings > VSS
  • Verify the VSS screen shows App and LDK tabs
  • Tap each tab and confirm the UI switches correctly

2. List LDK keys

  • Select the LDK tab
  • Tap "List Keys"
  • Verify keys are listed with index, version, and key name
  • Verify the count badge updates

3. Delete LDK key

  • From the listed LDK keys, tap the trash icon on a key
  • Verify the key is removed from the list
  • Verify a toast confirms the deletion

4. Export/share LDK key

  • From the listed LDK keys, tap the share icon on a key
  • Verify the system share sheet appears with the exported file
  • Verify the saved files is not blank

5. App tab unchanged

  • Switch to the App tab
  • Verify List Keys, individual delete, and Delete All work as before

# Conflicts:
#	app/src/main/java/to/bitkit/repositories/BackupRepo.kt
#	gradle/libs.versions.toml
@ovitrif ovitrif requested a review from jvsena42 February 12, 2026 00:42
@ovitrif ovitrif requested a review from pwltr February 12, 2026 09:48
jvsena42
jvsena42 previously approved these changes Feb 12, 2026
Copy link
Member

@jvsena42 jvsena42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tAck

Base automatically changed from fix/stale-graph-reset to master February 13, 2026 16:25
@ovitrif ovitrif dismissed jvsena42’s stale review February 13, 2026 16:25

The base branch was changed.

@ovitrif ovitrif requested a review from jvsena42 February 13, 2026 22:46
@ovitrif ovitrif enabled auto-merge February 13, 2026 22:46
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.

2 participants