fix: tryfix multikill grouping

record-daemon/Cargo.lock

@@ -2418,6 +2418,7 @@ dependencies = [
  "tokio-util",
  "toml",
  "tracing",
+ "tracing-appender",
  "tracing-subscriber",
  "uuid",
  "winapi 0.3.9",
@@ -3074,6 +3075,12 @@ version = "2.6.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "13c2bddecc57b384dee18652358fb23172facb8a2c51ccc10d74c157bdea3292"
 
+[[package]]
+name = "symlink"
+version = "0.1.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "a7973cce6668464ea31f176d85b13c7ab3bba2cb3b77a2ed26abd7801688010a"
+
 [[package]]
 name = "syn"
 version = "2.0.117"
@@ -3255,6 +3262,7 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "743bd48c283afc0388f9b8827b976905fb217ad9e647fae3a379a9283c4def2c"
 dependencies = [
  "deranged",
+ "itoa",
  "num-conv",
  "powerfmt",
  "serde_core",
@@ -3520,6 +3528,19 @@ dependencies = [
  "tracing-core",
 ]
 
+[[package]]
+name = "tracing-appender"
+version = "0.2.5"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "050686193eb999b4bb3bc2acfa891a13da00f79734704c4b8b4ef1a10b368a3c"
+dependencies = [
+ "crossbeam-channel",
+ "symlink",
+ "thiserror 2.0.18",
+ "time",
+ "tracing-subscriber",
+]
+
 [[package]]
 name = "tracing-attributes"
 version = "0.1.31"

record-daemon/Cargo.toml

@@ -34,6 +34,7 @@ anyhow = "1"
 # Logging
 tracing = "0.1"
 tracing-subscriber = { version = "0.3", features = ["env-filter"] }
+tracing-appender = "0.2"
 
 # UUID for recording IDs
 uuid = { version = "1", features = ["v4", "serde"] }

record-daemon/src/main.rs

@@ -42,6 +42,10 @@ struct Args {
     /// Socket path for IPC.
     #[arg(short, long)]
     socket: Option<std::path::PathBuf>,
+
+    /// Path to log file. If not specified, logs to stdout/stderr.
+    #[arg(short, long, value_name = "PATH")]
+    log_file: Option<std::path::PathBuf>,
 }
 
 /// Main daemon structure.
@@ -579,15 +583,49 @@ impl Daemon {
     }
 }
 
-/// Initialize logging.
-fn init_logging(level: &str) {
+/// Initialize logging with optional file output.
+fn init_logging(level: &str, log_file: Option<&std::path::Path>) {
     let filter = tracing_subscriber::EnvFilter::try_from_default_env()
         .unwrap_or_else(|_| tracing_subscriber::EnvFilter::new(level));
 
-    tracing_subscriber::registry()
-        .with(filter)
-        .with(tracing_subscriber::fmt::layer())
-        .init();
+    if let Some(log_path) = log_file {
+        // Ensure parent directory exists
+        if let Some(parent) = log_path.parent() {
+            let _ = std::fs::create_dir_all(parent);
+        }
+
+        // Use file appender for logging
+        let prefix = log_path
+            .file_stem()
+            .and_then(|s| s.to_str())
+            .unwrap_or("daemon");
+        
+        let file_appender = tracing_appender::rolling::RollingFileAppender::builder()
+            .rotation(tracing_appender::rolling::Rotation::DAILY)
+            .filename_prefix(prefix)
+            .filename_suffix("log")
+            .build(log_path.parent().unwrap_or(std::path::Path::new(".")))
+            .expect("Failed to create log file appender");
+
+        let (non_blocking, _guard) = tracing_appender::non_blocking(file_appender);
+        
+        // Store the guard to prevent it from being dropped
+        // We leak it intentionally to keep logging working for the daemon's lifetime
+        std::mem::forget(_guard);
+
+        tracing_subscriber::registry()
+            .with(filter)
+            .with(tracing_subscriber::fmt::layer().with_writer(non_blocking))
+            .init();
+
+        eprintln!("Logging to file: {}", log_path.display());
+    } else {
+        // Log to stdout/stderr
+        tracing_subscriber::registry()
+            .with(filter)
+            .with(tracing_subscriber::fmt::layer())
+            .init();
+    }
 
     // Set up panic hook to log panics
     std::panic::set_hook(Box::new(|panic_info| {
@@ -614,7 +652,7 @@ async fn main() -> Result<()> {
     let args = Args::parse();
 
     // Initialize logging
-    init_logging(&args.log_level);
+    init_logging(&args.log_level, args.log_file.as_deref());
 
     info!("Record Daemon v{} starting", record_daemon::VERSION);
 

tauri-app/src-tauri/src/daemon_ipc.rs

@@ -314,6 +314,20 @@ fn find_daemon_binary() -> Option<PathBuf> {
     None
 }
 
+/// Get the log file path for the daemon.
+///
+/// Returns a path in the app's data directory.
+fn get_log_file_path() -> Result<PathBuf, String> {
+    let data_dir = directories::ProjectDirs::from("com", "leaguerecorder", "record-daemon")
+        .ok_or_else(|| "Failed to get app data directory".to_string())?;
+    
+    let log_dir = data_dir.data_dir().join("logs");
+    std::fs::create_dir_all(&log_dir)
+        .map_err(|e| format!("Failed to create log directory: {}", e))?;
+    
+    Ok(log_dir.join("daemon.log"))
+}
+
 /// Spawn the record-daemon as a detached background process.
 ///
 /// Returns `Ok(())` if the process was successfully spawned.
@@ -321,10 +335,13 @@ fn spawn_daemon() -> Result<(), String> {
     let binary = find_daemon_binary()
         .ok_or_else(|| "record-daemon binary not found".to_string())?;
 
+    let log_file = get_log_file_path()?;
     eprintln!("[daemon] Spawning record-daemon: {}", binary.display());
+    eprintln!("[daemon] Log file: {}", log_file.display());
 
     let mut cmd = std::process::Command::new(&binary);
     cmd.arg("--foreground");
+    cmd.arg("--log-file").arg(&log_file);
 
     // Set the working directory to the daemon binary's directory so it can
     // find its bundled resources (OBS plugins, etc.) relative to itself.
@@ -536,3 +553,25 @@ pub async fn daemon_shutdown() -> Result<serde_json::Value, String> {
         Err(response.error.unwrap_or_else(|| "Unknown error".to_string()))
     }
 }
+
+/// Get the daemon log file path.
+#[tauri::command]
+pub fn daemon_get_log_path() -> Result<String, String> {
+    get_log_file_path()
+        .map(|p| p.to_string_lossy().to_string())
+}
+
+/// Read the daemon log file.
+///
+/// Returns the last `lines` lines of the log file (default 100, max 1000).
+#[tauri::command]
+pub fn daemon_read_logs(lines: Option<usize>) -> Result<String, String> {
+    let log_path = get_log_file_path()?;
+    let max_lines = lines.unwrap_or(100).min(1000);
+    
+    let content = std::fs::read_to_string(&log_path)
+        .map_err(|e| format!("Failed to read log file: {}", e))?;
+    
+    let lines: Vec<&str> = content.lines().rev().take(max_lines).collect();
+    Ok(lines.into_iter().rev().collect::<Vec<_>>().join("\n"))
+}

tauri-app/src-tauri/src/lib.rs

@@ -439,6 +439,8 @@ pub fn run() {
             daemon_ipc::daemon_start_recording,
             daemon_ipc::daemon_stop_recording,
             daemon_ipc::daemon_shutdown,
+            daemon_ipc::daemon_get_log_path,
+            daemon_ipc::daemon_read_logs,
         ])
         .run(tauri::generate_context!())
         .expect("error while running tauri application");

tauri-app/src-tauri/tauri.conf.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://schema.tauri.app/config/2",
   "productName": "leaguerecorder",
-  "version": "0.1.0",
+  "version": "0.2.0",
   "identifier": "v.leaguerecorder",
   "build": {
     "beforeDevCommand": "npm run dev",

tauri-app/src/components/HighlightsPanel.vue

@@ -42,6 +42,7 @@ const typeToggles = ref<Record<HighlightType, boolean>>({
   death: DEFAULT_HIGHLIGHT_SETTINGS.included_types.includes("death"),
   assist: DEFAULT_HIGHLIGHT_SETTINGS.included_types.includes("assist"),
   multi_kill: DEFAULT_HIGHLIGHT_SETTINGS.included_types.includes("multi_kill"),
+  trade_kill: DEFAULT_HIGHLIGHT_SETTINGS.included_types.includes("trade_kill"),
 });
 
 // Watch type toggles and update settings
@@ -82,6 +83,7 @@ const activeClipId = computed(() => {
 const killCount = computed(() => props.highlights.filter(h => h.highlight_type === "kill").length);
 const deathCount = computed(() => props.highlights.filter(h => h.highlight_type === "death").length);
 const multiKillCount = computed(() => props.highlights.filter(h => h.highlight_type === "multi_kill").length);
+const tradeKillCount = computed(() => props.highlights.filter(h => h.highlight_type === "trade_kill").length);
 const assistCount = computed(() => props.highlights.filter(h => h.highlight_type === "assist").length);
 const totalDuration = computed(() => {
   const total = props.highlights.reduce((sum, h) => sum + (h.end_time - h.start_time), 0);
@@ -173,6 +175,7 @@ function formatClipDuration(clip: HighlightClip): string {
               { type: 'death' as HighlightType, label: 'Deaths', icon: '💀' },
               { type: 'assist' as HighlightType, label: 'Assists', icon: '🤝' },
               { type: 'multi_kill' as HighlightType, label: 'Multi Kills', icon: '🔥' },
+              { type: 'trade_kill' as HighlightType, label: 'Trade Kills', icon: '🔄' },
             ]"
             :key="typeInfo.type"
             class="type-toggle"
@@ -230,6 +233,9 @@ function formatClipDuration(clip: HighlightClip): string {
       <div class="stat-pill" v-if="multiKillCount > 0">
         <span>🔥</span> {{ multiKillCount }}
       </div>
+      <div class="stat-pill" v-if="tradeKillCount > 0">
+        <span>🔄</span> {{ tradeKillCount }}
+      </div>
       <div class="stat-pill total">
         <span>⏱️</span> {{ formatDuration(totalDuration) }}
       </div>

tauri-app/src/types/timeline.ts

@@ -728,7 +728,7 @@ export type EventCategory =
 /**
  * The type of highlight clip.
  */
-export type HighlightType = "kill" | "death" | "assist" | "multi_kill";
+export type HighlightType = "kill" | "death" | "assist" | "multi_kill" | "trade_kill";
 
 /**
  * Configuration for highlight generation.
@@ -755,7 +755,7 @@ export const DEFAULT_HIGHLIGHT_SETTINGS: HighlightSettings = {
   buffer_before: 10,
   buffer_after: 3,
   min_duration: 5,
-  included_types: ["kill", "death", "multi_kill"],
+  included_types: ["kill", "death", "multi_kill", "trade_kill"],
   merge_overlapping: true,
   merge_gap_secs: 5,
 };
@@ -808,23 +808,46 @@ export function getKillHighlightType(
   return null; // Not involved — skip
 }
 
+/**
+ * Result of multi-kill detection, including whether it's a trade kill.
+ */
+export interface MultiKillGroup {
+  /** Kill events in the group. */
+  kills: TimestampedEvent[];
+  /** Death event if the player died during the multi-kill window. */
+  death: TimestampedEvent | null;
+  /** Whether this is a trade kill (player died during the sequence). */
+  isTradeKill: boolean;
+}
+
 /**
  * Detect multi-kills from a sequence of kill events.
  * Returns groups of kills that happened within `windowSecs` seconds.
+ * Also checks if the player died during the multi-kill window (trade kill).
  */
 export function detectMultiKills(
   killEvents: TimestampedEvent[],
+  deathEvents: TimestampedEvent[] = [],
   windowSecs: number = 12,
-): TimestampedEvent[][] {
-  if (killEvents.length === 0) return [];
+  localPlayerName?: string,
+): MultiKillGroup[] {
+  // Filter to only include kills where the player is the killer
+  const playerKills = localPlayerName
+    ? killEvents.filter((event) => {
+        const rawData = event.raw_data as { KillerName?: string } | null;
+        return rawData?.KillerName === localPlayerName;
+      })
+    : killEvents;
 
-  const sorted = [...killEvents].sort((a, b) => {
+  if (playerKills.length === 0) return [];
+
+  const sorted = [...playerKills].sort((a, b) => {
     const aTime = a.video_timestamp[0] + a.video_timestamp[1] / 1e9;
     const bTime = b.video_timestamp[0] + b.video_timestamp[1] / 1e9;
     return aTime - bTime;
   });
 
-  const groups: TimestampedEvent[][] = [];
+  const groups: MultiKillGroup[] = [];
   let currentGroup: TimestampedEvent[] = [sorted[0]];
 
   for (let i = 1; i < sorted.length; i++) {
@@ -837,19 +860,57 @@ export function detectMultiKills(
       currentGroup.push(sorted[i]);
     } else {
       if (currentGroup.length >= 2) {
-        groups.push(currentGroup);
+        // Check for death in this window
+        const firstTime = currentGroup[0].video_timestamp[0] + currentGroup[0].video_timestamp[1] / 1e9;
+        const lastTime = currentGroup[currentGroup.length - 1].video_timestamp[0] + currentGroup[currentGroup.length - 1].video_timestamp[1] / 1e9;
+        const death = findDeathInWindow(deathEvents, firstTime, lastTime, windowSecs);
+        
+        groups.push({
+          kills: currentGroup,
+          death,
+          isTradeKill: death !== null,
+        });
       }
       currentGroup = [sorted[i]];
     }
   }
 
+  // Handle last group
   if (currentGroup.length >= 2) {
-    groups.push(currentGroup);
+    const firstTime = currentGroup[0].video_timestamp[0] + currentGroup[0].video_timestamp[1] / 1e9;
+    const lastTime = currentGroup[currentGroup.length - 1].video_timestamp[0] + currentGroup[currentGroup.length - 1].video_timestamp[1] / 1e9;
+    const death = findDeathInWindow(deathEvents, firstTime, lastTime, windowSecs);
+    
+    groups.push({
+      kills: currentGroup,
+      death,
+      isTradeKill: death !== null,
+    });
   }
 
   return groups;
 }
 
+/**
+ * Find a death event within the multi-kill window.
+ * Checks from the first kill to the last kill, plus the window buffer.
+ */
+function findDeathInWindow(
+  deathEvents: TimestampedEvent[],
+  firstKillTime: number,
+  lastKillTime: number,
+  windowSecs: number,
+): TimestampedEvent | null {
+  for (const death of deathEvents) {
+    const deathTime = death.video_timestamp[0] + death.video_timestamp[1] / 1e9;
+    // Death must occur within the window starting from first kill
+    if (deathTime >= firstKillTime && deathTime <= lastKillTime + windowSecs) {
+      return death;
+    }
+  }
+  return null;
+}
+
 /**
  * Get the multi-kill label based on number of kills.
  */
@@ -872,6 +933,7 @@ export function getHighlightTypeColor(type: HighlightType): string {
     case "death": return "#f87171";       // red
     case "assist": return "#a78bfa";      // purple
     case "multi_kill": return "#f97316";  // orange
+    case "trade_kill": return "#fbbf24";  // amber/yellow
   }
 }
 
@@ -884,6 +946,7 @@ export function getHighlightTypeIcon(type: HighlightType): string {
     case "death": return "💀";
     case "assist": return "🤝";
     case "multi_kill": return "🔥";
+    case "trade_kill": return "🔄";
   }
 }
 
@@ -975,24 +1038,49 @@ export function computeHighlights(
     });
   }
 
-  // Detect multi-kills and create clips for them (only if enabled)
-  if (settings.included_types.includes("multi_kill")) {
-    const multiKillGroups = detectMultiKills(playerKills);
+  // Detect multi-kills and trade kills (only if enabled)
+  if (settings.included_types.includes("multi_kill") || settings.included_types.includes("trade_kill")) {
+    const multiKillGroups = detectMultiKills(playerKills, playerDeaths);
     for (const group of multiKillGroups) {
-      const firstTime = group[0].video_timestamp[0] + group[0].video_timestamp[1] / 1e9;
+      // Skip if neither multi_kill nor trade_kill is enabled
+      if (!settings.included_types.includes("multi_kill") && !settings.included_types.includes("trade_kill")) {
+        continue;
+      }
+      
+      // If it's a trade kill but trade_kill is not enabled, skip
+      if (group.isTradeKill && !settings.included_types.includes("trade_kill")) {
+        continue;
+      }
+      
+      // If it's a multi kill but multi_kill is not enabled, skip
+      if (!group.isTradeKill && !settings.included_types.includes("multi_kill")) {
+        continue;
+      }
+      
+      const firstTime = group.kills[0].video_timestamp[0] + group.kills[0].video_timestamp[1] / 1e9;
       const lastTime =
-        group[group.length - 1].video_timestamp[0] +
-        group[group.length - 1].video_timestamp[1] / 1e9;
-      const label = getMultiKillLabel(group.length);
+        group.kills[group.kills.length - 1].video_timestamp[0] +
+        group.kills[group.kills.length - 1].video_timestamp[1] / 1e9;
+      const label = getMultiKillLabel(group.kills.length);
+
+      // Include death event if present
+      const allEvents = group.death ? [...group.kills, group.death] : group.kills;
+      
+      // Extend end time if death occurred after last kill
+      const endTime = group.death
+        ? Math.max(lastTime, group.death.video_timestamp[0] + group.death.video_timestamp[1] / 1e9)
+        : lastTime;
 
       clips.push({
         id: clipId++,
         start_time: Math.max(0, firstTime - settings.buffer_before),
-        end_time: Math.min(duration, lastTime + settings.buffer_after),
-        highlight_type: "multi_kill",
-        events: group,
-        title: label || `${group.length}x Kill`,
-        subtitle: `${group.length} kills in quick succession`,
+        end_time: Math.min(duration, endTime + settings.buffer_after),
+        highlight_type: group.isTradeKill ? "trade_kill" : "multi_kill",
+        events: allEvents,
+        title: group.isTradeKill ? `Trade Kill` : (label || `${group.kills.length}x Kill`),
+        subtitle: group.isTradeKill
+          ? `${group.kills.length} kills but died in the trade`
+          : `${group.kills.length} kills in quick succession`,
       });
     }
   }
@@ -1013,7 +1101,7 @@ export function computeHighlights(
         current.end_time = Math.max(current.end_time, next.end_time);
         current.events = [...current.events, ...next.events];
         // Keep the "more important" highlight type
-        const typePriority: HighlightType[] = ["multi_kill", "kill", "death", "assist"];
+        const typePriority: HighlightType[] = ["multi_kill", "trade_kill", "kill", "death", "assist"];
         const currentIdx = typePriority.indexOf(current.highlight_type);
         const nextIdx = typePriority.indexOf(next.highlight_type);
         if (nextIdx < currentIdx) {
@@ -1021,14 +1109,39 @@ export function computeHighlights(
           current.title = next.title;
           current.subtitle = next.subtitle;
         }
-        // If both are kills, check if combined they form a multi-kill
-        const killEvents = current.events.filter(e => e.event_type === "kill");
-        if (killEvents.length >= 2) {
-          const label = getMultiKillLabel(killEvents.length);
+        // If both are kills, check if combined they form a multi-kill or trade kill
+        // Only count kills where the player is the killer, not all kill events
+        // Deduplicate by video_timestamp since same event can appear in multiple merged clips
+        const seenTimestamps = new Set<string>();
+        const playerKillEvents = current.events.filter(e => {
+          if (e.event_type !== "kill") return false;
+          const rawData = e.raw_data as { KillerName?: string } | null;
+          if (rawData?.KillerName !== localPlayerName) return false;
+          // Deduplicate by timestamp
+          const key = `${e.video_timestamp[0]}-${e.video_timestamp[1]}`;
+          if (seenTimestamps.has(key)) return false;
+          seenTimestamps.add(key);
+          return true;
+        });
+        const playerDeathEvents = current.events.filter(e => {
+          if (e.event_type !== "kill") return false;
+          const rawData = e.raw_data as { VictimName?: string } | null;
+          if (rawData?.VictimName !== localPlayerName) return false;
+          // Deduplicate by timestamp
+          const key = `${e.video_timestamp[0]}-${e.video_timestamp[1]}`;
+          if (seenTimestamps.has(key)) return false;
+          seenTimestamps.add(key);
+          return true;
+        });
+        if (playerKillEvents.length >= 2) {
+          const label = getMultiKillLabel(playerKillEvents.length);
           if (label) {
-            current.highlight_type = "multi_kill";
-            current.title = label;
-            current.subtitle = `${killEvents.length} kills`;
+            const isTradeKill = playerDeathEvents.length > 0;
+            current.highlight_type = isTradeKill ? "trade_kill" : "multi_kill";
+            current.title = isTradeKill ? `Trade Kill` : label;
+            current.subtitle = isTradeKill
+              ? `${playerKillEvents.length} kills but died in the trade`
+              : `${playerKillEvents.length} kills`;
           }
         }
       } else {