Skip to content

feat: expose playground URL on startup - #84

Merged
Asaf-prog merged 1 commit into
extra-org:mainfrom
happykawayigt:codex/playground-route
Aug 1, 2026
Merged

feat: expose playground URL on startup#84
Asaf-prog merged 1 commit into
extra-org:mainfrom
happykawayigt:codex/playground-route

Conversation

@happykawayigt

Copy link
Copy Markdown

Summary

  • rename the manager playground route from /demo to /playground
  • print a clickable playground URL before Uvicorn starts while preserving wildcard bind addresses
  • update the Makefile output and add route and CLI regression coverage, including IPv6 URL formatting

Closes #81

Validation

  • pytest tests/agent_manager/test_web.py tests/agent_manager/test_cli.py -q (12 passed)
  • ruff check src tests
  • mypy src tests (169 source files)
  • full suite excluding five existing Windows path-separator assertions (564 passed, 5 deselected)
  • generated stubs for both checked-in examples are unchanged
  • git diff --check

The unfiltered Windows run has the same five existing failures in tests/generate/test_plugins_manifest.py; they compare POSIX expected paths with Windows separators.

AI assistance

Implemented and reviewed with OpenAI Codex. The changes and validation results were reviewed before submission.

@happykawayigt
happykawayigt marked this pull request as ready for review August 1, 2026 18:21
@Asaf-prog
Asaf-prog self-requested a review August 1, 2026 21:59
@Asaf-prog
Asaf-prog merged commit dfadc4a into extra-org:main Aug 1, 2026
1 check passed
@Asaf-prog

Copy link
Copy Markdown
Collaborator

Hi! Could you log in with your GitHub account? Once you’re signed in, you’ll appear in our contributors list and get credit for your contributions. Thanks!

Asaf-prog pushed a commit that referenced this pull request Aug 2, 2026
The playground page added in #84 had no UI coverage. Add a Playwright
spec for the route wiring (page + widget.js content types), a clean
widget mount with the configured greeting, and a full send/stream
round-trip against a mocked conversation API.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.

Rename /demo playground route to /playground; print the actual URL on startup

2 participants