chore: update logic to be mac specific

Author: Noojuno

apps/web/src/components/Sidebar.logic.test.ts

@@ -2,7 +2,6 @@ import { describe, expect, it } from "vitest";
 
 import {
   hasUnseenCompletion,
-  isContextMenuPointerDown,
   resolveThreadRowClassName,
   resolveThreadStatusPill,
   shouldClearThreadSelectionOnMouseDown,
@@ -64,35 +63,6 @@ describe("shouldClearThreadSelectionOnMouseDown", () => {
   });
 });
 
-describe("isContextMenuPointerDown", () => {
-  it("treats secondary-button pointerdown as a context-menu gesture", () => {
-    expect(
-      isContextMenuPointerDown({
-        button: 2,
-        ctrlKey: false,
-      }),
-    ).toBe(true);
-  });
-
-  it("treats ctrl-primary-click as a context-menu gesture", () => {
-    expect(
-      isContextMenuPointerDown({
-        button: 0,
-        ctrlKey: true,
-      }),
-    ).toBe(true);
-  });
-
-  it("does not treat primary-button pointerdown as a context-menu gesture", () => {
-    expect(
-      isContextMenuPointerDown({
-        button: 0,
-        ctrlKey: false,
-      }),
-    ).toBe(false);
-  });
-});
-
 describe("resolveThreadStatusPill", () => {
   const baseThread = {
     interactionMode: "plan" as const,

apps/web/src/components/Sidebar.logic.ts

@@ -38,9 +38,20 @@ export function shouldClearThreadSelectionOnMouseDown(target: HTMLElement | null
   return !target.closest(THREAD_SELECTION_SAFE_SELECTOR);
 }
 
-export function isContextMenuPointerDown(input: { button: number; ctrlKey: boolean }): boolean {
+export function isMacOS(): boolean {
+  const nav = window.navigator as Navigator & {
+    userAgentData?: { platform: string };
+  };
+  return nav.userAgentData ? nav.userAgentData.platform === "macOS" : /Mac/i.test(nav.userAgent);
+}
+
+export function isContextMenuPointerDown(input: {
+  button: number;
+  ctrlKey: boolean;
+  isMac: boolean;
+}): boolean {
   if (input.button === 2) return true;
-  return input.button === 0 && input.ctrlKey;
+  return input.isMac && input.button === 0 && input.ctrlKey;
 }
 
 export function resolveThreadRowClassName(input: {

apps/web/src/components/Sidebar.tsx

@@ -93,6 +93,7 @@ import { formatWorktreePathForDisplay, getOrphanedWorktreePathForThread } from "
 import { isNonEmpty as isNonEmptyString } from "effect/String";
 import {
   isContextMenuPointerDown,
+  isMacOS,
   resolveThreadRowClassName,
   resolveThreadStatusPill,
   shouldClearThreadSelectionOnMouseDown,
@@ -927,6 +928,7 @@ export default function Sidebar() {
         isContextMenuPointerDown({
           button: event.button,
           ctrlKey: event.ctrlKey,
+          isMac: isMacOS(),
         })
       ) {
         // Keep context-menu gestures from arming the sortable drag sensor.