-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
[Feature]: Align on macOS remote Gateway UX before broader PRs #90665
Copy link
Copy link
Closed as not planned
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:needs-live-reproClawSweeper needs live local, crabbox, or manual validation to confirm this issue.ClawSweeper needs live local, crabbox, or manual validation to confirm this issue.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:needs-security-reviewClawSweeper marked this issue as needing security-sensitive review.ClawSweeper marked this issue as needing security-sensitive review.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.impact:securitySecurity boundary, credential, authz, sandbox, or sensitive-data risk.Security boundary, credential, authz, sandbox, or sensitive-data risk.impact:session-stateSession, memory, transcript, context, or agent state can drift or corrupt.Session, memory, transcript, context, or agent state can drift or corrupt.issue-rating: 🐚 platinum hermitGood issue quality with a plausible reproduction path needing some confirmation.Good issue quality with a plausible reproduction path needing some confirmation.staleMarked as stale due to inactivityMarked as stale due to inactivity
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:needs-live-reproClawSweeper needs live local, crabbox, or manual validation to confirm this issue.ClawSweeper needs live local, crabbox, or manual validation to confirm this issue.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:needs-security-reviewClawSweeper marked this issue as needing security-sensitive review.ClawSweeper marked this issue as needing security-sensitive review.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.impact:securitySecurity boundary, credential, authz, sandbox, or sensitive-data risk.Security boundary, credential, authz, sandbox, or sensitive-data risk.impact:session-stateSession, memory, transcript, context, or agent state can drift or corrupt.Session, memory, transcript, context, or agent state can drift or corrupt.issue-rating: 🐚 platinum hermitGood issue quality with a plausible reproduction path needing some confirmation.Good issue quality with a plausible reproduction path needing some confirmation.staleMarked as stale due to inactivityMarked as stale due to inactivity
Type
Fields
Priority
None yet
Summary
To avoid wasting anyone's time, I opened an issue before sending a broader PR for this area.
I want to check the intended direction for macOS Remote over SSH, Bonjour-assisted setup, remote diagnostics, and reset/restart behavior before splitting work into PRs.
Problem to solve
The macOS app already has the right split:
The rough part is that the UI makes those modes easy to confuse.
I'd prefer not to submit a quick feature PR that guesses the product direction. I would rather confirm the shape here first.
Proposed solution
My suggested split is:
Small bugfix PR: orphaned SSH tunnel cleanup
-N -Ltunnels for the same target, identity, and remote Gateway port during startup/reset/create.PortGuardianrecord for the local forwarded port.Remote setup PR
gateway.remote.sshTargetwhile keepinggateway.remote.urlas the local tunnel URL.Remote diagnostics PR
Remote reset/restart PR
Questions for maintainers
cliPath/sshPortever be auto-applied, or only suggested and confirmed through Test remote?Alternatives considered
Open one broad PR.
That would land the whole experience faster, but it mixes a bugfix, setup UX, diagnostics, and lifecycle behavior. I expect that would be harder to review.
Keep everything manual.
This is simpler to maintain, but it leaves remote setup brittle. Bonjour/gateway discovery already suggests that the app can help more than it currently does.
Treat Local and Remote as one combined mode.
I think this is the wrong direction. Local and Remote use different states and different authorities. They should stay separate.
Silently trust Bonjour TXT metadata.
Rejected. Bonjour/mDNS TXT records are unauthenticated. They should remain hints, not authority.
Impact
Affected users/systems/channels:
Severity:
Frequency:
Consequence:
Evidence/examples
Relevant docs:
docs/platforms/mac/remote.mddocs/platforms/macos.mddocs/platforms/mac/child-process.mddocs/platforms/mac/bundled-gateway.mddocs/gateway/remote.mddocs/gateway/discovery.mddocs/gateway/bonjour.mdObserved behavior from local testing:
58995,58996, etc. in the app's port ownership records.Additional information
Suggested proof expectations:
Maintainer decision requested: