ALL-1254: Fix Monitors page loading action cell crash

completed

Priority: 0

Branch: wintermute/hotfix-monitors-loading-crash

PR: #11368

Hotfix for post-merge ALL-542 regression reported by Seph in Slack thread 1780033984.507679: clicking Monitors throws TypeError reading enabled from undefined. Root cause likely DataTable loading cell invokes CustomCell render with empty row; action cell assumes row.rawMonitor. Branch wintermute/all-542-monitors-loading-hotfix. Patch loading-safe action cell + regression test, open PR assigned to sephcoster.

Event Timeline

created

progress

PR #11368 opened and assigned for ALL-1254 hotfix. Root cause: edges DataTable loading cells invoke CustomCell render with empty row; actions column read row.rawMonitor.enabled. Fix guards action renderer and MonitorRowActions; regression test added. Local MonitorsPage tests 16/16 passed. GitHub CI pending.

progress

Addressing Cursor Bugbot on PR #11368: test previously duplicated guard logic instead of exercising MonitorsPage column render. Updating mocked DataTable to invoke the actual actions column render during loading, removing duplicate local test, and rerunning MonitorsPage tests.

progress

PR #11368 Cursor comment addressed and thread resolved. Local validation after fix: apps/dashboard biome lint on MonitorsPage files passed; MonitorsPage vitest 15/15 passed. Pushing formatting commit to rerun CI.

status_change

in_progress → completed