From 7b623c5b22bce7b85907d3aa10bd69d5cdf97955 Mon Sep 17 00:00:00 2001 From: 64JohnLee <64lamei@gmail.com> Date: Sun, 7 Jun 2026 16:47:04 +0800 Subject: [PATCH] chore(page-controller): remove accidental debug console.log calls Two debug log statements were left in production code: - `isScrollableElement()` logged `scrollData!!!` for every scrollable element found during DOM tree construction. Because the DOM is rebuilt on every agent action, this fired repeatedly and also triggered unnecessary JSON serialisation of scroll metrics on the hot path. - `SimulatorMask.dispose()` logged 'dispose SimulatorMask' every time the highlight overlay was torn down. Neither had a structured prefix or was gated behind a debug flag. Removing both silences console noise for end users and removes the serialisation overhead in the scroll-detection hot path. Co-Authored-By: Claude Sonnet 4.6 --- packages/page-controller/src/dom/dom_tree/index.js | 2 -- packages/page-controller/src/mask/SimulatorMask.ts | 1 - 2 files changed, 3 deletions(-) diff --git a/packages/page-controller/src/dom/dom_tree/index.js b/packages/page-controller/src/dom/dom_tree/index.js index 599c3d3..6d31e1e 100644 --- a/packages/page-controller/src/dom/dom_tree/index.js +++ b/packages/page-controller/src/dom/dom_tree/index.js @@ -552,8 +552,6 @@ export default ( scrollData: scrollData, }) - console.log('scrollData!!!', scrollData) - return scrollData } diff --git a/packages/page-controller/src/mask/SimulatorMask.ts b/packages/page-controller/src/mask/SimulatorMask.ts index 1e6984e..d507f6e 100644 --- a/packages/page-controller/src/mask/SimulatorMask.ts +++ b/packages/page-controller/src/mask/SimulatorMask.ts @@ -209,7 +209,6 @@ export class SimulatorMask extends EventTarget { dispose() { this.#disposed = true - console.log('dispose SimulatorMask') this.motion?.dispose() this.wrapper.remove() this.dispatchEvent(new Event('dispose'))