Skip to content

CLI: custom visualizer#11677

Merged
malhotra5 merged 6 commits intomainfrom
cli-visualizer
Nov 7, 2025
Merged

CLI: custom visualizer#11677
malhotra5 merged 6 commits intomainfrom
cli-visualizer

Conversation

@malhotra5
Copy link
Copy Markdown
Collaborator

@malhotra5 malhotra5 commented Nov 7, 2025

Summary of PR

This PR includes a custom visualizer just for the CLI. This is largely the same as before except that the system prompt event is no longer emitted. Going forward we'd update the visualizer class in order to make any major changes to the CLI UI.

Change Type

  • Bug fix
  • [] New feature
  • Breaking change
  • Refactor
  • Other (dependency update, docs, typo fixes, etc.)

Checklist

  • I have read and reviewed the code and I understand what the code is doing.
  • I have tested the code to the best of my ability and ensured it works as expected.

Fixes

Resolves #(issue)

Release Notes

  • Include this change in the Release Notes.

To run this PR locally, use the following command:

GUI with Docker:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.openhands.dev/openhands/runtime:7a4f893-nikolaik   --name openhands-app-7a4f893   docker.openhands.dev/openhands/openhands:7a4f893

CLI with uvx:

uvx --python 3.12 --from git+https://github.com/OpenHands/OpenHands@cli-visualizer#subdirectory=openhands-cli openhands

@malhotra5 malhotra5 marked this pull request as draft November 7, 2025 18:58
@malhotra5 malhotra5 marked this pull request as ready for review November 7, 2025 19:20
Copy link
Copy Markdown
Collaborator

@enyst enyst left a comment

Choose a reason for hiding this comment

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

Ohh, interesting, thank you!

For a week+ I keep intending to ask / check if the CLI is using the same visualizer, because it seems to matter if we consider it more like a simplified/debug tool or a feature important to client apps. Happy to see this, and it's good to know!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 7, 2025

Coverage report

This PR does not seem to contain any modification to coverable code.

@malhotra5 malhotra5 enabled auto-merge (squash) November 7, 2025 19:28
@malhotra5 malhotra5 merged commit e0d26c1 into main Nov 7, 2025
27 checks passed
@malhotra5 malhotra5 deleted the cli-visualizer branch November 7, 2025 19:45
enyst added a commit to enyst/playground that referenced this pull request Nov 12, 2025
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