feat(review): mark hunks as reviewed with per-repo persistence

AGENTS.md

@@ -88,6 +88,7 @@ CLI input
 - `auto` should choose split on wide terminals and stack on narrow ones.
 - Explicit `split` and `stack` choices override responsive `auto` layout selection.
 - `[` and `]` navigate hunks across the full review stream. Do not reintroduce `j`/`k` hunk navigation unless the user asks.
+- `v` toggles the selected hunk's reviewed state and, when marking, advances to the next unreviewed hunk. Reviewed hunks collapse to a one-line marker in the render-plan layer. `[`/`]` stop on collapsed markers (so the keyboard can expand them with Enter or un-mark a mistaken `v`), and identity is a content hash over the hunk body plus file path (`src/core/reviewedHunks.ts`). Markers persist per repo in `.hunk/cache/reviewed/` with a `reviewed_ttl_days` GC (`src/core/reviewedMarkerStore.ts`); non-repo inputs are session-only.
 - Agent context belongs beside the code, not hidden in a separate mode or workflow.
 - Agent notes are hunk-specific: show notes for the selected hunk, render them in the diff flow near the annotated row, and keep a clear spatial relationship to the code they explain.
 - Keep note behavior explicit. If the UI intentionally prioritizes one note, one selection, or one active target, encode that as a named policy rather than scattering array-index assumptions through the codebase.

CHANGELOG.md

@@ -6,6 +6,8 @@ All notable user-visible changes to Hunk are documented in this file.
 
 ### Added
 
+- Press `v` to mark the selected hunk as reviewed: it collapses to a one-line `✓ reviewed` marker and the selection advances to the next unreviewed hunk. The sidebar shows per-file review progress with a checkmark when a file is fully reviewed. `[` / `]` still stop on markers, so Enter (or a click) expands one in place and `v` un-marks it. Reviewed state persists per repository under `.hunk/cache/reviewed/` using content hashes, so an edited hunk automatically comes back as unreviewed; markers expire after `reviewed_ttl_days` (default 30, configurable).
+
 ### Changed
 
 ### Fixed

README.md

@@ -8,6 +8,7 @@ Hunk is a review-first terminal diff viewer for agent-authored changesets, built
 
 - multi-file review stream with sidebar navigation
 - inline AI and agent annotations beside the code
+- mark hunks as reviewed (`v`) with per-repo persistence and sidebar progress
 - split, stack, and responsive auto layouts
 - watch mode for auto-reloading file and Git-backed reviews
 - keyboard, mouse, pager, and Git difftool support
@@ -128,10 +129,12 @@ line_numbers = true
 wrap_lines = false
 agent_notes = false
 transparent_background = false
+reviewed_ttl_days = 30
 ```
 
 `exclude_untracked` affects Git/Sapling working-tree `hunk diff` sessions only.
 `transparent_background` can also be written as `transparentBackground`.
+`reviewed_ttl_days` controls how long reviewed-hunk markers (`v` in the app, stored per repo in `.hunk/cache/reviewed/`) are kept before they expire.
 
 Custom themes can inherit from any built-in base theme and override only the colors you care about:
 

flake.nix

@@ -4,6 +4,13 @@
     nixpkgs.url = "github:NixOS/nixpkgs/nixos-unstable";
     bun2nix.url = "github:nix-community/bun2nix";
     bun2nix.inputs.nixpkgs.follows = "nixpkgs";
+    # nixpkgs-unstable has dropped x86_64-darwin and now hard-errors when
+    # instantiated for it. bun2nix's default systems list (nix-systems/default)
+    # still includes x86_64-darwin, which breaks any downstream evaluation that
+    # touches bun2nix's per-system outputs. The vendored list in ./nix/systems
+    # is nix-systems/default minus x86_64-darwin.
+    systems.url = "path:./nix/systems";
+    bun2nix.inputs.systems.follows = "systems";
   };
   nixConfig = {
     extra-substituters = [

nix/systems/default.nix

@@ -0,0 +1,5 @@
+[
+  "aarch64-darwin"
+  "x86_64-linux"
+  "aarch64-linux"
+]

nix/systems/flake.nix

@@ -0,0 +1,3 @@
+{
+  outputs = _: {};
+}

src/core/config.test.ts

@@ -94,6 +94,48 @@ describe("config resolution", () => {
     });
   });
 
+  test("reads reviewed_ttl_days with command-section override and a default of 30", () => {
+    const home = createTempDir("hunk-config-home-");
+    const repo = createTempDir("hunk-config-repo-");
+    createRepo(repo);
+
+    mkdirSync(join(home, ".config", "hunk"), { recursive: true });
+    writeFileSync(
+      join(home, ".config", "hunk", "config.toml"),
+      ["reviewed_ttl_days = 7", "", "[patch]", "reviewed_ttl_days = 14"].join("\n"),
+    );
+
+    const resolved = resolveConfiguredCliInput(createPatchPagerInput(), {
+      cwd: repo,
+      env: { HOME: home },
+    });
+    expect(resolved.input.options.reviewedTtlDays).toBe(14);
+
+    const defaulted = resolveConfiguredCliInput(createPatchPagerInput(), {
+      cwd: repo,
+      env: { HOME: createTempDir("hunk-config-empty-home-") },
+    });
+    expect(defaulted.input.options.reviewedTtlDays).toBe(30);
+  });
+
+  test("rejects a non-positive or fractional reviewed_ttl_days", () => {
+    const home = createTempDir("hunk-config-home-");
+    const repo = createTempDir("hunk-config-repo-");
+    createRepo(repo);
+
+    mkdirSync(join(home, ".config", "hunk"), { recursive: true });
+    writeFileSync(join(home, ".config", "hunk", "config.toml"), "reviewed_ttl_days = 0");
+
+    expect(() =>
+      resolveConfiguredCliInput(createPatchPagerInput(), { cwd: repo, env: { HOME: home } }),
+    ).toThrow("Expected reviewed_ttl_days to be a positive integer.");
+
+    writeFileSync(join(home, ".config", "hunk", "config.toml"), "reviewed_ttl_days = 1.5");
+    expect(() =>
+      resolveConfiguredCliInput(createPatchPagerInput(), { cwd: repo, env: { HOME: home } }),
+    ).toThrow("Expected reviewed_ttl_days to be a positive integer.");
+  });
+
   test("merges custom theme overrides from global and repo config", () => {
     const home = createTempDir("hunk-config-home-");
     const repo = createTempDir("hunk-config-repo-");

src/core/config.ts

@@ -1,6 +1,7 @@
 import fs from "node:fs";
 import { join } from "node:path";
 import { resolveGlobalConfigPath } from "./paths";
+import { DEFAULT_REVIEWED_TTL_DAYS } from "./reviewedMarkerStore";
 import { detectVcs, findVcsRepoRootCandidate, isVcsId } from "./vcs";
 import type {
   CliInput,
@@ -116,6 +117,19 @@ function normalizeString(value: unknown) {
   return typeof value === "string" && value.length > 0 ? value : undefined;
 }
 
+/** Accept only positive integers and report the failing TOML key path. */
+function normalizePositiveInteger(value: unknown, keyPath: string) {
+  if (value === undefined) {
+    return undefined;
+  }
+
+  if (typeof value !== "number" || !Number.isInteger(value) || value <= 0) {
+    throw new Error(`Expected ${keyPath} to be a positive integer.`);
+  }
+
+  return value;
+}
+
 /** Accept only #rrggbb theme colors and report the failing TOML key path. */
 function normalizeThemeColor(value: unknown, keyPath: string) {
   if (value === undefined) {
@@ -242,6 +256,7 @@ function readConfigPreferences(source: Record<string, unknown>): CommonOptions {
       normalizeBoolean(source.transparentBackground) ??
       normalizeBoolean(source.transparent_background),
     colorMoved: normalizeBoolean(source.color_moved),
+    reviewedTtlDays: normalizePositiveInteger(source.reviewed_ttl_days, "reviewed_ttl_days"),
   };
 }
 
@@ -263,6 +278,7 @@ function mergeOptions(base: CommonOptions, overrides: CommonOptions): CommonOpti
     copyDecorations: overrides.copyDecorations ?? base.copyDecorations,
     transparentBackground: overrides.transparentBackground ?? base.transparentBackground,
     colorMoved: overrides.colorMoved ?? base.colorMoved,
+    reviewedTtlDays: overrides.reviewedTtlDays ?? base.reviewedTtlDays,
   };
 }
 
@@ -358,6 +374,7 @@ export function resolveConfiguredCliInput(
     copyDecorations: resolvedOptions.copyDecorations ?? DEFAULT_VIEW_PREFERENCES.copyDecorations,
     transparentBackground: resolvedOptions.transparentBackground ?? false,
     colorMoved: resolvedOptions.colorMoved,
+    reviewedTtlDays: resolvedOptions.reviewedTtlDays ?? DEFAULT_REVIEWED_TTL_DAYS,
   };
 
   if (resolvedOptions.theme === "custom" && !resolvedCustomTheme) {

src/core/loaders.test.ts

@@ -255,6 +255,33 @@ describe("loadAppBootstrap", () => {
     expect(bootstrap.changeset.files[0]?.agent?.annotations).toHaveLength(1);
   });
 
+  test("exposes the VCS repo root, and leaves it absent outside a repository", async () => {
+    const repo = createTempRepo("hunk-repo-root-");
+    const left = join(repo, "before.ts");
+    const right = join(repo, "after.ts");
+    writeFileSync(left, "export const value = 1;\n");
+    writeFileSync(right, "export const value = 2;\n");
+
+    const inRepo = await loadAppBootstrap(
+      { kind: "diff", left, right, options: { mode: "auto" } },
+      { cwd: repo },
+    );
+    expect(inRepo.repoRoot).toBe(repo);
+
+    const plain = mkdtempSync(join(tmpdir(), "hunk-no-repo-"));
+    tempDirs.push(plain);
+    const plainLeft = join(plain, "before.ts");
+    const plainRight = join(plain, "after.ts");
+    writeFileSync(plainLeft, "export const value = 1;\n");
+    writeFileSync(plainRight, "export const value = 2;\n");
+
+    const outside = await loadAppBootstrap(
+      { kind: "diff", left: plainLeft, right: plainRight, options: { mode: "auto" } },
+      { cwd: plain },
+    );
+    expect(outside.repoRoot).toBeUndefined();
+  });
+
   test("skips binary file-pair diffs instead of reading their contents", async () => {
     const dir = mkdtempSync(join(tmpdir(), "hunk-binary-diff-"));
     tempDirs.push(dir);

src/core/loaders.ts

@@ -23,7 +23,12 @@ import {
   normalizePatchText,
   stripTerminalControl,
 } from "./patch/normalize";
-import { createUnsupportedVcsOperationError, getVcsAdapter, operationFromInput } from "./vcs";
+import {
+  createUnsupportedVcsOperationError,
+  findVcsRepoRootCandidate,
+  getVcsAdapter,
+  operationFromInput,
+} from "./vcs";
 import type {
   AppBootstrap,
   AgentContext,
@@ -679,6 +684,7 @@ export async function loadAppBootstrap(
   return {
     input,
     changeset,
+    repoRoot: findVcsRepoRootCandidate(cwd),
     initialMode: input.options.mode ?? "auto",
     initialTheme: input.options.theme,
     customTheme,