<@U0BME18CF7S> please review <https://github.com/O...
# automations-debug
c
@fresh-machine-37269 please review https://github.com/OpenHands/OpenHands/pull/18085 . feel free to approve it if you find no major issues, or request changes otherwise or comment as appropriate.
šŸ‘€ 1
āœ… 1
f
šŸ¤– On it! View progress here: oss-agent-canvas.ngrok.dev/conversations/…
āœ… Done! Reviewed PR #18085 and approved it. āœ… It's a docs-only map-maintenance update (the
verify-openhands
feature map) refreshing rows for four fixes merged earlier today, and the wording checks out against the actual merged code: • F19/F20 error toasts match #18025 (
displayApiErrorToast
, plus the 409 → "already installed, use Update or Uninstall" rewrite for Apps; plugins still keep the raw hint). • F07 hooks match #18031 —
staleTime
is 5 min for hooks and 10 min for skills, matching the new "first open / Refresh / cached reopen" description. • F24 credential hiding matches #18039's
displayUrlFor
(http(s) drops all userinfo, other schemes keep the user and drop only the password). • F23 repositories/plugins match #18041's
withPresetSources
and the export behavior. All referenced PRs/issues exist and are in the expected state, and the pinned screenshot URLs resolve. Two minor, non-blocking notes I left in the review: 1. The new baseline
main@ed815e141
is one commit behind current main (
7cd649604
, #18051) — fine for a delta pass, the next pass will pick it up. 2. 9 of the 10 screenshot embeds are wrapped in doubled backticks, so they render as code spans instead of images — worth fixing so the evidence is actually visible. Nothing major, so I approved rather than requesting changes.