chore(code): bump diffs package version

Author: adboio

apps/code/package.json

@@ -101,7 +101,7 @@
     "@opentelemetry/semantic-conventions": "^1.39.0",
     "@parcel/watcher": "^2.5.6",
     "@phosphor-icons/react": "^2.1.10",
-    "@pierre/diffs": "^1.1.21",
+    "@pierre/diffs": "^1.2.10",
     "@posthog/agent": "workspace:*",
     "@posthog/api-client": "workspace:*",
     "@posthog/core": "workspace:*",

packages/core/package.json

@@ -18,7 +18,7 @@
   "dependencies": {
     "@modelcontextprotocol/ext-apps": "^1.1.2",
     "@modelcontextprotocol/sdk": "^1.12.1",
-    "@pierre/diffs": "^1.1.21",
+    "@pierre/diffs": "^1.2.10",
     "@posthog/api-client": "workspace:*",
     "@posthog/di": "workspace:*",
     "@posthog/platform": "workspace:*",

packages/core/src/code-review/diffAnnotations.test.ts

@@ -1,6 +1,7 @@
 import type { FileDiffMetadata } from "@pierre/diffs";
 import { describe, expect, it } from "vitest";
 import {
+  buildCommentMergedOptions,
   buildDraftAnnotations,
   buildHunkAnnotations,
   getLastChangeLineNumber,
@@ -80,3 +81,24 @@ describe("buildDraftAnnotations", () => {
     });
   });
 });
+
+describe("buildCommentMergedOptions", () => {
+  it.each([
+    { hasOpenComment: false, expectedEnabled: true },
+    { hasOpenComment: true, expectedEnabled: false },
+  ])(
+    "with hasOpenComment=$hasOpenComment sets selection/gutter enabled=$expectedEnabled and routes both callbacks to the handler",
+    ({ hasOpenComment, expectedEnabled }) => {
+      const handler = () => {};
+      const merged = buildCommentMergedOptions(
+        undefined,
+        hasOpenComment,
+        handler,
+      );
+      expect(merged.enableLineSelection).toBe(expectedEnabled);
+      expect(merged.enableGutterUtility).toBe(expectedEnabled);
+      expect(merged.onLineSelectionEnd).toBe(handler);
+      expect(merged.onGutterUtilityClick).toBe(handler);
+    },
+  );
+});

packages/core/src/code-review/diffAnnotations.ts

@@ -61,5 +61,6 @@ export function buildCommentMergedOptions(
     enableLineSelection: !hasOpenComment,
     enableGutterUtility: !hasOpenComment,
     onLineSelectionEnd: handleLineSelectionEnd,
+    onGutterUtilityClick: handleLineSelectionEnd,
   };
 }

packages/ui/package.json

@@ -52,7 +52,7 @@
     "@lezer/highlight": "^1.2.3",
     "@modelcontextprotocol/ext-apps": "^1.1.2",
     "@modelcontextprotocol/sdk": "^1.12.1",
-    "@pierre/diffs": "^1.1.21",
+    "@pierre/diffs": "^1.2.10",
     "@posthog/agent": "workspace:*",
     "@posthog/api-client": "workspace:*",
     "@posthog/core": "workspace:*",

packages/ui/src/features/code-review/constants.ts

@@ -9,5 +9,5 @@ export const DIFF_METRICS = {
   lineHeight: 20,
   diffHeaderHeight: 31,
   hunkSeparatorHeight: 32,
-  fileGap: 8,
+  spacing: 8,
 } as const;

pnpm-lock.yaml

@@ -51,6 +51,7 @@ minimumReleaseAgeExclude:
   - '@anthropic-ai/claude-agent-sdk-win32-x64'
   - '@anthropic-ai/sdk'
   - '@pierre/diffs'
+  - '@pierre/theming'
   - '@posthog/quill'
   - '@posthog/quill-tokens'
   - '@tanstack/devtools-core'