@fresh-editor/fresh-editor 0.2.25 → 0.3.0

package/CHANGELOG.md

@@ -1,9 +1,161 @@
 # Release Notes
 
+## 0.3.0
+
+This version brings major features and many quality-of-life improvements and bug fixes:
+
+- A cool dashboard plugin
+- Devcontainer support
+- init.ts
+
+And more (see below). A large version is more likely to contain regression bugs, so please bear with me if you encounter problems, and open github issues without hesitation.
+
+### Features
+
+* **Dashboard plugin**: Built-in TUI dashboard that replaces the usual "[No Name]" with useful at-a-glance info.
+    - Default widgets: git status + repo URL, a "vs master" row (commits ahead/behind), and disk usage for common mounts.
+    - Opt-in widgets: weather, and open GitHub PRs for the current repo.
+    - Auto-open (on startup / last-buffer-close) is configurable — e.g. `editor.getPluginApi("dashboard")?.setAutoOpen(false)` in `init.ts`. When off, use the "Show Dashboard" command in the palette.
+    - Third-party plugins and `init.ts` can contribute their own rows via the `registerSection()` API. The `init.ts` starter template includes ready-to-paste snippets for enabling the opt-in widgets, toggling auto-open, and registering custom sections (see below).
+
+* **Devcontainer support** (thanks @masak1yu!): Fresh integrates with the [devcontainer CLI](https://github.com/devcontainers/cli) (install it yourself).
+    - Detects `.devcontainer/devcontainer.json` and offers Attach / Rebuild / Detach.
+    - Embedded terminal, filesystem, and LSP servers all run inside the devcontainer.
+    - `Dev Container: Create Config` scaffolds a config for projects that don't have one.
+    - `Dev Container: Show Ports` merges configured `forwardPorts` with live `docker port` output.
+    - `Dev Container: Show Logs` captures the container's recent stdout/stderr.
+    - Build log streams into a workspace split; failed attaches offer Retry / Show Logs / Detach via a recovery popup.
+    - `initializeCommand` runs on attach.
+
+* **`init.ts`**: Fresh now auto-loads `~/.config/fresh/init.ts`! Allows you to run plugin code on startup, which complements the purely declarative config system with imperative, environment-aware logic. Use command palette `init: Edit` to generate a template with some examples. Use `init: Reload` to run it after editing. Use `--no-init` / `--safe` to skip loading.  
+    - Tip: *Enable LSP* when editing `init.ts` to get help and completions. 
+    - Example (for the Dashboard plugin):
+    ```typescript
+    // in your init.ts file:
+    const dash = editor.getPluginApi("dashboard");
+    if (dash) {
+      dash.registerSection("env", async (ctx) => {
+        ctx.kv("USER", editor.getEnv("USER") || "?");
+      });
+    }
+    ```
+    Will add a line like this to your dashboard:
+    ```
+    │ ▎  ENV                              │
+    │    USER      someone                │
+    ```
+
+* **`{remote}` status-bar indicator**: Clickable status-bar element that lights up when you're attached to an SSH remote or devcontainer, with a context-aware menu (detach, show logs, retry attach, …). Surfaces `Connecting` / `Connected` / `FailedAttach` states. Fresh's config v1→v2 migration injects `{remote}` into customized `status_bar.left`.
+
+* **Hot-exit restore split from session restore**: `editor.restore_previous_session` config and the `--no-restore` / `--restore` CLI flags now control workspace/tab restoration separately from hot-exit content — unsaved scratch buffers come back even when you opt out of full session restore (#1404).
+
+* **File explorer — cut/copy/paste + multi-selection + right-click context menu** (thanks @theogravity!):
+    - `Ctrl+C` / `Ctrl+X` / `Ctrl+V` with same-dir auto-rename and per-file conflict prompt on cross-dir paste.
+    - `Shift+Up/Down` for multi-select.
+    - Right-click context menu (#1684) with the usual file operations, honoring the active multi-selection.
+    - Cut-pending items are dimmed until pasted; cancel a pending cut with Escape or by pasting back into the same directory.
+    - Renaming a file or directory relocates any open buffers inside it; deleting a file closes its buffer.
+
+* **File explorer — keyboard preview**: Moving the cursor with Up/Down in the explorer previews the highlighted file in a preview tab (#1570), so you can scan files without leaving the keyboard.
+
+* **Quick Open / Go-to Line live preview**: Typing `:<N>` in the file finder (or in the standalone `:` mode) scrolls the cursor to the target line live as you type; Enter commits, Escape reverts, mouse movement or clicks also commit.
+
+* **Terminal shell override (#1637)**: New `terminal.shell` config option lets you pick a different shell for the integrated terminal without reassigning `$SHELL` (which affects `format_on_save` and other features).
+
+* **Suspend process (Unix)**: New `Suspend Process` action sends Fresh to the background like Ctrl+Z in a shell. Routed through the client in session mode so the server stays up.
+
+* **Current-column highlight**: New `highlight_current_column` / `Toggle Current Column Highlight` — highlights the cursor's column for alignment work.
+
+* **Post-EOF shading** (#779): Rows past end-of-file render with a distinct background so the boundary is obvious; works alongside `show_tilde`.
+
+* **Regex replacement escapes**: `\n`, `\t`, `\r`, and `\\` in the replacement string are now interpreted when regex mode is on.
+
+### Improvements
+
+* **SSH URLs on the CLI**: `fresh ssh://user@host:port/path` launches a session whose filesystem and process authority point at the remote host.
+
+* **Redraw Screen command** (#1070): Added a `redraw_screen` action and palette entry that clears the terminal and fully repaints the UI, useful when an external program scribbles over the TUI.
+
+* **Terminal window title** (#1618): Fresh sets the terminal window title from the active buffer's filename, matching other editors.
+
+* **LSP status popup upgrades**: LSP popup now shows better options for enabling/disabling the nudge.
+
+* **Find Next centers vertically** (#1251): When the next match is off-screen, scroll it to roughly the middle of the viewport so you keep context above and below it. Matches that are already visible are not re-scrolled.
+
+* **Adaptive line-number gutter** (#1204): The gutter now grows with the buffer's line count rather than reserving 4 digits by default — a small file reclaims 2–3 columns of editor width.
+
+* **File explorer width in percent or columns** (#1118, #1212, #1213): `file_explorer.width` now accepts `"30%"` (percent of terminal width) or `"24"` (absolute columns). Dragging the divider preserves whichever form you configured. Legacy integer/fraction values keep working.
+
+* **Relative paths to theme files** (#1621): User themes in `config.json` can be spelled out as relative to your themes directory:
+  - "dark" or "builtin://dark" — any built-in by name
+  - "my-theme.json" or "subdir/dark.json" — nested relative path in your user themes dir - useful for sharing Fresh config.json in a dotfiles repo
+  - "file://${HOME}/themes/x.json" — absolute path; ${HOME}, ${XDG_CONFIG_HOME} are expanded
+  - "https://github.com/foo/themes#dark" — URL-packaged theme
+
+* **Plugin API additions**:
+    - `editor.overrideThemeColors(...)` for in-memory theme mutation.
+    - `editor.parseJsonc(...)` for host-side JSONC parsing.
+    - Plugin-created terminals now have an ephemeral lifetime — they close cleanly when the action that spawned them finishes.
+    - Plugin authors can augment `FreshPluginRegistry` to make `editor.getPluginApi("name")` return a typed interface (no `as`-cast needed). Augmentations are emitted to `~/.config/fresh/types/plugins.d.ts` at load time.
+    - `spawnHostProcess` now returns a handle with `kill()` (and a matching `KillHostProcess` command).
+    - `BufferInfo.splits` surfaces which splits display a buffer, for "focus-if-visible" dedupe.
+    - `editor.setRemoteIndicatorState(...)` / `clearRemoteIndicatorState()` let remote plugins drive the status-bar `{remote}` element.
+    - Dashboard gains `dash.registerSection()` (with a returned remover) and `dash.clearAllSections()` for plugin extension.
+
+* **JSONC language**: `.jsonc` files and well-known JSONC-with-`.json`-suffix files (`devcontainer.json`, `tsconfig.json`, `.eslintrc.json`, `.babelrc`, VS Code settings files) now get a dedicated `jsonc` language with comment-tolerant highlighting and LSP routing through `vscode-json-language-server` with the correct `languageId`.
+
+* **macOS Alt+Right / Option+Right stops at word end** (#1288): Selection no longer extends past trailing whitespace, matching TextEdit / VS Code on Mac.
+
+### Bug Fixes
+
+* **File Explorer `.gitignore` improvements** (#1388): Files are now visible only if they aren't hidden by ANY of the filters (hidden files, `.gitignore` files). Also, File Explorer will do a better job of auto-reloading when `.gitignore` changes.
+
+* **Scrollbar theme colours** (#1554): The scrollbar now honours `theme.scrollbar_track_fg` / `scrollbar_thumb_fg`. A few themes were updated to define this missing value.
+
+* **Fixed panic when clicking split + terminal** (#1620).
+
+* **Fixed LSP server crash loop** (#1612): When LSP fails on startup, restart bypassed the normal restart count limiter, now fixed.
+
+* **Fixed Markdown preview/compose wrapping when File Explorer is open**: When compose width was set (e.g. 80), opening the File Explorer sidebar pushed tables off the right edge. Separator rows no longer overflow when table cells are truncated.
+
+* **More settings propagate live**: File-explorer width and flag changes made in the Settings UI apply immediately on save, without a restart.
+
+* **Devcontainer: no re-prompt after restart**: Fresh no longer shows the "Attach?" prompt again after the post-attach self-restart.
+
+* **Dashboard polish**: Doesn't steal focus from a CLI-supplied file, underline only on clickable spans (not trailing padding), clicks dispatch only from underlined column ranges, immediate repaint on split resize.
+
+* **Quieter LSP**: Suppress `MethodNotFound` errors from LSP servers (#1649) — servers that don't implement an optional method no longer spam the log.
+
+* **Plugin action popups survive buffer switches**: Popups stay visible when the active buffer changes, and concurrent popups queue LIFO so the newest shows first.
+
+* **Encoding detection on CJK files** (#1635): Files whose only non-ASCII bytes sat past the 8 KB sample window were mis-detected; the sample boundary is now treated as truncation so the full file is considered before the encoding is guessed.
+
+* **Review diff — no fold jitter**: Toggling a fold no longer re-centers the viewport.
+
+* **LSP — cleaner disables**: No spurious warning when opening a file for a language whose LSP is explicitly disabled in config. The indicator shows buffer-skip state (e.g. file too large) instead of a generic warning.
+
+* **Windows — preserve UNC paths**: `pathJoin` plugin API now preserves `\\?\` UNC prefixes on Windows.
+
+* **Hardware cursor no longer bleeds through popups**: The terminal hardware cursor is hidden when an overlay popup covers it.
+
+* **Focus — tab clicks reset explorer context** (#1540): Clicking a tab or buffer no longer leaves the FileExplorer key context active.
+
+* **File explorer poll fixes**: Background refresh no longer collapses folders you've expanded, and resets the cursor to the root only when the selected path is genuinely gone.
+
+* **Review PR Branch — default-branch detection**: The prompt now pre-fills the repo's actual default branch (via `origin/HEAD`) instead of hard-coding `main`.
+
+* **Review: PageUp/PageDown**: Paging in review-branch mode now scrolls the commit list instead of moving the cursor by one row.
+
+### Under the Hood
+
+* **Authority abstraction**: Filesystem, process-spawning, and LSP routing are now consolidated behind a single `Authority` slot, with plugin ops (`editor.setAuthority` / `clearAuthority` / `spawnHostProcess`) for plugins that want to target the host even while attached elsewhere. This is what makes the devcontainer and `ssh://` flows work uniformly.
+
 ## 0.2.25
 
 ### Improvements
 
+* **Redraw Screen command** (#1070): Added a "Redraw Screen" entry to the command palette (action `redraw_screen`) that clears the terminal and fully repaints the UI. Useful when an external program (e.g. a macOS pasteboard diagnostic leaked by the host terminal on Ctrl+C) scribbles over the TUI and leaves ghost text behind.
+
 * **PageUp/PageDown in wrapped buffers**: Page motion is now view-row-aware, so paging through heavily wrapped text no longer stalls mid-buffer and the cursor stays visible after every press. Each page also keeps 3 rows of overlap with the previous page (matching vim / less) so you don't lose context across the jump.
 
 * **Smarter char-wrapping of long tokens**: When a token has to be split mid-word because it doesn't fit on a fresh line, the break now prefers a UAX #29 word boundary within a lookback window instead of an arbitrary grapheme position — e.g. `dialog.getButton(DialogInterface.BUTTON_NEUTRAL).setOnClickListener` now wraps after `BUTTON_NEUTRAL` rather than mid-identifier.

package/README.md

@@ -187,6 +187,12 @@ See [flatpak/README.md](flatpak/README.md) for building from source.
 
 Download the latest release for your platform from the [releases page](https://github.com/sinelaw/fresh/releases).
 
+### Using mise
+
+```bash
+mise use github:sinelaw/fresh
+```
+
 ### npm
 
 ```bash

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fresh-editor/fresh-editor",
-  "version": "0.2.25",
+  "version": "0.3.0",
   "description": "A modern terminal-based text editor with plugin support",
   "repository": {
     "type": "git",

package/plugins/audit_mode.i18n.json

@@ -14,7 +14,7 @@
     "cmd.refresh_review_branch_desc": "Re-fetch the commit list for the current base ref",
     "cmd.review_range": "Review Range (Commit or Branch)",
     "cmd.review_range_desc": "Review a commit or branch as a flattened diff (e.g. main..HEAD, HEAD~3..HEAD, or a single commit SHA)",
-    "prompt.branch_base": "Base ref to compare against (default: main):",
+    "prompt.branch_base": "Base ref to compare against (default: %{default}):",
     "prompt.review_range": "Review (range A..B or commit SHA): ",
     "status.review_range_empty": "No changes in %{range}",
     "status.review_branch_ready": "Reviewing %{count} commits in %{base}..HEAD",
@@ -92,7 +92,7 @@
     "cmd.review_range_desc": "Zrevidovat commit nebo větev jako sloučený rozdíl (např. main..HEAD, HEAD~3..HEAD nebo hash jednoho commitu)",
     "prompt.review_range": "Revize (rozsah A..B nebo SHA commitu): ",
     "status.review_range_empty": "Žádné změny v %{range}",
-    "prompt.branch_base": "Základní ref pro porovnání (výchozí: main):",
+    "prompt.branch_base": "Základní ref pro porovnání (výchozí: %{default}):",
     "status.review_branch_ready": "Kontroluje se %{count} commitů v %{base}..HEAD",
     "status.review_branch_empty": "Žádné commity v %{base}..HEAD — není co revidovat.",
     "panel.review_branch_header": "Commity (%{base}..HEAD)",
@@ -168,7 +168,7 @@
     "cmd.review_range_desc": "Einen Commit oder Branch als zusammengeführten Diff prüfen (z. B. main..HEAD, HEAD~3..HEAD oder ein einzelner Commit-SHA)",
     "prompt.review_range": "Prüfen (Bereich A..B oder Commit-SHA): ",
     "status.review_range_empty": "Keine Änderungen in %{range}",
-    "prompt.branch_base": "Basis-Ref zum Vergleichen (Standard: main):",
+    "prompt.branch_base": "Basis-Ref zum Vergleichen (Standard: %{default}):",
     "status.review_branch_ready": "%{count} Commits in %{base}..HEAD werden überprüft",
     "status.review_branch_empty": "Keine Commits in %{base}..HEAD — nichts zu überprüfen.",
     "panel.review_branch_header": "Commits (%{base}..HEAD)",
@@ -244,7 +244,7 @@
     "cmd.review_range_desc": "Revisar un commit o una rama como un diff aplanado (p. ej. main..HEAD, HEAD~3..HEAD o un SHA de commit)",
     "prompt.review_range": "Revisar (rango A..B o SHA de commit): ",
     "status.review_range_empty": "Sin cambios en %{range}",
-    "prompt.branch_base": "Ref base para comparar (por defecto: main):",
+    "prompt.branch_base": "Ref base para comparar (por defecto: %{default}):",
     "status.review_branch_ready": "Revisando %{count} commits en %{base}..HEAD",
     "status.review_branch_empty": "No hay commits en %{base}..HEAD — nada que revisar.",
     "panel.review_branch_header": "Commits (%{base}..HEAD)",
@@ -320,7 +320,7 @@
     "cmd.review_range_desc": "Revoir un commit ou une branche comme un diff aplati (par ex. main..HEAD, HEAD~3..HEAD ou un SHA de commit)",
     "prompt.review_range": "Revoir (plage A..B ou SHA de commit) : ",
     "status.review_range_empty": "Aucun changement dans %{range}",
-    "prompt.branch_base": "Ref de base pour la comparaison (par défaut : main) :",
+    "prompt.branch_base": "Ref de base pour la comparaison (par défaut : %{default}) :",
     "status.review_branch_ready": "Revue de %{count} commits dans %{base}..HEAD",
     "status.review_branch_empty": "Aucun commit dans %{base}..HEAD — rien à revoir.",
     "panel.review_branch_header": "Commits (%{base}..HEAD)",
@@ -396,7 +396,7 @@
     "cmd.review_range_desc": "Revisiona un commit o un ramo come un diff appiattito (es. main..HEAD, HEAD~3..HEAD o uno SHA di commit)",
     "prompt.review_range": "Revisiona (intervallo A..B o SHA di commit): ",
     "status.review_range_empty": "Nessuna modifica in %{range}",
-    "prompt.branch_base": "Ref base per il confronto (predefinito: main):",
+    "prompt.branch_base": "Ref base per il confronto (predefinito: %{default}):",
     "status.review_branch_ready": "Revisione di %{count} commit in %{base}..HEAD",
     "status.review_branch_empty": "Nessun commit in %{base}..HEAD — niente da revisionare.",
     "panel.review_branch_header": "Commit (%{base}..HEAD)",
@@ -472,7 +472,7 @@
     "cmd.review_range_desc": "コミットまたはブランチを平坦化した diff としてレビュー (例: main..HEAD、HEAD~3..HEAD、または単一コミットの SHA)",
     "prompt.review_range": "レビュー (範囲 A..B またはコミット SHA): ",
     "status.review_range_empty": "%{range} に変更はありません",
-    "prompt.branch_base": "比較対象のベース ref (デフォルト: main):",
+    "prompt.branch_base": "比較対象のベース ref (デフォルト: %{default}):",
     "status.review_branch_ready": "%{base}..HEAD の %{count} 件のコミットをレビュー中",
     "status.review_branch_empty": "%{base}..HEAD にコミットはありません — レビューする対象がありません。",
     "panel.review_branch_header": "コミット (%{base}..HEAD)",
@@ -548,7 +548,7 @@
     "cmd.review_range_desc": "커밋이나 브랜치를 평탄화된 diff로 리뷰 (예: main..HEAD, HEAD~3..HEAD 또는 단일 커밋 SHA)",
     "prompt.review_range": "리뷰 (범위 A..B 또는 커밋 SHA): ",
     "status.review_range_empty": "%{range}에 변경 사항 없음",
-    "prompt.branch_base": "비교할 기준 ref (기본값: main):",
+    "prompt.branch_base": "비교할 기준 ref (기본값: %{default}):",
     "status.review_branch_ready": "%{base}..HEAD의 %{count}개 커밋을 리뷰 중",
     "status.review_branch_empty": "%{base}..HEAD에 커밋이 없습니다 — 리뷰할 내용이 없습니다.",
     "panel.review_branch_header": "커밋 (%{base}..HEAD)",
@@ -624,7 +624,7 @@
     "cmd.review_range_desc": "Revisar um commit ou uma branch como um diff achatado (ex.: main..HEAD, HEAD~3..HEAD ou um SHA de commit)",
     "prompt.review_range": "Revisar (intervalo A..B ou SHA de commit): ",
     "status.review_range_empty": "Sem alterações em %{range}",
-    "prompt.branch_base": "Ref base para comparação (padrão: main):",
+    "prompt.branch_base": "Ref base para comparação (padrão: %{default}):",
     "status.review_branch_ready": "Revisando %{count} commits em %{base}..HEAD",
     "status.review_branch_empty": "Sem commits em %{base}..HEAD — nada para revisar.",
     "panel.review_branch_header": "Commits (%{base}..HEAD)",
@@ -700,7 +700,7 @@
     "cmd.review_range_desc": "Ревью коммита или ветки как плоского diff (например, main..HEAD, HEAD~3..HEAD или SHA одного коммита)",
     "prompt.review_range": "Ревью (диапазон A..B или SHA коммита): ",
     "status.review_range_empty": "Нет изменений в %{range}",
-    "prompt.branch_base": "Базовый ref для сравнения (по умолчанию: main):",
+    "prompt.branch_base": "Базовый ref для сравнения (по умолчанию: %{default}):",
     "status.review_branch_ready": "Ревью %{count} коммитов в %{base}..HEAD",
     "status.review_branch_empty": "Нет коммитов в %{base}..HEAD — нечего просматривать.",
     "panel.review_branch_header": "Коммиты (%{base}..HEAD)",
@@ -776,7 +776,7 @@
     "cmd.review_range_desc": "ตรวจสอบคอมมิตหรือสาขาเป็น diff แบบรวม (เช่น main..HEAD, HEAD~3..HEAD หรือ SHA ของคอมมิตเดียว)",
     "prompt.review_range": "ตรวจสอบ (ช่วง A..B หรือ SHA ของคอมมิต): ",
     "status.review_range_empty": "ไม่มีการเปลี่ยนแปลงใน %{range}",
-    "prompt.branch_base": "ref ฐานสำหรับเปรียบเทียบ (ค่าเริ่มต้น: main):",
+    "prompt.branch_base": "ref ฐานสำหรับเปรียบเทียบ (ค่าเริ่มต้น: %{default}):",
     "status.review_branch_ready": "กำลังตรวจสอบคอมมิต %{count} รายการใน %{base}..HEAD",
     "status.review_branch_empty": "ไม่มีคอมมิตใน %{base}..HEAD — ไม่มีอะไรต้องตรวจสอบ",
     "panel.review_branch_header": "คอมมิต (%{base}..HEAD)",
@@ -852,7 +852,7 @@
     "cmd.review_range_desc": "Рев'ю коміта або гілки як плаского diff (наприклад main..HEAD, HEAD~3..HEAD або SHA одного коміта)",
     "prompt.review_range": "Рев'ю (діапазон A..B або SHA коміта): ",
     "status.review_range_empty": "Немає змін у %{range}",
-    "prompt.branch_base": "Базовий ref для порівняння (за замовчуванням: main):",
+    "prompt.branch_base": "Базовий ref для порівняння (за замовчуванням: %{default}):",
     "status.review_branch_ready": "Рев'ю %{count} комітів у %{base}..HEAD",
     "status.review_branch_empty": "Немає комітів у %{base}..HEAD — нічого переглядати.",
     "panel.review_branch_header": "Коміти (%{base}..HEAD)",
@@ -928,7 +928,7 @@
     "cmd.review_range_desc": "Xem xét một commit hoặc nhánh dưới dạng diff đã gộp (ví dụ main..HEAD, HEAD~3..HEAD hoặc SHA của một commit)",
     "prompt.review_range": "Xem xét (khoảng A..B hoặc SHA commit): ",
     "status.review_range_empty": "Không có thay đổi trong %{range}",
-    "prompt.branch_base": "Ref cơ sở để so sánh (mặc định: main):",
+    "prompt.branch_base": "Ref cơ sở để so sánh (mặc định: %{default}):",
     "status.review_branch_ready": "Đang xem xét %{count} commit trong %{base}..HEAD",
     "status.review_branch_empty": "Không có commit nào trong %{base}..HEAD — không có gì để xem xét.",
     "panel.review_branch_header": "Commit (%{base}..HEAD)",
@@ -1004,7 +1004,7 @@
     "cmd.review_range_desc": "将提交或分支作为合并后的 diff 进行审查（例如 main..HEAD、HEAD~3..HEAD 或单个提交的 SHA）",
     "prompt.review_range": "审查（范围 A..B 或提交 SHA）：",
     "status.review_range_empty": "%{range} 中没有更改",
-    "prompt.branch_base": "用于比较的基础 ref（默认：main）：",
+    "prompt.branch_base": "用于比较的基础 ref（默认：%{default}）：",
     "status.review_branch_ready": "正在审查 %{base}..HEAD 中的 %{count} 个提交",
     "status.review_branch_empty": "%{base}..HEAD 中没有提交 — 无需审查。",
     "panel.review_branch_header": "提交 (%{base}..HEAD)",

package/plugins/audit_mode.ts

@@ -1579,7 +1579,7 @@ function on_review_mouse_click(data: {
                 else state.collapsedSections.add(cat);
                 applyFolds();
                 const sectionRow = state.sectionHeaderRows[cat];
-                if (sectionRow !== undefined) jumpDiffCursorToRow(sectionRow);
+                if (sectionRow !== undefined) jumpDiffCursorToRow(sectionRow, { recenter: false });
                 return;
             }
         }
@@ -1591,7 +1591,7 @@ function on_review_mouse_click(data: {
                 else state.collapsedFiles.add(key);
                 applyFolds();
                 const headerRow = state.fileHeaderRows[key];
-                if (headerRow !== undefined) jumpDiffCursorToRow(headerRow);
+                if (headerRow !== undefined) jumpDiffCursorToRow(headerRow, { recenter: false });
                 return;
             }
         }
@@ -1602,7 +1602,7 @@ function on_review_mouse_click(data: {
                 else state.collapsedHunks.add(hunkId);
                 applyFolds();
                 const hunkRow = state.hunkRowByHunkId[hunkId];
-                if (hunkRow !== undefined) jumpDiffCursorToRow(hunkRow);
+                if (hunkRow !== undefined) jumpDiffCursorToRow(hunkRow, { recenter: false });
                 return;
             }
         }
@@ -1812,7 +1812,7 @@ function review_toggle_file_collapse() {
         else state.collapsedSections.add(section);
         applyFolds();
         const sectionRow = state.sectionHeaderRows[section];
-        if (sectionRow !== undefined) jumpDiffCursorToRow(sectionRow);
+        if (sectionRow !== undefined) jumpDiffCursorToRow(sectionRow, { recenter: false });
         return;
     }
 
@@ -1824,7 +1824,7 @@ function review_toggle_file_collapse() {
         else state.collapsedFiles.add(key);
         applyFolds();
         const headerRow = state.fileHeaderRows[key];
-        if (headerRow !== undefined) jumpDiffCursorToRow(headerRow);
+        if (headerRow !== undefined) jumpDiffCursorToRow(headerRow, { recenter: false });
         return;
     }
 
@@ -1835,7 +1835,7 @@ function review_toggle_file_collapse() {
         else state.collapsedHunks.add(hunk.id);
         applyFolds();
         const hunkRow = state.hunkRowByHunkId[hunk.id];
-        if (hunkRow !== undefined) jumpDiffCursorToRow(hunkRow);
+        if (hunkRow !== undefined) jumpDiffCursorToRow(hunkRow, { recenter: false });
         return;
     }
 
@@ -1848,7 +1848,7 @@ function review_toggle_file_collapse() {
     else state.collapsedFiles.add(key);
     applyFolds();
     const headerRow = state.fileHeaderRows[key];
-    if (headerRow !== undefined) jumpDiffCursorToRow(headerRow);
+    if (headerRow !== undefined) jumpDiffCursorToRow(headerRow, { recenter: false });
 }
 registerHandler("review_toggle_file_collapse", review_toggle_file_collapse);
 
@@ -3147,23 +3147,31 @@ registerHandler("review_drill_down", review_drill_down);
 // --- Hunk navigation for side-by-side diff view ---
 
 /**
- * Move the diff panel's native cursor to the given 1-indexed row, scrolling
- * the viewport so the row is visible.
+ * Move the diff panel's native cursor to the given 1-indexed row.
+ *
+ * `options.recenter` controls whether the viewport is re-centered on the
+ * target row. The default is `true` for user-initiated navigation (next
+ * hunk, jump-to-comment, jump-to-file) — there the caller wants the
+ * target to land at a predictable position in the viewport. Callers
+ * that merely re-anchor the cursor to a nearby header (e.g. after a
+ * collapse/expand toggle) should pass `recenter: false` so the viewport
+ * stays put; `setBufferCursor` still runs `ensure_cursor_visible`, so
+ * the cursor is scrolled into view only when it would otherwise move
+ * off-screen. Without this opt-out every fold toggle re-centers the
+ * cursor's row at ~1/3 from the top of the viewport, which makes the
+ * diff jump around whenever the user is reading anywhere else.
  */
-function jumpDiffCursorToRow(row: number): void {
+function jumpDiffCursorToRow(row: number, options?: { recenter?: boolean }): void {
     const diffId = state.panelBuffers["diff"];
     if (diffId === undefined) return;
     const idx = row - 1;
     if (idx < 0 || idx >= state.diffLineByteOffsets.length) return;
 
-    // Set the cursor by absolute byte offset + scroll the viewport.
-    // Trust setBufferCursor — the previous N × executeAction("move_down")
-    // walk was O(target_row) round-trips into the editor and made
-    // collapsing big diffs visibly slow (thousands of round trips just
-    // to land the cursor on a header).
     const byteOffset = state.diffLineByteOffsets[idx];
     editor.setBufferCursor(diffId, byteOffset);
-    editor.scrollBufferToLine(diffId, idx);
+    if (options?.recenter !== false) {
+        editor.scrollBufferToLine(diffId, idx);
+    }
     state.diffCursorRow = row;
     applyCursorLineOverlay('diff');
     refreshStickyHeader(idx);
@@ -4388,12 +4396,47 @@ const branchState: ReviewBranchState = {
     detailBufferId: null,
     commits: [],
     selectedIndex: 0,
-    baseRef: "main",
+    // Empty means "not yet detected"; start_review_branch fills this in
+    // from the repo's actual default branch (main, master, or whatever
+    // origin/HEAD points at) before showing the prompt.
+    baseRef: "",
     detailCache: null,
     pendingDetailId: 0,
     logRowByteOffsets: [],
 };
 
+/**
+ * Best-effort detection of the repo's default branch. Checks, in order:
+ *   1. `origin/HEAD` (the remote's notion of the default branch)
+ *   2. local `main`
+ *   3. local `master`
+ * Falls back to `main` if none match, so the prompt still has a sensible
+ * default in an empty / unusual repo.
+ */
+async function detectDefaultBranch(): Promise<string> {
+    try {
+        const r = await editor.spawnProcess("git", [
+            "symbolic-ref", "--short", "refs/remotes/origin/HEAD",
+        ]);
+        if (r.exit_code === 0) {
+            const name = r.stdout.trim();
+            // Output looks like "origin/main"; strip the remote prefix.
+            const slash = name.indexOf("/");
+            const branch = slash >= 0 ? name.slice(slash + 1) : name;
+            if (branch) return branch;
+        }
+    } catch { /* fall through */ }
+    for (const candidate of ["main", "master"]) {
+        try {
+            const r = await editor.spawnProcess("git", [
+                "show-ref", "--verify", "--quiet", `refs/heads/${candidate}`,
+            ]);
+            if (r.exit_code === 0) return candidate;
+        } catch { /* fall through */ }
+    }
+    return "main";
+}
+
 // UTF-8 byte length helper, local copy so audit_mode doesn't pull in the one
 // from git_history (keeps the import list tiny).
 function branchUtf8Len(s: string): number {
@@ -4501,16 +4544,20 @@ async function start_review_branch(): Promise<void> {
         return;
     }
     // Prompt for the base ref so the user can review any PR, not just
-    // one branched off main.
-    const input = await editor.prompt(
-        editor.t("prompt.branch_base") || "Base ref (default: main):",
-        branchState.baseRef,
-    );
+    // one branched off main. The default offered is either what the user
+    // picked last time in this session, or the repo's actual default
+    // branch (main/master/etc.) on first use.
+    const suggested = branchState.baseRef || await detectDefaultBranch();
+    const rawPromptText = editor.t("prompt.branch_base", { default: suggested });
+    const promptText = (rawPromptText && !rawPromptText.startsWith("prompt."))
+        ? rawPromptText
+        : `Base ref to compare against (default: ${suggested}):`;
+    const input = await editor.prompt(promptText + " ", suggested);
     if (input === null) {
         editor.setStatus(editor.t("status.cancelled") || "Cancelled");
         return;
     }
-    const base = input.trim() || "main";
+    const base = input.trim() || suggested;
     branchState.baseRef = base;
 
     editor.setStatus(editor.t("status.loading") || "Loading commits…");
@@ -4637,17 +4684,11 @@ registerHandler("on_review_branch_cursor_moved", on_review_branch_cursor_moved);
 editor.defineMode(
     "review-branch",
     [
-        // Mode bindings replace globals, so we re-bind the editor's built-in
-        // motion actions here explicitly — without this, j/k and Up/Down
-        // do nothing in the commit list.
-        ["Up", "move_up"],
-        ["Down", "move_down"],
+        // vi-style aliases for Up/Down. Everything else (arrows,
+        // Page{Up,Down}, Home/End, selection motion, …) is inherited
+        // from the Normal keymap via `inheritNormalBindings: true`.
         ["k", "move_up"],
         ["j", "move_down"],
-        ["PageUp", "page_up"],
-        ["PageDown", "page_down"],
-        ["Home", "move_line_start"],
-        ["End", "move_line_end"],
         // Enter: focus the right-hand detail panel.
         ["Return", "review_branch_enter"],
         ["Tab", "review_branch_enter"],
@@ -4655,7 +4696,9 @@ editor.defineMode(
         ["q", "review_branch_close_or_back"],
         ["Escape", "review_branch_close_or_back"],
     ],
-    true,
+    true, // readOnly
+    false, // allowTextInput — keeps plain letters from inserting into the RO buffer
+    true, // inheritNormalBindings — PageUp/PageDown/arrows/Home/End come from Normal
 );
 
 // Register Modes and Commands