Skip to content

feat: print review URLs for configured hosts - #33

Merged
flexdinesh merged 1 commit into
mainfrom
codex/review-browser-url
Oct 4, 2026
Merged

flexdinesh merged 1 commit into
mainfrom
codex/review-browser-url

Conversation

@flexdinesh

Copy link
Copy Markdown
Owner

servediff review now prints the submitted diff's /contexts/{id} URL using the active listener's host and port, with localhost and LAN URLs for wildcard listeners.

  • Print localhost by default, configured IPs otherwise, and all LAN IPv4 URLs for 0.0.0.0.
  • Open the diff on supported desktops; skip SSH, headless Linux, missing launchers, and --no-browser.
  • Add URL, browser environment, and CLI submission tests; document behavior.

Validation: mise run check and the required mise run check:push hook passed, including 51 browser tests and Go race tests. CLI smoke verified the exact browser URL, LAN output, and SSH/--no-browser suppression. Embedded assets unchanged.

@flexdinesh
flexdinesh marked this pull request as ready for review October 4, 2026 13:01
@flexdinesh
flexdinesh merged commit 54c5643 into main Oct 4, 2026
1 check passed
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.

1 participant