Skip to content

Add selective key migration for os keyring migration CLI#86

Open
metalarm10 wants to merge 2 commits intomasterfrom
john/os-keyring-selective-migration
Open

Add selective key migration for os keyring migration CLI#86
metalarm10 wants to merge 2 commits intomasterfrom
john/os-keyring-selective-migration

Conversation

@metalarm10
Copy link
Contributor

@metalarm10 metalarm10 commented Feb 19, 2026

Description

Closes: https://app.clickup.com/t/868hh4h3a

Bulk migration is blocked on macOS: Keychain enforces per-item password prompts (2 dialogs per key), impractical for users with hundreds of keys.

This PR adds optional key name arguments to txd keys migrate-keyring, allowing users to migrate only the keys they need.

Usage:

txd keys migrate-keyring              # migrate all keys
txd keys migrate-keyring alice bob    # migrate only alice and bob

Reviewers checklist:

  • Try to write more meaningful comments with clear actions to be taken.
  • Nit-picking should be unblocking. Focus on core issues.

Authors checklist

  • Provide a concise and meaningful description
  • Review the code yourself first, before making the PR.
  • Annotate your PR in places that require explanation.
  • Think and try to split the PR to smaller PR if it is big.

This change is Reviewable

@metalarm10 metalarm10 requested a review from a team as a code owner February 19, 2026 07:42
@metalarm10 metalarm10 requested review from TxCorpi0x, masihyeganeh, miladz68 and ysv and removed request for a team February 19, 2026 07:42
Copy link
Contributor

@miladz68 miladz68 left a comment

Choose a reason for hiding this comment

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

@miladz68 reviewed 1 file and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on masihyeganeh, TxCorpi0x, and ysv).

Copy link
Contributor

@ysv ysv left a comment

Choose a reason for hiding this comment

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

@ysv reviewed 1 file and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on masihyeganeh and TxCorpi0x).

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.

3 participants

Comments