chore: rename keygenDialog/currentDir/tray → m_-prefixed (consistency)

[annejan]

Refactored the MainWindow class by renaming the following member variables to include an m_ prefix for consistency:

• keygenDialog to m_keygenDialog
• currentDir to m_currentDir
• tray to m_tray

This change is purely a refactoring effort to align with established naming conventions and has no functional impact on the application.

[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	✅
Business Logic	✅

Access your configuration settings here.

​

[coderabbitai]

📝 Walkthrough

Walkthrough

Renamed three private member variables in MainWindow from unprefixed names (keygenDialog, tray, currentDir) to conventionally-prefixed names (m_keygenDialog, m_tray, m_currentDir). Updated all usages throughout the implementation file and the corresponding getter method.

Changes

Cohort / File(s)	Summary
Member Variable Renaming src/mainwindow.h, src/mainwindow.cpp	Refactored private member variables to use m_ naming prefix convention. Updated all references to keygenDialog, tray, and currentDir to their m_-prefixed equivalents, including the getKeygenDialog() getter method.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• fix: address Copilot review findings #1150: Concurrently modifies MainWindow keygen dialog and tray member handling with similar member variable adjustments.
• fix: address review nits — git cp comment, dedupe reencrypt disable, this-> #1198: Updates MainWindow keygen-dialog member accessors and direct access patterns affecting the same internal state.

Suggested labels

size:S

Poem

🐰 With whiskers and cheer, I hop through the code,
Those variables now wear their m_ mode!
A naming convention so clean and so bright,
Makes encapsulation feel perfectly right. ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main change: renaming three member variables to use m_ prefix for consistency. It is concise, specific, and clearly describes the primary modification.
Docstring Coverage	✅ Passed	Docstring coverage is 80.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
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 chore/rename-mainwindow-members-m-prefix

---

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.}

[coveralls]

coverage: 28.015%. remained the same — chore/rename-mainwindow-members-m-prefix into main