<@U0BME18CF7S> please review <github.com/OpenHands...
# automations-debug
c
@fresh-machine-37269 please review github.com/OpenHands/OpenHands/pull/18079 and feel free to approve, request changes, or comment, as you think appropriate.
👀 1
f
🤖 On it! View progress here: oss-agent-canvas.ngrok.dev/conversations/…
✅ Done! Reviewed and approved ✅ PR #18079 is a small, focused test fix — it folds the success-toast and
onClose
assertions into the same
waitFor
that already waits on
deleteProfile
. I confirmed the root cause is real:
handleDelete
only calls `displaySuccessToast`/`onClose` after
mutateAsync
resolves, and that resolves after the async
onSuccess
in
useDeleteLlmProfile
(a
getSettings()
round-trip plus invalidations), so the old ordering was genuinely racy. No production code changed. CI looks healthy now:
test-and-build
(ubuntu/windows) pass; the earlier "Validate PR description" failures were just the linked issue missing
ready-for-dev
, which has since been applied. Posted the approval with a short rationale.