Skip to content

fix(desktop): extend remote cwd boot timeout#61449

Open
konsisumer wants to merge 1 commit into
NousResearch:mainfrom
konsisumer:fix/desktop-remote-cwd-boot-timeout
Open

fix(desktop): extend remote cwd boot timeout#61449
konsisumer wants to merge 1 commit into
NousResearch:mainfrom
konsisumer:fix/desktop-remote-cwd-boot-timeout

Conversation

@konsisumer

Copy link
Copy Markdown
Contributor

What changed and why

Per the reporter's clarification on July 9, 2026, the remote desktop path can still fail after Remote Hermes backend is ready with a renderer-side hermes:api timeout on current main. The desktop boot burst had already been widened for config/model/profile calls, but the remote desktopDefaultCwd() probe still used the Electron bridge's default 15s timeout. This PR moves that remote /api/fs/default-cwd request onto STARTUP_REQUEST_TIMEOUT_MS, so the post-connect boot path does not fail early on the leftover short timeout.

How to test

  1. Run npm --workspace apps/desktop exec -- vitest run src/lib/desktop-fs.test.ts src/hermes.test.ts.
  2. Launch Hermes Desktop against a remote gateway that is reachable but slow enough to exercise the post-connect startup burst.
  3. Verify that after Remote Hermes backend is ready, the renderer can complete the remote default-CWD probe without tripping the old 15s hermes:api timeout.

What platforms tested on

  • Not fully verified in this worker environment: the targeted Vitest command timed out locally without producing output.

Fixes #38061

@alt-glitch alt-glitch added type/bug Something isn't working comp/desktop Electron desktop app (apps/desktop/*) P3 Low — cosmetic, nice to have labels Jul 9, 2026

@tonydwb tonydwb left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review Summary

Verdict: LGTM

What the PR Does

Extend remote cwd boot timeout to prevent startup failures on slow remote environments.

Assessment

  • Small test-only change. Clean fix, well-scoped.

Reviewed by Hermes Agent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/desktop Electron desktop app (apps/desktop/*) P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Can't connect to Remote Gateway via Tailscale for Hermes Desktop

3 participants