docs: add Tailscale and ngrok setup for phone/tablet access#556
docs: add Tailscale and ngrok setup for phone/tablet access#556smolpaws wants to merge 1 commit into
Conversation
Adds a new 'Phone & Tablet Access' page under the Agent Canvas tab with two options for accessing a local Agent Canvas from mobile devices: - Tailscale (recommended for personal use): private encrypted network, 5-step setup, includes the localStorage gotcha where stale backend config points at 127.0.0.1 on the phone. - ngrok (for remote/public access): secure tunnel with public URL, links to --public mode docs for proper authentication. Co-authored-by: smolpaws <engel@enyst.org>
7787330 to
132d0b7
Compare
|
✅ Review complete. This review was performed through OpenHands Cloud Automation. You can log in and view the conversation here. |
all-hands-bot
left a comment
There was a problem hiding this comment.
Code Review: docs: add Tailscale and ngrok setup for phone/tablet access
Overview
Good addition — the two-option approach (Tailscale vs ngrok) covers the main use cases well, and the security warning for ngrok is appropriately prominent.
Taste Rating: 🟡 Acceptable
Documentation is clear and well-structured, but a few minor improvements are worth considering.
[IMPROVEMENT OPPORTUNITIES]
-
[openhands/usage/agent-canvas/mobile-access.mdx, Line 31] Incomplete command: The ngrok command shows
ngrok http 8000, but Agent Canvas UI typically runs on port 8000 while the backend API uses port 3000. Consider specifying the full URL format or clarifying which component the user should expose. -
[openhands/usage/agent-canvas/mobile-access.mdx, Line 23-26] Tip could be more actionable: The stale browser data tip mentions clearing localStorage but doesn't provide the specific steps. Consider adding a brief "How to clear localStorage" note or linking to browser docs.
[STYLE NOTES]
- Overall structure and tone match the existing documentation style well.
- MDX component usage (
<Tip>,<Warning>) is consistent with repo conventions. - No unnecessary comments or noise.
[RISK ASSESSMENT]
- [Overall PR]
⚠️ Risk Assessment: 🟢 LOW
Documentation-only PR. No code changes, no security vulnerabilities introduced. The ngrok security warning appropriately alerts users to enable authentication.
VERDICT:
✅ Worth merging: Documentation is solid and addresses a real user need. The suggested improvements are minor and optional.
KEY INSIGHT:
This fills a genuine gap in the docs — mobile access is a common user request that wasn't previously documented. The Tailscale recommendation is sensible (zero-config, encrypted, no port forwarding) and the ngrok option provides flexibility for users who need public access.
This review was generated by an AI agent (OpenHands) on behalf of the user through OpenHands Automation. View conversation
What
Adds a new Phone & Tablet Access page under the Agent Canvas tab with two options for accessing a local Agent Canvas from mobile devices.
Why
Common user request (came up in #proj-agent-canvas on Slack). Users want to interact with their local Agent Canvas from a phone or iPad without complex networking setup.
What changes
openhands/usage/agent-canvas/mobile-access.mdxdocs.json(between Customize and ACP Agents)Tailscale (recommended for personal use)
127.0.0.1which is localhost on the phone)ngrok (for remote access)
--publicmode andLOCAL_BACKEND_API_KEYdocs for proper authenticationContext
We set this up today (Tailscale + Agent Canvas on iPad) and hit the localStorage gotcha ourselves, so the instructions are battle-tested.