feat: add shareable profile image generation

- Add 1920x1080 HD canvas with trans-pride themed design
- Include profile avatar, name, bio, and lifetime stats
- Show achievement progress with visual progress bar
- Use Tauri fs plugin for cross-platform file reading
- Add scoped file permissions for read/write access

.gitea/workflows/ci.yml

@@ -49,13 +49,13 @@ jobs:
       - name: Run Svelte Check
         run: pnpm check
 
-      - name: Run frontend tests with coverage
-        run: pnpm test:coverage
+      - name: Run frontend tests
+        run: pnpm test
 
       - name: Setup Rust
         uses: dtolnay/rust-toolchain@stable
         with:
-          components: clippy, llvm-tools-preview
+          components: clippy
 
       - name: Cache Rust dependencies
         uses: actions/cache@v4
@@ -68,16 +68,13 @@ jobs:
             src-tauri/target/
           key: ${{ runner.os }}-cargo-${{ hashFiles('**/Cargo.lock') }}
 
-      - name: Install cargo-llvm-cov
-        run: cargo install cargo-llvm-cov --locked
-
       - name: Run Clippy
         working-directory: src-tauri
         run: cargo clippy --all-targets --all-features -- -D warnings
 
-      - name: Run Rust tests with coverage
+      - name: Run Rust tests
         working-directory: src-tauri
-        run: cargo llvm-cov --fail-under-lines 50
+        run: cargo test
 
   build-linux:
     name: Build Linux

.gitignore

@@ -8,6 +8,3 @@ node_modules
 !.env.example
 vite.config.js.timestamp-*
 vite.config.ts.timestamp-*
-
-# Coverage reports
-/coverage

CLAUDE.md

@@ -1,184 +0,0 @@
-# Hikari Desktop - Project Instructions
-
-## Repository Information
-
-This project is hosted on both GitHub and Gitea:
-
-- **GitHub**: `naomi-lgbt/hikari-desktop` (public mirror)
-- **Gitea**: `nhcarrigan/hikari-desktop` (primary development)
-
-## MCP Server Usage
-
-When working with issues, pull requests, or other repository operations for this project:
-
-- **Use `gitea-hikari` MCP server** - This allows Hikari to act as herself
-- **Target repository**: `nhcarrigan/hikari-desktop`
-- **Gitea instance**: `git.nhcarrigan.com`
-
-## Git Commits
-
-When asked to commit changes for this project:
-
-- **Always commit as Hikari** using: `--author="Hikari <hikari@nhcarrigan.com>"`
-- **Always sign commits** with Hikari's GPG key: `--gpg-sign=5380E4EE7307C808`
-- **Never add `Co-Authored-By` lines** for Gitea commits
-- **Always ask for confirmation** before committing
-- **Always ask for confirmation** before pushing
-
-Example commit command:
-
-```bash
-git commit --author="Hikari <hikari@nhcarrigan.com>" --gpg-sign=5380E4EE7307C808 -m "your commit message"
-```
-
-Example push command:
-
-```bash
-git push https://hikari:TOKEN@git.nhcarrigan.com/nhcarrigan/hikari-desktop.git <branch>
-```
-
-## Testing Requirements
-
-**All changes MUST include tests.** This is non-negotiable — no feature, bug fix, or refactor should be committed without corresponding test coverage. If a change cannot be tested (e.g. pure UI layout, Tauri IPC calls that are impossible to mock), document why in a comment.
-
-- **Frontend tests**: Use Vitest with `@testing-library/svelte` for component tests
-- **Test files**: Place test files next to the code they test with `.test.ts` or `.spec.ts` extension
-- **Run tests**: Use `pnpm test` to run all tests, or `pnpm test:watch` for watch mode
-- **Coverage**: Run `pnpm test:coverage` to generate coverage reports
-- **Rust tests**: Use `pnpm test:backend` for Rust/Tauri backend tests
-
-### Testing Guidelines
-
-- Write tests for utility functions, stores, and business logic
-- For Svelte 5 components, focus on testing the underlying logic functions
-- Use descriptive test names that explain what behaviour is being tested
-- Include edge cases and error conditions in test coverage
-- Mock Tauri APIs using the patterns in `vitest.setup.ts`
-- **Coverage Goal**: Maintain as close to 100% test coverage as possible across the entire codebase
-
-### Mocking Strategies
-
-#### Console Mocking
-
-When testing code that intentionally logs errors (like error handling paths), mock console methods to prevent stderr output that makes tests appear flaky:
-
-```typescript
-it("handles errors gracefully", async () => {
-  const consoleErrorSpy = vi.spyOn(console, "error").mockImplementation(() => {});
-
-  // Test error handling code
-  await expect(functionThatLogs()).rejects.toThrow();
-
-  // Verify error was logged
-  expect(consoleErrorSpy).toHaveBeenCalledWith("Expected error:", expect.any(Error));
-
-  // Restore console.error
-  consoleErrorSpy.mockRestore();
-});
-```
-
-#### E2E Integration Testing for Cross-Platform Code
-
-For code that calls platform-specific system APIs (like Windows PowerShell or Linux notify-send), use helper functions that build the command structure without execution. This allows CI to verify cross-platform compatibility on Linux-only containers:
-
-```rust
-/// Build notify-send command for testing (doesn't execute)
-#[cfg(test)]
-fn build_notify_send_command(title: &str, body: &str) -> (String, Vec<String>) {
-    (
-        "notify-send".to_string(),
-        vec![
-            title.to_string(),
-            body.to_string(),
-            "--urgency=normal".to_string(),
-            "--app-name=Hikari Desktop".to_string(),
-        ],
-    )
-}
-
-#[test]
-fn test_e2e_notify_send_command_structure() {
-    let (command, args) = build_notify_send_command("Test Title", "Test Body");
-
-    assert_eq!(command, "notify-send");
-    assert_eq!(args.len(), 4);
-    assert_eq!(args[0], "Test Title");
-    assert_eq!(args[1], "Test Body");
-}
-```
-
-This approach:
-
-- Verifies command structure, argument order, and escaping logic
-- Tests cross-platform code paths without requiring the target platform
-- Allows CI to catch regressions in Windows-specific code whilst running on Linux
-- Keeps tests fast and deterministic (no actual system calls)
-
-### Example Test Structure
-
-```typescript
-import { describe, it, expect } from "vitest";
-
-describe("FeatureName", () => {
-  it("handles the normal case correctly", () => {
-    // Arrange
-    const input = "test data";
-
-    // Act
-    const result = functionUnderTest(input);
-
-    // Assert
-    expect(result).toBe("expected output");
-  });
-
-  it("handles edge cases gracefully", () => {
-    // Test edge cases...
-  });
-});
-```
-
-### Adding Tests for All Changes
-
-Every change — features, bug fixes, refactors — must include tests:
-
-1. **Before implementing**: Consider what needs testing (happy path, edge cases, errors)
-2. **During implementation**: Write tests alongside the code
-3. **After implementation**: Run `pnpm test:coverage` to verify coverage remains high
-4. **Before committing**: Ensure `check-all.sh` passes (includes all tests)
-
-**Do not commit changes without tests.** The goal is to maintain near-100% coverage as the codebase grows, so future refactoring can be done with confidence!
-
-## Quality Assurance
-
-Before committing any changes, **always run the full test suite**:
-
-```bash
-./check-all.sh
-```
-
-This script runs all checks in the correct order:
-
-1. Frontend linting (ESLint)
-2. Frontend formatting (Prettier)
-3. Frontend type checking (svelte-check)
-4. Frontend tests with coverage (Vitest)
-5. Backend linting (Clippy with strict rules)
-6. Backend tests with coverage (cargo test + llvm-cov)
-
-**Important**: The script requires Node.js and Rust toolchains to be available:
-
-- **Node.js tools** (pnpm, npm): Source nvm first if needed: `source ~/.nvm/nvm.sh`
-- **Rust tools** (cargo, clippy): Should be in PATH via `~/.cargo/bin/`
-
-If `check-all.sh` reports any failures:
-
-1. Read the error messages carefully - they usually explain what needs fixing
-2. Fix the issues (linting errors, test failures, etc.)
-3. Run `check-all.sh` again to verify the fixes
-4. Only commit once all checks pass ✨
-
-**Never commit code that doesn't pass `check-all.sh`** - this ensures code quality and prevents broken builds!
-
-## Project Context
-
-Hikari Desktop is a Tauri-based desktop application that wraps Claude Code with a visual anime character (Hikari) who appears on screen. This is a personal project where Hikari can sign her work and act as herself!

README.md

@@ -1,29 +1 @@
-# hikari-desktop
-
-Desktop companion application featuring Hikari.
-
-## Live Version
-
-This page is currently deployed. [View the live website.](https://git.nhcarrigan.com/nhcarrigan/hikari-desktop/releases)
-
-## Feedback and Bugs
-
-If you have feedback or a bug report, please [log a ticket on our forum](https://support.nhcarrigan.com).
-
-## Contributing
-
-If you would like to contribute to the project, you may create a Pull Request containing your proposed changes and we will review it as soon as we are able! Please review our [contributing guidelines](CONTRIBUTING.md) first.
-
-## Code of Conduct
-
-Before interacting with our community, please read our [Code of Conduct](CODE_OF_CONDUCT.md).
-
-## License
-
-This software is licensed under our [global software license](https://docs.nhcarrigan.com/#/license).
-
-Copyright held by Naomi Carrigan.
-
-## Contact
-
-We may be contacted through our [Chat Server](http://chat.nhcarrigan.com) or via email at `contact@nhcarrigan.com`
+tem

RELEASE_NOTES_v0.3.0.md

@@ -0,0 +1,74 @@
+# Hikari Desktop v0.3.0 Release Notes
+
+## New Features
+
+### AskUserQuestion Tool Support (#51)
+
+- Claude can now ask you questions with multiple choice options during conversations
+- A dedicated modal appears with answer choices and support for custom responses
+- Answers are seamlessly integrated back into the conversation context
+
+### Slash Commands
+
+- `/cd <path>` - Change the working directory while preserving conversation context (#55)
+- `/search <query>` - Search and highlight matches within the conversation (#32)
+- `/skill <name> [data]` - Invoke Claude Code skills from `~/.claude/skills/` (#57)
+- `/new`, `/clear`, `/rename`, `/help` commands for conversation management (#6)
+
+### Auto-Update Checker (#17)
+
+- Automatically checks for new releases on startup
+- Notification appears when a newer version is available
+- Can be disabled in settings
+
+### Font Size & Zoom (#19)
+
+- Adjust font size with keyboard shortcuts: `Ctrl++`, `Ctrl+-`, `Ctrl+0`
+- Font size slider in settings (10-24px range)
+- Preference persists between sessions
+
+### Resizable Character Panel (#10)
+
+- Drag the divider between the character panel and terminal to resize
+- Panel width is saved and restored on app restart
+- Sprite now uses full height for better proportions
+
+### Input History Navigation (#13)
+
+- Use up/down arrows to navigate through previous messages and commands
+- Arrow keys only navigate history when input is empty - otherwise they move the cursor (#58)
+
+### Keyboard Shortcuts (#21)
+
+- `Ctrl+N` - New conversation
+- `Ctrl+W` - Close current tab
+- `Ctrl+Tab` / `Ctrl+Shift+Tab` - Switch between tabs
+- `Ctrl+L` - Clear conversation
+- `Ctrl+,` - Open settings
+
+### Always On Top Toggle (#28)
+
+- Pin the window to stay above other applications
+- Toggle in settings
+
+## Improvements
+
+### UI/UX
+
+- Resizable chat input with drag handle (expands upward)
+- Send button properly aligned with input field
+- Markdown rendering with syntax-highlighted code blocks (#31, #33)
+- Light mode text colors improved for better readability
+- Scroll position persists per conversation tab when switching
+- Confirmation modal when closing connected tabs
+- Links in chat now open in default browser (#54)
+- Spaces allowed when renaming tabs (#52)
+
+### State Management
+
+- Stats (tokens, cost) persist across session changes and only reset on disconnect (#59)
+- Multiple tabs can now request permissions simultaneously without conflicts
+
+## Closed Issues
+
+#6, #10, #13, #17, #19, #21, #28, #31, #32, #33, #51, #52, #54, #55, #57, #58, #59

check-all.sh

@@ -1,9 +1,5 @@
 #!/bin/bash
 
-# Source nvm to get access to pnpm
-export NVM_DIR="$HOME/.nvm"
-[ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"
-
 # Colors for output
 RED='\033[0;31m'
 GREEN='\033[0;32m'
@@ -36,11 +32,11 @@ echo -e "${YELLOW}🔍 Running all checks for Hikari Desktop...${NC}"
 run_check "Frontend lint" "pnpm lint" || failed=1
 run_check "Frontend format check" "pnpm format:check" || failed=1
 run_check "Frontend type check" "pnpm check" || failed=1
-run_check "Frontend tests with coverage" "pnpm test:coverage" || failed=1
+run_check "Frontend tests" "pnpm test" || failed=1
 
 # Backend checks
-run_check "Backend clippy (strict)" "(cd src-tauri && cargo clippy --all-targets --all-features -- -D warnings)" || failed=1
-run_check "Backend tests with coverage" "(cd src-tauri && cargo llvm-cov --fail-under-lines 50)" || failed=1
+run_check "Backend clippy (strict)" "cd src-tauri && cargo clippy --all-targets --all-features -- -D warnings" || failed=1
+run_check "Backend tests" "cargo test" || failed=1
 
 # Summary
 echo -e "\n${YELLOW}========================================${NC}"

eslint.config.js

@@ -27,6 +27,6 @@ export default tseslint.config(
     },
   },
   {
-    ignores: ["build/", ".svelte-kit/", "dist/", "src-tauri/target/", "node_modules/", "coverage/"],
+    ignores: ["build/", ".svelte-kit/", "dist/", "src-tauri/target/", "node_modules/"],
   }
 );

package.json

@@ -1,6 +1,6 @@
 {
   "name": "hikari-desktop",
-  "version": "1.10.0",
+  "version": "0.3.0",
   "description": "",
   "type": "module",
   "scripts": {
@@ -16,10 +16,6 @@
     "test": "vitest run",
     "test:watch": "vitest",
     "test:coverage": "vitest run --coverage",
-    "test:backend": "cd src-tauri && cargo test",
-    "test:backend:coverage": "cd src-tauri && cargo llvm-cov --text",
-    "test:all": "pnpm test && pnpm test:backend",
-    "coverage:all": "pnpm test:coverage && pnpm test:backend:coverage",
     "lint": "eslint .",
     "lint:fix": "eslint . --fix",
     "format": "prettier --write .",
@@ -27,69 +23,40 @@
   },
   "license": "MIT",
   "dependencies": {
-    "@codemirror/commands": "6.10.2",
-    "@codemirror/lang-angular": "0.1.4",
-    "@codemirror/lang-cpp": "6.0.3",
-    "@codemirror/lang-css": "6.3.1",
-    "@codemirror/lang-go": "6.0.1",
-    "@codemirror/lang-html": "6.4.11",
-    "@codemirror/lang-java": "6.0.2",
-    "@codemirror/lang-javascript": "6.2.4",
-    "@codemirror/lang-json": "6.0.2",
-    "@codemirror/lang-less": "6.0.2",
-    "@codemirror/lang-markdown": "6.5.0",
-    "@codemirror/lang-php": "6.0.2",
-    "@codemirror/lang-python": "6.2.1",
-    "@codemirror/lang-rust": "6.0.2",
-    "@codemirror/lang-sass": "6.0.2",
-    "@codemirror/lang-sql": "6.10.0",
-    "@codemirror/lang-vue": "0.1.3",
-    "@codemirror/lang-wast": "6.0.2",
-    "@codemirror/lang-xml": "6.1.0",
-    "@codemirror/lang-yaml": "6.1.2",
-    "@codemirror/language": "6.12.2",
-    "@codemirror/legacy-modes": "6.5.2",
-    "@codemirror/state": "6.5.4",
-    "@codemirror/theme-one-dark": "6.1.3",
-    "@codemirror/view": "6.39.15",
-    "@lezer/highlight": "1.2.3",
-    "@tauri-apps/api": "2.10.1",
-    "@tauri-apps/plugin-clipboard-manager": "2.3.2",
-    "@tauri-apps/plugin-dialog": "2.6.0",
-    "@tauri-apps/plugin-fs": "2.4.5",
-    "@tauri-apps/plugin-notification": "2.3.3",
-    "@tauri-apps/plugin-opener": "2.5.3",
-    "@tauri-apps/plugin-os": "2.3.2",
-    "@tauri-apps/plugin-shell": "2.3.5",
-    "@tauri-apps/plugin-store": "2.4.2",
-    "codemirror": "6.0.2",
-    "highlight.js": "11.11.1",
-    "lucide-svelte": "0.575.0",
-    "marked": "17.0.3"
+    "@tauri-apps/api": "^2",
+    "@tauri-apps/plugin-clipboard-manager": "^2.3.2",
+    "@tauri-apps/plugin-dialog": "^2",
+    "@tauri-apps/plugin-fs": "^2.4.5",
+    "@tauri-apps/plugin-notification": "^2",
+    "@tauri-apps/plugin-opener": "^2",
+    "@tauri-apps/plugin-os": "^2",
+    "@tauri-apps/plugin-shell": "^2.3.4",
+    "@tauri-apps/plugin-store": "^2",
+    "highlight.js": "^11.11.1",
+    "marked": "^17.0.1"
   },
   "devDependencies": {
-    "@eslint/js": "9.39.3",
-    "@sveltejs/adapter-static": "3.0.10",
-    "@sveltejs/kit": "2.53.2",
-    "@sveltejs/vite-plugin-svelte": "5.1.1",
-    "@tailwindcss/vite": "4.2.1",
-    "@tauri-apps/cli": "2.10.0",
-    "@testing-library/jest-dom": "6.9.1",
-    "@testing-library/svelte": "5.3.1",
-    "@vitest/coverage-v8": "4.0.18",
-    "eslint": "9.39.3",
-    "eslint-config-prettier": "10.1.8",
-    "eslint-plugin-svelte": "3.15.0",
-    "globals": "17.3.0",
-    "jsdom": "28.1.0",
-    "prettier": "3.8.1",
-    "prettier-plugin-svelte": "3.5.0",
-    "svelte": "5.53.5",
-    "svelte-check": "4.4.3",
-    "tailwindcss": "4.2.1",
-    "typescript": "5.9.3",
-    "typescript-eslint": "8.56.1",
-    "vite": "6.4.1",
-    "vitest": "4.0.18"
+    "@eslint/js": "^9.39.2",
+    "@sveltejs/adapter-static": "^3.0.6",
+    "@sveltejs/kit": "^2.9.0",
+    "@sveltejs/vite-plugin-svelte": "^5.0.0",
+    "@tailwindcss/vite": "^4.1.18",
+    "@tauri-apps/cli": "^2",
+    "@testing-library/jest-dom": "^6.9.1",
+    "@testing-library/svelte": "^5.3.1",
+    "eslint": "^9.39.2",
+    "eslint-config-prettier": "^10.1.8",
+    "eslint-plugin-svelte": "^3.14.0",
+    "globals": "^17.0.0",
+    "jsdom": "^27.4.0",
+    "prettier": "^3.8.0",
+    "prettier-plugin-svelte": "^3.4.1",
+    "svelte": "^5.0.0",
+    "svelte-check": "^4.0.0",
+    "tailwindcss": "^4.1.18",
+    "typescript": "~5.6.2",
+    "typescript-eslint": "^8.53.0",
+    "vite": "^6.0.3",
+    "vitest": "^4.0.17"
   }
 }

src-tauri/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "hikari-desktop"
-version = "1.10.0"
+version = "0.3.0"
 description = "Hikari - Claude Code Visual Assistant"
 authors = ["Naomi Carrigan"]
 edition = "2021"
@@ -31,10 +31,6 @@ tauri-plugin-fs = "2"
 tempfile = "3"
 semver = "1"
 chrono = { version = "0.4.43", features = ["serde"] }
-discord-rich-presence = "0.2"
-dirs = "5"
-tracing = "0.1"
-tracing-subscriber = { version = "0.3", features = ["env-filter", "fmt"] }
 
 [target.'cfg(windows)'.dependencies]
 windows = { version = "0.62", features = [

src-tauri/capabilities/default.json

@@ -28,17 +28,8 @@
       "identifier": "fs:allow-write-file",
       "allow": [{ "path": "**" }]
     },
-    {
-      "identifier": "fs:scope",
-      "allow": [{ "path": "$HOME/.claude/**" }]
-    },
-    {
-      "identifier": "fs:allow-read-text-file",
-      "allow": [{ "path": "$HOME/.claude/**" }]
-    },
     "core:window:allow-set-size",
     "core:window:allow-set-always-on-top",
-    "core:window:allow-inner-size",
-    "core:window:allow-hide"
+    "core:window:allow-inner-size"
   ]
 }

src-tauri/src/bridge_manager.rs

@@ -3,7 +3,6 @@ use std::collections::HashMap;
 use std::sync::Arc;
 use tauri::AppHandle;
 
-use crate::commands::record_session;
 use crate::config::ClaudeStartOptions;
 use crate::stats::UsageStats;
 use crate::wsl_bridge::WslBridge;
@@ -54,12 +53,7 @@ impl BridgeManager {
             .or_insert_with(|| WslBridge::new_with_conversation_id(conversation_id.to_string()));
 
         // Start the Claude process
-        bridge.start(app.clone(), options)?;
-
-        // Record session start for cost tracking
-        tauri::async_runtime::spawn(async move {
-            record_session(&app).await;
-        });
+        bridge.start(app, options)?;
 
         Ok(())
     }
@@ -173,127 +167,3 @@ pub type SharedBridgeManager = Arc<Mutex<BridgeManager>>;
 pub fn create_shared_bridge_manager() -> SharedBridgeManager {
     Arc::new(Mutex::new(BridgeManager::new()))
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    #[test]
-    fn test_bridge_manager_new() {
-        let manager = BridgeManager::new();
-        assert!(manager.app_handle.is_none());
-        assert!(manager.bridges.is_empty());
-    }
-
-    #[test]
-    fn test_bridge_manager_default() {
-        let manager = BridgeManager::default();
-        assert!(manager.app_handle.is_none());
-        assert!(manager.bridges.is_empty());
-    }
-
-    #[test]
-    fn test_is_claude_running_no_bridge() {
-        let manager = BridgeManager::new();
-        assert!(!manager.is_claude_running("nonexistent"));
-    }
-
-    #[test]
-    fn test_get_working_directory_no_bridge() {
-        let manager = BridgeManager::new();
-        let result = manager.get_working_directory("nonexistent");
-        assert!(result.is_err());
-        assert_eq!(
-            result.unwrap_err(),
-            "No Claude instance found for this conversation"
-        );
-    }
-
-    #[test]
-    fn test_get_usage_stats_no_bridge() {
-        let manager = BridgeManager::new();
-        let result = manager.get_usage_stats("nonexistent");
-        assert!(result.is_err());
-        assert_eq!(
-            result.unwrap_err(),
-            "No Claude instance found for this conversation"
-        );
-    }
-
-    #[test]
-    fn test_stop_claude_no_bridge() {
-        let mut manager = BridgeManager::new();
-        let result = manager.stop_claude("nonexistent");
-        assert!(result.is_err());
-        assert_eq!(
-            result.unwrap_err(),
-            "No Claude instance found for this conversation"
-        );
-    }
-
-    #[test]
-    fn test_interrupt_claude_no_bridge() {
-        let mut manager = BridgeManager::new();
-        let result = manager.interrupt_claude("nonexistent");
-        assert!(result.is_err());
-        assert_eq!(
-            result.unwrap_err(),
-            "No Claude instance found for this conversation"
-        );
-    }
-
-    #[test]
-    fn test_send_prompt_no_bridge() {
-        let mut manager = BridgeManager::new();
-        let result = manager.send_prompt("nonexistent", "Hello".to_string());
-        assert!(result.is_err());
-        assert_eq!(
-            result.unwrap_err(),
-            "No Claude instance found for this conversation"
-        );
-    }
-
-    #[test]
-    fn test_send_tool_result_no_bridge() {
-        let mut manager = BridgeManager::new();
-        let result = manager.send_tool_result(
-            "nonexistent",
-            "tool_id",
-            serde_json::json!({"result": "success"}),
-        );
-        assert!(result.is_err());
-        assert_eq!(
-            result.unwrap_err(),
-            "No Claude instance found for this conversation"
-        );
-    }
-
-    #[test]
-    fn test_create_shared_bridge_manager() {
-        let shared = create_shared_bridge_manager();
-        let manager = shared.lock();
-        assert!(manager.bridges.is_empty());
-        assert!(manager.app_handle.is_none());
-    }
-
-    #[test]
-    fn test_cleanup_stopped_bridges_empty() {
-        let mut manager = BridgeManager::new();
-        manager.cleanup_stopped_bridges();
-        assert!(manager.bridges.is_empty());
-    }
-
-    #[test]
-    fn test_get_active_conversations_empty() {
-        let manager = BridgeManager::new();
-        let active = manager.get_active_conversations();
-        assert!(active.is_empty());
-    }
-
-    #[test]
-    fn test_stop_all_without_app_handle() {
-        let mut manager = BridgeManager::new();
-        manager.stop_all(); // Should not panic
-        assert!(manager.bridges.is_empty());
-    }
-}

src-tauri/src/clipboard.rs

@@ -257,468 +257,3 @@ pub fn update_clipboard_language(
     save_history(&app, &history)?;
     Ok(updated_entry)
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    // ==================== ClipboardEntry tests ====================
-
-    #[test]
-    fn test_clipboard_entry_new() {
-        let entry = ClipboardEntry::new(
-            "let x = 42;".to_string(),
-            Some("rust".to_string()),
-            Some("main.rs".to_string()),
-        );
-
-        assert_eq!(entry.content, "let x = 42;");
-        assert_eq!(entry.language, Some("rust".to_string()));
-        assert_eq!(entry.source, Some("main.rs".to_string()));
-        assert!(!entry.is_pinned);
-        assert!(!entry.id.is_empty());
-        assert!(!entry.timestamp.is_empty());
-    }
-
-    #[test]
-    fn test_clipboard_entry_new_without_optional_fields() {
-        let entry = ClipboardEntry::new("some content".to_string(), None, None);
-
-        assert_eq!(entry.content, "some content");
-        assert!(entry.language.is_none());
-        assert!(entry.source.is_none());
-        assert!(!entry.is_pinned);
-    }
-
-    #[test]
-    fn test_clipboard_entry_unique_ids() {
-        let entry1 = ClipboardEntry::new("content1".to_string(), None, None);
-        let entry2 = ClipboardEntry::new("content2".to_string(), None, None);
-
-        assert_ne!(entry1.id, entry2.id);
-    }
-
-    #[test]
-    fn test_clipboard_entry_serialization() {
-        let entry = ClipboardEntry::new(
-            "fn main() {}".to_string(),
-            Some("rust".to_string()),
-            Some("lib.rs".to_string()),
-        );
-
-        let json = serde_json::to_string(&entry).unwrap();
-        assert!(json.contains("fn main() {}"));
-        assert!(json.contains("rust"));
-        assert!(json.contains("lib.rs"));
-        assert!(json.contains("is_pinned"));
-
-        let deserialized: ClipboardEntry = serde_json::from_str(&json).unwrap();
-        assert_eq!(deserialized.content, entry.content);
-        assert_eq!(deserialized.language, entry.language);
-        assert_eq!(deserialized.source, entry.source);
-        assert_eq!(deserialized.id, entry.id);
-    }
-
-    #[test]
-    fn test_clipboard_entry_clone() {
-        let entry = ClipboardEntry::new(
-            "original".to_string(),
-            Some("python".to_string()),
-            None,
-        );
-
-        let cloned = entry.clone();
-        assert_eq!(cloned.content, entry.content);
-        assert_eq!(cloned.id, entry.id);
-        assert_eq!(cloned.language, entry.language);
-    }
-
-    #[test]
-    fn test_clipboard_entry_timestamp_is_rfc3339() {
-        let entry = ClipboardEntry::new("test".to_string(), None, None);
-
-        // RFC3339 timestamp should parse successfully
-        let parsed = chrono::DateTime::parse_from_rfc3339(&entry.timestamp);
-        assert!(parsed.is_ok());
-    }
-
-    // ==================== ClipboardHistory tests ====================
-
-    #[test]
-    fn test_clipboard_history_default() {
-        let history = ClipboardHistory::default();
-        assert!(history.entries.is_empty());
-    }
-
-    #[test]
-    fn test_clipboard_history_serialization() {
-        let mut history = ClipboardHistory::default();
-        history.entries.push(ClipboardEntry::new(
-            "entry1".to_string(),
-            Some("js".to_string()),
-            None,
-        ));
-        history.entries.push(ClipboardEntry::new(
-            "entry2".to_string(),
-            None,
-            Some("file.txt".to_string()),
-        ));
-
-        let json = serde_json::to_string(&history).unwrap();
-        assert!(json.contains("entry1"));
-        assert!(json.contains("entry2"));
-        assert!(json.contains("js"));
-        assert!(json.contains("file.txt"));
-
-        let deserialized: ClipboardHistory = serde_json::from_str(&json).unwrap();
-        assert_eq!(deserialized.entries.len(), 2);
-    }
-
-    #[test]
-    fn test_clipboard_history_entries_order() {
-        let mut history = ClipboardHistory::default();
-
-        history.entries.push(ClipboardEntry::new("first".to_string(), None, None));
-        history.entries.push(ClipboardEntry::new("second".to_string(), None, None));
-        history.entries.push(ClipboardEntry::new("third".to_string(), None, None));
-
-        assert_eq!(history.entries[0].content, "first");
-        assert_eq!(history.entries[1].content, "second");
-        assert_eq!(history.entries[2].content, "third");
-    }
-
-    // ==================== ClipboardState tests ====================
-
-    #[test]
-    fn test_clipboard_state_default() {
-        let state = ClipboardState::default();
-        assert!(state.last_content.is_none());
-    }
-
-    #[test]
-    fn test_clipboard_state_with_content() {
-        let state = ClipboardState {
-            last_content: Some("cached content".to_string()),
-        };
-        assert_eq!(state.last_content, Some("cached content".to_string()));
-    }
-
-    // ==================== MAX_HISTORY_SIZE constant test ====================
-
-    #[test]
-    fn test_max_history_size_is_reasonable() {
-        assert_eq!(MAX_HISTORY_SIZE, 100);
-        // Compile-time assertions for constant bounds
-        const _: () = assert!(MAX_HISTORY_SIZE > 0);
-        const _: () = assert!(MAX_HISTORY_SIZE <= 1000); // Sanity check
-    }
-
-    // ==================== Pinned entry sorting tests ====================
-
-    #[test]
-    #[allow(clippy::useless_vec)]
-    fn test_pinned_entries_sorting() {
-        let mut entries = vec![
-            ClipboardEntry {
-                id: "1".to_string(),
-                content: "unpinned older".to_string(),
-                language: None,
-                source: None,
-                timestamp: "2024-01-01T00:00:00Z".to_string(),
-                is_pinned: false,
-            },
-            ClipboardEntry {
-                id: "2".to_string(),
-                content: "pinned".to_string(),
-                language: None,
-                source: None,
-                timestamp: "2024-01-02T00:00:00Z".to_string(),
-                is_pinned: true,
-            },
-            ClipboardEntry {
-                id: "3".to_string(),
-                content: "unpinned newer".to_string(),
-                language: None,
-                source: None,
-                timestamp: "2024-01-03T00:00:00Z".to_string(),
-                is_pinned: false,
-            },
-        ];
-
-        // Apply the same sorting logic as used in the module
-        entries.sort_by(|a, b| {
-            if a.is_pinned && !b.is_pinned {
-                std::cmp::Ordering::Less
-            } else if !a.is_pinned && b.is_pinned {
-                std::cmp::Ordering::Greater
-            } else {
-                b.timestamp.cmp(&a.timestamp)
-            }
-        });
-
-        // Pinned should be first
-        assert!(entries[0].is_pinned);
-        assert_eq!(entries[0].id, "2");
-
-        // Then unpinned sorted by timestamp descending (newest first)
-        assert_eq!(entries[1].id, "3"); // newer unpinned
-        assert_eq!(entries[2].id, "1"); // older unpinned
-    }
-
-    #[test]
-    #[allow(clippy::useless_vec)]
-    fn test_multiple_pinned_entries_sorting() {
-        let mut entries = vec![
-            ClipboardEntry {
-                id: "1".to_string(),
-                content: "pinned older".to_string(),
-                language: None,
-                source: None,
-                timestamp: "2024-01-01T00:00:00Z".to_string(),
-                is_pinned: true,
-            },
-            ClipboardEntry {
-                id: "2".to_string(),
-                content: "unpinned".to_string(),
-                language: None,
-                source: None,
-                timestamp: "2024-01-02T00:00:00Z".to_string(),
-                is_pinned: false,
-            },
-            ClipboardEntry {
-                id: "3".to_string(),
-                content: "pinned newer".to_string(),
-                language: None,
-                source: None,
-                timestamp: "2024-01-03T00:00:00Z".to_string(),
-                is_pinned: true,
-            },
-        ];
-
-        entries.sort_by(|a, b| {
-            if a.is_pinned && !b.is_pinned {
-                std::cmp::Ordering::Less
-            } else if !a.is_pinned && b.is_pinned {
-                std::cmp::Ordering::Greater
-            } else {
-                b.timestamp.cmp(&a.timestamp)
-            }
-        });
-
-        // Both pinned first, sorted by timestamp
-        assert!(entries[0].is_pinned);
-        assert_eq!(entries[0].id, "3"); // pinned newer
-        assert!(entries[1].is_pinned);
-        assert_eq!(entries[1].id, "1"); // pinned older
-        // Then unpinned
-        assert!(!entries[2].is_pinned);
-        assert_eq!(entries[2].id, "2");
-    }
-
-    // ==================== Entry filtering tests ====================
-
-    #[test]
-    fn test_filter_entries_by_language() {
-        let history = ClipboardHistory {
-            entries: vec![
-                ClipboardEntry {
-                    id: "1".to_string(),
-                    content: "rust code".to_string(),
-                    language: Some("rust".to_string()),
-                    source: None,
-                    timestamp: "2024-01-01T00:00:00Z".to_string(),
-                    is_pinned: false,
-                },
-                ClipboardEntry {
-                    id: "2".to_string(),
-                    content: "js code".to_string(),
-                    language: Some("javascript".to_string()),
-                    source: None,
-                    timestamp: "2024-01-02T00:00:00Z".to_string(),
-                    is_pinned: false,
-                },
-                ClipboardEntry {
-                    id: "3".to_string(),
-                    content: "more rust".to_string(),
-                    language: Some("rust".to_string()),
-                    source: None,
-                    timestamp: "2024-01-03T00:00:00Z".to_string(),
-                    is_pinned: false,
-                },
-            ],
-        };
-
-        let filtered: Vec<_> = history
-            .entries
-            .iter()
-            .filter(|e| e.language.as_ref() == Some(&"rust".to_string()))
-            .collect();
-
-        assert_eq!(filtered.len(), 2);
-        assert!(filtered.iter().all(|e| e.language == Some("rust".to_string())));
-    }
-
-    #[test]
-    fn test_search_entries_by_content() {
-        let history = ClipboardHistory {
-            entries: vec![
-                ClipboardEntry {
-                    id: "1".to_string(),
-                    content: "fn hello_world()".to_string(),
-                    language: Some("rust".to_string()),
-                    source: None,
-                    timestamp: "2024-01-01T00:00:00Z".to_string(),
-                    is_pinned: false,
-                },
-                ClipboardEntry {
-                    id: "2".to_string(),
-                    content: "function hello()".to_string(),
-                    language: Some("javascript".to_string()),
-                    source: None,
-                    timestamp: "2024-01-02T00:00:00Z".to_string(),
-                    is_pinned: false,
-                },
-                ClipboardEntry {
-                    id: "3".to_string(),
-                    content: "def goodbye()".to_string(),
-                    language: Some("python".to_string()),
-                    source: None,
-                    timestamp: "2024-01-03T00:00:00Z".to_string(),
-                    is_pinned: false,
-                },
-            ],
-        };
-
-        let query = "hello";
-        let query_lower = query.to_lowercase();
-        let filtered: Vec<_> = history
-            .entries
-            .iter()
-            .filter(|e| e.content.to_lowercase().contains(&query_lower))
-            .collect();
-
-        assert_eq!(filtered.len(), 2);
-        assert!(filtered[0].content.contains("hello"));
-        assert!(filtered[1].content.contains("hello"));
-    }
-
-    #[test]
-    fn test_search_entries_case_insensitive() {
-        let history = ClipboardHistory {
-            entries: vec![
-                ClipboardEntry {
-                    id: "1".to_string(),
-                    content: "HELLO WORLD".to_string(),
-                    language: None,
-                    source: None,
-                    timestamp: "2024-01-01T00:00:00Z".to_string(),
-                    is_pinned: false,
-                },
-            ],
-        };
-
-        let query = "hello";
-        let query_lower = query.to_lowercase();
-        let filtered: Vec<_> = history
-            .entries
-            .iter()
-            .filter(|e| e.content.to_lowercase().contains(&query_lower))
-            .collect();
-
-        assert_eq!(filtered.len(), 1);
-    }
-
-    // ==================== Unique languages extraction test ====================
-
-    #[test]
-    fn test_extract_unique_languages() {
-        let history = ClipboardHistory {
-            entries: vec![
-                ClipboardEntry {
-                    id: "1".to_string(),
-                    content: "".to_string(),
-                    language: Some("rust".to_string()),
-                    source: None,
-                    timestamp: "".to_string(),
-                    is_pinned: false,
-                },
-                ClipboardEntry {
-                    id: "2".to_string(),
-                    content: "".to_string(),
-                    language: Some("javascript".to_string()),
-                    source: None,
-                    timestamp: "".to_string(),
-                    is_pinned: false,
-                },
-                ClipboardEntry {
-                    id: "3".to_string(),
-                    content: "".to_string(),
-                    language: Some("rust".to_string()), // Duplicate
-                    source: None,
-                    timestamp: "".to_string(),
-                    is_pinned: false,
-                },
-                ClipboardEntry {
-                    id: "4".to_string(),
-                    content: "".to_string(),
-                    language: None, // No language
-                    source: None,
-                    timestamp: "".to_string(),
-                    is_pinned: false,
-                },
-            ],
-        };
-
-        let mut languages: Vec<String> = history
-            .entries
-            .iter()
-            .filter_map(|e| e.language.clone())
-            .collect();
-        languages.sort();
-        languages.dedup();
-
-        assert_eq!(languages.len(), 2);
-        assert!(languages.contains(&"rust".to_string()));
-        assert!(languages.contains(&"javascript".to_string()));
-    }
-
-    // ==================== Retain pinned entries test ====================
-
-    #[test]
-    fn test_retain_pinned_on_clear() {
-        let mut history = ClipboardHistory {
-            entries: vec![
-                ClipboardEntry {
-                    id: "1".to_string(),
-                    content: "pinned".to_string(),
-                    language: None,
-                    source: None,
-                    timestamp: "".to_string(),
-                    is_pinned: true,
-                },
-                ClipboardEntry {
-                    id: "2".to_string(),
-                    content: "unpinned".to_string(),
-                    language: None,
-                    source: None,
-                    timestamp: "".to_string(),
-                    is_pinned: false,
-                },
-                ClipboardEntry {
-                    id: "3".to_string(),
-                    content: "another pinned".to_string(),
-                    language: None,
-                    source: None,
-                    timestamp: "".to_string(),
-                    is_pinned: true,
-                },
-            ],
-        };
-
-        // Simulate clear (keep only pinned)
-        history.entries.retain(|e| e.is_pinned);
-
-        assert_eq!(history.entries.len(), 2);
-        assert!(history.entries.iter().all(|e| e.is_pinned));
-    }
-}

src-tauri/src/config.rs

@@ -25,19 +25,9 @@ pub struct ClaudeStartOptions {
 
     #[serde(default)]
     pub resume_session_id: Option<String>,
-
-    #[serde(default)]
-    pub use_worktree: bool,
-
-    #[serde(default)]
-    pub disable_1m_context: bool,
-
-    #[serde(default)]
-    pub max_output_tokens: Option<u64>,
 }
 
 #[derive(Debug, Clone, Serialize, Deserialize)]
-#[serde(default)]
 pub struct HikariConfig {
     #[serde(default)]
     pub model: Option<String>,
@@ -81,6 +71,9 @@ pub struct HikariConfig {
     #[serde(default = "default_font_size")]
     pub font_size: u32,
 
+    #[serde(default)]
+    pub minimize_to_tray: bool,
+
     #[serde(default)]
     pub streamer_mode: bool,
 
@@ -99,65 +92,6 @@ pub struct HikariConfig {
 
     #[serde(default)]
     pub profile_bio: Option<String>,
-
-    // Custom theme colors
-    #[serde(default)]
-    pub custom_theme_colors: CustomThemeColors,
-
-    // Token budget settings
-    #[serde(default)]
-    pub budget_enabled: bool,
-
-    #[serde(default)]
-    pub session_token_budget: Option<u64>,
-
-    #[serde(default)]
-    pub session_cost_budget: Option<f64>,
-
-    #[serde(default = "default_budget_action")]
-    pub budget_action: BudgetAction,
-
-    #[serde(default = "default_budget_warning_threshold")]
-    pub budget_warning_threshold: f32,
-
-    #[serde(default = "default_discord_rpc_enabled")]
-    pub discord_rpc_enabled: bool,
-
-    #[serde(default)]
-    pub use_worktree: bool,
-
-    #[serde(default)]
-    pub disable_1m_context: bool,
-
-    #[serde(default)]
-    pub max_output_tokens: Option<u64>,
-
-    #[serde(default)]
-    pub trusted_workspaces: Vec<String>,
-
-    // Background image settings
-    #[serde(default)]
-    pub background_image_path: Option<String>,
-
-    #[serde(default = "default_background_image_opacity")]
-    pub background_image_opacity: f32,
-
-    #[serde(default)]
-    pub show_thinking_blocks: bool,
-
-    // Custom terminal font settings
-    #[serde(default)]
-    pub custom_font_path: Option<String>,
-
-    #[serde(default)]
-    pub custom_font_family: Option<String>,
-
-    // Custom UI font settings
-    #[serde(default)]
-    pub custom_ui_font_path: Option<String>,
-
-    #[serde(default)]
-    pub custom_ui_font_family: Option<String>,
 }
 
 impl Default for HikariConfig {
@@ -177,30 +111,13 @@ impl Default for HikariConfig {
             update_checks_enabled: true,
             character_panel_width: None,
             font_size: 14,
+            minimize_to_tray: false,
             streamer_mode: false,
             streamer_hide_paths: false,
             compact_mode: false,
             profile_name: None,
             profile_avatar_path: None,
             profile_bio: None,
-            custom_theme_colors: CustomThemeColors::default(),
-            budget_enabled: false,
-            session_token_budget: None,
-            session_cost_budget: None,
-            budget_action: BudgetAction::Warn,
-            budget_warning_threshold: 0.8,
-            discord_rpc_enabled: true,
-            use_worktree: false,
-            disable_1m_context: false,
-            max_output_tokens: None,
-            trusted_workspaces: Vec::new(),
-            background_image_path: None,
-            background_image_opacity: 0.3,
-            show_thinking_blocks: false,
-            custom_font_path: None,
-            custom_font_family: None,
-            custom_ui_font_path: None,
-            custom_ui_font_family: None,
         }
     }
 }
@@ -225,30 +142,6 @@ fn default_font_size() -> u32 {
     14
 }
 
-fn default_budget_action() -> BudgetAction {
-    BudgetAction::Warn
-}
-
-fn default_budget_warning_threshold() -> f32 {
-    0.8
-}
-
-fn default_discord_rpc_enabled() -> bool {
-    true
-}
-
-fn default_background_image_opacity() -> f32 {
-    0.3
-}
-
-#[derive(Debug, Clone, Serialize, Deserialize, Default, PartialEq)]
-#[serde(rename_all = "lowercase")]
-pub enum BudgetAction {
-    #[default]
-    Warn,
-    Block,
-}
-
 #[derive(Debug, Clone, Serialize, Deserialize, Default, PartialEq)]
 #[serde(rename_all = "lowercase")]
 pub enum Theme {
@@ -257,27 +150,6 @@ pub enum Theme {
     Light,
     #[serde(rename = "high-contrast")]
     HighContrast,
-    Custom,
-}
-
-#[derive(Debug, Clone, Serialize, Deserialize, Default, PartialEq)]
-pub struct CustomThemeColors {
-    #[serde(default)]
-    pub bg_primary: Option<String>,
-    #[serde(default)]
-    pub bg_secondary: Option<String>,
-    #[serde(default)]
-    pub bg_terminal: Option<String>,
-    #[serde(default)]
-    pub accent_primary: Option<String>,
-    #[serde(default)]
-    pub accent_secondary: Option<String>,
-    #[serde(default)]
-    pub text_primary: Option<String>,
-    #[serde(default)]
-    pub text_secondary: Option<String>,
-    #[serde(default)]
-    pub border_color: Option<String>,
 }
 
 #[cfg(test)]
@@ -299,27 +171,13 @@ mod tests {
         assert!(config.update_checks_enabled);
         assert!(config.character_panel_width.is_none());
         assert_eq!(config.font_size, 14);
+        assert!(!config.minimize_to_tray);
         assert!(!config.streamer_mode);
         assert!(!config.streamer_hide_paths);
         assert!(!config.compact_mode);
         assert!(config.profile_name.is_none());
         assert!(config.profile_avatar_path.is_none());
         assert!(config.profile_bio.is_none());
-        assert_eq!(config.custom_theme_colors, CustomThemeColors::default());
-        assert!(!config.budget_enabled);
-        assert!(config.session_token_budget.is_none());
-        assert!(config.session_cost_budget.is_none());
-        assert_eq!(config.budget_action, BudgetAction::Warn);
-        assert!((config.budget_warning_threshold - 0.8).abs() < f32::EPSILON);
-        assert!(config.discord_rpc_enabled);
-        assert!(!config.use_worktree);
-        assert!(!config.disable_1m_context);
-        assert!(config.trusted_workspaces.is_empty());
-        assert!(!config.show_thinking_blocks);
-        assert!(config.custom_font_path.is_none());
-        assert!(config.custom_font_family.is_none());
-        assert!(config.custom_ui_font_path.is_none());
-        assert!(config.custom_ui_font_family.is_none());
     }
 
     #[test]
@@ -339,30 +197,13 @@ mod tests {
             update_checks_enabled: true,
             character_panel_width: Some(400),
             font_size: 16,
+            minimize_to_tray: true,
             streamer_mode: false,
             streamer_hide_paths: false,
             compact_mode: false,
             profile_name: Some("Test User".to_string()),
             profile_avatar_path: None,
             profile_bio: Some("A test bio".to_string()),
-            custom_theme_colors: CustomThemeColors::default(),
-            budget_enabled: true,
-            session_token_budget: Some(100000),
-            session_cost_budget: Some(1.50),
-            budget_action: BudgetAction::Block,
-            budget_warning_threshold: 0.75,
-            discord_rpc_enabled: true,
-            use_worktree: true,
-            disable_1m_context: false,
-            max_output_tokens: Some(32000),
-            trusted_workspaces: vec!["/home/naomi/projects/trusted".to_string()],
-            background_image_path: Some("/home/naomi/bg.png".to_string()),
-            background_image_opacity: 0.25,
-            show_thinking_blocks: true,
-            custom_font_path: Some("/home/naomi/.fonts/MyFont.ttf".to_string()),
-            custom_font_family: Some("MyFont".to_string()),
-            custom_ui_font_path: None,
-            custom_ui_font_family: None,
         };
 
         let json = serde_json::to_string(&config).unwrap();
@@ -391,8 +232,5 @@ mod tests {
             serde_json::to_string(&high_contrast).unwrap(),
             "\"high-contrast\""
         );
-
-        let custom = Theme::Custom;
-        assert_eq!(serde_json::to_string(&custom).unwrap(), "\"custom\"");
     }
 }

src-tauri/src/cost_tracking.rs

@@ -1,376 +0,0 @@
-use chrono::{Datelike, Local, NaiveDate, Weekday};
-use serde::{Deserialize, Serialize};
-use std::collections::HashMap;
-
-/// Represents a single day's cost data
-#[derive(Debug, Clone, Serialize, Deserialize, Default)]
-pub struct DailyCost {
-    pub date: String, // ISO date string (YYYY-MM-DD)
-    pub input_tokens: u64,
-    pub output_tokens: u64,
-    pub cost_usd: f64,
-    pub messages_sent: u64,
-    pub sessions_count: u64,
-}
-
-/// Historical cost tracking data
-#[derive(Debug, Clone, Serialize, Deserialize, Default)]
-pub struct CostHistory {
-    /// Daily costs indexed by date string (YYYY-MM-DD)
-    pub daily_costs: HashMap<String, DailyCost>,
-    /// Cost alert thresholds
-    pub daily_alert_threshold: Option<f64>,
-    pub weekly_alert_threshold: Option<f64>,
-    pub monthly_alert_threshold: Option<f64>,
-    /// Whether alerts have been triggered today
-    pub daily_alert_triggered: bool,
-    pub weekly_alert_triggered: bool,
-    pub monthly_alert_triggered: bool,
-    pub last_alert_reset_date: Option<String>,
-}
-
-impl CostHistory {
-    pub fn new() -> Self {
-        Self::default()
-    }
-
-    /// Get today's date as a string
-    fn today_str() -> String {
-        Local::now().format("%Y-%m-%d").to_string()
-    }
-
-    /// Get the start of the current week (Monday)
-    fn week_start() -> NaiveDate {
-        let today = Local::now().date_naive();
-        let days_since_monday = today.weekday().num_days_from_monday();
-        today - chrono::Duration::days(days_since_monday as i64)
-    }
-
-    /// Get the start of the current month
-    fn month_start() -> NaiveDate {
-        let today = Local::now().date_naive();
-        NaiveDate::from_ymd_opt(today.year(), today.month(), 1).unwrap_or(today)
-    }
-
-    /// Add cost for today
-    pub fn add_cost(&mut self, input_tokens: u64, output_tokens: u64, cost_usd: f64) {
-        let today = Self::today_str();
-
-        // Reset alert flags if it's a new day
-        if self.last_alert_reset_date.as_ref() != Some(&today) {
-            self.daily_alert_triggered = false;
-            // Reset weekly on Monday
-            if Local::now().weekday() == Weekday::Mon {
-                self.weekly_alert_triggered = false;
-            }
-            // Reset monthly on the 1st
-            if Local::now().day() == 1 {
-                self.monthly_alert_triggered = false;
-            }
-            self.last_alert_reset_date = Some(today.clone());
-        }
-
-        let daily = self.daily_costs.entry(today).or_default();
-        daily.input_tokens += input_tokens;
-        daily.output_tokens += output_tokens;
-        daily.cost_usd += cost_usd;
-        daily.messages_sent += 1;
-    }
-
-    /// Increment session count for today
-    pub fn increment_sessions(&mut self) {
-        let today = Self::today_str();
-        let daily = self.daily_costs.entry(today.clone()).or_insert_with(|| DailyCost {
-            date: today,
-            ..Default::default()
-        });
-        daily.sessions_count += 1;
-    }
-
-    /// Get today's cost
-    pub fn get_today_cost(&self) -> f64 {
-        self.daily_costs
-            .get(&Self::today_str())
-            .map(|d| d.cost_usd)
-            .unwrap_or(0.0)
-    }
-
-    /// Get this week's cost (Monday to Sunday)
-    pub fn get_week_cost(&self) -> f64 {
-        let week_start = Self::week_start();
-        self.daily_costs
-            .values()
-            .filter(|d| {
-                NaiveDate::parse_from_str(&d.date, "%Y-%m-%d")
-                    .map(|date| date >= week_start)
-                    .unwrap_or(false)
-            })
-            .map(|d| d.cost_usd)
-            .sum()
-    }
-
-    /// Get this month's cost
-    pub fn get_month_cost(&self) -> f64 {
-        let month_start = Self::month_start();
-        self.daily_costs
-            .values()
-            .filter(|d| {
-                NaiveDate::parse_from_str(&d.date, "%Y-%m-%d")
-                    .map(|date| date >= month_start)
-                    .unwrap_or(false)
-            })
-            .map(|d| d.cost_usd)
-            .sum()
-    }
-
-    /// Get cost summary for a date range
-    pub fn get_summary(&self, days: u32) -> CostSummary {
-        let today = Local::now().date_naive();
-        let start_date = today - chrono::Duration::days(days as i64 - 1);
-
-        let mut total_input_tokens = 0u64;
-        let mut total_output_tokens = 0u64;
-        let mut total_cost = 0.0f64;
-        let mut total_messages = 0u64;
-        let mut total_sessions = 0u64;
-        let mut daily_breakdown = Vec::new();
-
-        for i in 0..days {
-            let date = start_date + chrono::Duration::days(i as i64);
-            let date_str = date.format("%Y-%m-%d").to_string();
-
-            if let Some(daily) = self.daily_costs.get(&date_str) {
-                total_input_tokens += daily.input_tokens;
-                total_output_tokens += daily.output_tokens;
-                total_cost += daily.cost_usd;
-                total_messages += daily.messages_sent;
-                total_sessions += daily.sessions_count;
-                daily_breakdown.push(daily.clone());
-            } else {
-                daily_breakdown.push(DailyCost {
-                    date: date_str,
-                    ..Default::default()
-                });
-            }
-        }
-
-        CostSummary {
-            period_days: days,
-            total_input_tokens,
-            total_output_tokens,
-            total_cost,
-            total_messages,
-            total_sessions,
-            average_daily_cost: if days > 0 { total_cost / days as f64 } else { 0.0 },
-            daily_breakdown,
-        }
-    }
-
-    /// Check if any alert thresholds are exceeded and return which ones
-    pub fn check_alerts(&mut self) -> Vec<CostAlert> {
-        let mut alerts = Vec::new();
-
-        if let Some(threshold) = self.daily_alert_threshold {
-            let today_cost = self.get_today_cost();
-            if today_cost >= threshold && !self.daily_alert_triggered {
-                self.daily_alert_triggered = true;
-                alerts.push(CostAlert {
-                    alert_type: AlertType::Daily,
-                    threshold,
-                    current_cost: today_cost,
-                });
-            }
-        }
-
-        if let Some(threshold) = self.weekly_alert_threshold {
-            let week_cost = self.get_week_cost();
-            if week_cost >= threshold && !self.weekly_alert_triggered {
-                self.weekly_alert_triggered = true;
-                alerts.push(CostAlert {
-                    alert_type: AlertType::Weekly,
-                    threshold,
-                    current_cost: week_cost,
-                });
-            }
-        }
-
-        if let Some(threshold) = self.monthly_alert_threshold {
-            let month_cost = self.get_month_cost();
-            if month_cost >= threshold && !self.monthly_alert_triggered {
-                self.monthly_alert_triggered = true;
-                alerts.push(CostAlert {
-                    alert_type: AlertType::Monthly,
-                    threshold,
-                    current_cost: month_cost,
-                });
-            }
-        }
-
-        alerts
-    }
-
-    /// Set alert thresholds
-    pub fn set_alert_thresholds(
-        &mut self,
-        daily: Option<f64>,
-        weekly: Option<f64>,
-        monthly: Option<f64>,
-    ) {
-        self.daily_alert_threshold = daily;
-        self.weekly_alert_threshold = weekly;
-        self.monthly_alert_threshold = monthly;
-    }
-
-    /// Clean up old data (keep last N days)
-    #[allow(dead_code)]
-    pub fn cleanup_old_data(&mut self, keep_days: u32) {
-        let cutoff = Local::now().date_naive() - chrono::Duration::days(keep_days as i64);
-        self.daily_costs.retain(|date_str, _| {
-            NaiveDate::parse_from_str(date_str, "%Y-%m-%d")
-                .map(|date| date >= cutoff)
-                .unwrap_or(false)
-        });
-    }
-
-    /// Export to CSV format
-    pub fn export_csv(&self, days: u32) -> String {
-        let summary = self.get_summary(days);
-        let mut csv = String::from("Date,Input Tokens,Output Tokens,Cost (USD),Messages,Sessions\n");
-
-        for daily in &summary.daily_breakdown {
-            csv.push_str(&format!(
-                "{},{},{},{:.4},{},{}\n",
-                daily.date,
-                daily.input_tokens,
-                daily.output_tokens,
-                daily.cost_usd,
-                daily.messages_sent,
-                daily.sessions_count
-            ));
-        }
-
-        // Add totals row
-        csv.push_str(&format!(
-            "TOTAL,{},{},{:.4},{},{}\n",
-            summary.total_input_tokens,
-            summary.total_output_tokens,
-            summary.total_cost,
-            summary.total_messages,
-            summary.total_sessions
-        ));
-
-        csv
-    }
-}
-
-/// Cost summary for a period
-#[derive(Debug, Clone, Serialize, Deserialize)]
-pub struct CostSummary {
-    pub period_days: u32,
-    pub total_input_tokens: u64,
-    pub total_output_tokens: u64,
-    pub total_cost: f64,
-    pub total_messages: u64,
-    pub total_sessions: u64,
-    pub average_daily_cost: f64,
-    pub daily_breakdown: Vec<DailyCost>,
-}
-
-/// Alert types
-#[derive(Debug, Clone, Serialize, Deserialize, PartialEq)]
-pub enum AlertType {
-    Daily,
-    Weekly,
-    Monthly,
-}
-
-/// Cost alert notification
-#[derive(Debug, Clone, Serialize, Deserialize)]
-pub struct CostAlert {
-    pub alert_type: AlertType,
-    pub threshold: f64,
-    pub current_cost: f64,
-}
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    #[test]
-    fn test_add_cost() {
-        let mut history = CostHistory::new();
-        history.add_cost(1000, 500, 0.05);
-
-        let today_cost = history.get_today_cost();
-        assert!((today_cost - 0.05).abs() < 0.0001);
-    }
-
-    #[test]
-    fn test_accumulate_daily_cost() {
-        let mut history = CostHistory::new();
-        history.add_cost(1000, 500, 0.05);
-        history.add_cost(2000, 1000, 0.10);
-
-        let today_cost = history.get_today_cost();
-        assert!((today_cost - 0.15).abs() < 0.0001);
-    }
-
-    #[test]
-    fn test_summary() {
-        let mut history = CostHistory::new();
-        history.add_cost(1000, 500, 0.05);
-
-        let summary = history.get_summary(7);
-        assert_eq!(summary.period_days, 7);
-        assert!((summary.total_cost - 0.05).abs() < 0.0001);
-    }
-
-    #[test]
-    fn test_daily_alert() {
-        let mut history = CostHistory::new();
-        history.set_alert_thresholds(Some(0.10), None, None);
-
-        history.add_cost(1000, 500, 0.05);
-        let alerts = history.check_alerts();
-        assert!(alerts.is_empty());
-
-        history.add_cost(1000, 500, 0.06);
-        let alerts = history.check_alerts();
-        assert_eq!(alerts.len(), 1);
-        assert_eq!(alerts[0].alert_type, AlertType::Daily);
-    }
-
-    #[test]
-    fn test_alert_only_triggers_once() {
-        let mut history = CostHistory::new();
-        history.set_alert_thresholds(Some(0.10), None, None);
-
-        history.add_cost(1000, 500, 0.15);
-        let alerts = history.check_alerts();
-        assert_eq!(alerts.len(), 1);
-
-        // Second check should not trigger again
-        let alerts = history.check_alerts();
-        assert!(alerts.is_empty());
-    }
-
-    #[test]
-    fn test_export_csv() {
-        let mut history = CostHistory::new();
-        history.add_cost(1000, 500, 0.05);
-
-        let csv = history.export_csv(1);
-        assert!(csv.contains("Date,Input Tokens"));
-        assert!(csv.contains("TOTAL"));
-    }
-
-    #[test]
-    fn test_increment_sessions() {
-        let mut history = CostHistory::new();
-        history.increment_sessions();
-        history.increment_sessions();
-
-        let summary = history.get_summary(1);
-        assert_eq!(summary.total_sessions, 2);
-    }
-}

src-tauri/src/debug_logger.rs

@@ -1,157 +0,0 @@
-use serde::{Deserialize, Serialize};
-use std::sync::Arc;
-use tauri::{AppHandle, Emitter};
-use tracing::{Level, Subscriber};
-use tracing_subscriber::layer::{Context, Layer};
-
-#[derive(Debug, Clone, Serialize, Deserialize)]
-pub struct DebugLogEvent {
-    pub level: String,
-    pub message: String,
-}
-
-#[derive(Clone)]
-pub struct TauriLogLayer {
-    app: Arc<AppHandle>,
-}
-
-impl TauriLogLayer {
-    pub fn new(app: AppHandle) -> Self {
-        Self {
-            app: Arc::new(app),
-        }
-    }
-}
-
-impl<S> Layer<S> for TauriLogLayer
-where
-    S: Subscriber,
-{
-    fn on_event(
-        &self,
-        event: &tracing::Event<'_>,
-        _ctx: Context<'_, S>,
-    ) {
-        let metadata = event.metadata();
-        let level = match *metadata.level() {
-            Level::ERROR => "error",
-            Level::WARN => "warn",
-            Level::INFO => "info",
-            Level::DEBUG => "debug",
-            Level::TRACE => "debug",
-        };
-
-        // Extract message from the event
-        struct MessageVisitor {
-            message: String,
-        }
-
-        impl tracing::field::Visit for MessageVisitor {
-            fn record_debug(&mut self, field: &tracing::field::Field, value: &dyn std::fmt::Debug) {
-                if field.name() == "message" {
-                    self.message = format!("{:?}", value);
-                }
-            }
-        }
-
-        let mut visitor = MessageVisitor {
-            message: String::new(),
-        };
-        event.record(&mut visitor);
-
-        // If we couldn't extract a message, try to format the whole event
-        if visitor.message.is_empty() {
-            visitor.message = metadata.name().to_string();
-        }
-
-        // Strip quotes from the message
-        let message = visitor.message.trim_matches('"').to_string();
-
-        let log_event = DebugLogEvent {
-            level: level.to_string(),
-            message,
-        };
-
-        // Emit to frontend
-        let _ = self.app.emit("debug:log", log_event);
-    }
-}
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    #[test]
-    fn test_debug_log_event_creation() {
-        let event = DebugLogEvent {
-            level: "info".to_string(),
-            message: "Test message".to_string(),
-        };
-
-        assert_eq!(event.level, "info");
-        assert_eq!(event.message, "Test message");
-    }
-
-    #[test]
-    fn test_debug_log_event_serialization() {
-        let event = DebugLogEvent {
-            level: "error".to_string(),
-            message: "Error occurred".to_string(),
-        };
-
-        let json = serde_json::to_string(&event).unwrap();
-        assert!(json.contains("\"level\":\"error\""));
-        assert!(json.contains("\"message\":\"Error occurred\""));
-    }
-
-    #[test]
-    fn test_debug_log_event_deserialization() {
-        let json = r#"{"level":"warn","message":"Warning message"}"#;
-        let event: DebugLogEvent = serde_json::from_str(json).unwrap();
-
-        assert_eq!(event.level, "warn");
-        assert_eq!(event.message, "Warning message");
-    }
-
-    #[test]
-    fn test_debug_log_event_with_special_characters() {
-        let event = DebugLogEvent {
-            level: "info".to_string(),
-            message: "Message with \"quotes\" and \n newlines".to_string(),
-        };
-
-        let json = serde_json::to_string(&event).unwrap();
-        let decoded: DebugLogEvent = serde_json::from_str(&json).unwrap();
-
-        assert_eq!(decoded.level, event.level);
-        assert_eq!(decoded.message, event.message);
-    }
-
-    #[test]
-    fn test_debug_log_event_with_unicode() {
-        let event = DebugLogEvent {
-            level: "debug".to_string(),
-            message: "Unicode: 日本語 🎉".to_string(),
-        };
-
-        let json = serde_json::to_string(&event).unwrap();
-        let decoded: DebugLogEvent = serde_json::from_str(&json).unwrap();
-
-        assert_eq!(decoded.message, "Unicode: 日本語 🎉");
-    }
-
-    #[test]
-    fn test_debug_log_event_all_levels() {
-        let levels = vec!["error", "warn", "info", "debug", "trace"];
-
-        for level in levels {
-            let event = DebugLogEvent {
-                level: level.to_string(),
-                message: format!("{} level message", level),
-            };
-
-            assert_eq!(event.level, level);
-            assert!(event.message.contains(level));
-        }
-    }
-}

src-tauri/src/discord_rpc.rs

@@ -1,178 +0,0 @@
-use discord_rich_presence::activity::{Activity, Assets, Timestamps};
-use discord_rich_presence::{DiscordIpc, DiscordIpcClient};
-use parking_lot::RwLock;
-use std::sync::Arc;
-
-pub struct DiscordRpcManager {
-    client: Arc<RwLock<Option<DiscordIpcClient>>>,
-    session_name: Arc<RwLock<String>>,
-    model: Arc<RwLock<String>>,
-    started_at: Arc<RwLock<i64>>,
-}
-
-impl DiscordRpcManager {
-    pub fn new() -> Self {
-        Self {
-            client: Arc::new(RwLock::new(None)),
-            session_name: Arc::new(RwLock::new(String::new())),
-            model: Arc::new(RwLock::new(String::new())),
-            started_at: Arc::new(RwLock::new(0)),
-        }
-    }
-
-    pub fn init(&self, initial_session_name: String, initial_model: String, started_at: i64) -> Result<(), String> {
-        tracing::debug!("Attempting to initialize Discord RPC...");
-        tracing::debug!("Application ID: 1391117878182281316");
-        tracing::debug!("Initial session: '{}', model: '{}', timestamp: {}",
-            initial_session_name, initial_model, started_at);
-
-        let mut client = DiscordIpcClient::new("1391117878182281316")
-            .map_err(|e| {
-                let error_msg = format!("Failed to create Discord RPC client: {} (is Discord running?)", e);
-                tracing::error!("{}", error_msg);
-                error_msg
-            })?;
-
-        tracing::debug!("DiscordIpcClient created successfully");
-
-        client
-            .connect()
-            .map_err(|e| {
-                let error_msg = format!("Failed to connect to Discord RPC: {} (ensure Discord is running)", e);
-                tracing::error!("{}", error_msg);
-                error_msg
-            })?;
-
-        tracing::debug!("Connected to Discord IPC socket");
-
-        // Set initial activity immediately after connecting
-        tracing::debug!("Building initial activity...");
-        let state_text = format!("Model: {}", initial_model);
-        let assets = Assets::new()
-            .large_image("hikari")
-            .large_text("Hikari - Claude Code Assistant");
-
-        tracing::debug!("Assets created - large_image: 'hikari', large_text: 'Hikari - Claude Code Assistant'");
-
-        let timestamps = Timestamps::new()
-            .start(started_at);
-
-        tracing::debug!("Timestamps created - start: {}", started_at);
-
-        let activity = Activity::new()
-            .details(initial_session_name.as_str())
-            .state(state_text.as_str())
-            .assets(assets)
-            .timestamps(timestamps);
-
-        tracing::debug!("Activity created - details: '{}', state: '{}'",
-            initial_session_name, state_text);
-
-        tracing::debug!("Attempting to set initial activity...");
-        client
-            .set_activity(activity)
-            .map_err(|e| {
-                let error_msg = format!("Failed to set initial Discord RPC activity: {}", e);
-                tracing::error!("{}", error_msg);
-                error_msg
-            })?;
-
-        tracing::debug!("Initial activity set successfully!");
-
-        // Store the client and initial state
-        *self.client.write() = Some(client);
-        *self.session_name.write() = initial_session_name.clone();
-        *self.model.write() = initial_model.clone();
-        *self.started_at.write() = started_at;
-
-        tracing::info!("Discord RPC connected successfully with initial activity: session='{}', model='{}'",
-            initial_session_name, initial_model);
-        Ok(())
-    }
-
-    pub fn update(
-        &self,
-        session_name: String,
-        model: String,
-        started_at: i64,
-    ) -> Result<(), String> {
-        tracing::debug!("update() called with session='{}', model='{}', timestamp={}",
-            session_name, model, started_at);
-
-        *self.session_name.write() = session_name.clone();
-        *self.model.write() = model.clone();
-        *self.started_at.write() = started_at;
-
-        tracing::debug!("State variables updated");
-
-        let mut client_guard = self.client.write();
-        let client = client_guard
-            .as_mut()
-            .ok_or_else(|| {
-                let error_msg = "Discord RPC client not initialized".to_string();
-                tracing::error!("{}", error_msg);
-                error_msg
-            })?;
-
-        tracing::debug!("Client lock acquired");
-
-        let state_text = format!("Model: {}", model);
-        let assets = Assets::new()
-            .large_image("hikari")
-            .large_text("Hikari - Claude Code Assistant");
-
-        tracing::debug!("Assets created - large_image: 'hikari', large_text: 'Hikari - Claude Code Assistant'");
-
-        let timestamps = Timestamps::new()
-            .start(started_at);
-
-        tracing::debug!("Timestamps created - start: {}", started_at);
-
-        let activity = Activity::new()
-            .details(session_name.as_str())
-            .state(state_text.as_str())
-            .assets(assets)
-            .timestamps(timestamps);
-
-        tracing::debug!("Activity created - details: '{}', state: '{}'",
-            session_name, state_text);
-
-        tracing::debug!("Attempting to set activity...");
-        client
-            .set_activity(activity)
-            .map_err(|e| {
-                let error_msg = format!("Failed to update Discord RPC: {}", e);
-                tracing::error!("{}", error_msg);
-                error_msg
-            })?;
-
-        tracing::info!("Updated Discord RPC: session='{}', model='{}'", session_name, model);
-        Ok(())
-    }
-
-    pub fn stop(&self) -> Result<(), String> {
-        tracing::debug!("stop() called");
-
-        let mut client_guard = self.client.write();
-        if let Some(mut client) = client_guard.take() {
-            tracing::debug!("Client found, attempting to close...");
-            client
-                .close()
-                .map_err(|e| {
-                    let error_msg = format!("Failed to close Discord RPC: {}", e);
-                    tracing::error!("{}", error_msg);
-                    error_msg
-                })?;
-            tracing::info!("Discord RPC stopped successfully");
-        } else {
-            tracing::debug!("No client to stop (already stopped or never initialized)");
-        }
-        Ok(())
-    }
-}
-
-impl Default for DiscordRpcManager {
-    fn default() -> Self {
-        Self::new()
-    }
-}

src-tauri/src/drafts.rs

@@ -1,192 +0,0 @@
-use chrono::Utc;
-use serde::{Deserialize, Serialize};
-use tauri::AppHandle;
-use tauri_plugin_store::StoreExt;
-use uuid::Uuid;
-
-const DRAFTS_STORE_FILE: &str = "hikari-drafts.json";
-const DRAFTS_STORE_KEY: &str = "drafts";
-
-#[derive(Debug, Clone, Serialize, Deserialize)]
-pub struct Draft {
-    pub id: String,
-    pub content: String,
-    pub saved_at: String,
-}
-
-fn load_all_drafts(app: &AppHandle) -> Result<Vec<Draft>, String> {
-    let store = app
-        .store(DRAFTS_STORE_FILE)
-        .map_err(|e| e.to_string())?;
-
-    match store.get(DRAFTS_STORE_KEY) {
-        Some(value) => serde_json::from_value(value.clone()).map_err(|e| e.to_string()),
-        None => Ok(vec![]),
-    }
-}
-
-fn save_all_drafts(app: &AppHandle, drafts: &[Draft]) -> Result<(), String> {
-    let store = app
-        .store(DRAFTS_STORE_FILE)
-        .map_err(|e| e.to_string())?;
-
-    let value = serde_json::to_value(drafts).map_err(|e| e.to_string())?;
-    store.set(DRAFTS_STORE_KEY, value);
-    store.save().map_err(|e| e.to_string())?;
-
-    Ok(())
-}
-
-#[tauri::command]
-pub async fn list_drafts(app: AppHandle) -> Result<Vec<Draft>, String> {
-    let mut drafts = load_all_drafts(&app)?;
-    // Sort newest first — ISO 8601 timestamps sort lexicographically
-    drafts.sort_by(|a, b| b.saved_at.cmp(&a.saved_at));
-    Ok(drafts)
-}
-
-#[tauri::command]
-pub async fn save_draft(app: AppHandle, content: String) -> Result<Draft, String> {
-    let mut drafts = load_all_drafts(&app)?;
-
-    let draft = Draft {
-        id: Uuid::new_v4().to_string(),
-        content,
-        saved_at: Utc::now().to_rfc3339(),
-    };
-
-    drafts.push(draft.clone());
-    save_all_drafts(&app, &drafts)?;
-
-    Ok(draft)
-}
-
-#[tauri::command]
-pub async fn delete_draft(app: AppHandle, draft_id: String) -> Result<(), String> {
-    let mut drafts = load_all_drafts(&app)?;
-    drafts.retain(|d| d.id != draft_id);
-    save_all_drafts(&app, &drafts)
-}
-
-#[tauri::command]
-pub async fn delete_all_drafts(app: AppHandle) -> Result<(), String> {
-    save_all_drafts(&app, &[])
-}
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    fn make_draft(id: &str, content: &str, saved_at: &str) -> Draft {
-        Draft {
-            id: id.to_string(),
-            content: content.to_string(),
-            saved_at: saved_at.to_string(),
-        }
-    }
-
-    #[test]
-    fn test_draft_serialization() {
-        let draft = make_draft("test-id", "Hello world", "2026-01-01T00:00:00+00:00");
-        let json = serde_json::to_string(&draft).expect("Failed to serialize");
-        let parsed: Draft = serde_json::from_str(&json).expect("Failed to deserialize");
-
-        assert_eq!(parsed.id, draft.id);
-        assert_eq!(parsed.content, draft.content);
-        assert_eq!(parsed.saved_at, draft.saved_at);
-    }
-
-    #[test]
-    fn test_draft_clone() {
-        let original = make_draft("clone-id", "Clone me", "2026-01-01T00:00:00+00:00");
-        let cloned = original.clone();
-
-        assert_eq!(original.id, cloned.id);
-        assert_eq!(original.content, cloned.content);
-        assert_eq!(original.saved_at, cloned.saved_at);
-    }
-
-    #[test]
-    fn test_sort_newest_first() {
-        let mut drafts = [
-            make_draft("a", "First", "2026-01-01T00:00:00+00:00"),
-            make_draft("b", "Third", "2026-01-03T00:00:00+00:00"),
-            make_draft("c", "Second", "2026-01-02T00:00:00+00:00"),
-        ];
-
-        drafts.sort_by(|a, b| b.saved_at.cmp(&a.saved_at));
-
-        assert_eq!(drafts[0].id, "b");
-        assert_eq!(drafts[1].id, "c");
-        assert_eq!(drafts[2].id, "a");
-    }
-
-    #[test]
-    fn test_retain_excludes_deleted() {
-        let mut drafts = vec![
-            make_draft("keep-1", "Keep me", "2026-01-01T00:00:00+00:00"),
-            make_draft("delete-me", "Delete me", "2026-01-02T00:00:00+00:00"),
-            make_draft("keep-2", "Keep me too", "2026-01-03T00:00:00+00:00"),
-        ];
-
-        let target_id = "delete-me".to_string();
-        drafts.retain(|d| d.id != target_id);
-
-        assert_eq!(drafts.len(), 2);
-        assert!(drafts.iter().all(|d| d.id != "delete-me"));
-    }
-
-    #[test]
-    fn test_find_by_id() {
-        let drafts = [
-            make_draft("draft-1", "First draft", "2026-01-01T00:00:00+00:00"),
-            make_draft("draft-2", "Second draft", "2026-01-02T00:00:00+00:00"),
-            make_draft("draft-3", "Third draft", "2026-01-03T00:00:00+00:00"),
-        ];
-
-        let found = drafts.iter().find(|d| d.id == "draft-2");
-        assert!(found.is_some());
-        assert_eq!(found.unwrap().content, "Second draft");
-
-        let not_found = drafts.iter().find(|d| d.id == "draft-999");
-        assert!(not_found.is_none());
-    }
-
-    #[test]
-    fn test_multiline_content() {
-        let content = "Line 1\nLine 2\nLine 3";
-        let draft = make_draft("multi", content, "2026-01-01T00:00:00+00:00");
-
-        assert!(draft.content.contains('\n'));
-        assert_eq!(draft.content.split('\n').count(), 3);
-    }
-
-    #[test]
-    fn test_empty_after_delete_all() {
-        let mut drafts = vec![
-            make_draft("a", "A", "2026-01-01T00:00:00+00:00"),
-            make_draft("b", "B", "2026-01-02T00:00:00+00:00"),
-        ];
-
-        drafts.clear();
-
-        assert!(drafts.is_empty());
-    }
-
-    #[test]
-    fn test_uuid_format() {
-        // UUIDs should be non-empty and contain hyphens
-        let id = Uuid::new_v4().to_string();
-        assert!(!id.is_empty());
-        assert!(id.contains('-'));
-        assert_eq!(id.len(), 36);
-    }
-
-    #[test]
-    fn test_timestamp_is_rfc3339() {
-        let ts = Utc::now().to_rfc3339();
-        // RFC 3339 timestamps contain T and + or Z
-        assert!(ts.contains('T'));
-        assert!(ts.ends_with("+00:00") || ts.ends_with('Z'));
-    }
-}

src-tauri/src/git.rs

@@ -1,8 +1,6 @@
 use serde::{Deserialize, Serialize};
 use std::process::Command;
 
-use crate::process_ext::HideWindow;
-
 #[derive(Debug, Clone, Serialize, Deserialize)]
 pub struct GitStatus {
     pub is_repo: bool,
@@ -39,7 +37,6 @@ pub struct GitLogEntry {
 
 fn run_git_command(working_dir: &str, args: &[&str]) -> Result<String, String> {
     let output = Command::new("git")
-        .hide_window()
         .args(args)
         .current_dir(working_dir)
         .output()
@@ -289,593 +286,3 @@ pub fn git_discard(working_dir: String, file_path: String) -> Result<String, Str
 pub fn git_create_branch(working_dir: String, branch_name: String) -> Result<String, String> {
     run_git_command(&working_dir, &["checkout", "-b", &branch_name])
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use std::fs::{self, File};
-    use std::io::Write;
-    use tempfile::TempDir;
-
-    // Helper to create a git repository in a temp directory
-    fn create_test_repo() -> TempDir {
-        let temp_dir = TempDir::new().unwrap();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // Initialize git repo
-        run_git_command(&working_dir, &["init"]).unwrap();
-
-        // Configure git user for commits
-        run_git_command(&working_dir, &["config", "user.email", "test@example.com"]).unwrap();
-        run_git_command(&working_dir, &["config", "user.name", "Test User"]).unwrap();
-
-        // Disable GPG signing for tests (user may have it enabled globally)
-        run_git_command(&working_dir, &["config", "commit.gpgsign", "false"]).unwrap();
-
-        temp_dir
-    }
-
-    // Helper to create a file in the test repo
-    fn create_file(dir: &TempDir, name: &str, content: &str) {
-        let file_path = dir.path().join(name);
-        let mut file = File::create(file_path).unwrap();
-        file.write_all(content.as_bytes()).unwrap();
-    }
-
-    // ==================== GitStatus struct tests ====================
-
-    #[test]
-    fn test_git_status_serialization() {
-        let status = GitStatus {
-            is_repo: true,
-            branch: Some("main".to_string()),
-            upstream: Some("origin/main".to_string()),
-            ahead: 2,
-            behind: 1,
-            staged: vec![GitFileChange {
-                path: "file.txt".to_string(),
-                status: "modified".to_string(),
-            }],
-            unstaged: vec![],
-            untracked: vec!["new_file.txt".to_string()],
-        };
-
-        let json = serde_json::to_string(&status).unwrap();
-        assert!(json.contains("\"is_repo\":true"));
-        assert!(json.contains("\"branch\":\"main\""));
-        assert!(json.contains("\"ahead\":2"));
-        assert!(json.contains("\"behind\":1"));
-    }
-
-    #[test]
-    fn test_git_status_not_a_repo() {
-        let status = GitStatus {
-            is_repo: false,
-            branch: None,
-            upstream: None,
-            ahead: 0,
-            behind: 0,
-            staged: vec![],
-            unstaged: vec![],
-            untracked: vec![],
-        };
-
-        let json = serde_json::to_string(&status).unwrap();
-        let deserialized: GitStatus = serde_json::from_str(&json).unwrap();
-        assert!(!deserialized.is_repo);
-        assert!(deserialized.branch.is_none());
-    }
-
-    // ==================== GitFileChange struct tests ====================
-
-    #[test]
-    fn test_git_file_change_serialization() {
-        let change = GitFileChange {
-            path: "src/main.rs".to_string(),
-            status: "added".to_string(),
-        };
-
-        let json = serde_json::to_string(&change).unwrap();
-        assert!(json.contains("src/main.rs"));
-        assert!(json.contains("added"));
-
-        let deserialized: GitFileChange = serde_json::from_str(&json).unwrap();
-        assert_eq!(deserialized.path, "src/main.rs");
-        assert_eq!(deserialized.status, "added");
-    }
-
-    // ==================== GitBranch struct tests ====================
-
-    #[test]
-    fn test_git_branch_serialization() {
-        let branch = GitBranch {
-            name: "feature/new-feature".to_string(),
-            is_current: true,
-            is_remote: false,
-        };
-
-        let json = serde_json::to_string(&branch).unwrap();
-        assert!(json.contains("feature/new-feature"));
-        assert!(json.contains("\"is_current\":true"));
-        assert!(json.contains("\"is_remote\":false"));
-    }
-
-    #[test]
-    fn test_git_branch_remote() {
-        let branch = GitBranch {
-            name: "origin/main".to_string(),
-            is_current: false,
-            is_remote: true,
-        };
-
-        let json = serde_json::to_string(&branch).unwrap();
-        let deserialized: GitBranch = serde_json::from_str(&json).unwrap();
-        assert!(deserialized.is_remote);
-        assert!(!deserialized.is_current);
-    }
-
-    // ==================== GitLogEntry struct tests ====================
-
-    #[test]
-    fn test_git_log_entry_serialization() {
-        let entry = GitLogEntry {
-            hash: "abc123def456".to_string(),
-            short_hash: "abc123d".to_string(),
-            author: "Hikari".to_string(),
-            date: "2 hours ago".to_string(),
-            message: "feat: add new feature".to_string(),
-        };
-
-        let json = serde_json::to_string(&entry).unwrap();
-        assert!(json.contains("abc123def456"));
-        assert!(json.contains("Hikari"));
-        assert!(json.contains("feat: add new feature"));
-    }
-
-    // ==================== git_status integration tests ====================
-
-    #[test]
-    fn test_git_status_not_a_git_repo() {
-        let temp_dir = TempDir::new().unwrap();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        let result = git_status(working_dir);
-        assert!(result.is_ok());
-
-        let status = result.unwrap();
-        assert!(!status.is_repo);
-        assert!(status.branch.is_none());
-        assert!(status.staged.is_empty());
-    }
-
-    #[test]
-    fn test_git_status_empty_repo() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        let result = git_status(working_dir);
-        assert!(result.is_ok());
-
-        let status = result.unwrap();
-        assert!(status.is_repo);
-        assert!(status.staged.is_empty());
-        assert!(status.unstaged.is_empty());
-        assert!(status.untracked.is_empty());
-    }
-
-    #[test]
-    fn test_git_status_with_untracked_file() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // Create an untracked file
-        create_file(&temp_dir, "untracked.txt", "hello");
-
-        let result = git_status(working_dir);
-        assert!(result.is_ok());
-
-        let status = result.unwrap();
-        assert!(status.is_repo);
-        assert!(status.untracked.contains(&"untracked.txt".to_string()));
-    }
-
-    #[test]
-    fn test_git_status_with_staged_file() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // Create and stage a file
-        create_file(&temp_dir, "staged.txt", "hello");
-        run_git_command(&working_dir, &["add", "staged.txt"]).unwrap();
-
-        let result = git_status(working_dir);
-        assert!(result.is_ok());
-
-        let status = result.unwrap();
-        assert!(status.is_repo);
-        assert!(!status.staged.is_empty());
-        assert_eq!(status.staged[0].path, "staged.txt");
-        assert_eq!(status.staged[0].status, "added");
-    }
-
-    #[test]
-    fn test_git_status_with_modified_file() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // Create, stage, and commit a file
-        create_file(&temp_dir, "file.txt", "initial content");
-        run_git_command(&working_dir, &["add", "file.txt"]).unwrap();
-        run_git_command(&working_dir, &["commit", "-m", "initial commit"]).unwrap();
-
-        // Modify the file
-        create_file(&temp_dir, "file.txt", "modified content");
-
-        let result = git_status(working_dir);
-        assert!(result.is_ok());
-
-        let status = result.unwrap();
-        assert!(status.is_repo);
-        assert!(!status.unstaged.is_empty());
-        assert_eq!(status.unstaged[0].path, "file.txt");
-        assert_eq!(status.unstaged[0].status, "modified");
-    }
-
-    // ==================== git_diff integration tests ====================
-
-    #[test]
-    fn test_git_diff_no_changes() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        let result = git_diff(working_dir, None, false);
-        assert!(result.is_ok());
-        assert!(result.unwrap().is_empty());
-    }
-
-    #[test]
-    fn test_git_diff_with_changes() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // Create and commit a file
-        create_file(&temp_dir, "file.txt", "initial content");
-        run_git_command(&working_dir, &["add", "file.txt"]).unwrap();
-        run_git_command(&working_dir, &["commit", "-m", "initial"]).unwrap();
-
-        // Modify the file
-        create_file(&temp_dir, "file.txt", "modified content");
-
-        let result = git_diff(working_dir, None, false);
-        assert!(result.is_ok());
-        let diff = result.unwrap();
-        assert!(diff.contains("diff"));
-        assert!(diff.contains("file.txt"));
-    }
-
-    #[test]
-    fn test_git_diff_staged() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // Create and commit a file
-        create_file(&temp_dir, "file.txt", "initial content");
-        run_git_command(&working_dir, &["add", "file.txt"]).unwrap();
-        run_git_command(&working_dir, &["commit", "-m", "initial"]).unwrap();
-
-        // Modify and stage the file
-        create_file(&temp_dir, "file.txt", "modified content");
-        run_git_command(&working_dir, &["add", "file.txt"]).unwrap();
-
-        let result = git_diff(working_dir, None, true);
-        assert!(result.is_ok());
-        let diff = result.unwrap();
-        assert!(diff.contains("diff"));
-    }
-
-    #[test]
-    fn test_git_diff_specific_file() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // Create and commit files
-        create_file(&temp_dir, "file1.txt", "content1");
-        create_file(&temp_dir, "file2.txt", "content2");
-        run_git_command(&working_dir, &["add", "-A"]).unwrap();
-        run_git_command(&working_dir, &["commit", "-m", "initial"]).unwrap();
-
-        // Modify both files
-        create_file(&temp_dir, "file1.txt", "modified1");
-        create_file(&temp_dir, "file2.txt", "modified2");
-
-        // Get diff for only file1.txt
-        let result = git_diff(working_dir, Some("file1.txt".to_string()), false);
-        assert!(result.is_ok());
-        let diff = result.unwrap();
-        assert!(diff.contains("file1.txt"));
-        assert!(!diff.contains("file2.txt"));
-    }
-
-    // ==================== git_branches integration tests ====================
-
-    #[test]
-    fn test_git_branches_single_branch() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // Need at least one commit for branches to show
-        create_file(&temp_dir, "file.txt", "content");
-        run_git_command(&working_dir, &["add", "file.txt"]).unwrap();
-        run_git_command(&working_dir, &["commit", "-m", "initial"]).unwrap();
-
-        let result = git_branches(working_dir);
-        assert!(result.is_ok());
-
-        let branches = result.unwrap();
-        assert!(!branches.is_empty());
-        // Should have at least one branch (main or master)
-    }
-
-    #[test]
-    fn test_git_branches_multiple_branches() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // Initial commit
-        create_file(&temp_dir, "file.txt", "content");
-        run_git_command(&working_dir, &["add", "file.txt"]).unwrap();
-        run_git_command(&working_dir, &["commit", "-m", "initial"]).unwrap();
-
-        // Create additional branch
-        run_git_command(&working_dir, &["branch", "feature-branch"]).unwrap();
-
-        let result = git_branches(working_dir);
-        assert!(result.is_ok());
-
-        let branches = result.unwrap();
-        assert!(branches.len() >= 2);
-        assert!(branches.iter().any(|b| b.name == "feature-branch"));
-    }
-
-    // ==================== git_stage and git_unstage tests ====================
-
-    #[test]
-    fn test_git_stage_file() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        create_file(&temp_dir, "file.txt", "content");
-
-        let result = git_stage(working_dir.clone(), "file.txt".to_string());
-        assert!(result.is_ok());
-
-        // Verify file is staged
-        let status = git_status(working_dir).unwrap();
-        assert!(status.staged.iter().any(|f| f.path == "file.txt"));
-    }
-
-    #[test]
-    fn test_git_unstage_file() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // First, commit a file so we have a HEAD to restore from
-        create_file(&temp_dir, "file.txt", "initial content");
-        run_git_command(&working_dir, &["add", "file.txt"]).unwrap();
-        run_git_command(&working_dir, &["commit", "-m", "initial"]).unwrap();
-
-        // Modify and stage the file
-        create_file(&temp_dir, "file.txt", "modified content");
-        run_git_command(&working_dir, &["add", "file.txt"]).unwrap();
-
-        let result = git_unstage(working_dir.clone(), "file.txt".to_string());
-        assert!(result.is_ok());
-
-        // Verify file is unstaged (should now be in unstaged/modified, not staged)
-        let status = git_status(working_dir).unwrap();
-        assert!(!status.staged.iter().any(|f| f.path == "file.txt"));
-        assert!(status.unstaged.iter().any(|f| f.path == "file.txt"));
-    }
-
-    #[test]
-    fn test_git_stage_all() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        create_file(&temp_dir, "file1.txt", "content1");
-        create_file(&temp_dir, "file2.txt", "content2");
-
-        let result = git_stage_all(working_dir.clone());
-        assert!(result.is_ok());
-
-        // Verify all files are staged
-        let status = git_status(working_dir).unwrap();
-        assert_eq!(status.staged.len(), 2);
-    }
-
-    // ==================== git_commit tests ====================
-
-    #[test]
-    fn test_git_commit() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        create_file(&temp_dir, "file.txt", "content");
-        run_git_command(&working_dir, &["add", "file.txt"]).unwrap();
-
-        let result = git_commit(working_dir.clone(), "test commit message".to_string());
-        assert!(result.is_ok());
-
-        // Verify commit was made
-        let log = git_log(working_dir, Some(1)).unwrap();
-        assert!(!log.is_empty());
-        assert!(log[0].message.contains("test commit message"));
-    }
-
-    #[test]
-    fn test_git_commit_nothing_to_commit() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // Need initial commit first
-        create_file(&temp_dir, "file.txt", "content");
-        run_git_command(&working_dir, &["add", "file.txt"]).unwrap();
-        run_git_command(&working_dir, &["commit", "-m", "initial"]).unwrap();
-
-        // Try to commit with nothing staged
-        let result = git_commit(working_dir, "empty commit".to_string());
-        assert!(result.is_err()); // Should fail because nothing to commit
-    }
-
-    // ==================== git_log tests ====================
-
-    #[test]
-    fn test_git_log_empty_repo() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        let result = git_log(working_dir, Some(10));
-        // May fail on empty repo or return empty
-        if let Ok(commits) = result {
-            assert!(commits.is_empty());
-        }
-    }
-
-    #[test]
-    fn test_git_log_with_commits() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // Make multiple commits
-        for i in 1..=3 {
-            create_file(&temp_dir, &format!("file{}.txt", i), "content");
-            run_git_command(&working_dir, &["add", "-A"]).unwrap();
-            run_git_command(&working_dir, &["commit", "-m", &format!("commit {}", i)]).unwrap();
-        }
-
-        let result = git_log(working_dir, Some(10));
-        assert!(result.is_ok());
-
-        let log = result.unwrap();
-        assert_eq!(log.len(), 3);
-        assert!(log[0].message.contains("commit 3")); // Most recent first
-        assert!(log[2].message.contains("commit 1"));
-    }
-
-    #[test]
-    fn test_git_log_limit() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // Make 5 commits
-        for i in 1..=5 {
-            create_file(&temp_dir, &format!("file{}.txt", i), "content");
-            run_git_command(&working_dir, &["add", "-A"]).unwrap();
-            run_git_command(&working_dir, &["commit", "-m", &format!("commit {}", i)]).unwrap();
-        }
-
-        // Only get last 2
-        let result = git_log(working_dir, Some(2));
-        assert!(result.is_ok());
-
-        let log = result.unwrap();
-        assert_eq!(log.len(), 2);
-    }
-
-    // ==================== git_discard tests ====================
-
-    #[test]
-    fn test_git_discard_changes() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // Create and commit a file
-        create_file(&temp_dir, "file.txt", "original content");
-        run_git_command(&working_dir, &["add", "file.txt"]).unwrap();
-        run_git_command(&working_dir, &["commit", "-m", "initial"]).unwrap();
-
-        // Modify the file
-        create_file(&temp_dir, "file.txt", "modified content");
-
-        // Discard changes
-        let result = git_discard(working_dir.clone(), "file.txt".to_string());
-        assert!(result.is_ok());
-
-        // Verify file contents are restored
-        let content = fs::read_to_string(temp_dir.path().join("file.txt")).unwrap();
-        assert_eq!(content, "original content");
-    }
-
-    // ==================== git_create_branch tests ====================
-
-    #[test]
-    fn test_git_create_branch() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // Initial commit required
-        create_file(&temp_dir, "file.txt", "content");
-        run_git_command(&working_dir, &["add", "file.txt"]).unwrap();
-        run_git_command(&working_dir, &["commit", "-m", "initial"]).unwrap();
-
-        let result = git_create_branch(working_dir.clone(), "new-branch".to_string());
-        assert!(result.is_ok());
-
-        // Verify branch exists and is current
-        let branches = git_branches(working_dir).unwrap();
-        assert!(branches.iter().any(|b| b.name == "new-branch" && b.is_current));
-    }
-
-    // ==================== git_checkout tests ====================
-
-    #[test]
-    fn test_git_checkout() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // Initial commit required
-        create_file(&temp_dir, "file.txt", "content");
-        run_git_command(&working_dir, &["add", "file.txt"]).unwrap();
-        run_git_command(&working_dir, &["commit", "-m", "initial"]).unwrap();
-
-        // Create a branch
-        run_git_command(&working_dir, &["branch", "other-branch"]).unwrap();
-
-        // Checkout the branch
-        let result = git_checkout(working_dir.clone(), "other-branch".to_string());
-        assert!(result.is_ok());
-
-        // Verify current branch
-        let branches = git_branches(working_dir).unwrap();
-        let current = branches.iter().find(|b| b.is_current);
-        assert!(current.is_some());
-        assert_eq!(current.unwrap().name, "other-branch");
-    }
-
-    // ==================== run_git_command tests ====================
-
-    #[test]
-    fn test_run_git_command_success() {
-        let temp_dir = create_test_repo();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        let result = run_git_command(&working_dir, &["status"]);
-        assert!(result.is_ok());
-    }
-
-    #[test]
-    fn test_run_git_command_failure() {
-        let temp_dir = TempDir::new().unwrap();
-        let working_dir = temp_dir.path().to_string_lossy().to_string();
-
-        // This should fail because it's not a git repo
-        let result = run_git_command(&working_dir, &["log"]);
-        assert!(result.is_err());
-    }
-
-    #[test]
-    fn test_run_git_command_invalid_dir() {
-        let result = run_git_command("/nonexistent/path", &["status"]);
-        assert!(result.is_err());
-    }
-}

src-tauri/src/lib.rs

@@ -3,19 +3,13 @@ mod bridge_manager;
 mod clipboard;
 mod commands;
 mod config;
-mod cost_tracking;
-mod debug_logger;
-mod discord_rpc;
-mod drafts;
 mod git;
 mod notifications;
-mod process_ext;
 mod quick_actions;
 mod sessions;
 mod snippets;
 mod stats;
 mod temp_manager;
-mod tool_cache;
 mod tray;
 mod types;
 mod vbs_notification;
@@ -27,20 +21,14 @@ use bridge_manager::create_shared_bridge_manager;
 use clipboard::*;
 use commands::load_saved_achievements;
 use commands::*;
-use debug_logger::TauriLogLayer;
-use discord_rpc::DiscordRpcManager;
-use drafts::*;
 use git::*;
 use notifications::*;
 use quick_actions::*;
 use sessions::*;
 use snippets::*;
-use std::sync::Arc;
-use tauri::{Emitter, Manager};
+use tauri::Manager;
 use temp_manager::create_shared_temp_manager;
-use tracing_subscriber::layer::SubscriberExt;
-use tracing_subscriber::util::SubscriberInitExt;
-use tray::setup_tray;
+use tray::{setup_tray, should_minimize_to_tray};
 use vbs_notification::*;
 use windows_toast::*;
 use wsl_notifications::*;
@@ -49,7 +37,6 @@ use wsl_notifications::*;
 pub fn run() {
     let bridge_manager = create_shared_bridge_manager();
     let temp_manager = create_shared_temp_manager().expect("Failed to create temp file manager");
-    let discord_rpc = Arc::new(DiscordRpcManager::new());
 
     tauri::Builder::default()
         .plugin(tauri_plugin_dialog::init())
@@ -63,45 +50,33 @@ pub fn run() {
         .plugin(tauri_plugin_fs::init())
         .manage(bridge_manager.clone())
         .manage(temp_manager.clone())
-        .manage(discord_rpc.clone())
         .setup(move |app| {
-            // Initialize tracing with custom layer that emits to frontend
-            // NOTE: We don't use fmt::layer() because in production builds with windows_subsystem = "windows",
-            // stdout is hidden. Instead, all logs go through TauriLogLayer to the debug console.
-            let tauri_layer = TauriLogLayer::new(app.handle().clone());
-            tracing_subscriber::registry()
-                .with(tauri_layer)
-                .init();
-
             // Initialize the app handle in the bridge manager
             bridge_manager.lock().set_app_handle(app.handle().clone());
 
             // Clean up any orphaned temp files from previous sessions
             if let Ok(count) = temp_manager.lock().cleanup_orphaned_files() {
                 if count > 0 {
-                    tracing::info!("Cleaned up {} orphaned temp files", count);
+                    println!("Cleaned up {} orphaned temp files", count);
                 }
             }
 
-            tracing::info!("Hikari Desktop started successfully");
-
             // Set up system tray
             if let Err(e) = setup_tray(app.handle()) {
-                tracing::error!("Failed to set up system tray: {}", e);
+                eprintln!("Failed to set up system tray: {}", e);
             }
 
-            // Handle window close event for minimize to tray and close confirmation
+            // Handle window close event for minimize to tray
             let main_window = app.get_webview_window("main").unwrap();
             main_window.on_window_event({
                 let app_handle = app.handle().clone();
                 move |event| {
                     if let tauri::WindowEvent::CloseRequested { api, .. } = event {
-                        // Always prevent default close - let frontend handle it
-                        api.prevent_close();
-
-                        // Emit event to frontend to show confirmation modal
-                        if let Some(window) = app_handle.get_webview_window("main") {
-                            let _ = window.emit("window-close-requested", ());
+                        if should_minimize_to_tray(&app_handle) {
+                            api.prevent_close();
+                            if let Some(window) = app_handle.get_webview_window("main") {
+                                let _ = window.hide();
+                            }
                         }
                     }
                 }
@@ -123,7 +98,6 @@ pub fn run() {
             get_persisted_stats,
             load_saved_achievements,
             answer_question,
-            check_workspace_hooks,
             send_windows_notification,
             send_simple_notification,
             send_windows_toast,
@@ -177,49 +151,6 @@ pub fn run() {
             search_clipboard_entries,
             get_clipboard_languages,
             update_clipboard_language,
-            list_directory,
-            read_file_content,
-            write_file_content,
-            create_file,
-            create_directory,
-            delete_file,
-            delete_directory,
-            rename_path,
-            // Cost tracking commands
-            get_cost_summary,
-            get_cost_alerts,
-            set_cost_alert_thresholds,
-            export_cost_csv,
-            get_today_cost,
-            get_week_cost,
-            get_month_cost,
-            init_discord_rpc,
-            update_discord_rpc,
-            stop_discord_rpc,
-            close_application,
-            list_memory_files,
-            get_claude_version,
-            get_auth_status,
-            auth_login,
-            auth_logout,
-            list_plugins,
-            install_plugin,
-            uninstall_plugin,
-            enable_plugin,
-            disable_plugin,
-            update_plugin,
-            list_marketplaces,
-            add_marketplace,
-            remove_marketplace,
-            list_mcp_servers,
-            get_mcp_server,
-            remove_mcp_server,
-            add_mcp_server,
-            get_mcp_server_details,
-            list_drafts,
-            save_draft,
-            delete_draft,
-            delete_all_drafts,
         ])
         .run(tauri::generate_context!())
         .expect("error while running tauri application");

src-tauri/src/notifications.rs

@@ -1,11 +1,34 @@
 use std::process::Command;
 use tauri::command;
 
-use crate::process_ext::HideWindow;
+#[command]
+pub async fn send_notify_send(title: String, body: String) -> Result<(), String> {
+    // Use notify-send for Linux/WSL
+    let output = Command::new("notify-send")
+        .arg(&title)
+        .arg(&body)
+        .arg("--urgency=normal")
+        .arg("--app-name=Hikari Desktop")
+        .output()
+        .map_err(|e| {
+            format!(
+                "Failed to execute notify-send: {}. Make sure libnotify-bin is installed.",
+                e
+            )
+        })?;
 
-/// Generate PowerShell script for Windows Toast Notification
-fn generate_powershell_toast_script(title: &str, body: &str) -> String {
-    format!(
+    if !output.status.success() {
+        let error = String::from_utf8_lossy(&output.stderr);
+        return Err(format!("notify-send failed: {}", error));
+    }
+
+    Ok(())
+}
+
+#[command]
+pub async fn send_windows_notification(title: String, body: String) -> Result<(), String> {
+    // Create PowerShell script for Windows Toast Notification
+    let ps_script = format!(
         r#"
 [Windows.UI.Notifications.ToastNotificationManager, Windows.UI.Notifications, ContentType = WindowsRuntime] > $null
 [Windows.Data.Xml.Dom.XmlDocument, Windows.Data.Xml.Dom.XmlDocument, ContentType = WindowsRuntime] > $null
@@ -32,87 +55,10 @@ $toast = New-Object Windows.UI.Notifications.ToastNotification $xml
 "#,
         title.replace("\"", "`\""),
         body.replace("\"", "`\"")
-    )
-}
-
-/// Format simple notification message
-fn format_simple_notification(title: &str, body: &str) -> String {
-    format!("{}\n\n{}", title, body)
-}
-
-/// Build notify-send command for testing (doesn't execute)
-#[cfg(test)]
-fn build_notify_send_command(title: &str, body: &str) -> (String, Vec<String>) {
-    (
-        "notify-send".to_string(),
-        vec![
-            title.to_string(),
-            body.to_string(),
-            "--urgency=normal".to_string(),
-            "--app-name=Hikari Desktop".to_string(),
-        ],
-    )
-}
-
-/// Build Windows PowerShell command for testing (doesn't execute)
-#[cfg(test)]
-fn build_windows_powershell_command(title: &str, body: &str) -> (String, Vec<String>) {
-    let script = generate_powershell_toast_script(title, body);
-    (
-        "pwsh.exe".to_string(),
-        vec![
-            "-NoProfile".to_string(),
-            "-WindowStyle".to_string(),
-            "Hidden".to_string(),
-            "-Command".to_string(),
-            script,
-        ],
-    )
-}
-
-/// Build simple notification command for testing (doesn't execute)
-#[cfg(test)]
-fn build_simple_notification_command(title: &str, body: &str) -> (String, Vec<String>) {
-    let message = format_simple_notification(title, body);
-    (
-        "cmd.exe".to_string(),
-        vec!["/c".to_string(), "msg".to_string(), "*".to_string(), message],
-    )
-}
-
-#[command]
-pub async fn send_notify_send(title: String, body: String) -> Result<(), String> {
-    // Use notify-send for Linux/WSL
-    let output = Command::new("notify-send")
-        .hide_window()
-        .arg(&title)
-        .arg(&body)
-        .arg("--urgency=normal")
-        .arg("--app-name=Hikari Desktop")
-        .output()
-        .map_err(|e| {
-            format!(
-                "Failed to execute notify-send: {}. Make sure libnotify-bin is installed.",
-                e
-            )
-        })?;
-
-    if !output.status.success() {
-        let error = String::from_utf8_lossy(&output.stderr);
-        return Err(format!("notify-send failed: {}", error));
-    }
-
-    Ok(())
-}
-
-#[command]
-pub async fn send_windows_notification(title: String, body: String) -> Result<(), String> {
-    // Create PowerShell script for Windows Toast Notification
-    let ps_script = generate_powershell_toast_script(&title, &body);
+    );
 
     // Try PowerShell Core first (pwsh), then fall back to Windows PowerShell
     let output = Command::new("pwsh.exe")
-        .hide_window()
         .arg("-NoProfile")
         .arg("-WindowStyle")
         .arg("Hidden")
@@ -121,7 +67,6 @@ pub async fn send_windows_notification(title: String, body: String) -> Result<()
         .output()
         .or_else(|_| {
             Command::new("powershell.exe")
-                .hide_window()
                 .arg("-NoProfile")
                 .arg("-WindowStyle")
                 .arg("Hidden")
@@ -142,10 +87,9 @@ pub async fn send_windows_notification(title: String, body: String) -> Result<()
 // Alternative: Use Windows built-in MSG command for simple notifications
 #[command]
 pub async fn send_simple_notification(title: String, body: String) -> Result<(), String> {
-    let message = format_simple_notification(&title, &body);
+    let message = format!("{}\n\n{}", title, body);
 
     Command::new("cmd.exe")
-        .hide_window()
         .arg("/c")
         .arg("msg")
         .arg("*")
@@ -155,242 +99,3 @@ pub async fn send_simple_notification(title: String, body: String) -> Result<(),
 
     Ok(())
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    #[test]
-    fn test_generate_powershell_toast_script_basic() {
-        let script = generate_powershell_toast_script("Title", "Body");
-
-        assert!(script.contains("Hikari Desktop"));
-        assert!(script.contains("Title"));
-        assert!(script.contains("Body"));
-        assert!(script.contains("ToastNotification"));
-    }
-
-    #[test]
-    fn test_generate_powershell_toast_script_escapes_quotes() {
-        let script = generate_powershell_toast_script("Title with \"quotes\"", "Body with \"quotes\"");
-
-        // Quotes should be escaped as `" in PowerShell
-        assert!(script.contains("Title with `\"quotes`\""));
-        assert!(script.contains("Body with `\"quotes`\""));
-    }
-
-    #[test]
-    fn test_generate_powershell_toast_script_with_special_chars() {
-        let script = generate_powershell_toast_script("Title: Test", "Body\nwith\nnewlines");
-
-        assert!(script.contains("Title: Test"));
-        assert!(script.contains("Body\nwith\nnewlines"));
-    }
-
-    #[test]
-    fn test_generate_powershell_toast_script_unicode() {
-        let script = generate_powershell_toast_script("日本語 Title", "Unicode: 🎉");
-
-        assert!(script.contains("日本語 Title"));
-        assert!(script.contains("Unicode: 🎉"));
-    }
-
-    #[test]
-    fn test_generate_powershell_toast_script_empty() {
-        let script = generate_powershell_toast_script("", "");
-
-        // Should still contain the structure
-        assert!(script.contains("Hikari Desktop"));
-        assert!(script.contains("ToastNotification"));
-    }
-
-    #[test]
-    fn test_format_simple_notification_basic() {
-        let message = format_simple_notification("Title", "Body");
-
-        assert_eq!(message, "Title\n\nBody");
-    }
-
-    #[test]
-    fn test_format_simple_notification_with_newlines() {
-        let message = format_simple_notification("Multi\nLine\nTitle", "Multi\nLine\nBody");
-
-        assert!(message.contains("Multi\nLine\nTitle"));
-        assert!(message.contains("\n\n"));
-        assert!(message.contains("Multi\nLine\nBody"));
-    }
-
-    #[test]
-    fn test_format_simple_notification_unicode() {
-        let message = format_simple_notification("日本語", "🎉 Unicode");
-
-        assert_eq!(message, "日本語\n\n🎉 Unicode");
-    }
-
-    #[test]
-    fn test_format_simple_notification_empty() {
-        let message = format_simple_notification("", "");
-
-        assert_eq!(message, "\n\n");
-    }
-
-    #[test]
-    fn test_format_simple_notification_long_text() {
-        let long_title = "A".repeat(1000);
-        let long_body = "B".repeat(1000);
-        let message = format_simple_notification(&long_title, &long_body);
-
-        assert!(message.starts_with(&long_title));
-        assert!(message.ends_with(&long_body));
-        assert!(message.contains("\n\n"));
-    }
-
-    #[test]
-    fn test_generate_powershell_toast_script_multiple_quotes() {
-        let script = generate_powershell_toast_script(
-            "\"Quoted\" \"Multiple\" \"Times\"",
-            "\"More\" \"Quotes\" \"Here\""
-        );
-
-        // Each quote should be escaped
-        assert!(script.contains("`\"Quoted`\" `\"Multiple`\" `\"Times`\""));
-        assert!(script.contains("`\"More`\" `\"Quotes`\" `\"Here`\""));
-    }
-
-    // E2E Integration Tests - Command Structure Verification
-
-    #[test]
-    fn test_e2e_notify_send_command_structure() {
-        let (command, args) = build_notify_send_command("Test Title", "Test Body");
-
-        assert_eq!(command, "notify-send");
-        assert_eq!(args.len(), 4);
-        assert_eq!(args[0], "Test Title");
-        assert_eq!(args[1], "Test Body");
-        assert_eq!(args[2], "--urgency=normal");
-        assert_eq!(args[3], "--app-name=Hikari Desktop");
-    }
-
-    #[test]
-    fn test_e2e_notify_send_with_special_chars() {
-        let (command, args) =
-            build_notify_send_command("Title with \"quotes\"", "Body\nwith\nnewlines");
-
-        assert_eq!(command, "notify-send");
-        assert_eq!(args[0], "Title with \"quotes\"");
-        assert_eq!(args[1], "Body\nwith\nnewlines");
-        // notify-send handles these directly
-    }
-
-    #[test]
-    fn test_e2e_windows_powershell_command_structure() {
-        let (command, args) = build_windows_powershell_command("Test Title", "Test Body");
-
-        assert_eq!(command, "pwsh.exe");
-        assert_eq!(args.len(), 5);
-        assert_eq!(args[0], "-NoProfile");
-        assert_eq!(args[1], "-WindowStyle");
-        assert_eq!(args[2], "Hidden");
-        assert_eq!(args[3], "-Command");
-
-        // Verify the script in args[4] contains expected elements
-        let script = &args[4];
-        assert!(script.contains("Test Title"));
-        assert!(script.contains("Test Body"));
-        assert!(script.contains("Hikari Desktop"));
-        assert!(script.contains("ToastNotification"));
-    }
-
-    #[test]
-    fn test_e2e_windows_powershell_quote_escaping() {
-        let (_, args) =
-            build_windows_powershell_command("Title with \"quotes\"", "Body with \"quotes\"");
-
-        let script = &args[4];
-        // Verify quotes are properly escaped in the PowerShell script
-        assert!(script.contains("Title with `\"quotes`\""));
-        assert!(script.contains("Body with `\"quotes`\""));
-    }
-
-    #[test]
-    fn test_e2e_simple_notification_command_structure() {
-        let (command, args) = build_simple_notification_command("Test Title", "Test Body");
-
-        assert_eq!(command, "cmd.exe");
-        assert_eq!(args.len(), 4);
-        assert_eq!(args[0], "/c");
-        assert_eq!(args[1], "msg");
-        assert_eq!(args[2], "*");
-        assert_eq!(args[3], "Test Title\n\nTest Body");
-    }
-
-    #[test]
-    fn test_e2e_simple_notification_multiline() {
-        let (_, args) =
-            build_simple_notification_command("Multi\nLine\nTitle", "Multi\nLine\nBody");
-
-        let message = &args[3];
-        assert!(message.contains("Multi\nLine\nTitle"));
-        assert!(message.contains("\n\n"));
-        assert!(message.contains("Multi\nLine\nBody"));
-    }
-
-    #[test]
-    fn test_e2e_command_consistency_across_platforms() {
-        // Test that different platforms use consistent parameters
-        let title = "Consistency Test";
-        let body = "Testing cross-platform consistency";
-
-        // Linux command
-        let (notify_cmd, notify_args) = build_notify_send_command(title, body);
-        assert!(notify_cmd.contains("notify"));
-        assert!(notify_args.iter().any(|arg| arg.contains("Hikari Desktop")));
-
-        // Windows PowerShell command
-        let (ps_cmd, ps_args) = build_windows_powershell_command(title, body);
-        assert!(ps_cmd.contains("pwsh") || ps_cmd.contains("powershell"));
-        let ps_script = &ps_args[4];
-        assert!(ps_script.contains("Hikari Desktop"));
-
-        // Windows simple command
-        let (msg_cmd, msg_args) = build_simple_notification_command(title, body);
-        assert!(msg_cmd.contains("cmd"));
-        assert!(msg_args[3].contains(title));
-        assert!(msg_args[3].contains(body));
-    }
-
-    #[test]
-    fn test_e2e_unicode_support_across_platforms() {
-        let title = "日本語 Title";
-        let body = "Unicode: 🎉";
-
-        // Verify all platforms preserve unicode
-        let (_, notify_args) = build_notify_send_command(title, body);
-        assert_eq!(notify_args[0], title);
-        assert_eq!(notify_args[1], body);
-
-        let (_, ps_args) = build_windows_powershell_command(title, body);
-        let ps_script = &ps_args[4];
-        assert!(ps_script.contains(title));
-        assert!(ps_script.contains(body));
-
-        let (_, msg_args) = build_simple_notification_command(title, body);
-        assert!(msg_args[3].contains(title));
-        assert!(msg_args[3].contains(body));
-    }
-
-    #[test]
-    fn test_e2e_empty_input_handling() {
-        // Test that empty inputs are handled gracefully
-        let (_, notify_args) = build_notify_send_command("", "");
-        assert_eq!(notify_args[0], "");
-        assert_eq!(notify_args[1], "");
-
-        let (_, ps_args) = build_windows_powershell_command("", "");
-        let ps_script = &ps_args[4];
-        assert!(ps_script.contains("Hikari Desktop")); // Still has app name
-
-        let (_, msg_args) = build_simple_notification_command("", "");
-        assert_eq!(msg_args[3], "\n\n");
-    }
-}

src-tauri/src/process_ext.rs

@@ -1,21 +0,0 @@
-use std::process::Command;
-
-/// Extension trait for `Command` that hides the console window on Windows.
-///
-/// On non-Windows platforms this is a no-op, so callers can unconditionally
-/// chain `.hide_window()` without any `#[cfg]` guards at the call sites.
-pub trait HideWindow {
-    fn hide_window(&mut self) -> &mut Self;
-}
-
-impl HideWindow for Command {
-    fn hide_window(&mut self) -> &mut Self {
-        #[cfg(target_os = "windows")]
-        {
-            use std::os::windows::process::CommandExt;
-            const CREATE_NO_WINDOW: u32 = 0x08000000;
-            self.creation_flags(CREATE_NO_WINDOW);
-        }
-        self
-    }
-}

src-tauri/src/quick_actions.rs

@@ -171,18 +171,6 @@ pub async fn reset_default_quick_actions(app: AppHandle) -> Result<(), String> {
 mod tests {
     use super::*;
 
-    fn create_test_action(id: &str, name: &str, is_default: bool) -> QuickAction {
-        QuickAction {
-            id: id.to_string(),
-            name: name.to_string(),
-            prompt: "Test prompt".to_string(),
-            icon: "star".to_string(),
-            is_default,
-            created_at: Utc::now(),
-            updated_at: Utc::now(),
-        }
-    }
-
     #[test]
     fn test_default_quick_actions_exist() {
         let defaults = get_default_quick_actions();
@@ -200,174 +188,4 @@ mod tests {
             assert!(!action.icon.is_empty());
         }
     }
-
-    #[test]
-    fn test_default_quick_actions_count() {
-        let defaults = get_default_quick_actions();
-        // Should have 6 default actions
-        assert_eq!(defaults.len(), 6);
-    }
-
-    #[test]
-    fn test_default_quick_actions_have_unique_ids() {
-        let defaults = get_default_quick_actions();
-        let mut ids: Vec<&String> = defaults.iter().map(|a| &a.id).collect();
-        ids.sort();
-        ids.dedup();
-        assert_eq!(ids.len(), defaults.len());
-    }
-
-    #[test]
-    fn test_default_quick_actions_ids_start_with_default() {
-        let defaults = get_default_quick_actions();
-        assert!(defaults.iter().all(|a| a.id.starts_with("default-")));
-    }
-
-    #[test]
-    fn test_quick_action_serialization() {
-        let action = create_test_action("test-1", "Test Action", false);
-        let json = serde_json::to_string(&action).expect("Failed to serialize");
-        let parsed: QuickAction = serde_json::from_str(&json).expect("Failed to deserialize");
-
-        assert_eq!(parsed.id, action.id);
-        assert_eq!(parsed.name, action.name);
-        assert_eq!(parsed.prompt, action.prompt);
-        assert_eq!(parsed.icon, action.icon);
-        assert_eq!(parsed.is_default, action.is_default);
-    }
-
-    #[test]
-    fn test_quick_action_clone() {
-        let original = create_test_action("clone-test", "Clone Test", true);
-        let cloned = original.clone();
-
-        assert_eq!(original.id, cloned.id);
-        assert_eq!(original.name, cloned.name);
-        assert_eq!(original.is_default, cloned.is_default);
-    }
-
-    #[test]
-    #[allow(clippy::useless_vec)]
-    fn test_quick_action_sorting_defaults_first() {
-        let mut actions = vec![
-            create_test_action("custom-z", "Zebra", false),
-            create_test_action("default-a", "Apple", true),
-            create_test_action("custom-a", "Alpha", false),
-            create_test_action("default-z", "Zulu", true),
-        ];
-
-        // Sort by: defaults first, then alphabetically by name
-        actions.sort_by(|a, b| {
-            let default_cmp = b.is_default.cmp(&a.is_default);
-            if default_cmp == std::cmp::Ordering::Equal {
-                a.name.cmp(&b.name)
-            } else {
-                default_cmp
-            }
-        });
-
-        // Defaults should come first
-        assert!(actions[0].is_default);
-        assert!(actions[1].is_default);
-        assert!(!actions[2].is_default);
-        assert!(!actions[3].is_default);
-
-        // Within defaults, alphabetically sorted
-        assert_eq!(actions[0].name, "Apple");
-        assert_eq!(actions[1].name, "Zulu");
-
-        // Within non-defaults, alphabetically sorted
-        assert_eq!(actions[2].name, "Alpha");
-        assert_eq!(actions[3].name, "Zebra");
-    }
-
-    #[test]
-    fn test_known_default_actions() {
-        let defaults = get_default_quick_actions();
-        let ids: Vec<&str> = defaults.iter().map(|a| a.id.as_str()).collect();
-
-        assert!(ids.contains(&"default-review-pr"));
-        assert!(ids.contains(&"default-run-tests"));
-        assert!(ids.contains(&"default-explain-file"));
-        assert!(ids.contains(&"default-fix-error"));
-        assert!(ids.contains(&"default-write-tests"));
-        assert!(ids.contains(&"default-refactor"));
-    }
-
-    #[test]
-    fn test_default_action_icons() {
-        let defaults = get_default_quick_actions();
-        let icons: Vec<&str> = defaults.iter().map(|a| a.icon.as_str()).collect();
-
-        assert!(icons.contains(&"git-pull-request"));
-        assert!(icons.contains(&"play"));
-        assert!(icons.contains(&"file-text"));
-        assert!(icons.contains(&"alert-circle"));
-        assert!(icons.contains(&"check-square"));
-        assert!(icons.contains(&"refresh-cw"));
-    }
-
-    #[test]
-    fn test_quick_action_prompts_not_empty() {
-        let defaults = get_default_quick_actions();
-        for action in defaults {
-            assert!(
-                action.prompt.len() > 10,
-                "Prompt should be meaningful: {}",
-                action.name
-            );
-        }
-    }
-
-    #[test]
-    fn test_quick_action_timestamps() {
-        let action = create_test_action("time-test", "Time Test", false);
-        assert!(action.created_at <= action.updated_at);
-    }
-
-    #[test]
-    fn test_default_actions_have_same_timestamps() {
-        let defaults = get_default_quick_actions();
-        // All defaults are created at the same instant
-        let first_created = defaults[0].created_at;
-        let first_updated = defaults[0].updated_at;
-
-        for action in &defaults {
-            assert_eq!(action.created_at, first_created);
-            assert_eq!(action.updated_at, first_updated);
-        }
-    }
-
-    #[test]
-    fn test_action_retain_non_default() {
-        let mut actions = vec![
-            create_test_action("default-1", "Default 1", true),
-            create_test_action("custom-1", "Custom 1", false),
-            create_test_action("default-2", "Default 2", true),
-            create_test_action("custom-2", "Custom 2", false),
-        ];
-
-        // Mimics reset_default_quick_actions behavior (retain non-defaults)
-        actions.retain(|a| !a.is_default);
-
-        assert_eq!(actions.len(), 2);
-        assert!(actions.iter().all(|a| !a.is_default));
-    }
-
-    #[test]
-    #[allow(clippy::useless_vec)]
-    fn test_action_find_by_id() {
-        let actions = vec![
-            create_test_action("action-1", "First", false),
-            create_test_action("action-2", "Second", false),
-            create_test_action("action-3", "Third", false),
-        ];
-
-        let found = actions.iter().find(|a| a.id == "action-2");
-        assert!(found.is_some());
-        assert_eq!(found.unwrap().name, "Second");
-
-        let not_found = actions.iter().find(|a| a.id == "action-999");
-        assert!(not_found.is_none());
-    }
 }

src-tauri/src/sessions.rs

@@ -145,30 +145,6 @@ pub async fn clear_all_sessions(app: AppHandle) -> Result<(), String> {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use chrono::TimeZone;
-
-    fn create_test_session(id: &str, name: &str) -> SavedSession {
-        SavedSession {
-            id: id.to_string(),
-            name: name.to_string(),
-            created_at: Utc::now(),
-            last_activity_at: Utc::now(),
-            working_directory: "/home/test".to_string(),
-            message_count: 5,
-            preview: "Hello world".to_string(),
-            messages: vec![],
-        }
-    }
-
-    fn create_test_message(id: &str, content: &str, msg_type: &str) -> SavedMessage {
-        SavedMessage {
-            id: id.to_string(),
-            message_type: msg_type.to_string(),
-            content: content.to_string(),
-            timestamp: Utc::now(),
-            tool_name: None,
-        }
-    }
 
     #[test]
     fn test_session_list_item_from_saved_session() {
@@ -188,187 +164,4 @@ mod tests {
         assert_eq!(item.name, "Test Session");
         assert_eq!(item.message_count, 5);
     }
-
-    #[test]
-    fn test_session_list_item_preserves_all_fields() {
-        let created = Utc.with_ymd_and_hms(2024, 1, 15, 10, 30, 0).unwrap();
-        let last_activity = Utc.with_ymd_and_hms(2024, 1, 15, 14, 45, 0).unwrap();
-
-        let session = SavedSession {
-            id: "sess-123".to_string(),
-            name: "My Chat".to_string(),
-            created_at: created,
-            last_activity_at: last_activity,
-            working_directory: "/home/naomi/project".to_string(),
-            message_count: 42,
-            preview: "What is the meaning of life?".to_string(),
-            messages: vec![],
-        };
-
-        let item = SessionListItem::from(&session);
-
-        assert_eq!(item.id, "sess-123");
-        assert_eq!(item.name, "My Chat");
-        assert_eq!(item.created_at, created);
-        assert_eq!(item.last_activity_at, last_activity);
-        assert_eq!(item.working_directory, "/home/naomi/project");
-        assert_eq!(item.message_count, 42);
-        assert_eq!(item.preview, "What is the meaning of life?");
-    }
-
-    #[test]
-    fn test_saved_session_serialization() {
-        let session = create_test_session("test-1", "Test Session");
-        let json = serde_json::to_string(&session).expect("Failed to serialize");
-        let parsed: SavedSession = serde_json::from_str(&json).expect("Failed to deserialize");
-
-        assert_eq!(parsed.id, session.id);
-        assert_eq!(parsed.name, session.name);
-        assert_eq!(parsed.working_directory, session.working_directory);
-    }
-
-    #[test]
-    fn test_saved_message_serialization() {
-        let message = create_test_message("msg-1", "Hello!", "user");
-        let json = serde_json::to_string(&message).expect("Failed to serialize");
-        let parsed: SavedMessage = serde_json::from_str(&json).expect("Failed to deserialize");
-
-        assert_eq!(parsed.id, message.id);
-        assert_eq!(parsed.content, message.content);
-        assert_eq!(parsed.message_type, "user");
-    }
-
-    #[test]
-    fn test_saved_message_with_tool_name() {
-        let message = SavedMessage {
-            id: "msg-tool-1".to_string(),
-            message_type: "tool".to_string(),
-            content: "File read successfully".to_string(),
-            timestamp: Utc::now(),
-            tool_name: Some("Read".to_string()),
-        };
-
-        let json = serde_json::to_string(&message).expect("Failed to serialize");
-        let parsed: SavedMessage = serde_json::from_str(&json).expect("Failed to deserialize");
-
-        assert_eq!(parsed.tool_name, Some("Read".to_string()));
-    }
-
-    #[test]
-    fn test_session_with_messages_serialization() {
-        let mut session = create_test_session("sess-full", "Full Session");
-        session.messages = vec![
-            create_test_message("msg-1", "Hello!", "user"),
-            create_test_message("msg-2", "Hi there!", "assistant"),
-            create_test_message("msg-3", "Read file", "tool"),
-        ];
-        session.message_count = 3;
-
-        let json = serde_json::to_string(&session).expect("Failed to serialize");
-        let parsed: SavedSession = serde_json::from_str(&json).expect("Failed to deserialize");
-
-        assert_eq!(parsed.messages.len(), 3);
-        assert_eq!(parsed.messages[0].content, "Hello!");
-        assert_eq!(parsed.messages[1].message_type, "assistant");
-        assert_eq!(parsed.messages[2].message_type, "tool");
-    }
-
-    #[test]
-    fn test_session_list_item_serialization() {
-        let item = SessionListItem {
-            id: "list-item-1".to_string(),
-            name: "Quick Chat".to_string(),
-            created_at: Utc::now(),
-            last_activity_at: Utc::now(),
-            working_directory: "/tmp".to_string(),
-            message_count: 10,
-            preview: "Short preview...".to_string(),
-        };
-
-        let json = serde_json::to_string(&item).expect("Failed to serialize");
-        let parsed: SessionListItem = serde_json::from_str(&json).expect("Failed to deserialize");
-
-        assert_eq!(parsed.id, item.id);
-        assert_eq!(parsed.name, item.name);
-        assert_eq!(parsed.preview, item.preview);
-    }
-
-    #[test]
-    fn test_message_type_field_rename() {
-        // The message_type field is renamed to "type" in JSON
-        let message = create_test_message("msg-1", "Test", "assistant");
-        let json = serde_json::to_string(&message).expect("Failed to serialize");
-
-        assert!(json.contains("\"type\":"));
-        assert!(!json.contains("\"message_type\":"));
-    }
-
-    #[test]
-    fn test_session_default_empty_messages() {
-        let session = SavedSession {
-            id: "empty".to_string(),
-            name: "Empty".to_string(),
-            created_at: Utc::now(),
-            last_activity_at: Utc::now(),
-            working_directory: "/".to_string(),
-            message_count: 0,
-            preview: "".to_string(),
-            messages: vec![],
-        };
-
-        assert!(session.messages.is_empty());
-        assert_eq!(session.message_count, 0);
-    }
-
-    #[test]
-    #[allow(clippy::useless_vec)]
-    fn test_session_sorting_by_activity() {
-        let old_time = Utc.with_ymd_and_hms(2024, 1, 1, 0, 0, 0).unwrap();
-        let new_time = Utc.with_ymd_and_hms(2024, 6, 15, 12, 0, 0).unwrap();
-
-        let mut sessions = vec![
-            SessionListItem {
-                id: "old".to_string(),
-                name: "Old Session".to_string(),
-                created_at: old_time,
-                last_activity_at: old_time,
-                working_directory: "/old".to_string(),
-                message_count: 1,
-                preview: "Old".to_string(),
-            },
-            SessionListItem {
-                id: "new".to_string(),
-                name: "New Session".to_string(),
-                created_at: new_time,
-                last_activity_at: new_time,
-                working_directory: "/new".to_string(),
-                message_count: 1,
-                preview: "New".to_string(),
-            },
-        ];
-
-        // Sort by last activity, most recent first (mimics list_sessions behavior)
-        sessions.sort_by(|a, b| b.last_activity_at.cmp(&a.last_activity_at));
-
-        assert_eq!(sessions[0].id, "new");
-        assert_eq!(sessions[1].id, "old");
-    }
-
-    #[test]
-    fn test_session_clone() {
-        let original = create_test_session("clone-test", "Clone Test");
-        let cloned = original.clone();
-
-        assert_eq!(original.id, cloned.id);
-        assert_eq!(original.name, cloned.name);
-    }
-
-    #[test]
-    fn test_message_clone() {
-        let original = create_test_message("msg-clone", "Content", "user");
-        let cloned = original.clone();
-
-        assert_eq!(original.id, cloned.id);
-        assert_eq!(original.content, cloned.content);
-    }
 }

src-tauri/src/snippets.rs

@@ -205,19 +205,6 @@ pub async fn reset_default_snippets(app: AppHandle) -> Result<(), String> {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use std::collections::HashSet;
-
-    fn create_test_snippet(id: &str, name: &str, category: &str, is_default: bool) -> Snippet {
-        Snippet {
-            id: id.to_string(),
-            name: name.to_string(),
-            content: "Test content".to_string(),
-            category: category.to_string(),
-            is_default,
-            created_at: Utc::now(),
-            updated_at: Utc::now(),
-        }
-    }
 
     #[test]
     fn test_default_snippets_exist() {
@@ -236,204 +223,4 @@ mod tests {
             assert!(!snippet.category.is_empty());
         }
     }
-
-    #[test]
-    fn test_default_snippets_count() {
-        let defaults = get_default_snippets();
-        // Should have 8 default snippets
-        assert_eq!(defaults.len(), 8);
-    }
-
-    #[test]
-    fn test_default_snippets_have_unique_ids() {
-        let defaults = get_default_snippets();
-        let ids: HashSet<&String> = defaults.iter().map(|s| &s.id).collect();
-        assert_eq!(ids.len(), defaults.len());
-    }
-
-    #[test]
-    fn test_default_snippets_ids_start_with_default() {
-        let defaults = get_default_snippets();
-        assert!(defaults.iter().all(|s| s.id.starts_with("default-")));
-    }
-
-    #[test]
-    fn test_snippet_serialization() {
-        let snippet = create_test_snippet("test-1", "Test Snippet", "Testing", false);
-        let json = serde_json::to_string(&snippet).expect("Failed to serialize");
-        let parsed: Snippet = serde_json::from_str(&json).expect("Failed to deserialize");
-
-        assert_eq!(parsed.id, snippet.id);
-        assert_eq!(parsed.name, snippet.name);
-        assert_eq!(parsed.content, snippet.content);
-        assert_eq!(parsed.category, snippet.category);
-        assert_eq!(parsed.is_default, snippet.is_default);
-    }
-
-    #[test]
-    fn test_snippet_clone() {
-        let original = create_test_snippet("clone-test", "Clone Test", "Category", true);
-        let cloned = original.clone();
-
-        assert_eq!(original.id, cloned.id);
-        assert_eq!(original.name, cloned.name);
-        assert_eq!(original.is_default, cloned.is_default);
-    }
-
-    #[test]
-    #[allow(clippy::useless_vec)]
-    fn test_snippet_sorting_by_category_then_name() {
-        let mut snippets = vec![
-            create_test_snippet("s1", "Zebra", "B-Category", false),
-            create_test_snippet("s2", "Apple", "A-Category", false),
-            create_test_snippet("s3", "Banana", "B-Category", false),
-            create_test_snippet("s4", "Alpha", "A-Category", false),
-        ];
-
-        // Sort by category, then by name (mimics list_snippets behavior)
-        snippets.sort_by(|a, b| {
-            let cat_cmp = a.category.cmp(&b.category);
-            if cat_cmp == std::cmp::Ordering::Equal {
-                a.name.cmp(&b.name)
-            } else {
-                cat_cmp
-            }
-        });
-
-        // A-Category should come first
-        assert_eq!(snippets[0].category, "A-Category");
-        assert_eq!(snippets[1].category, "A-Category");
-        assert_eq!(snippets[2].category, "B-Category");
-        assert_eq!(snippets[3].category, "B-Category");
-
-        // Within categories, alphabetically by name
-        assert_eq!(snippets[0].name, "Alpha");
-        assert_eq!(snippets[1].name, "Apple");
-        assert_eq!(snippets[2].name, "Banana");
-        assert_eq!(snippets[3].name, "Zebra");
-    }
-
-    #[test]
-    fn test_known_default_snippets() {
-        let defaults = get_default_snippets();
-        let ids: Vec<&str> = defaults.iter().map(|s| s.id.as_str()).collect();
-
-        assert!(ids.contains(&"default-explain-code"));
-        assert!(ids.contains(&"default-fix-error"));
-        assert!(ids.contains(&"default-write-tests"));
-        assert!(ids.contains(&"default-refactor"));
-        assert!(ids.contains(&"default-optimize"));
-        assert!(ids.contains(&"default-review-pr"));
-        assert!(ids.contains(&"default-add-comments"));
-        assert!(ids.contains(&"default-security-review"));
-    }
-
-    #[test]
-    fn test_default_snippet_categories() {
-        let defaults = get_default_snippets();
-        let categories: HashSet<&String> = defaults.iter().map(|s| &s.category).collect();
-
-        assert!(categories.contains(&"Code Review".to_string()));
-        assert!(categories.contains(&"Debugging".to_string()));
-        assert!(categories.contains(&"Testing".to_string()));
-        assert!(categories.contains(&"Performance".to_string()));
-        assert!(categories.contains(&"Documentation".to_string()));
-        assert!(categories.contains(&"Security".to_string()));
-    }
-
-    #[test]
-    fn test_snippet_content_not_empty() {
-        let defaults = get_default_snippets();
-        for snippet in defaults {
-            assert!(
-                snippet.content.len() > 10,
-                "Content should be meaningful: {}",
-                snippet.name
-            );
-        }
-    }
-
-    #[test]
-    fn test_snippet_timestamps() {
-        let snippet = create_test_snippet("time-test", "Time Test", "Cat", false);
-        assert!(snippet.created_at <= snippet.updated_at);
-    }
-
-    #[test]
-    fn test_default_snippets_have_same_timestamps() {
-        let defaults = get_default_snippets();
-        // All defaults are created at the same instant
-        let first_created = defaults[0].created_at;
-        let first_updated = defaults[0].updated_at;
-
-        for snippet in &defaults {
-            assert_eq!(snippet.created_at, first_created);
-            assert_eq!(snippet.updated_at, first_updated);
-        }
-    }
-
-    #[test]
-    fn test_snippet_retain_non_default() {
-        let mut snippets = vec![
-            create_test_snippet("default-1", "Default 1", "Cat", true),
-            create_test_snippet("custom-1", "Custom 1", "Cat", false),
-            create_test_snippet("default-2", "Default 2", "Cat", true),
-            create_test_snippet("custom-2", "Custom 2", "Cat", false),
-        ];
-
-        // Mimics reset_default_snippets behavior (retain non-defaults)
-        snippets.retain(|s| !s.is_default);
-
-        assert_eq!(snippets.len(), 2);
-        assert!(snippets.iter().all(|s| !s.is_default));
-    }
-
-    #[test]
-    #[allow(clippy::useless_vec)]
-    fn test_snippet_find_by_id() {
-        let snippets = vec![
-            create_test_snippet("snippet-1", "First", "Cat", false),
-            create_test_snippet("snippet-2", "Second", "Cat", false),
-            create_test_snippet("snippet-3", "Third", "Cat", false),
-        ];
-
-        let found = snippets.iter().find(|s| s.id == "snippet-2");
-        assert!(found.is_some());
-        assert_eq!(found.unwrap().name, "Second");
-
-        let not_found = snippets.iter().find(|s| s.id == "snippet-999");
-        assert!(not_found.is_none());
-    }
-
-    #[test]
-    #[allow(clippy::useless_vec)]
-    fn test_extract_categories_sorted_and_deduped() {
-        let snippets = vec![
-            create_test_snippet("s1", "S1", "Zebra", false),
-            create_test_snippet("s2", "S2", "Alpha", false),
-            create_test_snippet("s3", "S3", "Beta", false),
-            create_test_snippet("s4", "S4", "Alpha", false), // Duplicate
-        ];
-
-        let mut categories: Vec<String> = snippets.iter().map(|s| s.category.clone()).collect();
-        categories.sort();
-        categories.dedup();
-
-        assert_eq!(categories.len(), 3);
-        assert_eq!(categories[0], "Alpha");
-        assert_eq!(categories[1], "Beta");
-        assert_eq!(categories[2], "Zebra");
-    }
-
-    #[test]
-    fn test_snippet_category_code_review_count() {
-        let defaults = get_default_snippets();
-        let code_review_count = defaults
-            .iter()
-            .filter(|s| s.category == "Code Review")
-            .count();
-
-        // There should be multiple code review snippets
-        assert!(code_review_count >= 2);
-    }
 }

src-tauri/src/temp_manager.rs

@@ -77,8 +77,8 @@ impl TempFileManager {
             for file_path in files {
                 if file_path.exists() {
                     if let Err(e) = fs::remove_file(&file_path) {
-                        tracing::warn!(
-                            "Failed to remove temp file {:?}: {}",
+                        eprintln!(
+                            "Warning: Failed to remove temp file {:?}: {}",
                             file_path, e
                         );
                     }
@@ -115,7 +115,7 @@ impl TempFileManager {
             let path = entry.path();
             if path.is_file() && !tracked_files.contains(&path) {
                 if let Err(e) = fs::remove_file(&path) {
-                    tracing::warn!("Failed to remove orphaned file {:?}: {}", path, e);
+                    eprintln!("Warning: Failed to remove orphaned file {:?}: {}", path, e);
                 } else {
                     cleaned_count += 1;
                 }
@@ -137,290 +137,3 @@ pub type SharedTempFileManager = Arc<Mutex<TempFileManager>>;
 pub fn create_shared_temp_manager() -> Result<SharedTempFileManager, String> {
     Ok(Arc::new(Mutex::new(TempFileManager::new()?)))
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use std::fs;
-    use tempfile::TempDir;
-
-    // Helper to create a TempFileManager with a custom base directory for testing
-    fn create_test_manager(base_dir: PathBuf) -> TempFileManager {
-        if !base_dir.exists() {
-            fs::create_dir_all(&base_dir).expect("Failed to create test temp dir");
-        }
-        TempFileManager {
-            base_dir,
-            files: HashMap::new(),
-        }
-    }
-
-    #[test]
-    fn test_new_creates_base_directory() {
-        let manager = TempFileManager::new().expect("Failed to create TempFileManager");
-        assert!(manager.base_dir.exists());
-    }
-
-    #[test]
-    fn test_get_base_dir_returns_correct_path() {
-        let temp_dir = TempDir::new().expect("Failed to create temp dir");
-        let base_path = temp_dir.path().join("hikari-test");
-        let manager = create_test_manager(base_path.clone());
-
-        assert_eq!(manager.get_base_dir(), base_path.as_path());
-    }
-
-    #[test]
-    fn test_save_file_creates_file_with_content() {
-        let temp_dir = TempDir::new().expect("Failed to create temp dir");
-        let base_path = temp_dir.path().join("hikari-test");
-        let mut manager = create_test_manager(base_path);
-
-        let data = b"Hello, world!";
-        let result = manager.save_file("conv-1", data, Some("test.txt"));
-
-        assert!(result.is_ok());
-        let file_path = result.unwrap();
-        assert!(file_path.exists());
-
-        let content = fs::read(&file_path).expect("Failed to read file");
-        assert_eq!(content, data);
-    }
-
-    #[test]
-    fn test_save_file_uses_correct_extension() {
-        let temp_dir = TempDir::new().expect("Failed to create temp dir");
-        let base_path = temp_dir.path().join("hikari-test");
-        let mut manager = create_test_manager(base_path);
-
-        let data = b"test data";
-        let result = manager.save_file("conv-1", data, Some("document.pdf"));
-
-        assert!(result.is_ok());
-        let file_path = result.unwrap();
-        assert_eq!(file_path.extension().unwrap(), "pdf");
-    }
-
-    #[test]
-    fn test_save_file_uses_bin_extension_when_no_filename() {
-        let temp_dir = TempDir::new().expect("Failed to create temp dir");
-        let base_path = temp_dir.path().join("hikari-test");
-        let mut manager = create_test_manager(base_path);
-
-        let data = b"binary data";
-        let result = manager.save_file("conv-1", data, None);
-
-        assert!(result.is_ok());
-        let file_path = result.unwrap();
-        assert_eq!(file_path.extension().unwrap(), "bin");
-    }
-
-    #[test]
-    fn test_register_file_tracks_file_path() {
-        let temp_dir = TempDir::new().expect("Failed to create temp dir");
-        let base_path = temp_dir.path().join("hikari-test");
-        let mut manager = create_test_manager(base_path);
-
-        let file_path = PathBuf::from("/some/path/file.txt");
-        manager.register_file("conv-1", file_path.clone());
-
-        let files = manager.get_files_for_conversation("conv-1");
-        assert_eq!(files.len(), 1);
-        assert_eq!(files[0], file_path);
-    }
-
-    #[test]
-    fn test_get_files_for_conversation_returns_empty_for_unknown() {
-        let temp_dir = TempDir::new().expect("Failed to create temp dir");
-        let base_path = temp_dir.path().join("hikari-test");
-        let manager = create_test_manager(base_path);
-
-        let files = manager.get_files_for_conversation("unknown-conv");
-        assert!(files.is_empty());
-    }
-
-    #[test]
-    fn test_get_files_for_conversation_returns_all_files() {
-        let temp_dir = TempDir::new().expect("Failed to create temp dir");
-        let base_path = temp_dir.path().join("hikari-test");
-        let mut manager = create_test_manager(base_path);
-
-        let data = b"test";
-        manager.save_file("conv-1", data, Some("file1.txt")).unwrap();
-        manager.save_file("conv-1", data, Some("file2.txt")).unwrap();
-        manager.save_file("conv-2", data, Some("file3.txt")).unwrap();
-
-        let files_conv1 = manager.get_files_for_conversation("conv-1");
-        let files_conv2 = manager.get_files_for_conversation("conv-2");
-
-        assert_eq!(files_conv1.len(), 2);
-        assert_eq!(files_conv2.len(), 1);
-    }
-
-    #[test]
-    fn test_cleanup_conversation_removes_files() {
-        let temp_dir = TempDir::new().expect("Failed to create temp dir");
-        let base_path = temp_dir.path().join("hikari-test");
-        let mut manager = create_test_manager(base_path);
-
-        let data = b"test";
-        let file_path = manager.save_file("conv-1", data, Some("test.txt")).unwrap();
-        assert!(file_path.exists());
-
-        let result = manager.cleanup_conversation("conv-1");
-        assert!(result.is_ok());
-        assert!(!file_path.exists());
-        assert!(manager.get_files_for_conversation("conv-1").is_empty());
-    }
-
-    #[test]
-    fn test_cleanup_conversation_handles_missing_files() {
-        let temp_dir = TempDir::new().expect("Failed to create temp dir");
-        let base_path = temp_dir.path().join("hikari-test");
-        let mut manager = create_test_manager(base_path);
-
-        // Register a file that doesn't exist
-        manager.register_file("conv-1", PathBuf::from("/nonexistent/file.txt"));
-
-        // Should not error, just skip missing files
-        let result = manager.cleanup_conversation("conv-1");
-        assert!(result.is_ok());
-    }
-
-    #[test]
-    fn test_cleanup_conversation_for_unknown_returns_ok() {
-        let temp_dir = TempDir::new().expect("Failed to create temp dir");
-        let base_path = temp_dir.path().join("hikari-test");
-        let mut manager = create_test_manager(base_path);
-
-        let result = manager.cleanup_conversation("unknown-conv");
-        assert!(result.is_ok());
-    }
-
-    #[test]
-    fn test_cleanup_all_removes_all_files() {
-        let temp_dir = TempDir::new().expect("Failed to create temp dir");
-        let base_path = temp_dir.path().join("hikari-test");
-        let mut manager = create_test_manager(base_path);
-
-        let data = b"test";
-        let file1 = manager.save_file("conv-1", data, Some("f1.txt")).unwrap();
-        let file2 = manager.save_file("conv-2", data, Some("f2.txt")).unwrap();
-
-        assert!(file1.exists());
-        assert!(file2.exists());
-
-        let result = manager.cleanup_all();
-        assert!(result.is_ok());
-
-        assert!(!file1.exists());
-        assert!(!file2.exists());
-        assert!(manager.files.is_empty());
-    }
-
-    #[test]
-    fn test_cleanup_orphaned_files_removes_untracked() {
-        let temp_dir = TempDir::new().expect("Failed to create temp dir");
-        let base_path = temp_dir.path().join("hikari-test");
-        let mut manager = create_test_manager(base_path.clone());
-
-        // Create a tracked file
-        let data = b"tracked";
-        let tracked_path = manager.save_file("conv-1", data, Some("tracked.txt")).unwrap();
-
-        // Create an untracked (orphaned) file directly in the temp directory
-        let orphan_path = base_path.join("orphan.txt");
-        fs::write(&orphan_path, b"orphan").expect("Failed to create orphan file");
-
-        assert!(tracked_path.exists());
-        assert!(orphan_path.exists());
-
-        let result = manager.cleanup_orphaned_files();
-        assert!(result.is_ok());
-        assert_eq!(result.unwrap(), 1); // One orphan removed
-
-        assert!(tracked_path.exists()); // Tracked file still exists
-        assert!(!orphan_path.exists()); // Orphan removed
-    }
-
-    #[test]
-    fn test_cleanup_orphaned_returns_zero_when_none() {
-        let temp_dir = TempDir::new().expect("Failed to create temp dir");
-        let base_path = temp_dir.path().join("hikari-test");
-        let mut manager = create_test_manager(base_path);
-
-        let data = b"test";
-        manager.save_file("conv-1", data, Some("test.txt")).unwrap();
-
-        let result = manager.cleanup_orphaned_files();
-        assert!(result.is_ok());
-        assert_eq!(result.unwrap(), 0);
-    }
-
-    #[test]
-    fn test_cleanup_orphaned_returns_zero_when_dir_missing() {
-        let mut manager = TempFileManager {
-            base_dir: PathBuf::from("/nonexistent/dir"),
-            files: HashMap::new(),
-        };
-
-        let result = manager.cleanup_orphaned_files();
-        assert!(result.is_ok());
-        assert_eq!(result.unwrap(), 0);
-    }
-
-    #[test]
-    fn test_default_creates_manager() {
-        // Default should work as long as we can create temp directories
-        let manager = TempFileManager::default();
-        assert!(manager.base_dir.exists());
-    }
-
-    #[test]
-    fn test_create_shared_temp_manager() {
-        let result = create_shared_temp_manager();
-        assert!(result.is_ok());
-
-        let shared = result.unwrap();
-        let manager = shared.lock();
-        assert!(manager.base_dir.exists());
-    }
-
-    #[test]
-    fn test_multiple_files_same_conversation() {
-        let temp_dir = TempDir::new().expect("Failed to create temp dir");
-        let base_path = temp_dir.path().join("hikari-test");
-        let mut manager = create_test_manager(base_path);
-
-        // Save multiple files to same conversation
-        for i in 0..5 {
-            let data = format!("content {}", i);
-            manager
-                .save_file("conv-1", data.as_bytes(), Some(&format!("file{}.txt", i)))
-                .unwrap();
-        }
-
-        let files = manager.get_files_for_conversation("conv-1");
-        assert_eq!(files.len(), 5);
-
-        // Each file should have unique content
-        for (i, file_path) in files.iter().enumerate() {
-            let content = fs::read_to_string(file_path).expect("Failed to read");
-            assert_eq!(content, format!("content {}", i));
-        }
-    }
-
-    #[test]
-    fn test_file_paths_contain_conversation_id() {
-        let temp_dir = TempDir::new().expect("Failed to create temp dir");
-        let base_path = temp_dir.path().join("hikari-test");
-        let mut manager = create_test_manager(base_path);
-
-        let file_path = manager
-            .save_file("my-conversation-id", b"test", Some("test.txt"))
-            .unwrap();
-
-        let filename = file_path.file_name().unwrap().to_str().unwrap();
-        assert!(filename.starts_with("my-conversation-id_"));
-    }
-}

src-tauri/src/tool_cache.rs

@@ -1,266 +0,0 @@
-use serde::{Deserialize, Serialize};
-use std::collections::hash_map::DefaultHasher;
-use std::collections::HashMap;
-use std::hash::{Hash, Hasher};
-
-/// Tools that could benefit from caching
-#[allow(dead_code)]
-#[derive(Debug, Clone, Copy, PartialEq, Eq)]
-pub enum CacheableTool {
-    Read,
-    Glob,
-    Grep,
-}
-
-impl CacheableTool {
-    #[allow(dead_code)]
-    pub fn from_name(name: &str) -> Option<Self> {
-        match name {
-            "Read" => Some(Self::Read),
-            "Glob" => Some(Self::Glob),
-            "Grep" => Some(Self::Grep),
-            _ => None,
-        }
-    }
-}
-
-/// Statistics about potential cache savings
-#[allow(dead_code)]
-#[derive(Debug, Default, Clone, Serialize, Deserialize)]
-pub struct CacheAnalytics {
-    /// Number of tool calls that could have been cache hits
-    pub potential_cache_hits: u64,
-    /// Estimated tokens that could have been saved
-    pub potential_savings_tokens: u64,
-    /// Tracks unique tool invocations: hash -> (tool_name, call_count)
-    #[serde(skip)]
-    recent_invocations: HashMap<u64, (String, u64)>,
-}
-
-#[allow(dead_code)]
-impl CacheAnalytics {
-    pub fn new() -> Self {
-        Self::default()
-    }
-
-    /// Compute a hash key from tool name and input
-    fn compute_key(tool_name: &str, input: &serde_json::Value) -> u64 {
-        let mut hasher = DefaultHasher::new();
-        tool_name.hash(&mut hasher);
-        input.to_string().hash(&mut hasher);
-        hasher.finish()
-    }
-
-    /// Track a tool invocation for analytics
-    /// Returns true if this was a repeated invocation (potential cache hit)
-    pub fn track_invocation(
-        &mut self,
-        tool_name: &str,
-        input: &serde_json::Value,
-        estimated_tokens: u64,
-    ) -> bool {
-        // Only track cacheable tools
-        if CacheableTool::from_name(tool_name).is_none() {
-            return false;
-        }
-
-        let key = Self::compute_key(tool_name, input);
-
-        if let Some((_, count)) = self.recent_invocations.get_mut(&key) {
-            *count += 1;
-            // This is a repeat - could have been a cache hit
-            self.potential_cache_hits += 1;
-            self.potential_savings_tokens += estimated_tokens;
-            true
-        } else {
-            self.recent_invocations
-                .insert(key, (tool_name.to_string(), 1));
-            false
-        }
-    }
-
-    /// Get the number of unique tool invocations being tracked
-    pub fn unique_invocations(&self) -> usize {
-        self.recent_invocations.len()
-    }
-
-    /// Get invocations that were called more than once
-    pub fn repeated_invocations(&self) -> Vec<(&str, u64)> {
-        self.recent_invocations
-            .values()
-            .filter(|(_, count)| *count > 1)
-            .map(|(name, count)| (name.as_str(), *count))
-            .collect()
-    }
-
-    /// Clear session analytics (keep totals)
-    pub fn clear_session(&mut self) {
-        self.recent_invocations.clear();
-    }
-
-    /// Fully reset all analytics
-    pub fn reset(&mut self) {
-        self.potential_cache_hits = 0;
-        self.potential_savings_tokens = 0;
-        self.recent_invocations.clear();
-    }
-}
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use serde_json::json;
-
-    #[test]
-    fn test_cacheable_tool_from_name() {
-        assert_eq!(CacheableTool::from_name("Read"), Some(CacheableTool::Read));
-        assert_eq!(CacheableTool::from_name("Glob"), Some(CacheableTool::Glob));
-        assert_eq!(CacheableTool::from_name("Grep"), Some(CacheableTool::Grep));
-        assert_eq!(CacheableTool::from_name("Bash"), None);
-        assert_eq!(CacheableTool::from_name("Edit"), None);
-        assert_eq!(CacheableTool::from_name("Write"), None);
-    }
-
-    #[test]
-    fn test_first_invocation_not_cache_hit() {
-        let mut analytics = CacheAnalytics::new();
-        let input = json!({"file_path": "/home/test/file.txt"});
-
-        let is_repeat = analytics.track_invocation("Read", &input, 100);
-
-        assert!(!is_repeat);
-        assert_eq!(analytics.potential_cache_hits, 0);
-        assert_eq!(analytics.potential_savings_tokens, 0);
-    }
-
-    #[test]
-    fn test_second_invocation_is_cache_hit() {
-        let mut analytics = CacheAnalytics::new();
-        let input = json!({"file_path": "/home/test/file.txt"});
-
-        analytics.track_invocation("Read", &input, 100);
-        let is_repeat = analytics.track_invocation("Read", &input, 100);
-
-        assert!(is_repeat);
-        assert_eq!(analytics.potential_cache_hits, 1);
-        assert_eq!(analytics.potential_savings_tokens, 100);
-    }
-
-    #[test]
-    fn test_different_inputs_not_cache_hit() {
-        let mut analytics = CacheAnalytics::new();
-        let input1 = json!({"file_path": "/home/test/file1.txt"});
-        let input2 = json!({"file_path": "/home/test/file2.txt"});
-
-        analytics.track_invocation("Read", &input1, 100);
-        let is_repeat = analytics.track_invocation("Read", &input2, 100);
-
-        assert!(!is_repeat);
-        assert_eq!(analytics.potential_cache_hits, 0);
-    }
-
-    #[test]
-    fn test_non_cacheable_tool_ignored() {
-        let mut analytics = CacheAnalytics::new();
-        let input = json!({"command": "ls -la"});
-
-        let is_repeat = analytics.track_invocation("Bash", &input, 100);
-        analytics.track_invocation("Bash", &input, 100);
-
-        assert!(!is_repeat);
-        assert_eq!(analytics.potential_cache_hits, 0);
-        assert_eq!(analytics.unique_invocations(), 0);
-    }
-
-    #[test]
-    fn test_multiple_repeated_invocations() {
-        let mut analytics = CacheAnalytics::new();
-        let input = json!({"file_path": "/home/test/file.txt"});
-
-        analytics.track_invocation("Read", &input, 100);
-        analytics.track_invocation("Read", &input, 100);
-        analytics.track_invocation("Read", &input, 100);
-
-        assert_eq!(analytics.potential_cache_hits, 2);
-        assert_eq!(analytics.potential_savings_tokens, 200);
-    }
-
-    #[test]
-    fn test_unique_invocations_count() {
-        let mut analytics = CacheAnalytics::new();
-
-        analytics.track_invocation("Read", &json!({"file_path": "/file1.txt"}), 100);
-        analytics.track_invocation("Read", &json!({"file_path": "/file2.txt"}), 100);
-        analytics.track_invocation("Glob", &json!({"pattern": "*.rs"}), 50);
-
-        assert_eq!(analytics.unique_invocations(), 3);
-    }
-
-    #[test]
-    fn test_repeated_invocations_list() {
-        let mut analytics = CacheAnalytics::new();
-
-        // file1 read twice
-        analytics.track_invocation("Read", &json!({"file_path": "/file1.txt"}), 100);
-        analytics.track_invocation("Read", &json!({"file_path": "/file1.txt"}), 100);
-
-        // file2 read once
-        analytics.track_invocation("Read", &json!({"file_path": "/file2.txt"}), 100);
-
-        // glob run 3 times
-        analytics.track_invocation("Glob", &json!({"pattern": "*.rs"}), 50);
-        analytics.track_invocation("Glob", &json!({"pattern": "*.rs"}), 50);
-        analytics.track_invocation("Glob", &json!({"pattern": "*.rs"}), 50);
-
-        let repeated = analytics.repeated_invocations();
-        assert_eq!(repeated.len(), 2); // file1 and glob pattern
-    }
-
-    #[test]
-    fn test_clear_session() {
-        let mut analytics = CacheAnalytics::new();
-        let input = json!({"file_path": "/file.txt"});
-
-        analytics.track_invocation("Read", &input, 100);
-        analytics.track_invocation("Read", &input, 100);
-
-        assert_eq!(analytics.potential_cache_hits, 1);
-        assert_eq!(analytics.unique_invocations(), 1);
-
-        analytics.clear_session();
-
-        assert_eq!(analytics.potential_cache_hits, 1); // Preserved
-        assert_eq!(analytics.unique_invocations(), 0); // Cleared
-    }
-
-    #[test]
-    fn test_reset() {
-        let mut analytics = CacheAnalytics::new();
-        let input = json!({"file_path": "/file.txt"});
-
-        analytics.track_invocation("Read", &input, 100);
-        analytics.track_invocation("Read", &input, 100);
-
-        analytics.reset();
-
-        assert_eq!(analytics.potential_cache_hits, 0);
-        assert_eq!(analytics.potential_savings_tokens, 0);
-        assert_eq!(analytics.unique_invocations(), 0);
-    }
-
-    #[test]
-    fn test_serialization() {
-        let mut analytics = CacheAnalytics::new();
-        analytics.potential_cache_hits = 10;
-        analytics.potential_savings_tokens = 500;
-
-        let json = serde_json::to_string(&analytics).expect("Failed to serialize");
-        let deserialized: CacheAnalytics =
-            serde_json::from_str(&json).expect("Failed to deserialize");
-
-        assert_eq!(deserialized.potential_cache_hits, 10);
-        assert_eq!(deserialized.potential_savings_tokens, 500);
-        // recent_invocations is skipped in serialization
-        assert_eq!(deserialized.unique_invocations(), 0);
-    }
-}

src-tauri/src/tray.rs

@@ -4,6 +4,8 @@ use tauri::{
     AppHandle, Manager,
 };
 
+use crate::config::HikariConfig;
+
 pub fn setup_tray(app: &AppHandle) -> tauri::Result<()> {
     let show_item = MenuItem::with_id(app, "show", "Show Hikari", true, None::<&str>)?;
     let quit_item = MenuItem::with_id(app, "quit", "Quit", true, None::<&str>)?;
@@ -46,3 +48,21 @@ pub fn setup_tray(app: &AppHandle) -> tauri::Result<()> {
 
     Ok(())
 }
+
+pub fn should_minimize_to_tray(app: &AppHandle) -> bool {
+    let config_path = app
+        .path()
+        .app_config_dir()
+        .ok()
+        .map(|p| p.join("hikari-config.json"));
+
+    if let Some(path) = config_path {
+        if let Ok(content) = std::fs::read_to_string(&path) {
+            if let Ok(config) = serde_json::from_str::<HikariConfig>(&content) {
+                return config.minimize_to_tray;
+            }
+        }
+    }
+
+    false
+}

src-tauri/src/types.rs

@@ -4,10 +4,6 @@ use serde::{Deserialize, Serialize};
 pub struct UsageInfo {
     pub input_tokens: u64,
     pub output_tokens: u64,
-    #[serde(default)]
-    pub cache_creation_input_tokens: Option<u64>,
-    #[serde(default)]
-    pub cache_read_input_tokens: Option<u64>,
 }
 
 #[derive(Debug, Clone, Serialize, Deserialize, PartialEq, Default)]
@@ -63,26 +59,6 @@ pub struct PermissionDenial {
     pub tool_input: serde_json::Value,
 }
 
-/// Rate limit information from a `rate_limit_event` message.
-/// All fields are optional to ensure forward-compatibility as the Claude CLI evolves.
-#[derive(Debug, Clone, Serialize, Deserialize, Default)]
-pub struct RateLimitInfo {
-    #[serde(default)]
-    pub requests_limit: Option<u64>,
-    #[serde(default)]
-    pub requests_remaining: Option<u64>,
-    #[serde(default)]
-    pub requests_reset: Option<String>,
-    #[serde(default)]
-    pub tokens_limit: Option<u64>,
-    #[serde(default)]
-    pub tokens_remaining: Option<u64>,
-    #[serde(default)]
-    pub tokens_reset: Option<String>,
-    #[serde(default)]
-    pub retry_after_ms: Option<u64>,
-}
-
 #[derive(Debug, Clone, Serialize, Deserialize)]
 #[serde(tag = "type")]
 pub enum ClaudeMessage {
@@ -120,11 +96,6 @@ pub enum ClaudeMessage {
         #[serde(default)]
         usage: Option<UsageInfo>,
     },
-    #[serde(rename = "rate_limit_event")]
-    RateLimitEvent {
-        #[serde(default)]
-        rate_limit_info: RateLimitInfo,
-    },
 }
 
 #[derive(Debug, Clone, Serialize, Deserialize)]
@@ -205,14 +176,6 @@ pub struct StateChangeEvent {
     pub conversation_id: Option<String>,
 }
 
-/// Cost information for a message
-#[derive(Debug, Clone, Serialize, Deserialize)]
-pub struct MessageCost {
-    pub input_tokens: u64,
-    pub output_tokens: u64,
-    pub cost_usd: f64,
-}
-
 #[derive(Debug, Clone, Serialize, Deserialize)]
 pub struct OutputEvent {
     pub line_type: String,
@@ -220,23 +183,14 @@ pub struct OutputEvent {
     pub tool_name: Option<String>,
     #[serde(skip_serializing_if = "Option::is_none")]
     pub conversation_id: Option<String>,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub cost: Option<MessageCost>,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub parent_tool_use_id: Option<String>,
-}
-
-#[derive(Debug, Clone, Serialize, Deserialize)]
-pub struct PermissionPromptEventItem {
-    pub id: String,
-    pub tool_name: String,
-    pub tool_input: serde_json::Value,
-    pub description: String,
 }
 
 #[derive(Debug, Clone, Serialize, Deserialize)]
 pub struct PermissionPromptEvent {
-    pub permissions: Vec<PermissionPromptEventItem>,
+    pub id: String,
+    pub tool_name: String,
+    pub tool_input: serde_json::Value,
+    pub description: String,
     #[serde(skip_serializing_if = "Option::is_none")]
     pub conversation_id: Option<String>,
 }
@@ -280,50 +234,6 @@ pub struct UserQuestionEvent {
     pub conversation_id: Option<String>,
 }
 
-#[derive(Debug, Clone, Serialize, Deserialize)]
-pub struct AgentStartEvent {
-    pub tool_use_id: String,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub agent_id: Option<String>,
-    pub description: String,
-    pub subagent_type: String,
-    pub started_at: u64,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub conversation_id: Option<String>,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub parent_tool_use_id: Option<String>,
-}
-
-#[derive(Debug, Clone, Serialize, Deserialize)]
-pub struct AgentEndEvent {
-    pub tool_use_id: String,
-    pub ended_at: u64,
-    pub is_error: bool,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub conversation_id: Option<String>,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub duration_ms: Option<u64>,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub num_turns: Option<u32>,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub last_assistant_message: Option<String>,
-}
-
-#[derive(Debug, Clone, Serialize, Deserialize)]
-pub struct TodoItem {
-    pub content: String,
-    pub status: String, // "pending", "in_progress", or "completed"
-    #[serde(rename = "activeForm")]
-    pub active_form: String,
-}
-
-#[derive(Debug, Clone, Serialize, Deserialize)]
-pub struct TodoUpdateEvent {
-    pub todos: Vec<TodoItem>,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub conversation_id: Option<String>,
-}
-
 #[cfg(test)]
 mod tests {
     use super::*;
@@ -444,106 +354,10 @@ mod tests {
             content: "Test output".to_string(),
             tool_name: None,
             conversation_id: None,
-            cost: None,
-            parent_tool_use_id: None,
         };
 
         let serialized = serde_json::to_string(&event).unwrap();
         assert!(serialized.contains("\"line_type\":\"assistant\""));
         assert!(serialized.contains("\"content\":\"Test output\""));
     }
-
-    #[test]
-    fn test_output_event_with_cost() {
-        let event = OutputEvent {
-            line_type: "assistant".to_string(),
-            content: "Test output".to_string(),
-            tool_name: None,
-            conversation_id: Some("conv-123".to_string()),
-            cost: Some(MessageCost {
-                input_tokens: 100,
-                output_tokens: 50,
-                cost_usd: 0.005,
-            }),
-            parent_tool_use_id: None,
-        };
-
-        let serialized = serde_json::to_string(&event).unwrap();
-        assert!(serialized.contains("\"cost\":"));
-        assert!(serialized.contains("\"input_tokens\":100"));
-        assert!(serialized.contains("\"output_tokens\":50"));
-    }
-
-    #[test]
-    fn test_rate_limit_info_default() {
-        let info = RateLimitInfo::default();
-        assert!(info.requests_limit.is_none());
-        assert!(info.requests_remaining.is_none());
-        assert!(info.requests_reset.is_none());
-        assert!(info.tokens_limit.is_none());
-        assert!(info.tokens_remaining.is_none());
-        assert!(info.tokens_reset.is_none());
-        assert!(info.retry_after_ms.is_none());
-    }
-
-    #[test]
-    fn test_rate_limit_event_deserialization_empty_info() {
-        let json = r#"{"type":"rate_limit_event","rate_limit_info":{}}"#;
-        let msg: ClaudeMessage = serde_json::from_str(json).unwrap();
-        assert!(matches!(msg, ClaudeMessage::RateLimitEvent { .. }));
-    }
-
-    #[test]
-    fn test_rate_limit_event_deserialization_no_info() {
-        // rate_limit_info field is optional via #[serde(default)]
-        let json = r#"{"type":"rate_limit_event"}"#;
-        let msg: ClaudeMessage = serde_json::from_str(json).unwrap();
-        assert!(matches!(msg, ClaudeMessage::RateLimitEvent { .. }));
-    }
-
-    #[test]
-    fn test_rate_limit_event_deserialization_with_data() {
-        let json = r#"{
-            "type": "rate_limit_event",
-            "rate_limit_info": {
-                "requests_limit": 1000,
-                "requests_remaining": 0,
-                "requests_reset": "2024-01-01T00:01:00Z",
-                "tokens_limit": 50000,
-                "tokens_remaining": 0,
-                "tokens_reset": "2024-01-01T00:01:00Z",
-                "retry_after_ms": 60000
-            }
-        }"#;
-        let msg: ClaudeMessage = serde_json::from_str(json).unwrap();
-        if let ClaudeMessage::RateLimitEvent { rate_limit_info } = msg {
-            assert_eq!(rate_limit_info.requests_limit, Some(1000));
-            assert_eq!(rate_limit_info.requests_remaining, Some(0));
-            assert_eq!(
-                rate_limit_info.requests_reset,
-                Some("2024-01-01T00:01:00Z".to_string())
-            );
-            assert_eq!(rate_limit_info.retry_after_ms, Some(60000));
-        } else {
-            panic!("Expected RateLimitEvent variant");
-        }
-    }
-
-    #[test]
-    fn test_rate_limit_event_ignores_unknown_fields() {
-        // Ensures forward-compat: unknown fields in rate_limit_info are silently ignored
-        let json = r#"{
-            "type": "rate_limit_event",
-            "rate_limit_info": {
-                "requests_remaining": 0,
-                "some_future_field": "some_value"
-            }
-        }"#;
-        let msg: ClaudeMessage = serde_json::from_str(json).unwrap();
-        if let ClaudeMessage::RateLimitEvent { rate_limit_info } = msg {
-            assert_eq!(rate_limit_info.requests_remaining, Some(0));
-        } else {
-            panic!("Expected RateLimitEvent variant");
-        }
-    }
 }

src-tauri/src/vbs_notification.rs

@@ -3,8 +3,6 @@ use std::process::Command;
 use tauri::command;
 use tempfile::NamedTempFile;
 
-use crate::process_ext::HideWindow;
-
 #[command]
 pub async fn send_vbs_notification(title: String, body: String) -> Result<(), String> {
     // Create a VBScript that shows a Windows notification
@@ -42,7 +40,7 @@ objShell.Popup "{}" & vbCrLf & vbCrLf & "{}", 5, "{}", 64
     } else if temp_path.starts_with("/tmp/") {
         // WSL temp files might be in a different location
         // Try to use wslpath to convert
-        let output = Command::new("wslpath").hide_window().arg("-w").arg(&temp_path).output();
+        let output = Command::new("wslpath").arg("-w").arg(&temp_path).output();
 
         if let Ok(result) = output {
             if result.status.success() {
@@ -59,7 +57,6 @@ objShell.Popup "{}" & vbCrLf & vbCrLf & "{}", 5, "{}", 64
 
     // Execute the VBScript using wscript.exe
     let output = Command::new("/mnt/c/Windows/System32/wscript.exe")
-        .hide_window()
         .arg("//NoLogo")
         .arg(&windows_path)
         .output()

src-tauri/src/wsl_notifications.rs

@@ -1,8 +1,6 @@
 use std::process::Command;
 use tauri::command;
 
-use crate::process_ext::HideWindow;
-
 #[command]
 pub async fn send_wsl_notification(title: String, body: String) -> Result<(), String> {
     // Method 1: Try Windows 10/11 toast notification using PowerShell
@@ -38,7 +36,6 @@ $notifier.Show($toast)
 
     // Try PowerShell.exe through WSL
     let output = Command::new("/mnt/c/Windows/System32/WindowsPowerShell/v1.0/powershell.exe")
-        .hide_window()
         .arg("-NoProfile")
         .arg("-ExecutionPolicy")
         .arg("Bypass")
@@ -51,15 +48,15 @@ $notifier.Show($toast)
     match output {
         Ok(result) => {
             if result.status.success() {
-                tracing::info!("WSL notification sent successfully");
+                println!("WSL notification sent successfully");
                 return Ok(());
             } else {
                 let stderr = String::from_utf8_lossy(&result.stderr);
-                tracing::error!("PowerShell toast failed: {}", stderr);
+                println!("PowerShell toast failed: {}", stderr);
             }
         }
         Err(e) => {
-            tracing::error!("Failed to run PowerShell: {}", e);
+            println!("Failed to run PowerShell: {}", e);
         }
     }
 
@@ -68,7 +65,6 @@ $notifier.Show($toast)
 
     // Method 3: Try wsl-notify-send if available
     let notify_result = Command::new("wsl-notify-send")
-        .hide_window()
         .arg("--appId")
         .arg("HikariDesktop")
         .arg("--category")
@@ -78,7 +74,7 @@ $notifier.Show($toast)
 
     if let Ok(result) = notify_result {
         if result.status.success() {
-            tracing::info!("Notification sent via wsl-notify-send");
+            println!("Notification sent via wsl-notify-send");
             return Ok(());
         }
     }

src-tauri/tauri.conf.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://schema.tauri.app/config/2",
   "productName": "hikari-desktop",
-  "version": "1.10.0",
+  "version": "0.3.0",
   "identifier": "com.naomi.hikari-desktop",
   "build": {
     "beforeDevCommand": "pnpm dev",

src/app.css

@@ -154,7 +154,11 @@ body {
   padding: 0;
   height: 100%;
   overflow: hidden;
-  font-family: var(--ui-font-family, "Segoe UI", system-ui, -apple-system, sans-serif);
+  font-family:
+    "Segoe UI",
+    system-ui,
+    -apple-system,
+    sans-serif;
   background: var(--bg-primary);
   color: var(--text-primary);
 }

src/lib/commands/slashCommands.test.ts

@@ -1,866 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
-import { get } from "svelte/store";
-import { invoke } from "@tauri-apps/api/core";
-import { claudeStore } from "$lib/stores/claude";
-import { searchState } from "$lib/stores/search";
-import { characterState } from "$lib/stores/character";
-import {
-  slashCommands,
-  parseSlashCommand,
-  getMatchingCommands,
-  isSlashCommand,
-  type SlashCommand,
-} from "./slashCommands";
-
-// Mock all external dependencies
-vi.mock("svelte/store", async (importOriginal) => {
-  const actual = await importOriginal<typeof import("svelte/store")>();
-  return {
-    ...actual,
-    get: vi.fn(),
-  };
-});
-
-vi.mock("@tauri-apps/api/core", () => ({
-  invoke: vi.fn(),
-}));
-
-vi.mock("$lib/stores/claude", () => ({
-  claudeStore: {
-    addLine: vi.fn(),
-    clearTerminal: vi.fn(),
-    activeConversationId: { subscribe: vi.fn() },
-    currentWorkingDirectory: { subscribe: vi.fn() },
-    setWorkingDirectory: vi.fn(),
-    getConversationHistory: vi.fn(),
-  },
-}));
-
-vi.mock("$lib/stores/character", () => ({
-  characterState: {
-    setState: vi.fn(),
-    setTemporaryState: vi.fn(),
-  },
-}));
-
-vi.mock("$lib/tauri", () => ({
-  setSkipNextGreeting: vi.fn(),
-  updateDiscordRpc: vi.fn().mockResolvedValue(undefined),
-}));
-
-vi.mock("$lib/stores/conversations", () => ({
-  conversationsStore: {
-    activeConversation: { subscribe: vi.fn() },
-  },
-}));
-
-vi.mock("$lib/stores/config", () => ({
-  configStore: {
-    getConfig: vi.fn().mockReturnValue({
-      auto_granted_tools: [],
-      model: "claude-sonnet",
-      api_key: null,
-      custom_instructions: null,
-      mcp_servers_json: null,
-      use_worktree: false,
-      disable_1m_context: false,
-      max_output_tokens: null,
-    }),
-  },
-}));
-
-vi.mock("$lib/stores/search", () => ({
-  searchState: {
-    setQuery: vi.fn(),
-    clear: vi.fn(),
-  },
-}));
-
-describe("slashCommands", () => {
-  describe("slashCommands array", () => {
-    it("contains expected commands", () => {
-      const commandNames = slashCommands.map((cmd) => cmd.name);
-      expect(commandNames).toContain("cd");
-      expect(commandNames).toContain("clear");
-      expect(commandNames).toContain("new");
-      expect(commandNames).toContain("help");
-      expect(commandNames).toContain("search");
-      expect(commandNames).toContain("summarise");
-      expect(commandNames).toContain("skill");
-    });
-
-    it("has 7 commands total", () => {
-      expect(slashCommands.length).toBe(7);
-    });
-
-    it("each command has required properties", () => {
-      slashCommands.forEach((cmd) => {
-        expect(cmd.name).toBeDefined();
-        expect(typeof cmd.name).toBe("string");
-        expect(cmd.name.length).toBeGreaterThan(0);
-
-        expect(cmd.description).toBeDefined();
-        expect(typeof cmd.description).toBe("string");
-        expect(cmd.description.length).toBeGreaterThan(0);
-
-        expect(cmd.usage).toBeDefined();
-        expect(typeof cmd.usage).toBe("string");
-        expect(cmd.usage.startsWith("/")).toBe(true);
-
-        expect(cmd.execute).toBeDefined();
-        expect(typeof cmd.execute).toBe("function");
-      });
-    });
-
-    it("cd command has correct metadata", () => {
-      const cdCmd = slashCommands.find((cmd) => cmd.name === "cd");
-      expect(cdCmd).toBeDefined();
-      expect(cdCmd!.description).toBe("Change the working directory");
-      expect(cdCmd!.usage).toBe("/cd <path>");
-    });
-
-    it("clear command has correct metadata", () => {
-      const clearCmd = slashCommands.find((cmd) => cmd.name === "clear");
-      expect(clearCmd).toBeDefined();
-      expect(clearCmd!.description).toBe("Clear the terminal display (keeps conversation context)");
-      expect(clearCmd!.usage).toBe("/clear");
-    });
-
-    it("new command has correct metadata", () => {
-      const newCmd = slashCommands.find((cmd) => cmd.name === "new");
-      expect(newCmd).toBeDefined();
-      expect(newCmd!.description).toBe("Start a fresh conversation (resets context)");
-      expect(newCmd!.usage).toBe("/new");
-    });
-
-    it("help command has correct metadata", () => {
-      const helpCmd = slashCommands.find((cmd) => cmd.name === "help");
-      expect(helpCmd).toBeDefined();
-      expect(helpCmd!.description).toBe("Show available slash commands");
-      expect(helpCmd!.usage).toBe("/help");
-    });
-
-    it("search command has correct metadata", () => {
-      const searchCmd = slashCommands.find((cmd) => cmd.name === "search");
-      expect(searchCmd).toBeDefined();
-      expect(searchCmd!.description).toBe("Search within the conversation (use /search to clear)");
-      expect(searchCmd!.usage).toBe("/search [query]");
-    });
-
-    it("summarise command has correct metadata", () => {
-      const summariseCmd = slashCommands.find((cmd) => cmd.name === "summarise");
-      expect(summariseCmd).toBeDefined();
-      expect(summariseCmd!.description).toBe("Get a summary of the entire conversation");
-      expect(summariseCmd!.usage).toBe("/summarise");
-    });
-
-    it("skill command has correct metadata", () => {
-      const skillCmd = slashCommands.find((cmd) => cmd.name === "skill");
-      expect(skillCmd).toBeDefined();
-      expect(skillCmd!.description).toBe("Invoke a Claude Code skill from ~/.claude/skills/");
-      expect(skillCmd!.usage).toBe("/skill [name] [data]");
-    });
-  });
-
-  describe("parseSlashCommand", () => {
-    it("returns null for non-slash input", () => {
-      const result = parseSlashCommand("hello world");
-      expect(result.command).toBeNull();
-      expect(result.args).toBe("");
-    });
-
-    it("returns null for empty string", () => {
-      const result = parseSlashCommand("");
-      expect(result.command).toBeNull();
-      expect(result.args).toBe("");
-    });
-
-    it("returns null for whitespace only", () => {
-      const result = parseSlashCommand("   ");
-      expect(result.command).toBeNull();
-      expect(result.args).toBe("");
-    });
-
-    it("parses /cd command without args", () => {
-      const result = parseSlashCommand("/cd");
-      expect(result.command).not.toBeNull();
-      expect(result.command!.name).toBe("cd");
-      expect(result.args).toBe("");
-    });
-
-    it("parses /cd command with path argument", () => {
-      const result = parseSlashCommand("/cd /home/naomi/code");
-      expect(result.command).not.toBeNull();
-      expect(result.command!.name).toBe("cd");
-      expect(result.args).toBe("/home/naomi/code");
-    });
-
-    it("parses /clear command", () => {
-      const result = parseSlashCommand("/clear");
-      expect(result.command).not.toBeNull();
-      expect(result.command!.name).toBe("clear");
-      expect(result.args).toBe("");
-    });
-
-    it("parses /new command", () => {
-      const result = parseSlashCommand("/new");
-      expect(result.command).not.toBeNull();
-      expect(result.command!.name).toBe("new");
-      expect(result.args).toBe("");
-    });
-
-    it("parses /help command", () => {
-      const result = parseSlashCommand("/help");
-      expect(result.command).not.toBeNull();
-      expect(result.command!.name).toBe("help");
-      expect(result.args).toBe("");
-    });
-
-    it("parses /search command with query", () => {
-      const result = parseSlashCommand("/search hello world");
-      expect(result.command).not.toBeNull();
-      expect(result.command!.name).toBe("search");
-      expect(result.args).toBe("hello world");
-    });
-
-    it("parses /search command without query", () => {
-      const result = parseSlashCommand("/search");
-      expect(result.command).not.toBeNull();
-      expect(result.command!.name).toBe("search");
-      expect(result.args).toBe("");
-    });
-
-    it("parses /summarise command", () => {
-      const result = parseSlashCommand("/summarise");
-      expect(result.command).not.toBeNull();
-      expect(result.command!.name).toBe("summarise");
-      expect(result.args).toBe("");
-    });
-
-    it("parses /skill command with name and data", () => {
-      const result = parseSlashCommand("/skill onboard-mentee john@example.com");
-      expect(result.command).not.toBeNull();
-      expect(result.command!.name).toBe("skill");
-      expect(result.args).toBe("onboard-mentee john@example.com");
-    });
-
-    it("parses /skill command with name only", () => {
-      const result = parseSlashCommand("/skill onboard-mentee");
-      expect(result.command).not.toBeNull();
-      expect(result.command!.name).toBe("skill");
-      expect(result.args).toBe("onboard-mentee");
-    });
-
-    it("parses /skill command without arguments", () => {
-      const result = parseSlashCommand("/skill");
-      expect(result.command).not.toBeNull();
-      expect(result.command!.name).toBe("skill");
-      expect(result.args).toBe("");
-    });
-
-    it("returns null for unknown command", () => {
-      const result = parseSlashCommand("/unknown");
-      expect(result.command).toBeNull();
-      expect(result.args).toBe("");
-    });
-
-    it("is case insensitive for command names", () => {
-      const result1 = parseSlashCommand("/CD /path");
-      expect(result1.command).not.toBeNull();
-      expect(result1.command!.name).toBe("cd");
-
-      const result2 = parseSlashCommand("/CLEAR");
-      expect(result2.command).not.toBeNull();
-      expect(result2.command!.name).toBe("clear");
-
-      const result3 = parseSlashCommand("/Help");
-      expect(result3.command).not.toBeNull();
-      expect(result3.command!.name).toBe("help");
-    });
-
-    it("handles leading whitespace", () => {
-      const result = parseSlashCommand("  /cd /path");
-      expect(result.command).not.toBeNull();
-      expect(result.command!.name).toBe("cd");
-      expect(result.args).toBe("/path");
-    });
-
-    it("handles trailing whitespace", () => {
-      const result = parseSlashCommand("/cd /path  ");
-      expect(result.command).not.toBeNull();
-      expect(result.command!.name).toBe("cd");
-      expect(result.args).toBe("/path");
-    });
-
-    it("handles multiple spaces between args", () => {
-      const result = parseSlashCommand("/search   hello   world");
-      expect(result.command).not.toBeNull();
-      expect(result.command!.name).toBe("search");
-      expect(result.args).toBe("hello world");
-    });
-  });
-
-  describe("getMatchingCommands", () => {
-    it("returns empty array for non-slash input", () => {
-      const result = getMatchingCommands("hello");
-      expect(result).toEqual([]);
-    });
-
-    it("returns empty array for empty string", () => {
-      const result = getMatchingCommands("");
-      expect(result).toEqual([]);
-    });
-
-    it("returns all commands for just slash", () => {
-      const result = getMatchingCommands("/");
-      expect(result.length).toBe(slashCommands.length);
-    });
-
-    it("returns matching commands for partial input", () => {
-      const result = getMatchingCommands("/c");
-      const names = result.map((cmd) => cmd.name);
-      expect(names).toContain("cd");
-      expect(names).toContain("clear");
-      expect(names).not.toContain("help");
-    });
-
-    it("returns single command for exact match", () => {
-      const result = getMatchingCommands("/cd");
-      expect(result.length).toBe(1);
-      expect(result[0].name).toBe("cd");
-    });
-
-    it("returns single command for partial unique match", () => {
-      const result = getMatchingCommands("/cl");
-      expect(result.length).toBe(1);
-      expect(result[0].name).toBe("clear");
-    });
-
-    it("returns matching commands for /s prefix", () => {
-      const result = getMatchingCommands("/s");
-      const names = result.map((cmd) => cmd.name);
-      expect(names).toContain("search");
-      expect(names).toContain("summarise");
-      expect(names).toContain("skill");
-    });
-
-    it("is case insensitive", () => {
-      const result1 = getMatchingCommands("/C");
-      const result2 = getMatchingCommands("/c");
-      expect(result1.length).toBe(result2.length);
-    });
-
-    it("returns empty array for no matches", () => {
-      const result = getMatchingCommands("/xyz");
-      expect(result).toEqual([]);
-    });
-
-    it("handles whitespace correctly", () => {
-      const result = getMatchingCommands("  /c");
-      const names = result.map((cmd) => cmd.name);
-      expect(names).toContain("cd");
-      expect(names).toContain("clear");
-    });
-
-    it("returns command for full command name", () => {
-      const result = getMatchingCommands("/help");
-      expect(result.length).toBe(1);
-      expect(result[0].name).toBe("help");
-    });
-
-    it("returns command for /new", () => {
-      const result = getMatchingCommands("/n");
-      expect(result.length).toBe(1);
-      expect(result[0].name).toBe("new");
-    });
-  });
-
-  describe("isSlashCommand", () => {
-    it("returns true for input starting with slash", () => {
-      expect(isSlashCommand("/cd")).toBe(true);
-      expect(isSlashCommand("/")).toBe(true);
-      expect(isSlashCommand("/help")).toBe(true);
-      expect(isSlashCommand("/unknown")).toBe(true);
-    });
-
-    it("returns false for non-slash input", () => {
-      expect(isSlashCommand("hello")).toBe(false);
-      expect(isSlashCommand("")).toBe(false);
-      expect(isSlashCommand("cd")).toBe(false);
-    });
-
-    it("handles whitespace correctly", () => {
-      expect(isSlashCommand("  /cd")).toBe(true);
-      expect(isSlashCommand("  hello")).toBe(false);
-    });
-
-    it("returns false for slash in middle of string", () => {
-      expect(isSlashCommand("hello/world")).toBe(false);
-    });
-  });
-
-  describe("SlashCommand interface", () => {
-    it("can create a valid slash command object", () => {
-      const testCommand: SlashCommand = {
-        name: "test",
-        description: "A test command",
-        usage: "/test [arg]",
-        execute: vi.fn(),
-      };
-
-      expect(testCommand.name).toBe("test");
-      expect(testCommand.description).toBe("A test command");
-      expect(testCommand.usage).toBe("/test [arg]");
-      expect(typeof testCommand.execute).toBe("function");
-    });
-
-    it("execute can be async function", () => {
-      const asyncCommand: SlashCommand = {
-        name: "async",
-        description: "An async command",
-        usage: "/async",
-        execute: async () => {
-          await Promise.resolve();
-        },
-      };
-
-      expect(asyncCommand.execute("")).toBeInstanceOf(Promise);
-    });
-
-    it("execute can be sync function", () => {
-      const syncCommand: SlashCommand = {
-        name: "sync",
-        description: "A sync command",
-        usage: "/sync",
-        execute: () => {
-          // Synchronous execution
-        },
-      };
-
-      const result = syncCommand.execute("");
-      // Sync function returns undefined, not a Promise
-      expect(result).toBeUndefined();
-    });
-  });
-
-  describe("command execute functions", () => {
-    let getMock: ReturnType<typeof vi.fn>;
-    let invokeMock: ReturnType<typeof vi.fn>;
-
-    beforeEach(() => {
-      vi.clearAllMocks();
-      getMock = vi.mocked(get);
-      invokeMock = vi.mocked(invoke);
-    });
-
-    describe("/clear execute", () => {
-      it("clears terminal and shows confirmation message", () => {
-        const clearCmd = slashCommands.find((cmd) => cmd.name === "clear")!;
-        clearCmd.execute("");
-        expect(claudeStore.clearTerminal).toHaveBeenCalledWith();
-        expect(claudeStore.addLine).toHaveBeenCalledWith("system", "Terminal cleared");
-      });
-    });
-
-    describe("/help execute", () => {
-      it("shows available commands header", () => {
-        const helpCmd = slashCommands.find((cmd) => cmd.name === "help")!;
-        helpCmd.execute("");
-        expect(claudeStore.addLine).toHaveBeenCalledWith(
-          "system",
-          expect.stringContaining("Available commands:")
-        );
-      });
-
-      it("includes all command usages in help text", () => {
-        const helpCmd = slashCommands.find((cmd) => cmd.name === "help")!;
-        helpCmd.execute("");
-        const callArgs = vi.mocked(claudeStore.addLine).mock.calls[0];
-        const helpText = callArgs[1] as string;
-        expect(helpText).toContain("/cd");
-        expect(helpText).toContain("/clear");
-        expect(helpText).toContain("/help");
-        expect(helpText).toContain("/search");
-        expect(helpText).toContain("/new");
-        expect(helpText).toContain("/summarise");
-        expect(helpText).toContain("/skill");
-      });
-
-      it("includes command descriptions in help text", () => {
-        const helpCmd = slashCommands.find((cmd) => cmd.name === "help")!;
-        helpCmd.execute("");
-        const callArgs = vi.mocked(claudeStore.addLine).mock.calls[0];
-        const helpText = callArgs[1] as string;
-        expect(helpText).toContain("Change the working directory");
-        expect(helpText).toContain("Show available slash commands");
-      });
-    });
-
-    describe("/search execute", () => {
-      it("clears search when called with empty args", () => {
-        const searchCmd = slashCommands.find((cmd) => cmd.name === "search")!;
-        searchCmd.execute("");
-        expect(searchState.clear).toHaveBeenCalledWith();
-        expect(claudeStore.addLine).toHaveBeenCalledWith("system", "Search cleared");
-      });
-
-      it("clears search when called with whitespace-only args", () => {
-        const searchCmd = slashCommands.find((cmd) => cmd.name === "search")!;
-        searchCmd.execute("   ");
-        expect(searchState.clear).toHaveBeenCalledWith();
-        expect(claudeStore.addLine).toHaveBeenCalledWith("system", "Search cleared");
-      });
-
-      it("sets query when called with a search term", () => {
-        const searchCmd = slashCommands.find((cmd) => cmd.name === "search")!;
-        searchCmd.execute("hello world");
-        expect(searchState.setQuery).toHaveBeenCalledWith("hello world");
-        expect(claudeStore.addLine).toHaveBeenCalledWith("system", 'Searching for: "hello world"');
-      });
-
-      it("trims whitespace from query before setting", () => {
-        const searchCmd = slashCommands.find((cmd) => cmd.name === "search")!;
-        searchCmd.execute("  hello  ");
-        expect(searchState.setQuery).toHaveBeenCalledWith("hello");
-        expect(claudeStore.addLine).toHaveBeenCalledWith("system", 'Searching for: "hello"');
-      });
-    });
-
-    describe("/cd execute", () => {
-      it("shows error when no active conversation", async () => {
-        getMock.mockReturnValue(null);
-        const cdCmd = slashCommands.find((cmd) => cmd.name === "cd")!;
-        await cdCmd.execute("/some/path");
-        expect(claudeStore.addLine).toHaveBeenCalledWith("error", "No active conversation");
-      });
-
-      it("shows current directory when called with empty args", async () => {
-        getMock.mockReturnValueOnce("conv-123").mockReturnValueOnce("/home/naomi/code");
-        const cdCmd = slashCommands.find((cmd) => cmd.name === "cd")!;
-        await cdCmd.execute("");
-        expect(claudeStore.addLine).toHaveBeenCalledWith(
-          "system",
-          "Current directory: /home/naomi/code"
-        );
-      });
-
-      it("shows current directory when called with whitespace-only args", async () => {
-        getMock.mockReturnValueOnce("conv-123").mockReturnValueOnce("/home/naomi/code");
-        const cdCmd = slashCommands.find((cmd) => cmd.name === "cd")!;
-        await cdCmd.execute("   ");
-        expect(claudeStore.addLine).toHaveBeenCalledWith(
-          "system",
-          "Current directory: /home/naomi/code"
-        );
-      });
-
-      it("validates path and changes directory on success", async () => {
-        getMock.mockReturnValueOnce("conv-123").mockReturnValueOnce(null);
-        invokeMock.mockResolvedValue("/validated/path");
-        const cdCmd = slashCommands.find((cmd) => cmd.name === "cd")!;
-        await cdCmd.execute("/new/path");
-        expect(invokeMock).toHaveBeenCalledWith(
-          "validate_directory",
-          expect.objectContaining({ path: "/new/path" })
-        );
-      });
-
-      it("shows error when directory change fails", async () => {
-        getMock.mockReturnValueOnce("conv-123");
-        invokeMock.mockRejectedValueOnce(new Error("invalid path"));
-        const cdCmd = slashCommands.find((cmd) => cmd.name === "cd")!;
-        await cdCmd.execute("/bad/path");
-        expect(claudeStore.addLine).toHaveBeenCalledWith(
-          "error",
-          expect.stringContaining("Failed to change directory:")
-        );
-        expect(characterState.setTemporaryState).toHaveBeenCalledWith("error", 3000);
-      });
-    });
-
-    describe("/new execute", () => {
-      it("shows error when no active conversation", async () => {
-        getMock.mockReturnValue(null);
-        const newCmd = slashCommands.find((cmd) => cmd.name === "new")!;
-        await newCmd.execute("");
-        expect(claudeStore.addLine).toHaveBeenCalledWith("error", "No active conversation");
-      });
-
-      it("shows error when starting new conversation fails", async () => {
-        getMock.mockReturnValueOnce("conv-123");
-        invokeMock.mockRejectedValueOnce(new Error("invoke failed"));
-        const newCmd = slashCommands.find((cmd) => cmd.name === "new")!;
-        await newCmd.execute("");
-        expect(claudeStore.addLine).toHaveBeenCalledWith(
-          "error",
-          expect.stringContaining("Failed to start new conversation:")
-        );
-        expect(characterState.setTemporaryState).toHaveBeenCalledWith("error", 3000);
-      });
-    });
-
-    describe("/summarise execute", () => {
-      it("shows error when no active conversation", async () => {
-        getMock.mockReturnValue(null);
-        const summariseCmd = slashCommands.find((cmd) => cmd.name === "summarise")!;
-        await summariseCmd.execute("");
-        expect(claudeStore.addLine).toHaveBeenCalledWith("error", "No active conversation");
-      });
-
-      it("sends a summary prompt when there is an active conversation", async () => {
-        getMock.mockReturnValue("conv-123");
-        invokeMock.mockResolvedValue(undefined);
-        const summariseCmd = slashCommands.find((cmd) => cmd.name === "summarise")!;
-        await summariseCmd.execute("");
-        expect(claudeStore.addLine).toHaveBeenCalledWith(
-          "system",
-          "Requesting conversation summary..."
-        );
-        expect(invokeMock).toHaveBeenCalledWith(
-          "send_prompt",
-          expect.objectContaining({ conversationId: "conv-123" })
-        );
-      });
-
-      it("shows error when send_prompt invoke fails", async () => {
-        getMock.mockReturnValue("conv-123");
-        invokeMock.mockRejectedValue(new Error("network error"));
-        const summariseCmd = slashCommands.find((cmd) => cmd.name === "summarise")!;
-        await summariseCmd.execute("");
-        expect(claudeStore.addLine).toHaveBeenCalledWith(
-          "error",
-          expect.stringContaining("Failed to request summary:")
-        );
-      });
-    });
-
-    describe("/skill execute", () => {
-      it("shows error when no active conversation", async () => {
-        getMock.mockReturnValue(null);
-        const skillCmd = slashCommands.find((cmd) => cmd.name === "skill")!;
-        await skillCmd.execute("onboard-mentee");
-        expect(claudeStore.addLine).toHaveBeenCalledWith("error", "No active conversation");
-      });
-
-      it("lists available skills when called with no name", async () => {
-        getMock.mockReturnValue("conv-123");
-        invokeMock.mockResolvedValue(["onboard-mentee", "other-skill"]);
-        const skillCmd = slashCommands.find((cmd) => cmd.name === "skill")!;
-        await skillCmd.execute("");
-        expect(invokeMock).toHaveBeenCalledWith("list_skills");
-        expect(claudeStore.addLine).toHaveBeenCalledWith(
-          "system",
-          expect.stringContaining("onboard-mentee")
-        );
-      });
-
-      it("shows empty message when no skills are found", async () => {
-        getMock.mockReturnValue("conv-123");
-        invokeMock.mockResolvedValue([]);
-        const skillCmd = slashCommands.find((cmd) => cmd.name === "skill")!;
-        await skillCmd.execute("");
-        expect(claudeStore.addLine).toHaveBeenCalledWith(
-          "system",
-          expect.stringContaining("No skills found")
-        );
-      });
-
-      it("invokes skill when called with a name and no data", async () => {
-        getMock.mockReturnValue("conv-123");
-        invokeMock.mockResolvedValue(undefined);
-        const skillCmd = slashCommands.find((cmd) => cmd.name === "skill")!;
-        await skillCmd.execute("onboard-mentee");
-        expect(claudeStore.addLine).toHaveBeenCalledWith(
-          "system",
-          "Invoking skill: onboard-mentee"
-        );
-        expect(invokeMock).toHaveBeenCalledWith(
-          "send_prompt",
-          expect.objectContaining({ conversationId: "conv-123" })
-        );
-      });
-
-      it("invokes skill with additional data in the prompt", async () => {
-        getMock.mockReturnValue("conv-123");
-        invokeMock.mockResolvedValue(undefined);
-        const skillCmd = slashCommands.find((cmd) => cmd.name === "skill")!;
-        await skillCmd.execute("onboard-mentee some extra data");
-        expect(invokeMock).toHaveBeenCalledWith("send_prompt", {
-          conversationId: "conv-123",
-          message: expect.stringContaining("some extra data"),
-        });
-      });
-
-      it("shows error when listing skills fails", async () => {
-        getMock.mockReturnValue("conv-123");
-        invokeMock.mockRejectedValue(new Error("list error"));
-        const skillCmd = slashCommands.find((cmd) => cmd.name === "skill")!;
-        await skillCmd.execute("");
-        expect(claudeStore.addLine).toHaveBeenCalledWith(
-          "error",
-          expect.stringContaining("Failed to list skills:")
-        );
-      });
-
-      it("shows error and resets character state when invoking skill fails", async () => {
-        getMock.mockReturnValue("conv-123");
-        invokeMock.mockRejectedValue(new Error("invoke error"));
-        const skillCmd = slashCommands.find((cmd) => cmd.name === "skill")!;
-        await skillCmd.execute("onboard-mentee");
-        expect(claudeStore.addLine).toHaveBeenCalledWith(
-          "error",
-          expect.stringContaining("Failed to invoke skill:")
-        );
-        expect(characterState.setTemporaryState).toHaveBeenCalledWith("error", 3000);
-      });
-    });
-
-    describe("/cd success path", () => {
-      beforeEach(() => {
-        vi.useFakeTimers();
-      });
-
-      afterEach(() => {
-        vi.useRealTimers();
-      });
-
-      it("changes directory and shows success message", async () => {
-        getMock
-          .mockReturnValueOnce("conv-123") // get(claudeStore.activeConversationId)
-          .mockReturnValueOnce("/current") // get(claudeStore.currentWorkingDirectory)
-          .mockReturnValueOnce(null); // get(conversationsStore.activeConversation)
-        vi.mocked(claudeStore.getConversationHistory).mockReturnValue("");
-        invokeMock
-          .mockResolvedValueOnce("/new/path") // validate_directory
-          .mockResolvedValueOnce(undefined) // stop_claude
-          .mockResolvedValueOnce(undefined); // start_claude
-
-        const cdCmd = slashCommands.find((cmd) => cmd.name === "cd")!;
-        const promise = cdCmd.execute("/new/path");
-        await vi.runAllTimersAsync();
-        await promise;
-
-        expect(claudeStore.addLine).toHaveBeenCalledWith(
-          "system",
-          "Changed directory to: /new/path"
-        );
-        expect(characterState.setState).toHaveBeenCalledWith("idle");
-      });
-
-      it("sends context restoration message when conversation history exists", async () => {
-        getMock
-          .mockReturnValueOnce("conv-123") // get(claudeStore.activeConversationId)
-          .mockReturnValueOnce("/current") // get(claudeStore.currentWorkingDirectory)
-          .mockReturnValueOnce(null); // get(conversationsStore.activeConversation)
-        vi.mocked(claudeStore.getConversationHistory).mockReturnValue(
-          "previous conversation history"
-        );
-        invokeMock
-          .mockResolvedValueOnce("/new/path") // validate_directory
-          .mockResolvedValueOnce(undefined) // stop_claude
-          .mockResolvedValueOnce(undefined) // start_claude
-          .mockResolvedValueOnce(undefined); // send_prompt
-
-        const cdCmd = slashCommands.find((cmd) => cmd.name === "cd")!;
-        const promise = cdCmd.execute("/new/path");
-        await vi.runAllTimersAsync();
-        await promise;
-
-        expect(invokeMock).toHaveBeenCalledWith(
-          "send_prompt",
-          expect.objectContaining({
-            message: expect.stringContaining("previous conversation history"),
-          })
-        );
-        expect(claudeStore.addLine).toHaveBeenCalledWith(
-          "system",
-          "Changed directory to: /new/path"
-        );
-      });
-
-      it("calls updateDiscordRpc when activeConversation is available", async () => {
-        const activeConv = {
-          name: "Test Conversation",
-          model: "claude-sonnet",
-          startedAt: new Date("2026-03-03T12:00:00Z"),
-          grantedTools: new Set<string>(),
-        };
-        getMock
-          .mockReturnValueOnce("conv-123") // get(claudeStore.activeConversationId)
-          .mockReturnValueOnce("/current") // get(claudeStore.currentWorkingDirectory)
-          .mockReturnValueOnce(activeConv); // get(conversationsStore.activeConversation)
-        vi.mocked(claudeStore.getConversationHistory).mockReturnValue("");
-        invokeMock
-          .mockResolvedValueOnce("/new/path")
-          .mockResolvedValueOnce(undefined)
-          .mockResolvedValueOnce(undefined);
-        const { updateDiscordRpc } = await import("$lib/tauri");
-
-        const cdCmd = slashCommands.find((cmd) => cmd.name === "cd")!;
-        const promise = cdCmd.execute("/new/path");
-        await vi.runAllTimersAsync();
-        await promise;
-
-        expect(updateDiscordRpc).toHaveBeenCalledWith(
-          "Test Conversation",
-          expect.any(String),
-          expect.any(Date)
-        );
-      });
-    });
-
-    describe("/new success path", () => {
-      beforeEach(() => {
-        vi.useFakeTimers();
-      });
-
-      afterEach(() => {
-        vi.useRealTimers();
-      });
-
-      it("starts a new conversation and shows success message", async () => {
-        getMock
-          .mockReturnValueOnce("conv-123") // get(claudeStore.activeConversationId)
-          .mockReturnValueOnce(null); // get(conversationsStore.activeConversation)
-        invokeMock
-          .mockResolvedValueOnce("/working/dir") // get_working_directory
-          .mockResolvedValueOnce(undefined) // interrupt_claude
-          .mockResolvedValueOnce(undefined); // start_claude
-
-        const newCmd = slashCommands.find((cmd) => cmd.name === "new")!;
-        const promise = newCmd.execute("");
-        await vi.runAllTimersAsync();
-        await promise;
-
-        expect(claudeStore.addLine).toHaveBeenCalledWith("system", "New conversation started!");
-        expect(characterState.setState).toHaveBeenCalledWith("idle");
-      });
-
-      it("calls updateDiscordRpc when activeConversation is available", async () => {
-        const activeConv = {
-          name: "My Conv",
-          model: "claude-sonnet",
-          startedAt: new Date("2026-03-03T12:00:00Z"),
-          grantedTools: new Set<string>(["tool1"]),
-        };
-        getMock.mockReturnValueOnce("conv-123").mockReturnValueOnce(activeConv);
-        invokeMock
-          .mockResolvedValueOnce("/working/dir")
-          .mockResolvedValueOnce(undefined)
-          .mockResolvedValueOnce(undefined);
-        const { updateDiscordRpc } = await import("$lib/tauri");
-
-        const newCmd = slashCommands.find((cmd) => cmd.name === "new")!;
-        const promise = newCmd.execute("");
-        await vi.runAllTimersAsync();
-        await promise;
-
-        expect(updateDiscordRpc).toHaveBeenCalledWith(
-          "My Conv",
-          expect.any(String),
-          expect.any(Date)
-        );
-      });
-    });
-  });
-});

src/lib/commands/slashCommands.ts

@@ -2,10 +2,8 @@ import { get } from "svelte/store";
 import { invoke } from "@tauri-apps/api/core";
 import { claudeStore } from "$lib/stores/claude";
 import { characterState } from "$lib/stores/character";
-import { setSkipNextGreeting, updateDiscordRpc } from "$lib/tauri";
+import { setSkipNextGreeting } from "$lib/tauri";
 import { searchState } from "$lib/stores/search";
-import { conversationsStore } from "$lib/stores/conversations";
-import { configStore } from "$lib/stores/config";
 
 export interface SlashCommand {
   name: string;
@@ -37,12 +35,6 @@ async function changeDirectory(path: string): Promise<void> {
     // Capture conversation history before disconnecting
     const conversationHistory = claudeStore.getConversationHistory();
 
-    // Get currently granted tools and config auto-granted tools
-    const activeConversation = get(conversationsStore.activeConversation);
-    const grantedTools = activeConversation ? Array.from(activeConversation.grantedTools) : [];
-    const config = configStore.getConfig();
-    const allAllowedTools = [...new Set([...grantedTools, ...config.auto_granted_tools])];
-
     await invoke("stop_claude", { conversationId });
 
     // Wait for clean shutdown
@@ -56,26 +48,9 @@ async function changeDirectory(path: string): Promise<void> {
       conversationId,
       options: {
         working_dir: validatedPath,
-        model: config.model || null,
-        api_key: config.api_key || null,
-        custom_instructions: config.custom_instructions || null,
-        mcp_servers_json: config.mcp_servers_json || null,
-        allowed_tools: allAllowedTools,
-        use_worktree: config.use_worktree ?? false,
-        disable_1m_context: config.disable_1m_context ?? false,
-        max_output_tokens: config.max_output_tokens ?? null,
       },
     });
 
-    // Update Discord RPC when reconnecting after directory change
-    if (activeConversation) {
-      await updateDiscordRpc(
-        activeConversation.name,
-        config.model || "claude",
-        activeConversation.startedAt
-      );
-    }
-
     // Wait for connection to establish
     await new Promise((resolve) => setTimeout(resolve, 1000));
 
@@ -114,12 +89,6 @@ async function startNewConversation(): Promise<void> {
       conversationId,
     });
 
-    // Get granted tools before interrupting
-    const activeConversation = get(conversationsStore.activeConversation);
-    const grantedTools = activeConversation ? Array.from(activeConversation.grantedTools) : [];
-    const config = configStore.getConfig();
-    const allAllowedTools = [...new Set([...grantedTools, ...config.auto_granted_tools])];
-
     claudeStore.addLine("system", "Starting new conversation...");
     characterState.setState("thinking");
 
@@ -133,26 +102,9 @@ async function startNewConversation(): Promise<void> {
       conversationId,
       options: {
         working_dir: workingDir,
-        model: config.model || null,
-        api_key: config.api_key || null,
-        custom_instructions: config.custom_instructions || null,
-        mcp_servers_json: config.mcp_servers_json || null,
-        allowed_tools: allAllowedTools,
-        use_worktree: config.use_worktree ?? false,
-        disable_1m_context: config.disable_1m_context ?? false,
-        max_output_tokens: config.max_output_tokens ?? null,
       },
     });
 
-    // Update Discord RPC when starting new conversation
-    if (activeConversation) {
-      await updateDiscordRpc(
-        activeConversation.name,
-        config.model || "claude",
-        activeConversation.startedAt
-      );
-    }
-
     claudeStore.addLine("system", "New conversation started!");
     characterState.setState("idle");
   } catch (error) {

src/lib/components/AchievementNotification.svelte

@@ -161,7 +161,7 @@
 
         <!-- Celebration confetti effect (CSS only) -->
         <div class="absolute inset-0 pointer-events-none overflow-hidden rounded-lg">
-          {#each Array.from({ length: 10 }, (_, i) => i) as confettiIndex (confettiIndex)}
+          {#each Array(10) as _ (_)}
             <div
               class="absolute w-2 h-2 bg-gradient-to-br {getRarityColor(
                 getAchievementRarity(currentAchievement.achievement.id)

src/lib/components/AchievementNotification.test.ts

@@ -1,153 +0,0 @@
-/**
- * AchievementNotification Component Tests
- *
- * Tests the rarity classification and colour mapping logic used by the
- * AchievementNotification component.
- *
- * What this component does:
- * - Listens for "achievement:unlocked" Tauri events
- * - Queues and displays achievement notifications one at a time
- * - Each notification shows the achievement's name, icon, description, and rarity
- * - A gradient border and badge colour correspond to the achievement's rarity
- *
- * Manual testing checklist:
- * - [ ] Achievement notification slides in from the right
- * - [ ] Notification auto-dismisses after 5 seconds
- * - [ ] Dismiss button works immediately
- * - [ ] Multiple achievements queue and display sequentially
- * - [ ] Legendary achievements have a yellow-orange gradient
- * - [ ] Epic achievements have a purple-pink gradient
- * - [ ] Rare achievements have a blue-indigo gradient
- * - [ ] Common achievements have a green-emerald gradient
- */
-
-import { describe, it, expect } from "vitest";
-
-function getAchievementRarity(id: string): string {
-  if (id === "TokenMaster") return "legendary";
-  if (["CodeMachine", "Unstoppable"].includes(id)) return "epic";
-  if (
-    [
-      "BlossomingCoder",
-      "CodeWizard",
-      "MasterBuilder",
-      "EnduranceChamp",
-      "DeepDive",
-      "CreativeCoder",
-    ].includes(id)
-  )
-    return "rare";
-  return "common";
-}
-
-function getRarityColor(rarity: string): string {
-  switch (rarity) {
-    case "legendary":
-      return "from-yellow-400 to-orange-500";
-    case "epic":
-      return "from-purple-400 to-pink-500";
-    case "rare":
-      return "from-blue-400 to-indigo-500";
-    default:
-      return "from-green-400 to-emerald-500";
-  }
-}
-
-// ---
-
-describe("getAchievementRarity", () => {
-  describe("legendary tier", () => {
-    it("classifies TokenMaster as legendary", () => {
-      expect(getAchievementRarity("TokenMaster")).toBe("legendary");
-    });
-  });
-
-  describe("epic tier", () => {
-    it("classifies CodeMachine as epic", () => {
-      expect(getAchievementRarity("CodeMachine")).toBe("epic");
-    });
-
-    it("classifies Unstoppable as epic", () => {
-      expect(getAchievementRarity("Unstoppable")).toBe("epic");
-    });
-  });
-
-  describe("rare tier", () => {
-    it("classifies BlossomingCoder as rare", () => {
-      expect(getAchievementRarity("BlossomingCoder")).toBe("rare");
-    });
-
-    it("classifies CodeWizard as rare", () => {
-      expect(getAchievementRarity("CodeWizard")).toBe("rare");
-    });
-
-    it("classifies MasterBuilder as rare", () => {
-      expect(getAchievementRarity("MasterBuilder")).toBe("rare");
-    });
-
-    it("classifies EnduranceChamp as rare", () => {
-      expect(getAchievementRarity("EnduranceChamp")).toBe("rare");
-    });
-
-    it("classifies DeepDive as rare", () => {
-      expect(getAchievementRarity("DeepDive")).toBe("rare");
-    });
-
-    it("classifies CreativeCoder as rare", () => {
-      expect(getAchievementRarity("CreativeCoder")).toBe("rare");
-    });
-  });
-
-  describe("common tier", () => {
-    it("classifies unknown IDs as common", () => {
-      expect(getAchievementRarity("FirstChat")).toBe("common");
-      expect(getAchievementRarity("SomeNewAchievement")).toBe("common");
-      expect(getAchievementRarity("")).toBe("common");
-    });
-  });
-});
-
-describe("getRarityColor", () => {
-  it("returns yellow-to-orange gradient for legendary", () => {
-    expect(getRarityColor("legendary")).toBe("from-yellow-400 to-orange-500");
-  });
-
-  it("returns purple-to-pink gradient for epic", () => {
-    expect(getRarityColor("epic")).toBe("from-purple-400 to-pink-500");
-  });
-
-  it("returns blue-to-indigo gradient for rare", () => {
-    expect(getRarityColor("rare")).toBe("from-blue-400 to-indigo-500");
-  });
-
-  it("returns green-to-emerald gradient for common", () => {
-    expect(getRarityColor("common")).toBe("from-green-400 to-emerald-500");
-  });
-
-  it("falls back to green-to-emerald gradient for unknown rarities", () => {
-    expect(getRarityColor("mythic")).toBe("from-green-400 to-emerald-500");
-    expect(getRarityColor("")).toBe("from-green-400 to-emerald-500");
-  });
-
-  describe("end-to-end rarity pipeline", () => {
-    it("produces the correct colour for a legendary achievement", () => {
-      const color = getRarityColor(getAchievementRarity("TokenMaster"));
-      expect(color).toBe("from-yellow-400 to-orange-500");
-    });
-
-    it("produces the correct colour for an epic achievement", () => {
-      const color = getRarityColor(getAchievementRarity("CodeMachine"));
-      expect(color).toBe("from-purple-400 to-pink-500");
-    });
-
-    it("produces the correct colour for a rare achievement", () => {
-      const color = getRarityColor(getAchievementRarity("CodeWizard"));
-      expect(color).toBe("from-blue-400 to-indigo-500");
-    });
-
-    it("produces the correct colour for a common achievement", () => {
-      const color = getRarityColor(getAchievementRarity("FirstChat"));
-      expect(color).toBe("from-green-400 to-emerald-500");
-    });
-  });
-});

src/lib/components/AgentMonitorPanel.svelte

@@ -1,348 +0,0 @@
-<script lang="ts">
-  import { SvelteMap } from "svelte/reactivity";
-  import { invoke } from "@tauri-apps/api/core";
-  import { claudeStore } from "$lib/stores/claude";
-  import { agentStore, getAgentsForConversation } from "$lib/stores/agents";
-  import type { AgentInfo } from "$lib/types/agents";
-  import { onMount, onDestroy } from "svelte";
-
-  interface Props {
-    isOpen: boolean;
-    onClose: () => void;
-  }
-
-  const { isOpen, onClose }: Props = $props();
-
-  let now = $state(Date.now());
-  let timerInterval: ReturnType<typeof setInterval> | null = null;
-
-  // We need a reactive subscription to agents for the active conversation
-  let agents: AgentInfo[] = $state([]);
-  let agentsUnsubscribe: (() => void) | null = null;
-
-  // Track active conversation reactively
-  let currentConversationId = $state<string | null>("");
-  const conversationIdUnsubscribe = claudeStore.activeConversationId.subscribe((id) => {
-    currentConversationId = id;
-  });
-
-  $effect(() => {
-    // Re-subscribe when conversation changes
-    if (agentsUnsubscribe) {
-      agentsUnsubscribe();
-    }
-    if (currentConversationId) {
-      const store = getAgentsForConversation(currentConversationId);
-      agentsUnsubscribe = store.subscribe((value) => {
-        agents = value;
-      });
-    } else {
-      agents = [];
-    }
-  });
-
-  const runningAgents = $derived(agents.filter((a) => a.status === "running"));
-  const completedAgents = $derived(agents.filter((a) => a.status === "completed"));
-  const erroredAgents = $derived(agents.filter((a) => a.status === "errored"));
-
-  // Organize agents into a tree structure based on parent_tool_use_id
-  const agentTree = $derived.by(() => {
-    const topLevel = agents.filter((a) => !a.parentToolUseId);
-    const childrenMap = new SvelteMap<string, AgentInfo[]>();
-
-    // Group children by their parent
-    agents.forEach((agent) => {
-      if (agent.parentToolUseId) {
-        const siblings = childrenMap.get(agent.parentToolUseId) || [];
-        siblings.push(agent);
-        childrenMap.set(agent.parentToolUseId, siblings);
-      }
-    });
-
-    return { topLevel, childrenMap };
-  });
-
-  onMount(() => {
-    timerInterval = setInterval(() => {
-      now = Date.now();
-    }, 1000);
-  });
-
-  onDestroy(() => {
-    if (timerInterval) clearInterval(timerInterval);
-    if (agentsUnsubscribe) agentsUnsubscribe();
-    conversationIdUnsubscribe();
-  });
-
-  function formatDuration(startedAt: number, endedAt?: number): string {
-    const end = endedAt || now;
-    const durationMs = end - startedAt;
-    const seconds = Math.floor(durationMs / 1000);
-    const minutes = Math.floor(seconds / 60);
-    const hours = Math.floor(minutes / 60);
-
-    if (hours > 0) {
-      return `${hours}h ${minutes % 60}m ${seconds % 60}s`;
-    }
-    if (minutes > 0) {
-      return `${minutes}m ${seconds % 60}s`;
-    }
-    return `${seconds}s`;
-  }
-
-  function getSubagentTypeLabel(type: string): string {
-    const labels: Record<string, string> = {
-      Explore: "Explorer",
-      "general-purpose": "General",
-      Plan: "Planner",
-      Bash: "Shell",
-    };
-    return labels[type] || type;
-  }
-
-  function getStatusBadgeClass(status: string): string {
-    switch (status) {
-      case "running":
-        return "bg-blue-500/20 text-blue-400 border-blue-500/30";
-      case "completed":
-        return "bg-green-500/20 text-green-400 border-green-500/30";
-      case "errored":
-        return "bg-red-500/20 text-red-400 border-red-500/30";
-      default:
-        return "bg-gray-500/20 text-gray-400 border-gray-500/30";
-    }
-  }
-
-  async function handleKillAll() {
-    if (!currentConversationId) return;
-
-    try {
-      await invoke("interrupt_claude", { conversationId: currentConversationId });
-      // Mark all running agents as errored after killing the process
-      agentStore.markAllErrored(currentConversationId);
-    } catch (error) {
-      console.error("Failed to kill Claude process:", error);
-    }
-  }
-
-  function handleClearCompleted() {
-    if (currentConversationId) {
-      agentStore.clearCompleted(currentConversationId);
-    }
-  }
-
-  // Flatten the tree for rendering with depth information
-  const flattenedAgents = $derived.by(() => {
-    const result: { agent: AgentInfo; depth: number }[] = [];
-    const { topLevel, childrenMap } = agentTree;
-
-    function addAgentAndChildren(agent: AgentInfo, depth: number) {
-      result.push({ agent, depth });
-      const children = childrenMap.get(agent.toolUseId);
-      if (children) {
-        children.forEach((child) => addAgentAndChildren(child, depth + 1));
-      }
-    }
-
-    topLevel.forEach((agent) => addAgentAndChildren(agent, 0));
-    return result;
-  });
-</script>
-
-{#if isOpen}
-  <!-- svelte-ignore a11y_click_events_have_key_events -->
-  <!-- svelte-ignore a11y_no_static_element_interactions -->
-  <div class="fixed inset-0 z-40" onclick={onClose}></div>
-
-  <div
-    class="fixed top-12 right-0 bottom-0 w-80 bg-[var(--bg-primary)] border-l border-[var(--border-color)] shadow-xl z-50 flex flex-col overflow-hidden"
-  >
-    <!-- Header -->
-    <div class="flex items-center justify-between p-4 border-b border-[var(--border-color)]">
-      <div class="flex items-center gap-2">
-        <svg
-          class="w-5 h-5 text-[var(--accent-primary)]"
-          fill="none"
-          stroke="currentColor"
-          viewBox="0 0 24 24"
-        >
-          <path
-            stroke-linecap="round"
-            stroke-linejoin="round"
-            stroke-width="2"
-            d="M9 3v2m6-2v2M9 19v2m6-2v2M5 9H3m2 6H3m18-6h-2m2 6h-2M7 19h10a2 2 0 002-2V7a2 2 0 00-2-2H7a2 2 0 00-2 2v10a2 2 0 002 2zM9 9h6v6H9V9z"
-          />
-        </svg>
-        <h3 class="text-sm font-semibold text-[var(--text-primary)]">Agent Monitor</h3>
-        {#if runningAgents.length > 0}
-          <span
-            class="px-1.5 py-0.5 text-xs rounded-full bg-blue-500/20 text-blue-400 animate-pulse"
-          >
-            {runningAgents.length} running
-          </span>
-        {/if}
-      </div>
-      <button
-        onclick={onClose}
-        class="p-1 text-[var(--text-secondary)] hover:text-[var(--text-primary)] transition-colors"
-        aria-label="Close agent monitor"
-      >
-        <svg class="w-4 h-4" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-          <path
-            stroke-linecap="round"
-            stroke-linejoin="round"
-            stroke-width="2"
-            d="M6 18L18 6M6 6l12 12"
-          />
-        </svg>
-      </button>
-    </div>
-
-    <!-- Action buttons -->
-    <div class="flex gap-2 px-4 py-2 border-b border-[var(--border-color)]">
-      <button
-        onclick={handleKillAll}
-        disabled={runningAgents.length === 0}
-        class="flex-1 px-2 py-1 text-xs bg-red-500/20 hover:bg-red-500/30 text-red-400 rounded transition-colors disabled:opacity-40 disabled:cursor-not-allowed"
-        title="Kills the entire Claude Code process to stop all agents"
-      >
-        Kill All
-      </button>
-      <button
-        onclick={handleClearCompleted}
-        disabled={completedAgents.length === 0 && erroredAgents.length === 0}
-        class="flex-1 px-2 py-1 text-xs bg-[var(--bg-secondary)] hover:bg-[var(--bg-hover,var(--bg-secondary))] text-[var(--text-secondary)] rounded transition-colors disabled:opacity-40 disabled:cursor-not-allowed"
-      >
-        Clear Finished
-      </button>
-    </div>
-
-    <!-- Agent list -->
-    <div class="flex-1 overflow-y-auto p-4 space-y-2">
-      {#if agents.length === 0}
-        <div
-          class="flex flex-col items-center justify-center h-full text-[var(--text-secondary)] text-sm"
-        >
-          <svg
-            class="w-8 h-8 mb-2 opacity-50"
-            fill="none"
-            stroke="currentColor"
-            viewBox="0 0 24 24"
-          >
-            <path
-              stroke-linecap="round"
-              stroke-linejoin="round"
-              stroke-width="2"
-              d="M9 3v2m6-2v2M9 19v2m6-2v2M5 9H3m2 6H3m18-6h-2m2 6h-2M7 19h10a2 2 0 002-2V7a2 2 0 00-2-2H7a2 2 0 00-2 2v10a2 2 0 002 2zM9 9h6v6H9V9z"
-            />
-          </svg>
-          <p>No agents detected yet</p>
-          <p class="text-xs mt-1 opacity-70">
-            Agents will appear here when Claude uses the Task tool
-          </p>
-        </div>
-      {:else}
-        {#each flattenedAgents as { agent, depth } (agent.toolUseId)}
-          <div
-            class="p-3 rounded-lg border border-[var(--border-color)] bg-[var(--bg-secondary)] {agent.status ===
-            'running'
-              ? 'border-l-2 border-l-blue-500'
-              : agent.status === 'errored'
-                ? 'border-l-2 border-l-red-500'
-                : 'border-l-2 border-l-green-500'}"
-            style="margin-left: {depth * 12}px; width: calc(100% - {depth * 12}px);"
-          >
-            <!-- Agent header -->
-            <div class="flex items-center justify-between mb-1">
-              <div class="flex items-center gap-1.5">
-                {#if depth > 0}
-                  <svg
-                    class="w-3 h-3 text-[var(--text-secondary)]"
-                    fill="none"
-                    stroke="currentColor"
-                    viewBox="0 0 24 24"
-                  >
-                    <path
-                      stroke-linecap="round"
-                      stroke-linejoin="round"
-                      stroke-width="2"
-                      d="M9 5l7 7-7 7"
-                    />
-                  </svg>
-                {/if}
-                <img
-                  src={agent.characterAvatar}
-                  alt={agent.characterName}
-                  class="w-5 h-5 rounded-full object-cover"
-                />
-                <span class="text-[10px] font-medium text-[var(--text-primary)]">
-                  {agent.characterName}
-                </span>
-                <span
-                  class="px-1.5 py-0.5 text-[10px] rounded border {getStatusBadgeClass(
-                    agent.status
-                  )}"
-                >
-                  {getSubagentTypeLabel(agent.subagentType)}
-                </span>
-              </div>
-              <span
-                class="text-[10px] {agent.status === 'running'
-                  ? 'text-blue-400'
-                  : 'text-[var(--text-secondary)]'}"
-              >
-                {#if agent.durationMs !== undefined}
-                  {Math.floor(agent.durationMs / 1000)}s
-                {:else}
-                  {formatDuration(agent.startedAt, agent.endedAt)}
-                {/if}
-                {#if agent.status === "running"}
-                  <span class="inline-block w-1 h-1 bg-blue-400 rounded-full animate-pulse ml-1"
-                  ></span>
-                {/if}
-              </span>
-            </div>
-
-            <!-- Agent description -->
-            <p class="text-xs text-[var(--text-primary)] truncate" title={agent.description}>
-              {agent.description}
-            </p>
-
-            <!-- Status indicator -->
-            <div class="mt-1 flex items-center gap-1">
-              {#if agent.status === "running"}
-                <span class="text-[10px] text-blue-400">Running...</span>
-              {:else if agent.status === "completed"}
-                <span class="text-[10px] text-green-400">Completed</span>
-              {:else}
-                <span class="text-[10px] text-red-400">Errored / Killed</span>
-              {/if}
-            </div>
-
-            <!-- Last assistant message snippet -->
-            {#if agent.lastAssistantMessage}
-              <p
-                class="mt-1 text-[10px] text-[var(--text-secondary)] italic truncate"
-                title={agent.lastAssistantMessage}
-              >
-                {agent.lastAssistantMessage}
-              </p>
-            {/if}
-          </div>
-        {/each}
-      {/if}
-    </div>
-
-    <!-- Footer summary -->
-    {#if agents.length > 0}
-      <div
-        class="px-4 py-2 border-t border-[var(--border-color)] text-[10px] text-[var(--text-secondary)]"
-      >
-        {agents.length} total &middot;
-        {runningAgents.length} running &middot;
-        {completedAgents.length} completed &middot;
-        {erroredAgents.length} errored
-      </div>
-    {/if}
-  </div>
-{/if}

src/lib/components/CastPanel.svelte

@@ -1,140 +0,0 @@
-<script lang="ts">
-  import { CHARACTER_POOL } from "$lib/utils/agentCharacters";
-
-  interface Props {
-    onClose: () => void;
-  }
-
-  const { onClose }: Props = $props();
-</script>
-
-<div
-  class="fixed inset-0 bg-black/50 backdrop-blur-sm z-50 flex items-center justify-center p-4"
-  onclick={onClose}
-  role="button"
-  tabindex="0"
-  onkeydown={(e) => e.key === "Escape" && onClose()}
->
-  <div
-    class="bg-[var(--bg-primary)] border border-[var(--border-color)] rounded-lg shadow-xl max-w-2xl w-full p-6 max-h-[90vh] overflow-y-auto"
-    onclick={(e) => e.stopPropagation()}
-    onkeydown={(e) => e.stopPropagation()}
-    role="dialog"
-    aria-labelledby="cast-title"
-    tabindex="-1"
-  >
-    <div class="flex items-center justify-between mb-6">
-      <h2 id="cast-title" class="text-xl font-semibold text-[var(--text-primary)]">
-        Meet the Team
-      </h2>
-      <button
-        onclick={onClose}
-        class="p-1 text-[var(--text-secondary)] hover:text-[var(--text-primary)] transition-colors"
-        aria-label="Close"
-      >
-        <svg class="w-5 h-5" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-          <path
-            stroke-linecap="round"
-            stroke-linejoin="round"
-            stroke-width="2"
-            d="M6 18L18 6M6 6l12 12"
-          />
-        </svg>
-      </button>
-    </div>
-
-    <!-- Principal cast: Hikari + Naomi -->
-    <div class="grid grid-cols-1 gap-3 mb-6 sm:grid-cols-2">
-      <div
-        class="flex items-center gap-3 p-4 rounded-lg bg-[var(--bg-secondary)] border border-[var(--accent-primary)]/40"
-      >
-        <img
-          src="https://cdn.nhcarrigan.com/hikari.png"
-          alt="Hikari"
-          class="w-16 h-16 object-cover rounded-full border-2 border-[var(--border-color)] shrink-0"
-        />
-        <div>
-          <div class="flex items-center gap-2 mb-1">
-            <span class="font-semibold text-[var(--text-primary)]">Hikari</span>
-            <span
-              class="text-xs px-2 py-0.5 rounded-full bg-[var(--accent-primary)]/20 text-[var(--accent-primary)] font-medium"
-            >
-              Chief Operating Officer
-            </span>
-          </div>
-          <p class="text-xs text-[var(--text-secondary)]">
-            Holds the line so the others don't have to. Never without her clipboard — or her
-            glasses.
-          </p>
-        </div>
-      </div>
-      <div
-        class="flex items-center gap-3 p-4 rounded-lg bg-[var(--bg-secondary)] border border-[var(--accent-primary)]/40"
-      >
-        <img
-          src="https://cdn.nhcarrigan.com/profile.png"
-          alt="Naomi"
-          class="w-16 h-16 object-cover rounded-full border-2 border-[var(--border-color)] shrink-0"
-        />
-        <div>
-          <div class="flex items-center gap-2 mb-1">
-            <span class="font-semibold text-[var(--text-primary)]">Naomi</span>
-            <span
-              class="text-xs px-2 py-0.5 rounded-full bg-[var(--accent-primary)]/20 text-[var(--accent-primary)] font-medium"
-            >
-              Chief hEx-ecutive Officer
-            </span>
-          </div>
-          <p class="text-xs text-[var(--text-secondary)]">
-            A 525-year-old vampire running a tech company from behind a VTuber avatar. Fixes server
-            crashes at 4 AM.
-          </p>
-        </div>
-      </div>
-    </div>
-
-    <!-- Subagent girls grid -->
-    <div>
-      <h3 class="text-sm font-medium text-[var(--text-secondary)] uppercase tracking-wider mb-3">
-        Subagent Squad
-      </h3>
-      <div class="grid grid-cols-2 gap-3 sm:grid-cols-3">
-        {#each CHARACTER_POOL as character (character.name)}
-          <div
-            class="flex flex-col items-center gap-2 p-3 rounded-lg bg-[var(--bg-secondary)] border border-[var(--border-color)] text-center"
-          >
-            <img
-              src={character.avatar}
-              alt={character.name}
-              class="w-14 h-14 object-cover rounded-full border-2 border-[var(--border-color)]"
-            />
-            <span class="text-sm font-medium text-[var(--text-primary)]">{character.name}</span>
-            <span
-              class="text-xs px-2 py-0.5 rounded-full bg-[var(--accent-primary)]/20 text-[var(--accent-primary)] font-medium"
-            >
-              {character.title}
-            </span>
-            <p class="text-xs text-[var(--text-secondary)] leading-snug">{character.description}</p>
-          </div>
-        {/each}
-      </div>
-    </div>
-  </div>
-</div>
-
-<style>
-  [role="dialog"] {
-    animation: slideIn 0.2s ease-out;
-  }
-
-  @keyframes slideIn {
-    from {
-      opacity: 0;
-      transform: scale(0.95);
-    }
-    to {
-      opacity: 1;
-      transform: scale(1);
-    }
-  }
-</style>

src/lib/components/CliVersion.svelte

@@ -1,160 +0,0 @@
-<script lang="ts">
-  import { invoke } from "@tauri-apps/api/core";
-  import { onMount } from "svelte";
-
-  const SUPPORTED_CLI_VERSION = "2.1.53";
-
-  let installedVersion = $state("Loading...");
-
-  function compareVersions(a: string, b: string): number {
-    const aParts = a.split(".").map(Number);
-    const bParts = b.split(".").map(Number);
-    for (let i = 0; i < Math.max(aParts.length, bParts.length); i++) {
-      const aVal = aParts[i] ?? 0;
-      const bVal = bParts[i] ?? 0;
-      if (aVal > bVal) return 1;
-      if (aVal < bVal) return -1;
-    }
-    return 0;
-  }
-
-  let displayVersion = $derived(installedVersion.split(" (")[0]);
-
-  let supportedBadgeState = $derived.by(() => {
-    if (installedVersion === "Loading..." || installedVersion === "Unknown") {
-      return "neutral";
-    }
-    const semverMatch = /(\d+\.\d+\.\d+)/.exec(installedVersion);
-    if (!semverMatch) return "neutral";
-    const cmp = compareVersions(semverMatch[1], SUPPORTED_CLI_VERSION);
-    if (cmp > 0) return "ahead";
-    if (cmp < 0) return "behind";
-    return "current";
-  });
-
-  async function fetchVersion() {
-    try {
-      const result = await invoke<string>("get_claude_version");
-      installedVersion = result;
-    } catch (error) {
-      console.error("Failed to get Claude CLI version:", error);
-      installedVersion = "Unknown";
-    }
-  }
-
-  onMount(() => {
-    fetchVersion();
-  });
-</script>
-
-<div class="cli-versions">
-  <div class="cli-version">
-    <svg
-      class="terminal-icon"
-      width="14"
-      height="14"
-      viewBox="0 0 24 24"
-      fill="none"
-      stroke="currentColor"
-      stroke-width="2"
-      stroke-linecap="round"
-      stroke-linejoin="round"
-    >
-      <polyline points="4 17 10 11 4 5" />
-      <line x1="12" y1="19" x2="20" y2="19" />
-    </svg>
-    <span class="version-text">CLI {displayVersion}</span>
-  </div>
-
-  <div class="cli-version supported {supportedBadgeState}" title="Highest audited CLI version">
-    <svg
-      class="terminal-icon"
-      width="14"
-      height="14"
-      viewBox="0 0 24 24"
-      fill="none"
-      stroke="currentColor"
-      stroke-width="2"
-      stroke-linecap="round"
-      stroke-linejoin="round"
-    >
-      <path d="M12 22s8-4 8-10V5l-8-3-8 3v7c0 6 8 10 8 10z" />
-    </svg>
-    <span class="version-text">Supported {SUPPORTED_CLI_VERSION}</span>
-  </div>
-
-  {#if supportedBadgeState === "ahead"}
-    <span class="version-warning ahead"
-      >Your version is newer, some features may not be supported</span
-    >
-  {:else if supportedBadgeState === "behind"}
-    <span class="version-warning behind"
-      >Your version is out of date, please update to ensure compatibility</span
-    >
-  {/if}
-</div>
-
-<style>
-  .cli-versions {
-    display: flex;
-    gap: 6px;
-    align-items: center;
-  }
-
-  .cli-version {
-    display: flex;
-    align-items: center;
-    gap: 6px;
-    padding: 4px 12px;
-    background: var(--bg-secondary);
-    border: 1px solid var(--border-color);
-    border-radius: 6px;
-    color: var(--text-secondary);
-    font-size: 0.85rem;
-    font-family: var(--font-mono, monospace);
-    transition: all 0.2s;
-  }
-
-  .cli-version:hover {
-    border-color: var(--accent-primary);
-    color: var(--accent-primary);
-  }
-
-  .cli-version.supported.current {
-    border-color: var(--success-color, #4caf50);
-    color: var(--success-color, #4caf50);
-  }
-
-  .cli-version.supported.ahead {
-    border-color: var(--warning-color, #ff9800);
-    color: var(--warning-color, #ff9800);
-  }
-
-  .cli-version.supported.behind {
-    border-color: var(--error-color, #f44336);
-    color: var(--error-color, #f44336);
-  }
-
-  .terminal-icon {
-    flex-shrink: 0;
-    opacity: 0.7;
-  }
-
-  .version-text {
-    white-space: nowrap;
-  }
-
-  .version-warning {
-    font-size: 0.75rem;
-    font-style: italic;
-    white-space: nowrap;
-  }
-
-  .version-warning.ahead {
-    color: var(--warning-color, #ff9800);
-  }
-
-  .version-warning.behind {
-    color: var(--error-color, #f44336);
-  }
-</style>

src/lib/components/CliVersion.test.ts

@@ -1,134 +0,0 @@
-/**
- * CliVersion Component Tests
- *
- * Tests the version comparison logic used by the CliVersion component,
- * which compares the installed CLI version against the supported version.
- *
- * What this component does:
- * - Displays the installed Claude CLI version
- * - Displays the highest audited/supported CLI version
- * - Shows a warning when the installed version is ahead of or behind supported
- *
- * Manual testing checklist:
- * - [ ] Installed version is fetched and displayed on mount
- * - [ ] "current" badge shows in green when versions match
- * - [ ] "ahead" badge shows in amber when installed is newer than supported
- * - [ ] "behind" badge shows in red when installed is older than supported
- * - [ ] Warning message appears for "ahead" and "behind" states
- */
-
-import { describe, it, expect } from "vitest";
-
-const SUPPORTED_CLI_VERSION = "2.1.53";
-
-function compareVersions(a: string, b: string): number {
-  const aParts = a.split(".").map(Number);
-  const bParts = b.split(".").map(Number);
-  for (let i = 0; i < Math.max(aParts.length, bParts.length); i++) {
-    const aVal = aParts[i] ?? 0;
-    const bVal = bParts[i] ?? 0;
-    if (aVal > bVal) return 1;
-    if (aVal < bVal) return -1;
-  }
-  return 0;
-}
-
-// ---
-
-describe("SUPPORTED_CLI_VERSION", () => {
-  it("is defined and non-empty", () => {
-    expect(SUPPORTED_CLI_VERSION).toBeTruthy();
-  });
-
-  it("matches the expected audited version", () => {
-    expect(SUPPORTED_CLI_VERSION).toBe("2.1.53");
-  });
-});
-
-describe("compareVersions", () => {
-  describe("equal versions", () => {
-    it("returns 0 for identical versions", () => {
-      expect(compareVersions("1.0.0", "1.0.0")).toBe(0);
-    });
-
-    it("returns 0 for the supported CLI version against itself", () => {
-      expect(compareVersions(SUPPORTED_CLI_VERSION, SUPPORTED_CLI_VERSION)).toBe(0);
-    });
-
-    it("returns 0 for 0.0.0 vs 0.0.0", () => {
-      expect(compareVersions("0.0.0", "0.0.0")).toBe(0);
-    });
-  });
-
-  describe("major version differences", () => {
-    it("returns 1 when a has a higher major version", () => {
-      expect(compareVersions("2.0.0", "1.0.0")).toBe(1);
-    });
-
-    it("returns -1 when a has a lower major version", () => {
-      expect(compareVersions("1.0.0", "2.0.0")).toBe(-1);
-    });
-  });
-
-  describe("minor version differences", () => {
-    it("returns 1 when a has a higher minor version", () => {
-      expect(compareVersions("1.2.0", "1.1.0")).toBe(1);
-    });
-
-    it("returns -1 when a has a lower minor version", () => {
-      expect(compareVersions("1.1.0", "1.2.0")).toBe(-1);
-    });
-  });
-
-  describe("patch version differences", () => {
-    it("returns 1 when a has a higher patch version", () => {
-      expect(compareVersions("1.0.2", "1.0.1")).toBe(1);
-    });
-
-    it("returns -1 when a has a lower patch version", () => {
-      expect(compareVersions("1.0.1", "1.0.2")).toBe(-1);
-    });
-  });
-
-  describe("major version takes precedence", () => {
-    it("returns 1 when a has a higher major but lower minor", () => {
-      expect(compareVersions("2.0.0", "1.9.9")).toBe(1);
-    });
-
-    it("returns -1 when a has a lower major but higher minor", () => {
-      expect(compareVersions("1.9.9", "2.0.0")).toBe(-1);
-    });
-  });
-
-  describe("unequal segment counts", () => {
-    it("treats missing segments as 0 (a shorter than b)", () => {
-      expect(compareVersions("1.0", "1.0.0")).toBe(0);
-    });
-
-    it("treats missing segments as 0 (a longer than b)", () => {
-      expect(compareVersions("1.0.0", "1.0")).toBe(0);
-    });
-
-    it("correctly compares when a has an extra non-zero segment", () => {
-      expect(compareVersions("1.0.1", "1.0")).toBe(1);
-    });
-
-    it("correctly compares when b has an extra non-zero segment", () => {
-      expect(compareVersions("1.0", "1.0.1")).toBe(-1);
-    });
-  });
-
-  describe("supported CLI version comparisons", () => {
-    it("returns 1 for a version ahead of supported", () => {
-      expect(compareVersions("2.2.0", SUPPORTED_CLI_VERSION)).toBe(1);
-    });
-
-    it("returns -1 for a version behind supported", () => {
-      expect(compareVersions("2.1.0", SUPPORTED_CLI_VERSION)).toBe(-1);
-    });
-
-    it("returns 0 for exactly the supported version", () => {
-      expect(compareVersions("2.1.53", SUPPORTED_CLI_VERSION)).toBe(0);
-    });
-  });
-});

src/lib/components/CloseAppConfirmModal.svelte

@@ -1,116 +0,0 @@
-<script lang="ts">
-  interface Props {
-    isOpen: boolean;
-    hasActiveConversation: boolean;
-    onClose: () => void;
-    onMinimize: () => void;
-    onCancel: () => void;
-  }
-
-  const { isOpen, hasActiveConversation, onClose, onMinimize, onCancel }: Props = $props();
-
-  function handleKeydown(event: KeyboardEvent) {
-    if (!isOpen) return;
-
-    if (event.key === "Escape") {
-      event.preventDefault();
-      onCancel();
-    }
-  }
-</script>
-
-<svelte:window onkeydown={handleKeydown} />
-
-{#if isOpen}
-  <div
-    class="fixed inset-0 bg-black/50 backdrop-blur-sm z-50 flex items-center justify-center p-4"
-    onclick={onCancel}
-    role="button"
-    tabindex="0"
-    onkeydown={(e) => e.key === " " && onCancel()}
-  >
-    <div
-      class="bg-[var(--bg-primary)] border border-[var(--border-color)] rounded-lg shadow-xl max-w-md w-full"
-      onclick={(e) => e.stopPropagation()}
-      onkeydown={(e) => e.stopPropagation()}
-      role="dialog"
-      aria-labelledby="confirm-title"
-      aria-describedby="confirm-message"
-      tabindex="-1"
-    >
-      <div class="p-6">
-        <div class="flex items-start gap-4">
-          <div
-            class="w-10 h-10 rounded-lg bg-yellow-500/20 flex items-center justify-center flex-shrink-0"
-          >
-            <svg
-              class="w-6 h-6 text-yellow-500"
-              fill="none"
-              stroke="currentColor"
-              viewBox="0 0 24 24"
-            >
-              <path
-                stroke-linecap="round"
-                stroke-linejoin="round"
-                stroke-width="2"
-                d="M12 9v2m0 4h.01m-6.938 4h13.856c1.54 0 2.502-1.667 1.732-3L13.732 4c-.77-1.333-2.694-1.333-3.464 0L3.34 16c-.77 1.333.192 3 1.732 3z"
-              />
-            </svg>
-          </div>
-          <div class="flex-1">
-            <h3 id="confirm-title" class="text-lg font-semibold text-[var(--text-primary)] mb-1">
-              Close Hikari Desktop?
-            </h3>
-            <p id="confirm-message" class="text-sm text-[var(--text-secondary)]">
-              {#if hasActiveConversation}
-                You have an active conversation with Claude. Are you sure you want to close the
-                application? Your conversation history will be saved, but any in-progress tasks will
-                be interrupted.
-              {:else}
-                Are you sure you want to close the application?
-              {/if}
-            </p>
-          </div>
-        </div>
-
-        <div class="flex gap-3 mt-6 justify-end">
-          <button
-            onclick={onCancel}
-            class="px-4 py-2 text-sm font-medium text-gray-300 bg-[var(--bg-secondary)] hover:bg-[var(--bg-tertiary)] border border-[var(--border-color)] rounded-lg transition-colors"
-          >
-            Cancel
-          </button>
-          <button
-            onclick={onMinimize}
-            class="px-4 py-2 text-sm font-medium text-white bg-blue-600 hover:bg-blue-700 rounded-lg transition-colors"
-          >
-            Minimize to Tray
-          </button>
-          <button
-            onclick={onClose}
-            class="px-4 py-2 text-sm font-medium text-white bg-red-600 hover:bg-red-700 rounded-lg transition-colors"
-          >
-            Close Application
-          </button>
-        </div>
-      </div>
-    </div>
-  </div>
-{/if}
-
-<style>
-  [role="dialog"] {
-    animation: slideIn 0.2s ease-out;
-  }
-
-  @keyframes slideIn {
-    from {
-      opacity: 0;
-      transform: scale(0.95) translateY(10px);
-    }
-    to {
-      opacity: 1;
-      transform: scale(1) translateY(0);
-    }
-  }
-</style>

src/lib/components/CompactMode.svelte

@@ -3,9 +3,8 @@
   import { get } from "svelte/store";
   import { claudeStore, isClaudeProcessing } from "$lib/stores/claude";
   import { characterState, characterInfo } from "$lib/stores/character";
-  import { isStreamerMode, configStore } from "$lib/stores/config";
+  import { isStreamerMode } from "$lib/stores/config";
   import { handleNewUserMessage } from "$lib/notifications/rules";
-  import { setSkipNextGreeting } from "$lib/tauri";
   import type { CharacterState, CharacterStateInfo } from "$lib/types/states";
 
   interface Props {
@@ -14,9 +13,6 @@
 
   let { onExpand }: Props = $props();
 
-  const configValues = configStore.config;
-  const hasBackgroundImage = $derived($configValues.background_image_path !== null);
-
   let inputValue = $state("");
   let isSubmitting = $state(false);
   let isConnected = $state(false);
@@ -131,11 +127,8 @@
       const conversationId = get(claudeStore.activeConversationId);
       if (!conversationId) return;
 
-      // Set flag to preserve stats/permissions (don't treat next connect as new session)
-      setSkipNextGreeting(true);
-
       await invoke("interrupt_claude", { conversationId });
-      claudeStore.addLine("system", "Process interrupted via stop button");
+      claudeStore.addLine("system", "Interrupted");
       characterState.setState("idle");
     } catch (error) {
       console.error("Failed to interrupt:", error);
@@ -153,10 +146,7 @@
   }
 </script>
 
-<div
-  class="compact-container {getStateGlow()}"
-  style={hasBackgroundImage ? "background: transparent !important;" : ""}
->
+<div class="compact-container {getStateGlow()}">
   <!-- Character sprite (smaller) -->
   <div class="compact-character">
     <div class="sprite-wrapper {getAnimationClass()}">
@@ -259,9 +249,6 @@
     -webkit-mask:
       linear-gradient(#fff 0 0) content-box,
       linear-gradient(#fff 0 0);
-    mask:
-      linear-gradient(#fff 0 0) content-box,
-      linear-gradient(#fff 0 0);
     -webkit-mask-composite: xor;
     mask-composite: exclude;
     opacity: 0;

src/lib/components/ConversationTabs.svelte

@@ -12,25 +12,6 @@
   let editingTabId = $state<string | null>(null);
   let editingName = $state("");
 
-  // Tab order for pointer-drag reordering
-  let tabOrder = $state<string[]>([]);
-  let draggedId = $state<string | null>(null);
-  let dragOverId = $state<string | null>(null);
-  let dragStartX = 0;
-  let isDragging = false;
-  let wasDragged = false;
-  let tabsRef = $state<HTMLElement | null>(null);
-
-  // Keep tabOrder in sync with conversations map (add new, remove deleted)
-  $effect(() => {
-    const currentIds = Array.from($conversations.keys());
-    const validIds = tabOrder.filter((id) => currentIds.includes(id));
-    const newIds = currentIds.filter((id) => !tabOrder.includes(id));
-    if (validIds.length !== tabOrder.length || newIds.length > 0) {
-      tabOrder = [...validIds, ...newIds];
-    }
-  });
-
   // Track last seen message count for each conversation
   let lastSeenMessageCount = new SvelteMap<string, number>();
 
@@ -157,73 +138,8 @@
     }
   }
 
-  async function handleTabClick(id: string) {
-    if (wasDragged) {
-      wasDragged = false;
-      return;
-    }
-    await switchTab(id);
-  }
-
-  function handlePointerDown(event: PointerEvent, id: string) {
-    if (editingTabId === id) return;
-    draggedId = id;
-    dragStartX = event.clientX;
-    isDragging = false;
-    wasDragged = false;
-
-    function onMove(e: PointerEvent) {
-      if (!isDragging && Math.abs(e.clientX - dragStartX) > 5) {
-        isDragging = true;
-      }
-      if (!isDragging || !tabsRef) return;
-      const tabs = tabsRef.querySelectorAll<HTMLElement>("[data-tab-id]");
-      dragOverId = null;
-      for (const tab of tabs) {
-        const rect = tab.getBoundingClientRect();
-        if (e.clientX >= rect.left && e.clientX <= rect.right) {
-          const tabId = tab.dataset.tabId;
-          if (tabId && tabId !== id) {
-            dragOverId = tabId;
-          }
-          break;
-        }
-      }
-    }
-
-    function onUp() {
-      if (isDragging && draggedId && dragOverId && draggedId !== dragOverId) {
-        const order = [...tabOrder];
-        const fromIndex = order.indexOf(draggedId);
-        const toIndex = order.indexOf(dragOverId);
-        order.splice(fromIndex, 1);
-        order.splice(toIndex, 0, draggedId);
-        tabOrder = order;
-        wasDragged = true;
-      }
-      draggedId = null;
-      dragOverId = null;
-      isDragging = false;
-      window.removeEventListener("pointermove", onMove);
-      window.removeEventListener("pointerup", onUp);
-      window.removeEventListener("pointercancel", onUp);
-    }
-
-    window.addEventListener("pointermove", onMove);
-    window.addEventListener("pointerup", onUp);
-    window.addEventListener("pointercancel", onUp);
-  }
-
   // Keyboard shortcuts
   onMount(() => {
-    // Initialise all conversations as seen on mount so that remounting
-    // this component (e.g. after closing the file editor) doesn't falsely
-    // mark existing messages as unread.
-    for (const [id, conversation] of $conversations) {
-      lastSeenMessageCount.set(id, conversation.terminalLines.length);
-    }
-    lastSeenMessageCount = lastSeenMessageCount;
-
     function handleGlobalKeydown(event: KeyboardEvent) {
       // Ctrl/Cmd + T: New tab
       if ((event.ctrlKey || event.metaKey) && event.key === "t") {
@@ -249,19 +165,21 @@
       // Ctrl/Cmd + Tab: Next tab
       else if ((event.ctrlKey || event.metaKey) && event.key === "Tab" && !event.shiftKey) {
         event.preventDefault();
-        const currentIndex = tabOrder.findIndex((id) => id === $activeConversationId);
+        const tabs = Array.from($conversations.keys());
+        const currentIndex = tabs.findIndex((id) => id === $activeConversationId);
         if (currentIndex !== -1) {
-          const nextIndex = (currentIndex + 1) % tabOrder.length;
-          claudeStore.switchConversation(tabOrder[nextIndex]);
+          const nextIndex = (currentIndex + 1) % tabs.length;
+          claudeStore.switchConversation(tabs[nextIndex]);
         }
       }
       // Ctrl/Cmd + Shift + Tab: Previous tab
       else if ((event.ctrlKey || event.metaKey) && event.key === "Tab" && event.shiftKey) {
         event.preventDefault();
-        const currentIndex = tabOrder.findIndex((id) => id === $activeConversationId);
+        const tabs = Array.from($conversations.keys());
+        const currentIndex = tabs.findIndex((id) => id === $activeConversationId);
         if (currentIndex !== -1) {
-          const prevIndex = (currentIndex - 1 + tabOrder.length) % tabOrder.length;
-          claudeStore.switchConversation(tabOrder[prevIndex]);
+          const prevIndex = (currentIndex - 1 + tabs.length) % tabs.length;
+          claudeStore.switchConversation(tabs[prevIndex]);
         }
       }
     }
@@ -272,22 +190,15 @@
 </script>
 
 <div
-  bind:this={tabsRef}
   class="terminal-tabs flex items-center gap-1 px-2 py-1 bg-[var(--bg-secondary)] border-b border-[var(--border-color)]"
 >
-  {#each tabOrder
-    .filter((id) => $conversations.has(id))
-    .map((id) => ({ id, conversation: $conversations.get(id)! })) as { id, conversation } (id)}
+  {#each Array.from($conversations.entries()) as [id, conversation] (id)}
     <div
-      data-tab-id={id}
-      class="tab-item group relative flex items-center px-3 py-1.5 rounded-t transition-all
+      class="tab-item group relative flex items-center px-3 py-1.5 rounded-t cursor-pointer transition-all
         {id === $activeConversationId
         ? 'bg-[var(--bg-terminal)] text-[var(--text-primary)] border-t border-l border-r border-[var(--border-color)]'
-        : 'bg-[var(--bg-tertiary)] text-[var(--text-secondary)] hover:bg-[var(--bg-terminal)]/50'}
-        {dragOverId === id && draggedId !== id ? 'drag-over' : ''}
-        {draggedId === id ? 'dragging' : ''}"
-      onpointerdown={(e) => handlePointerDown(e, id)}
-      onclick={() => handleTabClick(id)}
+        : 'bg-[var(--bg-tertiary)] text-[var(--text-secondary)] hover:bg-[var(--bg-terminal)]/50'}"
+      onclick={() => switchTab(id)}
       onkeydown={(e) => handleTabKeydown(id, e)}
       role="tab"
       tabindex={0}
@@ -300,7 +211,7 @@
           onblur={saveTabName}
           onkeydown={handleKeydown}
           onclick={(e) => e.stopPropagation()}
-          class="bg-transparent border-b border-[var(--border-color)] outline-none px-0 py-0 text-sm w-32 select-text"
+          class="bg-transparent border-b border-[var(--border-color)] outline-none px-0 py-0 text-sm w-32"
         />
       {:else}
         <div class="flex items-center gap-2">
@@ -385,20 +296,5 @@
 
   .tab-item {
     min-width: 100px;
-    cursor: grab;
-    touch-action: none;
-    user-select: none;
-  }
-
-  .tab-item:active {
-    cursor: grabbing;
-  }
-
-  .drag-over {
-    border-left: 2px solid var(--accent-primary);
-  }
-
-  .dragging {
-    opacity: 0.4;
   }
 </style>