Skip to content

FU-P12-T2-1: Fix stacking click event listeners in updateLatencyTable#58

Merged
SoundBlaster merged 6 commits into
mainfrom
feature/FU-P12-T2-1-fix-stacking-click-listeners
Feb 16, 2026
Merged

FU-P12-T2-1: Fix stacking click event listeners in updateLatencyTable#58
SoundBlaster merged 6 commits into
mainfrom
feature/FU-P12-T2-1-fix-stacking-click-listeners

Conversation

@SoundBlaster

Copy link
Copy Markdown
Owner

Description

Fixes a listener-stacking bug in the Web UI dashboard: updateLatencyTable was called on every polling cycle (~2 s) and each call added a new click listener to the same <tbody> element. After N refreshes, a single button click triggered N fetches to /api/analytics/param-patterns. The delegated click handler is now registered once in setupEventHandlers (attached to el("latency-table")) and removed from updateLatencyTable.

Changed file: src/mcpbridge_wrapper/webui/static/dashboard.js

Type of Change

  • Bug fix

Quality Gates

  • make test - All tests pass with ≥90% coverage (465 passed, 95.6% coverage)
  • make lint - No linting errors (ruff check src/ — all checks passed)
  • make format - Code is properly formatted
  • make typecheck - Type checking passes
  • make doccheck - Documentation is synced with DocC (if docs changed)

Documentation Sync

  • Documentation changes are synced with DocC catalog (or N/A) — no docs changed

Testing

  • Added/updated tests for new functionality — no JS test suite; Python tests unaffected
  • All tests pass locally
  • Manually tested the changes

Checklist

  • Code follows the project's style guidelines
  • Self-review completed
  • Comments added for complex code
  • Documentation updated (if needed)
  • No new warnings generated
  • PR title is descriptive

@SoundBlaster SoundBlaster merged commit ae1802f into main Feb 16, 2026
9 checks passed
@SoundBlaster SoundBlaster deleted the feature/FU-P12-T2-1-fix-stacking-click-listeners branch February 16, 2026 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant