feat: complete v0.2.8 — all tasks done, roadmap updated for release

- All 39 Task Master tasks completed (10 deferred to future releases)
- ROADMAP.md rewritten: removed shipped v0.2.7 section, added all v0.2.8
  tasks organized by category (performance, text shaping, LSP, viewers,
  bug fixes), updated Known Issues with fixed items checked off
- Handover prompt updated for release readiness
- Key v0.2.8 features: rendered view perf (AST cache, viewport culling,
  block height cache), HarfRust text shaping, LSP Phases 1-2, image/PDF
  viewer tabs, background file loading, per-frame O(N) elimination,
  terminal CJK wide chars, Windows borderless fix, 10+ bug fixes

Made-with: Cursor

Author: OlaProeis

Cargo.lock

@@ -52,8 +52,8 @@ walkdir = "2"
 # Git integration (no SSH/HTTPS - we only need local repo status)
 git2 = { version = "0.19", default-features = false }
 
-# Icon loading + markdown image rendering (PNG, JPEG, GIF, WebP)
-image = { version = "0.25", default-features = false, features = ["png", "jpeg", "gif", "webp"] }
+# Icon loading + markdown image rendering (PNG, JPEG, GIF, WebP, BMP)
+image = { version = "0.25", default-features = false, features = ["png", "jpeg", "gif", "webp", "bmp"] }
 font-kit = "0.14.3"
 
 # Internationalization
@@ -97,6 +97,10 @@ poll-promise = { version = "0.3", optional = true }
 harfrust = "0.5.2"
 unicode-script = "0.5"
 unicode-segmentation = "1.11"
+unicode-width = "0.1"
+
+# PDF rendering (pure Rust, CPU-based)
+hayro = "0.5"
 
 # High-performance memory allocators (platform-specific)
 # mimalloc for Windows: faster allocation, reduced fragmentation

Cargo.toml

@@ -14,7 +14,7 @@ Rust (edition 2021) + egui 0.28 markdown editor. Immediate-mode GUI — no retai
 ## Tech Stack
 - **Language:** Rust 2021, egui 0.28 + eframe (immediate-mode GUI)
 - **Text:** ropey (rope buffer), comrak (Markdown AST), syntect (syntax highlighting), harfrust (OTL shaping)
-- **Terminal:** portable-pty + vte | **VCS:** git2 | **Dialogs:** rfd | **i18n:** rust-i18n | **Hashing:** blake3
+- **Terminal:** portable-pty + vte | **VCS:** git2 | **Dialogs:** rfd | **i18n:** rust-i18n | **Hashing:** blake3 | **PDF:** hayro
 - **Memory:** mimalloc (Windows), jemalloc (Unix)
 
 ## Architecture
@@ -63,6 +63,8 @@ fn process(text: &str) -> Vec<&str> { text.lines().collect() }
 - **State:** `Tab` for per-tab, `AppState` for global
 - **Errors:** User-facing via `show_toast()`, technical via `log::error!`
 - **Large files (>1MB):** Hash-based `is_modified()`, reduced undo stack (10 vs 100), no `original_bytes`
+- **Background file loading (≥5MB):** `open_file_smart()` on `FerriteApp` spawns background thread; `Tab.tab_content` (`TabContent::Loading`/`Ready`/`Error`) tracks state; `FileLoadMsg` channel polled in `update()`
+- **Per-frame caching:** `Tab.content_version` (u64) gates cached `is_modified()`, `text_stats()`, `needs_cjk_cached()`, `needs_complex_script_cached()` — never scan full content per frame
 
 ## Where Things Live (common)
 
@@ -76,6 +78,7 @@ fn process(text: &str) -> Vec<&str> { text.lines().collect() }
 | Modify markdown parsing | `markdown/parser.rs` |
 | Modify central panel | `app/central_panel.rs` |
 | Add special tab | `state.rs` → `SpecialTabKind`, `app/central_panel.rs` |
+| Add viewer tab | `state.rs` → `TabKind` variant + state struct, `app/central_panel.rs` → render method |
 | Add global/per-tab state | `state.rs` → `AppState` / `Tab` struct |
 | Add i18n string | `locales/en.yaml` + `t!("key")` |
 | Mermaid diagrams | `markdown/mermaid/` (flowchart has `types`, `parser`, `layout/`, `render/`) |

ROADMAP.md

@@ -15,69 +15,26 @@
 - Prefer Task Master MCP tools over CLI when available.
 - Use Context7 MCP when needed.
 
-## Current Task: 35 — LSP Server Activation Based on Open Files
-
-### Task Details
-| Field | Value |
-|-------|-------|
-| **ID** | 35 |
-| **Title** | LSP Server Activation Based on Open Files |
-| **Complexity** | 8 |
-| **Priority** | high |
-| **Dependencies** | Tasks 24, 25, 33 — all done |
-
-### Critical Note
-**This task is likely already complete.** Task 33 (On-Demand LSP Server Startup) implemented ALL functionality described in Task 35:
-- On-demand server start via `sync_active_doc_to_lsp()` in `src/app/file_ops.rs`
-- Per-server doc count tracking (`lsp_open_doc_count`) and tab-to-server mapping (`lsp_tab_server`)
-- 30-second idle shutdown via `check_lsp_idle_shutdown()`
-- `didClose` on tab close via `cleanup_tab_state()` in `src/app/mod.rs`
-- All edge cases (multi-tab same lang, rapid switching, crashes, unknown extensions)
-
-**Recommended action:** Review the implementation (see docs below), verify the test scenarios, and mark as done if satisfied.
-
-### Description
-Implement on-demand LSP server activation that detects which language servers are needed based on currently open file tabs, starting servers only when matching files are opened and stopping them after idle timeout when the last tab for a language is closed.
-
-### Implementation Details
-1. **Tab Activation Detection** (`src/app/file_ops.rs`):
-   - `sync_active_doc_to_lsp()` checks active tab's extension via `detect_lsp_server_for_path()`
-   - If server status is `Disconnected` → calls `start_lsp_server_on_demand()`
-   - If `Starting`/`Initializing` → waits; if `Ready` → sends `didOpen`/`didChange`
-
-2. **Server Lifecycle** (`src/app/file_ops.rs` + `src/app/mod.rs`):
-   - `lsp_open_doc_count: HashMap<String, usize>` tracks open docs per server
-   - `lsp_tab_server: HashMap<usize, (PathBuf, String)>` maps tab ID to (path, server_key)
-   - `cleanup_tab_state()` sends `didClose` on tab close and decrements count
-   - `check_lsp_idle_shutdown()` stops servers idle ≥ 30s
-
-3. **Extension Mapping** (`src/lsp/detection.rs`):
-   - `detect_lsp_server_for_path()` maps extensions to server specs
-   - User overrides applied via `lsp_server_overrides` in settings
-
-### Key Files
-| File | Purpose |
-|------|---------|
-| `src/app/file_ops.rs` | `sync_active_doc_to_lsp()`, `start_lsp_server_on_demand()`, `check_lsp_idle_shutdown()`, `handle_lsp_events()` |
-| `src/app/mod.rs` | `FerriteApp` fields (`lsp_tab_server`, `lsp_open_doc_count`, `lsp_idle_since`), `cleanup_tab_state()` |
-| `src/lsp/detection.rs` | `detect_lsp_server_for_path()` extension-to-server mapping |
-| `src/lsp/manager.rs` | `LspManager` — server spawn, lifecycle, crash recovery |
-| `src/state.rs` | `start_lsp_for_workspace()` (now no-op) |
-| `docs/technical/lsp/lsp-on-demand-startup.md` | Full feature documentation |
-
-### Test Strategy
-1. Fresh workspace: Open Rust tab → server spawns. Switch to Markdown tab → NO server spawn.
-2. Tab reactivation: Close Rust tab → idle timer starts. Reopen within 30s → no respawn.
-3. Idle shutdown: Close all Rust tabs → server shuts after 30s (log check).
-4. Multi-lang: Rust+Python tabs → 2 servers. Close all Rust → 1 server.
-5. Multiple tabs same language: Open two Rust tabs → one server. Close one → server stays. Close both → idle shutdown.
-6. Unknown extensions: Open .xyz file → no server, no errors.
-7. Crash recovery: Kill rust-analyzer → reactivate Rust tab → restarts.
-8. Status bar shows 'rust-analyzer: Starting…→Ready'. Toggle `lsp_enabled=false` → no spawn.
+## Current Status: v0.2.8 Release Ready
+
+### All Tasks Complete
+All 39 Task Master tasks are `done`. 10 tasks are `deferred`:
+- Tasks 11-17: Executable Code Blocks → deferred to v0.2.9+
+- Tasks 27-28: Alt-Key Menu Bar → deferred to v0.3.0
+- Task 38: eframe 0.28→0.31 upgrade → deferred to v0.2.9
+
+### Release Readiness
+- ROADMAP.md updated with all 49 tasks cross-referenced
+- v0.2.7 section removed from roadmap (in CHANGELOG.md)
+- Known Issues section updated (fixed items checked off)
+- All documentation in `docs/technical/` up to date
+- `docs/index.md` current
+
+### Suggested Next Steps
+1. **Cut v0.2.8 release** — all planned work complete; tag and build
+2. **Add new tasks** — Use `task-master add-task` or parse a new PRD for v0.2.9
+3. **Start Task 38** — If ready to attempt the egui 0.28→0.31 upgrade
 
 ## Verification
-Before starting:
+Before starting any new task:
 `cargo build`
-
-## Model Selection
-Task complexity 8 → **Default model**

docs/ai-context.md

@@ -50,6 +50,7 @@
 | [Cursor Position Mapping](./technical/editor/cursor-position-mapping.md) | Raw-to-displayed text position mapping for formatted content editing |
 | [Galley Cursor Positioning](./technical/editor/galley-cursor-positioning.md) | Pixel-accurate cursor placement using egui Galley text layout |
 | [Undo/Redo System](./technical/editor/undo-redo.md) | Per-tab undo/redo with keyboard shortcuts (Ctrl+Z, Ctrl+Y) |
+| [Undo Hash Change Detection](./technical/editor/undo-hash-change-detection.md) | Blake3 hash-based undo snapshot to eliminate per-frame content clones |
 | [Find and Replace](./technical/editor/find-replace.md) | Search functionality with regex, match highlighting, replace operations |
 | [Go to Line](./technical/editor/go-to-line.md) | Ctrl+G modal dialog for line navigation, viewport centering |
 | [Duplicate Line](./technical/editor/duplicate-line.md) | Ctrl+Shift+D line/selection duplication, char-to-byte index handling |
@@ -60,6 +61,8 @@
 | [Semantic Minimap](./technical/editor/semantic-minimap.md) | Semantic minimap with clickable heading labels, content type indicators, density bars |
 | [Editor Minimap (Legacy)](./technical/editor/minimap.md) | VS Code-style pixel minimap (replaced by semantic minimap) |
 | [Search Highlight](./technical/editor/search-highlight.md) | Search-in-files result navigation with transient highlight, auto Raw mode switch |
+| [Search Highlight Rendered View](./technical/editor/search-highlight-rendered-view.md) | Rendered view search highlights (incl. tables) and floating panel z-order stacking |
+| [Search Highlight Edit Recompute](./technical/editor/search-highlight-edit-recompute.md) | Fix stale search highlights after document edits by recomputing match positions |
 | [Syntax Highlighting](./technical/editor/syntax-highlighting.md) | Syntect integration for code block highlighting |
 | [Auto-close Brackets](./technical/editor/auto-close-brackets.md) | Auto-pair insertion, selection wrapping, skip-over behavior for brackets/quotes |
 | [Bracket Matching](./technical/editor/bracket-matching.md) | Highlight matching brackets and parentheses |
@@ -135,6 +138,8 @@
 | [Block-Level Height Cache](./technical/markdown/block-level-height-cache.md) | Per-block blake3-keyed LRU height cache for off-screen block measurement skip |
 | [Strict Line Breaks](./technical/markdown/strict-line-breaks.md) | Optional setting treating single newlines as hard `<br>` breaks |
 | [Lazy Block Height Estimation](./technical/markdown/lazy-block-height-estimation.md) | Heuristic heights for unmeasured blocks, render budget cap, progressive refinement |
+| [Paragraph Trailing Spaces](./technical/markdown/paragraph-trailing-spaces.md) | Fix for trailing spaces lost in plain paragraphs via persistent edit buffer |
+| [Rendered Paragraph Block Spacing](./technical/markdown/rendered-paragraph-block-spacing.md) | Trailing space after block paragraphs and code blocks; viewport height alignment |
 
 ### Data Viewers
 
@@ -146,7 +151,10 @@
 | [CSV Header Detection](./technical/viewers/csv-header-detection.md) | Auto-detect header rows with heuristics, toggle UI, column alignment |
 | [CSV Rainbow Columns](./technical/viewers/csv-rainbow-columns.md) | Subtle alternating column colors using Oklch, status bar toggle |
 | [CSV Raw View Caching](./technical/viewers/csv-raw-view-caching.md) | Blake3 hash-guarded raw text cache to eliminate per-frame string allocation |
+| [Image Viewer](./technical/viewers/image-viewer.md) | Dedicated image viewer tabs (PNG/JPEG/GIF/WebP/BMP) with zoom and metadata |
+| [PDF Viewer](./technical/viewers/pdf-viewer.md) | Read-only PDF viewer tabs with hayro rendering, page navigation, zoom |
 | [Tree Viewer](./technical/viewers/tree-viewer.md) | JSON/YAML/TOML tree viewer with inline editing, expand/collapse, path copying |
+| [Tree Viewer Caching](./technical/viewers/tree-viewer-caching.md) | Blake3-guarded parse cache and raw text buffer to avoid per-frame work |
 | [Live Pipeline](./technical/viewers/live-pipeline.md) | JSON/YAML command piping through shell commands (jq, yq), recent history |
 | [Document Export](./technical/viewers/document-export.md) | HTML export with themed CSS, Copy-as-HTML clipboard functionality |
 
@@ -171,6 +179,7 @@
 | [Terminal UI](./technical/terminal/terminal-ui.md) | Terminal panel with tabs, split panes, floating windows, drag-and-drop |
 | [Terminal Themes](./technical/terminal/terminal-themes.md) | Terminal color schemes (Solarized, Dracula, Monokai, Nord, etc.) |
 | [Terminal Layout](./technical/terminal/terminal-layout.md) | Split pane layouts (horizontal/vertical), grid creation, layout save/load |
+| [Terminal CJK Wide Chars](./technical/terminal/terminal-cjk-wide-chars.md) | Double-width CJK character rendering, cursor advancement, selection snapping |
 
 ### Productivity Hub
 
@@ -192,7 +201,8 @@
 | [Custom Title Bar](./technical/platform/custom-title-bar.md) | Windows-style custom title bar implementation |
 | [Window Resize](./technical/platform/window-resize.md) | Custom resize handles for borderless windows, edge detection |
 | [Windows Borderless Window](./technical/platform/windows-borderless-window.md) | Top edge resize fix, fullscreen toggle (F10), title bar button area exclusion |
-| [Windows Path Normalization](./technical/platform/windows-path-normalization.md) | Strip Windows `\\?\` prefix from canonicalized paths |
+| [Windows Borderless Transparent Fix](./technical/platform/windows-borderless-transparent-fix.md) | Fix rendering offset (black bars) on Intel GPUs via `with_transparent(true)` DWM workaround |
+|| [Windows Path Normalization](./technical/platform/windows-path-normalization.md) | Strip Windows `\\?\` prefix from canonicalized paths |
 | [Linux Cursor Flicker Fix](./technical/platform/linux-cursor-flicker-fix.md) | Title bar exclusion zone to prevent cursor conflicts with window controls |
 | **[Idle Mode Optimization](./technical/platform/idle-mode-optimization.md)** | **Tiered idle repaint system to reduce CPU usage on all platforms** |
 | **[SignPath Code Signing](./technical/platform/signpath-code-signing.md)** | **Windows code signing via SignPath for OSS** |
@@ -270,6 +280,13 @@
 | **[Mermaid Crate Plan](./mermaid-crate-plan.md)** | **Extract Mermaid renderer as standalone pure-Rust crate** |
 | **[Math Support Plan](./math-support-plan.md)** | **v0.4.0 planning: Native LaTeX/TeX math rendering (pure Rust)** |
 
+### Performance
+
+| Document | Description |
+|----------|-------------|
+| **[Per-Frame Cache Elimination](./technical/performance/per-frame-cache-elimination.md)** | **content_version-based caching to eliminate 7 O(N) per-frame operations for large files** |
+| [Background File Loading](./technical/performance/background-file-loading.md) | Background thread loading for 5MB+ files with progress bar, cancellation support |
+
 ### Core (Remaining)
 
 | Document | Description |

docs/current-handover-prompt.md

@@ -80,6 +80,38 @@ pub struct EditorOutput {
 }
 ```
 
+### Scroll Persistence & egui Widget IDs
+
+egui persists `ScrollArea` state (scroll offset) per widget `Id`. To prevent scroll positions leaking across tab switches, every central-panel editor/preview widget ID must be scoped with `tab.id`:
+
+```rust
+// ✅ DO: Scope widget ID with tab.id — each tab gets independent scroll state
+let editor_widget_id = egui::Id::new("main_editor_raw").with(tab.id);
+EditorWidget::new(tab)
+    .id(editor_widget_id)
+    .show(ui);
+
+// ✅ DO: MarkdownEditor can inline it (only borrows tab.content, not tab)
+MarkdownEditor::new(&mut tab.content)
+    .id(egui::Id::new("main_editor_rendered").with(tab.id))
+    .show(ui);
+
+// ❌ DON'T: Fixed ID leaks scroll offset between tabs
+EditorWidget::new(tab)
+    .id(egui::Id::new("main_editor_raw"))
+    .show(ui);
+```
+
+Note: For `EditorWidget`, capture the ID before the builder chain since `EditorWidget::new(tab)` takes a mutable borrow of the entire `Tab`, making `tab.id` inaccessible in the chain.
+
+The four scoped IDs in `src/app/central_panel.rs`:
+- `"main_editor_raw"` — Raw mode, single pane
+- `"main_editor_rendered"` — Rendered mode, single pane
+- `"split_editor_raw"` — Split view, left (raw) pane
+- `"split_preview_rendered"` — Split view, right (preview) pane
+
+`FerriteEditor` storage (raw editor) is separately keyed by `tab_id` in a `HashMap` inside `FerriteEditorStorage` (see `src/editor/widget.rs`), so its virtual scrolling is already per-tab independent of the widget ID.
+
 ## Dependencies Used
 
 - `egui` - TextEdit::multiline, ScrollArea, FontId

docs/index.md

@@ -0,0 +1,67 @@
+# Search Highlight Recomputation After Document Edits
+
+## Problem
+
+When the user edits a document while the Ctrl+F find panel is open, search highlight rectangles drift out of alignment with actual match positions. The yellow highlight boxes shift horizontally and vertically, no longer overlaying the correct text.
+
+## Root Cause
+
+`FindState.matches` stores search results as byte-position pairs `(start_byte, end_byte)` computed against the document text at search time. When the user inserts or deletes text (e.g., pressing Enter), byte positions of all subsequent matches shift, but `find_matches()` was only called when:
+
+1. The search term changed (debounced via `dialogs.rs`)
+2. The find panel was first opened (`find_replace.rs`)
+3. After replace operations
+
+No recomputation was triggered by document edits, so stale byte offsets were passed to the highlight renderer each frame.
+
+## Fix
+
+In `src/app/central_panel.rs`, after each editor's content-change detection (`editor_output.changed == true`), if the find panel is active with a non-empty search term, `find_matches()` is re-run against the current `tab.content`.
+
+### Affected View Modes
+
+| View Mode | Detection | Location |
+|-----------|-----------|----------|
+| Raw | `editor_output.changed` from `EditorWidget` | After tab borrow ends (~line 857) |
+| Split | `editor_output.changed` from left raw editor OR `md_editor_output.changed` from right rendered pane | After both panes render (~line 1740) |
+| Rendered | `editor_output.changed` from `MarkdownEditor` | After tab borrow ends (~line 1960) |
+
+### Pattern
+
+Each view mode uses a flag variable to capture the change signal inside the mutable `tab` borrow scope, then recomputes after the borrow ends:
+
+```rust
+let mut content_changed_in_editor = false;
+if let Some(tab) = self.state.active_tab_mut() {
+    // ... editor rendering ...
+    if editor_output.changed {
+        content_changed_in_editor = true;
+    }
+}
+// After tab borrow ends:
+if content_changed_in_editor
+    && self.state.ui.show_find_replace
+    && !self.state.ui.find_state.search_term.is_empty()
+{
+    if let Some(content) = self.state.active_tab().map(|t| t.content.clone()) {
+        self.state.ui.find_state.find_matches(&content);
+    }
+}
+```
+
+## Performance
+
+- **Only runs on actual edits** — gated by `editor_output.changed`, not per-frame
+- **O(N) text clone + regex search** — same cost as the existing debounced search path
+- `find_matches()` uses compiled regex internally, which is efficient for typical document sizes
+- For 100K+ line files, the search is still user-initiated (typing causes the edit), keeping within the O(N) user-initiated tier from the performance rules
+
+## Key Files
+
+| File | Role |
+|------|------|
+| `src/app/central_panel.rs` | Recomputation trigger after edit detection in all 3 view modes |
+| `src/editor/find_replace.rs` | `FindState::find_matches()` — the search engine |
+| `src/editor/widget.rs` | `SearchHighlights` struct passed to `FerriteEditor` |
+| `src/editor/ferrite/search.rs` | `set_search_matches()` — converts byte ranges to `SearchMatch` with pre-computed line numbers |
+| `src/editor/ferrite/highlights.rs` | `render_search_highlights()` — paints highlight rectangles |