[AUTOMATION] fix(clawpatch): ignore stale dashboard event fetch errors#284
[AUTOMATION] fix(clawpatch): ignore stale dashboard event fetch errors#284michiosw wants to merge 1 commit into
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Greptile SummaryThis PR prevents stale guard dashboard event fetch errors from updating the visible banner. The main changes are:
Confidence Score: 4/5This looks safe to merge, but one stale banner path should be cleaned up.
Important Files Changed
|

Where We Are
The guard dashboard can show an error from session A after the user has already switched to session B. The stale request is ignored on success, but the stale error still reaches the banner.
Where We Want To Go
Only the active session should be allowed to update the event error state. Switching sessions should not surface a false error from an older request.
How do we get there
Apply the same
selectedRef.current !== idguard to thefetchEvents(id)error path inweb/guard-dashboard/src/App.tsxthat already exists on the success path. Rebuild the embedded dashboard assets so the shipped UI matches the source change. Validation:go vet ./...,npm exec --yes --package pnpm@10.0.0 -- pnpm install --frozen-lockfile,npm exec --yes --package pnpm@10.0.0 -- pnpm --dir web/guard-dashboard typecheck,./scripts/build-guard-dashboard-assets.sh, andgit diff --checkpassed.go test ./...is blocked by unrelated flaky timing failures ininternal/guard/judge(TestStartLlamaServerHealthCheckAndStop,TestStartLlamaServerEarlyExitDoesNotWaitForStopTimeout).