fix: remove macos notification icon handling

3 files changed, 7 insertions, 42 deletions

diff --git a/.pi/docs/CONFIGURATION.md b/.pi/docs/CONFIGURATION.md
index 4da0c565..24bbe8c5 100644
--- a/.pi/docs/CONFIGURATION.md
+++ b/.pi/docs/CONFIGURATION.md
@@ -310,9 +310,9 @@ for best macos support, install `terminal-notifier`:
 brew install terminal-notifier
 ```
 
-when `terminal-notifier` is installed, herdr shows the herdr icon in system notifications. herdr also tries to focus the hosting terminal when a notification is clicked. click-to-return is supported for detected ghostty, iterm2, wezterm, kitty, alacritty, and terminal.app sessions.
+when `terminal-notifier` is installed, herdr tries to focus the hosting terminal when a notification is clicked. click-to-return is supported for detected ghostty, iterm2, wezterm, kitty, alacritty, and terminal.app sessions.
 
-without `terminal-notifier`, herdr falls back to built-in `osascript`. this still shows a macos notification, but it cannot set the herdr icon and clicking the notification may focus the apple script runner instead of returning to your terminal.
+without `terminal-notifier`, herdr falls back to built-in `osascript`. this still shows a macos notification, but clicking the notification may focus the apple script runner instead of returning to your terminal.
 
 compatibility note:
 - older configs may still use `ui.toast.enabled = true|false`
diff --git a/Cargo.toml b/Cargo.toml
index 554ebd96..5b5f819c 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -9,7 +9,7 @@ repository = "https://github.com/ogulcancelik/herdr"
 homepage = "https://herdr.dev"
 keywords = ["terminal", "tui", "ai", "agents", "multiplexer"]
 categories = ["command-line-utilities"]
-include = ["src/**/*", "assets/logo.png", "assets/sounds/*", "README.md", "LICENSE", "Cargo.toml"]
+include = ["src/**/*", "assets/sounds/*", "README.md", "LICENSE", "Cargo.toml"]
 
 [dependencies]
 base64 = "0.22.1"
diff --git a/src/platform/macos.rs b/src/platform/macos.rs
index 417251ce..f085ee91 100644
--- a/src/platform/macos.rs
+++ b/src/platform/macos.rs
@@ -8,7 +8,6 @@ use std::sync::OnceLock;
 use super::{ClipboardCommand, ForegroundJob, ForegroundProcess, Signal};
 
 const PROC_PGRP_ONLY: u32 = 2;
-const HERDR_LOGO_PNG: &[u8] = include_bytes!("../../assets/logo.png");
 
 /// Collect the foreground terminal job for a given child PID.
 pub fn foreground_job(child_pid: u32) -> Option<ForegroundJob> {
@@ -205,9 +204,9 @@ pub fn write_clipboard(bytes: &[u8]) -> bool {
 
 /// Show a native macOS notification.
 ///
-/// Prefer `terminal-notifier` when it is installed because it supports Herdr's
-/// icon and can activate the hosting terminal on click. Fall back to built-in
-/// AppleScript notifications when it is not available.
+/// Prefer `terminal-notifier` when it is installed because it can activate the
+/// hosting terminal on click. Fall back to built-in AppleScript notifications
+/// when it is not available.
 pub fn show_desktop_notification(title: &str, body: Option<&str>) -> std::io::Result<bool> {
     show_desktop_notification_with_command(title, body, |program| Command::new(program))
 }
@@ -229,12 +228,10 @@ fn show_terminal_notifier_notification(
     body: Option<&str>,
     command: &mut impl FnMut(&str) -> Command,
 ) -> std::io::Result<bool> {
-    let icon_path = cached_notification_icon_path().ok();
     let activate_bundle_id = verified_terminal_bundle_identifier(command);
     show_terminal_notifier_notification_with_options(
         title,
         body,
-        icon_path.as_deref(),
         activate_bundle_id.as_deref(),
         command,
     )
@@ -243,12 +240,11 @@ fn show_terminal_notifier_notification(
 fn show_terminal_notifier_notification_with_options(
     title: &str,
     body: Option<&str>,
-    icon_path: Option<&Path>,
     activate_bundle_id: Option<&str>,
     command: &mut impl FnMut(&str) -> Command,
 ) -> std::io::Result<bool> {
     let mut cmd = command("terminal-notifier");
-    build_terminal_notifier_command(&mut cmd, title, body, icon_path, activate_bundle_id);
+    build_terminal_notifier_command(&mut cmd, title, body, activate_bundle_id);
     run_notification_command(cmd)
 }
 
@@ -256,14 +252,10 @@ fn build_terminal_notifier_command(
     cmd: &mut Command,
     title: &str,
     body: Option<&str>,
-    icon_path: Option<&Path>,
     activate_bundle_id: Option<&str>,
 ) {
     cmd.arg("-title").arg(title);
     cmd.arg("-message").arg(body.unwrap_or_default());
-    if let Some(icon_path) = icon_path {
-        cmd.arg("-appIcon").arg(icon_path);
-    }
     if let Some(bundle_id) = activate_bundle_id {
         cmd.arg("-activate").arg(bundle_id);
     }
@@ -286,28 +278,6 @@ fn show_osascript_notification(
     run_notification_command(cmd)
 }
 
-fn cached_notification_icon_path() -> std::io::Result<PathBuf> {
-    static ICON_PATH: OnceLock<std::io::Result<PathBuf>> = OnceLock::new();
-    match ICON_PATH.get_or_init(write_notification_icon) {
-        Ok(path) => Ok(path.clone()),
-        Err(err) => Err(std::io::Error::new(err.kind(), err.to_string())),
-    }
-}
-
-fn write_notification_icon() -> std::io::Result<PathBuf> {
-    let cache_dir = if let Ok(home) = std::env::var("HOME") {
-        PathBuf::from(home).join("Library/Caches/herdr")
-    } else {
-        std::env::temp_dir().join("herdr")
-    };
-    std::fs::create_dir_all(&cache_dir)?;
-    let path = cache_dir.join("logo.png");
-    if std::fs::read(&path).ok().as_deref() != Some(HERDR_LOGO_PNG) {
-        std::fs::write(&path, HERDR_LOGO_PNG)?;
-    }
-    Ok(path)
-}
-
 fn verified_terminal_bundle_identifier(
     command: &mut impl FnMut(&str) -> Command,
 ) -> Option<String> {
@@ -694,7 +664,6 @@ mod tests {
             &mut cmd,
             "pi finished",
             Some("workspace 1"),
-            Some(Path::new("/tmp/herdr-logo.png")),
             Some("com.mitchellh.ghostty"),
         );
         let args = cmd
@@ -708,8 +677,6 @@ mod tests {
                 "pi finished",
                 "-message",
                 "workspace 1",
-                "-appIcon",
-                "/tmp/herdr-logo.png",
                 "-activate",
                 "com.mitchellh.ghostty"
             ]
@@ -735,7 +702,6 @@ mod tests {
         let shown = show_terminal_notifier_notification_with_options(
             "title",
             Some("body"),
-            Some(Path::new("/tmp/herdr-logo.png")),
             Some("com.mitchellh.ghostty"),
             &mut command,
         )
@@ -745,7 +711,6 @@ mod tests {
         let args = std::fs::read_to_string(&path).expect("args file");
         let _ = std::fs::remove_file(&path);
         assert!(args.starts_with("terminal-notifier:"), "{args}");
-        assert!(args.contains("-appIcon /tmp/herdr-logo.png"), "{args}");
         assert!(args.contains("-activate com.mitchellh.ghostty"), "{args}");
         assert!(!args.contains("osascript"), "{args}");
     }