From 24d515eeca389c04d1b12dcd7c4583ccc3216fb6 Mon Sep 17 00:00:00 2001 From: Brad Harris Date: Sun, 12 Jul 2026 03:04:21 -0600 Subject: [PATCH] fix(web): remove unnecessary double type cast in unified-diff-view MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The EventMap onClick handler from react-diff-view already provides React.MouseEvent which includes shiftKey — the `as unknown as MouseEvent` double cast was unnecessary tech debt. Co-Authored-By: Claude Opus 4.6 --- apps/web/src/components/app/unified-diff-view.tsx | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/apps/web/src/components/app/unified-diff-view.tsx b/apps/web/src/components/app/unified-diff-view.tsx index 1b4df4ad..5dc771cd 100644 --- a/apps/web/src/components/app/unified-diff-view.tsx +++ b/apps/web/src/components/app/unified-diff-view.tsx @@ -264,9 +264,7 @@ export const UnifiedDiffView = memo(function UnifiedDiffView({ const ln = getNewLineNumber(change); if (ln === null) return; - const mouseEvent = e as unknown as MouseEvent; - - if (mouseEvent.shiftKey && lineSelection) { + if (e.shiftKey && lineSelection) { const start = Math.min(lineSelection.anchorLine, ln); const end = Math.max(lineSelection.anchorLine, ln); onLineSelection({