feat: modules exit after SIGKILL of app

[0xbrayo]

Closes #190

---

Important

Ensure child processes terminate with parent using job objects on Windows and pipes on Unix in manager.rs.

• Behavior:
  • On Windows, create_job_object() in manager.rs creates a job object to ensure child processes are killed when the parent is terminated.
  • On Unix, monitor_parent_process() in manager.rs uses a pipe to detect parent process termination and sends SIGTERM to child processes.
• Dependencies:
  • Add libc to Cargo.toml for Unix-specific functionality.
  • Update winapi features in Cargo.toml to include jobapi2, handleapi, processthreadsapi, and winnt for job object management.
• Misc:
  • Add error handling for job object creation and assignment in start_generic_module_thread() and start_notify_module_thread() in manager.rs.

^{This description was created by for 4cd0558. You can customize this summary. It will automatically update as commits are pushed.}

[0xbrayo]

Haven't tested on Windows yet

[ellipsis-dev]

Caution

Changes requested ❌

Reviewed everything up to 4cd0558 in 2 minutes and 29 seconds. Click for details.

• Reviewed 366 lines of code in 3 files
• Skipped 1 files when reviewing.
• Skipped posting 3 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. src-tauri/src/manager.rs:728

• Draft comment:
  In the aw-notify thread, after manually reading stdout via BufReader (lines ~730–760), the call to wait_with_output may yield an Output with empty stdout since it was already consumed. Verify this is intended for logging purposes.
• Reason this comment was not posted:
  Comment was not on a location in the diff, so it can't be submitted as a review comment.

2. src-tauri/src/manager.rs:490

• Draft comment:
  There is duplicated logic between start_generic_module_thread and start_notify_module_thread. Consider refactoring shared code to improve maintainability.
• Reason this comment was not posted:
  Comment was not on a location in the diff, so it can't be submitted as a review comment.

3. src-tauri/src/manager.rs:497

• Draft comment:
  The error message on line 497 uses the placeholder {name}. Please verify if this is intentional, as it doesn't appear to interpolate a variable. If no variable is intended, consider removing or correcting the placeholder.
• Reason this comment was not posted:
  Comment looked like it was already resolved.

Workflow ID: wflow_Nb3Zc3XY6QalqJdy

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

[0xbrayo]

Tested on windows, works perfectly