Skip to content

Unify hosted Sessions and Automations#95

Merged
burakgon merged 1 commit into
mainfrom
fix/unified-terminal-workspace
Jul 19, 2026
Merged

Unify hosted Sessions and Automations#95
burakgon merged 1 commit into
mainfrom
fix/unified-terminal-workspace

Conversation

@burakgon

Copy link
Copy Markdown
Owner

What changed

  • Remove the nested hosted terminal iframe and its duplicate control-plane chrome.
  • Hand authenticated Sessions and Automations routes to the full RoamCode workspace in the same tab.
  • Keep single-Node entry automatic and present an explicit full-page Node choice when several eligible Nodes exist.
  • Keep the terminal PWA's own navigation and Node switcher visible in hosted mode.
  • Prepare stable version 1.4.3 with user-facing release notes.

Why

The hosted control plane wrapped the terminal application inside a second navigation, header, Node picker, and framed surface. Sessions and Automations therefore looked and behaved like a separate app embedded inside RoamCode.

Impact

Desktop and mobile now use one product shell and one navigation plane. Existing terminal behavior, relay enrollment, session management, and Automations remain on the existing terminal implementation.

Validation

  • pnpm lint
  • pnpm typecheck
  • 751 web tests
  • 48 hosted-site tests
  • Hosted production build
  • Isolated Chrome product audit with 11 screenshots and no blocking layout, accessibility, routing, or iframe findings

@burakgon
burakgon marked this pull request as ready for review July 19, 2026 07:25
Copilot AI review requested due to automatic review settings July 19, 2026 07:25
@burakgon
burakgon merged commit 492a129 into main Jul 19, 2026
3 checks passed
@burakgon
burakgon deleted the fix/unified-terminal-workspace branch July 19, 2026 07:25

Copilot AI 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.

Pull request overview

This PR removes the hosted “workbench” iframe surface for Sessions and Automations and instead hands those routes off to the full RoamCode terminal workspace in the same tab, unifying navigation and product chrome across desktop and mobile. It also updates the hosted audit harness and bumps the repo to version 1.4.3 with release notes.

Changes:

  • Replace embedded /terminal/*?embed=1 iframe workbench with a product-handoff flow that navigates to the full /terminal/* workspace (auto-handoff for a single eligible Node; explicit chooser when multiple Nodes are eligible).
  • Remove the web client’s embedded-cloud mode handling so terminal navigation and the Node switcher are always visible in hosted/cloud mode.
  • Prepare release metadata for 1.4.3 (package versions + changelog entry) and update hosted product audit expectations.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
site/src/app-shell.ts Reworks Sessions/Automations hosted routing to hand off into /terminal/* without iframe embedding; adds auto-handoff + multi-node chooser UX.
site/src/app-shell.test.ts Updates hosted-shell tests to assert handoff navigation and adds coverage for the multi-node explicit choice flow.
site/src/app-shell.css Removes iframe workbench styling and adds styles for the new handoff and node-launcher surfaces.
site/scripts/hosted-product-audit.mjs Updates audit routes/fixtures/contracts to validate the unified terminal surface and assert no iframes remain.
packages/web/src/config.ts Removes the embedded-cloud flag now that embedding is no longer supported.
packages/web/src/App.tsx Always renders terminal navigation + host switcher in cloud mode (no embedded mode branching).
packages/web/package.json Bumps @roamcode.ai/web version to 1.4.3.
packages/server/package.json Bumps @roamcode.ai/server version to 1.4.3.
packages/cli/package.json Bumps roamcode CLI version to 1.4.3.
package.json Bumps workspace version to 1.4.3.
CHANGELOG.md Adds 1.4.3 release notes describing the hosted Sessions/Automations surface unification.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants