fix(desktop): open DiffPane comment composer on pointer-up

apps/desktop/src/renderer/routes/_authenticated/_dashboard/v2-workspace/$workspaceId/hooks/usePaneRegistry/components/DiffPane/DiffPane.tsx

@@ -88,7 +88,7 @@ export function DiffPane({
 
 	const {
 		composerAnnotationsByItemId,
-		onSelectedLinesChange,
+		onLineSelectionEnd,
 		onGutterUtilityClick,
 		clear: clearComposer,
 		submit: submitComposer,
@@ -127,8 +127,9 @@ export function DiffPane({
 			enableLineSelection: true,
 			enableGutterUtility: true,
 			onGutterUtilityClick,
+			onLineSelectionEnd,
 		}),
-		[options, onGutterUtilityClick],
+		[options, onGutterUtilityClick, onLineSelectionEnd],
 	);
 
 	const renderHeaderPrefix = useCallback(
@@ -250,7 +251,6 @@ export function DiffPane({
 			style={style}
 			items={items}
 			options={codeViewOptions}
-			onSelectedLinesChange={onSelectedLinesChange}
 			renderHeaderPrefix={renderHeaderPrefix}
 			renderHeaderMetadata={renderHeaderMetadata}
 			renderAnnotation={renderAnnotation}

apps/desktop/src/renderer/routes/_authenticated/_dashboard/v2-workspace/$workspaceId/hooks/usePaneRegistry/components/DiffPane/hooks/useDiffCommentComposer/useDiffCommentComposer.ts

@@ -1,5 +1,5 @@
 import type {
-	CodeViewLineSelection,
+	CodeViewOptions,
 	DiffLineAnnotation,
 	SelectedLineRange,
 } from "@pierre/diffs";
@@ -55,12 +55,16 @@ interface UseDiffCommentComposerArgs {
 	) => Promise<{ terminalId: string } | null>;
 }
 
+type OnLineSelectionEnd = NonNullable<
+	CodeViewOptions<DiffAnnotationMetadata>["onLineSelectionEnd"]
+>;
+
 interface UseDiffCommentComposerResult {
 	composerAnnotationsByItemId: ReadonlyMap<
 		string,
 		DiffLineAnnotation<DiffAnnotationMetadata>[]
 	> | null;
-	onSelectedLinesChange: (selection: CodeViewLineSelection | null) => void;
+	onLineSelectionEnd: OnLineSelectionEnd;
 	onGutterUtilityClick: () => void;
 	clear: () => void;
 	submit: (input: DiffCommentSubmitInput) => Promise<void>;
@@ -102,27 +106,24 @@ export function useDiffCommentComposer({
 		[clear],
 	);
 
-	const onSelectedLinesChange = useCallback(
-		(selection: CodeViewLineSelection | null) => {
-			if (!selection) {
+	const onLineSelectionEnd = useCallback<OnLineSelectionEnd>(
+		(range, context) => {
+			if (context.type !== "diff") return;
+			if (!range) {
 				setComposer(null);
 				return;
 			}
-			setComposer({ itemId: selection.id, range: selection.range });
+			setComposer({ itemId: context.item.id, range });
 		},
 		[],
 	);
 
 	// Pierre gates the gutter "+" button's pointer flow behind a non-null
 	// onGutterUtilityClick (InteractionManager.startGutterSelectionFromPointerDown
-	// early-returns otherwise). We mirror the open from the CodeView's
-	// current selection — pierre updates that during the pointer session.
-	const onGutterUtilityClick = useCallback(() => {
-		const selection = codeViewRef.current?.getSelectedLines();
-		if (selection) {
-			setComposer({ itemId: selection.id, range: selection.range });
-		}
-	}, [codeViewRef]);
+	// early-returns otherwise). The gutter pointer-up path also fires
+	// notifySelectionEnd → onLineSelectionEnd, so the open is handled there;
+	// this stays as a required stub.
+	const onGutterUtilityClick = useCallback(() => {}, []);
 
 	const composerAnnotationsByItemId = useMemo(() => {
 		if (!composer) return null;
@@ -204,7 +205,7 @@ export function useDiffCommentComposer({
 
 	return {
 		composerAnnotationsByItemId,
-		onSelectedLinesChange,
+		onLineSelectionEnd,
 		onGutterUtilityClick,
 		clear,
 		submit,