diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index a90d2cc4..e13c732f 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -92,3 +92,29 @@ jobs: run: npm ci - name: Tauri debug build run: npm run tauri -- build --debug --no-bundle + + build-windows: + runs-on: windows-latest + needs: + - lint + - typecheck + - test-js + steps: + - uses: actions/checkout@v4 + - uses: actions/setup-node@v4 + with: + node-version: "20" + cache: "npm" + - uses: dtolnay/rust-toolchain@stable + - name: Install LLVM (bindgen) + run: choco install llvm -y --no-progress + - name: Configure LLVM (bindgen) + run: | + echo "LIBCLANG_PATH=C:\\Program Files\\LLVM\\bin" >> $env:GITHUB_ENV + echo "C:\\Program Files\\LLVM\\bin" >> $env:GITHUB_PATH + - name: Install dependencies + run: npm ci + - name: Doctor (Windows) + run: npm run doctor:win + - name: Tauri debug build (Windows) + run: npm run tauri -- build --debug --no-bundle --config src-tauri/tauri.windows.conf.json diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 6025a9f9..85f084de 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -219,12 +219,74 @@ jobs: name: appimage-${{ matrix.arch }} path: src-tauri/target/release/bundle/appimage/*.AppImage* + build_windows: + runs-on: windows-latest + environment: release + env: + TAURI_SIGNING_PRIVATE_KEY_PASSWORD: ${{ secrets.TAURI_SIGNING_PRIVATE_KEY_PASSWORD }} + TAURI_SIGNING_PRIVATE_KEY_B64: ${{ secrets.TAURI_SIGNING_PRIVATE_KEY_B64 }} + steps: + - uses: actions/checkout@v4 + + - name: setup node + uses: actions/setup-node@v4 + with: + node-version: lts/* + cache: npm + + - name: install Rust stable + uses: dtolnay/rust-toolchain@stable + + - name: Install LLVM (bindgen) + run: choco install llvm -y --no-progress + + - name: Configure LLVM (bindgen) + run: | + echo "LIBCLANG_PATH=C:\\Program Files\\LLVM\\bin" >> $env:GITHUB_ENV + echo "C:\\Program Files\\LLVM\\bin" >> $env:GITHUB_PATH + + - name: install frontend dependencies + run: npm ci + + - name: Write Tauri signing key + shell: bash + run: | + set -euo pipefail + python - <<'PY' + import base64 + import os + from pathlib import Path + + raw = base64.b64decode(os.environ["TAURI_SIGNING_PRIVATE_KEY_B64"]) + home = Path.home() + target = home / ".tauri" + target.mkdir(parents=True, exist_ok=True) + (target / "codexmonitor.key").write_bytes(raw) + PY + + - name: build windows bundles + shell: bash + run: | + set -euo pipefail + export TAURI_SIGNING_PRIVATE_KEY + TAURI_SIGNING_PRIVATE_KEY="$(cat "$HOME/.tauri/codexmonitor.key")" + npm run tauri:build:win + + - name: Upload Windows artifacts + uses: actions/upload-artifact@v4 + with: + name: windows-artifacts + path: | + src-tauri/target/release/bundle/nsis/*.exe* + src-tauri/target/release/bundle/msi/*.msi* + release: runs-on: ubuntu-latest environment: release needs: - build_macos - build_linux + - build_windows steps: - name: Checkout uses: actions/checkout@v4 @@ -244,6 +306,12 @@ jobs: path: release-artifacts merge-multiple: true + - name: Download Windows artifacts + uses: actions/download-artifact@v4 + with: + name: windows-artifacts + path: release-artifacts + - name: Build latest.json run: | set -euo pipefail @@ -345,6 +413,27 @@ jobs: "signature": sig_path.read_text().strip(), } + exe_candidates = sorted(artifacts_dir.rglob("*.exe"), key=lambda p: p.name.lower()) + windows_installer = None + for candidate in exe_candidates: + lowered = candidate.name.lower() + if "setup" in lowered or "installer" in lowered: + windows_installer = candidate + break + if windows_installer is None and exe_candidates: + windows_installer = exe_candidates[0] + if windows_installer is None: + raise SystemExit("No Windows installer (.exe) found for latest.json") + + win_sig_path = windows_installer.with_suffix(windows_installer.suffix + ".sig") + if not win_sig_path.exists(): + raise SystemExit(f"Missing signature for {windows_installer.name}") + + platforms["windows-x86_64"] = { + "url": f"https://github.com/Dimillian/CodexMonitor/releases/download/v${VERSION}/{windows_installer.name}", + "signature": win_sig_path.read_text().strip(), + } + payload = { "version": "${VERSION}", "notes": notes, @@ -378,6 +467,8 @@ jobs: release-artifacts/CodexMonitor.app.tar.gz \ release-artifacts/CodexMonitor.app.tar.gz.sig \ release-artifacts/*.AppImage* \ + release-artifacts/nsis/*.exe* \ + release-artifacts/msi/*.msi* \ release-artifacts/latest.json - name: Bump version and open PR diff --git a/PLAN.md b/PLAN.md new file mode 100644 index 00000000..8909d902 --- /dev/null +++ b/PLAN.md @@ -0,0 +1,58 @@ +# Windows Support Execution Plan + +Source of truth for requirements: `SPEC.md`. +This plan is **live** and tracks what has landed and what remains for the Windows milestone (auto-updater + dictation). + +## Workstream + +### 1) Docs + +- [x] Add `SPEC.md` (Windows scope, updater + dictation required). +- [x] Keep `PLAN.md` current as work lands. + +### 2) Git safety + PR + +- [x] Push `feature/windows-support` to a fork remote and set upstream to avoid accidental `main` pushes. +- [x] Open a draft PR early so CI runs on every push. + +### 3) Windows UX + path correctness + +- [x] Make “Reveal in Finder” platform-aware (Explorer on Windows). +- [x] Fix path joining in the frontend so Windows absolute/relative paths behave. +- [x] Make backend `open_workspace_in` work cross-platform (macOS/Windows/Linux). +- [x] Make default “Open in” targets sensible on Windows (Explorer + command-based editors). +- [x] Make default shortcuts Windows-friendly (Ctrl/Alt; avoid Cmd+Ctrl collapse). + +### 4) Dictation on Windows (required) + +- [x] Enable Whisper dictation on Windows (`whisper-rs` + `cpal`) by removing the Windows stub. +- [x] Update Windows build checks (`doctor:win`) to require LLVM/Clang + CMake. +- [x] Fix `doctor:win` dependency detection on Unix (no shell builtins). + +### 5) CI (required) + +- [x] Add a Windows CI job that runs a Tauri debug build with `src-tauri/tauri.windows.conf.json`. + +### 6) Release + updater (required) + +- [x] Enable Windows updater artifacts in `src-tauri/tauri.windows.conf.json`. +- [x] Add a Windows release build job to `.github/workflows/release.yml`. +- [x] Extend `latest.json` generation to include Windows URL + signature. + +## Validation (run after each step) + +- `npm run lint` +- `npm run test` +- `npm run typecheck` +- Rust checks are executed in CI for macOS + Windows jobs added by this plan. + +## Manual checklist (Windows) + +- [ ] `npm run tauri:build:win` succeeds on Windows 10/11. +- [ ] App launches and can open workspaces. +- [ ] Adding a workspace succeeds when `codex --version` works in Windows Terminal. +- [ ] “Reveal in Explorer” opens the right folder. +- [ ] Shortcut hints use Ctrl/Alt labels and work on Windows. +- [ ] Theme dropdown options are readable in Dark/Dim with Reduce Transparency off. +- [ ] Auto-updater finds and applies the latest release. +- [ ] Dictation works end-to-end (download → hold-to-talk → transcript). diff --git a/README.md b/README.md index 7773e668..4f50829a 100644 --- a/README.md +++ b/README.md @@ -2,7 +2,7 @@ ![CodexMonitor](screenshot.png) -CodexMonitor is a macOS Tauri app for orchestrating multiple Codex agents across local workspaces. It provides a sidebar to manage projects, a home screen for quick actions, and a conversation view backed by the Codex app-server protocol. +CodexMonitor is a Tauri app for orchestrating multiple Codex agents across local workspaces. It provides a sidebar to manage projects, a home screen for quick actions, and a conversation view backed by the Codex app-server protocol. ## Features @@ -31,7 +31,7 @@ CodexMonitor is a macOS Tauri app for orchestrating multiple Codex agents across ### Files & Prompts -- File tree with search, file-type icons, and Reveal in Finder. +- File tree with search, file-type icons, and Reveal in Finder/Explorer. - Prompt library for global/workspace prompts: create/edit/delete/move and run in current or new threads. ### UI & Experience @@ -40,13 +40,14 @@ CodexMonitor is a macOS Tauri app for orchestrating multiple Codex agents across - Responsive layouts (desktop/tablet/phone) with tabbed navigation. - Sidebar usage and credits meter for account rate limits plus a home usage snapshot. - Terminal dock with multiple tabs for background commands (experimental). -- In-app updates with toast-driven download/install, debug panel copy/clear, sound notifications, and macOS overlay title bar with vibrancy + reduced transparency toggle. +- In-app updates with toast-driven download/install, debug panel copy/clear, sound notifications, plus platform-specific window effects (macOS overlay title bar + vibrancy) and a reduced transparency toggle. ## Requirements - Node.js + npm - Rust toolchain (stable) -- CMake (required for native dependencies; Whisper/dictation uses it on non-Windows) +- CMake (required for native dependencies; dictation/Whisper uses it) +- LLVM/Clang (required on Windows to build dictation dependencies via bindgen) - Codex installed on your system and available as `codex` in `PATH` - Git CLI (used for worktree operations) - GitHub CLI (`gh`) for the Issues panel (optional) @@ -74,13 +75,13 @@ npm run tauri dev ## Release Build -Build the production Tauri bundle (app + dmg): +Build the production Tauri bundle: ```bash npm run tauri build ``` -The macOS app bundle will be in `src-tauri/target/release/bundle/macos/`. +Artifacts will be in `src-tauri/target/release/bundle/` (platform-specific subfolders). ### Windows (opt-in) @@ -94,8 +95,8 @@ Artifacts will be in: - `src-tauri/target/release/bundle/nsis/` (installer exe) - `src-tauri/target/release/bundle/msi/` (msi) - -Note: dictation is currently disabled on Windows builds (to avoid requiring LLVM/libclang for `whisper-rs`/bindgen). + +Note: building from source on Windows requires LLVM/Clang (for `bindgen` / `libclang`) in addition to CMake. ## Type Checking diff --git a/SPEC.md b/SPEC.md new file mode 100644 index 00000000..66fdfed1 --- /dev/null +++ b/SPEC.md @@ -0,0 +1,92 @@ +# Windows Support Spec + +## Goal + +Ship a **fully functioning Windows version** of CodexMonitor with: + +- **Auto-updater** enabled and wired into the release workflow (`latest.json` includes Windows). +- **Dictation** working on Windows (same UX + model management as other platforms). + +This spec is the source of truth for Windows support requirements and acceptance criteria. + +## Target Platforms + +- Windows 10/11, **x86_64** (GitHub Actions `windows-latest`). + +## Non-Goals (for this milestone) + +- Authenticode / EV code signing (nice-to-have; not required to be functional). +- Windows ARM64 builds. + +## Functional Requirements + +### Updater (Required) + +- Windows release artifacts must be produced by CI and attached to GitHub Releases. +- `latest.json` must include a **Windows platform entry** with: + - correct asset URL + - signature generated by Tauri updater signing +- In-app updater must be enabled for Windows builds (same endpoints + pubkey as other platforms). +- The updater entry for Windows uses the **NSIS installer** (`.exe` + `.exe.sig`) as the updater bundle. + +### Dictation (Required) + +Dictation must work on Windows with the same surface area as other platforms: + +- Model status: missing/downloading/ready/error +- Model download/cancel/remove +- Session lifecycle: start/listening/stop->processing/transcript/cancel +- Emits the same Tauri events: + - `dictation-download` + - `dictation-event` + +Implementation choice for this milestone: + +- Use the existing Whisper-based implementation on Windows (via `whisper-rs` + `cpal`). + +### Windows UX Correctness (Required) + +- “Reveal in Finder” strings must be platform-aware (“Explorer” on Windows). +- Opening paths in an editor or file manager must behave correctly on Windows. +- Shortcut hints and formatting must be platform-aware on Windows (Ctrl/Alt labels; no macOS-only glyphs). +- Settings selects (for example Theme) must remain readable in Dark/Dim with Reduce Transparency off. + +## Build & Tooling Requirements + +### Windows Build Prereqs (dev + CI) + +Required on Windows to build dictation (Whisper + bindgen): + +- CMake +- LLVM/Clang (for `bindgen` / `libclang`) + +`npm run doctor:win` must fail fast with actionable instructions if missing. +`npm run doctor:win` must correctly detect installed dependencies on Windows/macOS/Linux. + +## CI / Release Requirements + +### CI (Required) + +- Add a Windows job to `.github/workflows/ci.yml` that: + - installs deps + - runs a Windows Tauri debug build (`--no-bundle`) using the Windows config + +### Release (Required) + +Update `.github/workflows/release.yml` to include Windows: + +- Build Windows bundles on `windows-latest` +- Upload `.msi` / `.exe` plus updater `.sig` artifacts +- Include Windows in generated `latest.json` + +## Acceptance Criteria + +- `npm run tauri:build:win` succeeds on Windows. +- Windows release workflow publishes installers and `latest.json` that enables in-app updates. +- Dictation works on Windows end-to-end (model download → hold-to-talk → transcript). +- Adding a workspace works when `codex --version` runs in Windows Terminal (Codex PATH handling is correct). +- Repo checks pass: + - `npm run lint` + - `npm run test` + - `npm run typecheck` + - Rust checks executed in CI for Windows/macOS as configured diff --git a/docs/changelog.html b/docs/changelog.html index 52df64ca..d90c6c33 100644 --- a/docs/changelog.html +++ b/docs/changelog.html @@ -81,7 +81,7 @@

Changelog

Codex Monitor app icon Codex Monitor -

macOS Codex agents orchestration, built by and for individuals who ship fast.

+

Desktop Codex agent orchestration, built by and for individuals who ship fast.

Features diff --git a/docs/index.html b/docs/index.html index 892eda6a..dac37be0 100644 --- a/docs/index.html +++ b/docs/index.html @@ -6,7 +6,7 @@ Codex Monitor - Orchestrate Codex agents across your workspaces
-
macOS command center for Codex
+
Desktop command center for Codex

Monitor your Codex situation.

Orchestrate any number of Codex agents across any number of projects in a beautifully crafted @@ -77,7 +77,7 @@

Monitor your Codex situation.

Platform - macOS + Linux + macOS + Windows + Linux
@@ -129,7 +129,7 @@

Skills + prompts

Updater + polish

-

Toast-driven updates, resizable panels, and a macOS overlay title bar.

+

Toast-driven updates, resizable panels, and platform-specific window chrome.

@@ -412,7 +412,7 @@

Ready to monitor every agent run?

Codex Monitor app icon Codex Monitor -

macOS Codex agents orchestration, built by and for individuals who ship fast.

+

Desktop Codex agent orchestration, built by and for individuals who ship fast.

Features diff --git a/scripts/doctor.mjs b/scripts/doctor.mjs index b247cb4a..2480275e 100644 --- a/scripts/doctor.mjs +++ b/scripts/doctor.mjs @@ -1,16 +1,47 @@ -import { spawnSync } from "node:child_process"; +import fs from "node:fs"; +import path from "node:path"; const strict = process.argv.includes("--strict"); +function canExecute(filePath) { + try { + const stat = fs.statSync(filePath); + if (!stat.isFile()) return false; + if (process.platform === "win32") return true; + fs.accessSync(filePath, fs.constants.X_OK); + return true; + } catch { + return false; + } +} + function hasCommand(command) { - const checker = process.platform === "win32" ? "where" : "command"; - const checkerArgs = process.platform === "win32" ? [command] : ["-v", command]; - const result = spawnSync(checker, checkerArgs, { stdio: "ignore" }); - return result.status === 0; + const pathValue = process.env.PATH; + if (!pathValue) return false; + + const dirs = pathValue.split(path.delimiter).filter(Boolean); + + if (process.platform !== "win32") { + return dirs.some((dir) => canExecute(path.join(dir, command))); + } + + const pathExtValue = process.env.PATHEXT ?? ".EXE;.CMD;.BAT;.COM"; + const exts = pathExtValue.split(";").filter(Boolean); + const hasExtension = path.extname(command) !== ""; + + for (const dir of dirs) { + if (hasExtension && canExecute(path.join(dir, command))) return true; + for (const ext of exts) { + if (canExecute(path.join(dir, `${command}${ext}`))) return true; + } + } + + return false; } const missing = []; if (!hasCommand("cmake")) missing.push("cmake"); +if (process.platform === "win32" && !hasCommand("clang")) missing.push("llvm"); if (missing.length === 0) { console.log("Doctor: OK"); @@ -29,8 +60,9 @@ switch (process.platform) { console.log("Arch: sudo pacman -S cmake"); break; case "win32": - console.log("Install: choco install cmake"); + console.log("Install: choco install cmake llvm"); console.log("Or download from: https://cmake.org/download/"); + console.log("If bindgen fails, set LIBCLANG_PATH to your LLVM bin directory."); break; default: console.log("Install CMake from: https://cmake.org/download/"); @@ -38,4 +70,3 @@ switch (process.platform) { } process.exit(strict ? 1 : 0); - diff --git a/src-tauri/Cargo.toml b/src-tauri/Cargo.toml index 809ce302..9762b153 100644 --- a/src-tauri/Cargo.toml +++ b/src-tauri/Cargo.toml @@ -46,8 +46,6 @@ toml = "0.8" [target."cfg(not(any(target_os = \"android\", target_os = \"ios\")))".dependencies] tauri-plugin-updater = "2" tauri-plugin-window-state = "2" - -[target."cfg(not(target_os = \"windows\"))".dependencies] cpal = "0.15" whisper-rs = "0.12" sha2 = "0.10" diff --git a/src-tauri/src/backend/app_server.rs b/src-tauri/src/backend/app_server.rs index 96801a28..4a8b7564 100644 --- a/src-tauri/src/backend/app_server.rs +++ b/src-tauri/src/backend/app_server.rs @@ -13,10 +13,13 @@ use tokio::sync::{mpsc, oneshot, Mutex}; use tokio::time::timeout; use crate::backend::events::{AppServerEvent, EventSink}; -use crate::shared::process_core::tokio_command; -use crate::codex::args::apply_codex_args; +use crate::shared::process_core::{kill_child_process_tree, tokio_command}; +use crate::codex::args::parse_codex_args; use crate::types::WorkspaceEntry; +#[cfg(target_os = "windows")] +use crate::shared::process_core::{build_cmd_c_command, resolve_windows_executable}; + fn extract_thread_id(value: &Value) -> Option { let params = value.get("params")?; @@ -84,73 +87,143 @@ impl WorkspaceSession { } pub(crate) fn build_codex_path_env(codex_bin: Option<&str>) -> Option { - let mut paths: Vec = env::var("PATH") - .unwrap_or_default() - .split(':') - .filter(|value| !value.is_empty()) - .map(|value| value.to_string()) - .collect(); - let mut extras = vec![ - "/opt/homebrew/bin", - "/usr/local/bin", - "/usr/bin", - "/bin", - "/usr/sbin", - "/sbin", - ] - .into_iter() - .map(|value| value.to_string()) - .collect::>(); - if let Ok(home) = env::var("HOME") { - extras.push(format!("{home}/.local/bin")); - extras.push(format!("{home}/.local/share/mise/shims")); - extras.push(format!("{home}/.cargo/bin")); - extras.push(format!("{home}/.bun/bin")); - let nvm_root = Path::new(&home).join(".nvm/versions/node"); - if let Ok(entries) = std::fs::read_dir(nvm_root) { - for entry in entries.flatten() { - let bin_path = entry.path().join("bin"); - if bin_path.is_dir() { - extras.push(bin_path.to_string_lossy().to_string()); + let mut paths: Vec = env::var_os("PATH") + .map(|value| env::split_paths(&value).collect()) + .unwrap_or_default(); + + let mut extras: Vec = Vec::new(); + + #[cfg(not(target_os = "windows"))] + { + extras.extend([ + "/opt/homebrew/bin", + "/usr/local/bin", + "/usr/bin", + "/bin", + "/usr/sbin", + "/sbin", + ].into_iter().map(PathBuf::from)); + + if let Ok(home) = env::var("HOME") { + let home_path = Path::new(&home); + extras.push(home_path.join(".local/bin")); + extras.push(home_path.join(".local/share/mise/shims")); + extras.push(home_path.join(".cargo/bin")); + extras.push(home_path.join(".bun/bin")); + let nvm_root = home_path.join(".nvm/versions/node"); + if let Ok(entries) = std::fs::read_dir(nvm_root) { + for entry in entries.flatten() { + let bin_path = entry.path().join("bin"); + if bin_path.is_dir() { + extras.push(bin_path); + } } } } } + + #[cfg(target_os = "windows")] + { + if let Ok(appdata) = env::var("APPDATA") { + extras.push(Path::new(&appdata).join("npm")); + } + if let Ok(local_app_data) = env::var("LOCALAPPDATA") { + extras.push( + Path::new(&local_app_data) + .join("Microsoft") + .join("WindowsApps"), + ); + } + if let Ok(home) = env::var("USERPROFILE").or_else(|_| env::var("HOME")) { + let home_path = Path::new(&home); + extras.push(home_path.join(".cargo").join("bin")); + extras.push(home_path.join("scoop").join("shims")); + } + if let Ok(program_data) = env::var("PROGRAMDATA") { + extras.push(Path::new(&program_data).join("chocolatey").join("bin")); + } + } + if let Some(bin_path) = codex_bin.filter(|value| !value.trim().is_empty()) { - let parent = Path::new(bin_path).parent(); - if let Some(parent) = parent { - extras.push(parent.to_string_lossy().to_string()); + if let Some(parent) = Path::new(bin_path).parent() { + extras.push(parent.to_path_buf()); } } + for extra in extras { - if !paths.contains(&extra) { + if !paths.iter().any(|path| path == &extra) { paths.push(extra); } } + if paths.is_empty() { - None - } else { - Some(paths.join(":")) + return None; } + + env::join_paths(paths) + .ok() + .map(|joined| joined.to_string_lossy().to_string()) } -pub(crate) fn build_codex_command_with_bin(codex_bin: Option) -> Command { +pub(crate) fn build_codex_command_with_bin( + codex_bin: Option, + codex_args: Option<&str>, + args: Vec, +) -> Result { let bin = codex_bin .clone() .filter(|value| !value.trim().is_empty()) .unwrap_or_else(|| "codex".into()); - let mut command = tokio_command(bin); - if let Some(path_env) = build_codex_path_env(codex_bin.as_deref()) { + + let path_env = build_codex_path_env(codex_bin.as_deref()); + let mut command_args = parse_codex_args(codex_args)?; + command_args.extend(args); + + #[cfg(target_os = "windows")] + let mut command = { + let bin_trimmed = bin.trim(); + let resolved = resolve_windows_executable(bin_trimmed, path_env.as_deref()); + let resolved_path = resolved + .as_deref() + .unwrap_or_else(|| Path::new(bin_trimmed)); + let ext = resolved_path + .extension() + .and_then(|ext| ext.to_str()) + .map(|ext| ext.to_ascii_lowercase()); + + if matches!(ext.as_deref(), Some("cmd") | Some("bat")) { + let mut command = tokio_command("cmd"); + let command_line = build_cmd_c_command(resolved_path, &command_args)?; + command.arg("/D"); + command.arg("/S"); + command.arg("/C"); + command.arg(command_line); + command + } else { + let mut command = tokio_command(resolved_path); + command.args(command_args); + command + } + }; + + #[cfg(not(target_os = "windows"))] + let mut command = { + let mut command = tokio_command(bin.trim()); + command.args(command_args); + command + }; + + if let Some(path_env) = path_env { command.env("PATH", path_env); } - command + Ok(command) } pub(crate) async fn check_codex_installation( codex_bin: Option, ) -> Result, String> { - let mut command = build_codex_command_with_bin(codex_bin); - command.arg("--version"); + let mut command = + build_codex_command_with_bin(codex_bin, None, vec!["--version".to_string()])?; command.stdout(std::process::Stdio::piped()); command.stderr(std::process::Stdio::piped()); @@ -209,10 +282,12 @@ pub(crate) async fn spawn_workspace_session( .or(default_codex_bin); let _ = check_codex_installation(codex_bin.clone()).await?; - let mut command = build_codex_command_with_bin(codex_bin); - apply_codex_args(&mut command, codex_args.as_deref())?; + let mut command = build_codex_command_with_bin( + codex_bin, + codex_args.as_deref(), + vec!["app-server".to_string()], + )?; command.current_dir(&entry.path); - command.arg("app-server"); if let Some(codex_home) = codex_home { command.env("CODEX_HOME", codex_home); } @@ -348,7 +423,7 @@ pub(crate) async fn spawn_workspace_session( Ok(response) => response, Err(_) => { let mut child = session.child.lock().await; - let _ = child.kill().await; + kill_child_process_tree(&mut child).await; return Err( "Codex app-server did not respond to initialize. Check that `codex app-server` works in Terminal." .to_string(), diff --git a/src-tauri/src/codex/mod.rs b/src-tauri/src/codex/mod.rs index d417dbe2..39c289df 100644 --- a/src-tauri/src/codex/mod.rs +++ b/src-tauri/src/codex/mod.rs @@ -24,7 +24,6 @@ use crate::remote_backend; use crate::shared::codex_core; use crate::state::AppState; use crate::types::WorkspaceEntry; -use self::args::apply_codex_args; pub(crate) async fn spawn_workspace_session( entry: WorkspaceEntry, @@ -66,10 +65,11 @@ pub(crate) async fn codex_doctor( .or(default_args); let path_env = build_codex_path_env(resolved.as_deref()); let version = check_codex_installation(resolved.clone()).await?; - let mut command = build_codex_command_with_bin(resolved.clone()); - apply_codex_args(&mut command, resolved_args.as_deref())?; - command.arg("app-server"); - command.arg("--help"); + let mut command = build_codex_command_with_bin( + resolved.clone(), + resolved_args.as_deref(), + vec!["app-server".to_string(), "--help".to_string()], + )?; command.stdout(std::process::Stdio::piped()); command.stderr(std::process::Stdio::piped()); let app_server_ok = match timeout(Duration::from_secs(5), command.output()).await { diff --git a/src-tauri/src/dictation/mod.rs b/src-tauri/src/dictation/mod.rs index 8fe265c3..fe9b8f97 100644 --- a/src-tauri/src/dictation/mod.rs +++ b/src-tauri/src/dictation/mod.rs @@ -1,9 +1,4 @@ -#[cfg(not(target_os = "windows"))] #[path = "real.rs"] mod imp; -#[cfg(target_os = "windows")] -#[path = "stub.rs"] -mod imp; - pub(crate) use imp::*; diff --git a/src-tauri/src/dictation/stub.rs b/src-tauri/src/dictation/stub.rs deleted file mode 100644 index 5cb8a14e..00000000 --- a/src-tauri/src/dictation/stub.rs +++ /dev/null @@ -1,195 +0,0 @@ -use serde::Serialize; -use tauri::{AppHandle, Emitter, State}; - -use crate::state::AppState; - -const DEFAULT_MODEL_ID: &str = "base"; -const UNSUPPORTED_MESSAGE: &str = "Dictation is not supported on Windows builds."; - -#[derive(Debug, Serialize, Clone, Copy, PartialEq, Eq)] -#[serde(rename_all = "lowercase")] -pub(crate) enum DictationModelState { - Missing, - Downloading, - Ready, - Error, -} - -#[derive(Debug, Serialize, Clone)] -pub(crate) struct DictationDownloadProgress { - #[serde(rename = "downloadedBytes")] - pub(crate) downloaded_bytes: u64, - #[serde(rename = "totalBytes")] - pub(crate) total_bytes: Option, -} - -#[derive(Debug, Serialize, Clone)] -pub(crate) struct DictationModelStatus { - pub(crate) state: DictationModelState, - #[serde(rename = "modelId")] - pub(crate) model_id: String, - pub(crate) progress: Option, - pub(crate) error: Option, - pub(crate) path: Option, -} - -#[derive(Debug, Serialize, Clone, Copy, PartialEq, Eq)] -#[serde(rename_all = "lowercase")] -pub(crate) enum DictationSessionState { - Idle, - Listening, - Processing, -} - -#[derive(Debug, Serialize, Clone)] -#[serde(tag = "type", rename_all = "snake_case")] -pub(crate) enum DictationEvent { - State { state: DictationSessionState }, - Level { value: f32 }, - Transcript { text: String }, - Error { message: String }, - Canceled { message: String }, -} - -pub(crate) struct DictationState { - pub(crate) model_status: DictationModelStatus, - pub(crate) session_state: DictationSessionState, -} - -impl Default for DictationState { - fn default() -> Self { - Self { - model_status: DictationModelStatus { - state: DictationModelState::Error, - model_id: DEFAULT_MODEL_ID.to_string(), - progress: None, - error: Some(UNSUPPORTED_MESSAGE.to_string()), - path: None, - }, - session_state: DictationSessionState::Idle, - } - } -} - -fn emit_status(app: &AppHandle, status: &DictationModelStatus) { - let _ = app.emit("dictation-download", status); -} - -fn emit_event(app: &AppHandle, event: DictationEvent) { - let _ = app.emit("dictation-event", event); -} - -fn windows_unsupported_status(model_id: Option) -> DictationModelStatus { - DictationModelStatus { - state: DictationModelState::Error, - model_id: model_id.unwrap_or_else(|| DEFAULT_MODEL_ID.to_string()), - progress: None, - error: Some(UNSUPPORTED_MESSAGE.to_string()), - path: None, - } -} - -#[tauri::command] -pub(crate) async fn dictation_model_status( - app: AppHandle, - state: State<'_, AppState>, - model_id: Option, -) -> Result { - let status = windows_unsupported_status(model_id); - { - let mut dictation = state.dictation.lock().await; - dictation.model_status = status.clone(); - dictation.session_state = DictationSessionState::Idle; - } - emit_status(&app, &status); - Ok(status) -} - -#[tauri::command] -pub(crate) async fn dictation_download_model( - app: AppHandle, - state: State<'_, AppState>, - model_id: Option, -) -> Result { - let status = dictation_model_status(app.clone(), state, model_id).await?; - emit_event( - &app, - DictationEvent::Error { - message: status - .error - .clone() - .unwrap_or_else(|| "Dictation is unavailable on Windows.".to_string()), - }, - ); - Ok(status) -} - -#[tauri::command] -pub(crate) async fn dictation_cancel_download( - app: AppHandle, - state: State<'_, AppState>, - model_id: Option, -) -> Result { - dictation_model_status(app, state, model_id).await -} - -#[tauri::command] -pub(crate) async fn dictation_remove_model( - app: AppHandle, - state: State<'_, AppState>, - model_id: Option, -) -> Result { - dictation_model_status(app, state, model_id).await -} - -#[tauri::command] -pub(crate) async fn dictation_start( - _preferred_language: Option, - app: AppHandle, - state: State<'_, AppState>, -) -> Result { - { - let mut dictation = state.dictation.lock().await; - dictation.session_state = DictationSessionState::Idle; - } - let message = UNSUPPORTED_MESSAGE.to_string(); - emit_event(&app, DictationEvent::Error { message: message.clone() }); - Err(message) -} - -#[tauri::command] -pub(crate) async fn dictation_request_permission(_app: AppHandle) -> Result { - Err(UNSUPPORTED_MESSAGE.to_string()) -} - -#[tauri::command] -pub(crate) async fn dictation_stop( - app: AppHandle, - state: State<'_, AppState>, -) -> Result { - { - let mut dictation = state.dictation.lock().await; - dictation.session_state = DictationSessionState::Idle; - } - let message = UNSUPPORTED_MESSAGE.to_string(); - emit_event(&app, DictationEvent::Error { message: message.clone() }); - Err(message) -} - -#[tauri::command] -pub(crate) async fn dictation_cancel( - app: AppHandle, - state: State<'_, AppState>, -) -> Result { - { - let mut dictation = state.dictation.lock().await; - dictation.session_state = DictationSessionState::Idle; - } - emit_event( - &app, - DictationEvent::Canceled { - message: "Canceled".to_string(), - }, - ); - Ok(DictationSessionState::Idle) -} diff --git a/src-tauri/src/shared/process_core.rs b/src-tauri/src/shared/process_core.rs index d9eb90eb..8fd9a26b 100644 --- a/src-tauri/src/shared/process_core.rs +++ b/src-tauri/src/shared/process_core.rs @@ -1,6 +1,11 @@ +#[cfg(target_os = "windows")] +use std::env; use std::ffi::OsStr; +#[cfg(target_os = "windows")] +use std::path::{Path, PathBuf}; +use std::process::Stdio; -use tokio::process::Command; +use tokio::process::{Child, Command}; /// On Windows, spawning a console app from a GUI subsystem app will open a new /// console window unless we explicitly disable it. @@ -17,4 +22,129 @@ pub(crate) fn tokio_command(program: impl AsRef) -> Command { let mut command = Command::new(program); hide_console_on_windows(command.as_std_mut()); command -} \ No newline at end of file +} + +pub(crate) async fn kill_child_process_tree(child: &mut Child) { + #[cfg(windows)] + { + if let Some(pid) = child.id() { + let _ = tokio_command("taskkill") + .arg("/PID") + .arg(pid.to_string()) + .arg("/T") + .arg("/F") + .stdout(Stdio::null()) + .stderr(Stdio::null()) + .status() + .await; + } + } + + let _ = child.kill().await; +} + +#[cfg(target_os = "windows")] +pub(crate) fn resolve_windows_executable(program: &str, path_env: Option<&str>) -> Option { + let trimmed = program.trim(); + if trimmed.is_empty() { + return None; + } + + let program = trimmed + .strip_prefix('"') + .and_then(|value| value.strip_suffix('"')) + .unwrap_or(trimmed) + .trim(); + + if program.is_empty() { + return None; + } + + let has_separators = program.contains('\\') || program.contains('/'); + let has_drive = matches!(program.as_bytes().get(1), Some(b':')); + let looks_like_path = has_separators || has_drive; + + let path_candidates = if Path::new(program).extension().is_some() { + vec![program.to_string()] + } else { + vec![ + format!("{program}.exe"), + format!("{program}.cmd"), + format!("{program}.bat"), + format!("{program}.com"), + ] + }; + + if looks_like_path { + for candidate in path_candidates { + let path = PathBuf::from(candidate); + if path.is_file() { + return Some(path); + } + } + return None; + } + + let paths: Vec = if let Some(value) = path_env { + let trimmed = value.trim(); + if trimmed.is_empty() { + Vec::new() + } else { + env::split_paths(trimmed).collect() + } + } else { + env::var_os("PATH") + .map(|value| env::split_paths(&value).collect()) + .unwrap_or_default() + }; + + for root in paths { + for candidate in &path_candidates { + let path = root.join(candidate); + if path.is_file() { + return Some(path); + } + } + } + + None +} + +#[cfg(target_os = "windows")] +fn validate_cmd_token(value: &str) -> Result<(), String> { + if value.contains('\0') { + return Err("Windows cmd wrapper does not support NUL bytes.".to_string()); + } + if value.contains('\n') || value.contains('\r') { + return Err("Windows cmd wrapper does not support newline characters.".to_string()); + } + if value.contains('"') { + return Err( + "Windows cmd wrapper does not support double quotes in arguments.".to_string(), + ); + } + Ok(()) +} + +#[cfg(target_os = "windows")] +fn quote_cmd_token(value: &str) -> Result { + validate_cmd_token(value)?; + Ok(format!("\"{value}\"")) +} + +/// Builds a single `cmd.exe /C ""` payload that safely treats each argument as data +/// (protects cmd metacharacters like `&`, `|`, `>`, `<`, `^`) by always quoting tokens. +/// +/// Returns a string that already includes the required outer quotes, suitable to be passed as +/// *one* argument after `/C` (usually with `/S`). +#[cfg(target_os = "windows")] +pub(crate) fn build_cmd_c_command(program: &Path, args: &[String]) -> Result { + let program_str = program.to_string_lossy(); + let mut parts: Vec = Vec::with_capacity(args.len() + 1); + parts.push(quote_cmd_token(program_str.as_ref())?); + for arg in args { + parts.push(quote_cmd_token(arg)?); + } + let inner = parts.join(" "); + Ok(format!("\"{inner}\"")) +} diff --git a/src-tauri/src/shared/workspaces_core.rs b/src-tauri/src/shared/workspaces_core.rs index 76bd1bf5..5c521b45 100644 --- a/src-tauri/src/shared/workspaces_core.rs +++ b/src-tauri/src/shared/workspaces_core.rs @@ -8,6 +8,7 @@ use tokio::sync::Mutex; use crate::backend::app_server::WorkspaceSession; use crate::codex::args::resolve_workspace_codex_args; use crate::codex::home::resolve_workspace_codex_home; +use crate::shared::process_core::kill_child_process_tree; use crate::storage::write_workspaces; use crate::types::{ AppSettings, WorkspaceEntry, WorkspaceInfo, WorkspaceKind, WorkspaceSettings, WorktreeInfo, @@ -196,7 +197,7 @@ where workspaces.remove(&entry.id); } let mut child = session.child.lock().await; - let _ = child.kill().await; + kill_child_process_tree(&mut child).await; return Err(error); } @@ -412,7 +413,7 @@ async fn kill_session_by_id( ) { if let Some(session) = sessions.lock().await.remove(id) { let mut child = session.child.lock().await; - let _ = child.kill().await; + kill_child_process_tree(&mut child).await; } } @@ -960,7 +961,7 @@ where .insert(entry_snapshot.id.clone(), new_session) { let mut child = old_session.child.lock().await; - let _ = child.kill().await; + kill_child_process_tree(&mut child).await; } } if codex_home_changed || codex_args_changed { @@ -1003,7 +1004,7 @@ where .insert(child.id.clone(), new_session) { let mut child = old_session.child.lock().await; - let _ = child.kill().await; + kill_child_process_tree(&mut child).await; } } } diff --git a/src-tauri/src/types.rs b/src-tauri/src/types.rs index 3fee17d0..3ef5e219 100644 --- a/src-tauri/src/types.rs +++ b/src-tauri/src/types.rs @@ -537,12 +537,11 @@ fn default_usage_show_remaining() -> bool { } fn default_ui_font_family() -> String { - "\"SF Pro Text\", \"SF Pro Display\", -apple-system, \"Helvetica Neue\", sans-serif" - .to_string() + "system-ui, -apple-system, BlinkMacSystemFont, \"Segoe UI\", Roboto, \"Helvetica Neue\", Arial, sans-serif".to_string() } fn default_code_font_family() -> String { - "\"SF Mono\", \"SFMono-Regular\", Menlo, Monaco, monospace".to_string() + "ui-monospace, \"Cascadia Mono\", \"Segoe UI Mono\", Menlo, Monaco, Consolas, \"Liberation Mono\", \"Courier New\", monospace".to_string() } fn default_code_font_size() -> u8 { @@ -550,15 +549,30 @@ fn default_code_font_size() -> u8 { } fn default_composer_model_shortcut() -> Option { - Some("cmd+shift+m".to_string()) + let value = if cfg!(target_os = "macos") { + "cmd+shift+m" + } else { + "ctrl+shift+m" + }; + Some(value.to_string()) } fn default_composer_access_shortcut() -> Option { - Some("cmd+shift+a".to_string()) + let value = if cfg!(target_os = "macos") { + "cmd+shift+a" + } else { + "ctrl+shift+a" + }; + Some(value.to_string()) } fn default_composer_reasoning_shortcut() -> Option { - Some("cmd+shift+r".to_string()) + let value = if cfg!(target_os = "macos") { + "cmd+shift+r" + } else { + "ctrl+shift+r" + }; + Some(value.to_string()) } fn default_interrupt_shortcut() -> Option { @@ -575,51 +589,107 @@ fn default_composer_collaboration_shortcut() -> Option { } fn default_new_agent_shortcut() -> Option { - Some("cmd+n".to_string()) + let value = if cfg!(target_os = "macos") { "cmd+n" } else { "ctrl+n" }; + Some(value.to_string()) } fn default_new_worktree_agent_shortcut() -> Option { - Some("cmd+shift+n".to_string()) + let value = if cfg!(target_os = "macos") { + "cmd+shift+n" + } else { + "ctrl+shift+n" + }; + Some(value.to_string()) } fn default_new_clone_agent_shortcut() -> Option { - Some("cmd+alt+n".to_string()) + let value = if cfg!(target_os = "macos") { + "cmd+alt+n" + } else { + "ctrl+alt+n" + }; + Some(value.to_string()) } fn default_archive_thread_shortcut() -> Option { - Some("cmd+ctrl+a".to_string()) + let value = if cfg!(target_os = "macos") { + "cmd+ctrl+a" + } else { + "ctrl+alt+a" + }; + Some(value.to_string()) } fn default_toggle_projects_sidebar_shortcut() -> Option { - Some("cmd+shift+p".to_string()) + let value = if cfg!(target_os = "macos") { + "cmd+shift+p" + } else { + "ctrl+shift+p" + }; + Some(value.to_string()) } fn default_toggle_git_sidebar_shortcut() -> Option { - Some("cmd+shift+g".to_string()) + let value = if cfg!(target_os = "macos") { + "cmd+shift+g" + } else { + "ctrl+shift+g" + }; + Some(value.to_string()) } fn default_toggle_debug_panel_shortcut() -> Option { - Some("cmd+shift+d".to_string()) + let value = if cfg!(target_os = "macos") { + "cmd+shift+d" + } else { + "ctrl+shift+d" + }; + Some(value.to_string()) } fn default_toggle_terminal_shortcut() -> Option { - Some("cmd+shift+t".to_string()) + let value = if cfg!(target_os = "macos") { + "cmd+shift+t" + } else { + "ctrl+shift+t" + }; + Some(value.to_string()) } fn default_cycle_agent_next_shortcut() -> Option { - Some("cmd+ctrl+down".to_string()) + let value = if cfg!(target_os = "macos") { + "cmd+ctrl+down" + } else { + "ctrl+alt+down" + }; + Some(value.to_string()) } fn default_cycle_agent_prev_shortcut() -> Option { - Some("cmd+ctrl+up".to_string()) + let value = if cfg!(target_os = "macos") { + "cmd+ctrl+up" + } else { + "ctrl+alt+up" + }; + Some(value.to_string()) } fn default_cycle_workspace_next_shortcut() -> Option { - Some("cmd+shift+down".to_string()) + let value = if cfg!(target_os = "macos") { + "cmd+shift+down" + } else { + "ctrl+alt+shift+down" + }; + Some(value.to_string()) } fn default_cycle_workspace_prev_shortcut() -> Option { - Some("cmd+shift+up".to_string()) + let value = if cfg!(target_os = "macos") { + "cmd+shift+up" + } else { + "ctrl+alt+shift+up" + }; + Some(value.to_string()) } fn default_notification_sounds_enabled() -> bool { @@ -711,50 +781,109 @@ fn default_workspace_groups() -> Vec { } fn default_open_app_targets() -> Vec { + if cfg!(target_os = "macos") { + return vec![ + OpenAppTarget { + id: "vscode".to_string(), + label: "VS Code".to_string(), + kind: "app".to_string(), + app_name: Some("Visual Studio Code".to_string()), + command: None, + args: Vec::new(), + }, + OpenAppTarget { + id: "cursor".to_string(), + label: "Cursor".to_string(), + kind: "app".to_string(), + app_name: Some("Cursor".to_string()), + command: None, + args: Vec::new(), + }, + OpenAppTarget { + id: "zed".to_string(), + label: "Zed".to_string(), + kind: "app".to_string(), + app_name: Some("Zed".to_string()), + command: None, + args: Vec::new(), + }, + OpenAppTarget { + id: "ghostty".to_string(), + label: "Ghostty".to_string(), + kind: "app".to_string(), + app_name: Some("Ghostty".to_string()), + command: None, + args: Vec::new(), + }, + OpenAppTarget { + id: "antigravity".to_string(), + label: "Antigravity".to_string(), + kind: "app".to_string(), + app_name: Some("Antigravity".to_string()), + command: None, + args: Vec::new(), + }, + OpenAppTarget { + id: "finder".to_string(), + label: "Finder".to_string(), + kind: "finder".to_string(), + app_name: None, + command: None, + args: Vec::new(), + }, + ]; + } + + let file_manager_label = if cfg!(windows) { + "Explorer" + } else { + "File Manager" + }; + vec![ OpenAppTarget { id: "vscode".to_string(), label: "VS Code".to_string(), - kind: "app".to_string(), - app_name: Some("Visual Studio Code".to_string()), - command: None, + kind: "command".to_string(), + app_name: None, + command: Some("code".to_string()), args: Vec::new(), }, OpenAppTarget { id: "cursor".to_string(), label: "Cursor".to_string(), - kind: "app".to_string(), - app_name: Some("Cursor".to_string()), - command: None, + kind: "command".to_string(), + app_name: None, + command: Some("cursor".to_string()), args: Vec::new(), }, OpenAppTarget { id: "zed".to_string(), label: "Zed".to_string(), - kind: "app".to_string(), - app_name: Some("Zed".to_string()), - command: None, + kind: "command".to_string(), + app_name: None, + command: Some("zed".to_string()), args: Vec::new(), }, OpenAppTarget { id: "ghostty".to_string(), label: "Ghostty".to_string(), - kind: "app".to_string(), - app_name: Some("Ghostty".to_string()), - command: None, + kind: "command".to_string(), + app_name: None, + command: Some("ghostty".to_string()), args: Vec::new(), }, OpenAppTarget { id: "antigravity".to_string(), label: "Antigravity".to_string(), - kind: "app".to_string(), - app_name: Some("Antigravity".to_string()), - command: None, + kind: "command".to_string(), + app_name: None, + command: Some("antigravity".to_string()), args: Vec::new(), }, OpenAppTarget { id: "finder".to_string(), - label: "Finder".to_string(), + label: file_manager_label.to_string(), kind: "finder".to_string(), app_name: None, command: None, @@ -764,7 +893,11 @@ fn default_open_app_targets() -> Vec { } fn default_selected_open_app_id() -> String { - "vscode".to_string() + if cfg!(windows) { + "finder".to_string() + } else { + "vscode".to_string() + } } impl Default for AppSettings { @@ -846,17 +979,23 @@ mod tests { assert!(settings.remote_backend_token.is_none()); assert_eq!(settings.default_access_mode, "current"); assert_eq!(settings.review_delivery_mode, "inline"); + let expected_primary = if cfg!(target_os = "macos") { "cmd" } else { "ctrl" }; + let expected_model = format!("{expected_primary}+shift+m"); + let expected_access = format!("{expected_primary}+shift+a"); + let expected_reasoning = format!("{expected_primary}+shift+r"); + let expected_toggle_debug = format!("{expected_primary}+shift+d"); + let expected_toggle_terminal = format!("{expected_primary}+shift+t"); assert_eq!( settings.composer_model_shortcut.as_deref(), - Some("cmd+shift+m") + Some(expected_model.as_str()) ); assert_eq!( settings.composer_access_shortcut.as_deref(), - Some("cmd+shift+a") + Some(expected_access.as_str()) ); assert_eq!( settings.composer_reasoning_shortcut.as_deref(), - Some("cmd+shift+r") + Some(expected_reasoning.as_str()) ); assert_eq!( settings.composer_collaboration_shortcut.as_deref(), @@ -870,39 +1009,59 @@ mod tests { assert_eq!(settings.interrupt_shortcut.as_deref(), Some(expected_interrupt)); assert_eq!( settings.archive_thread_shortcut.as_deref(), - Some("cmd+ctrl+a") + Some(if cfg!(target_os = "macos") { + "cmd+ctrl+a" + } else { + "ctrl+alt+a" + }) ); assert_eq!( settings.toggle_debug_panel_shortcut.as_deref(), - Some("cmd+shift+d") + Some(expected_toggle_debug.as_str()) ); assert_eq!( settings.toggle_terminal_shortcut.as_deref(), - Some("cmd+shift+t") + Some(expected_toggle_terminal.as_str()) ); assert_eq!( settings.cycle_agent_next_shortcut.as_deref(), - Some("cmd+ctrl+down") + Some(if cfg!(target_os = "macos") { + "cmd+ctrl+down" + } else { + "ctrl+alt+down" + }) ); assert_eq!( settings.cycle_agent_prev_shortcut.as_deref(), - Some("cmd+ctrl+up") + Some(if cfg!(target_os = "macos") { + "cmd+ctrl+up" + } else { + "ctrl+alt+up" + }) ); assert_eq!( settings.cycle_workspace_next_shortcut.as_deref(), - Some("cmd+shift+down") + Some(if cfg!(target_os = "macos") { + "cmd+shift+down" + } else { + "ctrl+alt+shift+down" + }) ); assert_eq!( settings.cycle_workspace_prev_shortcut.as_deref(), - Some("cmd+shift+up") + Some(if cfg!(target_os = "macos") { + "cmd+shift+up" + } else { + "ctrl+alt+shift+up" + }) ); assert!(settings.last_composer_model_id.is_none()); assert!(settings.last_composer_reasoning_effort.is_none()); assert!((settings.ui_scale - 1.0).abs() < f64::EPSILON); assert_eq!(settings.theme, "system"); assert!(!settings.usage_show_remaining); - assert!(settings.ui_font_family.contains("SF Pro Text")); - assert!(settings.code_font_family.contains("SF Mono")); + assert!(settings.ui_font_family.contains("system-ui")); + assert!(settings.code_font_family.contains("ui-monospace")); assert_eq!(settings.code_font_size, 11); assert!(settings.notification_sounds_enabled); assert!(settings.system_notifications_enabled); @@ -924,7 +1083,8 @@ mod tests { assert!(!settings.composer_list_continuation); assert!(!settings.composer_code_block_copy_use_modifier); assert!(settings.workspace_groups.is_empty()); - assert_eq!(settings.selected_open_app_id, "vscode"); + let expected_open_id = if cfg!(windows) { "finder" } else { "vscode" }; + assert_eq!(settings.selected_open_app_id, expected_open_id); assert_eq!(settings.open_app_targets.len(), 6); assert_eq!(settings.open_app_targets[0].id, "vscode"); } diff --git a/src-tauri/src/workspaces/commands.rs b/src-tauri/src/workspaces/commands.rs index f7e7f217..e4ce435d 100644 --- a/src-tauri/src/workspaces/commands.rs +++ b/src-tauri/src/workspaces/commands.rs @@ -1,4 +1,7 @@ use std::path::PathBuf; + +#[cfg(target_os = "windows")] +use std::path::Path; use std::process::Stdio; use std::sync::Arc; @@ -27,7 +30,9 @@ use crate::codex::args::resolve_workspace_codex_args; use crate::codex::home::resolve_workspace_codex_home; use crate::git_utils::resolve_git_root; use crate::remote_backend; -use crate::shared::process_core::tokio_command; +use crate::shared::process_core::{kill_child_process_tree, tokio_command}; +#[cfg(target_os = "windows")] +use crate::shared::process_core::{build_cmd_c_command, resolve_windows_executable}; use crate::shared::workspaces_core; use crate::state::AppState; use crate::storage::write_workspaces; @@ -255,7 +260,7 @@ pub(crate) async fn add_clone( workspaces.remove(&entry.id); } let mut child = session.child.lock().await; - let _ = child.kill().await; + kill_child_process_tree(&mut child).await; let _ = tokio::fs::remove_dir_all(&destination_path).await; return Err(error); } @@ -816,17 +821,74 @@ pub(crate) async fn open_workspace_in( .unwrap_or_else(|| "target".to_string()); let status = if let Some(command) = command { - let mut cmd = std::process::Command::new(command); - cmd.args(args).arg(path); + let trimmed = command.trim(); + if trimmed.is_empty() { + return Err("Missing app or command".to_string()); + } + + #[cfg(target_os = "windows")] + let mut cmd = { + let resolved = resolve_windows_executable(trimmed, None); + let resolved_path = resolved + .as_deref() + .unwrap_or_else(|| Path::new(trimmed)); + let ext = resolved_path + .extension() + .and_then(|ext| ext.to_str()) + .map(|ext| ext.to_ascii_lowercase()); + + if matches!(ext.as_deref(), Some("cmd") | Some("bat")) || ext.is_none() { + let mut cmd = tokio_command("cmd"); + let mut command_args = args.clone(); + command_args.push(path.clone()); + let command_line = build_cmd_c_command(resolved_path, &command_args)?; + cmd.arg("/D"); + cmd.arg("/S"); + cmd.arg("/C"); + cmd.arg(command_line); + cmd + } else { + let mut cmd = tokio_command(resolved_path); + cmd.args(&args).arg(&path); + cmd + } + }; + + #[cfg(not(target_os = "windows"))] + let mut cmd = { + let mut cmd = tokio_command(trimmed); + cmd.args(&args).arg(&path); + cmd + }; + cmd.status() + .await .map_err(|error| format!("Failed to open app ({target_label}): {error}"))? } else if let Some(app) = app { - let mut cmd = std::process::Command::new("open"); - cmd.arg("-a").arg(app).arg(path); - if !args.is_empty() { - cmd.arg("--args").args(args); + let trimmed = app.trim(); + if trimmed.is_empty() { + return Err("Missing app or command".to_string()); } + + #[cfg(target_os = "macos")] + let mut cmd = { + let mut cmd = tokio_command("open"); + cmd.arg("-a").arg(trimmed).arg(&path); + if !args.is_empty() { + cmd.arg("--args").args(&args); + } + cmd + }; + + #[cfg(not(target_os = "macos"))] + let mut cmd = { + let mut cmd = tokio_command(trimmed); + cmd.args(&args).arg(&path); + cmd + }; + cmd.status() + .await .map_err(|error| format!("Failed to open app ({target_label}): {error}"))? } else { return Err("Missing app or command".to_string()); diff --git a/src-tauri/tauri.windows.conf.json b/src-tauri/tauri.windows.conf.json index 12d4499d..6884d2fa 100644 --- a/src-tauri/tauri.windows.conf.json +++ b/src-tauri/tauri.windows.conf.json @@ -17,6 +17,6 @@ ] }, "bundle": { - "createUpdaterArtifacts": false + "createUpdaterArtifacts": true } } diff --git a/src/features/app/components/MainHeader.tsx b/src/features/app/components/MainHeader.tsx index 867f4086..19bb0eb2 100644 --- a/src/features/app/components/MainHeader.tsx +++ b/src/features/app/components/MainHeader.tsx @@ -5,6 +5,7 @@ import Terminal from "lucide-react/dist/esm/icons/terminal"; import { revealItemInDir } from "@tauri-apps/plugin-opener"; import type { BranchInfo, OpenAppTarget, WorkspaceInfo } from "../../../types"; import type { ReactNode } from "react"; +import { revealInFileManagerLabel } from "../../../utils/platformPaths"; import { OpenAppMenu } from "./OpenAppMenu"; import { LaunchScriptButton } from "./LaunchScriptButton"; import { LaunchScriptEntryButton } from "./LaunchScriptEntryButton"; @@ -372,7 +373,7 @@ export function MainHeader({ }} data-tauri-drag-region="false" > - Reveal in Finder + {revealInFileManagerLabel()}
diff --git a/src/features/app/components/OpenAppMenu.tsx b/src/features/app/components/OpenAppMenu.tsx index 06d54f02..51fed084 100644 --- a/src/features/app/components/OpenAppMenu.tsx +++ b/src/features/app/components/OpenAppMenu.tsx @@ -62,9 +62,14 @@ export function OpenAppMenu({ const fallbackTarget: OpenTarget = { id: DEFAULT_OPEN_APP_ID, - label: DEFAULT_OPEN_APP_TARGETS[0]?.label ?? "Open", + label: + DEFAULT_OPEN_APP_TARGETS.find((target) => target.id === DEFAULT_OPEN_APP_ID) + ?.label ?? + DEFAULT_OPEN_APP_TARGETS[0]?.label ?? + "Open", icon: getKnownOpenAppIcon(DEFAULT_OPEN_APP_ID) ?? GENERIC_APP_ICON, target: + DEFAULT_OPEN_APP_TARGETS.find((target) => target.id === DEFAULT_OPEN_APP_ID) ?? DEFAULT_OPEN_APP_TARGETS[0] ?? { id: DEFAULT_OPEN_APP_ID, label: "VS Code", diff --git a/src/features/app/constants.ts b/src/features/app/constants.ts index 1f06f4ad..dacc6d2a 100644 --- a/src/features/app/constants.ts +++ b/src/features/app/constants.ts @@ -1,50 +1,99 @@ import type { OpenAppTarget } from "../../types"; +import { + fileManagerName, + isMacPlatform, + isWindowsPlatform, +} from "../../utils/platformPaths"; export const OPEN_APP_STORAGE_KEY = "open-workspace-app"; -export const DEFAULT_OPEN_APP_ID = "vscode"; +export const DEFAULT_OPEN_APP_ID = isWindowsPlatform() ? "finder" : "vscode"; export type OpenAppId = string; -export const DEFAULT_OPEN_APP_TARGETS: OpenAppTarget[] = [ - { - id: "vscode", - label: "VS Code", - kind: "app", - appName: "Visual Studio Code", - args: [], - }, - { - id: "cursor", - label: "Cursor", - kind: "app", - appName: "Cursor", - args: [], - }, - { - id: "zed", - label: "Zed", - kind: "app", - appName: "Zed", - args: [], - }, - { - id: "ghostty", - label: "Ghostty", - kind: "app", - appName: "Ghostty", - args: [], - }, - { - id: "antigravity", - label: "Antigravity", - kind: "app", - appName: "Antigravity", - args: [], - }, - { - id: "finder", - label: "Finder", - kind: "finder", - args: [], - }, -]; +export const DEFAULT_OPEN_APP_TARGETS: OpenAppTarget[] = isMacPlatform() + ? [ + { + id: "vscode", + label: "VS Code", + kind: "app", + appName: "Visual Studio Code", + args: [], + }, + { + id: "cursor", + label: "Cursor", + kind: "app", + appName: "Cursor", + args: [], + }, + { + id: "zed", + label: "Zed", + kind: "app", + appName: "Zed", + args: [], + }, + { + id: "ghostty", + label: "Ghostty", + kind: "app", + appName: "Ghostty", + args: [], + }, + { + id: "antigravity", + label: "Antigravity", + kind: "app", + appName: "Antigravity", + args: [], + }, + { + id: "finder", + label: fileManagerName(), + kind: "finder", + args: [], + }, + ] + : [ + { + id: "vscode", + label: "VS Code", + kind: "command", + command: "code", + args: [], + }, + { + id: "cursor", + label: "Cursor", + kind: "command", + command: "cursor", + args: [], + }, + { + id: "zed", + label: "Zed", + kind: "command", + command: "zed", + args: [], + }, + { + id: "ghostty", + label: "Ghostty", + kind: "command", + command: "ghostty", + args: [], + }, + { + id: "antigravity", + label: "Antigravity", + kind: "command", + command: "antigravity", + args: [], + }, + { + id: "finder", + label: fileManagerName(), + kind: "finder", + args: [], + }, + ]; diff --git a/src/features/app/utils/openAppIcons.ts b/src/features/app/utils/openAppIcons.ts index 82d8e967..836520b9 100644 --- a/src/features/app/utils/openAppIcons.ts +++ b/src/features/app/utils/openAppIcons.ts @@ -4,6 +4,7 @@ import antigravityIcon from "../../../assets/app-icons/antigravity.png"; import ghosttyIcon from "../../../assets/app-icons/ghostty.png"; import vscodeIcon from "../../../assets/app-icons/vscode.png"; import zedIcon from "../../../assets/app-icons/zed.png"; +import { isMacPlatform } from "../../../utils/platformPaths"; const GENERIC_APP_SVG = ""; @@ -12,6 +13,13 @@ export const GENERIC_APP_ICON = `data:image/svg+xml;utf8,${encodeURIComponent( GENERIC_APP_SVG, )}`; +const GENERIC_FOLDER_SVG = + ""; + +export const GENERIC_FOLDER_ICON = `data:image/svg+xml;utf8,${encodeURIComponent( + GENERIC_FOLDER_SVG, +)}`; + export function getKnownOpenAppIcon(id: string): string | null { switch (id) { case "vscode": @@ -25,7 +33,7 @@ export function getKnownOpenAppIcon(id: string): string | null { case "antigravity": return antigravityIcon; case "finder": - return finderIcon; + return isMacPlatform() ? finderIcon : GENERIC_FOLDER_ICON; default: return null; } diff --git a/src/features/files/components/FileTreePanel.tsx b/src/features/files/components/FileTreePanel.tsx index 98c8fab9..c8a259f8 100644 --- a/src/features/files/components/FileTreePanel.tsx +++ b/src/features/files/components/FileTreePanel.tsx @@ -30,6 +30,7 @@ import { PanelTabs, type PanelTabId } from "../../layout/components/PanelTabs"; import { readWorkspaceFile } from "../../../services/tauri"; import type { OpenAppTarget } from "../../../types"; import { languageFromPath } from "../../../utils/syntax"; +import { joinWorkspacePath, revealInFileManagerLabel } from "../../../utils/platformPaths"; import { FilePreviewPopover } from "./FilePreviewPopover"; type FileTreeNode = { @@ -370,10 +371,7 @@ export function FileTreePanel({ const resolvePath = useCallback( (relativePath: string) => { - const base = workspacePath.endsWith("/") - ? workspacePath.slice(0, -1) - : workspacePath; - return `${base}/${relativePath}`; + return joinWorkspacePath(workspacePath, relativePath); }, [workspacePath], ); @@ -585,7 +583,7 @@ export function FileTreePanel({ }, }), await MenuItem.new({ - text: "Reveal in Finder", + text: revealInFileManagerLabel(), action: async () => { await revealItemInDir(resolvePath(relativePath)); }, diff --git a/src/features/home/components/Home.tsx b/src/features/home/components/Home.tsx index d9664021..a3c56b75 100644 --- a/src/features/home/components/Home.tsx +++ b/src/features/home/components/Home.tsx @@ -1,3 +1,4 @@ +import FolderOpen from "lucide-react/dist/esm/icons/folder-open"; import RefreshCw from "lucide-react/dist/esm/icons/refresh-cw"; import type { LocalUsageSnapshot } from "../../../types"; import { formatRelativeTime } from "../../../utils/time"; @@ -243,7 +244,7 @@ export function Home({ data-tauri-drag-region="false" > - ⌘ + Open Project diff --git a/src/features/messages/hooks/useFileLinkOpener.ts b/src/features/messages/hooks/useFileLinkOpener.ts index 94d3e31e..879d5259 100644 --- a/src/features/messages/hooks/useFileLinkOpener.ts +++ b/src/features/messages/hooks/useFileLinkOpener.ts @@ -8,6 +8,11 @@ import * as Sentry from "@sentry/react"; import { openWorkspaceIn } from "../../../services/tauri"; import { pushErrorToast } from "../../../services/toasts"; import type { OpenAppTarget } from "../../../types"; +import { + isAbsolutePath, + joinWorkspacePath, + revealInFileManagerLabel, +} from "../../../utils/platformPaths"; type OpenTarget = { id: string; @@ -44,11 +49,10 @@ function resolveFilePath(path: string, workspacePath?: string | null) { if (!workspacePath) { return trimmed; } - if (trimmed.startsWith("/") || trimmed.startsWith("~/")) { + if (isAbsolutePath(trimmed)) { return trimmed; } - const base = workspacePath.replace(/\/+$/, ""); - return `${base}/${trimmed}`; + return joinWorkspacePath(workspacePath, trimmed); } function stripLineSuffix(path: string) { @@ -56,20 +60,6 @@ function stripLineSuffix(path: string) { return match ? match[1] : path; } -function revealLabel() { - const platform = - (navigator as Navigator & { userAgentData?: { platform?: string } }) - .userAgentData?.platform ?? navigator.platform ?? ""; - const normalized = platform.toLowerCase(); - if (normalized.includes("mac")) { - return "Reveal in Finder"; - } - if (normalized.includes("win")) { - return "Show in Explorer"; - } - return "Reveal in File Manager"; -} - export function useFileLinkOpener( workspacePath: string | null, openTargets: OpenAppTarget[], @@ -164,7 +154,7 @@ export function useFileLinkOpener( const canOpen = canOpenTarget(target); const openLabel = target.kind === "finder" - ? revealLabel() + ? revealInFileManagerLabel() : target.kind === "command" ? command ? `Open in ${target.label}` @@ -184,7 +174,7 @@ export function useFileLinkOpener( ? [] : [ await MenuItem.new({ - text: revealLabel(), + text: revealInFileManagerLabel(), action: async () => { try { await revealItemInDir(resolvedPath); diff --git a/src/features/settings/components/SettingsView.test.tsx b/src/features/settings/components/SettingsView.test.tsx index 8c00912d..f7cf3687 100644 --- a/src/features/settings/components/SettingsView.test.tsx +++ b/src/features/settings/components/SettingsView.test.tsx @@ -48,9 +48,9 @@ const baseSettings: AppSettings = { theme: "system", usageShowRemaining: false, uiFontFamily: - "\"SF Pro Text\", \"SF Pro Display\", -apple-system, \"Helvetica Neue\", sans-serif", + 'system-ui, -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, "Helvetica Neue", Arial, sans-serif', codeFontFamily: - "\"SF Mono\", \"SFMono-Regular\", Menlo, Monaco, monospace", + 'ui-monospace, "Cascadia Mono", "Segoe UI Mono", Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace', codeFontSize: 11, notificationSoundsEnabled: true, systemNotificationsEnabled: true, @@ -316,12 +316,12 @@ describe("SettingsView Display", () => { await waitFor(() => { expect(onUpdateAppSettings).toHaveBeenCalledWith( expect.objectContaining({ - uiFontFamily: expect.stringContaining("SF Pro Text"), + uiFontFamily: expect.stringContaining("system-ui"), }), ); expect(onUpdateAppSettings).toHaveBeenCalledWith( expect.objectContaining({ - codeFontFamily: expect.stringContaining("SF Mono"), + codeFontFamily: expect.stringContaining("ui-monospace"), }), ); }); diff --git a/src/features/settings/components/SettingsView.tsx b/src/features/settings/components/SettingsView.tsx index 88386bbf..5e0d3972 100644 --- a/src/features/settings/components/SettingsView.tsx +++ b/src/features/settings/components/SettingsView.tsx @@ -25,6 +25,12 @@ import type { WorkspaceInfo, } from "../../../types"; import { formatDownloadSize } from "../../../utils/formatting"; +import { + fileManagerName, + isMacPlatform, + isWindowsPlatform, + openInFileManagerLabel, +} from "../../../utils/platformPaths"; import { buildShortcutValue, formatShortcut, @@ -425,6 +431,12 @@ export function SettingsView({ const globalConfigSaveLabel = globalConfigExists ? "Save" : "Create"; const globalConfigSaveDisabled = globalConfigLoading || globalConfigSaving || !globalConfigDirty; const globalConfigRefreshDisabled = globalConfigLoading || globalConfigSaving; + const optionKeyLabel = isMacPlatform() ? "Option" : "Alt"; + const metaKeyLabel = isMacPlatform() + ? "Command" + : isWindowsPlatform() + ? "Windows" + : "Meta"; const selectedDictationModel = useMemo(() => { return ( DICTATION_MODELS.find( @@ -1571,7 +1583,7 @@ export function SettingsView({
System notifications
- Show a macOS notification when a long-running agent finishes while the window is unfocused. + Show a system notification when a long-running agent finishes while the window is unfocused.
- Default: {formatShortcut("cmd+ctrl+a")} + Default:{" "} + {formatShortcut(isMacPlatform() ? "cmd+ctrl+a" : "ctrl+alt+a")}
@@ -2379,7 +2392,10 @@ export function SettingsView({
- Default: {formatShortcut("cmd+ctrl+down")} + Default:{" "} + {formatShortcut( + isMacPlatform() ? "cmd+ctrl+down" : "ctrl+alt+down", + )}
@@ -2403,7 +2419,10 @@ export function SettingsView({
- Default: {formatShortcut("cmd+ctrl+up")} + Default:{" "} + {formatShortcut( + isMacPlatform() ? "cmd+ctrl+up" : "ctrl+alt+up", + )}
@@ -2427,7 +2446,12 @@ export function SettingsView({
- Default: {formatShortcut("cmd+shift+down")} + Default:{" "} + {formatShortcut( + isMacPlatform() + ? "cmd+shift+down" + : "ctrl+alt+shift+down", + )}
@@ -2451,7 +2475,10 @@ export function SettingsView({
- Default: {formatShortcut("cmd+shift+up")} + Default:{" "} + {formatShortcut( + isMacPlatform() ? "cmd+shift+up" : "ctrl+alt+shift+up", + )}
@@ -2531,7 +2558,7 @@ export function SettingsView({ > - + {target.kind === "app" && ( @@ -2658,8 +2685,10 @@ export function SettingsView({ Add app
- Commands receive the selected path as the final argument. Apps use macOS open - with optional args. + Commands receive the selected path as the final argument.{" "} + {isMacPlatform() + ? "Apps open via `open -a` with optional args." + : "Apps run as an executable with optional args."}
@@ -3139,11 +3168,11 @@ export function SettingsView({
Config file
- Open the Codex config in Finder. + Open the Codex config in {fileManagerName()}.
{openConfigError && ( diff --git a/src/features/settings/hooks/useAppSettings.test.ts b/src/features/settings/hooks/useAppSettings.test.ts index 5a1ccbd7..44f70840 100644 --- a/src/features/settings/hooks/useAppSettings.test.ts +++ b/src/features/settings/hooks/useAppSettings.test.ts @@ -49,8 +49,8 @@ describe("useAppSettings", () => { expect(result.current.settings.uiScale).toBe(UI_SCALE_MAX); expect(result.current.settings.theme).toBe("system"); - expect(result.current.settings.uiFontFamily).toContain("SF Pro Text"); - expect(result.current.settings.codeFontFamily).toContain("SF Mono"); + expect(result.current.settings.uiFontFamily).toContain("system-ui"); + expect(result.current.settings.codeFontFamily).toContain("ui-monospace"); expect(result.current.settings.codeFontSize).toBe(16); expect(result.current.settings.experimentalPersonality).toBe("default"); expect(result.current.settings.backendMode).toBe("remote"); @@ -66,8 +66,8 @@ describe("useAppSettings", () => { expect(result.current.settings.uiScale).toBe(UI_SCALE_DEFAULT); expect(result.current.settings.theme).toBe("system"); - expect(result.current.settings.uiFontFamily).toContain("SF Pro Text"); - expect(result.current.settings.codeFontFamily).toContain("SF Mono"); + expect(result.current.settings.uiFontFamily).toContain("system-ui"); + expect(result.current.settings.codeFontFamily).toContain("ui-monospace"); expect(result.current.settings.backendMode).toBe("local"); expect(result.current.settings.dictationModelId).toBe("base"); expect(result.current.settings.interruptShortcut).toBeTruthy(); @@ -110,8 +110,8 @@ describe("useAppSettings", () => { expect.objectContaining({ theme: "system", uiScale: 0.1, - uiFontFamily: expect.stringContaining("SF Pro Text"), - codeFontFamily: expect.stringContaining("SF Mono"), + uiFontFamily: expect.stringContaining("system-ui"), + codeFontFamily: expect.stringContaining("ui-monospace"), codeFontSize: 9, notificationSoundsEnabled: false, }), diff --git a/src/features/settings/hooks/useAppSettings.ts b/src/features/settings/hooks/useAppSettings.ts index 49641cc9..438003ba 100644 --- a/src/features/settings/hooks/useAppSettings.ts +++ b/src/features/settings/hooks/useAppSettings.ts @@ -15,11 +15,13 @@ import { OPEN_APP_STORAGE_KEY, } from "../../app/constants"; import { normalizeOpenAppTargets } from "../../app/utils/openApp"; -import { getDefaultInterruptShortcut } from "../../../utils/shortcuts"; +import { getDefaultInterruptShortcut, isMacPlatform } from "../../../utils/shortcuts"; const allowedThemes = new Set(["system", "light", "dark", "dim"]); const allowedPersonality = new Set(["default", "friendly", "pragmatic"]); +const isMac = isMacPlatform(); + const defaultSettings: AppSettings = { codexBin: null, codexArgs: null, @@ -28,23 +30,23 @@ const defaultSettings: AppSettings = { remoteBackendToken: null, defaultAccessMode: "current", reviewDeliveryMode: "inline", - composerModelShortcut: "cmd+shift+m", - composerAccessShortcut: "cmd+shift+a", - composerReasoningShortcut: "cmd+shift+r", + composerModelShortcut: isMac ? "cmd+shift+m" : "ctrl+shift+m", + composerAccessShortcut: isMac ? "cmd+shift+a" : "ctrl+shift+a", + composerReasoningShortcut: isMac ? "cmd+shift+r" : "ctrl+shift+r", composerCollaborationShortcut: "shift+tab", interruptShortcut: getDefaultInterruptShortcut(), - newAgentShortcut: "cmd+n", - newWorktreeAgentShortcut: "cmd+shift+n", - newCloneAgentShortcut: "cmd+alt+n", - archiveThreadShortcut: "cmd+ctrl+a", - toggleProjectsSidebarShortcut: "cmd+shift+p", - toggleGitSidebarShortcut: "cmd+shift+g", - toggleDebugPanelShortcut: "cmd+shift+d", - toggleTerminalShortcut: "cmd+shift+t", - cycleAgentNextShortcut: "cmd+ctrl+down", - cycleAgentPrevShortcut: "cmd+ctrl+up", - cycleWorkspaceNextShortcut: "cmd+shift+down", - cycleWorkspacePrevShortcut: "cmd+shift+up", + newAgentShortcut: isMac ? "cmd+n" : "ctrl+n", + newWorktreeAgentShortcut: isMac ? "cmd+shift+n" : "ctrl+shift+n", + newCloneAgentShortcut: isMac ? "cmd+alt+n" : "ctrl+alt+n", + archiveThreadShortcut: isMac ? "cmd+ctrl+a" : "ctrl+alt+a", + toggleProjectsSidebarShortcut: isMac ? "cmd+shift+p" : "ctrl+shift+p", + toggleGitSidebarShortcut: isMac ? "cmd+shift+g" : "ctrl+shift+g", + toggleDebugPanelShortcut: isMac ? "cmd+shift+d" : "ctrl+shift+d", + toggleTerminalShortcut: isMac ? "cmd+shift+t" : "ctrl+shift+t", + cycleAgentNextShortcut: isMac ? "cmd+ctrl+down" : "ctrl+alt+down", + cycleAgentPrevShortcut: isMac ? "cmd+ctrl+up" : "ctrl+alt+up", + cycleWorkspaceNextShortcut: isMac ? "cmd+shift+down" : "ctrl+alt+shift+down", + cycleWorkspacePrevShortcut: isMac ? "cmd+shift+up" : "ctrl+alt+shift+up", lastComposerModelId: null, lastComposerReasoningEffort: null, uiScale: UI_SCALE_DEFAULT, diff --git a/src/main.tsx b/src/main.tsx index 21b7dc69..5da8c2a3 100644 --- a/src/main.tsx +++ b/src/main.tsx @@ -3,6 +3,26 @@ import ReactDOM from "react-dom/client"; import * as Sentry from "@sentry/react"; import App from "./App"; +function sentryPlatformLabel(): string { + if (typeof navigator === "undefined") { + return "unknown"; + } + const platform = + (navigator as Navigator & { userAgentData?: { platform?: string } }) + .userAgentData?.platform ?? navigator.platform ?? ""; + const normalized = platform.toLowerCase(); + if (normalized.includes("mac")) { + return "macos"; + } + if (normalized.includes("win")) { + return "windows"; + } + if (normalized.includes("linux")) { + return "linux"; + } + return "unknown"; +} + const sentryDsn = import.meta.env.VITE_SENTRY_DSN ?? "https://8ab67175daed999e8c432a93d8f98e49@o4510750015094784.ingest.us.sentry.io/4510750016012288"; @@ -16,7 +36,7 @@ Sentry.init({ Sentry.metrics.count("app_open", 1, { attributes: { env: import.meta.env.MODE, - platform: "macos", + platform: sentryPlatformLabel(), }, }); diff --git a/src/styles/approval-toasts.css b/src/styles/approval-toasts.css index ff71a5a1..a488dfab 100644 --- a/src/styles/approval-toasts.css +++ b/src/styles/approval-toasts.css @@ -77,7 +77,7 @@ } .approval-toast-detail-code { - font-family: "SF Mono", Menlo, monospace; + font-family: var(--code-font-family); font-size: 11px; color: var(--text-muted); white-space: pre-wrap; diff --git a/src/styles/plan.css b/src/styles/plan.css index 9f4770de..162e96cd 100644 --- a/src/styles/plan.css +++ b/src/styles/plan.css @@ -60,7 +60,7 @@ } .plan-step-status { - font-family: "SF Mono", Menlo, monospace; + font-family: var(--code-font-family); font-size: 11px; color: var(--text-faint); flex: 0 0 auto; diff --git a/src/styles/settings.css b/src/styles/settings.css index 14e6fe55..eb4e2fa4 100644 --- a/src/styles/settings.css +++ b/src/styles/settings.css @@ -259,7 +259,7 @@ } .settings-input--shortcut { - font-family: "SF Mono", "Fira Mono", "Menlo", monospace; + font-family: var(--code-font-family); letter-spacing: 0.02em; } @@ -286,6 +286,11 @@ font-size: 12px; } +.settings-select option { + background-color: var(--surface-popover); + color: var(--text-strong); +} + .settings-select--compact { padding: 6px 8px; font-size: 11px; diff --git a/src/styles/themes.dark.css b/src/styles/themes.dark.css index 73ac753c..6fb78809 100644 --- a/src/styles/themes.dark.css +++ b/src/styles/themes.dark.css @@ -1,14 +1,15 @@ :root { - font-family: "SF Pro Text", "SF Pro Display", -apple-system, "Helvetica Neue", - sans-serif; + font-family: system-ui, -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, + "Helvetica Neue", Arial, sans-serif; color: var(--text-primary); background-color: transparent; font-synthesis: none; text-rendering: optimizeLegibility; color-scheme: light dark; - --ui-font-family: "SF Pro Text", "SF Pro Display", -apple-system, "Helvetica Neue", - sans-serif; - --code-font-family: "SF Mono", "SFMono-Regular", Menlo, Monaco, monospace; + --ui-font-family: system-ui, -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, + "Helvetica Neue", Arial, sans-serif; + --code-font-family: ui-monospace, "Cascadia Mono", "Segoe UI Mono", Menlo, + Monaco, Consolas, "Liberation Mono", "Courier New", monospace; --code-font-size: 11px; --code-font-weight: 400; --code-line-height: 1.28; diff --git a/src/styles/update-toasts.css b/src/styles/update-toasts.css index 79768a83..581fd3b4 100644 --- a/src/styles/update-toasts.css +++ b/src/styles/update-toasts.css @@ -71,7 +71,7 @@ } .update-toast-error { - font-family: "SF Mono", Menlo, monospace; + font-family: var(--code-font-family); font-size: 11px; color: var(--text-muted); white-space: pre-wrap; diff --git a/src/utils/fonts.ts b/src/utils/fonts.ts index 2a1a4584..f086f62d 100644 --- a/src/utils/fonts.ts +++ b/src/utils/fonts.ts @@ -1,8 +1,8 @@ export const DEFAULT_UI_FONT_FAMILY = - "\"SF Pro Text\", \"SF Pro Display\", -apple-system, \"Helvetica Neue\", sans-serif"; + 'system-ui, -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, "Helvetica Neue", Arial, sans-serif'; export const DEFAULT_CODE_FONT_FAMILY = - "\"SF Mono\", \"SFMono-Regular\", Menlo, Monaco, monospace"; + 'ui-monospace, "Cascadia Mono", "Segoe UI Mono", Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace'; export const CODE_FONT_SIZE_DEFAULT = 11; export const CODE_FONT_SIZE_MIN = 9; diff --git a/src/utils/platformPaths.ts b/src/utils/platformPaths.ts new file mode 100644 index 00000000..7d7799de --- /dev/null +++ b/src/utils/platformPaths.ts @@ -0,0 +1,113 @@ +type PlatformKind = "mac" | "windows" | "linux" | "unknown"; + +function platformKind(): PlatformKind { + if (typeof navigator === "undefined") { + return "unknown"; + } + const platform = + (navigator as Navigator & { userAgentData?: { platform?: string } }) + .userAgentData?.platform ?? navigator.platform ?? ""; + const normalized = platform.toLowerCase(); + if (normalized.includes("mac")) { + return "mac"; + } + if (normalized.includes("win")) { + return "windows"; + } + if (normalized.includes("linux")) { + return "linux"; + } + return "unknown"; +} + +export function isMacPlatform(): boolean { + return platformKind() === "mac"; +} + +export function isWindowsPlatform(): boolean { + return platformKind() === "windows"; +} + +export function fileManagerName(): string { + const platform = platformKind(); + if (platform === "mac") { + return "Finder"; + } + if (platform === "windows") { + return "Explorer"; + } + return "File Manager"; +} + +export function revealInFileManagerLabel(): string { + const platform = platformKind(); + if (platform === "mac") { + return "Reveal in Finder"; + } + if (platform === "windows") { + return "Show in Explorer"; + } + return "Reveal in File Manager"; +} + +export function openInFileManagerLabel(): string { + return `Open in ${fileManagerName()}`; +} + +function looksLikeWindowsAbsolutePath(value: string): boolean { + if (/^[A-Za-z]:[\\/]/.test(value)) { + return true; + } + if (value.startsWith("\\\\") || value.startsWith("//")) { + return true; + } + if (value.startsWith("\\\\?\\")) { + return true; + } + return false; +} + +export function isAbsolutePath(value: string): boolean { + const trimmed = value.trim(); + if (!trimmed) { + return false; + } + if (trimmed.startsWith("/") || trimmed.startsWith("~/") || trimmed.startsWith("~\\")) { + return true; + } + return looksLikeWindowsAbsolutePath(trimmed); +} + +function stripTrailingSeparators(value: string) { + return value.replace(/[\\/]+$/, ""); +} + +function stripLeadingSeparators(value: string) { + return value.replace(/^[\\/]+/, ""); +} + +function looksLikeWindowsPathPrefix(value: string): boolean { + const trimmed = value.trim(); + return looksLikeWindowsAbsolutePath(trimmed) || trimmed.includes("\\"); +} + +export function joinWorkspacePath(base: string, path: string): string { + const trimmedBase = base.trim(); + const trimmedPath = path.trim(); + if (!trimmedBase) { + return trimmedPath; + } + if (!trimmedPath || isAbsolutePath(trimmedPath)) { + return trimmedPath; + } + + const isWindows = looksLikeWindowsPathPrefix(trimmedBase); + const baseWithoutTrailing = stripTrailingSeparators(trimmedBase); + const pathWithoutLeading = stripLeadingSeparators(trimmedPath); + if (isWindows) { + const normalizedRelative = pathWithoutLeading.replace(/\//g, "\\"); + return `${baseWithoutTrailing}\\${normalizedRelative}`; + } + const normalizedRelative = pathWithoutLeading.replace(/\\/g, "/"); + return `${baseWithoutTrailing}/${normalizedRelative}`; +} diff --git a/src/utils/shortcuts.ts b/src/utils/shortcuts.ts index 0d6eb1cf..8bbaac93 100644 --- a/src/utils/shortcuts.ts +++ b/src/utils/shortcuts.ts @@ -7,13 +7,20 @@ export type ShortcutDefinition = { }; const MODIFIER_ORDER = ["cmd", "ctrl", "alt", "shift"] as const; -const MODIFIER_LABELS: Record = { +const MODIFIER_LABELS_MAC: Record = { cmd: "⌘", ctrl: "⌃", alt: "⌥", shift: "⇧", }; +const MODIFIER_LABELS_OTHER: Record = { + cmd: "Ctrl", + ctrl: "Ctrl", + alt: "Alt", + shift: "Shift", +}; + const KEY_LABELS: Record = { " ": "Space", space: "Space", @@ -85,25 +92,32 @@ export function formatShortcut(value: string | null | undefined): string { if (!parsed) { return value; } + const useSymbols = isMacPlatform(); + const modifierLabels = useSymbols ? MODIFIER_LABELS_MAC : MODIFIER_LABELS_OTHER; const modifiers = MODIFIER_ORDER.flatMap((modifier) => { if (modifier === "cmd" && parsed.meta) { - return MODIFIER_LABELS.cmd; + return modifierLabels.cmd; } if (modifier === "ctrl" && parsed.ctrl) { - return MODIFIER_LABELS.ctrl; + return modifierLabels.ctrl; } if (modifier === "alt" && parsed.alt) { - return MODIFIER_LABELS.alt; + return modifierLabels.alt; } if (modifier === "shift" && parsed.shift) { - return MODIFIER_LABELS.shift; + return modifierLabels.shift; } return []; }); + const uniqueModifiers = useSymbols + ? modifiers + : modifiers.filter((modifier, index) => modifiers.indexOf(modifier) === index); const keyLabel = KEY_LABELS[parsed.key] ?? (parsed.key.length === 1 ? parsed.key.toUpperCase() : parsed.key); - return [...modifiers, keyLabel].join(""); + return useSymbols + ? [...uniqueModifiers, keyLabel].join("") + : [...uniqueModifiers, keyLabel].join("+"); } export function buildShortcutValue(event: KeyboardEvent): string | null { @@ -141,12 +155,22 @@ export function matchesShortcut(event: KeyboardEvent, value: string | null | und if (!key || key !== parsed.key) { return false; } - return ( - parsed.meta === event.metaKey && - parsed.ctrl === event.ctrlKey && - parsed.alt === event.altKey && - parsed.shift === event.shiftKey - ); + const isMac = isMacPlatform(); + const metaMatches = parsed.meta + ? isMac + ? event.metaKey + : event.ctrlKey || event.metaKey + : !event.metaKey; + if (!metaMatches) { + return false; + } + + const ctrlMatches = parsed.ctrl + ? event.ctrlKey + : parsed.meta && !isMac + ? true + : !event.ctrlKey; + return ctrlMatches && parsed.alt === event.altKey && parsed.shift === event.shiftKey; } export function isMacPlatform(): boolean {