Merge pull request #8 from OpenKnots/okcode/default-collapse-diffs

Default new diff files to collapsed

Author: BunsDev

apps/web/src/components/DiffPanel.tsx

@@ -679,7 +679,7 @@ export default function DiffPanel({ mode = "inline" }: DiffPanelProps) {
                   const themedFileKey = `${fileKey}:${resolvedTheme}`;
                   const fileReviewState = activeReviewState[filePath] ?? {
                     accepted: false,
-                    collapsed: false,
+                    collapsed: true,
                   };
                   return (
                     <DiffFileSection

apps/web/src/lib/diffFileReviewState.test.ts

@@ -18,9 +18,9 @@ describe("reconcileDiffFileReviewState", () => {
     });
   });
 
-  it("initializes new files as unaccepted and expanded", () => {
+  it("initializes new files as unaccepted and collapsed", () => {
     expect(reconcileDiffFileReviewState(["src/a.ts"], undefined)).toEqual({
-      "src/a.ts": { accepted: false, collapsed: false },
+      "src/a.ts": { accepted: false, collapsed: true },
     });
   });
 });
@@ -79,6 +79,7 @@ describe("expandDiffFile", () => {
     const state = {
       "src/a.ts": { accepted: false, collapsed: false },
     };
+    // File is already expanded, so the same object reference is returned.
     expect(expandDiffFile(state, "src/a.ts")).toBe(state);
   });
 });

apps/web/src/lib/diffFileReviewState.ts

@@ -6,7 +6,7 @@ export interface DiffFileReviewState {
 export type DiffFileReviewStateByPath = Record<string, DiffFileReviewState>;
 
 const DEFAULT_DIFF_FILE_REVIEW_STATE: DiffFileReviewState = {
-  collapsed: false,
+  collapsed: true,
   accepted: false,
 };