refactor: minor cleanup and migration of config file + version bump

[0xbrayo]

Important

Refactor configuration management, update dependencies, and improve code organization in lib.rs and manager.rs.

• Configuration Management:
  • Migrated configuration file handling to use UserDirs for Linux in lib.rs.
  • Renamed autolaunch to autostart in UserConfig in lib.rs.
• Version Updates:
  • Bumped version numbers for dependencies in Cargo.lock and Cargo.toml, including tauri, fern, nix, and notify.
• Code Refactoring:
  • Extracted handle_first_run() and listen_for_lockfile() functions in lib.rs for better code organization.
  • Updated SpecificFileWatcher to use Config::default().with_poll_interval() in lib.rs.
  • Improved tray menu update logic in ManagerState in manager.rs.

^{This description was created by for 3229db4. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 3229db4 in 1 minute and 30 seconds

More details

• Looked at 903 lines of code in 7 files
• Skipped 0 files when reviewing.
• Skipped posting 3 drafted comments based on config settings.

1. src-tauri/src/lib.rs:75

• Draft comment:
  Using thread::sleep is a temporary workaround and should be removed or replaced with a more robust solution before production.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR refactors the code to separate the handling of the first run and lockfile listening into distinct functions. This improves code readability and maintainability. However, the thread::sleep in handle_first_run is a temporary workaround and should be addressed before production. The use of unwrap in multiple places can lead to panics if not handled properly. Consider using more robust error handling strategies.

2. src-tauri/src/lib.rs:83

• Draft comment:
  Using unwrap can lead to panics if the notification fails to show. Consider handling the error more gracefully.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR refactors the code to separate the handling of the first run and lockfile listening into distinct functions. This improves code readability and maintainability. However, the use of unwrap in multiple places can lead to panics if not handled properly. Consider using more robust error handling strategies.

3. src-tauri/src/lib.rs:100

• Draft comment:
  Using unwrap can lead to panics if the window is not found. Consider handling the error more gracefully.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR refactors the code to separate the handling of the first run and lockfile listening into distinct functions. This improves code readability and maintainability. However, the use of unwrap in multiple places can lead to panics if not handled properly. Consider using more robust error handling strategies.

Workflow ID: wflow_5GA00PANMi0uDHJA

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[0xbrayo]

Updated SpecificFileWatcher to use Config::default().with_poll_interval() in lib.rs
@ellipsis-dev I can't find this in the git diff

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 3229db4 in 1 minute and 56 seconds

More details

• Looked at 903 lines of code in 7 files
• Skipped 0 files when reviewing.
• Skipped posting 3 drafted comments based on config settings.

1. src-tauri/src/lib.rs:189

• Draft comment:
  The get_config_path function for Linux still manually constructs the config path using UserDirs. Consider updating it to use ProjectDirs for consistency with other platforms.
• Reason this comment was not posted:
  Decided after close inspection that this draft comment was likely wrong and/or not actionable:

1. The code explicitly uses different implementations for Linux vs non-Linux. 2. This seems intentional given the cfg attributes. 3. Linux has different conventions for config paths. 4. The manual path construction follows the XDG Base Directory specification for Linux. 5. Using ProjectDirs may not be the right approach for Linux.
   I may be missing some benefits of using ProjectDirs on Linux. Perhaps there are edge cases where the manual path construction could fail.
   The manual path follows Linux conventions and XDG spec. ProjectDirs may actually be less appropriate on Linux where ~/.config is the standard location.
   The comment should be deleted. The Linux-specific implementation appears intentional and follows Linux conventions correctly.

2. src-tauri/src/lib.rs:161

• Draft comment:
  The UserConfig struct has autostart and autostart_minimized fields, which is consistent with the PR description. No changes needed here.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The PR description mentions renaming autolaunch to autostart in UserConfig, but the UserConfig struct still has autostart and autostart_minimized fields. This is consistent with the description, so no issue here.

3. src-tauri/src/lib.rs:258

• Draft comment:
  The change from user_config.autolaunch to user_config.autostart is consistent with the PR description and intent. No issues here.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The PR description mentions renaming autolaunch to autostart in UserConfig, and this change is reflected in the code where user_config.autolaunch is replaced with user_config.autostart. This is consistent with the description and intent of the PR.

Workflow ID: wflow_xoENM081Bc1Ze6Ci

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.