Skip to content

fix(workspace): surface actionable remote worker diagnostics#1538

Open
pascalandr wants to merge 5 commits intodifferent-ai:devfrom
Pagecran:fix/1419-remote-worker-diagnostics
Open

fix(workspace): surface actionable remote worker diagnostics#1538
pascalandr wants to merge 5 commits intodifferent-ai:devfrom
Pagecran:fix/1419-remote-worker-diagnostics

Conversation

@pascalandr
Copy link
Copy Markdown
Contributor

Summary

  • Fixes [Bug]: Remote worker Test connection and Recover fail without actionable diagnostics #1419 by restoring actionable remote worker diagnostics in the React app.
  • Ports the old remote connection test/recover behavior into the current shell via apps/app/src/react-app/domains/workspace/remote-connection.ts.
  • Wires connection state and test/recover handlers into session-route.tsx and settings-route.tsx, and shows the real error message in the workspace sidebar instead of a generic status.

Testing

  • git diff --check upstream/dev...fix/1419-remote-worker-diagnostics OK
  • pnpm --filter @openwork/app typecheck could not provide a clean signal in this environment due local TypeScript install issues (react-router-dom, lucide-react, zustand, @tanstack/react-virtual missing) plus an existing streamdown / React types mismatch

Notes

  • This ended up being a larger React re-port than the original one-file Solid patch because the old workspace connection flow was no longer wired into the shell.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Apr 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-landing Ready Ready Preview, Comment, Open in v0 Apr 30, 2026 6:30am

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Apr 23, 2026

@pascalandr is attempting to deploy a commit to the Different AI Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

Copy link
Copy Markdown
Collaborator

@src-opn src-opn left a comment

Choose a reason for hiding this comment

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

it looks good

@src-opn
Copy link
Copy Markdown
Collaborator

src-opn commented Apr 29, 2026

@pascalandr please fix conflicts

…er-diagnostics

# Conflicts:
#	apps/app/src/react-app/shell/session-route.tsx
#	apps/app/src/react-app/shell/settings-route.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Remote worker Test connection and Recover fail without actionable diagnostics

2 participants