Firstp1ck · Firstp1ck · Feb 1, 2026 · Feb 1, 2026 · Feb 1, 2026 · Feb 1, 2026
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -48,9 +48,15 @@ chrono = "0.4.43"
 portable-pty = "0.9"
 strip-ansi-escapes = "0.2"
 lru = "0.16.3"
-ego-tree = "0.10"
+ego-tree = "0.10.0"
 rand = "0.9.2"
 
+[target.'cfg(unix)'.dependencies]
+nix = { version = "0.31", features = ["poll", "fs"], default-features = false }
+
+[target.'cfg(windows)'.dependencies]
+windows-sys = { version = "0.52", features = ["Win32_Foundation", "Win32_System_Threading"] }
+
 [dev-dependencies]
 tempfile = "3.24.0"
 syn = { version = "2.0.114", features = ["full", "parsing", "extra-traits"] }

diff --git a/config/settings.conf b/config/settings.conf
@@ -191,3 +191,11 @@ locale =
 # Default is 30 seconds. Increase this value on systems with slow I/O or many packages to reduce resource usage.
 # Minimum value is 1 second.
 updates_refresh_interval = 30
+
+# Terminal Theme
+# When true, uses the terminal's theme colors instead of theme.conf via OSC 10/11 queries.
+# When false (default), uses theme.conf colors.
+# Note: Terminal theme is also used automatically when theme.conf is missing/invalid
+# and the terminal is supported (alacritty, kitty, konsole, ghostty, xterm,
+# gnome-terminal, xfce-terminal, tilix, mate-terminal, wezterm, wezterm-gui).
+use_terminal_theme = false
diff --git a/dev/COMMIT/COMMIT_MSG.md b/dev/COMMIT/COMMIT_MSG.md
@@ -0,0 +1,5 @@
+fix: avoid modal spam on package-details fetch failures
+
+- fix: skip Alert modal for official/AUR package-details-unavailable errors; only log.
+- fix: show modal only for other network errors so UX stays clear.
+- docs: update PR to document package-details error handling in event_loop.
diff --git a/dev/IMPROVEMENTS/TERMINAL_THEME_FALLBACK.md b/dev/IMPROVEMENTS/TERMINAL_THEME_FALLBACK.md
diff --git a/dev/PR/PR_feat-edit-visual-picker-REVIEW.md b/dev/PR/PR_feat-edit-visual-picker-REVIEW.md
diff --git a/dev/PR/PR_feat-terminal-colors.md b/dev/PR/PR_feat-terminal-colors.md
@@ -0,0 +1,107 @@
+<!-- Thank you for contributing to Pacsea!
+
+**Important references:**
+- [CONTRIBUTING.md](../CONTRIBUTING.md) — Full contribution guidelines and PR process
+- [PR_DESCRIPTION.md](../Documents/PR_DESCRIPTION.md) — Detailed PR description template
+- [Development Wiki](https://github.com/Firstp1ck/Pacsea/wiki/Development) — Development tools and debugging
+
+Please ensure you've reviewed these before submitting your PR.
+-->
+
+## Summary
+- **Problem**: When no theme is set (or user wants to match the terminal), Pacsea always wrote a default skeleton (Catppuccin Mocha) to `theme.conf` and used it. There was no way to use the terminal’s foreground/background colors so the UI could blend with the terminal palette.
+- **Solution**:
+  - Add `use_terminal_theme` setting in `settings.conf` (default: `false`). When `true`, theme resolution tries to query the terminal via OSC 10/11 (foreground/background), parses the reply, builds Pacsea’s 16-color theme from fg/bg and derivation rules, and uses it. If the terminal does not support query or the query fails (e.g. non-TTY, timeout), resolution falls back to `theme.conf` or the default skeleton.
+  - Add theme resolution order: prefer terminal theme when `use_terminal_theme=true` and terminal is supported; otherwise use file theme or skeleton.
+  - Add OSC 10/11 query implementation with timeout and robust parsing (`terminal_query.rs`), terminal support detection (`terminal_detect.rs`), and resolution logic (`resolve.rs`).
+  - Improve robustness of OSC query (handling reply format, ST terminator, non-TTY).
+  - On Unix: query terminal colors via `/dev/tty` with nix `poll` so stdin is not used and crossterm is not raced.
+  - Add WezTerm and wezterm-gui to supported terminals (unit test added).
+  - On Windows: cancellable reader thread (WaitForSingleObject with short timeout + cancel flag) so join() returns quickly and does not hang if the OSC query never yields bytes; avoids freezing startup/theme reload.
+  - On other non-Unix: bounded join so the caller does not block indefinitely if the reader is stuck on stdin.
+  - In headless/test (`PACSEA_TEST_HEADLESS=1`): skip mouse capture disable/enable for clean output.
+  - Remove dead `load_theme_from_file` from theme_loader; add rustdoc and error logging in `ensure_theme_file_exists`. `ensure_theme_file_exists()` always targets `config_dir()/theme.conf` for creation (never legacy pacsea.conf); reading continues to use `resolve_theme_config_path()` elsewhere.
+  - Fix AUR cache handling for long filenames in updates feed.
+  - Do not show a "Connection issue" modal for each failed official/AUR package-details fetch when scrolling; only log. Reduces modal spam when network is flaky or circuit breaker is open.
+  - Build: add nix (Unix) dependency with poll and fs features; add windows-sys (Windows) for timeout-aware stdin wait in OSC reader.
+- **Doc**: Add `dev/IMPROVEMENTS/TERMINAL_THEME_FALLBACK.md` describing design, OSC sequences, and implementation choices.
+
+## Type of change
+- [x] feat (new feature)
+- [x] fix (bug fix)
+- [ ] docs (documentation only)
+- [x] refactor (no functional change)
+- [ ] perf (performance)
+- [ ] test (add/update tests)
+- [x] chore (build/infra/CI)
+- [x] ui (visual/interaction changes)
+- [ ] breaking change (incompatible behavior)
+
+## Related issues
+Closes #122
+
+## How to test
+1. Format, lint, check, and test:
+```bash
+cargo fmt --all
+cargo clippy --all-targets --all-features -- -D warnings
+cargo check
+cargo test -- --test-threads=1
+```
+2. Terminal theme on supported terminal (e.g. Alacritty, Kitty, WezTerm): set `use_terminal_theme = true` in `settings.conf`, run Pacsea; UI should match terminal fg/bg–derived colors. Change terminal theme and reload theme in Pacsea to see updates. On Unix, query uses `/dev/tty` (no stdin).
+3. Unsupported / non-TTY: run with `use_terminal_theme = true` in a pipe or unsupported terminal; should fall back to theme.conf or default skeleton without hanging.
+4. AUR updates: run update flow on a system with AUR packages; long cache filenames should not cause failures.
+
+```bash
+RUST_LOG=pacsea=debug cargo run -- --dry-run
+```
+
+## Checklist
+
+**Code Quality:**
+- [ ] Code compiles locally (`cargo check`)
+- [ ] `cargo fmt --all` ran without changes
+- [ ] `cargo clippy --all-targets --all-features -- -D warnings` is clean
+- [ ] `cargo test -- --test-threads=1` passes
+- [ ] Complexity checks pass for new code (`cargo test complexity -- --nocapture`)
+- [ ] All new functions/methods have rustdoc comments (What, Inputs, Output, Details)
+- [ ] No `unwrap()` or `expect()` in non-test code
+
+**Testing:**
+- [ ] Added or updated tests where it makes sense
+- [ ] For bug fixes: created failing tests first, then fixed the issue
+- [ ] Tests are meaningful and cover the functionality
+
+**Documentation:**
+- [ ] Updated README if behavior, options, or keybinds changed (keep high-level, reference wiki)
+- [ ] Updated relevant wiki pages if needed:
+  - [How to use Pacsea](https://github.com/Firstp1ck/Pacsea/wiki/How-to-use-Pacsea)
+  - [Configuration](https://github.com/Firstp1ck/Pacsea/wiki/Configuration)
+  - [Keyboard Shortcuts](https://github.com/Firstp1ck/Pacsea/wiki/Keyboard-Shortcuts)
+- [x] Updated config examples in `config/` directory if config keys changed (`settings.conf`: `use_terminal_theme`)
+- [ ] For UI changes: included screenshots and updated `Images/` if applicable
+
+**Compatibility:**
+- [x] Changes respect `--dry-run` flag
+- [x] Code degrades gracefully if `pacman`/`paru`/`yay` are unavailable
+- [x] No breaking changes (or clearly documented if intentional): new setting is opt-in, default `false`
+
+**Other:**
+- [x] Not a packaging change for AUR (otherwise propose in `pacsea-bin` or `pacsea-git` repos)
+
+## Notes for reviewers
+- **Theme resolution** (`src/theme/resolve.rs`): Order is (1) if `use_terminal_theme` and terminal supported → query OSC, use terminal theme; (2) else use theme from file or skeleton. Fallback on query failure avoids hanging in pipes/CI/unsupported terminals.
+- **OSC query** (`src/theme/terminal_query.rs`): Uses blocking read with timeout; parses `rgb:rrrr/gggg/bbbb` and optional `rgba`; handles both `\033\\` and BEL as ST. Non-TTY or no reply → returns error so resolution can fall back.
+- **Terminal detection** (`src/theme/terminal_detect.rs`): Heuristic (e.g. `COLORTERM`, `TERM`) to decide if we try OSC query; WezTerm/wezterm-gui added; not all terminals that set these support *query* (some only support *set*), so we still rely on query success/failure.
+- **Unix query** (`src/theme/terminal_query.rs`): Colors are read from `/dev/tty` via nix `poll` so stdin is untouched and there is no race with crossterm.
+- **Windows**: Reader uses WaitForSingleObject on stdin handle with short timeout and a cancel flag; main sets cancel on recv timeout then join() returns quickly (no hang when OSC query never yields bytes). Other non-Unix: bounded join so the caller does not block indefinitely.
+- **Headless tests**: When `PACSEA_TEST_HEADLESS=1`, mouse capture disable/enable is skipped for clean test output.
+- **AUR updates** (`src/sources/feeds/updates.rs`): Long filename fix is a small, separate change in the same branch; included in this PR.
+- **Package details errors** (`src/app/runtime/event_loop.rs`): When official or AUR package details fail to fetch (e.g. SSL/network or circuit breaker), the error is logged but no modal is shown, so scrolling through packages does not spam "Connection issue" dialogs.
+
+## Breaking changes
+None. New setting `use_terminal_theme` defaults to `false`; existing configs unchanged.
+
+## Additional context
+- Design and OSC details: `dev/IMPROVEMENTS/TERMINAL_THEME_FALLBACK.md`
+- Terminals that support OSC 10/11 *query* (e.g. xterm, Alacritty, Kitty, WezTerm) will use terminal theme when `use_terminal_theme = true`; others fall back to file/skeleton.