Repository navigation
docs(readme): add Console start to Quick Start - #173
Conversation
Root README never mentioned the hosted Console. Additive only: a Console link in the top navigation and in the Packages/Website line, and a new first Quick Start subsection that explains starting from the Console, that setup opens the Console to approve the machine, and login/logout for headless or skipped connects. No existing text removed or reworded; no code or package change.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 229dd10185
ℹ️ 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".
| `login` prints a verification URL and a short code (`--no-open` for headless | ||
| use); `agentveil-mcp-proxy logout` disconnects this machine. See |
There was a problem hiding this comment.
--no-open
For a headless user following this recommendation, login --no-open does not initially print a usable URL or code. run_console_login_cli first invokes _acquire_console_credential_via_attach, which suppresses opening start.attach_url but still blocks in poll_for_token; pairing—and the documented verification URL/code—is attempted only after attach fails or expires. Since attach responses may specify up to a 3600-second lifetime, the advertised headless flow can appear hung for up to an hour. Either make --no-open skip attach and start device pairing immediately, or do not recommend it as the headless path.
Useful? React with 👍 / 👎.
Summary
setupopens the Console to approve the machine;login/logoutfor headless or skipped connects.Additive only: no existing README text removed or reworded. Docs only: no code or package change, no PyPI release.
Verification
git diff --check: cleantests/test_mcp_packaging.py(reads README.md): 21 passed