feat(review): nerd icons, mouse support, and outline folding

Cargo.toml

@@ -25,6 +25,7 @@ clap-verbosity-flag = "3.0.4"
 clap_complete = { version = "4.6.5", features = ["unstable-dynamic"] }
 clap_mangen = "0.3.0"
 crossterm = "0.29.0"
+devicons = "0.6"
 dialoguer = { version = "0.12.0", features = ["fuzzy-select"] }
 dirs = "6.0"
 env_logger = "0.11.10"

docs/adr/035-review-theming-base16-hybrid.md

@@ -132,6 +132,18 @@ read left untested (see `terminal_query.rs`).
   to render; `FgSpan` loses its `Color` field in favor of a capture index. Existing render
   tests that assert concrete colors must resolve through a fixed test `Palette`.
 
+## Revised (CS2, visual-polish pass)
+
+The "chrome that is never a theme knob (error/warn/current-marker) stays ANSI/const in
+`render.rs`" clause above is superseded. Those three colors are now `Palette` fields
+(`error_fg`/`warn_fg`/`current_fg`, mapped to base08/base0A/base0B) rather than module
+consts — the user explicitly approved revisiting this boundary during the icons/semantic-fg
+polish pass. `dark()` keeps the shipped RGB values verbatim (the same pixel-identity
+precedent the diff/cursor tints follow); `light()` takes `ONE_LIGHT`'s base08/base0A/base0B;
+`from_terminal()` takes the probed scheme's base08/base0A/base0B directly, same reasoning as
+the syntax slots (matching the terminal, not curated-tint-borrowing). No other part of the
+hybrid boundary changes: this only moves three named colors from `const` to palette fields.
+
 ## References
 
 - [ADR-034](034-review-git-native-config-schema.md) — `workon.review.theme` config key

git-workon-review/Cargo.toml

@@ -35,6 +35,7 @@ vendored = ["git-workon-lib/vendored", "git2/vendored-libgit2", "git2/vendored-o
 clap.workspace = true
 clap_complete.workspace = true
 crossterm.workspace = true
+devicons.workspace = true
 dirs.workspace = true
 git-workon-lib.workspace = true
 git2.workspace = true
@@ -53,6 +54,7 @@ tree-sitter-md.workspace = true
 tree-sitter-rust.workspace = true
 tree-sitter-toml-ng.workspace = true
 tree-sitter-typescript.workspace = true
+unicode-width.workspace = true
 
 [package.metadata.dist]
 # Redundant with publish = false today; load-bearing at the M3 flip so

git-workon-review/src/config.rs

@@ -18,6 +18,7 @@
 //! ```gitconfig
 //! [workon "review"]
 //!   theme = dark               ; auto | dark | light (default: auto)
+//!   icons = nerd               ; nerd | none (default: none)
 //!
 //! [workon "review.diff.bind"]
 //!   stage-hunk = s x           ; action = key tokens (space-separated)
@@ -32,19 +33,20 @@
 //!   width = 32
 //!   mode = tree
 //!   order = base-first         ; head-first | base-first (default: head-first)
-//!   icons = nerd               ; nerd | none (default: none)
 //!
 //! [workon "review.diff"]
 //!   layout = split
 //!   zoom = combined
 //! ```
 //!
-//! ## `outline.icons` (CS5)
+//! ## `icons`
 //!
-//! Opt-in nerd-font file/dir icons in the outline pane. There is deliberately NO auto-detection
+//! Opt-in nerd-font iconography — top-level next to `theme` (`workon.review.icons`), NOT an
+//! outline setting: the mode gates the outline's file/dir icons, the summary panel's glyphs,
+//! and the winbar's marker/diffstat/file icons alike. There is deliberately NO auto-detection
 //! — a terminal cannot report whether the user's font is patched with the nerd-font glyphs, so
 //! guessing would silently render tofu/mojibake for anyone without one. Default is `none`
-//! (today's plain text); set `icons = nerd` explicitly once your terminal font supports it. See
+//! (plain text); set `icons = nerd` explicitly once your terminal font supports it. See
 //! [`crate::icons`] for the glyph table.
 
 use git2::Repository;
@@ -112,7 +114,7 @@ pub struct RawViewConfig {
     pub outline_width: Option<i64>,
     pub outline_mode: Option<String>,
     pub outline_order: Option<String>,
-    pub outline_icons: Option<String>,
+    pub icons: Option<String>,
     pub diff_layout: Option<String>,
     pub diff_zoom: Option<String>,
 }
@@ -220,10 +222,15 @@ impl<'repo> ReviewConfig<'repo> {
         self.get_view_string(View::Outline, "order")
     }
 
-    /// Get `workon.review.outline.icons`, raw. `None` if unset — callers apply the current
-    /// default ([`crate::icons::OutlineIcons::None`], CS5: no auto-detection story exists).
-    pub fn outline_icons(&self) -> Result<Option<String>, git2::Error> {
-        self.get_view_string(View::Outline, "icons")
+    /// Get `workon.review.icons`, raw. `None` if unset — callers apply the current default
+    /// ([`crate::icons::IconMode::None`]; no auto-detection story exists). Top-level like
+    /// `theme`, not a view setting: icon mode gates the outline, summary panel, AND winbar.
+    pub fn icons(&self) -> Result<Option<String>, git2::Error> {
+        let config = self.repo.config()?;
+        match config.get_string("workon.review.icons") {
+            Ok(val) => Ok(Some(val)),
+            Err(_) => Ok(None),
+        }
     }
 
     /// Get `workon.review.diff.layout`, raw. `None` if unset.
@@ -248,7 +255,7 @@ impl<'repo> ReviewConfig<'repo> {
             outline_width: self.outline_width().ok().flatten(),
             outline_mode: self.outline_mode().ok().flatten(),
             outline_order: self.outline_order().ok().flatten(),
-            outline_icons: self.outline_icons().ok().flatten(),
+            icons: self.icons().ok().flatten(),
             diff_layout: self.diff_layout().ok().flatten(),
             diff_zoom: self.diff_zoom().ok().flatten(),
         }
@@ -433,7 +440,7 @@ mod tests {
             .config("workon.review.outline.width", "40")
             .config("workon.review.outline.mode", "tree")
             .config("workon.review.outline.order", "base-first")
-            .config("workon.review.outline.icons", "nerd")
+            .config("workon.review.icons", "nerd")
             .config("workon.review.diff.layout", "split")
             .config("workon.review.diff.zoom", "staged")
             .build()
@@ -450,10 +457,7 @@ mod tests {
             config.outline_order().expect("order"),
             Some("base-first".to_string())
         );
-        assert_eq!(
-            config.outline_icons().expect("icons"),
-            Some("nerd".to_string())
-        );
+        assert_eq!(config.icons().expect("icons"), Some("nerd".to_string()));
         assert_eq!(
             config.diff_layout().expect("layout"),
             Some("split".to_string())
@@ -473,7 +477,7 @@ mod tests {
         assert_eq!(config.outline_width().expect("width"), None);
         assert_eq!(config.outline_mode().expect("mode"), None);
         assert_eq!(config.outline_order().expect("order"), None);
-        assert_eq!(config.outline_icons().expect("icons"), None);
+        assert_eq!(config.icons().expect("icons"), None);
         assert_eq!(config.diff_layout().expect("layout"), None);
         assert_eq!(config.diff_zoom().expect("zoom"), None);
     }