Skip to content

Show a remote host's state and next step on its reviews - #973

Draft
thesiti92 wants to merge 4 commits into
remote/10-gateway-transportfrom
remote/11-host-status
Draft

thesiti92 wants to merge 4 commits into
remote/10-gateway-transportfrom
remote/11-host-status

Conversation

@thesiti92

@thesiti92 thesiti92 commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Shows a remote host's state the same way everywhere, with one next step, instead of raw error text.

  • A dropped review stream shows a yellow chip in the top bar ("alex-box offline") instead of the "Connection lost. Reconnecting…" banner. Its click runs the host's next step: Retry, Install, or Open Settings. Retry also reopens the review stream at once instead of after its backoff.
  • reviewHostStatus gives each host state one short label, a sentence and that next step. The chip, Home, Settings and the gateway's messages use it.
  • A remote review that has not loaded says what its host is doing, with the next step, instead of a blank page.
  • On Home, an unavailable review says why, with the next step, and opens once its host is back.
  • Settings offers Retry to offline hosts, shows progress and errors for Retry and Install, and shows the host's detail without the state name in front. Open Settings from a review scrolls to Remote hosts.
  • The top bar's context group can shrink, so the chip and agent badges end in an ellipsis instead of pushing Dismiss off the bar.
  • Gateway errors no longer say "did not answer: it did not answer", and unknown network codes read "it failed (CODE)".

Stacked on #949.

Test plan

  • With a host added, pause it (remote.mjs pause a): the chip shows " offline" with Retry, and hovering shows the reason.
  • Reload the window while the host is paused: the review tab shows the host's state and Retry.
  • On Home, click the paused host's review: the message gives the reason and Retry. Resume the host: the review opens.
  • Settings: the paused host's row has Retry. Open Settings from a review's chip on an auth-failed host scrolls to Remote hosts.
  • remote-host journey passes.

@thesiti92
thesiti92 force-pushed the remote/10-diffr-npm-packages branch from 9814fe6 to aeb3f67 Compare October 7, 2026 13:31
@thesiti92
thesiti92 force-pushed the remote/11-host-status branch 3 times, most recently from f385694 to 6ce70b3 Compare October 8, 2026 12:59
@thesiti92
thesiti92 changed the base branch from remote/10-diffr-npm-packages to remote/10-gateway-transport October 8, 2026 12:59
@thesiti92 thesiti92 mentioned this pull request Oct 8, 2026
2 of 4 tasks
@thesiti92
thesiti92 force-pushed the remote/10-gateway-transport branch from 76c4cfc to 3a81dda Compare October 8, 2026 13:10
@thesiti92
thesiti92 force-pushed the remote/11-host-status branch 2 times, most recently from 47f0a9f to 2e9f03b Compare October 11, 2026 13:28
@thesiti92
thesiti92 force-pushed the remote/10-gateway-transport branch from 3a81dda to 78d4737 Compare October 11, 2026 13:28
The shared review stream waits up to 30 seconds between reconnect
attempts. reconnect() starts a new attempt now, with a fresh backoff.

Agent-Session: 13cd2035-0029-45cb-a2c9-65f0c01b6460
Agent-Session: e9b9e23e-de55-4291-9391-cd234f736397
Agent-Session: 5368a6f5-df0c-4411-8eec-4b9cccbfbef0
Agent-Session: f086ac47-f054-41fb-8279-be4df7aa2dee
Agent-Session: 3a3376c5-3f72-46df-930c-a66d5c3bcce5
Agent-Session: 7633c50b-998d-4b1e-8ca2-ffb350a26066
Agent-Session: 9a4d92ed-7a0d-42a1-9169-574bca60095b
Agent-Session: 1c093463-922b-435b-a925-664d5d5c3c0a
Agent-Session: 2b51b153-90b5-4a72-a052-fccf0beb9f1c
reviewHostStatus gives every host state one short status, one label, a
sentence and the action that helps: Retry, Install or Settings. The
gateway uses its sentence when a down host has no detail, so the window
no longer reads "alex-box is auth-failed.".

The gateway also stops saying "did not answer: it did not answer", and
an unknown network error reads "it failed (CODE)", not the bare code.

Agent-Session: 5368a6f5-df0c-4411-8eec-4b9cccbfbef0
Agent-Session: f086ac47-f054-41fb-8279-be4df7aa2dee
Agent-Session: 3a3376c5-3f72-46df-930c-a66d5c3bcce5
Agent-Session: 7633c50b-998d-4b1e-8ca2-ffb350a26066
Agent-Session: 9a4d92ed-7a0d-42a1-9169-574bca60095b
Agent-Session: 1c093463-922b-435b-a925-664d5d5c3c0a
Agent-Session: 2b51b153-90b5-4a72-a052-fccf0beb9f1c
A dropped review stream shows a yellow chip in the top bar, such as
"alex-box offline", instead of the "Connection lost" banner. A click
runs the step that helps: Retry, Install or Open Settings. Retry also
reopens the review stream at once.

A remote review that has not loaded shows the same, not a blank page.
On Home, an unavailable review says why, offers the step, and opens
once its host is back.

Agent-Session: 5368a6f5-df0c-4411-8eec-4b9cccbfbef0
Agent-Session: f086ac47-f054-41fb-8279-be4df7aa2dee
Agent-Session: 3a3376c5-3f72-46df-930c-a66d5c3bcce5
Agent-Session: 7633c50b-998d-4b1e-8ca2-ffb350a26066
Agent-Session: 9a4d92ed-7a0d-42a1-9169-574bca60095b
Agent-Session: 1c093463-922b-435b-a925-664d5d5c3c0a
Agent-Session: 2b51b153-90b5-4a72-a052-fccf0beb9f1c
An offline host now gets Retry, and Retry and Install show progress
and any error. A host's line shows its detail alone, without the state
name before it. Open Settings from a review scrolls to Remote hosts.

Agent-Session: 5368a6f5-df0c-4411-8eec-4b9cccbfbef0
Agent-Session: f086ac47-f054-41fb-8279-be4df7aa2dee
Agent-Session: 3a3376c5-3f72-46df-930c-a66d5c3bcce5
Agent-Session: 7633c50b-998d-4b1e-8ca2-ffb350a26066
Agent-Session: 9a4d92ed-7a0d-42a1-9169-574bca60095b
Agent-Session: 1c093463-922b-435b-a925-664d5d5c3c0a
Agent-Session: 2b51b153-90b5-4a72-a052-fccf0beb9f1c
@thesiti92
thesiti92 force-pushed the remote/10-gateway-transport branch from 78d4737 to a3f7279 Compare October 11, 2026 13:34
@thesiti92
thesiti92 force-pushed the remote/11-host-status branch from 2e9f03b to 2fe9306 Compare October 11, 2026 13:34

This branch has not been deployed

No deployments
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