Add AgentsView as an optional sessions integration - #182
Conversation
Reviewer's GuideIntroduces Sequence diagram for launching AgentsView locallysequenceDiagram
participant User
participant Dotagents
participant PATH
participant AgentsView
User->>Dotagents: dotagents sessions [flags]
Dotagents->>PATH: exec.LookPath(agentsview)
PATH-->>Dotagents: executable path
Dotagents->>AgentsView: serve [--no-browser] [forwarded flags]
AgentsView-->>User: Local session dashboard
Sequence diagram for remote AgentsView SSH launchsequenceDiagram
participant User
participant Dotagents
participant AgentsView
User->>Dotagents: dotagents sessions --ssh-host user@host
Dotagents->>AgentsView: serve --no-browser [forwarded flags]
Dotagents-->>User: Loopback URL and ssh -L tunnel command
User->>AgentsView: Open tunneled dashboard
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Hey - I've reviewed your changes and they look great!
Sourcery assessment
Needs a human reviewer. This adds a launcher for an external service that indexes and exposes local session transcripts, telemetry, and usage data. If the integration is wrong, an index or other local state may remain after reverting, but it is bounded and can be cleared or rebuilt; no money movement, deletion, or irreversible authorization change is introduced.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 54c408f193
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| } | ||
|
|
||
| func announceSessionsRemote(port string, sshHost string) { | ||
| url := "http://127.0.0.1:" + port |
There was a problem hiding this comment.
Honor the forwarded bind host in tunnel hints
When a remote invocation forwards a specific non-loopback bind address, such as --host 192.0.2.10, AgentsView listens on that address but this URL is always built with 127.0.0.1; tunnelCommand consequently forwards to localhost:PORT, where nothing is listening. Since the CLI help and documentation advertise --host ADDR, either derive the tunnel target from that flag or force AgentsView to bind to loopback in remote mode.
AGENTS.md reference: AGENTS.md:L49-L51
Useful? React with 👍 / 👎.
What
dotagents sessionsas a thin launcher foragentsview serveVerification
go test ./...Summary by Sourcery
Add an optional sessions launcher that integrates AgentsView without making it a dotagents dependency.
New Features:
dotagents sessionsintegration for launching AgentsView session search, replay, telemetry, and usage views.Enhancements:
Documentation:
Tests: