Go to the next file when viewed

Author: kylecarbs

AGENTS.md

@@ -15,6 +15,9 @@ Use `bun` for everything - package management, tests.
 
 Tests should be minimal, not conflict with each other, and not be race-prone.
 
+If you make changes to a file that has tests, you should ensure the tests pass and add a test-case if one does not already exist covering it.
+We do not want duplicative tiny tests, but we want cases to be covered.
+
 Good:
 
 - `import { test } from "bun:test"`

src/browser/contexts/pr-review/index.test.ts

@@ -224,6 +224,16 @@ test("toggleViewed unmarks viewed file", () => {
   expect(store.getSnapshot().viewedFiles.has("src/index.ts")).toBe(false);
 });
 
+test("toggleViewed navigates to next file when marking current file as viewed", () => {
+  const store = createStore();
+  store.selectFile("src/index.ts");
+
+  store.toggleViewed("src/index.ts");
+
+  expect(store.getSnapshot().viewedFiles.has("src/index.ts")).toBe(true);
+  expect(store.getSnapshot().selectedFile).toBe("src/utils.ts");
+});
+
 test("toggleViewedMultiple marks multiple files", () => {
   const store = createStore();
 

src/browser/contexts/pr-review/index.tsx

@@ -524,13 +524,19 @@ export class PRReviewStore {
 
   toggleViewed = (filename: string) => {
     const next = new Set(this.state.viewedFiles);
-    if (next.has(filename)) {
+    const wasViewed = next.has(filename);
+    if (wasViewed) {
       next.delete(filename);
     } else {
       next.add(filename);
     }
     this.persistViewedFiles(next);
     this.set({ viewedFiles: next });
+
+    // When marking a file as viewed, navigate to the next file
+    if (!wasViewed && filename === this.state.selectedFile) {
+      this.navigateToFile("next");
+    }
   };
 
   toggleViewedMultiple = (filenames: string[]) => {