Omit unset diff query fields

- Skip undefined checkpoint diff inputs when building queries
- Tighten diff panel formatting and test typing

Author: BunsDev

apps/server/src/checkpointing/Layers/CheckpointDiffQuery.ts

@@ -133,8 +133,8 @@ const make = Effect.gen(function* () {
         fromCheckpointRef,
         toCheckpointRef,
         fallbackFromToHead: false,
-        relativePath: input.relativePath,
-        contextMode: input.contextMode,
+        ...(input.relativePath ? { relativePath: input.relativePath } : {}),
+        ...(input.contextMode ? { contextMode: input.contextMode } : {}),
       });
 
       const turnDiff: OrchestrationGetTurnDiffResultType = {

apps/web/src/components/DiffPanel.tsx

@@ -236,7 +236,9 @@ function DiffFileSection(props: {
     [contextMode, filePath, fullContextDiffQuery.data?.diff, resolvedTheme],
   );
   const resolvedFileDiff =
-    contextMode === "full" ? (resolveRenderableFileDiff(fullContextPatch, filePath) ?? fileDiff) : fileDiff;
+    contextMode === "full"
+      ? (resolveRenderableFileDiff(fullContextPatch, filePath) ?? fileDiff)
+      : fileDiff;
   const fullContextError =
     contextMode === "full" && fullContextDiffQuery.error
       ? fullContextDiffQuery.error instanceof Error
@@ -768,7 +770,9 @@ export default function DiffPanel({ mode = "inline" }: DiffPanelProps) {
                         threadId: activeThreadId,
                         fromTurnCount: activeCheckpointRange?.fromTurnCount ?? null,
                         toTurnCount: activeCheckpointRange?.toTurnCount ?? null,
-                        cacheScope: selectedTurn ? `turn:${selectedTurn.turnId}` : conversationCacheScope,
+                        cacheScope: selectedTurn
+                          ? `turn:${selectedTurn.turnId}`
+                          : conversationCacheScope,
                         enabled: isGitRepo,
                       }}
                       collapsed={fileReviewState.collapsed}

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

@@ -94,8 +94,8 @@ describe("expandDiffFile", () => {
 
   it("returns the same object when the file is already expanded", () => {
     const state = {
-      "src/a.ts": { accepted: false, collapsed: false, contextMode: "patch" },
-    };
+      "src/a.ts": { accepted: false, collapsed: false, contextMode: "patch" as const },
+    } satisfies Record<string, { accepted: boolean; collapsed: boolean; contextMode: "patch" }>;
     // File is already expanded, so the same object reference is returned.
     expect(expandDiffFile(state, "src/a.ts")).toBe(state);
   });