Skip to content

Restore --connect for remote sessions - #16

Closed
jeff-hykin wants to merge 8 commits into
mainfrom
jeff/fix/connect
Closed

Restore --connect for remote sessions#16
jeff-hykin wants to merge 8 commits into
mainfrom
jeff/fix/connect

Conversation

@jeff-hykin

Copy link
Copy Markdown
Member

restore --connect so we can connect to remote sessions

ruthwikdasyam and others added 7 commits March 14, 2026 01:01
…PC proxy

When --connect is provided (with optional URL), the viewer connects to an
existing gRPC proxy server instead of spawning its own. This mirrors the
stock rerun binary's --connect behavior. Defaults to rerun+http://127.0.0.1:<port>/proxy.

@github-actions github-actions Bot 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.

Hi! Thanks for opening this pull request.

Because this is your first time contributing to this repository, make sure you've read our Contributor Guide and Code of Conduct.

- Remove unused `parking_lot` dependency (cargo-shear)
- Add `[lints] workspace = true` to dimos/Cargo.toml (rerun lint)
- Fix TOML formatting in dimos/pyproject.toml (taplo)
- Rename error variables from `e` to `err` (rerun lint convention)
- Fix trailing whitespace in keyboard.rs
- Add blank lines before doc comments on struct fields/constants
- Use Display instead of Debug for error formatting in viewer.rs
- Commit updated Cargo.lock

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@spomichter

Copy link
Copy Markdown

THis PR only passes through --connect. Need ALL rerun args so we dont have to do this 10 more times in future

@spomichter spomichter closed this Mar 16, 2026
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.

3 participants