feat: add re-encrypt context menu for directories

[nogeenharrie]

Fixes: #269

This pull request introduces a new "Re-encrypt" context menu option for directories within the main application's tree view.

Key changes include:

• Context Menu Integration: When a directory is right-clicked, a "Re-encrypt" action is now available in the context menu.
• Re-encryption Functionality: A new reencryptPath function handles the re-encryption process.
  • It first validates the directory path.
  • It presents a confirmation dialog to the user, explaining that all password files within the selected directory will be re-encrypted using the current GPG recipients defined in .gpg-id, and that this action is not easily reversible.
  • Upon user confirmation, the main window is temporarily disabled to prevent further interaction, and the re-encryption operation is delegated to the ImitatePass backend for the specified directory.

[codeant-ai]

CodeAnt AI is reviewing your PR.

[coderabbitai]

📝 Walkthrough

Walkthrough

Added a "Re-encrypt" directory context-menu action in MainWindow that validates the selected folder, prompts the user for confirmation, disables UI interaction, and calls ImitatePass::reencryptPath() with the folder's absolute path to re-encrypt contained password files.

Changes

Cohort / File(s)	Summary
Directory Context Menu src/mainwindow.cpp	Added MainWindow::reencryptPath(QString) and a conditional "Re-encrypt" action in the tree view context menu. The handler validates the directory exists, shows a confirmation dialog, disables relevant UI/tree view on confirm, and delegates to QtPassSettings::getImitatePass()->reencryptPath() with a cleaned absolute path.

Sequence Diagram

sequenceDiagram
    participant User as User
    participant MW as MainWindow
    participant Dlg as ConfirmationDialog
    participant IP as ImitatePass
    participant FS as FileSystem

    User->>MW: Right-click folder + select "Re-encrypt"
    MW->>FS: verify directory exists
    alt not exists
        MW->>User: show critical error
    else exists
        MW->>Dlg: show confirmation (warn about .gpg-id recipients)
        User->>Dlg: confirm
        MW->>MW: disable UI / tree
        MW->>IP: reencryptPath(cleanedAbsoluteDir)
        IP->>FS: re-encrypt files using .gpg-id recipients
        IP-->>MW: completion
        MW->>MW: re-enable UI / tree
    end

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• fix: improve re-encryption security #891: Modifies ImitatePass::reencryptPath() and related re-encryption logic that this UI handler invokes; closely related at the code path level.

Poem

🐰 I nudge a folder, click anew,

Recipients set, the files pass through,
A warning hopped, confirm the night,
UI rests quiet, then springs to light,
Freshly wrapped secrets, snug and true.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: adding a re-encrypt context menu action for directories in the tree view.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feature/reencrypt-button

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/mainwindow.cpp`:
- Around line 1347-1351: The code currently disables the entire MainWindow with
this->setEnabled(false) before calling
QtPassSettings::getImitatePass()->reencryptPath(...), but the re-enable path
uses setUiElementsEnabled(true), leaving the window permanently unresponsive;
change the disable call to setUiElementsEnabled(false) so the disable/enable
operations are symmetric with setUiElementsEnabled and match the re-enabling
done in endReencryptPath()/related completion handlers.
- Around line 1058-1059: The code captures a folder path into QString dirPath
via Util::getDir but does not normalize it before use; fix by calling
QDir::cleanPath(dirPath) immediately after assignment (mirroring the existing
pattern around QDir::cleanPath used elsewhere) and use that cleaned QString when
invoking the backend re-encryption call that consumes the folder path; update
references to dirPath in both the local scope and the function/method that sends
the path to the backend so the cleaned path is always passed.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI (base), Organization UI (inherited)

Review profile: ASSERTIVE

Plan: Pro

Run ID: ba61c2d0-b6f9-4f0f-820d-99d5d0ad14f9

📥 Commits

Reviewing files that changed from the base of the PR and between e1590fe and c44e3d4.

📒 Files selected for processing (1)

• src/mainwindow.cpp

[coveralls]

Coverage is 21.015% — feature/reencrypt-button into main. No base build found for main.

[codeant-ai]

CodeAnt AI finished reviewing your PR.

[codecov]

Codecov Report

❌ Patch coverage is 0% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 27.11%. Comparing base (8f1eab1) to head (5001365).
⚠️ Report is 18 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
src/mainwindow.cpp	0.00%	20 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1009      +/-   ##
==========================================
- Coverage   27.27%   27.11%   -0.17%     
==========================================
  Files          39       39              
  Lines        3314     3334      +20     
==========================================
  Hits          904      904              
- Misses       2410     2430      +20     

Flag	Coverage Δ
qtpass	27.11% <0.00%> (-0.17%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[kody-ai]

Kody Review Complete

Great news! 🎉
No issues were found that match your current review configurations.

Keep up the excellent work! 🚀

Kody Guide: Usage and Configuration

Interacting with Kody

• Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

• Validate Business Logic: Ask Kody to validate your code against business rules by adding a comment with the @kody -v business-logic command.

• Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration

Review Options

The following review options are enabled or disabled:

Options	Enabled
Bug	✅
Performance	✅
Security	✅
Cross File	✅
Business Logic	✅

Access your configuration settings here.

​

[annejan]

I have tested this locally and it seems to work pretty well 👍🏼