Description
Follow-up from #6450 (durable TUI passive rotate-key indicator, implemented in the #6451/#6450/#6454 PR).
#6450 asked to map CipherError::UnknownKeyId on a stale in-memory cipher to an actionable message "in the TUI reveal path specifically" (the CLI reveal path already does this via describe_reveal_error). During implementation it was confirmed that no TUI reveal/decrypt path exists today — the TUI durable panel only has a passive D metadata toggle, never touching the cipher. Building a speculative reveal path just to wire this mapping was explicitly scoped out (see PR discussion / CHANGELOG "Known gap" note).
Expected Behavior
If/when a TUI reveal path is added, it should map CipherError::UnknownKeyId to an actionable message the same way the CLI's describe_reveal_error does today. describe_reveal_error was confirmed reusable as-is (a free function taking a CipherError) for this purpose.
Actual Behavior
No TUI reveal path exists, so this mapping has nowhere to be wired. Tracked here so the dropped half of #6450's original acceptance criteria isn't lost.
Environment
Description
Follow-up from #6450 (durable TUI passive rotate-key indicator, implemented in the #6451/#6450/#6454 PR).
#6450 asked to map
CipherError::UnknownKeyIdon a stale in-memory cipher to an actionable message "in the TUI reveal path specifically" (the CLI reveal path already does this viadescribe_reveal_error). During implementation it was confirmed that no TUI reveal/decrypt path exists today — the TUI durable panel only has a passiveDmetadata toggle, never touching the cipher. Building a speculative reveal path just to wire this mapping was explicitly scoped out (see PR discussion / CHANGELOG "Known gap" note).Expected Behavior
If/when a TUI reveal path is added, it should map
CipherError::UnknownKeyIdto an actionable message the same way the CLI'sdescribe_reveal_errordoes today.describe_reveal_errorwas confirmed reusable as-is (a free function taking aCipherError) for this purpose.Actual Behavior
No TUI reveal path exists, so this mapping has nowhere to be wired. Tracked here so the dropped half of #6450's original acceptance criteria isn't lost.
Environment