IJHack / QtPass / 28659639755

63%

Build Type

push

github

Committed by web-flow

Commit Message

fix: keep clipboard autoclear tracking when navigating between entries (#1607)

* fix: keep clipboard autoclear tracking when navigating between entries

clippedText records what copyTextToClipboard() placed on the clipboard so the
autoclear timer knows what to clear. But setClippedText() (called on every entry
show) and clearClippedText() (called on every tree navigation) both overwrote
it, so after copying a password and then selecting another entry the timer saw
clippedText != clipboard and refused to clear — leaving the copied password on
the clipboard indefinitely, past its autoclear window.

The staged value was never consumed anywhere (on-demand copy uses the field
value directly), so it only corrupted the autoclear tracker. Make clippedText a
pure record of the current clipboard contents: setClippedText() now only copies
in "always copy" mode (via copyTextToClipboard, which sets the tracker) and no
longer overwrites it otherwise, and the vestigial clearClippedText() and its
navigation call are removed.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JuQsrHonihp1nARE7bzstc

* docs: update setClippedText docstring to match conditional copy behavior

setClippedText only copies (and thus updates the autoclear tracker) in
"always copy" mode; document that instead of implying it always mutates the
tracked value.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JuQsrHonihp1nARE7bzstc

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Coverage Stats

0 of 2 new or added lines in 1 file covered. (0.0%)

1 existing line in 1 file now uncovered.

3811 of 6715 relevant lines covered (56.75%)

30.8 hits per line