fix(web-console): prevent infinite state updates in metrics

[emrberk]

Reworked the state management of this component, ensuring updates all go through the updateBuffer call, no redundant local state, no unnecessary rerenders, no mutations on context values.

• Metrics component was getting the buffers object from Dexie, setting a local state, and was not properly syncing local state with the storage.
• In some cases, we were running effect to set the local state because context value changed -> running effect to set the context value -> .... This was causing infinite loops. -> removed redundant local state and effects
• We had a focus listener but it did not have any effect -> removed
• We had a visibility check but ref was being set only when telemetry is disabled -> reversed, it is set when telemetry is enabled

[Copilot]

Pull Request Overview

This PR fixes infinite state update loops in the Metrics component by reworking state management to eliminate redundant local state and ensure all updates go through the updateBuffer call.

• Removed local state variables and replaced them with direct access to buffer state
• Eliminated redundant effects that were causing infinite loops between local and context state
• Fixed visibility check implementation to properly set ref when telemetry is enabled

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
packages/web-console/src/scenes/Editor/index.tsx	Added key prop to Metrics component to force remount on buffer changes
packages/web-console/src/scenes/Editor/Metrics/index.tsx	Removed local state management, fixed infinite update loops, and streamlined state synchronization
packages/browser-tests/questdb	Updated subproject commit reference

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[javier]

The error is fixed now