feat: switch notification format to json lines

[0xbrayo]

Important

Switch aw-notify notification parsing to JSON Lines format in manager.rs.

• Behavior:
  • start_notify_module_thread in manager.rs now parses notifications as JSON Lines instead of using a custom format with dashes.
  • Extracts title and message from JSON notifications and formats them as "title\nmessage".
  • Logs an error if JSON notification is missing title or message fields.
  • Falls back to logging non-JSON lines as potential info logs or old format.

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

[ellipsis-dev]

Important

Looks good to me! 👍

Reviewed everything up to 62c9cfb in 52 seconds. Click for details.

• Reviewed 63 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 2 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:719

• Draft comment:
  Consider checking for empty lines (e.g. using if line_content.trim().is_empty() { continue; }) before attempting to parse as JSON. This can reduce unnecessary parse attempts and debug log noise.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50% None

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

• Draft comment:
  Consider logging detailed serde_json errors when JSON parsing fails (instead of only logging non-JSON output) to aid in debugging malformed notifications.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50% None

Workflow ID: wflow_jnTTMb5u0xiBPUAA

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

[ErikBjare]

Nice