keepassxc icon indicating copy to clipboard operation
keepassxc copied to clipboard

Improve CLI Show and Clip Commands

Open droidmonkey opened this issue 9 months ago • 2 comments

  • Fixes #11767
  • When requesting TOTP, only output that value and then exit
  • Add explicit requests for UUID and Tags for both show and clip commands.
  • When showing all values, also include UUID and Tags.
  • Only hide the attribute names when specifically requesting them by name

Testing strategy

Tested new commands manually

Type of change

  • ✅ Bug fix (non-breaking change that fixes an issue)
  • ✅ New feature (change that adds functionality)

droidmonkey avatar May 10 '25 13:05 droidmonkey

@droidmonkey is this ready to review? CI is failing

louib avatar May 10 '25 17:05 louib

Codecov Report

Attention: Patch coverage is 94.59459% with 2 lines in your changes missing coverage. Please review.

Project coverage is 63.88%. Comparing base (f53c7e5) to head (87ccc85).

Files with missing lines Patch % Lines
src/cli/Clip.cpp 88.89% 2 Missing :warning:
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #12083      +/-   ##
===========================================
- Coverage    63.88%   63.88%   -0.00%     
===========================================
  Files          369      369              
  Lines        38938    38946       +8     
===========================================
+ Hits         24875    24880       +5     
- Misses       14063    14066       +3     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar May 18 '25 13:05 codecov[bot]