From b67faca60b21b0cf74d5c967f0f8d215ad601e93 Mon Sep 17 00:00:00 2001 From: amankumarsingh77 Date: Tue, 28 Jul 2026 20:30:49 +0530 Subject: refactor(domains): strip comment noise Removes module banners, step markers, and comments restating the code. Keeps one-line notes for safeJs's trust boundary, the select-match null sentinels, the TUI width clamp, the single-window tab rule, the input synthesis quirks (char events, keyCode, mousemove, focus, compositor), the stale AX value, ignored-wrapper recursion, and isolated-tab routing. Co-Authored-By: Claude Opus 5 (1M context) --- src/domains/scroll.ts | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) (limited to 'src/domains/scroll.ts') diff --git a/src/domains/scroll.ts b/src/domains/scroll.ts index 1819a48..832f584 100644 --- a/src/domains/scroll.ts +++ b/src/domains/scroll.ts @@ -38,13 +38,10 @@ export const scrollTool = defineBrowserTool({ const cy = args.y ?? Math.round(info.data.height / 2); const dx = args.deltaX ?? 0; const dy = args.deltaY ?? 300; - // The page target must be focused or Input events are dropped silently - // (mouseWheel hangs; synthesizeScrollGesture returns ok but no scroll). - // Page.bringToFront makes the target the active one in the browser. + // The page target must be focused or Input events are dropped silently: mouseWheel hangs and synthesizeScrollGesture returns ok without scrolling. const front = await cdpCall(client, "Page.bringToFront", {}); if (!front.success) return front; - // Establish compositor mouse position before mouseWheel; without this, - // CDP sometimes drops the wheel event with a timeout. + // Establish compositor mouse position first, or CDP sometimes drops the wheel event with a timeout. const moved = await cdpCall(client, "Input.dispatchMouseEvent", { type: "mouseMoved", x: cx, y: cy, button: "none", buttons: 0, }); -- cgit v1.2.3